Implement new method: CheckedFunction_#recover

fix #1381

[Current coverage][cc-pull] is 97.09%

Merging [#1385][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.07%

@@             master      #1385   diff @@
==========================================
  Files            86         86
  Lines         10099      10189    +90
  Methods           0          0
  Messages          0          0
  Branches       1818       1818
==========================================
+ Hits           9813       9893    +80
- Misses          158        167     +9
- Partials        128        129     +1

Powered by Codecov. Last updated by [c47d5c5...9b81749][cc-compare]
[cc-base-branch]: https://codecov.io/gh/javaslang/javaslang/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/javaslang/javaslang/compare/c47d5c57e01015ccc686f824eac7dd92f17563c3...9b817496d4099aa56d3137a1aea57c2756fedbaf
[cc-pull]: https://codecov.io/gh/javaslang/javaslang/pull/1385?src=pr

Tests for Function_#liftTry() will be created soon

That was fast! The implementation looks great.
But I think we need a second iteration. While looking at our current idea to use Tuples (because of the Function arity bound of 8) I think currying fits better. We can make the following simplifications:
Simplify method argument types
(Function0..2 have super interface in java.util.function)

Instead of Function0<R> we write Supplier<R>
Instead of Function1<T1, R> we write Function<T1, R>
Instead of Function2<T1, T2, R> we write BiFunction<T1, T2, R>
For CheckedFunctions we cannot make these 'optimizations'

Currying instead of Tuples:
(Curried type signatures look ugly in Java but are nice to write: x -> (t1, ..., tn) -> r)

Instead of CheckedFunction2<Tuple0, Throwable, R> we should use CheckedFunction1<Throwable, CheckedFunction0<R>>
Instead of CheckedFunction2<Tuple1<T1>, Throwable, R> we should use CheckedFunction1 <Throwable, CheckedFunction1<T1, R>>
Instead of CheckedFunction2<Tuple2<T1, T2>, Throwable, R> we should use CheckedFunction1<Throwable, CheckedFunction2<T1, T2, R>>
...
Instead of Function2<Tuple0, Throwable, R> we should use Function<Throwable, Supplier<R>>
Instead of Function2<Tuple1<T1>, Throwable, R> we should use Function<Throwable, Function<T1, R>>
Instead of Function2<Tuple2<T1, T2>, Throwable, R> we should use Function<Throwable, BiFunction<T1, T2, R>>
Instead of Function3<Tuple3<T1, T2, T3>, Throwable, R> we should use Function<Throwable, Function3<T1, T2, T3, R>>
...

I hope that it is not too much effort for you to apply the changes. The good thing is that we use a generator. There are already examples of switching functions (Supplier, Function, BiFunction).
Thank you very much!

@danieldietrich, fixed:

currying instead of Tuples applied to recover and unchecked
simplify method argument types applied to recover, unchecked and liftTry

This, Sir, looks awesome! Many thanks.


Virtual dir

removes old files operations to replace it with new class that makes a virtual file system and allows access to only allowed dirs

@lawnvi do you think you can check these new changes, it allows more control over dirs now you can share more than one dir instead of the one dir and uses path package to normalise stuff, if you have a case not covered let me know

@lawnvi do you think you can check these new changes, it allows more control over dirs now you can share more than one dir instead of the one dir and uses path package to normalise stuff, if you have a case not covered let me know

Apologies for the delay in getting back to you. I just saw your message and will try out the changes as soon as possible. If I encounter any issues, I’ll let you know.

windows ftp client seems to not support the SIZE command.

Thank you for your work, what do you think about the changes
I think using the path module is a cleaner approach and having the ability to share more than one dir is the original need for ftp
Do you have anymore ideas or missing features that needs to be added?

Selecting multiple files at once is indeed very useful, as the files you want to share are not always in the same directory.
In macOS Finder, some operations related to the server, such as renaming, deleting, and uploading files (drag and drop or paste), may not be displayed.
Additionally, characters like ‘|’ might be supported on one platform but not on another, which can lead to problems.

In macOS Finder, some operations related to the server, such as renaming, deleting, and uploading files (drag and drop or paste), may not be displayed. , i didnt try to connect to ftp server using finder, should it show these options? if thats the case will need to check what commands need to added
and for | i think there is a need to replace all the special chars to be safe on all platforms


Print logs when build fails in travis

Closes #76
Signed-off-by: Shishir Halaharvi<EMAIL_ADDRESS>

Should the logs only be displayed when the checks fail? Or should it be printed all the time?

oops didn't see your PR, I created another :)

Failure case should be fine.


Create a common test package for opentelemetry-lib remappers

Currently opentelemetry-lib includes no specific testing package. As we plan to merge more datastreams rather than hostmetrics (eg. kubernetes one), there are common fucntions that are being used in testing.
See eg here: https://github.com/elastic/opentelemetry-lib/pull/20/files#diff-973a344d29ac8e238d1e425d5a5ee843244cbe07fd85a3c1598704af3967a6d0
Plan of this issue is to move the common testing functions in a seperate testing package

Dont merge until https://github.com/elastic/opentelemetry-lib/pull/20#issuecomment-2165545553 is merged
cc @lahsivjar


More test required

Need to implement simple Gradle/Ant project in test/ dir and write tests that will build this project and run some tests on this project. To be sure that Flash Played Debugger and Xvfb work correctly.

Resolved in #2


Backface culling wipes out every other row in the triangle strip.

I've tried adjusting the winding order in terrain. I cannot figure out why this is happening. Enable culling to see the problem.

I took a look. And I can't think of anything off the top of my head... but I do notice that it seems like it does every other 2 rows.

http://www.gamedev.net/topic/174403-left-hand-right-hand-problem-with-triangles/
The second response (the one from John Bolton) might lend some insight.
I think we are basically missing a triangle after every two rows, from how we did our indexing logic.
That would explain why a full two rows gets culled.

http://www.gamedev.net/topic/259017-drawing-height-map-using-triangle-strip-with-ccw-problem/
just another link explaining triangle lists winding orders...

Fixed. I will be pushing the new code to the branch.


Add option for sending using TCP

Hi, i hope this will be useful.
AWS Case
Motivation
I need proper percentiles accounting, improve performance and problem with old statsd 1 cpu bound. This should simplify configuration and makes statsd setup stateful.
Description
I run Statsrelay on each local machine that previously have local statsd daemon. This improve performance and remove local statsd 1CPU bound.
Ok you can say why not just use statsite on each machine this will improve performance and remove cpu bound. Unfortunately this will still generate more cpu usage on instances, but for most cases bad percentile accounting based on each instance of app and server..
When we have many app instances on many VM's in PaaS infrastructure with same metric name we have local accounting of percentiles on each of this instances. To make percentile accounting valid we need central statsite setup. After that we should send all RAW statsd data from all apps instances to one statsite instance.
In AWS we like to use Elastic Load Balancer/Elastic IP before statsite instance to make sure that sending endpoint is always with same DNS name or Elastic IP. ELB handling TCP but have no option for UDP balancing. Thats why i add TCP send from statsrelay to any statsd server supported TCP listener.
UDP listen -> statsrelay -> UDP/TCP Send to statsd server

This makes whole setup on all instances with statsrelay transparent and stateful with no need to reconfigure if any statsite instance failed. We can now replace any instance behind ELB without any change in infrastructure.
Use cases
multiple instances with same metrics -> local statsrelay to one TCP endpoint -> ELB/EIP -> remote statsite/statsd

When we need percentiles to be centralised and be sure that all app instances sending data to one particular statsite. Metrics does not have any info from which instances came from.
multiple instances with different metrics -> local statsrelay pointing to many TCP endpoints -> multiple ELB/EIP -> multiple remote statsite

When we have multiple metrics, but metrics have info about source or they are uniq. Then we can send them using statrelay hashing to many statsite remote servers behind ELB or EIP as it is with current UDP setup.

I've been pondering something very similar for a bit as I'd like to move my setup to using a TCP statsd protocol for improved reliability and better support service discovery methods.  So, I'll take this.  ;-)

Thanks :)


on scroll event

Hi.
I would like to know if there a method to use the 'on scroll' function from scrollMonitor.
The idea is to use it for create a custom parallax effect rather than coding another scrolling event handler.
thank you, and especially thank for this lib !

Check out scrollmonitor-parallax, the docs aren't great but it may do what you need.
There's no way with the library to receive a scroll event, but you can use it to add and remove scroll listeners as needed and use it to read the scroll position.


File cleanup and more annotations

Divided the classes and libraries into folders and added annotations for Item and warp.

LGTM


Proposal: Deprecating support of incomplete indexing on MultiIndexes

First, here's the example DataFrame
                    0  1  2  3  4  5  6  7
first second third
bar   one    three  4  9  7  8  5  0  7  8
             four   6  8  1  5  9  9  1  7
      two    three  7  5  2  6  7  8  5  9
             four   0  8  8  5  3  5  8  3
baz   one    three  6  0  8  0  0  9  8  8
             four   9  3  2  0  2  7  4  9
      two    three  0  3  7  7  4  3  7  0
             four   6  3  2  8  3  9  7  8
foo   one    three  6  7  3  7  3  0  3  6
             four   5  8  0  8  1  5  1  5
      two    three  2  0  8  2  8  1  8  3
             four   9  0  2  7  0  6  8  3
qux   one    three  1  2  5  5  0  7  0  1
             four   6  6  7  0  0  4  5  3
      two    three  1  8  2  8  7  5  7  5
             four   1  1  3  8  8  6  0  3

For DataFrames with MultiIndexed rows, pandas allows this type of indexing
df.loc[('foo','bar'), ('one','two'), ('three','four')]
To be taken to mean
df.loc[(('foo','bar'), ('one','two'), ('three','four')), :]
But this type of indexing is ambiguous in the case when the number of indexing tuples is 2 since
df.loc[('foo','bar'), ('one','two')]
could mean incomplete indexing as in
df.loc[(('foo','bar'), ('one','two')),:]
or row,column indexing as in
df.loc[(('foo','bar'),), (('one','two'),)]
I appreciate that there is already a warning for this in the documentation, but I wonder if the functionality is worth the complications it adds to the code/docs.
Personally, I would suggest offloading the responsibility of complete indexing on a MultiIndex DataFrame to the user (obviously this doesn't apply to Series as they are 1d so to speak). This would take away the minor syntactical convenience of not specifying the column index, but it simplifies the code and gives the user only one way to index on a MultiIndex DataFrame (which makes usage less confusing).
The consequence to the user in the specific case of selecting multiple levels of a row-MultiIndex on a DataFrame is that instead of writing
df.loc['foo','one']
they would have to write
df.loc[('foo','one'), :]
And, in the syntactically worst case, instead of writing
df.loc[('foo','bar'), ('one','two'), ('three','four')]
they would have to write
df.loc[(('foo','bar'), ('one','two'), ('three','four')), :]
I'm fairly new to pandas (don't think I started using it until v0.16), so I realize I may be missing the bigger picture. If so, enlighten me!

it's not whether we should do this
it's just ambiguous and I don't think it's possible
however if you would like to try to fix be my guest

@jreback I'm not sure I understand you're comment. What I was saying is I don't think it is possible to do the incomplete indexing in the general case in any reasonable way which is why I was advocating deprecating incomplete indexing for multiIndexed dataframes.

@tgarc my point is that the following are both completely legitimate, but mean different things. Their is no way to disambiguate what is meant except for the user providing context (e.g. both axes). So how do you propose to deprecate this then?
In [35]: df = DataFrame(np.arange(12).reshape(4,3),columns=[0,2,1],index=MultiIndex.from_product([range(2),range(2)],names=['first','second'])).sortlevel()

In [36]: df
Out[36]:
              0   2   1
first second
0     0       0   1   2
      1       3   4   5
1     0       6   7   8
      1       9  10  11

In [37]: df.loc[(0,[1]),:]
Out[37]:
              0  2  1
first second
0     1       3  4  5

In [38]: df.loc[(0,[1])]
Out[38]:
        1
second
0       2
1       5

Closing since there is no obvious recommendation, other issues (#19110 for instance) face the same problem and there doesn't seem to be interest in just disabling incomplete indexing. @tgarc feel free to argument if you still think this is valid


[Application] Inform the applicant about the application result

Is your feature request related to a problem? Please describe.
When the application process is finished the applicant should be informed whether the application was rejected (otherwise he might never know).
This might need to be discussed with the StMas.
Describe the solution you'd like
An e-mail is sent to the applicant, that the application was rejected.
Describe alternatives you've considered
The Mitarbeiter der Kommune can decide him/herself whether an e-mail should be sent.
Additional context
Add any other context or screenshots about the feature request here.

Well, we currently don't have contact data for every region (and likely won't have it in the near future). Probably we have to refer him to the website of his region if he wants to directly get into contact with someone. (see here: https://www.lbe.bayern.de/engagement-anerkennen/ehrenamtskarte/antraege/index.php)

The websites are in fact already in the database (region -> website). Furthermore, the websites I checked out also list a contact phone number or email. I guess we refer the user to there :)

Well its actually a lot of work to collect these mail information. I think we could maybe send an email to the applicant when the card was created. (This can be set in the application next to the delete button) Even better would be to have a button on the application "Create Card" and pass the information to the create card view. Currently the process is very error prone, since the user has to check the application, note down surname, name and type of card and then switch to the card creation and type it in.

I think you misunderstood there. The idea was to just add a link to the website in the email. And the main idea behind sending the email was to inform the applicant about whether the application was rejected. As otherwise the applicant may never know.
One issue here might be that there already is a workflow when the application is rejected, where the "mitarbeiter of the region" calls the user or writes an email by hand.
When the application is successful the user should already receive a letter/email with the anmeldecode, so it might not be necessary to send an email in this case.

this issue was rejected since it's not yet a requirement by the customer


User interaction when running code

At the moment, stdin is submitted alongside the program source code. This means user interaction with the program is not possible. As a well-requested feature, I'll open the issue.

Input is a single feature, you can add many by seeing the website repl.it. It has a Shell, code GUI window and a console to see the output and add the input. You can see the design of this website “https://repl.it/login?goto=%2F~“ by creating a demo project on a free account.


Dealing with incomplete birth date

Incomplete birth dates are encoded as datetime="1986-0-0" which is not accepted by Date() when a string is used to initialize.
There is a simple solution by adding .replace(/-0/g,'-1') but then the zodiac sign is wrong. So here is what I use for the moment:
[...]
		const actorDates = [];
		const actorDatesFixed = [];
		var fixed=false;
		responseText.replace(/datetime="(\d{4}-\d{1,2}-\d{1,2})"/g, (match, date) => {actorDates.push(date), actorDatesFixed.push(date.replace(/-0/g,'-1'))} );
		if (actorDates[0] != actorDatesFixed[0]) fixed=true;
		const [actorBirthDate, actorDeathDate] = fixed ? actorDatesFixed : actorDates;
[...]
		span.innerHTML = `Was about ${actorInfo['age-when-the-movie-first-released']} years old` + (zodiac_signs_disabled || fixed ? '' : ` <br>(${getZodiacSign(month, day)})`);
[...]

I thought about it before actually, but I kind of wanted to be as accurate as possible, specially when calculating the actor age at the moment of viewing. The script already assumes the movie gets released once it is done shooting. Would be ideal to have the date in which they started shooting that movie, I guess 😄.

I often wondered the age of the actor at the time of filming. Even more when on imdb the picture assigned to the actor is one of an old person. That's when your userscript is useful.
But then, there is this film: https://www.imdb.com/title/tt1065073/
it took 12 years to shoot.   :)

That just complicates things. That movie looks interesting though, will have to watch it someday.


Netlink constants

Let's see what ci says

Some of the failures on nightly I think may be fixed on rebasing, otherwise looks like there may be some other legit failures.

SOL_NETLINK doesn't seem popular in headers yet, so I removed it.


How does "default_tasks" section work?

🌱 Hello.
I would like to know "How does :default_tasks section work?"
If I write in :default_tasks section like below,
in project.yml
  :default_tasks:
    - test:all --verbosity=obnoxious

I expect that it is the same as $ ceedling test:all --verbosity=obnoxious
but, result is ...
$ ceedling
🌱 Loaded project configuration at default location using ./project.yml

🌱 Ceedling set up completed in 297 milliseconds
rake aborted!
Don't know how to build task '--verbosity=obnoxious' (See the list of available tasks with `rake --tasks`)

Tasks: TOP => default
(See full trace by running task with --trace)
🪲 ERROR: Ceedling could not complete operations because of errors

of course, I've read docs, but I need more clarity or example.
I'm very looking forward 🌱 v1.0.0 !

Oh! This is a very good question. The command-line-parsing library Ceedling uses makes a difference between flags (things like --verbosity or -v) and tasks (things like test:all). The :default_tasks section allows you to list multiple tasks to be executed one after another, but it doesn't support flags.
It'd look like this:
:default_tasks:
  - clobber
  - test:all

At the moment, you'd still have to issue flags like so:
ceedling -v=4

which (if you have the very latest) is the same as saying
ceedling --verbosity=obnoxious

which is also the same as the following (because of the default tasks defined above)
ceedling --verbosity=obnoxious clobber test:all

I just pushed a revision to CeedlingPacket that clarifies the meaning and values for :project ↳ :default_tasks.


[Documentation] Missed documentation in Conv classes hierarchy

After merging I recognized that the KDocs for the Conv class hierarchy are missed
Please add KDocs for all classes, methods, and fields in the following classes

[ ] AbstractConv (class, properties, methods)
[ ] Conv1D (methods)
[ ] Conv2D (methods)
[ ] Conv3D (methods)
[ ] DepthwiseConv2D (methods)

I suppose that enough to define good KDoc for the parent (protected) method of property, the documentation is inherited by child methods

@avan1235 will you take this ticket?

Yes, of course, as it is my fault that I didn't provide them with the code


[Feature-#1897][gbasehk] annotation gbasehk connector

Purpose of this pull request

Which issue you fix
Fixes #1897 .
Checklist:

[ ] I have executed the 'mvn spotless:apply' command to format my code.
[ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

reviewed


CRF Layer rename sparse_target to one_hot_target

Hi everyone,
This is a recommendation for the CRF layer, written by @linxihui . I think author's acceptance as also broad acceptance by users is needed to proceed in any code change.
I would suggest the renaming of the parameter sparse_target. In my opinion, it is completely misleading. One has to use either sparse_target=False for one-hot vectors (labels) or sparse_target=True to use integers.
What is the intuition behind the definition of "sparsity" in this statement? I was completely misled until I run and show the data representation of the related example (BILSTM-CRF, CONLL).
I would suggest the clearest possible form e.g. one_hot_target. I would like to hear more opinion on that matter.

Agreed - the description sounds contradictory to the argument name, and the documentation does not specify what the value True entails.

I was confused as hell for an entire day before I found this issue. So please do this!


Hi all!

I'm starting on JS/Duktape and this works fine! [to my surprise with very low memory hungry]
Using on;
a powermac g4 [32-bit powerpc big-endian] on osx tiger 10.4 running! [compiled with gcc 4.x / xcode 2.5]
a desktop win7/64-bit on cygwin
and was running perfect on a [32-bit arm big-endian] property/closed os [unix-like]
I can run tests on this plataforms! [on new versions or patched versions!]

Good to hear - there's ongoing work for a 2.0.0 release so there'll surely be some testing ahead :)

fine!
the project have a google groups or an aproprieated channel ?

@denisdemaisbr The support mechanisms are listed on http://duktape.org/. Currently there's github for development related stuff (features, bugs, etc). There's also a github project for the duktape-wiki which is a natural place to request howtos and such. There's an IRC channel too, but no mailing list or similar at present.


Fix Y Axis labels being truncated on horizontal bar charts

Description

Resolves #1203
Checklist

[x] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset

Any sense why this one isn't showing all axis labels?

Any sense why this one isn't showing all axis labels?

Not sure; I'll take a look


Bug: Errors when adding secondary contacts

Describe the Bug:
Probability (how likely the bug is to happen, scored from 1-5): 4
Effect (how bad the bug is when it does happen, scored from 1-5): 4
Steps to reproduce the behaviour:

when you select "Add a Secondary Contact" the fields appear and then quickly disappear...prompting you to select the secondary contact option again and when you do, two sets of secondary contact fields show up, leaving you to remove one and then you can proceed to populate them - but not always able to apply to revision.

Another thing to note is the "behavior" of project contacts page under project details tab. When entering primary and secondary contacts they will flip on each other (making secondary the primary and the primary the secondary) This can righted by going back in to the drop downs and reselecting the contacts but thought I would mention it since we've had issues within this tab recently.

This needs reproducing, or check with LP
Screenshots:
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context

Unable to reproduce this behaviour in test or locally.


Auto Commenting when a PR is merged

Fixes #233

Please merge this pull request and add the labels as per gssoc guidelines.


[DO NOT MERGE] Backend prototype

Just opening this to start getting feedback. A lot of changes will break the existing stable CLI and dashboard, need to think about how to have backwards compatibility. Perhaps release a new version of spektate package that should not be consumed by existing client side.

Refactored all the classes into interfaces for better JSON handling after HTTP calls
Removed access token from interfaces or it may lead to exposure
Moved backend into its own yarn project

Note: build will fail, will need a new spektate package release.
Related to https://github.com/microsoft/bedrock/issues/980

@samiya what do we need to do before merging this?

@andrebriggs https://github.com/microsoft/bedrock/issues/1017 would be nice to polish it a little more before we merge it (since this is a prototype)

@samiyaakhtar ok. I just saw that all the task boxes were check in https://github.com/microsoft/bedrock/issues/980 and thought we were done


Error when compiling

Hello,
i am facing following error when try to build the project:
esp-idf/esp32/libesp32.a(cpu_start.c.obj):(.literal.main_task+0x10): undefined reference to app_main' esp-idf/esp32/libesp32.a(cpu_start.c.obj): In function main_task':
C:/Users/asenn/Desktop/esp-idf-2/components/esp32/cpu_start.c:514: undefined reference to `app_main'
collect2.exe: error: ld returned 1 exit status
I use this guide as in the readme file:
https://docs.espressif.com/projects/esp-idf/en/latest/get-started/windows-setup.html
but can't 'figured out what is wrong.

I don't use ESP32, but I frequently see that people have issues if they use a version that is too new. Maybe you should try an older version of ESP-IDF.

I am wondering this project with which IDE is created, which compiler is used and the OS?

I am wondering this project with which IDE is created, which compiler is used and the OS?

It's a simple makefile project but as @embeddedt mentioned ESP-IDF's build has completely changed. Therefore the project is not working with the latest version of ESP-IDF.
Take a look at this post in the Forum:
https://forum.littlevgl.com/t/how-to-create-complex-project-structures-in-esp-idf/619/15?u=kisvegabor
I'm using v3.2 of ESP-IDF which works well with this project.

I tried with v3.2, but still can't compile. The error is the same.

Please open a topic in the Forum about this question.
The Forum is visited by more people, so there is a higher chance to get help there.
https://forum.littlevgl.com/


silx.gui.data: Do not keep aspect ratio in NXdata image views when axes @units are different

This PR does not keep aspect ratio by default in the case the 2 axes of an image have different @units (either both are defined and different or only one is defined).
In all other cases, the aspect ratio is kept by default.
This is done for the image and complex image widgets, and not for the stack or 3D views.
closes #3626

attn @linupi, would that work for your use case?

Yes @t20100 , this would do the job in my use case. Thank you for considering my issue!


Card reader

Describe your idea
Could you add magnetic cards, RFID cards and card readers like there was in Immibis's peripherals ?
Describe alternatives you've considered if you've any
No response
Additional context
No response
Linked Issues
No response

I'd like to expand upon this because RFID and Magstrip cards are one of my favorite toys to have in Minecraft, especially with a ComputerCraft focus.
Specific changes for RFID compared to Immibis Version

RFID "holding" functionality where using the item on a reader returns a much lower distance than possible standing next to it. This is to improve speed as speed is proportionate to disatnce as well as make readers that require the user "tap" it like a magstrip. This is nice so that I can use the same RFID card for nearby checks as a security badge that must be intentionally used.

Specific changes for Magstrip compared to Immibis Version

I'd like to see card read + write (or update) functionality so that a card can have it's data read and written to without the user simply inserting a second card for the write part.

Changes to both compared to Immibis Version

Some decorative way to tell cards apart outside just coloring a piece of them and naming them. Even if it was just a small selection of alternate styles.
The ability to "sign" a card, which is unique to the player who is writting it. Basically just ensures you can make cards that for all intents are not copyable because it stores the user ID of the signer on it. Could be used for example for making ID cards for your base that can not be created by a 3rd party to gain access even if they read the card you gave them at some point.


Type error

Got this error recently:
Argument 3 passed to Amp\Parallel\Worker\TaskException::__construct()
must be of the type integer, string given,
called in .../vendor/amphp/parallel/lib/Worker/Internal/TaskFailure.php on line 54

This happens because PDOException::getCode() can return a string.

Thanks for reporting, #20 should fix this.

I think the actual fix for v0.2 should be to simply not reuse the exception code of the parent exception, but instead just use 0 there and remove the parameter from the constructor.

I agree here. The code is just part of the parent exception, it has no business in the wrapping exception.

But changing that now would be a BC break, so let's do it for v0.2 only.


take devtron url from UI instead of env variable

impacted component

[x] notifier
[x] external ci
[x] grafana

url scheme: http/https
domain on which devtron is available
Auto-fetch URL and inform user that URL has been updated.
URL can only be updated from super-admin account.
Global Config > Host URL (domain name from which this dashboard is accessible)
App > App metrics
App > Trigger
Workflow editor > Upfront + External CI
Notification > Show message upfront

App > App metrics  (grafana url tested)
App > Trigger https://zpl.io/ad6ODKn
Workflow editor > Upfront https://zpl.io/agON3X9
External CI https://zpl.io/aN6X7rv https://zpl.io/V4gWY8X
Notification > Show message upfront https://zpl.io/b6GrePj


Add build.zig

MicroTeX is amazing and I want to use it in my Zig projects.
The best way for a Zig programmer to be able to do that is if MicroTeX has a build.zig file in the source tree that can build the C wrapper. This lets downstream Zig users use the package manager to easily integrate MicroTeX into their projects.
Other benefits include:

Another trivial way to build (zig build to build).
Cross compilation for free (e.g. zig build -Dtarget=arm-linux-muslabi).

If this is deemed irrelevant, then please just close the PR. For my own projects I can depend on my fork, but I thought I would contribute back as other Zig users might also want to use MicroTeX.
Thanks!

Great job!


[HW] Rakesh Pavuluri

Name: Rakesh Pavuluri
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/rakeshpavuluri/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Looks great Rakesh, thank you. Here's your badge -> https://api.badgr.io/public/assertions/tHsA90u5QcuXrwUiJ3diIQ?identity__email=mr.rakeshpavuluri%40gmail.com


Fix dashboard error when click on Total subscribers

http://forge.prestashop.com/browse/BOOM-4104
Adjusted Dashboard Total subscribers link to point to ps_emailsubscription configuration page instead of blocknewsletter that doesn't exists anymore.

Tested and ok, thanks !

Thank you @gabdara


Add/google prices

This is a WIP that adds the ability to get actual on demand (and preemptible or spot) prices from the Google API. It's a really weird  API to figure out - you have to convert nanos into a unit, and then multiply for each of cpus / memory to get a price. I need to do some more spot checks, and likely factor in other small variables that influence price (that may not be reflected here).

Note that I don't expect tests to pass yet - there were changes to some of the data structures that need to be also changed for the tests to account for spot instance prices.


[BUG] Bloxstrap Exeption

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
idk wtf is this even mean i try to open roblox with bloxstrp new verison just not opening roblox pls help

Read the Wiki


Fix typos in chapter about filtrations in modal logic

Another student pointed out these typos to me.

Points in a filtration are equivalence classes. And in some places, the brackets that mark the equivalence class were missing.
Once the wrong variable is used. (u does not even appear in this proof)
Finally, the parts of a model are introduced with a non-conventional variable.

Thanks!


Angular Controller not effective on boolean flag

In the main div, we have initialized boolean flags using ng-init.

These flags are used in section div to hide show the div using ng-if

On event ng-click, respective ng-controller calls a method to toggle the flag.

But the required result isn't found.

Check scope of controller & ng-variables

**ng-if creates its own local scope for all variablese inside its scope. Confusing ?? :1234: **


Add information on how to create applications

I am new to Spinnaker and I am looking for a tutorial explaining how to create an application. This page explains how the UI had been made, but there is not information on how to create and application.

Thank you


Broken link to ScienceBase scanner

The Trouble-Shooting: Overview page (https://usfws.github.io/Science_Applications_Metadata_Guidance/content/trouble_shooting/trouble_shooting_overview.html) contains a broken link to "ScienceBase scanner" (http://calcommons.info/sb2/scansb.php)

That link is not longer working because the website is defunct. Removed that section in the Guidance, so the issue is resolved.

That link is not longer working because the website is defunct. Removed that section in the Guidance, so the issue is resolved.


add option to leave __() calls in the output

See #20. Wanted to get some code up so we can discuss

RE: https://github.com/privatenumber/webpack-localize-assets-plugin/pull/21#issuecomment-963476637

I still think #25/#26 is a better design, for what it's worth!

Since this change shifts localization to run-time, why not go full run-time at this point?
For example, by using ProvidePlugin to inject the localization function:
new webpack.ProvidePlugin({
  '__': ['~/localize.js', 'default'],
});

~/localize.js
// assume the correct json of strings are exposed via `localizedStrings` global

export default function (stringKey) {
	return window.localizedStrings[stringKey];
}

All of the strings will still be templated in at compile time, this just gives the option of injecting some additional runtime code (by defining a global __ function). I'm not anticipating that I'll be doing string lookups at runtime, just pluralisation and templating (which unavoidably has to happen at runtime).


Polyline

First round

Note: we lost some functionality in this change that needs to be restored before it can be merged. Namely:

Handling of closed polygons outlines, which form a continuous loop (instead of having caps on the first and last points). See closed_polygon flag.
Automatic removal of line segments falling on tile edges, which is needed for correctly rendering polygon outlines, since many logical polygons have been split into several polygons across tile boundaries (we are basically visually reassembling it to appear as a single polygon). See remove_tile_edges flag.

Polyline ready for review. Not caps and joins implemented yet

So does this just silently replace the old line style? Do we have or want a way to specify eg endcap style in the scene def?

Yep this is just a different way of implementing what we had before. There
aren't any additions for caps and joints yet, but there should be soon!
On Thursday, January 22, 2015, Peter Richardson<EMAIL_ADDRESS>wrote:

So does this just silently replace the old line style? Do we have or want
a way to specify eg endcap style in the scene def?
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/tangram/pull/79#issuecomment-71119661.


Not downloading all posts with a certain hashtag

When the search by hashtag retrieves a high number of results (> 1000 for example), not all posts are downloaded (and no error in printed).

On the first glance, this seems a duplicate of #20.
The number of posts with a certain hashtag may differ from the number of posts that can actually be accessed, as the hashtag count might include private posts. The more interesting question is whether an official Instagram app loads more hashtags posts than Instaloader does.
Are the "missing" posts accessible by other Instagram clients, such as the browser interface or the Instagram App?


Get rid of useless threading

The cheat is onethreaded, so I don't see the need to make a thread there.

You do make a solid point, merging :)

You do make a solid point, merging :)


[DO NOT MERGE] Enable strict concurrency

Motivation:
Catch potential data races at build time.
Modifications:

Enabled strict concurrency checking in the Package.swift.
Updated the ServiceDiscovery protocol, and all concrete implementations, to have @preconcurrency and added Sendable where needed.

Result:
Fewer potential data races can sneak in.
One issue, however - I'm not able to fix one remaining warning on AnyServiceDiscovery with the message: Conformance of 'AnyHashable' to 'Sendable' is unavailable. I couldn't find a way to silence it with any permutation of @preconcurrency. It (correctly) complains about AnyHashable not being Sendable, but ideally we'd have a way to silence that warning.
Due to that I had to disable warnings-as-errors even on Swift 6.0.
Test Plan
Ran tests locally, did not see any concurrency warnings or errors.

Do not merge until Swift 6.0.3 has been out for a bit, we rely on @preconcurrency fixes in that version.


When the amount of data is large, exporting data will be very slow.

Describe the bug
When the amount of data is large, exporting data will be very slow.
Here, it's not a failure caused by exporting data timeout. When I uploaded a text annotation data of 50,000 records, with a file size of approximately 30 megabytes, and tried to export it, it took nearly 4 minutes.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Screenshots
Environment (please complete the following information):

OS: [e.g. iOS]
Label Studio Version [e.g. 1.12.0]

Additional context
Add any other context about the problem here.

hey @zhoumingcheng
Thanks for the bug report, are you able to reproduce on the latest version? Furthermore, are you deploying your LS instance locally or is it hosted on a service? Wondering what the bottleneck is on this export process. What are your impressions?

I have a similar issue, but the amount of data is much smaller (1200 Taks). I cannot even export because I run into an HTTP Request Timeout of 30s. It would be better to export asynchronously and then to provide a download link once the export is done, like other applications do.


fix: handle unavail asset for approve tx

Description
Adds error handling for the AprovalAmount component usage in Transaction History in order to avoid an app crash when we cannot retrieve the asset information. We do not display any amount in this case, without displaying an error to the user, the "raw" value we have is not very significant to the user without the precision/symbol (and hard to display, especially for "infinite" amount due to their length in this case). The link to the block explorer will potentially give them more details on the asset/value in this regard.

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #3774
Risk
None.

Testing
Verify that amounts are still displayed in general for Approve transactions in the Transaction History, and for entries missing a Symbol for the Approved Asset, no amount should be displayed and the app should not crash anymore.

Engineering
☝

Operations
☝

Screenshots (if applicable)

After discussing with Engineering and Product we've settled to displaying "Unavailable" as the amount in the case of assets that can't be found by the AssetService. No more custom error is raised when the asset is missing, instead we handle it at the time we attempt to format the amount.
Updated screenshot:

After discussing with Engineering and Product we've settled to displaying "Unavailable" as the amount in the case of assets that can't be found by AssetService. No more custom error raised when the asset is missing, instead we handle it at the time we attempt to format the amount.
Updated screenshot:

That's awesome, definitely agree on the approach here. I'd like to see more of this kind of error handling for assets where we can.

Do you have steps to reproduce the unhappy path e.g which asset produces this?

Besides having an Approve transaction for an asset that isn't found by AssetService, not really. The asset that made me discover this odd bug is in the issue #3774 : eip155:1/erc20:0xa478c2975ab1ea89e8196811f51a7b7ade33eb11 (a LP token).


GitHub: Moved github docs into .github folder

Summary
N/A
Task Checklist
N/A
Steps To Validate Changes
N/A
Known Issues
N/A
PR Checklist

[ ] Development: N/A
[ ] Approval: Have discussed with and got approval from the team[^1]
[x] Tests: Tested accessibility on the local end
[x] Deployment: Deployed the app on the local end
[ ] Localization: Modified en-US string resources[^2]
[ ] Co-authors: N/A

Screenshots
N/A
[^1]: If the request hasn't been agreed by the team, this work might be rejected. Make sure that you get approval from the team before opening any PR related the request.
[^2]: If you removed any en-US string resources, make sure that there are no references of those resources. When you add a new en-US string resources, do not make any changes on other languages' resources; Crowdin will do that, instead.

I haven't changed that many lines; just diff have been broken.

I haven't changed that many lines; just diff have been broken.

Which lines have been changed?

NOTICE.md
Nothing. Just moved.
rich-commands.md
Added title and description placeholder.


Exercises-w2

Exercises
Name
Sajjad Ali
Class
202
GitHub username
https://github.com/Sajjad0405
Bootcamp exercises
https://github.com/Sajjad0405/js-bootcamp
Bootcamp notes
https://github.com/Sajjad0405/js-bootcamp

Your exercises have been checked and your grade is added to the admin sheet. Where are your notes?
Results in the next lesson!


MudSetpper click button step bypass validation

Hi,
if you clickc button step the validation is bypassed.

Best regards

News?

Will fix probably with the next release


Add basic infrastructure for future tracing work.

Tracing is configured with the optional DAP_LOG worker variable.
All existing logging has been converted to the tracing system.

hmm, probably DAP_LOG should be DAP_TRACING :)


x/tools/cmd/gopls: does not compile at latest

What version of Go are you using (go version)?

$ go version 1.12

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/wdy/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/wdy/pkg:/home/wdy/gitlab/vt/pkg"
GOPROXY="https://goproxy.cn"
GORACE=""
GOROOT="/home/wdy/go"
GOTMPDIR=""
GOTOOLDIR="/home/wdy/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/wdy/gopls/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build158961015=/tmp/go-build -gno-record-gcc-switches"

What did you do?

What did you expect to see?
succeed
What did you see instead?
go get -u -v golang.org/x/tools/gopls@latest
Fetching https://goproxy.cn/golang.org/x/tools/gopls/@v/list
Fetching https://goproxy.cn/golang.org/x/tools/@v/list
Fetching https://goproxy.cn/golang.org/x/sync/@v/list
Fetching https://goproxy.cn/golang.org/x/net/@v/list
Fetching https://goproxy.cn/golang.org/x/text/@v/list
Fetching https://goproxy.cn/golang.org/x/crypto/@v/list
Fetching https://goproxy.cn/golang.org/x/sys/@v/list
go: finding golang.org/x/sync latest
Fetching https://goproxy.cn/golang.org/x/sync/@latest
go: finding golang.org/x/crypto latest
Fetching https://goproxy.cn/golang.org/x/crypto/@latest
go: finding golang.org/x/net latest
Fetching https://goproxy.cn/golang.org/x/net/@latest
go: finding golang.org/x/sys latest
Fetching https://goproxy.cn/golang.org/x/sys/@latest
go: finding golang.org/x/tools latest
Fetching https://goproxy.cn/golang.org/x/tools/@latest
golang.org/x/tools/gopls
golang.org/x/tools/gopls
../pkg/pkg/mod/golang.org/x/tools/gopls@v0.1.3/main.go:20:41: not enough arguments in call to cmd.New
have (string, nil)
want (string, string, []string)

not enough arguments in call to cmd.New

cc @stamblerre @ianthehat

@kingeasternsun: Does the command work without -u? The @latest tag is a specific version, but adding -u gets the master of the dependencies, which could be incompatible.

@stamblerre same issue, with and without -u
go get golang.org/x/tools/gopls@latest
# golang.org/x/tools/gopls
../../../../go/pkg/mod/golang.org/x/tools/gopls@v0.1.3/main.go:20:41: not enough arguments in call to cmd.New
        have (string, nil)
        want (string, string, []string)

if you performed the command in a go module, be sure to delete the old dependencies before rerunning go get golang.org/x/tools/gopls@latest. that worked for me.

@deanveloper is there a single command to do that?

go mod tidy; go get golang.org/x/tools/gopls@latest should work

Thanks @deanveloper! Closing this issue.

@stamblerre  @deanveloper
Not working , still the same error!


Change the music player background to a nice looking solid colored/interactive small sized gif image.

Hey @Ayush-projects can you please Assign this issue to me. I am interested to work on this?

Assigned !!!


Fix download context

What changes are proposed in this pull request?
fix TypeError: descriptor '_download_context' for 'SampleCollection' objects doesn't apply to a 'DatasetView' object
repro:
model = foz.load_zoo_model("clip-vit-base32-torch")
q= q=fo.load_dataset('quickstart')
q.compute_embeddings(model, embeddings_field="clip-embeds")

How is this patch tested? If it is not, please explain why.
tested locally
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Summary by CodeRabbit

New Features

Enhanced management of download contexts in the dataset view.

Chores

Updated the version constraint for the fiftyone-brain dependency to allow for new features and fixes.

there is no download context in this (oss) repository


Optional string keys set with attribute? ignore input values

Structs with optional string keys defined via attribute? ... treat these keys as unknown. Symbolic keys work fine.
To Reproduce
class MyStruct < Dry::Struct
  schema schema.strict
  attribute "attrOne", Types::String
  attribute? "attrTwo", Types::String
  attribute? "attrThree", Types::String.optional
end

struct = MyStruct.new("attrOne" => "value1", "attrTwo" => "value2")
# => Dry::Struct::Error: [MyStruct.new] unexpected keys ["attrTwo"] in Hash input

Thanks to schema schema.strict I noticed this behaviour in my deeply nested structs.
Without it the structure swallows optional string keys silently and set values to nil.
Expected behavior
struct = MyStruct.new("attrOne" => "value1", "attrTwo" => "value2")
# => #<MyStruct attrOne="value1" attrTwo="value2" attrThree=nil>

struct = MyStruct.new("attrOne" => "value1")
# => #<MyStruct attrOne="value1" attrThree=nil>
# NOTE: imo `struct.to_hash` should not contain `"attrTwo" => nil` bc the key is defined as optional

Your environment

Affects my production application: YES
Ruby version: 2.6.5
OS: Official ruby Docker container

String keys are not supported and never were. You can define a transformation for keys like transform_keys(&:to_sym) that will turn strings into symbols. I'll report an issue to dry-types to reject schemas with non-symbol keys to make it clear.


Optimize variadic and/or.

This PR implements an optimization reducing variadic and and or to 4 constraints. When four or more elements are supplied to and! or or! macros, the variadic versions (which take a vector argument under the hood) are used.
This only saves 35 constraints and 55 inputs, but oh well.

bors merge

bors cancel

bors merge (this has no reviewers, in particular no codeowners, so it should be 100% rejected)

bors merge

bors merge (this has no reviewers, in particular no codeowners, so it should be 100% rejected)

Reminder: my bors merge command is a positive review!


Fix displaying of submitted response copy on summary page

What is the context of this PR?
When viewing the summary page of a schema with submitted responses, there's supposed to be a bit of copy telling the user they can print/copy their answers
How to review
In dev_settings.sh set EQ_DYNAMODB_ENABLED=True. Run test_view_submitted_response.json and go through to the summary, at the bottom you should see the copy 'You will have the opportunity...'.  Try a different survey and see that the copy isn't there.

@kierenbeckett once the conflicts are resolved this can be squashed and merged.


Clarifying log entries about app reopen and blocking process behavior…

… when updateTool is used
Code comments and log entries make it unclear that BLOCKING_PROCESS_ACTION is ignored when a label uses updateTool. Also clarified log entry regarding reopening apps.

thank you!


Feature Request: Play Videos in the Background

It would be awesome if YouTube videos or any videos could be played in the background.

For media (streaming video, etc.) I specifically made the fork https://github.com/tobykurien/WebMediaShare which allows you to extract the playlist/stream URL and share to VLC or Kodi. I initially wanted to add this to WebApps, but media sites work differently (many 3rd party dependencies, popups and redirects, etc.), hence the fork.

That works for me! Thank you. Now I can cast to my TV too. :D


Fix JS merge crashes from lovefield

Merge enum with expando.
Merge enum member with property assignment.
Merge interface-declared method declaration with
prototype-property-assignment method declaration.

The reason that the enum merges crash is that getTypeOfSymbol assumes that symbol flags are (basically) mutually exclusive. This assumption is shredded, badly, for JS merges.
One fix is to drop the assumption of exclusivity and instead order cases by least to most likely. This has the highest chance of working, but is also slow, since you would prefer to order cases by most likely first, not last.
The other fix, which is what I did here, is to add a last-chance re-dispatch at the bottom of getTypeOfVariableOrParameterOrPropertyWorker. This dispatch uses the valueDeclaration instead of the symbol flags.
Fixes #27352

Also fixes crash from phonegap.


Clean up CI/CD workflows

Rewrote workflows to be more concise, leveraging the setup-miniconda GitHub Action.
Changes:

Updated to use setup-miniconda
Removed now obsolete scripts
Separated linting into its own file
Updated workflow triggers for linting and testing

In addition to the failing test in master, the mac tests aren't passing for JIT compilation due to the removal of this line:
https://github.com/OceanParcels/parcels/blob/56e988ae0f627ec8994c656f5f4bab71154a2e47/.github/scripts/osx_unittest.sh#L5
This can be hardcoded into the action by checking the platform, which I'm happy to do in the coming days.

Testing is now working on mac. Ready for feedback (sorry for the noise)

Thanks @VeckoTheGecko, for this great improvement in the testing functionality!
A few quick comments:

[ ] The integration-tests doesn't seem to work yet?
[ ] Are the three warnings at e.g. Unittesting on ubuntu with python latest
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/upload-artifact@v2 something to worry about? Something that can be fixed?
[ ] There still seems to be a lot of code duplication between the unit-tests and the integration-tests; can that not be streamlined in some way?
[ ] I like the report in html a lot! Didn't know that was possible. But any way to not have it be zipped before downloading? (so that it's even simpler to open it)

[ ]  The integration-tests doesn't seem to work yet?

Not sure what happened there. Seems like they were skipped for some reason. We can check again on the next run.

[ ]  Are the three warnings ...

Easy fix. Thanks for raising that

[ ]  There still seems to be a lot of code duplication between the unit-tests and the integration-tests; can that not be streamlined in some way?

Actually, yes! Just went down a bit of a rabbit hole into composite actions, and updated accordingly. It should be working, let me know what you think.

[ ]  I like the report in html a lot! Didn't know that was possible. But any way to not have it be zipped before downloading? (so that it's even simpler to open it)

All artifacts are zipped, so you can't just download plain HTML (or "open in new tab"). I know for coverage there are platforms to upload coverage reports. There could be similar platforms for testing, but I think it might not be worth it. Runs will hopefully only fail occasionally (and when they do, downloading the report isn't much overhead).
Speaking of coverage, I'll implement it sometime after this PR has merged.

On another note, I think the Windows builds are all failing without logs (even on PR #1273, prior to the workflow changes). Would it be worth disabling Windows builds? I'm not sure what the problem is (relevant issue)

I've now fixed the vonmises issue in PR #1279, but don't get green ticks with Windows because of a strange error
D:\a\parcels\parcels>call activate parcels
[446](https://github.com/OceanParcels/parcels/actions/runs/3879215870/jobs/6624344113#step:3:447)

[447](https://github.com/OceanParcels/parcels/actions/runs/3879215870/jobs/6624344113#step:3:448)
EnvironmentNameNotFound: Could not find conda environment: parcels
[448](https://github.com/OceanParcels/parcels/actions/runs/3879215870/jobs/6624344113#step:3:449)
You can list all discoverable environments with `conda info --envs`.

@VeckoTheGecko, can you try to merge #1279 into this one, to see if it works then?

Looks like the tests are still failing on Windows, and I don't know if this will be a simple fix. If the macos/linux tests pass, I think we should:

revert memory swap fix
remove Windows from testing matrix
create new issue for Windows CI/CD
merge this PR if everything else is in order

I've now fixed it in #1274 (with the solution at https://github.com/OceanParcels/parcels/pull/1279/commits/36cdbd3af36270505caf3a797ad02db0cef4ab90); can you implement that too here? That should give us green ticks for CI?

Tests should pass now. If all is in order, feel free to squash and merge this (as most commits here are just noise).


why not let channel_identifier be unique ?

why not let channel_identifier=sha3(participant1,participant2,address(tokenNetwork) ?
so  channel identidifier can be used to as the id of a channel.  I thinks is useful for implementation of raiden network.
if  channel_identifier=sha3(participant1,participant2), there will be many channels with the same identifier, so we have to use channel_identifier and tokenNetwork as the id of a channel.

@nkbai , we have limited the number of channels. At the moment, two nodes in the system can only have one channel between them.
Please refer to https://github.com/raiden-network/spec/issues/55 and add there any pro's and con's that you have regarding our approach.
I will close this issue for now, as this is a spec related decision.
For the record, going from multiple channels between two nodes to only one was implemented in this PR: https://github.com/raiden-network/raiden-contracts/pull/58

    function getChannelIdentifier(address participant, address partner)
        pure
        public
        returns (bytes32)
    {
        require(participant != 0x0);
        require(partner != 0x0);

        // Participant addresses must be different
        require(participant != partner);

        // Lexicographic order of the channel addresses
        // This limits the number of channels that can be opened between two nodes to 1.
        if (participant < partner) {
            return keccak256(participant, partner,address(this));
        } else {
            return keccak256(partner, participant,address(this));
        }
    }

@loredanacirstea   return keccak256(participant, partner,address(this));
this is what i mean, two partners can only open one channel on the same token.

channel_identifier=sha3(participant1,participant2), there will be many channels with the same identifier

I see no problem here. For all the signed messages we always use channel_id + token_network_address + chain_id for replay attacks.

two partners can only open one channel on the same token

Correct. This is intended. Please check the spec issue I mentioned above.

I have already seen these spec above,
for example:
when i want to close a channel:
PATCH /api/<version>/channels/<channel_address>
this channel_address should be the channel_identifier above.
and if A and B open two channels on Token1 and Token2, they will have two channels with the same channel_identifier.
it's  confused for users. why not make  channle_ideitifer be unique?  and it follows all your spec

@nkbai , the Raiden APIs will also be changed and the TokenNetwork contract address will be added.
What you propose adds no benefit to the smart contract itself. The smart contract knows its address and there is no confusion in its logic. I think adding the contract address to the channel_identifier is redundant, as our goal with the contracts is to simplify logic.
Any other confusion can be handled outside of the contract. E.g. API paths that are clear. If something like PATCH /api/<version>/<token_network_address>/channels/<channel_identifier> is not clear enough, you can propose us to work with keccak256(channel_identifier, token_network_address) in the Raiden client implemention. Because we are starting to integrate the new contracts there.

@loredanacirstea if channel_identifier=sha3(participant1,participant2,tokennetworkaddress), then all the recover.. function (for example recoverAddressFromBalanceProof), doesn't need verify address(this) . I think my method would save more gas.
    function getChannelIdentifier(address participant, address partner)
        pure
        public
        returns (bytes32)
    {
        require(participant != 0x0);
        require(partner != 0x0);

        // Participant addresses must be different
        require(participant != partner);

        // Lexicographic order of the channel addresses
        // This limits the number of channels that can be opened between two nodes to 1.
        if (participant < partner) {
            return keccak256(abi.encodePacked(participant, partner,address(this)));
        } else {
            return keccak256(abi.encodePacked(partner, participant,address(this)));
        }
    }

 function recoverAddressFromBalanceProof(
        bytes32 channel_identifier,
        bytes32 balance_hash,
        uint256 nonce,
        bytes32 additional_hash,
        bytes signature
    )
        view
        internal
        returns (address signature_address)
    {
        bytes32 message_hash = keccak256(abi.encodePacked(
            balance_hash,
            nonce,
            additional_hash,
            channel_identifier,
           // address(this), no need this address  because channel_identifier already contains this information.
            chain_id
        ));

        signature_address = ECVerify.ecverify(message_hash, signature);
    }

this id have two advantages:

save gas (for cooperativeSettle,updateNonClosingBalanceProof,setTotalWithdraw)
more friendly for raiden app implementation. only one variable channel_identifier would identify a channel, no need of tokennetwork address


Setting a section to false will cause a crash

Description
I've disabled stashes in my config like so:
require('neogit').setup {
    -- kind="floating",
    disable_commit_confirmation = true,
    disable_hint = true,
    integrations = {diffview = true},
    sections = {stashes = false}
}

In a recent update this gave me:
Error detected while processing /home/theodorc/.config/nvim/init.vim:
line   14:
E5108: Error executing lua ...nvim/site/pack/packer/start/neogit/lua/neogit/config.lua:398: attempt to in
dex local 'section' (a boolean value)
stack traceback:
        ...nvim/site/pack/packer/start/neogit/lua/neogit/config.lua:398: in function 'validate_sections'
        ...nvim/site/pack/packer/start/neogit/lua/neogit/config.lua:568: in function 'validate_config'
        ...nvim/site/pack/packer/start/neogit/lua/neogit/config.lua:597: in function 'setup'
        .../share/nvim/site/pack/packer/start/neogit/lua/neogit.lua:32: in function 'setup'
        /home/theodorc/.config/nvim/lua/plugin_conf.lua:43: in main chunk
        [C]: in function 'require'
        [string ":lua"]:4: in main chunk
Press ENTER or type command to continue

Using the {folded=false} option fixes the crash, but not my issue. The README indicates that setting the whole section to false will not render the section
Neovim version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/__w/neovim/neovim/build/nvim.AppDir/usr/share/nvim"
Run :checkhealth for more info
Operating system and version
Ubuntu 22.04.1 LTS
(Very pleased with the plugin btw!)

Yup, sorry about that. You can now set { hidden = true } for the section to hide it.

No problem. Thanks!


evaluate2.sh: handle case of prepare file with no sdk use ... call

Closes #283
In block that installs SDKs, grep returns exit code 1 when no match. Add || true to prevent the script from exiting early.

cc @gunnarmorling

Excellent, thx!

@gunnarmorling FYI this runs all forks:
./evaluate2.sh `ls -1 ./calculate_average_*.sh | cut -d_ -f3 | cut -d. -f1`

I tested this (using a tiny measurements.txt script + adding timeout 1 before ./calculate_average_.sh) and found another bug: 1730ad1
So, the script works on my machine at least for all forks! If you want to try running all forks on the eval machine, I recommend adding timeout like this to catch forks that hang forever:
    numactl --physcpubind=0-7 hyperfine $HYPERFINE_OPTS "./calculate_average_$fork.sh 2>&1"

to
    timeout 600 numactl --physcpubind=0-7 hyperfine $HYPERFINE_OPTS "./calculate_average_$fork.sh 2>&1"

When a command fails (via timeout or other failure caught by hyperfine), this message appears and the $fork is excluded from the Summary/Leaderboard same as if the output did not match:


[BUG] Tool tip for Backup action in volume details page is incorrect

Describe the bug
Tool tip for Backup action in volume details page is incorrect
To Reproduce

Create a volume
Go to the volume details page
The tool tip is incorrect

Expected behavior
the tool tip should display appropriate message for Backup Action
Environment:

Longhorn version: master - 04/21/2020
Kubernetes version: 1.17.4
Node OS type and version: DO rke Linux

Validation: PASSED
Longhorn UI versio: wangsiye/longhorn-ui:82eb981


Convert dependency management to glide

Summary:
It is tedious for (glide based) projects depending on m3db to
also keep in sync the dependencies of m3x, m3cluster, and any other
deps of m3db. Using glide allows other projects to use this code much
more easily.
Also it eliminates the usage of git submodules which is almost always
the right choice...
Test Plan:
rm -rf ./vendor
make install-ci
make

Coverage decreased (-0.01%) to 80.062% when pulling 951e1aca58562dc23faa3c3a90f8f817a7b497dc on jra3/glide into f1db6730630eb7e39355608ce85c424b6539b2be on master.

Coverage decreased (-0.01%) to 80.062% when pulling 951e1aca58562dc23faa3c3a90f8f817a7b497dc on jra3/glide into f1db6730630eb7e39355608ce85c424b6539b2be on master.

a 0.01% coverage regression counts as a failure for coveralls. Boooooooo.

Tested this by running make install-license-bin
Note that this command uses the proper american spelling

Coverage remained the same at 80.074% when pulling a77e4100635deee96d8a19f590b006eb1b2e2232 on jra3/glide into f1db6730630eb7e39355608ce85c424b6539b2be on master.

Coverage remained the same at 80.074% when pulling a77e4100635deee96d8a19f590b006eb1b2e2232 on jra3/glide into f1db6730630eb7e39355608ce85c424b6539b2be on master.

LGTM

LGTM


Setting HOME messes up expected directories

When you set the HOME env variable (I do this on Windows so I can put my dotfiles where I want them), the plugins unzips the wpilib folder to user home, not where I set it to. Then, actually building a program expects the wpilib folder to be in where I've set HOME to, not the user home. So I have two directories:

C:\Users\333fr\wpilib - Where the wpilib folder is unzipped to.
C:\otherdir\wpilib - Where the eventual build expects to find the wpilib folder.

I assume the desired behavior here is to respect the env variable for the install?

Yes, I think so.

Yeah. Ideally, I should also be able to set an env variable like WPILIB_INSTALL_DIR or something of the like that and have it install there.

Isn't this why I added vagrant?
Does this work there?
Or would this be a build artifact has this problem?

This has nothing to do with vagrant. This an issue with the plugins, not the build.

The problem is the Eclipse plugins installing the wpilib files on users machine. I don't think vagrant helps there.
Somewhere we must be using different variables to figure out where to install vs where to look for the files when building. I think it will be easy to find and fix, but I want to get the 3rd party stuff done before I look at it.
-------- Original message --------
From: Jonathan Leitschuh<EMAIL_ADDRESS>Date: 10/11/16 12:56 PM (GMT-05:00)
To: wpilibsuite/EclipsePlugins<EMAIL_ADDRESS>Cc: Kevin OConnor<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [wpilibsuite/EclipsePlugins] Setting HOME messes up expected directories (#15)
Isn't this why I added vagrant?
Does this work there?
Or would this be a build artifact has this problem?

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/wpilibsuite/EclipsePlugins/issues/15#issuecomment-252977058, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALOgBhlDJr1o6Ddy1LNipcf2NIsV72Czks5qy797gaJpZM4KPnYF.

I think doing both "WPILIB_DIR" or whatever, and respecting HOME and using user.home if neither are there starts to get a bit messy. If you had to pick between "WPILIB_DIR" and "HOME", with "HOME" being the easier to implement, what are your thoughts?

WPILIB_DIR, without a doubt. The only reason I have HOME set is so that I can easily share my dotfiles among my Windows computers, and it doesn't really break anything on Windows. It'll break a lot on anything 'Nix based.

Ok, I'll try to figure out the one issue left to do that one then.
-------- Original message --------
From: Fred Silberberg<EMAIL_ADDRESS>Date: 10/18/16 6:15 PM (GMT-05:00)
To: wpilibsuite/EclipsePlugins<EMAIL_ADDRESS>Cc: Kevin OConnor<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [wpilibsuite/EclipsePlugins] Setting HOME messes up expected directories (#15)
WPILIB_DIR, without a doubt. The only reason I have HOME set is so that I can easily share my dotfiles among my Windows computers, and it doesn't really break anything on Windows. It'll break a lot on anything 'Nix based.

You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/wpilibsuite/EclipsePlugins/issues/15#issuecomment-254655253, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALOgBvmGQqF7lowHuX1QDOwMkVO7L0MNks5q1UTggaJpZM4KPnYF.

So, after doing some work on this one, allowing a different install location opens up a can of worms that I am not convinced is worth it.
Specifically, it complicates installer creation for 3rd party installers, requires these installers to be re-run if the var is created or changed after they are installed and requires a DS change to be able to auto-launch the SmartDashboard from multiple possible locations.
Considering all of that, I think the appropriate fix here is to always install to user.home and fix what is currently causing breakage in Fred's scenario of HOME being set on Windows.

Ok. If we fix that, it'll at least make builds work on my machine without having to mess with settings.

I have this working for new Eclipse installs (or existing if you delete the Eclipse variable for WPILIB in Window->Preferences->Run/Debug->String Substitution and restart Eclipse after starting the plugins). Is that good enough or do you want me to work on adding something to reset this for existing installs?

Confirmed working on my machine. Thanks Kevin.


LCFS - Compliance report, internal workflow, Recommended by manager status, IDIR Director role

Compliance report, internal workflow, Recommended by manager status, IDIR Director role
Description:
The report will have two buttons to Issue assessment and Return to compliance manager. IDIR users in Analyst or Compliance manager roles will not see these buttons.
Once the Director has clicked the Issue assessment button the report is put into Assessed status, this transaction is recorded in the report history section above the compliance manger recommend info as per mock-up. Note the BCeID version displays different text.
IDIR version
•	Assessed [date time stamp] PST by [IDIR director first name, last name].
BCeID version (government staff names are not displayed on BCeID version, the Recommended history is also not displayed, see card #1006 )
•	Assessed [date time stamp] PST by the director under the Low Carbon Fuels Act.
The transaction ledger should be updated to capture the change in the suppliers compliance unit balance.
If the director clicks the return button the report will be put back into Recommended by analyst status for the compliance manger to action further. This action is not recorded in the report history. When the compliance manager recommends the report for the second time the original recommended history entry will be updated. This could be a different compliance manager so the name should also be updated if required.
Wireframe:
https://preview.uxpin.com/e0e2f15fc1b52987fd0287ad8c8103452dd16d60#/pages/186169277
Purpose and benefit to user:
The purpose of this workflow is to facilitate the review process for compliance reports by adding decision-making capabilities for the IDIR Director. The Director can either assess or return reports to a compliance manager, and the history of these actions will be appropriately recorded. This improves auditability and ensures that compliance reports are handled efficiently.
Acceptance Criteria:

[ ] Two buttons are available on the compliance report for IDIR Director role: Issue assessment and Return to compliance manager.
[ ] The two buttons are only visible to IDIR Directors and not to users with Analyst or Compliance manager roles.
[ ] Clicking the Issue assessment button changes the report status to "Assessed."
[ ] A transaction is recorded in the report history section above the compliance manager recommendation, showing:
Assessed <date time stamp> PST by <IDIR director first name, last name>.
[ ] Clicking the Return to compliance manager button changes the report status to "Recommended by analyst."
[ ] No transaction is recorded in the report history for the return action.
[ ] If the Compliance manager recommends the report again after it was returned, the original recommendation entry is updated, including the compliance manager’s name (if different).

Development Checklist:

[ ] Implement conditional visibility for Issue assessment and Return to compliance manager buttons, visible only to users with IDIR Director role.
[ ] Create functionality to change the report status to "Assessed" when the Issue Assessment button is clicked.
[ ] Log the transaction in the report history with a date, timestamp, and IDIR Director’s name.
[ ] Create functionality to revert the report status to "Recommended by analyst" when the Return to compliance manager button is clicked, without logging the action.
[ ] Update the Compliance manager recommendation entry upon subsequent recommendations, ensuring the manager's name is updated if necessary.

Notes:

Ensure that the workflow follows the role-specific visibility for actions. The history log should be automatically updated upon assessment but not for the return action.

Date/Times are displayed in PST

Everything works great but the Issue assessment button gives a 500 error

The 500 error has been resolved, but the "issue assessment" and "return to compliance manager" buttons are missing for the director on the compliance report that has been recommended by the manager.


Cannot run GridDB docker in Fixed List mode

Running GridDB docker in fixed list mode e.g.
docker run -dt --network="host" -e NOTIFICATION_MEMBER=1 griddb/griddb:latest
leads to the following error:
Fixed List mode
Execution failed. See message logs
/start-griddb.sh: line 94: exec: griddb: not found

Had no issue running in multicast mode.
Specifications: Windows 10, Docker Desktop Docker version 24.0.2.

I have updated docker image for fixed list mode recently.
Could you try it ?

docker run -dt --network="host" -e NOTIFICATION_MEMBER=1 griddb/griddb:latest


Failure to replicate results？

Use the veri data set to train and evaluate. The results are as follows：
2023-10-24 21:53:58,401 transreid.test INFO: Validation Results
2023-10-24 21:53:58,402 transreid.test INFO: mAP: 75.5%
2023-10-24 21:53:58,402 transreid.test INFO: CMC curve, Rank-1  :92.0%
2023-10-24 21:53:58,402 transreid.test INFO: CMC curve, Rank-5  :94.4%
2023-10-24 21:53:58,402 transreid.test INFO: CMC curve, Rank-10 :95.9%
Rank_1 is 4.8% lower than in the paper...I double-checked the configs and ensured that the experiment settings were identical.
For reference, I'm attaching the training logs of the models:
train_log_cnn_prom_veri.txt

Traceback (most recent call last):
File "/media/lele/c/zuozhigang/CLIP_ReID/Base/train_clipreid.py", line 89, in
do_train_stage2(
File "/media/lele/c/zuozhigang/CLIP_ReID/Base/processor/processor_clipreid_stage2.py", line 98, in do_train_stage2
loss = loss_fn(score, feat, target, target_cam, logits)
TypeError: loss_func() takes 3 positional arguments but 5 were given
请问一下，你们有没有遇到这个问题，如果有是怎么解决的？


remove coverage.out

Stack:

#46
#45
#44 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #46


Update AWS Batch Executor to use AWSExecutor

Acceptance Criteria

[ ] Pass any required attributes from credentials_file, region, s3_bucket_name, profile, execution_role, and log_group_name to AWSExecutor super class
[ ] Re-structure executor in order to accord with abstract class methods as defined in RemoteExecutor, namely _upload_task, submit_task, get_status, _poll_task, _query_result, and cancel when appropriate
[ ] Update unit tests for executor
[ ] Update doc strings for executor

Closed as duplicate of https://github.com/AgnostiqHQ/covalent-awsbatch-plugin/issues/25


Content: Kruskal's Algorithm

Description
Add the article for Kruskal's algorithm. Use the following structure:

Introduction
Explanation
Algorithm
Code
Sample I/O
Complexity Analysis

@HarshCasper Hey, I would like to work on this. Thank you :)


Docs fix for using internal trackers

What does this PR do?
Fixed a minor mistake!
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


(PUP-7601) Remove LDAP Terminus and related tests

jenkins please test this

CLA signed by all contributors.


Introduce nodeFilter Predicate to filter Partitions

Bug Report
Current Behavior
When Lettuce is connected to a 2 node Redis Cluster (1 shard, 1 replica), and is configured for REPLICA_PREFERRED, and the replica ceases responding to TCP (such as a ungraceful hardware failure), Lettuce does not recover until the TCP retry counter expires for that connection (~926 seconds).
Input Code

Input Code
(Assumes the hostname `redis` resolves to all nodes in the group)
import io.lettuce.core.cluster.*;
import io.lettuce.core.cluster.api.sync.*;
import io.lettuce.core.cluster.api.*;
import io.lettuce.core.*;
import java.util.concurrent.TimeUnit;
import java.time.Duration;

public class RedisExample {
    public static void main(String[] args) {
        RedisURI redisUri = RedisURI.Builder.redis("redis").build();

        RedisClusterClient clusterClient = RedisClusterClient.create(redisUri);

        ClusterTopologyRefreshOptions topologyRefreshOptions = ClusterTopologyRefreshOptions
           .builder()
           .enablePeriodicRefresh(60, TimeUnit.SECONDS)
           .enableAllAdaptiveRefreshTriggers()
           .dynamicRefreshSources(true)
           .closeStaleConnections(true)
           .build();
       TimeoutOptions  timeoutOptions = TimeoutOptions
          .builder()
          .timeoutCommands()
          .fixedTimeout(Duration.ofMillis(400))
          .build();

        SocketOptions socketOptions = SocketOptions
          .builder()
          .connectTimeout(500, TimeUnit.MILLISECONDS)
          .build();

        clusterClient.setOptions(ClusterClientOptions
            .builder()
            .autoReconnect(true)
            .socketOptions(socketOptions)
            .cancelCommandsOnReconnectFailure(true)
            .timeoutOptions(timeoutOptions)
            .disconnectedBehavior(ClientOptions.DisconnectedBehavior.REJECT_COMMANDS)
            .topologyRefreshOptions(topologyRefreshOptions)
            .validateClusterNodeMembership(true)
            .suspendReconnectOnProtocolFailure(true)
            .build());

        StatefulRedisClusterConnection<String, String> connection = clusterClient.connect();
        RedisAdvancedClusterCommands<String, String> syncCommands = connection.sync();
        connection.setReadFrom(ReadFrom.REPLICA_PREFERRED);

        String value1 = syncCommands.set("foo", "bar");

        while (true) {
         try {
          Thread.sleep(1000);
          String value = syncCommands.get("foo");
          if (! value.equals(value2)) {
            System.out.println("bad response:" + value + ":" + value2 + ":");
          } else {
            System.out.println("Good response: " + value);
          }
         } catch (Exception e) {
            System.out.println("Error response: " + e);
         }
       }
    }
}

Expected behavior/code
When the timeout is reached and a dynamic topology refresh is triggered, connections to the node in "fail?" state should be considered stale and closed / abandoned.
Environment

Lettuce version(s): 6.1.5.RELEASE
Redis server v=6.2.6 sha=00000000:0 malloc=jemalloc-5.1.0 bits=64 build=3f28004270edf9dc
OpenJDK Runtime Environment (build 1.8.0_312-b07)
Amazon Linux 2, 5.10.75-79.358.amzn2.x86_64, t3a.small

Possible Solution
A workaround, undesirable as it is global-to-the-node in nature, is to shorten the TCP retry counter on the client:
echo 5 >/proc/sys/net/ipv4/tcp_retries2

At first glance it looks like adding a filter for failed/eventual_fail nodes at https://github.com/lettuce-io/lettuce-core/blob/cda3be6b9477da790365ad098c6e39c8687f5002/src/main/java/io/lettuce/core/cluster/topology/DefaultClusterTopologyRefresh.java#L292-L296  would cap the duration of the failure scenario at the periodic-topology-refresh interval.
Additional context
TCPdump clearly shows the client getting a topology refresh occurs, but the client does not recover until the existing TCP connection exits.
17:58:54.396570 IP <IP_ADDRESS>.6379 > <IP_ADDRESS>.46894: Flags [P.], seq 150:428, ack 117, win 490, options [nop,nop,TS val<PHONE_NUMBER> ecr 121987982], length 278: RESP "=270" "txt:e03b0b3b56ca33dc759fb6a122a903c7ac47d8f7 <IP_ADDRESS>:6379@16379 myself,master - 0 0 0 connected 0-16383" "215c649d39c0182c82aec8fc7e533cd57c052b9a <IP_ADDRESS>:6379@16379 slave,fail? e03b0b3b56ca33dc759fb6a122a903c7ac47d8f7<PHONE_NUMBER>742<PHONE_NUMBER>738 0 connected"

Failure of the replica node was simulated by dropping all Redis packets on the replica:
$ for x in INPUT OUTPUT; do for y in 6379 16379; do iptables -I $x -p tcp --dport $y -j DROP; done; done

Redis.conf contains:
port 6379
cluster-enabled yes
cluster-config-file nodes.conf
cluster-node-timeout 5000
appendonly yes
maxmemory 1gb

TCP KeepAlive's do not help here as the connection is not idle.

You can override RedisClusterClient.determinePartitions(…) to post-process which nodes are available in the topology view. The client obtains topology information from all cluster nodes and uses the topology view that makes the most sense. Since each Redis server can have a different perspective on the other nodes, there is no single source of truth but rather requires a consensus to select the most appropriate one. And the view of Redis servers may be different than the perspective of the client.
I'm not sure whether TCP keep-alive is helping in this case.

@mp911de Yeah I don't think TCP keep-alive helps either, but I wanted to call out that I had looked at it as a possibility.

I tested reworking https://github.com/lettuce-io/lettuce-core/blob/4110f2820766c4967951639aa2b6bdd9d50466be/src/main/java/io/lettuce/core/cluster/RedisClusterClient.java#L1032 to filter out FAIL and EVENTUAL_FAIL nodes and achieved the same recovery after the periodic-topology-refresh interval.

I wonder whether it makes generally sense to introduce a node filter (Predicate<RedisClusterNode>) that is used to filter Partitions. It's easier to provide a config value than subclassing the client.

I like the sound of that over a subclass.  I did try calling removeIf against the Partitions collection with a predicate and it threw a unimplemented-error, so I guess that'd have to be added as well.

That's in place now.

Is this the default behavior in latest lettuce without explicitly specifying the node filters?
If the answer is no, what is the reason for not making this the default behavior?

You can specify a Predicate now. This new functionality allows additional filtering so that we do not break existing applications.

Thanks for the reason!
Can this be made the default in the next major version? This seems like a good default for most use cases. Folks run into this problem frequently that there is even a guide to set the predicates in AWS's documentation https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/BestPractices.Clients-lettuce.html#:~:text=Set nodeFilter,retrying is exhausted.

Thanks @srgsanky for the background. Can you file a new ticket to flip the default?


[TypeScript][Virtual Assistant] Add botservices parity

Purpose
What is the context of this pull request? Why is it being done?
Add the changes for parity to botservices class

FYI - @matiasroldan6
I implemented the following changes taking into account C# side:

Change the location of the properties declaracion to be at the top of the class
Rename of cognitiveModelSet variable
Rollback of the cognitiveModelSet initialization in order to avoid the undefined values
You were re-generating the luisServices and qnaConfiguration in each iteration of the kbs, I rollbacked the cognitiveModelSet initialization as mentioned above
Remove the initialization of DispatchService and LuisService inside each assignment of each cognitiveModels
Cleanup of getCognitiveModels


Installation Error

Hello. I have installed the omniture library by downloading the repository, going into it and running this command within a conda environment:
python setup.py install
That seems okay but when I try to import the library in Python I get this error:

ModuleNotFoundError                       Traceback (most recent call last)
 in ()
1 #Import omniture library
----> 2 import omniture
~\anaconda\envs\biancasenv\lib\site-packages\omniture-0.3.1-py3.6.egg\omniture_init_.py in ()
1 # encoding: utf-8
2
----> 3 from account import Account, Suite
4 from elements import Value, Element, Segment
5 from query import Query
ModuleNotFoundError: No module named 'account'
Has anyone encountered this before?
I should mention that I am using Python 3.6.5.

It seems the setup file has not been updated in over 2 years and it's pointing to this:
download_url='http://www.github.com/stdbrouw/python-omniture/tarball/master'

Would be best to do the following via terminal:
$ pip install git+http://github.com/dancingcactus/python-omniture.git


Show distance and timer over pokemon annotations

Add Time and Timer labels
Add Distance label
Add settings to switch them

Also, @istornz what do you think about creating some kind of constants file and/or settings helper? There are a lot of string constants for settings and notifications and its easy to misspell.

For now distance is calculated from users position, not manually set point. Im not sure how to handle this case and when to switch between them.

Thx for your feature !
But there are some conflicts... Can you patch that (I don't have time now)

And yeah good idea for constants. It would be very appreciated :)

Can you please point me to conflicts? Im not familiar with merging PRs from other repo w/o write access.

Yes :

Merge conflict in iPokeGo/MapViewController.m
Merge conflict in iPokeGo/Base.lproj/Main.storyboard

I think you started to work on the older version of the project (without radar).
I made a lot of change especially in the mapviewcontroller.m

Yep. Should I pull master to Distance_and_Timer_Features and then fix conflicts?

If you want :)

@istornz any conflicts now?

No thx :)

Great update good work !

@evshade there's a bug with the timer, after the app Goes in background and becomes active again, the timer doesn't update to the real time left. For example: if there's a Pokémon with 10 min left, and I press the home button and 5 min later I open the app again...the timer will be at 10 min left instead of 5... these are screeshots from the web version and the iOS version


map not displaying on iPhone 5S device

map not displaying on iPhone 5S device.  Also, card view and list view are broke as well.  The card and list view will work if the search is changed to trending for example.  or if changed it back to nearby.

#33, #34 and #35 seem to be related to this issue.  I removed the ios platform and added it.  The plugins were installed and I was able to view the map on my iPhone 5S.  Current location was not shown until I used the option in the About view.
Card and list view are a separate issue.


Fix case of GitHub

Checklist

[x] You have read the Contribution Guidlines before creating this PR.

@jbampton
Thanks for contributing to the repo. We can surely improve as per your suggestions. But as per the HacktoberFest guideline, we will not be accepting minor text fixes as a PR. Please suggest and improve some quantifiable changes in the pull request to make your PR eligible for HacktoberFest.


Add support for AKS in Azure China Cloud

Address issue: https://github.com/rancher/rancher/issues/20881
Note: Azure service rejects with "httpApplicationRouting not supported" even if we set the addonProfile to httpApplicationRouting: {Enabled: false}, so we need to drop it for these regions.

LGTM

lgtm


fix: upgrade to version 3.0.722

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign abayer
You can assign the PR to them by writing /assign @abayer in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test vault

gsm
a89f0edfd4b04c6e5dac462db34fd2622aa00a13
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


I can't seem to follow the installation guide.

Patch something, okay put what file where. What directory. is there a command i should have done?
Is this even a guide, or more like pseudo theory talk..

I wrote some instructions in readme. Please let me know if there is something not clear.


Failed to deploy on testnet

Can't find resources to deploy on testnet (mainly storage, a public ipv4 node, a public ipv6 node)
https://github.com/threefoldtech/js-sdk/blob/ef00a2dea91ef5629ba11c486c347ccc6f350841/jumpscale/sals/marketplace/apps_chatflow.py#L92

Are you suggesting there is something wrong with the explorer behavior? The way how the capacity planning is done by the chatflow is not controlled by the explorer.

Are you suggesting there is something wrong with the explorer behavior? The way how the capacity planning is done by the chatflow is not controlled by the explorer.

It's OK but can't find resources.


Would you like to consider migration to kohttp dsl for requests?

https://github.com/alex-yh99/Readhub/blob/2b225e830245d34327cbf80a2c9c8a9ac523cafb/src/com/madfish/ide/util/RHApi.kt#L69
kohttp team could help with it :)

Thanks Sergey @rybalkinsd
This plugin is not actively maintained, I will try kohttp in other projects :)


Added Chrome and Firefox extension link to documentation

Link to a browser extension that enables using Ollama for interacting with web pages.

Hi thanks for the PR. Possible to rebase?


Reverse render

Hey there, is there a possibility to reverse render as in reverse the gcode parse function to remove that part of rendering while keeping the original render?

Hi there, I'm not sure I understand. Do you want to process the gcode
before rendering? Could you elaborate on your usecase?
Op di 15 jun. 2021 15:02 schreef anupam-tiwari @.***>:

Hey there, is there a possibility to reverse render as in reverse the
gcode parse function to remove that part of rendering while keeping the
original render?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/remcoder/gcode-preview/issues/22, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADQWUUBIM2STHCZ6LS2ZC3TS5FQFANCNFSM46XHHOTQ
.

sure, so what I want to do is take a gcode file and renders the model , which is does fantastically. I wanted to add a reverse function as in when I press the button instead of moving forward and rendering, it will go backward and remove gcode from the rendered model

You can reverse the animation by first parsing and displaying the gcode, and then lowering endLayer in a setTimeout/setInterval loop.
Something like:
 import * as GCodePreview from "gcode-preview";

  const preview = new GCodePreview.WebGLPreview({
      targetId: 'gcode-preview',
  });

  preview.processGCode(gcode);
  preview.render();
  let maxLayer = preview.layers.length + 1
  setInterval(()=> {
      preview.endLayer = --maxLayer;
      preview.render();
  }, 100);

this is perfect, can it be done using  lines instead of layers?

There is no param like maxLine or anything so no.
Another way would be to call preview.processGCode and parse a whole new
gcode fragment. Then every time you can make the gcode fragment shorter.
And in between you should then call preview.clear. This would not really
perform well though so YMMV.
Good luck!
Op ma 28 jun. 2021 23:06 schreef anupam-tiwari @.***>:

this is perfect, can it be done using lines instead of layers?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/remcoder/gcode-preview/issues/22#issuecomment-870041879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADQWUUUQRUL3YI43R5RKLLTVDP5TANCNFSM46XHHOTQ
.

Hi @anupam-tiwari, nice work. I can step through the gcode forwards and backwards. Really neat!
btw, I did notice a glitch I think. Here is see double lines after going backwards & forwards

Oh and it seems the forwards/backwards buttons seem unresponsive after clicking 'load'. What does 'load' do. exactly?
I think actually that I should update gcode-preview to make it easier to build such an interface, what do you think? What would be needed?
Also, for building interfaces on top of gcode-preview, you may be interested in my Vue.js example if you want use Vue.
Anyway I appreciate you reported back with your results, thank you. Good luck taking it further if that's your intention. Feel free to ask any questions about gocde-preview again. :-)


Ignore reformat files/folder

Fixes: #1488

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

@johnaohara I have added this as mendatory and its a part of configuration with prettier and another reason is lock file is still reformated by dependabot PRs.

@shivam-sharma7 I think this PR looks ok, is there anything else you want to add? if not, please can you move out of draft?

@johnaohara Thanks but for now it enough, we will look further.