(Feature) DAI chain support

Relates to https://github.com/poanetwork/metamask-extension/issues/152
DAI chain support is added

@dennis00010011b please actualize e2e tests to support DAI chain.

@vbaranov
Invalid custom RPC can be added

@dennis00010011b Is it reproducible in develop branch? If yes, please open a separate issue.

@dennis00010011b And we only validate accessibility of the provided URL endpoint. We don't check, that it is a valid RPC endpoint actually (see the original feature request https://github.com/poanetwork/metamask-extension/issues/76)

Hey guys,
We are making slight alterations to the name. Dai Chain is now called xDai Chain. The ticket will be xDAI.
Ill make comments along the files here that I can spot will need a change

@vbaranov
There is still 'Dai chain' in dropdown menu and in Settings ->Current Network

@dennis00010011b please, resolve merging conflicts related to e2e tests

@dennis00010011b please take a look to broken e2e tests and also to DeepScan suggestions ☝

@dennis00010011b all tests were passed successfully after merging with develop branch.

Dai Chain is now called xDai Chain. The ticket will be xDAI.

@SiddigZeidan it has been implemented. Is it final naming? Could we proceed to merge this PR to develop branch?

Dai Chain is now called xDai Chain. The ticket will be xDAI.

@SiddigZeidan it has been implemented. Is it final naming? Could we proceed to merge this PR to develop branch?

Yes sir, please proceed and merge


add miit.edu.mm

Domain - miit.edu.mm
School Name(EN) - Myanmar Institute of Information Technology
School Name(MM) - မြန်မာသတင်းအချက်အလက်နည်းပညာတက္ကသိုလ်

@peterzam First stage of verification passed, official domain is miit.edu.mm as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@peterzam Pull request merged. Thank you!


Supporting additional sensors like RotationRate, Attitude and Device Orientation

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

Is it possible to support the additional sensors values like RotationRate, Attitude, Device Orientation?

Describe the solution you'd like

Like the currently supported sensors Gyroscope, Accelerometer, Magnetometer, Barometer, Above mentioned sensors can also be supported

Describe alternatives you've considered

Expo sensors (https://docs.expo.io/versions/latest/sdk/sensors/) provides these requested sensors values along with the sensors supported by react-native-sensors. I've used them using uni_modules but they still missing the timestamp of the event.
While for Attitude I've used react-native-attitude (https://github.com/dpyeates/react-native-attitude).
Expo sensors do not support Attitude values.

Additional context
Described above.
@DanielMSchmidt

Hey @o16-waqas, love it!
Please keep in mind that one of the strict rules of this project is that each sensor need to be supported for both iOS and Android. As long as your PR contains both implementations I am more then happy to review it and get it in 💪

unstale


npm ci broken on node 14

Errors with
npm WARN prepare removing existing node_modules/ before installation
npm ERR! Cannot read property 'n3' of undefined

Seems like the cause is https://github.com/rdfjs/N3.js/issues/317


Performance testing

It may be faster to use a for loop to calculate the factorial, rather than the current method of recursion.
Some basic performance testing should be done to find which is the fastest.

Testing done using the benny package, using the code in the benchmark branch of this repository, was done in Node.js versions v14.17.0, v12.22.1, and v10.24.1
The existing method (recursion) was tested against other factorial implementations using a for and a while loop.
Results were consistent across these versions, and the results from v14.17.0 are below:
Running "Calculate factorials of 1...100" suite...
Progress: 100%

  Recursive function:
    1 925 ops/s, ±1.61%   | fastest

  While loop:
    1 846 ops/s, ±0.64%   | slowest, 4.1% slower

  For loop:
    1 897 ops/s, ±0.52%   | 1.45% slower

Finished 3 cases!
  Fastest: Recursive function
  Slowest: While loop
Running "Calculate factorial of 1717" suite...
Progress: 100%

  Recursive function:
    1 547 ops/s, ±1.17%   | fastest

  While loop:
    522 ops/s, ±2.22%     | slowest, 66.26% slower

  For loop:
    543 ops/s, ±0.58%     | 64.9% slower

Finished 3 cases!
  Fastest: Recursive function
  Slowest: While loop

Running "Calculate factorials of 1...1000" suite...
Progress: 100%

  Recursive function:
    10 ops/s, ±1.02%   | fastest

  While loop:
    4 ops/s, ±0.58%    | slowest, 60% slower

  For loop:
    5 ops/s, ±0.67%    | 50% slower

Finished 3 cases!
  Fastest: Recursive function
  Slowest: While loop
Running "Calculate factorial of 17171" suite...
Progress: 100%

  Recursive function:
    10.7 ops/s, ±0.44%   | fastest

  While loop:
    4.1 ops/s, ±0.43%    | 61.68% slower

  For loop:
    4 ops/s, ±0.73%      | slowest, 62.62% slower

Finished 3 cases!
  Fastest: Recursive function
  Slowest: For loop

The results seem to show that recursion is the most efficient method here.


Send by sdapi/v1/text2img

How to pass parameters in this api ' sdapi/v1/text2img'?

I don't entirely know, I don't work with the webui API much. Might have to learn how soon.
For now, check this issue: https://github.com/mcmonkeyprojects/sd-dynamic-thresholding/issues/20 probably subscribe to notifications there too.
If I find a better solution myself or somebody posts one at any point, it'll get posted into there.


Return to burger menu selection after back on mobile

https://github.com/kodadot/nft-gallery/assets/90852205/82427253-045c-4faa-a9c9-35892d21e60b
After you click back from the profile, you should be on the burger menu again - not closed.

there is also the weird flash of the home page; its not as smooth as for example, with explore etc.

lucky issue number <3

@yangwao maybe it will be luckily solved


ARROW-3495: [Java] Move validityBuffer to BaseValueVector,  delete the optim…

This patch does the following changes:

moves two common function "getNullCount" and "splitAndTransferValidityBuffer" to the top-level BaseValueVector. This change requires moving "validityBuffer" to the BaseValueVector class (as recommended in this TODO: https://github.com/apache/arrow/blob/master/java/vector/src/main/java/org/apache/arrow/vector/BaseFixedWidthVector.java#L89)
optimize the implementation of loadValidityBuffer (in the BaseValueVector) to just pass the reference for the validity buffer instead of optimizing it
optimize for the common boundary condition when all variables are valid (as done in the C++ code: https://github.com/apache/arrow/blob/master/cpp/src/arrow/array.h#L290)

The optimization delivers performance.
Tests: Read 50M integers from a single Int column (2GB).
Before the patch:
Baseline: 7.64 Gb/sec
With the Holder API : 9.99 Gb/sec
After the patch (with the bitmap condition checks)
Baseline: 12.13 Gb/sec (+58.7% gains)
With the Holder API: 16.03 Gb/sec (+60.4% gains)

@azimafroozeh thank you for the pull request. Given the size it is a little bit hard to review, I think it would better to split apart the refactoring from imporvements so a more careful review can be done.  It appears as is, this PR breaks the integration tests.
Second, @jacques-n has mentioned mentioned in the past that redundant code in some cases.  I'd like him to take a look at this to understand if this goes against design principles.
Third, for performance improvements it would be good to specify the existing benchmark that this improve or add a benchmark to the benchmark packages for what you are measuring.

(when splitting up PRs please open up a JIRA per PR).

@emkornfield Thanks for your feedback. I'll split my PR to two PRs.

@azimafroozeh , I am not quite sure of the performance benchmark. What change led to the improved throughput numbers you mentioned in the PR description?
Secondly, I think moving the validityBuffer to the super class might be a bad idea. I was the one who actually added that TODO when Java code was refactored. One pattern that is followed in Arrow is that sometimes not fully sticking to the object oriented principles of arranging code in inheritance hierarchy gives better performance.
In this case, we have moved validityBuffer to the top level super class BaseValueVector. Earlier it was maintained in BaseFixedWidthVector and BaseVariableWidthVector for scalars and within each vector or its immediate super class for complex types. So:
(1) the operations like getValidityBuffer, getValidityBufferAddress(), transferValidityBuffer(), isNull(), isSet() which are actually quite frequent on the hot code paths for data processing were all handled by immediate super classes.
(2) Also, the bit vector operations on the validity buffer in each concrete subclass vector used static methods which can probably be inlined by the JVM. These methods have also been moved to the super class.
I believe with this PR there might be performance degradation since (1) and (2) will no longer be true and there will be an extra indirection to super class method reference. While in general it is good practice w.r.t OOP principles, it comes with performance overhead.
I do agree that having duplicate implementations of splitAndTransferValidityBuffer() within each concrete vector is annoying and should probably be moved to the super class as a static method that takes validityBuffer as an argument May be we should focus on this aspect?
@jacques-n might have more thoughts on this

@azimafroozeh , are you planning to address the comments or we can close the PR for now?

@siddharthteotia Please close it for now.

Closing based on latest comments


Custom headers

Custom headers will help you to setup device_ids, username, password during handshake

Ok! But what about wss?

@pluzmedia : https://pypi.org/project/micropython-async-websocket-client/0.1.7/


Console: use console width in all commands

The console has a setting to specify the console width, but the help command (and possibly some other commands) doesn't use it when printing info to the screen.

Could be solved by #1394


Text formatting is not working

Describe the bug
Bold, Italics and Underline buttons don't anything.
To Reproduce

Run program
Click to one of them

Expected behavior
App should do formatting (I think formatting in .txt files is impossible)
Desktop (please complete the following information):

OS: Windows
Version: 10

Additional context
_tkinter.TclError: text doesn't contain any characters tagged with "sel" on every button's text_tags = text_field.tag_names(SEL_FIRST)

It is working for me.
What version of the Application are you using?

v1.1.1. What version of Python are you using? (I am using 3.9.6)

v1.1.1. What version of Python are you using? (I am using 3.9.6)

Application Version : Latest (v1.1.1)
Python Version : 3.9.5

There is no error for me when I try the Text Formatting. The text gets formatted perfectly.

Sorry for being late

Try running the .exe application. It should work if you run the Application.

Development

Do you have Tkinter installed?
Are you selecting the text before formatting it?

Currently, text can be formatted only by selecting it. So you will have to highlight the text you wish to format and then click on the buttons.

I could texts formatted by selecting but formatted texts is forgetting by app after reopen the file.

I could texts formatted by selecting but formatted texts is forgetting by app after reopen the file.

That is because it is a Text file, if we edit Markdown files, it would remember the formatting. The text formatting stays only while the app is running.
This was of course looked by me, and will be updated in the future in upcoming versions.


Can't get "-e VIRT_BRANCH=stable/7" to work on Windows

Hi,
I was wondering if you tried to run virtuoso-rpm-builder on Windows? If I try via the docker commandline, then it always appears to ignore the -e. If I try to set the environment variable in Kitematic, and to run it from there, same result. I always see it using the latest development branch instead of what is specified in VIRT_BRANCH. Looking at the code and everything should be allright, but it is unfortunately not :(
Thanks!

I reproduced the issue, I'll try to look into it later this week.

Hi @nvdk!
Wonderful thanks. For now, I had to recreate an image after changing the variable directly in the code. It works once I removed the \r from the files so I am good, but lack the initial intended flexibility :)
Otherwise great work with this! I will try to find the time to do the same to create DEB files for Virtuoso. This will be part of the version 3.4 of the OSF Installer:

https://github.com/structureddynamics/Open-Semantic-Framework-Installer/tree/3.4

Hey,
It seems the variable was unavailable to the subshell, commit 4e50fa6 should resolve this.


Create a website with hexo.

AS A author
I WANT TO automate the boring stuff
SO THAT I CAN focus on writing.

Fixed with 93f7dda0f7f59fdb705c9e5cde4cf556cd8bf9e4


Added history buttons for EventBrowser.

New History feature for the EventBrowser
Often when using RenderDoc, you get lost in the list of Events when following pixel history or when searching for an event.
The History records the last events visited in the EventBrowser and makes it easy to go back to previously visited events.
I added two menu buttons, next to the "step backwards" and "step forward" buttons:

Clicking the menu arrow on those buttons shows the list of visited events, and clicking on an item jumps to that event:

Once you go back in history, the "History Next" button can be used to go forward in the history.
There are two new keyboard shortcuts;

ctrl - goes to previous history event
ctrl = goes to next history event

I've thought about this and I don't think I want to merge this feature. I can understand the utility but I don't think it's significant enough beyond the existing functionality of marker regions, bookmarks, etc to justify the extra UI space and maintenance. I've had people ask for a history in the resource inspector which makes sense to me but I've seen people only rarely use bookmarks and I've not had anyone mention this as an issue before. Part of the challenge with a project like this is understanding which features are too niche.
I think this would be best kept as a local change - if you don't want to maintain a permanent fork this could also be implemented as a python UI extension.

This is sad, because this is the one feature I've always wanted in RenderDoc.
When analyzing big frames, I always end up following pixel history to understand where a value comes from, and by doing so I go to other events and always have trouble going back to the pass I was on at first. History solves that issue. It's like a web browser's back button but for events.
I guess I'll have to use bookmarks instead, which is a feature I use heavily!


Feature request: the ability to filter out certain creaters

In a similar fashion to how you can natively follow creators, I think there should be an inverse option to hide certain creators when generally browsing Nebula. It's not really possible to avoid certain creators right now. This feature would only effect the catalogue; it shouldn't prevent you from actually watching their videos.

Do you mean to hide creators from https://nebula.app/videos ?
Also keep in mind that this is an independent browser extension, so the solution probably won't be that pretty. I'd need to save which creators to ignore in the browser and then just hide the video, since there is afaik no way to filter in the API.

Do you mean to hide creators from https://nebula.app/videos ?

Precisely.

so the solution probably won't be that pretty

Not necessarily. I've just tested in my browser by deleting a video entry from the catalogue by removing its most-parent div and it doesn't leave a hole as the other videos move to fill in the gaps. Likewise, the infinite scrolling feature still works as expected.

I've just tested in my browser by deleting a video entry from the catalogue by removing its most-parent div
That's what I would've done as well

and it doesn't leave a hole as the other videos move to fill in the gaps. Likewise, the infinite scrolling feature still works as expected.
Not what I meant. Nebula fetches as videos such that they go evenly into rows. Just hiding some of the videos will mean that the last row will end unevenly. I also have to integrate the option to block/unblock creators into the UI...

Just hiding some of the videos will mean that the last row will end unevenly.

That is true. When removing the div, the law row of the catalogue is uneven but the infinite scrolling feature ignores this so the unevenness is always at the bottom of the page.

I have started work on it, please give it a go:
enhancer_for_nebula-1.0.09.zip
Currently, on the creator page it adds an icon to show and hide a particular creator. I'm planning on adding a list of hidden creators to the options.

I also just noticed that on the homepage, the slider really does not like having elements removed, so I just greyed it out there:

Again, please tell me what you think.

Updated extension: enhancer_for_nebula-1.0.09.zip
How to install: https://github.com/cpiber/NebulaEnhance/discussions/9#discussioncomment-1334893

Updated extension: enhancer_for_nebula-1.0.09.zip

Just tested this out and seems to work fine. The video is removed from https://nebula.app/videos and its various categories which is excellent. I do see what you mean about the sliders on https://nebula.app/ though. Since it's unavoidable, perhaps having some text over the thumbnail (or replacing the thumbnail and having it be less greyed out) which explains why it's greyed out, like "This creator is hidden by you."
Another thing is the show/hide buttons. I think they should be flipped. The image should, in my opinion, reflect the current state, rather than reflect the future state should you click the button. My first instinct when looking on creator pages is "Wait, I have this creator hidden?" when it's the button to hide the creator.

perhaps having some text over the thumbnail

good idea

The image should, in my opinion, reflect the current state

These are always a bit tricky ^^ But I guess you're right, that's probably more intuitive

I hope it doesn't stand out too much, what do you think?

Looks good 👍

The extension is updated on both stores now :)


[FEAT] Update modal to be closable from within the modal

:bulb: Feature request
Feature Name
Update modal to be closable from within the modal.
The Desired Behavior
The modal should be closable from within the modal (i.e. close icon button or text close button). Modal is currently only closable by clicking on the backdrop.
Your Use Case

Prior Work

Environment

Daffodil version: X.Y.Z
Angular version: X.Y.Z

For Tooling issues:
- Node version: XX
- Platform:

Others:

Fixed by #2832


Implement BiasAddGradHelper.

Registers the BiasAddGrad op as the gradient for BiasAdd by implementing BiasAddGradHelper.
Unlike the other tested ops in nn_grad_tests.cc, BiasAdd takes two arguments. The test I added does test both arguments, but not simultaneously. Not sure if that's acceptable. Guidance appreciated.
This is only my second PR to TensorFlow. When giving feedback, please assume that I barely know what I'm doing.

Can one of the admins verify this patch?

Thanks for the PR
@suharshs : Mind taking a look?

@suharshs back to you

Jenkins, test this please


"note show keyword" is case sensitive, need an option to search case insensitive...

Since it's impossible to remember which words have capitalized letters, it's important to be able to search case insensitively.

That's true.
I am going to make case insensitive by default and use the flag option --case-sensitive / -s (like ripgrep) if someone want to use case sensitive.

PR #13
Release https://github.com/gumieri/note/releases/tag/v1.2.0


Compatibility with Nanoc 4

It seems to me that this does not need any change to work with Nanoc 4 (but I haven't tried). Is it correct? Also, should this be moved to Nanoc 4.0?

@lifepillar So sorry for letting this lie around for so long. I’ve published it to Rubygems. To give you RubyGems push access, I need your e-mail address; can you send it to the address below?
echo ZGVuaXMuZGVmcmV5bmVAc3RvbmVzaGlwLm9yZwo= | base64 -d


Fix issue of handling multiple pairs of single quotes

Fixes https://github.com/vmg/redcarpet/issues/549
Trailing single quotes were treated by a special case condition causing *is_open to remain true.

👌 works for me!

Awesome, thank you very much ! :heart:


Website verification via .txt repeatedly fails

Having recently found Keybase, myself and a friend have made profiles and a team etc. All other services (Reddit, Twitter, Github) have successfully verified, but none of our websites have. I can't verify via DNS as one host forces the name to have the website address appended instead of just the @ the app requests, and my friend who can hasn't been able to verify either. I have tried the .txt on one site, which timed out (sent an email about it's failure) and just in case I used another host (with HTTPS this time, the other one was only HTTP) but that still hasn't worked. Any ideas?
The sites I tried are:
Hosted by X10 - no HTTPS - .txt is here.
Hosted by 000webhost - with HTTPS - .txt is here.
My friend's (@jackputtockk) sites, using the DNS method are:
Business
Personal

Update: After yet another try, both domains failed to verify.


Update server cipher suite selection to skip choices higher than server can negotiate

Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience._
Issue # (if available):
https://github.com/awslabs/s2n/issues/1461#issuecomment-601780922
Description of changes:
never pick a cipher suite that requires a version higher than the server is willing to negotiate.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@92e2e10). Click here to learn what that means.
The diff coverage is n/a.


Mlflow Support

Can we log fine tuned llama models using mlflow ?

Oh I'm unsure if MLFlow can be used - I think W&B works - you'll have to refer to HF docs

Thank you for your response. What would be your suggestion to host the model locally within company cloud infrastructure to avoid repeated downloads from hf for model/pipeline initialization ?

I have used MLflow with Unsloth for logging hyper-parameters and it works well for llama models, no experience yet in logging or serving models (looks like it could work) - you can refer LLM fine-tuning tutorial using PEFT from MLflow for more information

@Sruthi5797 Oh wait MLFlow works?

@danielhanchen Yes, it does - After adding the normal mlflow experimentation steps  inside TrainingArguments and at the end mlflow.log_params(training_arguments.to_dict())  does the job


Returns 500 Error if Attachment.language value is a random text

Describe the bug
For the Attachment data type ( https://www.hl7.org/fhir/datatypes.html#Attachment ) the "language" field will return a "Unexpected failure while validating resource", status 500-Internal Server Error, when used, if the value is different from values in the Common Languages value set.
To Reproduce
Steps to reproduce the behavior:

Make a PUT operation on any of the following resources : BodySite, Claim, Communication, CommunicationRequest, Consent, Contract, DiagnosticReport, DocumentManifest, DocumentReference, ExplanationOfBenefit, HealthcareService, Library, Media, Medication, Observation, Patient, Person, Practitioner, Questionnaire, QuestionnaireResponse and RelatedPerson
In PUT operation body add the field of type Attachment.
To the field added at step 2 give value "test". (Or any other value NOT contained in Common Languages value set)
Execute the PUT operation
Returns "Unexpected failure while validating resource", status 500-Internal Server Error

Expected behavior
Should have received a validating error message.
Environment (please complete the following information):

HAPI FHIR Version STU3 3.4.0
OS:  macOS High Sierra
Browser POSTMAN

Also this maybe be the common problem for the fields listed below witch also return 500-Internal Server Error if the value is different from values in the Common Languages value set.

CodeSystem.concept.designation.language
ValueSet.compose.include.concept.designation.language
ValueSet.expansion.contains.designation.language
ExpansionProfile.displayLanguage
ExpansionProfile.designation.include.designation.language
ExpansionProfile.designation.exclude.designation.language
BodySite.image.language


Port ARM snippets to bicep - part 4

Port arm snippets - part 4

Pushed some updates. Take a look when you get a chance.

Thanks @MarcusFelling. The changes look good to me.


Error in element 'set-backend-service' Backend with id 'backendId'  could not be found.

I have published a function app for production environment. Now I would want to puplish new version (v1) for the same app (separated function app). When I try to edit the <set-backend-service id="apim-generated-policy" backend-id="new-backend-app-id" /> directive I receive error 'Error in element 'set-backend-service' Backend with id 'backendId'  could not be found.'
Why can't I rewrite V1 to the right target at APIM and what is the point of versioning system If I have to target v0 and v1 to the only and the same backend service?

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

ID: 25a33ff6-8ffc-ad26-d429-3170bcca9498
Version Independent ID: fcd9c088-8b17-e53b-4c93-f90f2dc5c09c
Content: Publish versions of your API using Azure API Management
Content Source: articles/api-management/api-management-get-started-publish-versions.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@pikkuhanuristi Thanks for the feedback! We are currently investigating and will update you shortly.

@pikkuhanuristi I believe the exact scenario is covered on this doc describing the use of <set-backend-service> policy. Did you create a backend entity for the newer backend app before attempting to use the backend-id attribute? You can use either Powershell or API per the Note. If you've already created it and still getting an error, it also seems that id="apim-generated-policy" isn't part of the supported attributes. Also, instead of attempting to use the backend-id, have you tried using the base-url attribute?
Since your scenario isn't quite part of the scope of the content that this particular documentation covers, I will now proceed to close this thread. If there are further questions regarding this matter, please tag me here and I will gladly continue the discussion and I'd be happy to look further into this.


PR: Pass new environment variable to the kernel and remove some benign errors thanks to debugpy 1.6.0 (IPython console)

Description of Changes

Removed relevant errors from the benign error filter in ipythonconsole plugin
Added the environment variable PYDEVD_DISABLE_FILE_VALIDATION to IPython Console

The advent of debugpy=1.6.0 provided an environment variable to bypass the errors encountered in #16927, #16828, and #17552.
Currently, only 1.5.2 is available on conda-forge; when 1.6.0 is available, I'll mark as ready for review.
Issue(s) Resolved

Fixes #17552
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @mrclary

@ccordoba12 @dalthviz should we also require debugpy>=1.6.0? We don't have an explicit requirement on this right now, so I'm not sure what we want to do if users still have 1.5.1.

@ccordoba12, is there a reason why we are not testing Python 3.9 and conda for macOS? Just curious.

Because Github only offers 5 Mac Github Action slots per organization on the free plan, so we can spend them all in Spyder.
I think an option could be to run our tests only in Python 3.9 (instead of 3.7 and 3.9), so we can reduce the amount of slots we use per pull request. We'll take a look at that after 5.3.0 is released.

Because Github only offers 5 Mac Github Action slots per organization on the free plan, so we can spend them all in Spyder.
I think an option could be to run our tests only in Python 3.9 (instead of 3.7 and 3.9), so we can reduce the amount of slots we use per pull request. We'll take a look at that after 5.3.0 is released.

I see, thanks for the explanation. So that means we use 4 right now, correct? 3.7 pip slow, 3.7 pip fast, lite build, and full build? I have been considering for a while whether we really needed to make Lite builds for every pull request. If we limited pull requests to just the full builds, that may help; on releases, we can have both full and lite. But as you say, we can discuss after 5.3.0.

So that means we use 4 right now, correct? 3.7 pip slow, 3.7 pip fast, lite build, and full build?

Right.

I have been considering for a while whether we really needed to make Lite builds for every pull request

Well, the nice thing about the Lite builds is that they are like half-size, so faster to download, install and test. I use the Windows Lite one all the time.

@ccordoba12, what about requiring debugpy>=1.6.0? I think that this PR may cause problems for users that are still using 1.5.1. 1.6.0 is not yet available on conda-forge, either.

@ccordoba12 @dalthviz should we also require debugpy>=1.6.0?

If the env var is available since debugpy 1.6.0 then I would say so. Probably the requirement should be done over spyder-kernels, right? and then we change the minimum version for spyder-kernels over Spyder before doing the release.

Upon further consideration, I don't think we should require debugpy>=1.6.0. First, 1.6.0 is not yet available on conda-forge, so we cannot set the requirement until it becomes available. Second, while it is true that the environment variable is not recognized by debugpy<1.6.0, it will do no harm there either. Third, the particular benign errors addressed are exclusive to the standalone applications for which 1.6.0 will be the default version anyway. In short, it doesn't matter which version users have installed because the only contexts that matter (standalone apps) will use 1.6.0, so I don't see a need to add another requirement to either spyder or spyder-kernels.

Third, the particular benign errors addressed are exclusive to the standalone applications for which 1.6.0 will be the default version anyway. In short, it doesn't matter which version users have installed because the only contexts that matter (standalone apps) will use 1.6.0, so I don't see a need to add another requirement to either spyder or spyder-kernels.

Right, that's what I thought too (and sorry for letting that unanswered, I just missed the question).


React to occasion selection in booking process

What's the improvement: When the user is selecting an occasion, it should prompt him to redirect to the corresponding Deals page.
How should it look like (screenshot or sketch):

How should it work (write use case):
The prompt should appears when the user select an occasion.
Priority (tag accordingly): regular
Domain (tag accordingly): frontend
Branch: `
Work on it

Derive the branch from staging.
Tag with ongoing and move the card to ongoing.
Develop the feature
Commit your work
Create a PR and ask for validation. Move the card to staged.
Ask for validation.
Close the issue if all the work have been done

Validate

Check on mobile and on desktop if the feature have been correctly implemented.
If everything is working fine, move the card toaccepted column.
If the feature isn't completely implemented, re-open the issue with a comment clearly what the problem is and what is the desired result. Move the card to have issues column.

Please follow https://github.com/Whichost/frontend/wiki/Git-workflow

What have been done: Add a birthday deal redirection

Pushed to: react-to-occasion-selection-in-booking-process
Tag somebody: @RohrerHope
Where to test:
Asking for validation:

Check on mobile and on desktop if the issue have been correctly fixed.
If everything is correct, move the card to accepted column.
If the issue isn't fixed, re-open the comment clearly what the problem is and what is the desired result.
Move the card to have issues column.
Merge (or express approval in) the corresponding PR if needed. Move the card to the launch column.


Issue with different time zone

When we change time zone on iPhone from Minsk, Belarus to Pacific/Kiritimati weekly calendar works not correctly
Integration method - cocoapods
Device modal and iOS version - all devices(in my case iPhone XR and 7)
Xcode version - Xcode 11.7 and 12.3
FSCalendar version - FSCalendar 2.8.1
Same issue in Demo(wrong week)

https://user-images.githubusercontent.com/20534235/106722616-3c7c6b80-6617-11eb-8b94-20a77071f21a.MP4

any updates?


Added support for android using jni wrappers

Adds Jni wrappers to support liveview-client-jetpack .

@Kashif-E could you please take care of it. thanks

Android only supports dynamic libraries so adding 'cdylib' is necessary and same dor jni wrappers unlike swift android requires thw wrapper to conform with the package name of the application using the library. I think the jni part should remain in the repo. @KronicDeth can you add your suggestions please?

@Kashif-E I'm not disputing that those things are necessary for the library that contains the bindings, but this library doesn't contain any bindings, and doesn't need to do that. We specifically designed Core to be separate from the bindings, so that those could not only be maintained separately, but even be built and maintained without consulting the core team at all.
To go into a bit more detail on how this works: Swift at the highest level has the LiveViewNative Swift package, which has all of the high-level Swift UI code, and consumes the Swift bindings to Core. Those bindings are defined in the liveview-native-core-swift repo, where the LiveViewNativeCore Swift package is defined. That package builds against Core as a static archive, linking in the objects contained in that archive during its build. All of the Swift-specific binding code is contained in that package, not in Core.
For JNI, the thing to do is define a similar setup, but it doesn't have to be identical in implementation, it only needs to ensure that the JNI bindings are maintained in their own repo. One way to do this would be to have the JNI equivalent of LiveViewNativeCore (the Swift bindings package) be implemented as a Rust crate, which pulls in the Android-specific dependencies, as well as liveview_native_core, and simply contains the JNI bindings. You can then control the build output type, the library name, and any other configuration you like, without having to tamper with Core itself.
The only changes that should be made in Core, are to the features Core provides, or what it exposes via FFI. If Core needs additional Cargo "features" to control how certain Core features are implemented in order to support a particular platform, that is also fine, since that would necessarily be an issue with Core itself.
In any case, that's just my opinion, and how I approached the implementation. Ultimately it is @KronicDeth and @bcardarella who will decide how to maintain things going forward, so if they have a different take on things, feel free to disregard.

@bcardarella @KronicDeth  can you please add your opinions here, so we can decide how should we be moving forward?

As I understand it @Kashif-E you are requested a feature added to Core that would be specific to the jetpack client. @bitwalker is suggesting that you can still accomplish what you need by adding that feature to the jetpack client instead of Core. If this is the case I agree with @bitwalker we want to keep the surface area of Core specific to cross-client needs not start carving out exceptions for specific clients.

According to @Kashif-E that one could be closed without merging, cause that has been done in liveview-native/liveview-native-core-jetpack#1


listall: close the missing channel

Codecov Report
Base: 59.33% // Head: 59.30% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (e8c91b7) compared to base (2caf355).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head e8c91b7 differs from pull request most recent head 1ca5327. Consider uploading reports for the commit 1ca5327 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2866      +/-   ##
==========================================
- Coverage   59.33%   59.30%   -0.03%
==========================================
  Files         140      140
  Lines       24331    24332       +1
==========================================
- Hits        14436    14431       -5
- Misses       8044     8047       +3
- Partials     1851     1854       +3

Impacted Files
Coverage Δ

pkg/sync/sync.go
43.09% <0.00%> (-0.08%)
:arrow_down:

pkg/meta/sql.go
64.45% <0.00%> (-0.29%)
:arrow_down:

pkg/meta/tkv_mem.go
87.32% <0.00%> (+0.70%)
:arrow_up:

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

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


refactor(Collection): rename observable collections - fixes #213

The Stack and Set collections have now been brought into line with
the ObservableList by both now having a base class with the name
Observable included:

ObservableStack
ObservableSet

And the previous classes now become smaller concrete classes that
extend those bases to deliver a GameObjectObservableStack and a
GameObjectObservableSet.
The prefabs have been updated with the new script names.

I forced pushed the change as it was just a single fix.


Add contents to Libraries and Synths sections

I wish to add the following items

XSound - Web Audio API Library for Synths, Visualization, Effects ... etc
X Sound - Web Synthesizer by XSound.

Thanks for the contribution 👍


Optimize argument decode exception handling.

In the current implementation, org.apache.dubbo.rpc.protocol.dubbo.DecodeableRpcInvocation#decode(org.apache.dubbo.remoting.Channel, java.io.InputStream) will catch exceptions in readObject, and then only print WARN-level log, If the log level of the provider is ERROR, the error cannot be seen. A common problem is that a NullPointerException be throw when the provider executes business logic. Finally, the consumer gets a NullPointerException, which makes the problem difficult to troubleshoot.
For example, when the argument class has only one constructor with parameters, and the constructor parameter is a non-primitive type, Hession will initialize the parameter to be null. If the parameter is used in the constructor to execute business logic, it will trigger related exceptions, such as IllegalArgumentException, but in the current implementation, parameter decoding exceptions will be caught. The provider actually gets an null parameter when execute business logic. In most scenarios, a NullPointerException will be thrown to the consumer.
What is the purpose of the change
Allow the consumer to get the original argument decode exception to facilitate troubleshooting.
Brief changelog
Remove the exception catch and log printing logic when calling readObject, and throw it directly to the outer decode method so that the exception is set into the request instance, and finally sent to the consumer.

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

#9489

Codecov Report

Merging #9490 (ee3f6cd) into 3.0 (897cd0f) will decrease coverage by 0.19%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.0    #9490      +/-   ##
============================================
- Coverage     65.29%   65.10%   -0.20%
+ Complexity      354      353       -1
============================================
  Files          1204     1204
  Lines         52079    52020      -59
  Branches       7783     7729      -54
============================================
- Hits          34006    33866     -140
- Misses        14436    14519      +83
+ Partials       3637     3635       -2

Impacted Files
Coverage Δ

...org/apache/dubbo/rpc/filter/ClassLoaderFilter.java
88.88% <ø> (-11.12%)
:arrow_down:

...bo/rpc/protocol/dubbo/DecodeableRpcInvocation.java
75.92% <100.00%> (+1.81%)
:arrow_up:

...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java
24.35% <0.00%> (-44.88%)
:arrow_down:

...va/org/apache/dubbo/rpc/support/AccessLogData.java
53.16% <0.00%> (-37.98%)
:arrow_down:

.../remoting/api/SingleProtocolConnectionManager.java
61.53% <0.00%> (-23.08%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 41 more

Continue to review full report at Codecov.

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

How about changing the log level to error?
I am a little bit worried that this change will bring unexpected behavior to some of the previous users.

I checked the Git commit log and this code has been committed since 2012.07.01. The first version used e.printStackTrace(), and then 2012.07.06 was optimized to use WARN log printing. This code has not changed since then. It is not a good design to initialize to null when deserialization fails, which delays the time when the error occurs.
Consumers should be aware of deserialization exceptions, not just print logs on the provider.
For previous users, if the deserialization fails, they should deal with the deserialization problem instead of using null parameters, because it seems that the parameters are lost after the Dubbo call.
Reference: Java Techies Zone: Item 65: Don’t ignore exceptions


InfraworldExample/Server missing from sample project

After downloading and unzipping the linked example project InfraworldExample/Server folder does not exist.

@nikitamiroshnichenko I tried to setup the current master with the demo client.
fatal error C1189: #error:  This file was generated by an older version of protoc which is
Did I screw up something during installation or is the demo client outdated?

@nielslanting you need to regenerate from the proto file (which I believe is not included). As the release has a newer version then the example.

@lsjostro Thanks for the quick reply.  I could not find the proto file anywhere, I tried to recreate it.
https://gist.github.com/nielslanting/f819f456db57c8cfa5fcc28b0f3dfc86
I am uncertain how I should build the .proto file for this project. Am I supposed to use the infraworld-protobuild repository? The infraworld-cornerstone doesn't generate the wrappers. I tried to manually build them, protoc -I ./ --cpp_out=./ vizor_demonstration.proto  protoc -I ./ --plugin=protoc-gen-grpc=grpc_cpp_plugin.exe --grpc_out=./ vizor_demonstration.proto, not getting the right files out of this.
Can you tell me how I should build a proto file to work with this project?

something like this would work.
# Generate wrappers
~/UnrealEngine/Engine/Plugins/Marketplace/InfraworldRuntime/GrpcPrograms/Linux/protoc \
    -I $(realpath ~)/Projects/mygame/Api/ \
    --plugin=protoc-gen-grpc=$(realpath ~)/Projects/UnrealEngine/Engine/Plugins/Marketplace/InfraworldRuntime/GrpcPrograms/Linux/grpc_cpp_plugin \
    --cpp_out=Source/MyService/Private/Wrappers/coolapi \
    --grpc_out=Source/MyService/Private/Wrappers/coolapi \
    $(realpath ~)/Projects/mygame/Api/*.proto

@agearhart we created an separate repository for example project. Demo Server located at DemoServer directory. Please look https://github.com/vizor-games/InfraworldRuntimeExample

Fantastic, thank you!  I've gone another route entirely, but may tinker with this during free time.

I'm closing this issue for now. Feel free to open an another one if you have something's still unclear.


Deduplicate work when consuming the same collection stream in multiple ways

In the case where a single collection stream is processed (a) with combine* as a consumption operation and (b) with merge as a preprocessing operation, switchCollection will be applied twice, doing the same work in both cases.
Investigate ways to multicast switchCollection as an intermediate operation in a way that doesn't complicate the existing API.

Same issue as #7


RangeExtensions.SelectMany causes confusing compiler errors

The following code produces a very confusing error:
using Funcky;
using Funcky.Extensions; // <-- Note this using

var query =
    from x in Sequence.Return(1, 2, 3)
    from y in Sequence.Return(1, 2, 3)
//            ^^^^^^^^^^^^^^^^^^^^^^^^
//            Cannot implicitly convert type 'System.Collections.Generic.IReadOnlyList<int>' to 'System.Range'
    select x + y;

The error is reported because the SelectMany extension method from RangeExtensions is in scope:
SelectMany<TSource, TResult>(this IEnumerable<TSource> source, Func<TSource, Range> selector, Func<TSource, int, TResult> resultSelector)
It took me a while to figure that I was missing a using for System.Linq.
If there's no using for Funcky.Extensions the error is very clear and the IDE suggests to add the missing using for System.Linq:
using Funcky;

var query =
    from x in Sequence.Return(1, 2, 3)
    from y in Sequence.Return(1, 2, 3)
//            ^^^^^^^^^^^^^^^^^^^^^^^^
//            Could not find an implementation of the query pattern for source type 'IReadOnlyList<int>'.
//            'SelectMany' not found. Are you missing required assembly references or a using directive for 'System.Linq'?
    select x + y;

I'd suggest we don't add more SelectMany overloads like this one to IEnumerable (I think we talked about adding some for Option<T>).

Decision:

we do not add SelectMany on IEnumerable in other namespaces thant Funcky.Extensions.


Add quality grade filter to taxon page photo browser

E.g. at https://www.inaturalist.org/taxa/98351-Contia-longicaudae/browse_photos, provide ability to only view photos from RG observations.

Fixed in 380077edac8d81ebf7cf32b9a5fc3038335f39ea


Update README about how to run for production

A lot of people including me, @pgerochi and including this issue https://github.com/ReactJSResources/react-webpack-babel/issues/302 are seeing this issue when you open index.html from public directory after running yarn it won't display anything and no warning/error about why the page is blank.
Workaround for people who wants to run without webserver would be to enable hash in the URL, but if that's not what they want then they'll have to either setup an expressjs server or run simple python server to test their app quickly
python -m SimpleHTTPServer 8000

Obviously there are many other options including https://www.npmjs.com/package/browser-sync which is easy to setup, but we just need to update our README to avoid this issue in the future :)

https://github.com/ReactJSResources/react-webpack-babel/pull/305


throughput.py and release.py won't wait if images aren't already built

These scripts have a 15 second timeout on the call to run_pair.sh. Great, but that's too short if the images aren't already built. We should probably push the timeout down to connect_pair.py.
Here's a one-liner to pre-build your images:
for b in chrome firefox; do for v in stable beta canary; do ./image_make.sh $b $v; done; done

Super-annoying if you aren't intimately familiar with our Docker scripts, so keeping this as P2.

docker rmi uproxy/chrome-stable uproxy/chrome-beta uproxy/chrome-canary uproxy/firefox-stable uproxy/firefox-beta
for b in chrome firefox; do for v in stable beta canary; do ./image_make.sh $b $v; done; done

No longer relevant since we started uploading images to Docker Hub.


Add weather emoji ☀

Inspired by #106:
If time and geolocation is there, then even a weather emoji could be added automatically.
I guess there are enough of them:
🌤⛅🌥🌦☁🌧⛈🌩☀🌞⚡❄🌨☃⛄🌬💨🌪🌫🌈☔💧💦🌊🧣🧤🌂☂🔅🔆🥵🥶🧊ϟ☀☁☂☃☄☉☼☽☾♨❄❅❆
And some people did some libraries for that.
But of course there need to be a possibility to add an API key for a weather service in the settings.
Dart has at least some libraries for that, like this or this.
Or maybe just using the icons from OpenWeatherMap would be easier?

ui, emojis are great, they can't be used enough


[anonymous links] Distinguish between already checked and unchecked components

Steps

Go to sharing page on a project on which you have admin access
Press Create a link
Look at the options to associate components with the link

Expected
Already selected components should appear disabled and components that can be selected should appear active.
Actual
The current project is always already clicked, and components are listed below it. Since the text color is the same for the checked/selected components and active/unselected components, I always think that the unchecked components are also inactive. Does this make any sense?

I think the text with the purple arrow should somehow be distinguished from the text with the blue arrows. Perhaps make it a lighter shade of gray? If the text were to change when selected (such as becoming gray), this may make it clearer which components can be clicked and which can't.

I think this UI enhancement is necessary and also is not hard to do. Can I grab this one? Thanks :)

The effect after changing:

Similar unchecked box also exists in project setting page, make the similar change and it will get:

But after discussing with Caner, he pointed out that the grey checkbox has already shown the state of current component, which is checked but unselectable. From my point of view, grey color is to make the checked but unselectable state more obvious. It delivers the correct and useful information.
Some additional discussions are needed.

@haoyuchen1992 and @caileyfitz ... If I am understanding correct, does your example, "The effect after changing:" accomplish the needed task?

@lbanner It does accomplish the needed task. But there is a discuss that the current implementation is already good. Changing the text color may not a good option.

Already selected components appearing disabled is against feedback rules for UI. One could bold the text but that could denote hierachy and also influence the visibility.
@lbanner I already made my comments on this to Haoyu, unless someone specific can comment on it (in which case we need to tag them) we can move on and close this issue.
@caileyfitz

@caneruguz I don't think I completely understand... Doesn't the checkbox already appear disabled? Does this go against feedback rules for UI?

This is what can't be done " If the text were to change when selected (such as becoming gray), this may make it clearer which components can be clicked and which can't."
Also the check box is already disabled, isn't it?
disabled means you can't do an action on this right now. If that is the case we can use it, it doesn't mean "this is selected". I don't think there is any need to change here.

@caileyfitz -- Good goal but I think this one we leave as is for now ... unless over time we receive confused feedback from users. Closing to clear the decks. TY!

I stand my ground that this is still very confusing but see the need to focus energy into addressing more troublesome issues.


Disable Coc's completeopt

Similar to #9 but with coc.nvim

#13 should work for coc.nvim as well unless coc modifies completeopt on its side. I believe this should be handled in coc.nvim. Vim-clap has done what it can.

'completeopt' is a global value, setlocal is not work.
https://github.com/liuchengxu/vim-clap/blob/master/autoload/clap/popup.vim#L163
Your commit make a mistake, completeopt is cleared globally

@tracyone  Thanks, fixed.


toSentenceCase utility should consider consider abbreviations

toSentenceCase utility should consider abbreviations.
For example:
ImageWithTextAndCTA should be transformed in Image With Text And CTA

Fixed with https://github.com/hauptrolle/chakra-templates/commit/fc2d89dbc819f48677d0d8d625d63499ad9c0789

Fixed with https://github.com/hauptrolle/chakra-templates/commit/fc2d89dbc819f48677d0d8d625d63499ad9c0789


[WIP] Overworld world generation

The aim is to implement a basic generator with Perlin noise and biome support. Caves, trees, etc. will be added in a later PR.

[x] Lay groundwork for world generation with a ComposableGenerator based on http://cuberite.xoft.cz/docs/Generator.html
[ ] Implement a basic height map generator
[x] Implement a basic composition generator
[ ] Implement a biome grid generator
[ ] Generate height and composition based on biomes
[ ] Interpolate between biome height values to prevent sharp cutoffs

First steps!

Height map generator is a bit messed up at the moment. Hope to figure out what's causing this problem.

In Cuberite we moved away from heightmap generators in favor of shape generators https://github.com/cuberite/cuberite/pull/1604. Basically, a shape represents if a block should be empty (air) or filled. The reason for this is because it was much harder to create proper overhangs if we're generating heights.

Feel free to contact me with questions about terrain generation (I'm the author of most of the Cuberite Composable generator docs you mentioned).

Yes, that's what I plan to switch to—the height map generator is just temporary. The issue, though, is that the noise function doesn't appear to produce smooth results, and there are visible lines between chunks. So switching to a density map will still cause issues.
This image is a visualization of what the noise function outputs when called for generated in chunks:

See jackmott/rust-simd-noise#9.

@madmaxof so I have a question about the biome height interpolation method mentioned in the Cuberite docs. I can see how you could interpolate between heights by averaging when using a 2D height map, but the same doesn't apply when using 3D noise to generate a density map rather than a height map. What's your method for doing this?
I see that Glowstone uses a 2D height noise with added 3D detail and roughness noises to add overhangs and such. They then use the 2D height noise to calculate interpolation between biomes. Is this a method you would recommend?

We use a bit different method.
For the base, we have two 3D noises that are used for the density, another low-frequency 3D noise is used to switch between them (this allows for sudden "walls", overhangs etc.)
Then biomes averaged across 9 * 9 columns are used to select two "parameters" for each point, the ideal height and the ideal amplitude of the terrain. Each biome has those two hard-coded. The ideal height and the amplitude are then turned into a vertical "gradient" (height -> gradient midpoint, amplitude -> gradient slope) which is added to the noise. The gradient is slightly modified to account for some shaping - at the top of the chunk, the gradient makes the terrain disappear faster, and underground the gradient is 4 times "slopier" than above ground to make the ground have fewer holes. Finally, yet another 2D noise, added to each Y column, is used to modify the ground height.
Of course calculating all this would be too much for the CPU, so these noises are all linearly upscaled by a factor of 4 (so that only a 5 * 33 * 5 values from each are needed for a single chunk).
Here's the C++ code:
https://github.com/cuberite/cuberite/blob/7678d5e6ed6fcc9361416ef41c43fa09a1d49f6f/src/Generating/Noise3DGenerator.cpp#L585-L772

I plan to merge this PR within the next day and make incremental improvements to worldgen over time. @momothereal any thoughts?

Just tested the PR. Very impressive work!
Sounds good to me


Return value is printed out twice in the REPL

When I evaluate (first '(4 5 3)) using the ctrl-, b binding in the REPL (this is without Ink installed), I get:
=> 4
=> 4
--------------------
(first '(4 5 3))

Why is the result printed out twice? For that matter, why is the original expression repeated? Is the presumption that you'd want to modify it and re-evaluate it? Also, the re-printing of the evaluated expression doesn't happen if the evaluation is done on an expression in a regular editor - should output not be consistent across the REPL and editor invocations? Edit: just noticed that if the evaluation is invoked in the REPL using shift-enter, the original expression is not repeated (but the output is still doubled).

This doesn't happen normally. Someone reported this before as #33 (https://github.com/jasongilman/proto-repl/issues/33) It was caused by cider-nrepl in their profiles.clj

I tried all kinds of restarting, starting with a fresh .atom folder, etc, and finally figured out the cause — cider-nrepl. Removing cider-nrepl from plugins in .lein/profiles.clj solved the problem.

Can you see if you have a similar issue?
I tried proto repl with no ink enabled and evaluated (first '(1 2 3)) and I get the expected output in the repl
(first '(1 2 3))
=>
1
--------------------

You're right, removing cider-nrepl fixes this problem, but I'd still like to be able to use Emacs/Cider along with Atom so I'd like to keep it there. Maybe you could ask @Malabarba (Artur Malabarba, the author) in the gitter room where he hangs out what could be done to make things work despite cider-nrepl being present.

While I'm happy to help, let me clarify I'm a contributor (not the aurhor) and we should also ping @bbatsov (the maintainer).
As for the actual problem, I'm sad to say nothing comes to mind. Maybe with more information about how proto-repl works I'll be able to help, but I'm not aware of any cider-nrepl middleware that affects return values.
Cider-nrepl does carry some middleware that causes additional responses to be sent (such as the state_tracker).

I'm guessing proto-repl has some way to inspect the messages exchanged by it and the nREPL server - I'd suggest showing us those messages, so we can help you out.

Looking at the installation instructions it seems to me proto-repl doesn't use any additional middleware. Is this correct?

Thanks for helping.  Proto REPL doesn't use any additional middleware. It uses https://github.com/rksm/node-nrepl-client for connecting to nREPL. I printed out the messages returned after requesting (+ 1 1).
(GUIDs truncated)
{"id":"0048FDA8","ns":"user","session":"acf79b4f","value":"2"}
{"id":"0048FDA8","session":"acf79b4f","status":["done"]}

Then I tried using Proto REPL after installing cider-nrepl and the messages looked like this:
(GUIDS truncated)
{"id":"46BF16F1","ns":"user","session":"0c868cc4","value":"2"}
{"id":"46BF16F1","ns":"user","session":"0c868cc4","value":"2"}
{"id":"46BF16F1","session":"0c868cc4","status":["done"]}
{"id":"46BF16F1","session":"0c868cc4","status":["done"]}

In addition two other messages were returned with "state" status.
I tried debugging node-nrepl-client at a lower level and it looks like only one message is being returned so I'm not sure why I'm seeing the doubling. I can't tell if it's a bug in node-nrepl-client or something else. I wonder if there's a way to tell nREPL to avoid using a middleware.

Here's the logging inside node-nrepl-client when using cider-nrepl
nREPL message chunk received (3128 bytes)
nrepl message received (113 bytes, {
  id: 'E506CF61-5B0D-46CF-865F-6C73867CF117',
  ns: 'user',
  session: 'd92e5ef8-5ec8-43ec-8363-ca8176725d6a',
  value: '2'
})

nrepl message received (109 bytes, {
id: 'E506CF61-5B0D-46CF-865F-6C73867CF117',
  session: 'd92e5ef8-5ec8-43ec-8363-ca8176725d6a',
  status: [ 'done' ]
})

nrepl message received (2903 bytes, {
  changed-namespaces: Dict {
    'clojure.string': [Object],
    'proto-repl-demo.conditional': [Object],
    'proto-repl-demo.demo': [Object],
    'proto-repl-demo.demo-test': [Object] },
  id: 'E506CF61-5B0D-46CF-865F-6C73867CF117',
  repl-type: 'clj',
  session: 'd92e5ef8-5ec8-43ec-8363-ca8176725d6a',
  status: [ 'state' ]
})

I've traced this down to a single problem. There's a callback being made twice when executing *clojure-version*. This double callback was then adding two sets of handlers for printing out nREPL messages. I added some temporary code to handle that and all other messages returned so far only come back once.
At this point it looks like the problem is almost certainly not with cider-nrepl. I need to do some more investigation but in any case I'll have a fix for Proto REPL so that if the cider-nrepl middleware is used Proto REPL will still behave normally.

Great! Btw, you should investigate the functionality offered by cider-nrepl - it's really not Emacs specific and it can really help you add extra functionality to proto-repl easily. We've learned from experience that there's a limit to what you can do by relying simply on nREPL's eval. :-)

And if you do decide to use it, don't hesitate to ask questions on the repo or on gitter.
Anyway, I'm glad you've found the issue. 👍


Resolve deprecated set-output warning

Fixes https://github.com/joshlk/jenkins-githubaction/issues/4

Thanks for the contribution


chore(main): release 0.26.5

:robot: I have created a release beep boop
0.26.5 (2023-06-09)
Bug Fixes

repo issue and logger issue (86303c6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/appblocks-hub/BB-CLI/releases/tag/v0.26.5 :sunflower:


Persist state of pinned view

When pinning/unpinning a view, the xy and timeline charts lose state of their tree selections. Would be nice to persist the state of these charts when they are re-rendered due to pin/unpin action

Working on this issue

Working on this issue

You can assign yourself using Assignees on the right. I just did it for you. Thanks for picking this work up!


Read Consul HTTP Token from a file

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
CONSUL_HTTP_TOKEN seems to be the only documented way to use Consul ACLs.
In Docker Swarm configurations this token should be read from a file instead. As ENV vars are not treated as secret and will leak.
An alternative such as CONSUL_HTTP_TOKEN_FILE or better yet --consul.httpTokenFile and it's TOML equivalent would be much better for swarm security.

I think the scope is broader, and treafik should be adapted to read the contents of files when a file path is passed via special ENV variable
The suffix _FILE is a good alternative, and it's being used in official docker images like this.
So the issue cloud be something like : "Add _FILE suffix as an alternative to use file contents as the value of the variable."

My case is for ACME DNS Challenge for Cloudflare, and to save the CLOUDFLARE_API_KEY as a docker secret.
Just use a CLOUDFLARE_API_KEY_FILE instead of CLOUDFLARE_API_KEY could solve.
Ex.:
    environment:
      -<EMAIL_ADDRESS>      - CLOUDFLARE_API_KEY_FILE="/run/secrets/cloudflare_api_key"


Removed mock from everywhere

Current dependencies on/for this PR:

main

PR #403   👈

This comment was auto-generated by Graphite.


Cannot write data

Hi, I'm trying to send data between two VSPs created with:
socat -d -d pty,raw,echo=0 pty,raw,echo=0
Opening one port in my program using your library and the other port in putty I cannot send data from program to putty but only read strings from putty.
write always returns zero

I've not used socat before. I'd recommend trying your setup with two instances of pyserial or miniterm or something that is "known" to work.

I've just tried the communication between two instances of putty and it works just fine in both directions.
It's non mandatory using socat, if you know another method to create a pair of virtual serial ports let me know, i'm just trying to simulate a connection between my software and a robot controller that in this case is a "simulator" running on this pc.
thaks :)

in file unix.cc, line 551-557
while (bytes_read < size) {
int64_t timeout_remaining_ms = total_timeout.remaining();
if (timeout_remaining_ms <= 0) {
// Timed out
break;
}
...
}
timeout_remaining_ms is always a negative number so the function deos not write to the port

Make sure you've set the read timeout: https://github.com/wjwwood/serial/blob/1.2.1/include/serial/serial.h#L424-L459

I'm gonna close this. Comment if new information comes up.

I had a similar issue. Problem was actually on the other end of the Serial line not reading properly and the pselect timing out is the symptom I saw from this end.
If you are in a similar spot, try putting a loopback on the other end of the serial as described in the preamble of serial_example.cc to quickly test if you have a similar issue.


can't construct a list that is more than one element.

Hi,
I was trying List in project-m36.
I didn't find manuals about Lists. And I tried this on my own but failed.
TutorialD (master/main): data Hair = Bald | Brown | Blond | OtherColor Text
TutorialD (master/main): b :: { a List Hair }
TutorialD (master/main): :showexpr b
┌─────────────────┐
│a::List (a::Hair)│
└─────────────────┘
TutorialD (master/main): b := relation{tuple{ a (Cons Bald Empty) }}
TutorialD (master/main): b := relation{tuple{ a (Cons Bald (Cons Bald Empty)) }}
ERR: NoSuchFunctionError "Bald"

And a list in a relvar in my program shows like this (just one cons for three elements):
TutorialD (master/main): :showexpr Profile {profileSkills}
┌──────────────────────────────────────────┐
│profileSkills::List (a::Skill)            │
├──────────────────────────────────────────┤
│Cons Woodwork Ironwork Plumber_Electrician│
└──────────────────────────────────────────┘

I don't know if it is normal.

I fixed the list creation bug- luckily, it was just a parsing bug.
Could you provide instructions on how to reproduce the Cons cell with the wrong number of arguments? I see this:
TutorialD (master/main): b := relation{tuple{ a (Cons Bald Bald Empty) }}
ERR: ConstructedAtomArgumentCountMismatchError 2 3

which is the expected behavior, so there must a different way to construct the Profile relvar that I am missing.

It happens at toAtom in Atomable [a].
Luckily, It is an easy one, too.
λ> data A = A | B | C deriving (Show, Eq, Ord, Generic, NFData, Binary, Atomable)
λ> toAtom [A,B,C]
ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "A" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "B" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "C" (ConstructedAtomType "A" (fromList [])) []]
λ> atomToText it
"Cons A B C"

--Atomable.hs
-- toAtom (x:xs) = ConstructedAtom "Cons" (listAtomType (toAtomType (Proxy :: Proxy a))) (map toAtom (x:xs))
++ toAtom (x:xs) = ConstructedAtom "Cons" (listAtomType (toAtomType (Proxy :: Proxy a))) [toAtom x, toAtom (xs)]

λ> data A = A | B | C deriving (Show, Eq, Ord, Generic, NFData, Binary, Atomable)
λ> toAtom [A,B,C]
ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "A" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "B" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "C" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Empty" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) []]]]
λ> atomToText it
"Cons A (Cons B (Cons C Empty))"

Thanks for checking!
Could I just ask you some more questions?
Why does project-m36 need run-time data type creation?
Is it possible to make Set and List primary atom types?
I would like to express Skills in Set Skill but not sure why there are no built-in Set and List. Is it violating some relational algebra or 1NF thing?

And fromAtom also need to change.
λ> data A = A | B | C deriving (Show, Eq, Ord, Generic, NFData, Binary, Atomable)
λ> toAtom [A,B,C]
ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "A" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "B" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "C" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Empty" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) []]]]
λ> fromAtom it :: [A]
[A,*** Exception: no fromAtomG traversal found
CallStack (from HasCallStack):
  error, called at /root/project-m36/src/lib/ProjectM36/Atomable.hs:46:16 in main:ProjectM36.Atomable

-- fromAtom (ConstructedAtom "Cons" _ (x:xs)) = fromAtom x:map fromAtom xs
++ fromAtom (ConstructedAtom "Cons" _ (x:y:[])) = fromAtom x : fromAtom y

λ> data A = A | B | C deriving (Show, Eq, Ord, Generic, NFData, Binary, Atomable)
λ> toAtom [A,B,C]
ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "A" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "B" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Cons" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) [ConstructedAtom "C" (ConstructedAtomType "A" (fromList [])) [],ConstructedAtom "Empty" (ConstructedAtomType "List" (fromList [("a",ConstructedAtomType "A" (fromList []))])) []]]]
λ> fromAtom it :: [A]
[A,B,C]

Ah, thanks- I didn't try using Atomable directly.
Your question regarding why Project:M36 supports algebraic data types is a good one! This topic is specifically covered in the paper "Out of the Tarpit" where ADTs are part of an effort to reduce bugs by allowing developers to program in their assumptions instead of relying on progamming-language specific abstractions. I hope you agree that "Bald | Brown | Bald | OtherColor String" is more specific than "Int" or a simple enumeration. However, the paper recommends not allowing for product types due to concerns about normalization. I wasn't convinced by this argument, so, in the interest of being Haskell-like, I implemented support for product and sum types with the same syntax as Haskell.
Could we implement our data types without ADTs? Sure, just like we do with painful workarounds in SQL or JSON document databases. Furthermore, ADTs solve the pervasive NULL problem.
In conclusion, I would recommend avoiding Lists in a database and embracing ADTs, though Project:M36 does try to make Lists first class objects (instead of CSV strings).

Thanks for explanation! I agree the "OtherColor String" example is way more specific.
data Skill = A | B | C
So if I want to record a person with his/her skill set, then see who have skill A,
I guess I should make Data.Set in haskell space Atomable first.
Profile{ person Text, skills (Set Skill) }
(If I use relation to represent a set. I can't give it a unique key and the property of set will be lost.)
then create a read-only view that (kind of) ungroup the skill set to avoid repeating information.
View{ person Text, skill Skill }
then I can View where skill=A
Does these two things make sense for you in project-m36?

a read-only view just for viewing. Maybe a shortcut operator? ( the := operator creates a new relvar and won't be changed by the original one's update while :@ makes a shortcut for a view and the view's data is changed by the original relvar accordingly.)
View :@ Profile ungroup_container skills

a ungroup_container operator ungroup the ADT container instead of subrelation.

--
Another way is to just write boolean function for Set.  ^has (A, skills)
Could you give me some opinion about this question? I am scratching my head on this.

Well, with regards to normalization, I would suggest neither a list nor a set for this case. Instead, make use of the relational algebra. "Skill", itself, can be a key since it must support equality, so one can create a many-to-many relationship between Profiles and Skills. The Skill can definitely be a unique key, but maybe that needs clarification in the documentation.
TutorialD (master/main): data Skill = A | B
TutorialD (master/main): :showexpr relation{tuple{a A},tuple{a B},tuple{a B}}
┌────────┐
│a::Skill│
├────────┤
│A       │
│B       │
└────────┘

The advantage of using the relational algebra to make the relationships is the ease of being able to issue arbitrary queries to answer any question you can ask about the data. In your Set-based strategy above, for example, you would need to implement the Set operators on the atom in order to figure out which profiles include which skills, but the native relational algebra does not require additional atom-based operators.
Another strategy, to which I think you alluded, is to use a nested relation to represent the skills. Project:M36 supports this.
Ultimately, you are suggesting views because many of these representations are isomorphic! That's key to understanding data independence because the database can choose whichever representation it finds most ideal for the scenario or it can choose to represent them all of them in memory or on disk. So, in the end, it shouldn't matter which one of the isomorphic representations you choose- stay tuned for exciting new Project:M36 features on this front!

I see.
I guess I can turn a Set a into RelationAtom in Atomable.
So a Profile with a Set a field type will be a nested relation in Tupeable.
Then I can get a many-to-many relationship between Profiles and Skills by ungroup.
Thanks for enlightening!


Can't create a FileFilter with no name

Creating a FileFilter with an empty name (FileFilter::new("")) makes the OpenFile portal to error out. I think it is generally possible to create an unnamed filter, or at least this seems possible with GtkFileChooserNative, so I guess this is a bug.

That doesn't seem to be the case per https://flatpak.github.io/xdg-desktop-portal/#gdbus-method-org-freedesktop-portal-FileChooser.OpenFile or any of the FileChooser methods. Quoting the docs

filters a(sa(us))
List of serialized file filters.
Each item in the array specifies a single filter to offer to the user. The first string is a user-visible name for the filter. The a(us) specifies a list of filter strings, which can be either a glob-style pattern (indicated by 0) or a mimetype (indicated by 1). Patterns are case-sensitive. To match different capitalizations of, e.g. '.ico', use a pattern like '.[iI][cC][oO]'.
 The portal may still allow the user to select files that don't match any filter criteria, and applications must be prepared to handle that.

I will have a look at gtk to see what it does here.

That is what the gtk implementation calls, https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkfilefilter.c#L965 so it seems to always include the filter name

I would assume the issue comes from the portal itself, please reopen if the portals (xdg-desktop-portal) made it clear the filter name could be an empty string.


bug: Embeddings not provided, do not appear in enhanced context sources, after reload

Version
Main 9e06cafee3fb60b7e265ced935b2e55b57413838
Describe the bug
From @toolmantim via Slack: https://sourcegraph.slack.com/archives/C05AGQYD528/p1702249057007849?thread_ts=1702246529.082959&cid=C05AGQYD528
Repro steps:

Open a workspace with local embeddings, start a chat, note the embeddings appear in the enhanced context selector
Reload the window
Open the enhanced context selector

Expected behavior
Expected behavior: Embeddings still appear in the enhanced context selector
Actual behavior: Embeddings don't appear in the enhanced context selector
Additional context
Cody log indicates the workspace is being loaded before the access token is valid/set. Either local embeddings should not require valid access tokens to open indexes, or when one is available the controller should try to open the index for the last attempted repo.

Debugging this at trunk, it looks like local embeddings doesn't get an access token on window reload.


[Demo store] Discount route doesn't work when a cart doesn't exist

What is the location of your example repository?
https://github.com/Shopify/hydrogen/tree/2023-04/templates/demo-store
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2023.4.5
What version of Remix are you using?
1.17.1
Steps to Reproduce

Go to https://hydrogen.shop/discount/HYDROBOARD?redirect=/products/snowboard
Add the product to the cart
The discount isn't applied to the cart

Expected Behavior
The discount should be applied
Actual Behavior
The cart doesn't apply the discount. It only works if the user adds a product to the cart before going to the discount route

I noticed that this has been fixed on<EMAIL_ADDRESS>


Question: Change node AMI while master is up

Is it possible to change the node AMI for the cluster while the master is up?
Thanks

Before answering your question. Can we understand what are you trying to achieve here?

I am trying to preserve the master while changing the nodes AMI for the sge cluster

It's possible to create cluster with custom_ami. As per current design we use same ami for mater and compute node. You can't change compute node AMI while master instance is running.
I am not  sure what you want to change in compute node of sge cluster. Have you looked at post_install and pre_install script. you can have customized script to run only on compute node based on instance tags.

We have a service that needs to be updated and each update takes a considerable amount of time, if it were possible to swap out the AMI used instead would be a bit less work. I appreciate your time

We have a service that needs to be updated and each update takes a considerable amount of time, if it were possible to swap out the AMI used instead would be a bit less work. I appreciate your time
Also we would like the master IP address not to change between updates

Have you looked at Code Deploy In-place deployment? Again I don't know what your service does, so please validate before take my word on it.
Master instance has elastic ip address and we are not changing it in cluster life cycle.

Sorry I should've been more accurate, the private IP address. I'll take a look at Code Deploy thanks!

when you stop and start cfncluster, we do stop of master instance today (will might change this behavior), which will change public ip address associated to the master but not the private ip (also note eip is not affected here).

awesome, thanks a lot


Add shouldInterceptRequest method

Android WebView has a method called shouldInterceptRequest, it is used to support fully control over every request. We can use whatever we want as the response of the request by using this method.
It's good to have such method or any similar feature.
Here is my use case with Android WebView's shouldInterceptRequest:
I need to send some large messages(big image in base64 encoding, eg.) to js code, but the Android WebView has a maximum limit on the size of the data exchanged between js code and java code through the @JavascriptInterface(aka the js-bridge), so the ONLY way to achieve this is to make a "fake" HTTP request with a special url from js side, and "catch" this request in java side using shouldInterceptRequest, construct a HTTP response using the large message as the response body, and extract the large message from the HTTP response in js side later.
By the way, according to SO, it seems that we can use WKURLSchemeHandler to implement an iOS version of shouldInterceptRequest method.

@TelephoneTan Thanks for your suggestion! I will add it to our feature plan. Feel free to submit a PR for it as well!

Is there any progress on this feature?

I am looking for this as well. Maybe I could help contribute this in?

I am looking for this as well. Maybe I could help contribute this in?

@jccr Sure, feel free to do it!

Do you accept changes for Android only, or would every change need to support both ios and android? I have worked with implementing shouldInterceptRequest in react native


Binder.bind(PropertyAdapter source).to(HasValue<?> widget)

Hey!
I've been messing around a bit with GWT and Hexa Binding. I noticed one thing. If you have a model that already has data in it and you bind it to a Widget (HasValue<?>), the Widget won't take the models current value.
Maybe it's not always what you want, but in my case I'd like it.
In my case, I have a simple HashMap as my model. I create a one-way property adapter for it (implementing PropertyAdapter). This of course means that if I make changes to my Map, it won't be reflected to the bound component, but that is fine.
The Map has default values set on them, and when I bind each key to a GWT component, I want that GWT component to immediately get the models value.
Can we work this out, you think? Maybe a new overload, to(HasValue<?> widget, boolean useExistingModelValue);

Hi Anton,
Normally the first thing a data binding does is to propagate the value from
the source to the destination. So you should not have to call nothing to
geth that done.
This means there might be a problem else where.
I suggest two things :

when you create the data binding, add a call to the
log("myBindingLittleName") method. You should at least see a message saying
"propagating to destination". Is it the case ?
could you please provide me with a minimum code snippet to reproduce the
problem ?

Thanks a lot !
Arnaud
Le mar. 30 juin 2015 à 14:48, Anton Johansson<EMAIL_ADDRESS>a
écrit :

Hey!
I've been messing around a bit with GWT and Hexa Binding. I noticed one
thing. If you have a model that already has data in it and you bind it to a
Widget (HasValue<?>), the Widget won't take the models current value.
Maybe it's not always what you want, but in my case I'd like it.
In my case, I have a simple HashMap as my model. I create a one-way
property adapter for it (implementing PropertyAdapter). This of course
means that if I make changes to my Map, it won't be reflected to the bound
component, but that is fine.
The Map has default values set on them, and when I bind each key to a GWT
component, I want that GWT component to immediately get the models value.
Can we work this out, you think? Maybe a new overload, to(HasValue<?>
widget, boolean useExistingModelValue);
—
Reply to this email directly or view it on GitHub
https://github.com/ltearno/hexa.tools/issues/3.

A one-way binding does that yes, but I thought there was support for two-way binding. I.e. if the model is updated, the GUI is updated and if the GUI is updated, the model is updated.
Let me try the .log(String) method and I'll get back to you (with a code snippet too)!

Hi Anton,
Do you have any update on your problem ?
Please, maybe you could send few lines of code showing what does not work as expected ?
Thanks

So if the model has data in it before it is bound to the object it should map that data directly to its partner bind? This should already be the case from what I have tested.
        Binder.bind(this.setting, "type")
            .to(typeListBox);

This works fine, I would suggest you have a problem with your custom PropertyAdapter? When it calls getValue or setValue perhaps.

My GWT project is currently on hold so I haven't been working on this. However, my guess is that @BenDol is correct here.
Closing for now!


SSE_Event_Stream setup problem

I have just started using your SSE event stream experimentally, as Yadomi's event node seems to lose connection every month or so.
It seems with @tq-bit/node-red-contrib-server-sent-events v1.2.6 the SSE client node has perhaps changed the way it works. I'm using it for the first time, and wasn't able to get your inject nodes hooked up to it. Also it took me a while to get it working - I had to use an event of "message" and a header with my hue key, {"hue-application-key":"1234567"}. Either I might be misunderstanding the setup, or perhaps you could update the SSE_Event_Stream?
Thanks very much for your work by the way - your library here has been invaluable to me over the last few years.

@patrickharris
You need the
node-red-contrib-sse-client (0.2.4)
Then it should work.

Argh, my bad. Thanks very much for that Andreas. :)


Central Logout Not Working

I get the following:
Error Code: illegal_post_logout_redirect_uri
Description: The 'post_logout_redirect_uri' parameter must be an absolute URI that is whitelisted in the client app settings.

You have to add your app’s URL to the Logout redirect URIs on Okta to fix this.

On Mar 9, 2018, at 21:43, kdbeall<EMAIL_ADDRESS>wrote:
I get the following:
Error Code: illegal_post_logout_redirect_uri
Description: The 'post_logout_redirect_uri' parameter must be an absolute URI that is whitelisted in the client app settings.```
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you! @mraible

Looking at your 400 screenshot (in the address bar), it shows that your Logout Redirect URI should be http://localhost:49836/signout/callback.

Merci @mraible, J'ai eu le meme probleme et cela est la solution.

Hi,
I have set the value correctly in my App

Based on what is in the URL when logging out:
https://dev-4003858.okta.com/oauth2/default/v1/logout?post_logout_redirect_uri=https%3A%2F%2Fsitecore-demo-rg-170970-single.azurewebsites.net%2Fidentity%2Fpostexternallogout%3Fnonce%3Dc6671394839644a8acbb6e928743ee4e%26ReturnUrl%3Dhttps%253a%252f%252fsitecore-demo-rg-170970-single.azurewebsites.net%252fidentity%252flogin%252fshell%252fSitecoreIdentityServer&id_token_hint=eyJraWQiOiJxVjdIUkpidHkwRkhkcDVYaXcwdlJhRTJoY1VEVkEwSEZMei1EODFhTFdrIiwiYWxnIjoiUlMyNTYifQ.eyJzdWIiOiIwMHUxZGR6ZXJXNHlVdUVoZjVkNiIsInZlciI6MSwiaXNzIjoiaHR0cHM6Ly9kZXYtNDAwMzg1OC5va3RhLmNvbS9vYXV0aDIvZGVmYXVsdCIsImF1ZCI6IjBvYTFmMGtxNWxzMUNzcU5DNWQ2IiwiaWF0IjoxNjA2NzA2NDg0LCJleHAiOjE2MDY3MTAwODQsImp0aSI6IklELnhGR1NycHdna0JrSTJlSVZocExISGZsOVZJUHh1cXZRUXRSc2Jaek1KbTQiLCJhbXIiOlsicHdkIl0sImlkcCI6IjAwbzFkZHphejIzV2RuVXRDNWQ2Iiwibm9uY2UiOiI2Mzc0MjMwMzI4Mzg0MzQyNTguT0RZMk9UbGlZMlV0T1RRMVlTMDBaREF3TFdJME5qRXROalUxT0dJd056UTFPVGN5WVRka1pEWmhNakl0Tm1RMVpTMDBZbUppTFRsaU9UZ3RNbU14TWpRd01qY3laalJqIiwiYXV0aF90aW1lIjoxNjA2Njk2NzY0LCJhdF9oYXNoIjoid2dPWUt1VTdmdTZ5ZUdETTJ2SVduZyIsImNfaGFzaCI6IjIxWUlqMHhKbzFvTExiRnRwc1F2RWcifQ.ceS5Q7BQEH6WCiyjhS_ZJ-DCCyIx4y1rasTeij-o0b3RvqDowVhMS7PAxUr45fGp3PtpHRKgjogp9d-g6x2ANbr3ZkSvM48rUCJurPr9NXJlT822NsoL1NWCB9ZJaCeoAz4-CmP_zlJkRtPu1oHqBVDR2mHANmmDC3koj-iPrgOw-cBruMTedBoXkjpzOJ58-sgH3qE2i5vCTj0IZnC3IpfF8QC0NNeM0Pa2XeJ1LtspvsILmSBL8UwwHr-LYf72ci8Iw46gXU5BTeGi7tTZN4S8JXoDVgIUY1GIEHs9UiQzHwMy1sAY2-XHlYdm8eaUBz8K7I77X76Crhb0NMG9eg&x-client-SKU=ID_NET461&x-client-ver=<IP_ADDRESS>

But I am still having the same issue. Please help if you can.
Thank you.

Please send your issues to<EMAIL_ADDRESS>This issue is not monitored by developer support.

In my case I commented out the AuthenticationProperties that contains RedirectUri. This code was in my C# (MVC) Action !
HttpContext.GetOwinContext().Authentication.SignOut(
    //new AuthenticationProperties { RedirectUri = "/logout" },
    OpenIdConnectAuthenticationDefaults.AuthenticationType,
    CookieAuthenticationDefaults.AuthenticationType );

This worked for me. Hope this could help!


Create PDF version of website

Some users would prefer a PDF user guide. See e.g. https://github.com/MIT-LCP/mimic-code/issues/197

This is a duplicate of https://github.com/MIT-LCP/mimic-website/issues/6. It is straightforward to create a PDF by pandoccing the content directory from .md to .pdf, but the styling needs a little work.

oops x.x


[PYIC-2682] Create stub identities for correlation unhappy paths

Proposed changes
What changed
Added 2 new stub identities for DL IPV for correlation unhappy paths; non-correlating names and birthdates, respectively.
Why did it change
Added to enable non-correlation test scenarios
Issue tracking

PYI-2682

Checklists
Environment variables or secrets

No environment variables or secrets were added or changed

Other considerations

Unhappy paths being exercised by selecting cris from other identities instead


setup_loader() should not add unicode strings to os.environ

In djcelery/_init_.py:20, the function setup_loader() sets the CELERY_LOADER environment variable. However, the file has from __future__ import unicode_literals, so both key and values are actually set to unicode strings, while all other entries of os.environ normally are byte strings.
I can't quite interpret the official documentation as to whether unicode strings are allowed in the os.environ mapping object, but various other projects struggle with this and raise a TypeError: environment can only contain strings exception.
For example IPython, when starting its kernel, and https://github.com/mitsuhiko/werkzeug/issues/467, often used via django_extensions, both crash hard when djcelery is active.
This could be fixed by either removing the future import from this file, or by explicitly prefixing these literals as byte strings with b''. (Although I assume that syntax was only introduced in Python 2.7 and would break compatibility with <=2.6?)

This change needs to be reversed. With the change reversed, the tests work under Python2 and Python3.


Fix user page rankings

Currently, we call .annotate_with_rank().filter(user=user) to try to get user replays ranked. This doesn't work though, as the rank will apply to the filtered replays. Not really sure why, I don't understand DB stuff.
As a workaround, store ranks for all replays in a python dict, then apply them to a filtered queryset. This is cringe. Performance dropped by about 200ms. I'm sure there's a better way but idk what it is.
Fixes #353

This is too slow :/ Especially since if it's taking 200ms now, I bet the big-O answer will be even worse once we have more replays in the database. I'd rather see us just stop displaying ranks on pages other than the full scoreboard for now.
In the long term this problem does seem fixable; it should be possible to get the ranks of replays in a reasonable amount of time. Especially since we can clamp the big-O from both ends; we aren't fetching that many rows here, and in all cases we don't care about the ranks of anything but the top replays. But it might be tricky. These pages might be helpful in figuring out how to do it:
https://docs.djangoproject.com/en/4.2/topics/db/aggregation/#combining-multiple-aggregations
https://stackoverflow.com/questions/56567841/django-count-and-sum-annotations-interfere-with-each-other
Remember too that you can uncomment part of settings.py to log all SQL queries, which should make it clearer what's really going wrong.

Thanks for the links. I will try again some time next week, but if I can't get it to work, I'll just remove the feature for now.

You can always disable it for now and come back later when you have time to figure it out, too

Is using Django 4.2 an option? https://stackoverflow.com/questions/68289137/filtering-on-a-window-function-in-django

I don't see why not, if it works

However definitely do not use Django 4.2 in this PR; if it works, send that change alone as a separate PR, and then merge it in here

I still haven't found a way to get this to work with just a DB query. I'm still using a python dict, but I tried to get better performance by adding only top-3 replays to the dict.

On my machine, this function went from 0.045s to 0.145s.

Okay, I think that's fine then. In reality doing it this way isn't really that much slower than doing it on the db side; the main disadvantage comparatively is that it's kinda hacky (since you'd have to redo the same logic on other pages)

Ok, I extracted it and added some tests


Apply visibility-settings to form-validation

Short: The visibility-settings of fields don't apply to the validation- and saving-part of forms - just the display. (visibility = membership-level)
[applies to non-admins]
If I set the visibility of an form input to admin- and mod-only, the form itself is displayed correctly.
Means: A normal user can't see the input field.
But when the user saves/submits the form, the system validates even the "hidden" fields.
If I set the visibility of the username-field to admins-only, a normal user can't see it, but if he saves he gets an error (please check the form for correct input etc.).
If an admin-only field doesn't contain a validation-rule (example: not required), the data of this field will be empty, after the user saved.

Great finding!


Can GSS-NTLMSSP be used?

https://github.com/gssapi/gss-ntlmssp
https://packages.debian.org/sid/amd64/gss-ntlmssp/filelist
/etc/gss/mech.d/mech.ntlmssp.conf
/usr/lib/x86_64-linux-gnu/gssntlmssp/gssntlmssp.so
/usr/share/doc/gss-ntlmssp/README.md
/usr/share/doc/gss-ntlmssp/changelog.Debian.gz
/usr/share/doc/gss-ntlmssp/copyright
/usr/share/man/man8/gssntlmssp.8.gz

cat /etc/gss/mech.d/mech.ntlmssp.conf
# NTLMSSP mechanism plugin
#
# NOTE: to activate the NTLM SSP mechanism do the following
#     * FOR krb5 < 1.12: copy the content of this file in /etc/gss/mech
#     * FOR krb5 >= 1.12: copy this file in /etc/gss/mech.d
#
# Mechanism Name        Object Identifier               Shared Library Path                     Other Options
gssntlmssp_v1           <IP_ADDRESS>.4.1.3<IP_ADDRESS>          /usr/lib/x86_64-linux-gnu/gssntlmssp/gssntlmssp.so

I don't know, and at the moment I don't have the time (or need) to put in the research work. If you would like to do it, I will happily review a PR.


Fix compensate_phase of pulse.builder.frequency_offset

Summary
A factor of 2*pi was previously missing from the appended ShiftPhase instruction.
Details and comments

The previous behavior of the backends was an error and not as defined by are execution model. The update to the backends was a fix of the previous error.


Matching validation issues

I created the following issues using the live demo:
Issue 1:

Enter a password into the password field.
Enter an identical password into the confirmation field.
Before submit, go back and change the password field.
Passwords validate even though they do not match anymore.

Issue 2:

Remove the 'required' validation condition of the password and confirmation inputs.
Enter a password into the password field.
Passwords validate even though the confirmation is blank.

Yes I can see the behavior since only changing the value on the confirmation input will trigger a validation action, basically there is no watch on the parent input (which is why the input stays valid after changing your password). I'm not crazy on having Angular-Validation to add multiple watch, but I don't think there is any other way to do it.
So I will see if I can add a $watch on the parent input (like the password field for example). That would probably fix both your issues when it becomes available.

Added a $watch on the parent field (for example Password) so that the matching input gets notified (example Password Confirmation).
Also note that it affects all matching validators, that is:

match: n,
different: n

Both your issues should now be fixed, if not let me know
Thanks for reporting it


System not working as it should: All AIX systems (except one)

Please put the system name in the title of this issue.

Link to any log file showing the problem:

Please describe the issue:
While looking at the jenkins configurations (via https://ci.adoptium.net/computer/) I noticed that the NTP time was not in sync for the AIX servers (except for one).

Looking at the AIX servers there are a couple of concerns:
a) some do not have NTP configured or installed (while the ansible setup is suppossed to ensure that it is uniform)
b) the one good system does NOT have the expected configuration (using local time server).
c) The local time server is reached, but ut reports mode: unspecified, rather than "server".

I have opened a ticket with OSU to ask that they look at the time server. If this takes too long I'll reconfigure the other (in error) servers to use an external server.

Closing this - as the servers all seem to be consistent with the rest of the systems (either in sync, or 14-16 seconds behind), as are many other OS flavors.
Maybe the "14 sec" behind is being caused by the testing system being 14 seconds ahead of what it should be.


Update build.gradle

Update the gradle version.
Hello.  Would you mind making a commit that ups the gradle version.
Thanks.

I wanted to inform you that I've forked your AR Flutter plugin to create an updated version available here: https://pub.dev/packages/ar_flutter_plugin_flutterflow. This fork was created because the original plugin had not been updated since 2022. Changes include an update to the AR Core endpoint, a gradle upgrade, and compatibility with FlutterFlow. I hope this fork will be useful to the community.
Best Regards,