[1.18.2] Bug synchronized method with world (geckolib 3.0.57 latest 1.18.2 forge version)

Hello,
I have a bug with my Blockling Entity Class. I check in my predicate() method if it's the night in the world to do a specific animation with a shouldSleep() method:
    // in BlocklingEntity class
    public boolean shouldSleep() {
        return !this.getCommandSenderWorld().isDay();
    }

This method return always false in my predicate() method even though it's the night.
    // in BlocklingEntity class
    public boolean shouldSleep() {
        return !this.getCommandSenderWorld().isDay();
    }

    private  <E extends IAnimatable> PlayState predicate(AnimationEvent<E> event) {

        if (event.isMoving() && this.shouldSleep()) {
            event.getController().setAnimation(new AnimationBuilder().addAnimation("animation.blockling.walk2", ILoopType.EDefaultLoopTypes.LOOP));
        } else {
            event.getController().setAnimation(new AnimationBuilder()
                    .addAnimation("animation.blockling.idle3", ILoopType.EDefaultLoopTypes.PLAY_ONCE)
                    .addAnimation("animation.blockling.idle4", ILoopType.EDefaultLoopTypes.LOOP)
            );

        }

        return PlayState.CONTINUE;
    }

(So the animations in the else{...}  are all time set)
My class is extends to Animal and implements IAnimatable, IAnimationTickable. I noticed that only the methods coming from the Entity super class like getOnPos() or interactAt() are synchronized and return the values synchronized with the game (or the server but I don't know precisely):
     // in BlocklingEntity class
    private boolean shouldSleep = false;

    @Override
    public @NotNull InteractionResult interactAt(@NotNull Player p_19980_, @NotNull Vec3 p_19981_, @NotNull InteractionHand hand) {
        if (hand == InteractionHand.MAIN_HAND) {
            this.shouldSleep= !this.shouldSleep; // Work !
        }

        return super.interactAt(p_19980_, p_19981_, hand);
    }

    private  <E extends IAnimatable> PlayState predicate(AnimationEvent<E> event) {

        if (event.isMoving() && this.shouldSleep) { // Work !
            event.getController().setAnimation(new AnimationBuilder().addAnimation("animation.blockling.walk2", ILoopType.EDefaultLoopTypes.LOOP));
        } else {
            event.getController().setAnimation(new AnimationBuilder()
                    .addAnimation("animation.blockling.idle3", ILoopType.EDefaultLoopTypes.PLAY_ONCE)
                    .addAnimation("animation.blockling.idle4", ILoopType.EDefaultLoopTypes.LOOP)
            );

        }

        return PlayState.CONTINUE;
    }

I know your are not supporting this version but, consider this ticket to resolve the problem for all. Thank you !

I set the night in the world. you can see in the console first the position and in next the value return by the method shouldSleep

The position value are updated. The value return by the method is always false

But in the baseTick method, the method return true:

I up some boolean but nothing work. How to fix it ?

yes bro,today i meet the same problem ，so could you please tell me how do you work it out？

Hello ! I come back to you with the repositery (if I'll find it). I believe I fixed it in the past.


Theme customization

Enhancement Summary
In addition to including some additional TUI default themes, it would be nice to have the ability to define theme colors in a yaml/json file that is read by the CLI
Additional context
No response

The above PR added a couple more theme options via tuikit but support is still needed for yaml/json themes


[region-isolation] Finish adding support for the rest of the SIL instructions

Now the model is completely covered and we do not have any more unhandled instructions. I am still going to have to do some tweaks, but this at least will force anyone who adds new instructions to update this code or get a linker error.
rdar://115367810

@swift-ci test


Addon fluxcd could not be disabled or upgrade if one of the cluster is deleted

Describe the bug

Addon fluxcd could not be disabled or upgrade if one of the cluster is deleted. In this example, fluxcd is installed in cluster staging and the cluster is deleted before disabling fluxcd addon.
  - lastTransitionTime: "2021-12-22T17:18:16Z"
    reason: Available
    status: "True"
    type: Policy
  - lastTransitionTime: "2021-12-22T17:18:16Z"
    reason: Available
    status: "True"
    type: Render
  - lastTransitionTime: "2021-12-22T17:18:44Z"
    reason: Available
    status: "True"
    type: Workflow
  - lastTransitionTime: "2021-12-22T17:19:32Z"
    reason: ReconcileSuccess
    status: "True"
    type: Ready
  - lastTransitionTime: "2021-12-23T08:07:05Z"
    message: 'failed to sweep resourcetrackers to be deleted: failed to get resource
      v1/Namespace/staging//flux-system: no such cluster staging'
    reason: ReconcileError
    status: "False"
    type: Synced

To Reproduce

Expected behavior

Screenshots

KubeVela Version

Cluster information

Additional context

Yes we shouldn't block delete operation when cluster not exist


Allow regular producers with a dedup enabled namespace

Fixes #1225
Motivation
Fix  #1225
Modifications
When MessagePublishContext.isMarkerMessage is true, the pulsar deduplication check will always pass, so we can produce a regular message in KoP when dedup is enabled.
But the isMarkerMessage is not designed for this case, it uses to check the replicator marker. We might need more disscuss.
Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

Can you check the pulsar 2.8.4 has this check?

I will cherry pick 80f47d14774b85a23af050b09bfb8cb896163a48 to branch-2.8
but users will have to wait until a release, and given that 2.8.3 was released some days ago I don't know when a release will happen.

@Demogorgon314 I have verified that https://github.com/apache/pulsar/pull/14017 is on all active branches.
+1 to this patch


fix: overlapped price on market screen

❓ Context

Impacted projects: ledger-live-mobile

Linked resource(s): LIVE-1396

On the market screen of ledger-live-mobile, when the currency name is too long the price is overlapped.
✅ Checklist

[ ] Test coverage: Did you write any tests to cover the changes introduced by this pull request?
[X] Atomic delivery: Is this pull request standalone? In order words, does it depend on nothing else?
[ ] Breaking changes: Does this pull request contain breaking changes of any kind? If so, please explain why.

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Hey @pierrelouis-c, could you please add a changelog to your PR? (pnpm changelog)
Thanks 🙇


test(e2e): wait for deployment to exist in csv replacement test

Fixes a flake I noticed on https://github.com/operator-framework/operator-lifecycle-manager/pull/830
Might not be the only one, so I'll append more fixes if needed

/test unit

/retest

/test unit

/retest
deleting namespaces after a test seems to be hanging :/

/retest

/lgtm

/retest

/retest

/retest


[WIP] [Bug] Fix Onboarding dismiss issue (closes #122)

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] If this PR comes from a fork, please Allow edits from maintainers
[x] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes # 41)
[ ] Link your Pull Request to an issue (if applicable)
[x] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[x] Make sure that your PR is not introducing unnecessary reformatting (e.g., introduced by on-save hooks in your IDE)

Description

In a previous commit, many NotificationCenter notifications were removed. SceneDelegate relies on the .isOnboardedDidChange notification to dismiss the onboarding UI & show the main app UI. This broke the dismissing flow:

A quick fix would be to add the notification back into SecureStore. This is implemented here.

It's duplicated #128


[Bug]: NSFW selector 'X' doesn't work properly for images

What happened?
I'm assuming 'X' is higher than 'Mature' in which case, it doesn't appear to work properly. If not, disregard this ticket but document how it's supposed to work. The API documentation looks like there are supposed to be 4 increasing levels of maturity.
This model https://civitai.com/api/v1/images?modelId=98925 contains 9 images rated 'None', 0 rated 'Soft', 6 rated 'Mature' and 0 rated 'X'.
Specifying None, Soft and Mature work as expected returning only those images. Specifying 'X' returns 'Mature'
Thundercats! Ho!
Steps to reproduce the problem

Go to .... https://civitai.com/api/v1/images?modelId=98925&nsfw=X

What should have happened?
You should get back an empty list of images as there aren't any graded 'X' for that model.
What platforms do you use to access the site?
Linux
What browsers do you use to access the site?
Mozilla Firefox
Additional information, context and logs
This ticket assumes the maturity levels are graded in increasing maturity order from None to X. If that's not the case, the API documentation is insufficient to explain how the nsfw selector is supposed to work.

Hey @worthlesscog the way that filter works is that it will fetch images up to the nsfw level that you specify. For example, if you specify Mature, it will retrieve images marked as Soft and Mature, if you send Soft it'll bring just images marked as Soft and finally X will retrieve images with Soft, Mature and X
Hope this helps!

Then it's broken in a different way - specifying "Mature" only returns Mature not Mature and lower, which isn't how you've indicated it's supposed to work. With the model I linked None returns the 9 marked as None, Soft returns an empty list and Mature returns only the 6 marked Mature.
The only way to get the images marked None is to not specify at all or specify None.


python3Packages.pytest-subprocess: 1.1.0 -> 1.1.1

Motivation for this change
https://github.com/aklajnert/pytest-subprocess/blob/1.1.1/HISTORY.rst
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127377 at 5f5380a3 run on aarch64-linux 1

2 packages built successfully:

python38Packages.pytest-subprocess
python39Packages.pytest-subprocess


Magic bean recipe is broken (1.1.0)

Recipe seems to appear in tools like JEI, but looks like its giving stack size zero, and a weird metadata value. crafting doesnt work.

Fixed in 1.2.0


Prepare azure-search-documents for July 2022 Beta Release

API change check
API changes are not detected in this pull request.


Compute the number of dimensions N in build_solution

My code got broken with (probably) a recent version of SciML.

Hm indeed SciML does (for me) a bit too many breaking changes in non-breaking releases. Should we maybe also bump the compat entry then? (raise the version on the [compat]) If that is possible that is (i.e. we have that entry).

Hm, I will try to fix this soon. Apart from SciML sometimes doing breaking changes in non-breaking releases, ManifoldDiffEq.jl hooks into some of their internals so they can justifiably feel free to change them as they see fit.

Hm indeed SciML does (for me) a bit too many breaking changes in non-breaking releases.

This dimension was added in 2016. What's the right time interval for such changes?

Hm indeed SciML does (for me) a bit too many breaking changes in non-breaking releases.

This dimension was added in 2016. What's the right time interval for such changes?

I am not sure, I did not write this code, for now my main experience and interaction with SciML was sudden failures on CI – maybe because we used internals, I am not sure; so this looked like the same area, and I am not sure how the got to the parameter being 1, maybe because we adapted some example code?
I personally did not get much into that, since I was usually missing documentation every now and then.

Apart from SciML sometimes doing breaking changes in non-breaking release

What breaking change? Please help me understand what you are asking for here and I can help out. But this dimension parameter has not changed in 8 years so I do not understand what's being asked for.

No the parameter did not change, but how you handle it and how it is documented. I did not work with SciML for the last year or so, but when I did, e.g. implementing this function
https://github.com/JuliaManifolds/ManifoldDiffEq.jl/blob/4f283a0361488d57c36ba4498237934f3dde7cd8/src/frozen_solvers.jl#L30-L46
I have no clue (today) what any of the parameter is, I just copied that from an example. back then I could maybe figure out 20% of what the parameters mean because none of them are documented anywhere (and I am not a differential equations guy much). So when we do such guesswork, we maybe also guessed, the parameter 1 is the right thing to do from an example (where that was the case). Now in 2.40, you surely did not change the parameter itself, but how you interpret and work with it. Before you maybe were more relaxed about what the value was?
Now you are not, and since we guessed what it might mean, we were wrong but never noticed, and then...
what this ends up doing is that the commit linked above breaks other peoples code but is marked as nonbreaking. Usually this hits us often (I remember up to 5 times for now?) which I feel is annoying.
But maybe the main issue is documentation (and not the breaking part which one could also consider bug fixing).

What breaking change? Please help me understand what you are asking for here and I can help out.

Thank you for offering help but everything except this issue has already been resolved. SciML folks help resolve any issues quite quickly but still, our CI failures caused by upstream changes or bugs mostly originate in SciML. At the same time SciML most likely has more development than other dependencies, so per commit you may even be more reliable than other libraries but it's hard to tell.

Thanks for this very detailed answer. Concerning

I would highly recommend sticking to public APIs if you want stability.

I tried my best to do that ;)
For the ndims argument. The problem is that our tangent vectors might be more general than (just) an army, see for example https://github.com/JuliaManifolds/Manifolds.jl/blob/676b0f5d0751f4899bec67a0a81b7f313e1f6db7/src/manifolds/FixedRankMatrices.jl#L88-L106 (or any other <: TVector type). They have to still follow “Vectorian ideas” (addition, scaling etc.(

One additional comment on the tutorial you mentioned

https://docs.sciml.ai/DiffEqDevDocs/stable/contributing/adding_algorithms/

There I have exactly the problem with for example
function alg_cache(alg::RK_ALG,u,rate_prototype,uEltypeNoUnits,uBottomEltypeNoUnits,tTypeNoUnits,uprev,uprev2,f,t,dt,reltol,p,calck,::Val{true})
[...]

I am missing a bit what all the parameters of the function are. I think for the above-mentioned Manifold-Explicit-Eurer I (or Metusz) took exactly that tutorial and tried to make sense of the parameters and what they mean for the algorithm to use the DiffEq framework, but I probably understand far too less (still) about the rest of the framework.
Yes the public API is basically all algorithms, but one idea here, is to implement new algorithms (on Manifolds that is). So for that it seems not all we have to use (or we think we have to use) seems to be public then. How can we then best avoid this package to fail regularly? Because that happened a few times already.

I would highly recommend sticking to public APIs if you want stability. There are no guarantees to the outside that internal functions will not change any arguments or behavior. If that was the case then every release would have to be a breaking release! For information on Julia's public interfaces, I would recommend reading:

IIRC I needed to overload some internals (specifically, build_solution) to get the interpolation method I wanted. Or, at least the one close to what I wanted because it should be on-manifold Hermite interpolation but it is fairly difficult to implement.

Is there a more general interface function that should be considered here? I don't know of one in ArrayInterface.jl but I'd be interested to hear what the correct generalization is if it helps. I would think that the tensor dimensionality of a manifold object would be well-defined as that's more a definition of what kind of operator/vector an object is. The tensor dimensionality "are you a vector or matrix" is not the dimensionality of the manifold, i.e. the length of a state representation of an object in the vector space. And this would allow for example different charts/representations, because it's not the length of the data, and this would then use the ragged AbstractVectorOfArray structure as above.

The thing is, there are (to my knowledge) no generic and useful operations for arbitrary manifolds that generalize ndims. We either work on a specific representation for a specific manifold, and then ndims is well-defined and used as necessary. Or, we linearize representation using get_coordinates or get_parameters and we get plain vectors of numbers that have ndims equal to 1. We just didn't see the need to linearize to something other than a vector but it could be added. The thing about manifold linearizations is that you have to be explicit about how you linearize and that is by design.
We have charts and representations figured out here and it works perfectly fine without ndims. "Give me the number of indices" requires an assumption about indexing, and indexing is meaningless without a linearization or an embedding. Linearization needs to be explicitly specified by a certain additional object in JuliaManifolds, while the embedding can be a tuple of vectors and matrices. ArrayPartition sort of gives up here and pretends to be a vector but if that is enough, why can't we pretend that everything is a vector? If an operation can meaningfully distinguish between vectors and matrices, it will ignore the inner structure of ArrayPartition unless it's explicitly special-cased, which is prone to errors when we have about 10 or more other structs like that.

The issue was fixed by #33 .


Question: Is there a way to do camelCase in the bbt cite key?

Sorry, I must have clicked something too quickly:
I prefer to use camelCaseConvention (like that) when typing.  I was wondering if there was an option in the automatic creation of citation keys.  If I have two or three authors, I'd like to see this
dimitrakopoulosFarrellyGodoy2002
or
dimitrakopoulosFarrelly2002
[authEtAl] looks close to what I want, but still does the not-camelCase i'd like.
Thanks in advance

There isn't currently, no.

Cool.  No worries then.  Thanks.


Update year to 2023

Could you also update the selector? It's not actually working

Could you also update the selector? It's not actually working

That's weird since I only updated the comment.  It shouldn't affect the code at all.

I know but I'm saying it's altering a comment on a piece of code that doesn't work. Feels strange 😃

Could you also update the selector? It's not actually working

How do you know that it is not working? The newest version of the Add-on works in Firefox for me.

I think you're misunderstanding me. I'm saying you're making unnecessary changes to a feature that is currently not working. The line after the comment you edited is not working, the year is not being hidden:

I'm not saying that you broke it, I'm saying that I'm not going to accept menial PRs that alter comments around broken code.

Could you also update the selector? It's not actually working

It's working for me 🤔 I checked it's hidden by RGH

Indeed it was updated in https://github.com/refined-github/refined-github/pull/6415 and it's still broken in Safari 👍


KVM: add vm setting for nic multiqueue number and packed virtqueues

Description
This PR adds two vm setting for user vms on KVM

nic multiqueue number
packed virtqueues enabled . optional are true and false (false by default). It requires qemu>=4.2.0 and libvirt >=6.3.0

Tested ok on ubuntu 22 and rocky 8.4
(See test results in Screenshots part)

Types of changes

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

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Screenshots (if appropriate):

Deploy a vm
There are two new options in "Advanced mode" part.

Start the vm. check qemu process on kvm host

it shows

-netdev tap,fds=57:59:60:61:62:63,id=hostnet0,vhost=on,vhostfds=64:65:66:67:68:69
-device virtio-net-pci,packed=on,mq=on,vectors=14,netdev=hostnet0,id=net0,mac=02:00:3f:4f:00:0a,bus=pci.0,addr=0x3

(vectors = 2 * queue number + 2)

virsh dumpxml shows       <driver queues='6' packed='on'/>

log into the vm, and run some commands

enable multiqueue inside user vm

This works on centos7 as well.
but centos7 with tap device does not work. got an exception
org.libvirt.LibvirtException: Unable to create tap device tap0200492f0004: Invalid argument

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5723

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5724

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15.  It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan test keepEnv

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6016

Trillian test result (tid-6499)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45937 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7333-t6499-kvm-centos7.zip
Smoke tests completed. 109 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_10_reboot_cpvm_forced
Error
6.28
test_ssvm.py

LGTM, I've manually tested it and confirmed the failing test was passing in a separate run
Test force reboot CPVM ... === TestName: test_10_reboot_cpvm_forced | Status : SUCCESS ===

great, thanks @borisstoyanov for the testing !

@weizhouapache @borisstoyanov is this ready for merging or there's further testing on-going?

Merging based on manual tests by @borisstoyanov, CI tests and reviews.


Issues with Gradients exporting

I know this has been asked a million times but I am having trouble with gradient colors exporting. I made sure I was using gradient fill and not an effect. Made sure to save the project before exporting. Followed every advice I could find on this forum with no luck. I am working in the latest AE and version 4.11.1 of bodymovin. Please help. I am working on a much bigger projects with TONS of gradients and really need to figure out how to make gradients work.
I am attaching and AE file. The animations are simple because this is just a test project, the much bigger project is confidential but will be using the same things (ease and wizz expressions, etc). Any help is appreciated
Project DOWNLOAD LINK
http://dl.fc.media/content/Jobs/Series/2017/OpenHeart/Projects/HeartStory/Sequences/ExportTest.aep?dl=1

Nevermind! Ease and wizz was the problem. Solved it by converting the expression to keyframes and it works!

Would love for the developer to integrate ease and wizz in future versions

@kerrylofton ease and wizz should be working.
I'll check the .aep and get back to you

@kerrylofton are you using it with Bodymovin or Lottie?

Using it with bodymovin. I've found that ease and wizz works but not with the gradient. If I put ease and wizz on the gradient layer it doesn't export the correct colors, gradient just comes out black and white

interesting, I'll check it out.
thanks!

thank you! If you find to a way that it does work with ease and wizz let me know. The converting expression to keyframes works for getting it out but makes me very nervous about having to go back and make changes down the line with this big project I am working on.

@kerrylofton can you reupload the failing .aep so I can reproduce it?

Sooooo, as I got to work this morning and tried out the failing comp, it now works with ease and wizz and the gradient! lol No idea why but I promise it wasn't yesterday haha either way, glad that it works now with ease and wizz because that makes the work on this big project ALOT easier. thank you so much for the help and for this great program! I will send you the final project when we're done so you can see how it all came out!

Nice!
If you run into this issue again please let me know and I will check it out.


Plugin fails to extract driver if driver-archive contains folders

Downloading the edgedrivers currently fails im my projekt (e.g. edgedriver-windows-32bit-85.0.564.17) using version 3.2.0.
The Archive gets downloaded but can not be extracted:

[INFO] --- webdriverextensions-maven-plugin:3.2.0:install-drivers (default-cli) @ sednaft-engine-ml-admin ---
[INFO] Installation directory "C:\workspace\java\idea_workspace\...\drivers"
[INFO] Installing drivers from configuration
[INFO] edgedriver-windows-32bit version 85.0.564.17
[INFO]   Downloading "https://msedgedriver.azureedge.net/85.0.564.17/edgedriver_win32.zip" to "C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\downloads\edgedriver-windows-32bit-85.0.564.17\edgedriver_win32.zip"
[INFO]   Extracting "C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\downloads\edgedriver-windows-32bit-85.0.564.17\edgedriver_win32.zip" to temp folder
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  2.383 s
[INFO] Finished at: 2020-09-04T16:23:37+02:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.github.webdriverextensions:webdriverextensions-maven-plugin:3.2.0:install-drivers (default-cli) on project sednaft-engine-ml-admin: Failed to extract driver from "C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\downloads\edgedriver-windows-32bit-85.0.564.17\edgedriver_win32.zip" cause of java.nio.file.NoSuchFileException: C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\temp\Driver_Notes\credits.html
[ERROR]
[ERROR] driver: {
[ERROR]   "name": "edgedriver",
[ERROR]   "platform": "windows",
[ERROR]   "bit": "32",
[ERROR]   "version": "85.0.564.17",
[ERROR]   "url": "https://msedgedriver.azureedge.net/85.0.564.17/edgedriver_win32.zip"
[ERROR] }
[ERROR]
[ERROR] downloadDirectory:
[ERROR] C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\downloads
[ERROR] ├── edgedriver-windows-32bit-85.0.564.17\download.completed  0
[ERROR] └── edgedriver-windows-32bit-85.0.564.17\edgedriver_win32.zip     5,5 MB
[ERROR]
[ERROR] tempDirectory:
[ERROR] C:\WINDOWS\TEMP\webdriverextensions-maven-plugin9108440565592915644\temp is empty
[...]

The archive contains (in addition to the driver exe itself) a folder with a credits file inside ("Driver_Notes\credits.html") which seems to cause the problem. I deleted the extra folder and credits.html file, made the patched archive available under some URL and used that URL in my configuration (by <url> tag) : with that the error was gone.
It seems to me the plugin has a problem with driver-archives containing folders.

Older versions also seem to be affected, I can reproduce the error with 3.0.2, too.

As a workaround I tried filtering the archive contents using the  tag but failed so far. (.msedgedriver.)

Yeah same issue here, any solution for this?

I encountered the same issue and have some proposals on enhancements.
The issue with <fileMatchInside> isn't taking effect is that the Repository.enrichDriver isn't handling the  correctly. We should probably use the info from the pom if that is provided. It might be that there are some side effects to iron out.
I also think that the "fileMatchInside" property should be added for Edge (Chromium) and Opera in (https://raw.githubusercontent.com/webdriverextensions/webdriverextensions-maven-plugin-repository/master/repository-3.0.json). This would fix the issue for when one isn't specifying a particular driver version. The solution would be fileMatchInside: "MicrosoftWebDriver.*" for Edge and fileMatchInside: "operadriver.*" respectively according to my initial investigations. This also leads to the workaround I've found: if you roll your own repository.json, then you can add this line and mvn webdriverextensions:install-drivers will extract the correct file.

Pr for the repository file is created: PR #77.

This is fixed in my pull request. Edgedriver ZIP archive contains some file in subfolder, but it does not contain the subfolder.


Single hyphen option highlighted incorrectly

As of commit e27e53b some single hyphen options are highlighted with unknown-token.

I can't reproduce this, Is pacman aliased to sudo pacman in which case does #556 fix the issue?

Yes, pacman is aliased to sudo pacman and #556 does fix this issue.

#556 has been merged.


Get_historical_klines doesn't show full set of data

Hi,
I wanted to use get_historical_klines to pull information for past 4 hours (1 minute intervals), but I get data for the interval t-4h to t-2h. I can't see information for past 2 hrs. Do you know what is the cause of this?
Is there something else you would recommend to see information for past 4 hrs in 1 min intervals?

@MirkoZulj You don't have to use the get_historical_klines() to do this. You can just use the get_klines(). The get_klines() function will return the most recent klines up to the limit. Documentation says 500 klines but I think Binance raised the amount to 1000 (that's how many im currently getting). You only need to set the limit to 240 klines (60 * 4) since you only want 1 minute tickers for 4 hours.client.get_klines(symbol=coin,interval=client.KLINE_INTERVAL_1MINUTE,limit=240)
If you want to use the historical version please provide how you are setting your start_time. Note if doing get historical you don't need to set the end time so that it grabs up to the current time.

I have tried:
output =client.get_klines(symbol="VENETH", interval = Client.KLINE_INTERVAL_1MINUTE, limit=500, startTime=1529920688678, endTime=1529924288683)
And got empty array. If I go beyond 2 hrs it works without problem, but doesn't show last two hours
I get_historical_klines, the loop would stop because len(temp_data) = 0, so the loop breaks.

If I remove startTime and endTime and just put limit to 240, then I get data for t-6hr to t-2hr

I found the error. In the start time I asked for "4 hours ago" and in End time I selected "Now UTC". As I am UTC+2, it gave me data for 2 hours only


Create Siddhi-PR.md

Hi GSSOC team,
https://github.com/Siddhisony/READMEasy/tree/emojiandgif , this is the PR which has been assigned to me and it is not merged yet. It is a level 2 PR. Could you please review it? I am open to any kind of improvements.
Thank you!

that is not a PR link. add valid link

give proper link asap

that is not a PR link. add valid link

https://github.com/Siddhisony/READMEasy.git

give proper link asap

https://github.com/BamaCharanChhandogi/READMEasy/pull/129

Do I have to change anything?

no labels will be assigned for this @Siddhisony

I understand that it's a lot to merge or close the requests. May I know the reason why I haven't got assigned any labels?

Because, you didn't provide the correct link before 7pm


Admin Panel - Assessment Management - Product Scoping

NisAI : Admin Panel
PRD: https://docs.google.com/document/d/104pNxv_RbouK0fJzXNmbVNyWs1etYTch-maXckySTlM/edit
Very Rough sketch in Figma: https://www.figma.com/file/KEv3Bga4hygJiw4QPFMxx3/NL-App_May_2023?mode=dev

[x] Scoping of features
[x] PRD

OOS:

~[ ]  Setup of admin panel~
~[ ]  Auth implementation for Admin and Orgs~
~[ ]  Creation of Orgs~
~[ ]  Dataset builder integration~
~[ ]  Config creation flows~
~[ ]  Export config to Orgs~

Agreed Generation flow cc @rohitsamagra @karntrehan

Select Competency
Select Prompt Schema Format (3 default formats to chose from - Comprehension, QA, BSTT)
Select Question Type (Single select, multi-select, free-text)
Edit prompt for questionnaires generated automatically
Iterate for ’n’ number of examples specified.
Specify reasons for the generated questionnaires.
Save the prompt
Convert to GAP or see the final output.
Ingest as Questionnaires.

Competency to player mapping, how does it need to happen? @rohitsamagra @karntrehan to discuss with @tushar5526

1st iteration of the design seen.

For  V1, Question generation will not be there. Admin can upload , review, save and ingest the questions.
@karntrehan @amit-s19

@VargishChauhan to update PRD and scoping
Waiting for Dt to come back with updated designs

Dropped.


Merge Scala API for DeltaTable

GitOrigin-RevId: 243bc18522d75b05bc1c02b92921aaefdeabfe40
This PR is for https://github.com/delta-io/delta/issues/42, after the change, delta table has the ability to merge some source table/query with optional condition and update/insert rules.
This PR has been tested by MergeIntoScalaSuite.scala

this looks good. will merge.


Further enhanced the syntax support

Force pushed to #3 and now I can't reopen

I realise this is a pretty hefty PR, but I'm not sure what your thoughts are on this repo, or if you're planning on keeping in maintained.
I've added syntax support for nearly everything (I think anyway) -- there are a few small problems with some of the JS style attributes, etc., but I hope to have them sorted soon-ish.
If you'd rather, though (and I'm guessing this will be the case seeing as I've rewritten most of it, and to avoid any upstreaming requests), I can just "unfork" my repo to be maintained separately by me.
Let me know what your thoughts are :)

I don't think this is being maintained by @zealot128 anymore, as I've had a pull request pending since May 2022 with no response. Your best bet is to put in a pull request with the nvim-treesitter project itself to switch to your fork.

@JulesNP @mclaughlinconnor ups, I didn't noticed this PR.  The default notification was only on for Mentions/watched threads for both of your PR's. (Maybe because this is a fork itself...?)
Will check out if it works for "my" pug files, and if yes, I see no issues to give you commit rights. I only forked it myself to make it work for my use cases anyway. It would be best for this repo to be part of @pugjs group, but I am not in contact with anyone from that team.

On cursory glance it parses most of the vue-pug files that I took a look on. Will use the fork for a couple of workdays.
Maybe @JulesNP can also check out, if this PR fixes the quotation issue, because I think that merging both will be hard because of conflicts.

@JulesNP No, my code doesn't fix your string quoting issue. I probably could have, but I didn't want to shamelessly steal your code :)
I haven't tested the code, but I know for sure it will have conflicts.
I think you'll also want to use the same code for quoted_javascript nodes

Re living under @pugjs, I agree, but there doesn't seem to be much activity in the group. I'm not in contact with anyone either, but @ForbesLindesay is pretty active on Github, so maybe we'll get a response with his thoughts on moving this into the group.

On cursory glance it parses most of the vue-pug files that I took a look on. Will use the fork for a couple of workdays. Maybe @JulesNP can also check out, if this PR fixes the quotation issue, because I think that merging both will be hard because of conflicts.

It doesn't fix my quoting issue, but everything else with this version is working well. I'll withdraw my pull request and resubmit an updated version once @mclaughlinconnor's changes are merged.

Haven't noticed any regression in the highlighting of my Vue/pug files.
Great work!!
Only some wording in the README is Angular only. Can you adjust that please? Now the grammar is not only useful for us Vue or Angular folks, but almost general purpose.

Awesome! Thanks for the huge contribution.


Object wireframe markers show wrong orientation

Object wireframe box markers (e.g. from ObjectData2280) are showing wrong orientation due to duplicate (in case of non-ECU ObjectData) or unnecessary (in case of ECU ObjectData) conversion from 1/100 deg to rad.

Closed with #6.


Update stable version in repo from 1.1.4 to 1.2.2

Think about update stable version to 1.2.2
Version: stable=1.1.4 (790 days old) => latest=1.2.2 (15 days old)
Installs: stable=1451 (86.83%), latest=93 (5.57%), total=1671
Click to edit

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.2.2 but the current stable version is already 1.2.2.
So this issue should be closed.


test : add tests for document creation methods

Codecov Report

Merging #2 into master will increase coverage by 13.51%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master       #2       +/-   ##
===========================================
+ Coverage   83.78%   97.29%   +13.51%
===========================================
  Files           1        1
  Lines          37       37
  Branches        1        1
===========================================
+ Hits           31       36        +5
+ Misses          6        1        -5

Impacted Files
Coverage Δ

src/main/scala/es/weso/document/Document.scala
97.29% <0%> (+13.51%)
: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 87b3e72...7ebd7c4. Read the comment docs.


Add void.cc to the webring

Hey! I love your projects and the whole rabbits universe.
This idea of a webring triggers good memories from late nights from the 90's, where we had to wait for midnight to connect through a 56k modem and navigate in a world of ASCII zines, IRC channels and loopy GIFs.
I try to keep void.cc as my personal corner in the web with random bits excavated from my brain dump. I humbly hope it fits in yours universe.

Looks like a perfect fit to me :) Welcome to the ring


apk_builder.py fails with unzip:  cannot find or open lib.zip, lib.zip.zip or lib.zip.ZIP

I executed the following command line
$ python apk_builder.py --apk myapp.apk
but it fails with the following error message
[I] Preparing work directory...
[I] Expanding APK...
[I] Searching .smali
[I] Patching .smali
[I] Injecting libs
unzip:  cannot find or open lib.zip, lib.zip.zip or lib.zip.ZIP.
Traceback (most recent call last):
File "apk_builder.py", line 168, in
unzip_output = subprocess.check_output(["unzip", LIB_FILE_PATH, "-d", lib_dir])
File "/usr/lib/python2.7/subprocess.py", line 573, in check_output
raise CalledProcessError(retcode, cmd, output=output)
CalledProcessError: Command '['unzip', 'lib.zip', '-d', '/tmp/appmon_apk/myapp/lib']' returned non-zero exit status 9
What's wrong?

@rena2019 You need to first execute the script (provided with apk_builder.py)
"sh getlibs.sh" (without quotes)


Updating to prost-wkt 0.5 can break downstream builds

I had an existing project using prost-wkt 0.4 with chrono 0.4.24. When I updated to prost-wkt 0.5, the build broke because DateTime::from_naive_utc_and_offset is not in that version of chrono. The fix for my project was cargo update -p chrono, but this will bite others too.
I think the right fix is to change Cargo.toml to insist on a version of chrono that is recent enough to use. chrono = "0.4.31" would select the version that's current right now.

Thanks for raising this issue! This is a good catch. I will update the Cargo.toml to better specify the dependency of chrono. Will aim to release it in 0.5.1 within next week.

I released v0.5.1 which uses chrono 0.4.27 which contains DateTime::from_naive_utc_and_offset. Let me know if this release works well for you.


Sparql logs

Hello;
I'm a PhD student.  And I'm working on sparql logs in BI systems.
I found your project really interesting; I just ask you if you can share with me some sparql logs from your Endpoint deployed services.
I found, the LSQ logs (about DBpedia, British museum, LGD and SWDF) but for my tests, I need others.
Unfortunately, I can’t find anymore.
I'll be really grateful if you can send me some other logs.

Hi Dihia, we are working on a completely revamped and improved version of LSQ which will finish in the coming days. The model will be a bit different but also cleaner and more consistent; and it will allow storing results from multiple independent benchmarks.
Do you need additional RDFized logs or the raw logs? Publishing the RDFized logs is fine (as we salt the ips) but some of the raw ones have restrictions.
We have RDFization of new logs pending such as wikidata which should complete during August.


[Help] Explain utterancess identifier

Mandatory Step

[X] I am using latest production release of Elegant

I need help with
@jackdewinter would you like to add explanation in Comments — Enable Utterances  or Comments — Thread ID explaining what will be passed as utterances identifier.
Thank you.

@talha131 why did we choose to append siteurl as article identifier to commenting system (disqus/utterances), isn't slug is already unique on its own?

@talha131 https://github.com/talha131 why did we choose to append
siteurl
https://github.com/Pelican-Elegant/elegant/blob/0e7f5f8616c5eb6ce4bbba42601a6431f8750c65/templates/_includes/comments.html#L10
as
article identifier to commenting system (disqus/utterances), isn't slug is
already unique on its own?

Not necessarily. Like we have two sites, elegant and next.elegant. They
have same slugs. Base URL is important to differentiate.
Also I think, (I do not have code in front of me rug right now), these
sites require complete URL.
On Tue, Dec 10, 2019, 02:22 azzamsa<EMAIL_ADDRESS>wrote:

@talha131 https://github.com/talha131 why did we choose to append
siteurl
https://github.com/Pelican-Elegant/elegant/blob/0e7f5f8616c5eb6ce4bbba42601a6431f8750c65/templates/_includes/comments.html#L10
as article identifier to commenting system (disqus/utterances), isn't slug
is already unique on its own?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Pelican-Elegant/elegant/issues/547?email_source=notifications&email_token=AAILIRC72LSX4RCP5KDQSMLQX2ZLBA5CNFSM4JYSLNHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGKX6HQ#issuecomment-563445534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAILIRF6Q3E33MHVEODN6BLQX2ZLBANCNFSM4JYSLNHA
.

@talha131 why did we choose to append siteurl as article identifier to commenting system (disqus/utterances), isn't slug is already unique on its own?

Not necessarily. Like we have two sites, elegant and next.elegant. They
have same slugs. Base URL is important to differentiate.
Another reason is that official examples of all the comment systems, use a complete URL as the identifier. Elegant just does exactly the same.


Improved calc mode error correction parameters interpretation string

Pull Request Test Coverage Report for Build 132

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
1730 unchanged lines in 31 files lost coverage.
Overall coverage increased (+49.3%) to 77.529%

Files with Coverage Reduction
New Missed Lines
%

block_preds.rs
4
100.0%

file_writer.rs
5
55.36%

file_error.rs
8
24.24%

integer_utils.rs
9
56.25%

progress_report.rs
10
85.09%

file_reader.rs
11
72.22%

main.rs
12
90.63%

general_error.rs
13
65.0%

cli_repair.rs
15
83.33%

cli_decode.rs
15
96.67%

Totals

Change from base Build 131:
49.3%

Covered Lines:
2736

Relevant Lines:
3529

💛  - Coveralls

Pull Request Test Coverage Report for Build 132

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
1730 unchanged lines in 31 files lost coverage.
Overall coverage increased (+49.3%) to 77.529%

Files with Coverage Reduction
New Missed Lines
%

block_preds.rs
4
100.0%

file_writer.rs
5
55.36%

file_error.rs
8
24.24%

integer_utils.rs
9
56.25%

progress_report.rs
10
85.09%

file_reader.rs
11
72.22%

main.rs
12
90.63%

general_error.rs
13
65.0%

cli_repair.rs
15
83.33%

cli_decode.rs
15
96.67%

Totals

Change from base Build 131:
49.3%

Covered Lines:
2736

Relevant Lines:
3529

💛  - Coveralls

Codecov Report

Merging #63 into master will increase coverage by 0.26%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   77.26%   77.52%   +0.26%
==========================================
  Files          40       40
  Lines        3527     3529       +2
==========================================
+ Hits         2725     2736      +11
+ Misses        802      793       -9

Impacted Files
Coverage Δ

src/cli_calc.rs
98.96% <100%> (+4.17%)
:arrow_up:

src/main.rs
90.62% <0%> (+0.3%)
:arrow_up:

src/file_utils.rs
91.22% <0%> (+4.38%)
: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 b55ca1c...43b3c44. Read the comment docs.


refactor: fix wartremover errors

Summary by CodeRabbit

New Features

Added conditional logic in ColumnTypeString class for enhanced data handling.
Introduced an enclose method for better value representation in RecordValue.

Refactor

Changed several classes from case class to final case class for improved immutability and thread-safety.
Refactored test methods and input validation for clarity and efficiency.

Bug Fixes

Fixed type conversion issues in EdgeQuery and VertexQuery by explicitly converting count results to Long.

Chores

Removed unnecessary warnings and enabled coverage reporting.
Added @SuppressWarnings annotations to suppress specific warnings.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (91261c5) 78.86% compared to head (620ce5b) 78.86%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/main/scala/Main.scala
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #127   +/-   ##
=======================================
  Coverage   78.86%   78.86%
=======================================
  Files          25       25
  Lines         459      459
  Branches       43       43
=======================================
  Hits          362      362
  Misses         97       97

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support Menu key in addition to right click (on Microsoft Windows keyboards)

This key is present on some keyboard. It's used to open context menus.
https://en.wikipedia.org/wiki/Menu_key

I suppose the underlying use case here is to enable using context menus without lifting one's hand to the mouse? As in, press the Menu key, then select and activate a menu item via cursor/shortcut/Enter keys.
Fully implementing that will require enhancing the keyboard-based UI navigation, also enabling #144.


Sync Gradle-generated aar's version with SDK version

I'm not sure if this is required but it makes me uneasy that we just show version as 1.0 in the POM for the built artifacts such as ably_flutter_release.
Discovered if you flutter build aar --no-profile from root, and then:
ably-flutter % cat build/outputs/repo/io/ably/flutter/plugin/ably_flutter_release/1.0/ably_flutter_release-1.0.pom
<?xml version="1.0" encoding="UTF-8"?>
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>io.ably.flutter.plugin</groupId>
  <artifactId>ably_flutter_release</artifactId>
  <version>1.0</version>
  <packaging>aar</packaging>
  <dependencies>
    <dependency>
      <groupId>io.ably</groupId>
      <artifactId>ably-android</artifactId>
      <version>1.2.10</version>
      <scope>compile</scope>
    </dependency>
    <dependency>
      <groupId>com.google.firebase</groupId>
      <artifactId>firebase-messaging</artifactId>
      <version>22.0.0</version>
      <scope>compile</scope>
    </dependency>
  </dependencies>
</project>

I've investigated this and I don't think there's much we can do. To set the artifact version for AAR builds, flutter recommends using --build-number flag like flutter build aar --build-number 1.2.13, and apparently, there's no other way to do this.
I'm not entirely sure if it's even a problem for us - as far as I can tell, we do not release AAR anywhere - the release and publish flow only includes a GitHub release with source code, and pub.dev release. Neither of those use AAR files, so I think we don't have to worry about this. @QuintinWillison Let me know if it's okay to close this 🙂

Fair enough. I wasn't sure when I created this issue. Thanks for investigating - let's close this.


Source map parsing

https://github.com/clydeshaffer/GameTankEmulator/issues/33
Parse the dbgfile from ld65 to use as a source map
Currently requires the "bank" attribute printed in the dbgfile from cc65 fork: https://github.com/clydeshaffer/cc65
Like the memory map feature it assumes/requires a the directory structure and tooling of a gametank_sdk project

Interface added for adding breakpoint by C filename and line number


docs: update details about auto save/load options.

A few small corrections to readme around auto saving and loading that were previously missed.

Thanks!


Don't crash on non-literal computed property names during getPropertyAssignment

getPropertyAssignment is searching for a property with a given text; when it encounters a bad property name, it crashes. Just skip over the bad property name, as it can't be equal to the key being searched for anyway.
This test is weird; the failure mode doesn't look like the bug in this particular harness (but it works).
Test failure:
fourslash-server test tsconfigComputedPropertyError.ts runs correctly
  [.................................................................]

  0 passing (2s)
  1 failing

  1) fourslash-server tests
       tests/cases/fourslash/server/tsconfigComputedPropertyError.ts
         fourslash-server test tsconfigComputedPropertyError.ts runs correctly:
     Error: Debug Failure. False expression: Malformed response: response sequence number did not match request sequence number.

Fixes #47989

With that change cherry-picked, I do get a reasonable error without the fix:
  1) fourslash-server tests
       tests/cases/fourslash/server/tsconfigComputedPropertyError.ts
         fourslash-server test tsconfigComputedPropertyError.ts runs correctly:
     Error Error processing request. Debug Failure. Text of property name cannot be read from non-literal-valued ComputedPropertyNames
Error: Debug Failure. Text of property name cannot be read from non-literal-valued ComputedPropertyNames
    at getTextOfPropertyName (E:\work\TypeScript\src\compiler\utilities.ts:1006:30)
    at E:\work\TypeScript\src\compiler\utilities.ts:1581:34
    at Array.filter (<anonymous>)


Kusto queries are the same?!

[Enter feedback here]
In the https://docs.microsoft.com/en-us/power-bi/developer/embedded/monitor-power-bi-embedded#sample-kusto-queries section both queries are the same! Is this correct?

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

ID: 6acd0f15-412a-cc53-64fc-69840687dc7d
Version Independent ID: ddd31412-6865-be3d-f36b-72b66cf9a495
Content: Monitor Power BI Embedded - Power BI
Content Source: powerbi-docs/developer/embedded/monitor-power-bi-embedded.md
Service: powerbi
Sub-service: powerbi-developer
GitHub Login: @mberdugo
Microsoft Alias: monaberdugo

Thanks for your comment, @TiagoRente. This issue has been assigned to the doc author for review.

Thanks for pointing this out, @TiagoRente. I'll check with the devs and clarify this.


Update figure notations for 2101

Codecov Report

Merging #340 (2ccfc6e) into master (d1a0761) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #340   +/-   ##
=========================================
  Coverage     64.60%   64.60%
  Complexity      701      701
=========================================
  Files           112      112
  Lines          2557     2557
  Branches        332      332
=========================================
  Hits           1652     1652
  Misses          775      775
  Partials        130      130

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 d1a0761...2ccfc6e. Read the comment docs.


Know column where you click

First of all, thank you so much for the great work.
Is there any way to know which column I click on?
Thanks!

@Ixreb how did you solved it?


Make AllowedMethodsFilter immutable

It looks like that the set of allowed methods in AllowedMethodsFilter is not modified after it was created. We could embrace it and make immutable during the creation time. This also should provide some performance benefits, because immutable hash tables could be optimized during construction times (e.g. they can use linear probing instead chaining)

Looks good to me.  Any idea why appveyor has been failing recently?

Coverage decreased (-0.006%) to 84.398% when pulling a7a74ecfc37523f17b2bcf0141a25428d3b74338 on methods_filter into f56d66e3220472dbabc99ec2b8dd30537425fe8d on master.

The reason is that there is already a test table in some of the tests and so the CREATE TABLE test will fail. Whether it is because h2 isn't shutting down cleaning or tests are being executed concurrently, I'm not sure. The temporary fix would be to transform all CREATE TABLE test to CREATE TABLE test IF NOT EXISTS

I guess the pull requests with new Liquebase tests could be a culprit. It creates shared H2 databases to keep the state between different migrations.

Coverage decreased (-0.006%) to 84.398% when pulling 1e7e78dcc360e696651edfeef3c4ea2d7a725fe4 on methods_filter into f56d66e3220472dbabc99ec2b8dd30537425fe8d on master.

I think I fixed the issue with the JDBI errors. I added shutdown code in the JDBI tests to make sure the tests don't reuse the same database.


txn: Move ts to each command

Signed-off-by: longfangsong<EMAIL_ADDRESS>
What problem does this PR solve?
Minor refactor in src/txn/commands.rs.
What is changed and how it works?
Move ts to each command and abstract them with a CommandExt trait.
Check List
Tests

Unit test

@nrc PTAL.

I think it is worth planning the command macro syntax a bit. Prewrite, Vec<Result<()>>, prewrite, ts => start_ts is getting a little confusing and if we add more (which I think we will) then it will get worse. Could you have a think about what we'll want in the final version of the macro, and how to make that readable please?

This is exactly what I'm doing now.
Currently I think something like this one is good:
Prewrite:
    cmd_ty => Vec<Result<()>>,
    tag => prewrite,
    ts => start_ts,
    is_sys_cmd => false,
    can_pipelined => false,
    content => {
        /// The set of mutations to apply.
        mutations: Vec<Mutation>,
        // ...
    }

But this pattern got some trouble on readonly and several other methods, since they might need a &self argument to work, and we cannot use expressions like self.key_locks.is_empty() in macros.
Currently, the best solution I have is use another trait, eg:
pub trait CommandReadonly {
    fn readonly(&self) -> bool;
}
pub trait CommandExt: CommandReadonly {
    // ...
}

And if a value is provided in the macro, it will generate the impl block:
$(
        impl CommandReadonly for $cmd {
            fn readonly(&self) -> bool {
                $readonly
            }
        }
)?

If no value is provided, then we can impl the CommandReadonly trait out of the macro.
The problem of this solution is that the code which want to use the commands might need to import another traits, and I don't really know it is ok.

I think you could just use one trait and do the same thing - either implement in the macro, or implement outside for other types.

and we cannot use expressions like self.key_locks.is_empty() in macros.

I don't understand this, could you say why we can't use such expressions? Or give an example.

and we cannot use expressions like self.key_locks.is_empty() in macros.

I don't understand this, could you say why we can't use such expressions? Or give an example.

What I want is
command! {
    ResolveLock:
        cmd_ty: (), tag => resolve_lock, is_sys_cmd => true, readonly => self.key_locks.is_empty() {
        // ...
    }
}

but due to macro hygiene, we cannot use self here.\

I see. Thanks for the explanation. I think the way to do this is to have a default readonly method that returns false, an option for the macro to return true, and for ResolveLock to override the method with a manual implementation.

for ResolveLock to override the method with a manual implementation.

I can't figure out how to do this (override a single method in a trait which has other methods) without using a new trait. Can you show me an example?
And shall we merge this pr first?

And shall we merge this pr first or continue to push further refactors into this pr?

Yes lets merge this as is.
@youjiali1995 @sticnarf @MyonKeminta could one of you take a look please?

I can't figure out how to do this (override a single method in a trait which has other methods) without seperate this method into a new trait. Can you show me an example?

trait A {
    fn foo(&self) -> bool;
    fn bar(&self) -> bool {
        false
    }
}

struct B;
struct C;

// Macro generated
impl A for B {
    fn foo(&self) -> bool {
        true
    }
}

// Macro generated
impl A for C {
    fn foo(&self) -> bool {
        true
    }
}

// Manually implemented
impl A for C {
    fn bar(&self) -> bool {
        true
    }
}

I think this is a bit simpler than Command, but it shows the principle. Is it clear?

I can't figure out how to do this (override a single method in a trait which has other methods) without seperate this method into a new trait. Can you show me an example?

trait A {
    fn foo(&self) -> bool;
    fn bar(&self) -> bool {
        false
    }
}

struct B;
struct C;

// Macro generated
impl A for B {
    fn foo(&self) -> bool {
        true
    }
}

// Macro generated
impl A for C {
    fn foo(&self) -> bool {
        true
    }
}

// Manually implemented
impl A for C {
    fn bar(&self) -> bool {
        true
    }
}

I think this is a bit simpler than Command, but it shows the principle. Is it clear?

The compiler will complain about conflicting implementation for C in this way.

The compiler will complain about conflicting implementation for C in this way.

Ah, indeed, my mistake. You can have multiple impls with no trait, but not for trait impls.
So I think the macro needs to get a bit smarter. Either you have to take an ident token to use as self, or you could take the entire function as an item token stream (iirc). Either way, you can make it optional so that most macro instantiations don't need to use it.

/merge


OVRO-LWA slow recorder png writing error

Noticed that the drslow recorder service was trying and failing to write a file.
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:   File "/home/ubuntu/anaconda3/envs/casa/lib/python3.6/threading.py", line 916, in _bootstrap_inne
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:     self.run()
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:   File "/home/ubuntu/anaconda3/envs/casa/lib/python3.6/threading.py", line 864, in run
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:     self._target(*self._args, **self._kwargs)
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:   File "/home/ubuntu/ovro_data_recorder/dr_visibilities.py", line 524, in main
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:     mp.to_file(filename)
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:   File "/home/ubuntu/anaconda3/envs/casa/lib/python3.6/site-packages/mnc_python-0.1.1_23_ga86862e_
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]:     with open(name_or_handle, 'wb') as fh:
Apr 05 17:13:47 lxdlwagpu09 dr-vslow-2[9299]: PermissionError: [Errno 13] Permission denied: '059675_001314_baselines.png'

I've turned off the writing of the images to disk.  They will still be available through their respective monitoring points.


Reversing Dependency: Implementing Untyped-RPC on Top of Typed-RPC (draft)

In this Pull Request, I've reversed the dependency between the typed-RPC and
untyped-RPC implementations. Now, the untyped-RPC is built on top of the
typed-RPC, a shift from the previous structure.
This change is motivated by several factors:

It paves the way for a straightforward deprecation of the untyped-RPC in the
future, as we can simply remove the code when ready.

It reduces indirection and allocation in the typed-RPC implementation path.
With this new implementation, there's no need to map over sequences, which
should enhance the performance of the typed implementation and clarify the code.

While this modification isn't strictly necessary for the PR introducing the
typed-RPC interface (#48), I believe it's a valuable addition. This approach aligns
with my original vision during the exploration phase, and I recommend
implementing it without significant delay. Specifically, I suggest that the
typed interface implementation for other concurrency libraries should follow a
similar path, bypassing the initial exploratory phases of building the typed-RPC
on top of the untyped one.

Currently this PR is built on top of #48 thus duplicating the commit chain.
Also, currently I have only gone through the client side, I plan on making a similar change to the server side shortly.

@quernd and @tmcgilchrist I prefer this implementation over the one in #48. If you like it too, I will use this as the basis for the typed-rpc-eio implementation that I enable for code review. Please let me know what you think. Thank you!

I ended up re-using this code organization in #55.


feat: implement fmg endpoint and processing pipeline

#3540
#3541

Reviewers
Functional:
@atolopko-czi
Readability
@Bento007
Algorithmic
@pablo-gar
This PR adds a new endpoint (/markers) to the Gene Expression API. It queries a new data artifact in the snapshot (marker_genes) which is a TileDB array containing precomputed marker genes. This cube is now created as part of the weekly cube generation process (see backend/wmg/pipeline/summary_cubes/marker_genes.py).
Notable changes

Marker gene calculation module - marker gene calculation logic: backend/wmg/api/calculate_markers.py
Query criteria - marker genes cube and expression summary fmg cube: backend/wmg/api/query.py
API endpoint - /markers: backend/wmg/api/v1.py and backend/wmg/api/wmg-api.yml
Data artifacts - new data artifacts have been added to WmgSnapshot: backend/wmg/data/snapshot.py
WMG processing pipeline step - create_marker_genes_cube: backend/wmg/pipeline/summary_cubes/marker_genes.py
Test snapshot for marker gene tests - tests/unit/backend/wmg/fixtures/test-fmg-snapshot
Marker gene calculation tests - tests/unit/backend/wmg/test_marker_gene_calculation.py
Marker gene query tests - tests/unit/backend/wmg/test_query.py
Marker gene cube creation test - tests/unit/wmg_processing/test_marker_gene_cube_creation.py

Note to reviewers
If there is any missing test coverage, please let me know and I will add the corresponding tests.

Completed a full pass. Overall, looks good! Added a number of (mostly minor) in-line comments and these:

Can you add some documentation or a script that is used to build the new snapshot fixture? It should be as easy as possible for a future engineer to recreate this fixture. Tests that make use of it should comment that "If you need to update the fixture due to changes in expected test values, please refer to...".

Done! It's now in scripts/generate_fmg_test_snapshot.py.

Maybe as a separate issue/PR, but is there anything that can be added to backend.wmg.data.validation.validation.Validation to validate the FMG data artifacts?

I think @pablo-gar and I will work on the validation in a separate story.

Out-of-scope for this PR, but in keeping with the ETL structure of the pipeline job classes:

Should backend.wmg.pipeline.summary_cubes.expression_summary_fmg.job._load() be in load.py (and not job.py)?
Should backend.wmg.pipeline.summary_cubes.expression_summary_fmg.load.build_in_mem_cube() bin transform.py?

I copied the structure of the summary_cubes.expression_summary module, but your suggestions make sense in the context of ETL structure!


Update not-latest-version.md

The preview notice is showing for 9.0, which is the default doc load now.

I'll take this live now.


Missing config option for Ludwig's Holy Blade

In config file there is no options for this blade, no disabling, no balancing

And also will there be config option to balance armor characteristics?

It is outdated. Delete the config file, launch minecraft and generate a new one. It will then be present.
As for the armor bonuses, maybe in the future.


[AutoPR policyinsights/resource-manager] Specs for Microsoft.PolicyInsights GA version (2018-04-04)

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2929

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2122


Don't sort import declarations that only import for side effects

For example, the following should be left as-is:
import "./b.css"
import "./a.css"

See #369

dprint's CLI formats file in parallel in a non-deterministic order and plugins don't have access to the file system so it would be infeasible to support that (also, it would make things slow).


Error in uploading the large training set

error: this exceeds GitHub's file size limit of 100.00 MB

The last three commits were deleted by editing the rebase in interactive mode. After that, all the commits which were not pushed got deleted, and I have work on it again.
Reference: Rewriting Git History

Now, I am able to push, but still, I have to use git LFS (Large File Storage) to upload X.npy.

Git LFS is used to track ".npy" file.
Commands used:

git lfs install
git lfs track ".npy"


array_add deprecated laravel 5.8

array_add is deprecated since laravel 5.8. I shout be changed to Arr:add

Thanks for reminding me! :)

This issue is resolved in version 1.0.4.


[FR] - Log an ETA when cardano node is expected to finish replaying blocks

Internal/External
Internal
Area
Quality of life improvement
Describe the feature you'd like
When upgrading Cardano node it would be nice if it could log an ETA when the upgrade is expected to finish
Describe alternatives you've considered
None

When upgrading or syncing the node we currently don't have an ETA provided. Adding this feature could help us to better manage our resources during node upgrades or resynchronization.


I would like to add time and distance calculate feature in the speed calculator.

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like

A clear and concise description of what you want to happen.

i would like to work on the issue.

@jugeshraghav can you do some thing like what we have on P/B/H trigo
like if 3 input fields are there , distance , time , speed
if user enter 2 of them it should show the 3rd one

Ya I would
On Wed, Jun 23, 2021, 12:25 111arpit1 @.***> wrote:

@jugeshraghav https://github.com/jugeshraghav can you do some thing
like what we have on P/B/H trigo
like if 3 input fields are there , distance , time , speed
if user enter 2 of them it should show the 3rd one
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/makesmatheasy/makesmatheasy/issues/5304#issuecomment-866579936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVI6WOFTZJMK55RLCDX3G3TUGANZANCNFSM47FD2Z2Q
.

Ok then go ahead @jugeshraghav

Deadline 28 june

the issue is completed so i am closing this issue. Thank you so much @arpit456jain @sairish2001 you helped me a lot.


Auto-formatter

Hello 👋🏼 C4 + Structurizr is going to be a great addition to our team!
I was wondering how feasible an auto-formatter would be.
I tried exporting to DSL and overwriting the file, but as I half-expected, it didn't fix any indentation.
(This is mostly a curiosity, not a deal-breaker or urgent).

I have no plans to build an auto-formatter, but there are a couple of VS Code extensions and an IntelliJ plugin that might provide this.


A Submenu where we can search for our term in specific website

The Motivation
There are cases, which the user wants to search for a term in specific website and get the quick result.
The Solution
In some launchers, there is possibility to type what you want in text box, and it will search for the term in your preferred website. I have some examples here.
We can have a sub menu where we can click on, and then a text box opens up and when we type, it searches for our term in that website.
The Alternatives
Here is how my setting in uLauncher looks like

Thanks again. However, as I metioned in #346, Fly-Pie is currently in low-maintenance mode as I am investing most of the little spare time I have into its successor project, Kando. So I personally will not implement new features for Fly-Pie these days.
Also, Fly-Pie is a mouse- or touch-based tool, so if you want to search for something with your keyboard, maybe other tools are better suited. That being said, I'll keep this open to see if maybe others are interested.

I have made a small workaround for the process.
It looks like that.

Inside Fly Pie, I launch ULauncher and then type what I want inside that. It works, but it's not the best option, because I have 3 Launchers  now :) Fly Pie for when I work touch intensive works, ULauncher for searching inside Fly Pie and Rofi for when I work Keyboard intensive works.

Thanks again. However, as I metioned in https://github.com/Schneegans/Fly-Pie/discussions/346, Fly-Pie is currently in low-maintenance mode as I am investing most of the little spare time I have into its successor project, Kando. So I personally will not implement new features for Fly-Pie these days.

Really appreciate the idea of Gnome Pie and Fly pie. Love to see Kando. It looks very promising and would love to work with that. Thanks for developing and maintaining such a nice piece of code.

Also, Fly-Pie is a mouse- or touch-based tool, so if you want to search for something with your keyboard, maybe other tools are better suited. That being said, I'll keep this open to see if maybe others are interested.

Of course. Actually my use case is when I study something, or work with some patients data, when taking note, be able to search for some details in medical encyclopedias. It helps me to be much more productive. Also because of the sensitivity of the data I always work with Open Source and stable apps like Fly Pie.
Thanks for making the world better place.

Closing this as it's most likely not going to be added to Fly-Pie.


AttributeError: 'XAxis' object has no attribute '_gridOnMajor'

🐛 Bug Description

In example notebook workflow_by_code.ipynb, encountered AttributeError when running line analysis_model.model_performance_graph(pred_label)
To Reproduce
Steps to reproduce the behavior:

Open workflow_by_code.ipynb
Run all cells

Expected Behavior

All cells in notebook should run without error.
Screenshot

Environment
Note: User could run cd scripts && python collect_info.py all under project directory to get system information
and paste them here directly.
Linux
x86_64
Linux-5.8.0-43-generic-x86_64-with-glibc2.10
#49~20.04.1-Ubuntu SMP Fri Feb 5 09:57:56 UTC 2021
Python version: 3.8.5 (default, Sep  4 2020, 07:30:14)  [GCC 7.3.0]
Qlib version: <IP_ADDRESS>
numpy==1.19.2
pandas==1.2.2
scipy==1.6.0
requests==2.25.1
sacred==0.8.2
python-socketio==3.1.2
redis==3.5.3
python-redis-lock==3.7.0
schedule==1.0.0
cvxpy==1.0.21
hyperopt==0.1.1
fire==0.4.0
statsmodels==0.12.1
xlrd==2.0.1
plotly==4.12.0
matplotlib==3.3.4
tables==3.6.1
pyyaml==5.4.1
mlflow==1.13.1
tqdm==4.56.0
loguru==0.5.3
lightgbm==3.1.1
tornado==6.1
joblib==1.0.1
fire==0.4.0
ruamel.yaml==0.16.12

Commit number (optional, please provide it if you are using the dev version):
fa8f1cba06ba511744a0625afdf2cc3ac05302d0

Additional Notes

I have the same problem:

I have the same problem:

I believe this issue is resolved after getting latest source cbbf6cd82260e533662a7433ba72cf3391347619.

Not sure what happened, after getting commit 40416d8c301efdfadc572a8749be61cc7376d2c3, the issue happens again.

@arisliang Hi,  you can try updating matplotlib and plotly:pip install -U matplotlib==3.1.3 plotly==4.12.0

@kurucan Hi, you can try updating matplotlib and plotly: pip install -U matplotlib==3.1.3 plotly==4.12.0

After running the pip, it works now. Thanks!

Hi, @arisliang
Would you mind sending a PR to improve the docs about this Question?
Other users who have similar confusion will be grateful to you :)
Thanks
For example, add some docs here.
https://qlib.readthedocs.io/en/latest/FAQ/FAQ.html

A search in Issues on the error message would lead to this issue directly. Think it should be ok to leave it here.


Weird error message for object copy a file from staging

When testing out pelican client with my own director running as well as my own generated token, I get a panic error when running pelican object copy for the first time (accessing the file from the origin rather than the cache). This error only appears when grabbing from the origin and not when accessed from the cache. The error is below:

panic: increment required before ewma iteration update

goroutine 87 [running]:
github.com/vbauerster/mpb/v7.(*Bar).DecoratorEwmaUpdate.func1(0xc000de5e68?)
        /go/pkg/mod/github.com/vbauerster/mpb/v7@v7.5.3/bar.go:266 +0xb8
github.com/vbauerster/mpb/v7.(*Bar).serve(0xc000198ea0, {0x1a82828, 0xc000512d70}, 0xc0007fc5a0)
        /go/pkg/mod/github.com/vbauerster/mpb/v7@v7.5.3/bar.go:349 +0xfc
created by github.com/vbauerster/mpb/v7.newBar
        /go/pkg/mod/github.com/vbauerster/mpb/v7@v7.5.3/bar.go:84 +0x1c5

Seems to be some error with progress bars. The file does download successfully so this bug is not fatal but looks like it is because of the panic. Should be fixed

Looks like a bug in the underlying library -- is there a newer version available?

Yes, looks like the library is on v8 now instead of v7. I can try to make some changes to see if it'll fix it.


avoid auto github sync

I have markdown document which is synced with google drive and github repo. There are several commits created in github as and when I edit the document in stackedit. Is there any option to manually sync once I am done with the editing?

There is a feature to publish manually instead of auto synchronize. This issue can be closed.

Hi @rajanand
Could you point me to this option where auto-sync can be disabled?

As a workaround I added following custom setting:
autoSyncEvery:<PHONE_NUMBER>000
This will now automatically sync every 100 years instead of default every 90 seconds.


feat: initial OpenFGA v2beta1 protobuf API development

Description
Start the development branch for the v2beta1 protobuf API definitions for OpenFGA. This is to get things started with that new API development effort as we go into the new year.
References

Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

[ ] todo: add protobuf doc to every protobuf message and field as appropriate.
[ ] todo: add required field specifications and other protobuf annotations for OpenAPI generation


DmxOutput.h void write not const

Is there a reason write does not take a const uint8_t *universe? I'm just delving a little deeper into C++ itself and from my latest understanding, this would make more sense. I could be wrong, so wondering your thoughts.
Sincerely,
Marcus

Hi, Marcus!
You're absolutely right :) probably a little slip up that was made in the infancy of this library that was never caught.


Use fancy_regex instead of onig

This depends on https://github.com/fancy-regex/fancy-regex/pull/38
We want this because we experienced multiple onig segfaults during our testing.

Thanks for the PR! It's really appreciated!


added comments to Welcome controller, cleaned up gallery comments and…

… updated changelog correctly
Dear Captain Raj,
Please accept my pull request and also my deepest apologies for neglecting to update the changelog with the previous changes.
Sincerely yours,
First Mate Derek

Thank you first mate!


bugfix - fix handling nested properties in PactDslJsonBody

missing initial dot in matching key when property was escaped

Thank you for that fix!


Performance Testing - Testing time for the links from the menu to load.

Description : Verify the load time for the links from the main menu when they are accessed.
Preconditions : The website (https://oportunitatisicariere.ro/) is up and running.

Step 1 : Open the URL .
Expected result : The homepage is displayed.

Step 2 : Verify the links from the main menu.
Expected results : The links are displayed and functional.

Step 3: Verify the loading time for the each link of the menu,
Expected result  : Every link loads its part under 3 seconds.


fix Mac CI

macos-latest is now referring to Mac M1, not x86_64, so we need to use an older version to build for x86_64.

it says it's still waiting for the spack_macos-latest check. maybe that needs to be commented out?

it says it's still waiting for the spack_macos-latest check. maybe that needs to be commented out?

Should be fixed now (the test was renamed to spack_macos-12)

looks like the parameter server test needs to be shortened

looks like the parameter server test needs to be shortened

Sure, can you do this in a separate PR?


GCI - New Ideas (check this out)

i already see someone give an a using google maps as an a idea, and then this idea came to my mind .... add some "Dengue Danger Meter"
example like this area , In future i believe that will be possible to make an a AI that can recognize dengue danger possibilities
https://www.google.co.id/maps/@-8.6041211,115.1080274,3a,75y,93.33h,66.72t/data=!3m6!1e1!3m4!1sy4BHKsM5ryO_BznrlbfoWw!2e0!7i13312!8i6656?hl=en
because i think google maps already have enough information about dengue problem by only using it
and add some "Dengue Symtomps Analyzer", so the user will know what to do and what is the best medication for  he/she
sorry for my bad english XD

I hope you will hear me out, but won't giving facilities for self-diagnosis lead to increased anxiety and hypochondria? Because due to the placebo effect, if people believe they have a disease strongly enough, they will invent symphtoms in their mind. Best not to overtax the already overtaxed medical infrastructure, eh?


Add config for ESLint and Prettier.

Subject to resolution of comments on https://github.com/googleapis/nodejs-storage/pull/13, which might change this slightly.

btw you don't need to work from a fork


Email field not being saved in Customer.io

Hey @karlclement - could you elaborate?

Hey! I'll close as this has gotten stale now. It's also possible that a recent refactor of the plugin might have fixed this for you.


Short solution needed: "How to find duplicates in data frame" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to find duplicates in data frame (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to find duplicates in data frame


Skip and Take throw Exception.

An exception is thrown when outside the bounds but that's a problem when using seq aiming for laziness, it will force you to check first the length of the sequence.
That's how they're implemented in F# but I prefer the Linq implementation which works as Haskell's drop and take, in those cases it returns an empty list which logically represents 'no results'.

The F# equivalent of Take from LINQ is called truncate that does not throw if the sequence has fewer elements than the specified count.

So, may be can we leave them as they are and add another 2 functions: drop and truncate with that (safe) behaviour.

I used the name Limit instead of Truncate because Truncate will conflict with the method of the same name for Numerics.


Get the list of subjects dynamically

When the user clicks on become a tutor or add more subjects then the values should come dynamically from the database table of subjects.

Solved #42


FreeBSD build fails Error: No statfs implementation

I've found a couple of failures while building on FreeBSD - if these can be fixed I can add avalanchemq to FreeBSD ports/packages. I will report it in crystal repo & link back to here.
> shards version
1.0.0-alpha.27

> crystal version
Crystal 0.36.1 ()

LLVM: 10.0.1
Default target: x86_64-portbld-freebsd13.0

> uname -a
FreeBSD wintermute 13.0-RC4 FreeBSD ... amd64

> gsh
commit a8c83ffc5d458d672758be35ec78a4752151332c (HEAD -> main, origin/main, origin/HEAD)

> shards build --release --production
Resolving dependencies
Fetching https://github.com/cloudamqp/amq-protocol.cr.git
Fetching https://github.com/tbrand/router.cr.git
Fetching https://github.com/luislavena/radix.git
Fetching https://github.com/schovi/baked_file_system.git
Fetching https://github.com/cloudamqp/amqp-client.cr.git
Fetching https://github.com/84codes/http-protection.git
Fetching https://github.com/84codes/systemd.cr.git
Fetching https://github.com/crystal-lang/logger.cr.git
Using radix (0.4.0)
Using router (0.2.7)
Installing baked_file_system (0.9.8 at fd6ba88)
Installing amq-protocol (1.0.0)
Installing amqp-client (1.0.0)
Installing http-protection (0.2.0 at 893a191)
Using systemd (1.1.2)
Installing logger (0.1.0)
Building: avalanchemq
Error target avalanchemq failed to compile:
Showing last frame. Use --error-trace for full trace.

In src/avalanchemq.cr:2:1

 2 | require "./stdlib/*"
     ^
Error: No statfs implementation

FreeBSD's statfs(2) is simlar to Darwin:
      *	filesystem statistics
      */

     #define MFSNAMELEN	     16		     /*	length of type name including null */
     #define MNAMELEN	     1024	     /*	size of	on/from	name bufs */
     #define STATFS_VERSION  0x20140518	     /*	current	version	number */

     struct statfs {
     uint32_t f_version;	     /*	structure version number */
     uint32_t f_type;		     /*	type of	filesystem */
     uint64_t f_flags;		     /*	copy of	mount exported flags */
     uint64_t f_bsize;		     /*	filesystem fragment size */
     uint64_t f_iosize;		     /*	optimal	transfer block size */
     uint64_t f_blocks;		     /*	total data blocks in filesystem	*/
     uint64_t f_bfree;		     /*	free blocks in filesystem */
     int64_t  f_bavail;		     /*	free blocks avail to non-superuser */
     uint64_t f_files;		     /*	total file nodes in filesystem */
     int64_t  f_ffree;		     /*	free nodes avail to non-superuser */
     uint64_t f_syncwrites;	     /*	count of sync writes since mount */
     uint64_t f_asyncwrites;	     /*	count of async writes since mount */
     uint64_t f_syncreads;	     /*	count of sync reads since mount	*/
     uint64_t f_asyncreads;	     /*	count of async reads since mount */
     uint64_t f_spare[10];	     /*	unused spare */
     uint32_t f_namemax;	     /*	maximum	filename length	*/
     uid_t     f_owner;		     /*	user that mounted the filesystem */
     fsid_t    f_fsid;		     /*	filesystem id */
     char      f_charspare[80];		 /* spare string space */
     char      f_fstypename[MFSNAMELEN]; /* filesystem type name */
     char      f_mntfromname[MNAMELEN];	 /* mounted filesystem */
     char      f_mntonname[MNAMELEN];	 /* directory on which mounted */
     };

A partial patch for src/stdlib/filesystem.cr looks like this, but I have no idea yet how the fields (with C comments still) map to crystal syntax/structures:
  {% elsif flag?(:freebsd) %}
    struct Statfs
     #define STATFS_VERSION  0x20140518	     /*	current	version	number */
      version : UInt32
      type : UInt32
      flags : UInt64
      bsize : UInt64
      iosize : UInt64
      blocks : UInt64
      bfree : UInt64
      bavail : Int64
      files : UInt64
      ffree : Int64
      ffree : UInt64
      syncwrites : UInt64
      asyncwrites : UInt64
      syncreads : UInt64
      asyncreads : UInt64
     uint64_t f_spare[10];	     /*	unused spare */
      namemax : UInet32
     uid_t     f_owner;		     /*	user that mounted the filesystem */
     fsid_t    f_fsid;		     /*	filesystem id */
     char      f_charspare[80];		 /* spare string space */
     char      f_fstypename[MFSNAMELEN]; /* filesystem type name */
     char      f_mntfromname[MNAMELEN];	 /* mounted filesystem */
     char      f_mntonname[MNAMELEN];	 /* directory on which mounted */
    end

There's a similar error for getrlimit getrlimit(2)
Error target avalanchemq failed to compile:
Showing last frame. Use --error-trace for full trace.

In src/stdlib/resource.cr:21:22

 21 | fun getrlimit(Int, Rlimit*) : Int
                         ^-----
Error: undefined constant Rlimit

which appears to be https://cgit.freebsd.org/src/tree/sys/sys/resource.h#n112
#define	RLIMIT_NOFILE	8		/* number of open files */
#define	RLIMIT_SBSIZE	9		/* maximum size of all socket buffers */

I'd be happy to collaborate on this if you're willing to help out with the crystal syntax etc.

Ok, thank you for your findings!
Made PR to crystal for RLimit: https://github.com/crystal-lang/crystal/pull/10569
But also monkey patched here: https://github.com/cloudamqp/avalanchemq/commit/6560de727cdb210e4523333a0577442268c0ae45
And added Statfs for bsd platforms too:
https://github.com/cloudamqp/avalanchemq/commit/d47e2e2ada38b4c8fb664b395f19ed617e8501dc

Would be awesome to see a GH action that built avalanchemq in freebsd, maybe with https://github.com/vmactions/freebsd-vm ?

Thanks! I will look into that, although perhaps using qemu/ubuntu instead of ... macos .. as a target!
In the interim, if you're willing to add a github post-commit webhook, I can at least build it locally here each time.

Thanks! I will look into that, although perhaps using qemu/ubuntu instead of ... macos .. as a target!

The reason for it using macOS, I believe, is that GitHub Actions doesn't allow nested virtualization for the Azure VMs: https://github.com/actions/virtual-environments/issues/183

There are other CI solutions we could use, e.g. https://cirrus-ci.org/features/ which have FreeBSD

> shards build --release --production
Dependencies are satisfied
Building: avalanchemq
Error target avalanchemq failed to compile:
ld: error: undefined symbol: statfs64
>>> referenced by filesystem.cr:61 (/repos/avalanchemq/src/stdlib/filesystem.cr:61)
>>>               _main.o:(~procProc(Nil)@src/avalanchemq/server.cr:46)
cc: error: linker command failed with exit code 1 (use -v to see invocation)
Error: execution of command failed with code: 1: `cc "${@}" -o /repos/avalanchemq/bin/avalanchemq  -rdynamic -lz `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libssl || printf %s '-lssl -lcrypto'` `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libcrypto || printf %s '-lcrypto'` -lpcre -lm -lgc-threaded -lpthread /usr/local/lib/crystal/ext/libcrystal.a -L/usr/local/lib -levent  -lpthread`

I'm not really clear what the issue here is - I'll take a closer look next week once I have some more familiarity with crystal under my belt. But I guess the monkey-patching isn't sufficient -- the Darwin & FreeBSD structs are not so similar.

THank you for the tip @dentarg , set up cirrus: https://cirrus-ci.com/github/cloudamqp/avalanchemq

Got it working, but now there's some OpenSSL issue with crystal in freebsd: https://cirrus-ci.com/task/6046301113548800
Should probably be raised with crystal-lang. Also notice that crystal 0.36.1 is installed on freebsd rather than 1.0.0, maybe it's an issue solved since 0.36.1 :shrug:

yeah we should bump that. you'll need pkg install -r FreeBSD security/openssl for the above issue, I think. Or crystal is not looking in /usr/lib for its crypto goodies. thanks!

Ah, cool, thanks! It's progressing! https://cirrus-ci.com/task/5046482723143680


Show progress notification when queueing experiments

Raising this as a draft as not sure this is needed (the queue command is fast enough now).
Demo
https://user-images.githubusercontent.com/37993418/235814901-4b60dee3-a245-4da3-9a56-8b5aac12af2c.mov

Doing something else.


Services' tasks should not get rescheduled if their placement constraints get updated but are still satisfied

Today, if you update a service's placement constraints, all of its tasks get restarted. If you update a service's placement constraints and all of its current tasks are already running on nodes that satisfy those placement constraints, there's no need for those tasks to get restarted, so Swarmkit should avoid restarting the tasks in that case. It should only restart tasks that need to be moved to other nodes as a result of the placement constraint updates.

Doing this might probably require some architectural changes. AFAIK, it is the scheduler which looks at constraints to make scheduling decisions, while the orchestrator decides if the service needs to be updated (and old tasks shut down). If we did the above, the orchestrator would have to avoid doing that in the case when only service constraints are updated, and let the scheduler decide how to proceed. We'd also have to think about other side effects of not shutting down old tasks.

Avoiding restart if its not really needed seems like the right thing to do and a reasonable change in behavior to me.

Swarm already knows not to restart tasks if e.g. only changing the replica count or changing service labels (not container labels, they will always cause task restarts). Changes only affecting service metadata, or constraint changes where a current placement is still valid, really shouldn't affect those existing placements.

When a service is updated, Updater checks all the tasks to determine which tasks need updating. If Updater can check scheduling constraints and update task's placement constraints as below, it would avoid unnecessary task restarting.
What do you think about the below code?
https://github.com/docker/swarmkit/blob/master/manager/orchestrator/update/updater.go#L523
func (u *Updater) isTaskDirty(t *api.Task) bool {
	constraints, _ := constraint.Parse(u.newService.Spec.Task.Placement.Constraints)
	u.store.Update(func(tx store.Tx) error {
		n := store.GetNode(tx, t.NodeID)
		if constraint.NodeMatches(constraints, n) {
			if t.Spec.Placement == nil {
				t.Spec.Placement = &api.Placement{}
			}
			t.Spec.Placement.Constraints = u.newService.Spec.Task.Placement.Constraints
			return store.UpdateTask(tx, t)
		}
		return nil
	})IsTaskDirty

	return orchestrator.(u.newService, t)
}

Fixed in #2490. Closing.

#2490 wasn't merged. Should this still be closed?

It should have said https://github.com/docker/swarmkit/pull/2496 actually.

Yep, #2496 is the correct one.


Allow Modules to Store and Serve Data outside of the JMT

Background
Currently, modules have access to #[state] variables. Under the hood, these variables are stored in the authenticated data store backed by the JMT.
Recently, the EVM and Optimistic rollup workstreams have both encountered the need to serve data via RPC which does not need to be authenticated. (For the EVM, this is receipt and log data; for ORUs, this is storage proofs). While we always can store extra data in state, it's significantly preferable to avoid the overhead in these cases. Instead, we should add a new field type to modules which is write-only within the state machine, but can be read in RPC calls.
Desired Outcome
Modules should be able to efficiently store and retrieve arbitrary auxiliary data using a non-provable data store. The store should be accessible for reading in RPC handlers but must not be readable during execution.
Potential Designs
Off the top of my head, I can see at least three possible designs:
Option 1: Add a new API for registering module-defined DBs
In this design, we create a new trait which allows modules to bring their own (non-provable) database and have it opened/initialized during node startup. The motivating example for this design is to allow the EVM module to use Reth's existing LMDB/MDBX database rather than attempting to port their storage model to RocksDB.
A very rough sketch of the design is like this:
// sov-modules-api/lib.rs
pub trait UnprovableStore: Clone {
  type Config: DeserializeOwned;
  fn open(Config) -> Self;
  fn put<S: Schema>(&self, S::Key, S::Value) -> Result<()>;
  #[cfg(feature = "native")]
  fn get<S: Schema>(&self, S::Key) -> Result<Option<S::Value>>;
}

#[derive(Default)]
pub struct DbRegistry {
  dbs: HashMap<Address, Box<dyn UnprovableStore>>
}

//my-modules/lib.rs
/// A new initailization function to create the module store. This is invoked
/// during node startup
fn init(config: Config) -> Option<UnprovableStore>;

//runtime.rs
/// A new (macro-generated?) function to initalize all module dbs.
fn init(config: Config) ->  {
  let db_registry = Default::default();
  if let Some(db) = MyModule::init(config.my_module) {
    registry.add(MyModules::new().address(), db)
  }
}

//scratchpad.rs
pub struct WorkingSet<S> {
  // ...
  module_dbs: DbRegistry,
}

impl<S> WorkingSet<S> {
  pub fn get_unprovable(...);
  pub fn put_unprovable(...);
}

Option 2: Allow modules to define and write to their own RocksDB tables
In this design, we don't allow modules to bring their own DBs, but we do allow them to create and write to custom tables in our current DB. This will require us to separate our current SchemaDB crate into at least two pieces, and expose the schema definition traits/macros from sov-modules-api. We will also have to modify our DB intialization code...

Generate a list of module-defined column families from the runtime
Open all of these DBs on startup

Option 3: Allow modules to read/write to their own prefix in a single (predefined) RocksDB table
This is the least flexible option, but the simplest to implement. We can simply create a new database schema and add a couple of methods to the working set.

cc @bkolad

The functionality has to also supportWrokingSet reverts. In my opinion, option 2 does not bring a lot of advantage over 3, 1 would be great, but it brings some complexity, and it looks like all our use cases can be achieved with 3. Let's go with 3?

I would say that if possible it would be natural to use a column family here. We probably don't need 1 right now.

We should ensure that the DB writes are atomic across the JMT and non-JMT, so the data remains consistent even if the rollup shuts down in the middle of the db-write.


[Feature] Support Iceberg JDBC Catalog

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@stephen-shelby  Can you help me review this PR?
Some of code are copied from IcebergHadoopCatalog. I think it's hard to lower the duplication rate unless we refactor all IcebergCatalog code.