Support custom indicator outside of SliderView frame

As title, I want to display the indicator outside of SliderView frame, but it's fixed on xml layout of lib.
Could you support the custom indicator feature like this library:
https://github.com/daimajia/AndroidImageSlider/blob/e318cabdef668de985efdcc45ca304e2ac6f58b5/library/src/main/java/com/daimajia/slider/library/SliderLayout.java#L226
If it's done, this lib is perfect! ^_^ Please support this!

Please give an update on this. It's a really important feature to add.

I just work using PageIndicatorView in xml and disable indicator on sliderview.
then, combine them by :
slider.selectedColor = ContextCompat.getColor(requireActivity(), R.color.color_primary)
slider.unselectedColor = ContextCompat.getColor(requireActivity(), R.color.color_gray)
slider.setPageIndicatorView(slider_indicator)

I think this issue should be closed.


bump: tag and release v1.0.0

Release

[!IMPORTANT]
This PR is required to be merged only after completing the checklist #749.

This would mean tagging 80e3fc4e2eeb43ac00bc888cf41101f5c56f1535 as v1.0.0 to release.
Vote
We need at least 4 approvals from 6 maintainers to release notation v1.0.0.

[ ] Junjie Gao (@JeyJeyGao)
[ ] Milind Gokarn (@gokarnm)
[ ] Patrick Zheng (@Two-Hearts)
[ ] Pritesh Bandi (@priteshbandi)
[ ] Rakesh Gariganti (@rgnote)
[x] Shiwei Zhang (@shizhMSFT)

Changes
The code changes compared to v1.0.0-rc.7 include:

#692
#702
#703
#710
#711
#712
#594
#717
#716
#727
#724
#731
#730
#735
#739
#734
#737
#745
#742
#746
#744
#752
#751
#754
#750

Full Changelog: https://github.com/notaryproject/notation/compare/v1.0.0-rc.7...80e3fc4e2eeb43ac00bc888cf41101f5c56f1535
Action Requested
Please respond LGTM (approve) or REJECT (request for changes).

Should we do a dev build before 1.0 to do a round of sanity test?

Should we do a dev build before 1.0 to do a round of sanity test?

We have the build on the main branch passing. Meanwhile, we have released v1.0.0-rc.7 that the release pipeline has no issue. Thus, I think we are fine to not have a dev build.
If you have concerns, we can enable the weekly dev release workflow and manually trigger the workflow to publish a dev release.


Added my data

Please add your data above contributors.unshift @rutuja369

Good job @rutuja369
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.
Note that this repo is excluded from Hacktoberfest but this PR is still eligible for ClueLess Swags.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


H/V Selection Improvement

For features such as valves that allow users to choose their orientation, the current checkbox system isn't ideal and sometimes you have to click twice to get the orientation to change. Instead, if there were two buttons (one for H, on for V) that users can click on to choose orientation that would be a bit simpler to use.

Changing the orientation is now more consistent


Get datatable options from config file

Currently
Datatable options have to be customized by overriding the bread.browse view
Solution
Allow options to be loaded from voyager.dashboard.data_tables config entry
Plus fix responsive option not working since config('dashboard.data_tables.responsive') does not exist.

I dig it 👍
Thanks!

where should i add the relevent Json  for Export EXCEL and PDF buttons?

@corexz, again, please don't comment on unrelated issues/PRs that were closed/merged months ago.  If you have an issue, just open your own

so , you might consider opening a forum for new commers!!

@corexz, we have one.  It's a slack group, and is mentioned in the project README

If its of any help here is my config config/voyager.php for the export buttons:
'dashboard' => [
        'data_tables' => [
            'responsive' => true, // Use responsive extension for jQuery dataTables that are not server-side paginated
            'stateSave' => true,
            'columnDefs' => [
                ['responsivePriority' => 1, 'targets' => 0 ],
                ['responsivePriority' => 2, 'targets' => -1 ],
            ],
            'dom' => 'lfrtipB',
            'buttons' => [
                'copyHtml5',
                [
                    'extend' => 'print',
                    'exportOptions' => [
                        'columns' => "thead th:not(.actions,.noprint)"
                    ]
                ],
                'csvHtml5',
                'excelHtml5',
            ]
        ],

    'additional_css' => [
        //jQuery Datatable plugins
        'vendor/tcg/voyager/assets/lib/css/responsive.dataTables.min.css',
        'vendor/tcg/voyager/assets/lib/css/buttons.dataTables.min.css',
        'vendor/tcg/voyager/assets/js/datetimepicker/bootstrap-datetimepicker.min.css',
    ],

    'additional_js' => [
        //jQuery Datatable plugins
        'vendor/tcg/voyager/assets/lib/js/dataTables.responsive.min.js',
        'vendor/tcg/voyager/assets/lib/js/dataTables.buttons.min.js',
        'vendor/tcg/voyager/assets/lib/js/dataTables.buttons.html5.min.js',
        'vendor/tcg/voyager/assets/lib/js/dataTables.buttons.print.min.js',
        'vendor/tcg/voyager/assets/lib/js/jszip.min.js',
    ],


Data from warm up period updates indicator after warm up

Expected Behavior

The indicator should only be updated once in the warm-up period with the consolidator.
Actual Behavior

The consolidator generates consolidated bars in the warm-up period twice so the indicator was updated with the previous price data which throw the error This is a forward only indicator: SMA10 Input: 2013-10-01 00:00:00Z Previous: 2013-10-22 00:00:00Z.
The issue might relate to #2325.
Reproducing the Problem

class RenkoConsolidatorAlgorithm(QCAlgorithm):
    def Initialize(self):

        self.SetStartDate(2013, 11, 1)
        self.SetEndDate(2013, 11, 10)
        self.AddEquity("SPY", Resolution.Daily)
        renkoConsolidator = RenkoConsolidator(3)
        self.sma = SimpleMovingAverage(10)
        self.RegisterIndicator("SPY", self.sma, renkoConsolidator)
        renkoConsolidator.DataConsolidated += self.HandleRenko
        self.SubscriptionManager.AddConsolidator("SPY", renkoConsolidator)
        self.SetWarmUp(timedelta(days=30))

    def HandleRenko(self, sender, data):
        self.Debug("BarStart "+str(data.Time) + " BarEnd "+str(data.EndTime)+" renko price "+ str(data.Close))

System Information

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[ ] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Closed by https://github.com/QuantConnect/Lean/pull/2748


Seasons that aren't available are marked as partially available when synced with Jellyfin

Description
On every show I have with multiple seasons with only some downloaded, Jellyseerr incorrectly marks the missing seasons as "partially available" which prevents anyone from requesting them. I have noticed this happens only when doing a library scan with Jellyfin.

Without doing a library scan with Jellyfin and instead first setting up Sonarr with the "Enable scan" feature and running a Sonarr scan in Jobs & Cache, it correctly detects the undownloaded seasons as "Not requested". This can also be replicated by clearing data for the particular show and running a Sonarr scan - though it once again regresses to "partially available" once the Jellyfin library scan runs.

Version
1.10
Steps to Reproduce

Have one or only some seasons of a show available and downloaded
Run Jellyfin library scan

Screenshots
How the show looks like in Sonarr (as you can see, only one season is downloaded):

And in Jellyfin, where you can see only the same one season is available:

Logs
No response
Platform
desktop
Device
PC
Operating System
Linux
Browser
Firefox
Additional Context
No response
Code of Conduct

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

Hm. That is odd. Does that happen when you have like let's say season 4 without 1-3?
Because I also have several series w half downloaded but that hasn't happened to me

(Trying to replicate this error to figure it out)

I just tried the other way too. Having season 3 without season 1-2 for example and i could not replicate the error.

Could you try recreating jellyseerr and checking if the same thing happens? Or try the develop tag?

My feeling is that it is caused by jellyfin keeping some metadata about deleted seasons/episodes, however the jellyfin web ui can correctly determine so and wont show anything in the UI. On the other hand, jellyseerr does not handle this case and thus shows partially available / available status.

This what I have been thinking as well. There might be metadata leftover of some deleted seasons or episodes or like metadata there w no media. Jellyfin won't pick it up but jellyseerr would as it would look for the metadata instead of the video file

Shouldn't be this marked as bug then? Jellyseer needs to do some validation or something else, because even 3rd party jellyfin clients don't display the series which jellyseerr sees as (partially) available - suggesting that this can be determined by the Jellyfin API.

Yes I have the same issued and I was wondering what it is about. Is there any solution to this?

One temporary solution I could think of is, deleting any leftover metadata from jellyfin and then removing the data of that show from jellyseerr and rescanning. That was jellyseerr will only see the season that is available and everything else will be seen as not requested


Question: Is there a way to render to a picture?

I want to render a static part of my image to a back-buffer image. Is there a feature that allows to render off-screen to a bitmap?

The FBO example shows you how to render to texture. Is that what you're looking for?
https://github.com/memononen/nanovg/blob/master/example/example_fbo.c

Yes, that probably is. Thank you.


add timeout option to core.system() (SOFTWARE-646)

In core, add a per-command timeout option to __run_command(), system(),
and check_system().
This will remain unused by default; you have to explicitly pass
timeout=NN[.NN] to core.system() or core.check_system() in your tests if
you want to use it.
a TERM signal is used by default to kill the process group of
subprocesses that exceed their timeout.  Use timeout_signal='KILL', etc,
if some other signal is desired.

After learning more about fork/exec/waitpid, I'm mostly satisfied with the changes but I do have questions:

If I'm reading this correctly, check_system will fail on timeouts but will not differentiate between timeout and other failures. I think we should probably edit the failure message in the case of timeouts (and maybe create a specific timeout type of test failure status i.e. badskip?)
Have you run a timeout test in osg-test to make sure the TestRunner handles everything properly?

Ah!  You're right; while the status returned by core.system will be -1 on timeout, check_system doesn't do anything specific with it -- the failure message will include EXIT STATUS: -1, but (I'm guessing) it would be clearer if the failure message specifically said it timed out.  (We can add this in core.diagnose by checking if status == -1.)
(So, to repeat: yes, currently the only way for the caller of check_system to detect a timeout would be to see if the failure message matches EXIT STATUS: -1.)
But, currently the only way check_system communicates a failed check to the caller is by the failed assert throwing an AssertionError, right?  So as for the caller detecting that check_system had a timeout, I suppose we could have check_system throw a new exception type (TimeoutError or something).  I'm open to suggestions.
I haven't tested adding timeouts to any tests yet, no :wink:  ...I can get back to you on that...

We could create a new TimeoutError exception based on AssertionError (see osgunittest.py) so that it plays nicely with the unittest framework. This would also mean updates to the test framework to accommodate the new test result state in the reporting. Alternatively, we can just treat them as failures and check if status == -1 and prepend the user's message with TIMEOUT or some such similar text.
Opinions @osg-cat, @matyasselmeci ?

I <3 clarity, which I think a separate exception class achieves. We can always choose to treat that new exception in the same way as others.

I now have the timeout watcher process call _exit instead of regular exit or get killed with SIGKILL to avoid running any unittest exit handlers - @brianhlin, hopefully this will fix the double-output issue you were seeing...

Test results look good (http://vdt.cs.wisc.edu/tests/20160601-1442/results.html). Manual tests (adding a command that timed out) failed properly.


Refactor codebase to async/await - m365 aad o365group recyclebinitem remove until m365 aad o365group set

Closes #4942

Thank you @MathijsVerbeeck, we'll try to review it ASAP!

Hi @milanholemans
Thanks for the review. Please have another look!

Merged manually, thank you!


Implement The Law Goes Underground

Sorry it takes so long, will review after the WWE update.

Sorry it takes so long, will review after the WWE update.

No rush! WWE is clearly more important :).
Sounds like we're mostly good on implementing the new cards, but if there are any that need scripting happy to try to help out. I'm back at work after the holidays, but I can hopefully fit in some time here and there.


chore: avoid circular type dependencies

[x] move Library type from @prisma/engine-core to @prisma/internals
[x] move plusX utility dependency in @prisma/internals from @prisma/engine-core to @prisma/fetch-engine

Let's remove the dependency out of the package.json now :)

Let's recall we still have this, @millsp:
// https://github.com/prisma/prisma/blob/main/packages/internals/src/get-generators/getGenerators.ts#L2

import { fixBinaryTargets, getOriginalBinaryTargetsValue, printGeneratorConfig } from '@prisma/engine-core'

// ...

Let's recall we still have this, @millsp:

Nice catch, thanks. What should we do with this then? Do you think it belongs in internals?

Let's recall we still have this, @millsp:

Nice catch, thanks. What should we do with this then? Do you think it belongs in internals?

Hard to move things considering the current state of things.
Currently, fixBinaryTargets and printGeneratorConfig is used in a duplicated getFixedGenerator() method defined twice in @prisma/engine-core:
  // TODO Fixed as in "not broken" or fixed as in "written down"? If any of these, why and how and where?
  private getFixedGenerator(): string {
    const fixedGenerator = {
      ...this.config.generator!,
      binaryTargets: fixBinaryTargets(this.config.generator!.binaryTargets, this.platform!),
    }

    return printGeneratorConfig(fixedGenerator)
  }

This getFixedGenerator() is used once in BinaryEngine.ts and once in DefaultLibraryLoader.ts in a similar fashion, which are again in @prisma/engine-core.

// https://github.com/prisma/prisma/blob/main/packages/internals/src/get-generators/getGenerators.ts#L2

import { fixBinaryTargets, getOriginalBinaryTargetsValue, printGeneratorConfig } from '@prisma/engine-core'

Current problem about this line:

we initially thought we could potentially move get-generators to @prisma/client, then we realized @prisma/migrate uses get-generators as well, and as @prisma/cli depends on both @prisma/client and @prisma/migrate, we would simply move the circular dependency problem without solving it.

Sounds reasonable, client generation isn't a migration concern, after all. What this the benefit of passing tryRunGenerate as an argument to migration commands vs doing it it in the cli after just executing the migration command?

Sounds reasonable. Client generation isn't a migration concern, after all. What this the benefit of passing tryRunGenerate as an argument to migration commands vs doing it it in the cli after just executing the migration command?

There are migrate commands that invoke tryRunGenerate before the $Command:parse() method has completed its execution. One such example is prisma migrate dev, as seen in here.

As of this commit:

get-generators is now in @prisma/client/generator-build
@prisma/internals no longer depends on @prisma/engine-core
@prisma/migrate no longer depends on get-generators

Please take this commit as an evaluation of whether it made sense to move all these files around to achieve the scope of avoiding circular dependencies, and to highlight any file that shouldn't have been moved (if any).
Note:

@prisma/studio is looking for get-generators in @prisma/internals, thus building the cli won't work
I haven't moved around tests yet, that'll come later

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


fix(apiDocs): Fix typo in ProductsDocs

There is a typo in patchProductById, which leads to wrongly named hook generated for the Server UI, and this PR fixes the typo.

LGTM. Thanks @Etsija.


[BUG] com.sun.jna.platform.win32.Win32Exception: 占位程序接收到错误数据。when the WorkerServer work in the Windows

When I'm running the workerServer in Windows 10 System，I schedule a simple program like
dir

then the task run fail，and get some exception

Hello, workerServer recommends deploying linux or mac os. Currently, the overall deployment of ds does not support windows. Debugging under windows may encounter many problems.


basic support for react 15

Using these 2 changes currently in my codebase to use react templates with react 15.0.1 and it works fine

Thanks for the PR! Merged.


Generate helpers for begin_/end_ call pairs

Fixes #8
Thanks @mattrberry for the idea and collaboration!

I haven't finished the part about converting the demos. But all the main code generation is finished. See obj.cr.
Feedback about naming is still accepted for sure :)

Ah and also something I briefly thought about: should the pops perhaps be in an ensure block? But probably not, it's likely to break ImGui even more.

And: should the return value be propagated from the yield? Currently these have no return value.

Ah and also something I briefly thought about: should the pops perhaps be in an ensure block? But probably not, it's likely to break ImGui even more, if something is already broken.

Hmm that's just a wonky state overall. I think either option is fine as long as it's documented. I may lean slightly towards using an ensure block because my understanding of these blocks is that they'll remove any place the user needs to push/pop or begin/end. However, I'd still re-raise the exception because it's still up to the user to handle the error beyond that point. Just my initial thought

And: should the return value be propagated from the yield? Currently these have no return value.

I don't see a huge value in this since the user knows the condition is true whenever the block is entered. That said, I don't see any harm in yielding the value, so maybe it'd be good to do it just in case it's helpful in the way the user wants to write their code?

I am strongly considering removing the prefix from methods such as within_menu_bar → menu_bar. There's just the problem that within would be renamed to nothing according to that scheme sweat_smile What does begin mean - begin a window? Should we name this window then? Or what do you think?

Assuming that's the decision, I agree with begin → window since that's how it's described in imgui. My only thought about dropping the within, though, is that the name may not be unique. This holds true for the combo case, which is now another method overload. I think I actually still prefer it with the within dropped, but just something to think about.
If we're dropping the within, what do you think about the with on the push/pop blocks? I think I favor having it since it's just a little more explicit, but I'm curious what you think

Yes. Well, it's all finished now. Please give it a try, maybe check out how the new functions and the demo code look.

Just gave it a try with my own code. I like it! :shipit:


Default periodic process pruning via heuristics to false

Description
This optimization is very expensive and based on some analysis, it is not particularly useful in many environments. Default it to off, but have a way to turn it back on if needed
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI

/retest

😢


Shells/PIDs commands to Python and Java isolates

Description
When issuing shells or pids command in Pelix shell, while Java and Python isolates coexist in the same application, the call to the Java isolate would fail with message "These isolates didn't respond", while somewhere from none to all of the Python isolates won't reply at all. The number of Python isolates to answer appears to be indeterministic, however, for the same run the same Python isolates reply for each calls to shells and/or pids.
A call to shells or pids with a Python isolate name as an argument would succeed, even if that isolate doesn't show for the same call without arguments.
Reproduction
Create a Cohorte note with 3 different python isolate, and one java isolate. From within Pelix shell issue: shells.
Environment
Mac OS X 10.10.4 (14E46).
Python 3.4.3.
Java 1.8.0_51.

In python/cohorte/shell/agent.py method ShellAgentCommands.pids() once a failure occurs (on_error() is invoked) no more "success" occurs (on_success() is not called any more)!


Fix strict option in deserializer to support recursive calls

Current strict options in NpzDeserializer and HDF5Deserializer work well when the link does not have a hierarchical structure, however, it automatically changes strict option to True after the 2nd level even if users explicitly specify False. This PR aims to fix this problem.

Can you write test?

@okuta I added several tests. Can you check them?

LGTM!


Add a CLI command to clearly identify why 2 Gradle daemons are incompatible

Local devs can hit the following issue:

Starting a Gradle Daemon, 1 incompatible and 1 stopped Daemons could not be reused, use --status for details

$ gradlew --status
   PID STATUS   INFO
 44496 IDLE     8.0
 18376 IDLE     8.0

I can dig through daemon logs, and manually compare and contrast the array of startup values for each daemon. However, this is  not the best developer experience.
I would like a gradle CLI command that will allow us to clearly identify why two daemons are incompatible. Maybe something like:
./gradlew --status 44496 18376

And it could output information that shows the share values for each daemon, and the values that are different for each daemon.
Bonus, additional explanation could be provided, depending on the value as well.

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team but this area of Gradle is currently not a focus one so it might take a while before it gets implemented.


fix(cli): add back in resolveTableId export for use with mudConfig

Removed in #517 but didn't realize we needed this in https://github.com/latticexyz/v2sandbox/pull/7

Confirmed locally that this works without issue (the exported module doesn't use any node deps), so gonna self-approve and merge so I can unblock some follow up work


Added link towards Coriolis RTL2GDS toolchain.

Dear Mr. Olofsson,
I recently came across your very interesting list of softwares, and I think the Coriolis toolchain may well fit into it.
Best regards,

Please change link to point to official source repo.

Done. I do hope that it is on a gitlab and not on github is not a problem.

Thank you!


[MRG] Remove print statement and unused import

Remove a stray print statement and an unused import in the zenodo content provider.

ignoring test failure, which is due to external factors


css: Fix dropdown text alignment centered.

The issue was extra padding on a fixed-height box.
Screenshots and screen captures:
Before:

After:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Would be nice to check that it works in other modals too since this same CSS used for other modals. Also, I was able to reproduce the issue only on 110% zoom on my system, so would be good to check at couple of zoom levels as well.

For some reason I'm getting errors when I change the time zone, which I'm not seeing on main:

@alya That is strange because the PR does not touch any file related to changing timezone functionality. Still, I tried to change the timezone and it was a success on my end.

For some reason I'm getting errors when I change the time zone, which I'm not seeing on main:

I have got this a couple of times as well but have not been able to reproduce consistently. This is probably because we call the function to update bot data after recieving realm_user/update event for non-bot users. Will look into this.

OK, got it. Testing the PR again, it looks all right to me.
@timabbott This has been reviewed by @sahil839  and me -- please take a look. The errors above are unrelated.

Hmm, why is the right fix for this to remove the fixed height rather than adjusting the 4px/6x padding?

@timabbott By reducing padding we make the box a little shorter from its neighbors. Also, 4px/6px is being used as padding in a lot of boxes. Removing height makes its properties similar to its neighboring boxes.

But I think all of our other select elements are 30px in height, and most of them don't have this visual issue. I don't understand. @sahil839 perhaps you can explain?

The issue was present in particular modals only due to font-size being large for modals and only in some select elements with letters like g used in the option. And in my case, I was only able to reproduce the issue only when I was at 110% zoom and not at other values.
And I think the issue is essentially as we are setting height, because changing height to 30px just cuts the element from bottomand thus resulting in text appearing not aligned and more towards bottom.
I am also not sure what is the best solution here. The argument to change the height was to have consistenct with other inputs. But on noticing closely, the text inputs and select elements height still are not same after removing the 30px height.

Maybe we should move this discussion to a #frontend thread? I think it'd be helpful to have others help resolve how we want to resolve this issue.

@timabbott Just a reminder since it has been quite some time.

Can you add a link to the #frontend discussion on this, and a summary of its conclusions? I see nobody linked it in the discussion above, and I'd like confirmation that this is what we decided to do and what testing was done.

CZO

@timabbott Mentioning you for a quick re-review.

I've checked this out (in rebased form) across multiple viewports and zoom levels, and it looks good to me.

I changed the 2px to 0 in this PR and have marked this to merge once CI passes.
(Looking closely, I'd noticed that in a few dropdowns, even the 2px seems to adjust the positioning a bit off center, and we're already using mechanisms like vertical-align: center to position the labels inside the dropdowns, so I don't see what the purpose of that padding would have been)


"Mark records as duplicate" table has too much space

To see this page, create two person records with a common family name, search for the family name, click "duplicat marking mode", check both of them, and click "Mark records as duplicate".

@gimite
I couldn't view the screenshot. On clicking the link, I was asked enter my google credentials. On entering my gmail ID, it doesn't proceed further. Maybe the sreenshot reside in Google's private servers.
Let me know if I can work on this.
BR,
Viswa

Yeah sorry it was our internal page. I replaced it with an externally visible one.

It would be great if you work on it.

Hi,
I had created a pull request containing the fix for this issue. It is here : https://github.com/google/personfinder/pull/160 .
Please review & accept the merge, if deemed fit.

As I mentioned in #160, it has actually been fixed by #146. Sorry for miscommunication.


Prefix all pkgs with 'khi_'

This PR converts all of packages to khi_* for #4
It changes only package(directory) names.

I rebased on master to solve a conflict.
@matsui-hiro I think master currently has a problem about https://github.com/Kawasaki-Robotics/khi_robot/pull/8#issuecomment-479585050
Please consider to fix it.

@d-nakamichi wrote:

It changes only package(directory) names.

the PR seems to change more than just the directory names, correct?
I would also expect there to be more changes needed.

@gavanderhoorn

the PR seems to change more than just the directory names, correct?

Is that means it changes codes of package references like this, right?
It changes package(directory) names and those referenced codes. may be correct.

I would also expect there to be more changes needed.

I've already check it OK to build and execute moveit demo, although I'm concerned about https://github.com/Kawasaki-Robotics/khi_robot/pull/8#issuecomment-479585050 (this is about urdf/srdf robot name)
What the expected changes, for example?

All references to the old package names would need to be replaced with the new names.
For MoveIt configurations, it may be easier to just recreate them (as they're mostly auto-generated anyway by the Setup Assistant).
For the rest of the packages: if you've compile tested this (and Travis has done that as well) and tested the launch files things should be fine.
We might want to add more roslaunch tests to automatically test whether roslaunch can still find all the packages, just as I did in #5.

OK. I changed launch files in khi_*_moveit_config/ based on MoveIt! Setup Assistant.
I also added test files which launch moveit applications.
Thank you.


How to use

can you add a readme or some form of documentation on how to use this, I can get the mesh to obj, but how do i get obj back to apti_msh?

@smasher248
I have no experience with models
About how to use it after export dpc file use noesis to preview


osx-arm64 package on conda-forge (for apple silicon)?

Dear SH team
Could you please create a (set of) eo-learn builds for Apple Silicon (m1 / osx-arm64) on conda-forge, please? https://anaconda.org/conda-forge/eo-learn-geometry doesn't seem to have it, so it can't be installed. See https://anaconda.org/conda-forge/rasterio for an example that works :)
Thank you!

(meanwhile I'll try with pip-within-conda and report here if it works)

Uh oh. Yup, pip install within conda not working nicely for eo-learn on m1. It seems pip install eo-learn is trying to build wheels  for some if its dependencines, in particular for  numpy, and that's error-ing :/

Hi Julien,
Unfortunately, the conda-forge eo-learn recipes are not maintained by us, so you will have to open an issue there (e.g. https://github.com/conda-forge/eo-learn-io-feedstock)
Can you use the eo-learn docker image?

Hi @batic ! Docker image doesn't quite cut it, sadly. Are you familiar with compilation of the feedstock? Happy to work with you on bringing it up to date -- could be a good thing for EO-learn to own its own packaging, too, distribution-wise :)

Had this issue too on a new machine, problems are related to scipy, which requires numba. It seems like the numba issue is nearing a resolution: https://github.com/numba/numba/issues/7822

We recently tried to install eo-learn and managed to install it successfully with the following steps (updating conda seems the key step).

Download Anaconda from here
Install Anaconda (double-click on Anaconda3-2021.11-MacISX-x86_64.pkg)
once installed, check installation with conda list
update conda with conda update conda
create new environment with conda create -n test-eo-learn python=3.9
activate env conda activate test-eo-learn
add conda-forge conda config --add channels conda-forge and conda config --set channel_priority strict
installation of single packages like conda install eo-learn-core or conda install eo-learn-io version v0.10.1 works
installation of eo-learn from source as

clone from github git clone https://github.com/sentinel-hub/eo-learn.git
change directory to eo-learn
install in development mode as python install_all.py -e
this ran with no errors and we could import tasks from features from latest v1.0.0

Let us know if this works for you.

IIUC, we have recently gained control over the conda-forge, but not sure what was done in relation to fixing issues mentioned in this post.
In any case, closing this since there was no activity. Feel free to reopen if it's still relevant.

IIUC, we have recently gained control over the conda-forge, but not sure what was done in relation to fixing issues mentioned in this post.
In any case, closing this since there was no activity. Feel free to reopen if it's still relevant.

we're still not in a position to solve the conda issue, hopefully we can do more once we switch to a single-package installation


stopped working entirely on my pc

running 2.6
have it running 24/7
has been great all year, never any issue of any kind
yesterday at some point it just stopped.
its there, says no updates available, its not doing anything.
my feeds have results constantly all day long.
it all suddenly stopped yesterday.
no recent changes of any kind before it stopped.
no other pc issues and no changes to my pc of any kind recently.
same IP, same everything

Same problem here


Insert/remove range with bounds

Implementation of insert_range and remove_range functions that accept the RangeBounds trait instead of a Range struct.
This PR fix #5 and #113.
Done:

[x] Switch inner parameters to RangeBounds

[x] insert_range
[x] remove_range

[x] Fix boundary condition in insert_range/remove_range
[x] Add more test cases
[x] switching to RangeInclude on the Container and Store

[x] insert_range
[x] remove_range

Hi @Kerollmops
Thanks for your reviewing!
Recently I might have no time to code since I'm on my way to travel 👻. But I would come back ASAP.


Pom cleaning

Hi!
That PR related to the pom cleaning and bump version some libraries.
Thanks
Cheers,
Gonchik

What's the status here? Should this be merged?


Update project to use new EUS urls

This is not in a state to merge right now, some other changes need to be made first before this should be merged.

@littlemisssynth Should be ready to review now.


fix of gql operationName attribute name

@kvakos Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

trying to fix #741 by this PR

Thanks for your contribution @kvakos but I think #742 is already fixing that.


allow ancestral to be used for zulrah and other mage bosses

ancestral should be able to be used and cover requirements for bosses like zulrah instead of requiring me to have ahrims.

this have been resolved.

this have been resolved.


Adding EDIX

Hi!!!
I'm trying to add my university https://www.edix.com/
The mail domain that they are providing to studens is: @soy.edix.com

@elogin91 This organization does not seem to be a school at all. The homepage redirects to an HR agency website. I have to close the request because the organization's domain is not eligible for this repository.

It's not an HR agency, if you scroll down the website you can see all the formation that they have:


RTN20a

As-is, I don't think this test is providing anything. I'd rather leave this functionality out for now, and just have it marked in the spec spreadsheet instead of having this test lingering around.

Actually, scratch that, I doesn't seem hard to just do it now.


Reversing Entries

Reversing Entries are for much more than corrections.
Reversing Entries are routinely used for accrual entries, which have to be subsequently 'removed', in the sense of entirely offset, so that the ultimate entry can be entered without double entering amounts or calculating and entering only the differences between the original entry and the final entry.
Jerald Savin, CPA

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

ID: 6c0c4f82-94bf-ee44-9f15-15cab3277de1
Version Independent ID: b7e8a4e6-826f-c1d7-54ba-c2ef370f3c0d
Content: Undo a Posting by Posting a Reversing Entry - Business Central
Content Source: business-central/finance-how-reverse-journal-posting.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: edupont

Hello Jerald,
Thank you for your feedback and insights! We've added your feedback to our backlog and will use it to update this article later this year.

Now live.


Design: Allow script/update etc. to run when context is not in target project, but can be found in services

Listing DbContext's will return contexts that are registered in services, even if they reside in another project. Some commands (such as adding a migration) require the target project, since they generate code files, but others only need a context instance and could run by getting the context from services.
Perhaps we could also consider making this work if there is an IDbContextFactory in the target project.
This will help ASP.NET Core publishing.
cc @vijayrkn @mlorbetske

@Eilon And this one is "patch proposed" as of  this afternoon. Guess we have some conversions to have. 😺

@bricelam Just from reading this it isn't clear why this should go into the patch. Can you explain some more?

This was an ask from the Scaffolding team. Issue aspnet/Tooling#898 describes the end-to-end scenario.

This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch.


Chore/build

[x] 使用最新的 @mini-types/alipay，解决mixin ts报错问题，包括 value，及createForm mixin问题
[x] 修改 mini.project.json 修改，包括去掉 typescript transpileOnly
[ ] 修改typescript 错误

#583

Codecov Report
Base: 11.41% // Head: 11.67% // Increases project coverage by +0.25% :tada:

Coverage data is based on head (90d99b9) compared to base (4832178).
Patch coverage: 16.50% of modified lines in pull request are covered.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Failed to execute script deid-chrome

Hello,
Thank you for your great work !
I downloaded svs-deidentifier-windows-chrome-onefile.zip, unzipped it and ran it.
Two boxes opened :

A chrome box : "Error : 404 Not found"
A box Fatal error detected : "Failed to execute script deid-chrome"
My computer is a Windows 64
I am using Python 3.7
All packages have been installed like described here : https://github.com/pearcetm/svs-deidentifier/blob/master/wsideidentifier.py
Both chrome and Edge are installed.
I have the feeling that I am missing something, but I do not know what...
Please, could you let me know if you have any idea about the way to solve this issue ?

Kindly yours.

I haven't see an error like you're describing before - but I also haven't tested on a lot of different Windows operating systems. When you say "Windows 64" which OS are you actually using?
A couple of questions and things to try:

Does "one-dir" mode work?
If you launch from the command line are there any other error messages that give information about where the problem is?
If you build the project on your computer, does that work? You can try in either one-file or one-dir mode.
Have you tried it on other operating systems and/or computers?
Eel uses port 8000 by default. Is another local server already running on that port? Is that port allowed by your firewall/security settings?

Dear @pearcetm thank you for your answer
When you say "Windows 64" which OS are you actually using? => Windows 10 family 64x
Does "one-dir" mode work? => **Indeed it worked !**
If you launch from the command line are there any other error messages that give information about where the problem is? => apparently it was a problem of Eel. I I had module not found error although it was actually installed...
If you build the project on your computer, does that work? You can try in either one-file or one-dir mode. No, still eel module error
Have you tried it on other operating systems and/or computers? No, but I should definitely try
Eel uses port 8000 by default. Is another local server already running on that port? Is that port allowed by your firewall/security settings? I guess not

By the way, thank you very much for your help !
Kindly !


ADR 012: Handlers validation and execution separation

Closes: informalsystems/ibc-rs-tmp#10
Rendered view

PR author checklist:

[x] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[x] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).
[ ] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Shall we move this to cosmos/ibc-rs?

Moved to https://github.com/cosmos/ibc-rs/pull/153


Extract function doesn't preserve semantic with return statements

function foo(i: number) {
	let x = 10;
	for (let i = 0; i < 10; i++) {
		x++;
		return;
	}
}

select

		x++;
		return;

extract
you get

function foo(i: number) {
	let x = 10;
	for (let i = 0; i < 10; i++) {
        x = newFunction(x);
	}
}
function newFunction(x: number) {
    x++;
    return x;
}

which is not sematic preserving.

It only breaks when extracting into the global scope.  I'm guessing it's failing to handle the combination of a write and a void return (e.g. it works if the original has return 1).


add missing each in Modelica.Media.Examples.TestOnly.MixIdealGasAir

Because medium2.X is a vector modification of fixed attribute needs each prefix.

You could run the following commands from a Linux terminal to find a couple of more candidates:
mkdir temp
cd temp
wget -r -np -nd --accept=err https://libraries.openmodelica.org/branches/master/Modelica_trunk/files/
grep "missing" *.err

@thorade Thats a good idea, would it be possible for you to great a pull request from each valid candidate.

We have https://libraries.openmodelica.org/branches/master/ModelicaTest_trunk/files/ as well if you are interested.


ASK: How to detect if one of the action toolbar clicked?

Hi, I'm using<EMAIL_ADDRESS>Is there any way to detect if user just click one of the action provided in the list of action passed to the RichToolbar? My intention is I want to invoke some function when user click one of the action.

Try registerToolbar

#50

Thanks @stulip I do see the toolbar item being called in the new example.
Would it possible to trigger the action functionality on the selected text ?
For example on clicking bold button it logs Toolbar click, selected items (insert end callback): ["bold"], can I somehow call the action.setBold on the selectedText.

Thanks for @blitzmohit discovering
This is a bug that appears to fix the focus problem. I hurry to fix it.

Thank you for acknowledging & fixing the issue @stulip , I see the version on master is executing the action on selection.
I see some handling for actions.heading1 etc in RichTool.js but can't seem to follow how that is made available to a user.
If I wanted to turn on just heading1 and heading2 do I need to create a customAction still ? If so could you provide an example on how it would look like since it will need to get the selectedText.
Really appreciate all your efforts in creating and maintaining this library.

@blitzmohit
Please refer to master branch example or Online Example


Board should be a class not a typedef

The board is currently a typedef of a vector of type vector of type board value. The board should instead be refactored into a separate class with utility methods that interact with the board array.

Finished 26a37356dfcbe04298fd921ba298ab61c3dd3ea0


[Bug] labelLayout: labels are jumping, shifting

Version
5.3.1
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=funnel&code=PYBwLglsB2AEC8sDeAoWszGAG0iAXMmuhgE4QDmFApqYQOQRjUC29ANMegGbCksBDMMzqx6SAQF9YAHgBGpAPQA-JHOmEkAY0kBSesUmd02ajWgATTV1gWhAwgG16AZQAWwAO4cxAYWwQWgDWPvQAahAAzkyhAJLQAI4ArhCkAJ6hAPKkFrT0ALrGsEbEkbQQ1JFONqgkJNACLNQMAGJJ0NDU2Bw26GBpIM1i3O2d3UV1LBDQhAAMEySCAB6EAIyz872wU9AuEABeQ_Sz-gvoy3uHDOsnPXXokXxgDLmRWtSW0xR393wV0M9YAAiDzkfYwMACbBAs6wCgCAiwVawzCIoEAZhOMK2bmolDcgKBAHYsbDsAI5F1rPcSCBgNFIDBCEDTNwwAAVUDYmkmabUAASeIoBLWADZYegoZQZsDWWBuTzIh5PIQwKQktQJbAtDg-AwAMQAEQAgoaWgBRACcPxpzCWYAAQnxcqRfLrREC1QJoJEQAJSB95Vq_RYLF8nPNYJHI-tClsSjTyZTsAAZPnUxXK1XqzVbXmdFz9UwZnna93Mo2mi2WhU0hP3et1JNdFMCNLAJKAoilyUBCgylnUNm1nkAN1okC0UONfYHqJHNLShAArJs67C7JDqqWkLBR1CNYRRZGGk0GBEGfRilrd_vsIfYAAWE-NI7xZKpDLXvPIPcHoYAEwvmeYjZC6V6Nvct7_oQAAcwFHP4gQhN-O5_veQw3OwsCnkc7heBBWz5DYkjEMRkgANxAA
Steps to Reproduce

click on the item in the legend

Current Behavior

on initial load (usually) labels are placed according to the Y in labelLayout (on the top)
user clicks on the item in the legend
labels jump top/bottom each time the legend is clicked

Expected Behavior
Labels should stick to the position specified in labelLayout
Environment
- OS: Ubuntu 22.04.1 LTS
- Browser: google chrome Version 112.0.5615.121 (Official Build) (64-bit)
- Framework: react

Any additional comments?
No response

Looks related to the animation. One workaround is to disable the animation.

Alright, thanks for the workaround


[QUESTION] Offset Commit - Clarification

I'm just getting started with this package. I've gone through the documentation and parts of the code surrounding consuming a message and using committers. From other research into librdkafka, I know that under the covers the offset committal process is controlled by a background thread so there is a chance that an offset could be committed to kafka before the message was successfully consumed. i.e. Kafka could think a message got received and is told to move on when really the message failed so you lose out on 'At Least Once' processing. (librdkafka doc reference)
My question is around the offset commit functionality in your package. When I was looking through the default DefaultCommitterFactory and the consumer code, it looked like by default the consumer will wait until the message successfully processes before trying a commit. If it fails processing, it will try to commit to the dead letter queue where as if it was successful it will commit the offset AFTER PROCESSING to the original topic. Is my understanding here correct?
If not, can you explain more what is happening when commitMessage runs or when you set up a topic using ->withAutoCommit()?

Hi @shazaman23, sorry for my late response i'm having a really busy week. Your explanation is correct. When you enable auto commit, the kafka consumer commits the offset periodically when polling batches

@shazaman23 i'm closing this but feel free to reopen or create a new issue if you still have any questions. Thank you!


Escalation policy exception if individual user

The escalation policy linked to the service currently needs to be a schedule and doesn't handle being an individual user.
The code should be fixed to be able to handle both situations.

Fixed in https://github.com/jrasell/dashboard-duty/pull/2


Support running in sub directories

For monorepo setups one has to run this action in a sub-directory, and not the root. Adding an input for that would be very helpful.

This should now be OK with v1.4.0 (v1). There is a new directory option.
Let me know if it does't fulfill your needs.

Awesome, thanks :+1:


Need suggestion

I am getting following error in Android Studio while trying to build the app:
Error:A problem occurred configuring project ':aace'.

executing external native build for cmake /home/user/Documents/aac-sdk-1.0/platforms/android/aace/CMakeLists.txt

I am have followed guldelines from below links:
https://github.com/alexa/avs-device-sdk/wiki/Ubuntu-Linux-Quick-Start-Guide
https://github.com/alexa/aac-sdk/blob/1.0/builder/BuildWithCMake.md
https://github.com/alexa/aac-sdk/tree/1.0/samples/android
Ubuntu 16.04 Android Studio 3.0.1, NDK 17.1 ,Android SDK Platform 26
Any help will be appreciated.

Hi @itsani4u , the error seems to be related to Alexa Auto SDK, please request assistance from their team: https://github.com/alexa/aac-sdk

Hi @visayamv  My goal is to build Alexa Auto Sample app.
But while do so, CMakelist.txt of Android Platform code throwing error.
Could you please help me with the steps for build Android Sample app with NDK.

Hi @itsani4u , the particular error  you mentioned:
Error:A problem occurred configuring project ':aace'.
is Alexa Auto SDK related which we don't maintain.


Steam groups are not applied

I tried to add Portal 2 as a game, but nothing happened :(

This happend because Steam now uses https and not http, so my string replacement in RequestHandler:70 broke.

I added in the last commit debug messages. Maybe these help to identify the problem.
It also could be caused by the join check delay. I lowered it to one minute.

I'll close this issue now because of the lack of activity. But if the debug message tell an interesting story feel free to reopen this issue.


Valid URL is failing.

Hello,
When I run this URL:  https://www.rent-river-oaks.com/
It's flagging it as invalid.  But the URL is working just fine.
var urlExists = require("url-exists")

urlExists('https://www.rent-river-oaks.com', function(err, exists) {
  console.log(exists); // true
});

Thanks,

It looks like https://www.rent-river-oaks.com returns a 400 on HEAD requests, which is why exists is coming back as false.
I'm not sure what the best way to proceed is.  As the code is written, any 4xx response status code is considered to be a failure.

Yep.  I tried this using the REQUEST module and it's still 400 status code.
It's strange b/c the website is loading just fine in the browser.
On Thu, Nov 2, 2017 at 10:21 AM, Bob Lauer<EMAIL_ADDRESS>wrote:

It looks like https://www.rent-river-oaks.com returns a 400 on HEAD
requests, which is why exists is coming back as false.
I'm not sure what the best way to proceed is. As the code is written, any
4xx response status code is considered to be a failure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/boblauer/url-exists/issues/3#issuecomment-341456281,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFF2reEN6tdQYCI9VhwVADWbRTEDiJsks5syd3lgaJpZM4QPxsw
.

The difference is the browser does a GET request, where as this package does a HEAD request.  The reason I do a HEAD request is because I want the response to be as small as possible.  I don't want to have to read an entire webpage into memory just to determine if it exists.

Hi, @boblauer I think I can help to solve.
I already solved or "workarounded" on my code.
the https://www.rent-river-oaks.com/ is now recognized as online
and the https://amazon.com mentioned on https://github.com/boblauer/url-exists/issues/6 too
I think we don't have to trust only on HEAD method, because there's a lot of poor implemented or maybe a kind of "over-secured" websites that dont respond to HEAD methods, or respond with wrong response codes, giving us a false positive.
Look, I'm using your library and when the urlExists response is true I assume that the url is online.
But when your library give me a urlExists false, I also make another check with a traditional "GET" method and check the status code.
If is real offline, the get method will have no impact on load an entire webpage into memory, as you mentioned above...
If is online unfortunately, due to wrong head response, there will be a little resource usage, but I think that it's better than have a false positive.
What do you think?
here's a code snippet that I made the workaround.
// on node.js I need to install node-fetch...
// npm install node-fetch --save

const fetch = require('node-fetch');

urlExists(url, function(err, exists) {
  if (exists) {
    resolve(exists);
    return;
  }

  // did not resolved above, so... try do a normal "get" request and check status
 // to do this "fetch" you need to install a package https://github.com/node-fetch/node-fetch

  fetch(new URL(url))
    .then(response => {

      //using the same regex you use, to test the response status
      resolve(/4\d\d/.test(response.status) === false)
    })
    .catch(err => {

      //maybe you have to handle this error or just reject as "false"
      reject(err);
    });
});

I hope you can use part of this code and add to the core of your library, which would be a nice accuracy improvement.

Hi, @boblauer I think I can help to solve.
I already solved or "workarounded" on my code.
the https://www.rent-river-oaks.com/ is now recognized as online
and the https://amazon.com mentioned on https://github.com/boblauer/url-exists/issues/6 too
I think we don't have to trust only on HEAD method, because there's a lot of poor implemented or maybe a kind of "over-secured" websites that dont respond to HEAD methods, or respond with wrong response codes, giving us a false positive.
Look, I'm using your library and when the urlExists response is true I assume that the url is online.
But when your library give me a urlExists false, I also make another check with a traditional "GET" method and check the status code.
If is real offline, the get method will have no impact on load an entire webpage into memory, as you mentioned above...
If is online unfortunately, due to wrong head response, there will be a little resource usage, but I think that it's better than have a false positive.
What do you think?
here's a code snippet that I made the workaround.
// on node.js I need to install node-fetch...
// npm install node-fetch --save

const fetch = require('node-fetch');

urlExists(url, function(err, exists) {
  if (exists) {
    resolve(exists);
    return;
  }

  // did not resolved above, so... try do a normal "get" request and check status
 // to do this "fetch" you need to install a package https://github.com/node-fetch/node-fetch

  fetch(new URL(url))
    .then(response => {

      //using the same regex you use, to test the response status
      resolve(/4\d\d/.test(response.status) === false)
    })
    .catch(err => {

      //maybe you have to handle this error or just reject as "false"
      reject(err);
    });
});

I hope you can use part of this code and add to the core of your library, which would be a nice accuracy improvement.


Issue running db-init.sh on Azure DataBricks installation

An issue occurs within the “Deploy Microsoft.Spark.Worker” section above.

bash scripts db-init.sh calls install-worker.sh are uploaded to the DBFS file system
install-worker.sh uses two Linux commands wget and basename that are not installed on the cluster VMs

Cluster creation fails when based on this init script

Script is simplified as follows to avoid wget and basename

Cluster is created successfully with simplified script
Processing of a uploaded jar of Microsoft Spark  (using Set JAR) executes and produces output until first .Net UDF
Job fails, logs report: java.io.IOException: Cannot run program "Microsoft.Spark.Worker": error=2, No such file or directory

Simplified script:  (all commands verified by running locally through Python OS shell execution in Workspace)
cd /usr/local/bin
tar xzf /dbfs/spark-dotnet/Microsoft.Spark.Worker.netcoreapp2.1.linux-x64-0.3.0.tar.gz -C ./
sudo chmod 755 Microsoft.Spark.Worker-0.3.0/Microsoft.Spark.Worker
sudo ln -sf    Microsoft.Spark.Worker-0.3.0/Microsoft.Spark.Worker Microsoft.Spark.Worker
Please advise on how this should be done.

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

ID: d09d9935-2cc4-e579-dd1e-120569030ccd
Version Independent ID: 3a4307d6-10e4-546e-9cf5-ad59b62ea8f1
Content: Deploy a .NET for Apache Spark application to Databricks
Content Source: docs/spark/tutorials/databricks-deployment.md
Product: dotnet-spark
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

Tried adding the following to simplified script:
        DOTNET_SPARK_WORKER_INSTALLATION_PATH = "/usr/local/bin"
        export DOTNET_SPARK_WORKER_INSTALLATION_PATH

and, alternatively:
        DotnetWorkerPath = "/usr/local/bin"
        export DotnetWorkerPath

Still fails at  .Show() after:
        spark.Udf().Register<int?, string, string>(
            "my_udf",
            (age, name) => name + " with " + ((age.HasValue) ? age.Value.ToString() : "null"));

        sqlDf = spark.Sql("SELECT my_udf(*) FROM people");
        sqlDf.Show();

Also posted issue at https://github.com/dotnet/spark/issues/173

See dotnet/spark#173


bcmod issue

in ban/library/IBAN/Core/IBAN.php
on line 115 you use the code return bcmod($invertedIban, 97) === '1';
bcmod is (even in php 5.*) not always available since the Math extention should be configured
a fix could be
including a fallback bcmod function
if ( !function_exists('bcmod') ){ function bcmod( $x, $y )    {         // how many numbers to take at once? carefull not to exceed (int)         $take = 5;         $mod = '';         do  {             $a = (int)$mod.substr( $x, 0, $take );             $x = substr( $x, $take );             $mod = $a % $y;         }         while ( strlen($x) );         return (int)$mod;     } }

I've got this issue as well, I would appericiate if this method could be added to this library to prevent issues when using this on a version of PHP where this method doesn't exist. However, thank you for this example, I am going to try this!

resolved in https://github.com/jschaedl/Iban/pull/18 but not merged yet

As far as I can see it, this issue is not resolved in https://github.com/jschaedl/Iban/pull/18 There is no fallback bcmod function included yet. Maybe @bobmulder can do it and make a new pull request?

@jschaedl I could add the bcmod-method if you want?

do it

Where should I put the code? I did it in my own config-file...

The code you need:
/**
 * Function bcmod (for IBAN library)
 */
if (!function_exists('bcmod')) {
    function bcmod($x, $y)
    {
        $take = 5;
        $mod = '';

        do {
            $a = (int)$mod . substr($x, 0, $take);
            $x = substr($x, $take);
            $mod = $a % $y;
        } while (strlen($x));

        return (int)$mod;
    }
}

@bobmulder: Thank you for providing. I added the fallback method to https://github.com/jschaedl/Iban/blob/master/library/IBAN/Core/IBAN.php

Great @jschaedl. Guess this issue could be closed?

Exactly.


RAE incorrect encode function

Hello! Thank you for your project, great job!
I have a trouble with encoding function in RAE. I'm new in Theano, so I could have mistaken, but I think the code in lines 51-53 is not valid: x_var should be a matrix and encode_func should contain x_var as argument, not x. So, I suppose the correct code will be:
        x_var = T.matrix()
        self._encode_func = theano.function([x_var], self.layers[0].encode_func(x_var),
                        allow_input_downcast=True, mode=theano.Mode(linker=THEANO_LINKER))

Sorry if I mistake.

Hi,
Actually, the code for RAE is very very old, and is not well tested.
I'm think you are right, I have to make a testcase for it and fix the problem.