Lesson 01

This is actually my very first pull request, so it might be a little wonky.
Most of the text is from my SwiftUI by Tutorials book chapter. I haven't actually run RGBullsEye on my phone, but I noticed with WaitForIt that swiping left/right doesn't seem to work — it doesn't move to the next/previous element, just goes "boink".
Note that you must use Xcode 15.0.1 for this lesson. Xcode 15.2 still doesn't have a working accessibility inspector.

This is actually my very first pull request, so it might be a little wonky.
Most of the text is from my SwiftUI by Tutorials book chapter. I haven't actually run RGBullsEye on my phone, but I noticed with WaitForIt that swiping left/right doesn't seem to work — it doesn't move to the next/previous element, just goes "boink".
Note that you must use Xcode 15.0.1 for this lesson. Xcode 15.2 still doesn't have a working accessibility inspector.

See comments in the Asana task :)

I manually merged the files from the 3 branches to the release branch.
I didn't want to worry about conflicts tbh since some files were deleted from one branch and updated on another


Feature/mp4copy

Implement the functions needed to play and record ISO-BMFF for H.264 and AAC

Add examples/mp4copy
Add Mp4Writer and TrackWriter
Rename atoms to mp4box
Add common media types

Thanks!


Not compatible with React v15

This will no longer work with React v15
function rm (el) {
    el.removeAttribute('data-reactid');
  }

since React 15 dropped data-reactid.

Closing this because it's been a year and a half, the project hasn't had a commit in two years, and I'm pretty sure dragula just plain won't work with React > 15.


Casper tests should have a better "overall result" output

Now that we are running our casper tests as basically 10 suites run in a row, rather than a single long suite, we should generate some sort of overall, final, aggregate output showing the total time consumed and number of tests passed/failed.

can i work on this?

sure, go for it!  tools/test-js-with-casper is the main script for that.

Just elapsed time and number of tests passed/failed is I think all we need.

Hi, I am new to this project and open source in general. I have tried my hand at this issue and made my first pull request! @zulipbot claim

@zulipbot claim

Closing since we're planning to remove the casper tests in favor of puppeteer.


[bug]: Trojan detected during the installation from launcher

Is there an existing issue for this problem?

[X] I have searched the existing issues

Operating system
Windows
GPU vendor
Nvidia (CUDA)
GPU model
RTX 3060ti
GPU VRAM
No response
Version number
4.5.5RC1
Browser
Edge
Python dependencies
No response
What happened
During the installation with Invoke Launcher, I got trojan warning from my Windows Defender.

What you expected to happen
Why trojan?
How to reproduce the problem
No response
Additional context
No response
Discord username
No response

Can confirm, my windows 11 also flags same file.

Same here. Bitdefender kicked in, wouldn't let me install it.

Same here

Confirmed, same here. May be a false positive, but will hold off on installing until I see this fixed to be safe


Atlas hotfixes

About the PR

Cargo airlock now opens for people with Cargo access.
Uniform printer added for HoP in the room adjoining the hopline.
Added crew monitoring server in bridge's substation room (no room in telecomms)
CMO's locker fill has been replaced with hardsuitfill (no hardsuit storage was initially present).
Salvage expedition console spawns in salvage's gear-up room.
Botany's water tank has been replaced with a high-capacity water tank.
Defibrillators were added.
Medfab was added.
Service and Security camera routers are now definitely powered. (Was not able to replicate this issue initially.)
Service cameras now also exist. (There weren't any.)

I AM NOT THIS MAP'S MAINTAINER. Explicit approval for merging this is required by @deltanedas , who is this map's maintainer.
Why / Balance

https://github.com/space-wizards/space-station-14/issues/21776 (Issue closes when this is merged.)
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Defibs and medfab:

Uniform printer and crewmon server:

Exped console:

Botany water tank:

Changelog

no cl, no fun

:trollface: thanks for fixes


chore: add changelog entry for metallb v0.13.5 bump

Codecov Report

Merging #372 (877c1dc) into main (c57a1a9) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #372   +/-   ##
=====================================
  Coverage   8.57%   8.57%
=====================================
  Files          5       5
  Lines        828     828
=====================================
  Hits          71      71
  Misses       752     752
  Partials       5       5

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


DialogFragment overlap by status bar on Android

I have a issue with overlapping when displaying a form. Form layout is located under the status bar as you can see on the screenshot:

This only happens on devices without display cutout. Here's what it looks like with display cutout:

As you can see the status bar changes color.
My style.xml:
<style name="AppTheme" parent="Theme.AppCompat.DayNight.NoActionBar">
    <item name="android:windowTranslucentStatus">true</item>
    <item name="colorPrimary">@color/colorPrimary</item>
    ...
  </style>

Tested on Android 13.
Is there a way to prevent this?

Have the same issue on Android 13 (Pixel 4 API 33)

Thank you for your issue submission.
On December 1, we will be are closing out the issues tabs on the Usabilla GitHub repos. We will continue supporting the apps/SDKs via our Help Center and support team. Information on how to contact support will be provided in the readme sections of the repos. By consolidating our contact channels, our team can monitor and reply more efficiently.
If the issue you have reached out about persists, please send us an email at<EMAIL_ADDRESS>Please include all details provided here in your email, along with any other details that may be helpful.
-GetFeedback Digital Team


rootless-install.sh: update to 20.10.7

@AkihiroSuda @StefanScherer PTAL

thx! Bringing this one in


Enhancement: Anti-spam option

In the original, you can't press the buttons unless there is a note to hit as it will be a miss, but in kade it allows you to press whenever without consequence. I think it would be good to have an option to swap between these ways of treating a random or "wrong" press, because currently if every button was pressed every frame then you would beat any song (not with a great score, but beaten nonetheless).

probably in week 7 we'll add back old input


[FEATURE] Add new dataset.steppedLine type

FEATURE
Add new dataset.steppedLine type to simplify migration from Google Charts.
Suggested type: middle (or any other name)
Stepped lines in Google Charts begins outside bar-columns.
My current solution jsFiddle
Current Behavior

Google Charts Behavior

@bril-andrew seems a good enhancement, can you submit a PR with docs and tests?
@ericnkatz any thoughts, since you implemented before and after?

@bril-andrew  @simonbrunel looks good to me.
I used 'before' and 'after' because that's what D3 called the two styles of interpolation.

To obtain the desired 'middle' behaviour, wouldn't it suffice a small change to function lineTo in src/helpers/helpers.canvas.js? Something like adding these 5 lines (actually 4 when putting the else and following if in the same line):
    lineTo: function(ctx, previous, target, flip) {
        if (target.steppedLine) {
+            if (target.steppedLine === 'middle') {
+                var midx = (previous.x + target.x)/2;
+                ctx.lineTo(midx, previous.y);
+                ctx.lineTo(midx, target.y);
+            } else
             if ((target.steppedLine === 'after' && !flip) || (target.steppedLine !== 'after' && flip)) {

I guessed that the flip parameter should be a no-op in the case of 'middle'. Nevertheless, I am not familiar enough with the code, so this might be breaking stuff somewhere.


feat: Automate user invitation

the plugin doesnt work and creates a whole new REST API so its best to tear the whole stack down, closing


Improve ClearType

I would like to see much better rendering of type in Windows.

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

ID: bf2300b8-71e0-a6fe-b12f-f47b2b9564c1
Version Independent ID: e771c103-3be0-7747-0084-be7bdef7bf63
Content: Microsoft ClearType - Typography
Content Source: typographydocs/cleartype/index.md
Product: windows
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

This is product feedback, not a doc issue. For Windows 10, please provide this feedback using the Feedback Hub.

You can get much better rendering of type in Windows by disabling ClearType.

@PiotrGrochowski: This is product feedback, not a doc issue. For Windows 10, please provide this feedback using the Feedback Hub.


Add servicerouter information to docs

Add a reference to the servicerouter documentation and provide a simple usage example.

Added a few minor comments, related to naming and capitalisation.
changelog.md should also be updated.

In order to unblock today's release, I just updated the comments in "servicerouter.py".
I didn't modify the description of HAPROXY_GROUP, because I am not sure I completely understood what it does. We can update it later.


[Request for comment] Fix for fields being dropped in Custom Search Command.

Issue
An issue has been reported on the current Python SDK stating that the Custom Search Command drops fields where the fields are selectively set as part of the command.
As per the analysis of the existing code, fields are being dropped because the RecordWriter in splunklib/searchcommands/internals.py uses the first row of results in order to determine the field names.
Proposed solution
Below is our current approach to solve this issue:
To prevent fields from being dropped, we iterate over the results and create a set of all possible field names set before writing the headers for the output. This implementation can be found in a branch here: https://github.com/splunk/splunk-sdk-python/compare/develop...DVPL-8354.
Impact
The above solution does solve the original problem but it affects the performance of the Custom Search Command. As per the analysis, we found that the proposed solution would take approximately 15-25% more time depending on the system. This was tested with datasets of increasing sizes and the performance hit is relatively the same.
To effectively improve the performance, the underlying implementation of the Custom Search Command would require changes.
Feedback requested
We would love your feedback on this issue to help us decide on the fix.

Is a 15-25% more time in the performance of Custom Search Commands acceptable for your application in favor of having the bug fix? Please indicate with an upvote or downvote on this issue.
Please post comments below to provide any suggestions to improve the performance without changing underlying structure and implementation of Custom Search Commands.

I wonder if it would be worth implementing this as an option, which is default disabled, so previous functionality and performance is maintained, with an "easy button" solution for those that want it, or the ability to determine all needed fields in the custom code as was previously done to work around this issue.

We have modified our approach to handle the Custom Search Command issue regarding the dropping of the fields.

we have introduced a new method add_field, which is required to be called while adding any new custom fields to the records being streamed for Custom Search Command.
add_field method, maintains a list of custom fields which will be appended to the existing fields before writing it to the output file.

The implementation can be found here ..develop...DVPL-9943 The updated approach has ~10% impact on the performance . On how to use the updated approach please refer the updated README file in the above link.
Note : Use of add_field method to add a new custom field is _optional_. Hence the previous implementation and the functionality will still be valid and it won't have any performance impact as well.
Only for the CSCs where field dropping is observed, add_field method is to be used .
Feedback requested
We would love your feedback/suggestion on the update approach. Please feel free to comment your views on this.

I have attempted to implment https://github.com/splunk/splunk-sdk-python/pull/407/files in my streaming search command https://splunkbase.splunk.com/app/6161/ but I think there is a small bug.
How I have got the implementation working is by simply calling self._record_writer.custom_fields.add(key) directly when I am adding a new field.
The version currently deployed to Splunkbase uses the previous approch https://github.com/splunk/splunk-sdk-python/compare/develop...DVPL-8354 successfully, but I do like the level of control the new approch provides.
Honestly calling a function to create new fields manually is the most appealing, as it gives ultimate flexabiltiy to the developer.

Hi All, we have a new SDK release 1.6.18 with the above mentioned approach and the suggested changes. Try out the latest release and let us know if this help resolved your problems.

I just applied 1.6.18 to https://splunkbase.splunk.com/app/6161 and it appears to work perfectly using self.add_field


feature request: auto lock when time-out

Can you add auto lock and give option for example if the computer/phone is inactive for 5 or 10 minutes, it prompts the password to unlock?
thank you so much for implementing the encription feature, extremely useful!

Okay, but you need to wait a few days, because I'm busy with work these days.


More descriptive error than "Unable to Connect"

Two days ago I was able to start my bot and have it connect it was acting completely as expected. Last night when trying to start, I am unable to connect. Therefore I am wondering if it possible to produce a more descriptive error than, "Unable to Connect"
Actual behaviour:
Unable to connect
Expected behaviour:
Connect and continue on.
Error log:
[09:03] error: Reconnecting in 2 seconds.. (node:6348) UnhandledPromiseRejectionWarning: Unable to connect. (node:6348) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was  not handled with .catch(). (rejection id: 1) (node:6348) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Server configuration

Operating system: Windows 10
Node version (if applicable): 10.15.3
NPM version (if applicable): 6.4.1
tmi.js version: 1.4.5

I believe issue was caused by a server and fixed on its own. Closing issue.


Advices

Hellow @mgechev and @NathanWalker, i have created this repo to try use cordova with the seed.
I know that @NathanWalker wanted something like this on his seed, and initially i was to make a PR, but i think it will conflict with the already proposed there "the same multiplatforms (windows, ios, android, linux, osx and web of course)" and maybe give more complexity.
@mgechev, i have to make some changes at the gulp tasks (almost all) to change the package 'gulp' to vinyl-fs, because it allow me to use Symbolic Link (got that idea from the advanced-seed) and added some arguments to the dev_dest and prod_dest at seed.config. Got to ask, would you merge those at the original seed? (Not the cordova itself, only the gulp and seed.config changes). With symbolic link we enable use the same files from the web at the cordova and vice-versa.
Beside those questions, i would ask you guys to avaliate this project and see if have worthed the work!
Anyway, any advice is welcome!

@jvitor83 This is very exciting work, nice job.
I want to dig into this more when I have time. On the surface I might suggest integrating with Ionic v2 since it uses cordova. That would make for a pretty exciting integration I believe. Although it's also nice to have just cordova (I could see both being alternate seed paths).
For the advanced seed, the reason I wouldn't want to merge something like this to master is just simply more to maintain. The stack there already is a bit to maintain. But a separate seed like you have here is a great way forward. Will look more when I can and look fwd to chatting more about it in near future. 👍

Thanks for the feedback @NathanWalker.
In a near future i will take a look at Ionic2 integration (and how to apply to this seed).
Initially i don't think in let ionic2 out-of-box (because people could use others libs to the project, and that is the beauty of cordova), but maybe in some section at the readme.md or wiki to how to incorporate that. But maybe a branch could be a way to go too.
Any way, thanks and looking forward for you to try this seed!


[qtwebengine] fix debug builds

simply fix debug builds by applying the same change to LogStreamer<V,T,Ts...>

@Neumann-A While you are working on this, could you check if the binaries for the debug build are copied into the right directories? I seem to remember that I was missing a debug version of the QtWebEngineProcess executable. I was unable to pursue the matter, due to the other build error.
If this is out of scope for this pull request, I can check if the issue still exists once I can build the port and create a separate issue, if need be. Thank you.

Vcpkg generally tries to only keep release versions of executables around. Is the debug version really needed?

Vcpkg generally tries to only keep release versions of executables around. Is the debug version really needed?

For our use case, yes. We have tried to debug our application with the release build of the qt web engine process, but we could not get that to work.
If it is too much of a hassle, we can manually copy the debug binaries from the build tree to the respective folders, but it would be nice if we could skip that step.

Debug executables are ultra-heavy for static triplets.

is the debug version really needed?

Qtwebengine only allows dynamic builds.  So it is definitely out of scope for this PR.

@carsten-grimm-at-ipolog: Yeah the debug exe gets installed however it gets removed by vcpkg_copy_tools. So it is definitely out of scope for this PR.

Thank you for taking the time to look into this and thank you for the insight. This might help us build a better workaround on our end.

https://github.com/microsoft/vcpkg/pull/35640 is testing with the updated compiler


Add config = Engine flag for UCLASS default/project user configs

As far as I can see this doesn't cause any conflicts, although I couldn't find any info about GlobalUserConfig needing Config = Engine in it, so I just left that one for now.

I added this as a setting as requested. I attempted to build the xinput myself to test, but when loading the game there is an error [17:32:55] Fatal Error: Call to K32GetModuleBaseNameA failed. Error Code: 6 but I don't have time right now to figure that one out.

This seems to work as intended.


Payment Issues From USA

I just wanted to report that my bank informed me that, because the money was being sent to Belgium, they will decline every payment I initiate with you unless I call them in advance to notify them I will be sending money to Belgium.
It would be great if you guys accepted some other payment method than a credit card, such as cryptocurrency or Paypal.  This would be much more convenient for situations like this.

(note: the project I was trying to back was https://opencollective.com/manyverse/contribute )

@fuzzyTew Sad from your bank to be so drastic.
But I agree, this is why allowing a diversity of payment methods is important.

See these related issue about better support for PayPal:

https://github.com/opencollective/opencollective/issues/1141
https://github.com/opencollective/opencollective/issues/1655

Definitely something we need to work on. Closing this one to follow up in related issues


Unable to generate ONNX graph for GPT-2.

Hello, I'm Jongmin Kim, a student studying DNN accelerator, and I have been interested in your simulator.
According to the README, the following command should generate an ONNX graph for GPT-2:
python3 ./scripts/generate_transformer_onnx.py --model gpt2
However, it results in the following error message:

File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 2007, in _run_symbolic_function
raise errors.UnsupportedOperatorError(
torch.onnx.errors.UnsupportedOperatorError: Exporting the operator 'aten::scaled_dot_product_attention' to ONNX opset version 11 is not supported. Support for this operator was added in version 14, try exporting with this version.
Traceback (most recent call last):

I have verified that the version is 22, and I don’t believe this is the issue, as the latest versions of onnx and onnxruntime are installed, as specified in the Dockerfile.
I am currently attempting to export ONNX graphs for GPT-2, Llama 3, and OPT using the ONNX optimizer (AutoModelForXXX, ORTModelForXXX, etc.).
However, I’m wondering if there are any simpler or more efficient methods to resolve this issue.
Thank you for your time, and I look forward to your guidance.

Hi @JongminKim-KU ,
This issue was caused by a version mismatch and has been fixed and uploaded.  Additionally, a guide for the LLM mode has been added to the README.
If you encounter any further issues, please don't hesitate to let us know!


Impossible to Map Introspection Disabled GQL APIs for These Specific URLS (Apollo Servers Detected)

If anyone could give me a hand -- running into this issue with pretty much every GraphQL enumeration / mapping tool or library for the following URLs. I'm not sure if there is some additional variable I'm missing that I need to be including or potentially a different URL? No clue.

https://api.hypedrop.com/graphql
[Apollo | Introspection Disabled]

https://api.hypeup.com/graphql
[Apollo | Introspection Disabled]

http://api.csgoroll.com/graphql
[Apollo | Introspection Disabled]

Seriously any and all help appreciated as this has been driving me insane!
Thank you so much for an amazing repo & tool. I hope to be able to utilize it!
x

I am sorry for extremely late response. Basically, introspection is a feature of GraphQL server that is useful for debugging, because it allows client to request full API map. But in most correctly configured production server, introspection is disabled. This is when Clairvoyance is shining: it sends requests for objects with variety of names (e.g. "user", "company") and that's where another feature, called field suggestion, comes into play. For example, if there is a field called "users", then GraphQL will suggest us this field and by trying many similar fields Clairvoyance, in some way, bruteforce API map.
Hopefully, it answers your question. If so, feel free to close the issue. But in case if you have further questions, you can ask them below.


logstash_internal unauthorized on indices

The new logstash_internal user in configs cannot write on indices in runtime (v8.0.1):

unauthorized for user [logstash_internal] with roles [logstash_writer] on indices

Solution: In ELK in Stack Management/Users assign to "logstash_internal" user the role as for "elastic" user

@pkral3 thanks for reporting 👍
May I ask what indices you are writing to? The default role has permissions on the following indices:

logs-generic-default (the data stream targeted by Logstash's default config)
logstash-* (the index pattern when data streams are disabled in Logtash's config)
ecs-logstash-* (the index pattern when data streams are disabled in Logtash's config and ECS schema is enabled)
logstash (alias used for managing index lifecycle)
ecs-logstash (alias used for managing index lifecycle when ECS schema is enabled)

Custom indices that were ok before logstash_internal are not writeable now.
As an example in logstash/pipeline/logstash.conf define any esIndex
filter {
if [app_id] == "process1" {
grok {
match => { "message" => "[%{TIMESTAMP_ISO8601:logtimestamp}] [%{LOGLEVEL:loglevel}%{SPACE}] [%{GREEDYDATA:hostname}] [%{GREEDYDATA:environment}] [%{GREEDYDATA:project}] [%{GREEDYDATA:jobname}_%{GREEDYDATA:jobversion}] [%{GREEDYDATA:transaction}] - %{GREEDYDATA:logmessage}" }
}
# create an elasticsearch index per month
mutate { add_field => { "esIndex" => "process1-%{+YYYY.MM}" }}
}
}
process1 index even existing from before with data in ELK, is not writeable with logstash_internal user (until it has not the role as elastic user has). It worked ok before, nok with 8.0.1 just with update.

@pkral3 OK that's expected because logstash_internal is just assigned a default role during the initial stack startup (logstash_writer), but that role is only suitable for a fairly default log-oriented Logstash pipeline.
Obviously, we can not anticipate all the indices names that end users are going to target.
We can also not open the permissions to much because that would be quite unsecure, and we don't want to encourage people to use overprivileged users.
I can recommend a few options:

Create another role via Kibana (e.g. logstash_custom) and assign it the same permissions as the logstash_writer role, just with different indices names, based on your own Logstash config (e.g. process1-*).
Edit the existing logstash_writer role created by docker-elk, and change/rename the indices patterns it contains.
Give the logstash_internal user admin extended permissions, like you did, with the risk that this user can be abused to perform more changes than it is supposed to (e.g. perform modifications to the Elasticsearch cluster, read secrets, etc.)

So there is no solution just to update, build and up -d as before now.

Are you suggesting that we make logstash_internal a superadmin like before?


[Serial] USB has mysterious ~50 byte slew on connection

the issue still exists, but is now circumvented by the pin configuration query "handshake" which is repeated until complete. Thoughts on keeping this issue open?

Closing since it has been worked around


Type 'Element' is not assignable to type 'ReactNode'.

Component or Package Name: @jsx-email/all
Component or Package Version: 3.0.1
@jsxp-email/cli Version?: Unknown
Operating System (or Browser): Macos Sonoma
Node Version: 18.17.1
Link to reproduction (⚠ read below):

While building the next app this error occurs
$ next build
 ✓ Creating an optimized production build
 ✓ Compiled successfully
   Linting and checking validity of types  ...Failed to compile.

./email/templates/PersonalEmail.tsx:31:7
Type error: Type 'Element' is not assignable to type 'ReactNode'.
  Property 'children' is missing in type 'Element' but required in type 'ReactPortal'.

  29 |   return (
  30 |     <Tailwind>
> 31 |       <Html>
     |       ^
  32 |         <Head>
  33 |           <meta name="color-scheme" content="light dark" />
  34 |           <meta name="supported-color-schemes" content="light dark" />
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Expected Behavior
Should build properly as it was building with react-email
Actual Behavior
Error
Additional Information
None

Hey please give the instructions in the issue template another read. We require reproductions so we can help get your issue solved for your scenario.

Hey please give the instructions in the issue template another read. We require reproductions so we can help get your issue solved for your scenario.

This is the repo https://github.com/shahriyardx/shahriyardx

That's just your entire app :)

Provide a minimal repository link (Read https://gist.github.com/Rich-Harris/88c5fc2ac6dc941b22e7996af05d70ff for instructions).
Please use NPM for installing dependencies!
These may take more time to triage than the other options.

Really, really want to help you. But 🙏 please do read our instructions. They really are helpful.


Don't complain about weave bridge routes clashing

Added --ignore-iface option to netcheck and use it to ignore weave bridge route
Fixes #1076

needs to be branched off 1.0
it should be possible to supply multiple --ignore-iface options.

it should be possible to supply multiple --ignore-iface options.

Why?

Why?

It's an obvious generalisation. If you were to write a netcheck independently of weave, you'd certainly want this.

Do you want them all on 1.0, or do you want to maintain the separate threads of development?

Let's not bother putting this on 1.0 then; until we have a few users complaining about it.

OK; it's in branch 1076-ignore-iface-1.0 when we need it.
(I had already done it by the time I'd come back to read your comment)

OK; it's in branch 1076-ignore-iface-1.0 when we need it.

Have looked at it. Drags in too much. Delete. (though perhaps push the branch to your own repo).

I did it with a map instead.

needs re-basing, by the looks of it.

Rebased


使用pip install fairy-doc[cpu]显示没有找到版本

ERROR: Could not find a version that satisfies the requirement fairy-doc[cpu] (from versions: none)
ERROR: No matching distribution found for fairy-doc[cpu]

it seems caused by incorrect python version, you can try the following instructions
conda create -n py310 python=3.10
conda activate py310
pip install fairy-doc[cpu]

没什么卵用啊兄嘚


Frontend - Ribbon App Titles & Branding

Context
Desire to rebrand/redesign the Grant Identification Tool
Users of the Grant Finder also want a way to easily access their "My Profile" user account and manage their team(s) no matter what screen they are currently on
This is a ticket for Grant Finder only and assumption is that no changes will impact ARPA users
Open Questions

None

Current State

In the Grant Finder Tool, a Quick Menu current exists in the top ribbon, with limited fields available (see below).
Image
Future State (design)
The Grant Finder will have an improved Quick Menu that can be easily accessed from the top ribbon of the tool. improvements include:

updating "Grant Identification Tool" name to "Federal Grant Finder"
updating colors/fonts/styles to match figma file
updating display of profile on the header to display Profile circle with initials, user's full name, and name of Organization

Future State Mock Figma:
https://www.figma.com/file/gi975Am6lHBFbjCZeVBzb5/Settings?type=design&node-id=697-18353&mode=design&t=b1u9OLalI1W1AUwl-4
Federal Grant Finder

@ClaireValdivia we have other tickets tracking the profile changes, right? I have a branch now with the toolbar color change and tool name change, but not sure if I should wait on doing anything with the profile.


HWY-167: Validate incoming votes for LNC & include endorsements for our own votes.

This code requires that whenever A directly cites B, and B cites an equivocator without accusing them as faulty, then B must be endorsed. That's a much stronger criterion than the LNC: It implies the LNC, but may redundantly require much more endorsements.
If that's intended, I'm okay with merging it that way, but maybe we should add a TODO that this should still be refined?
(Or shouldn't it? In that case we need to discuss and evaluate whether this criterion does the job, too.)

This code requires that whenever A directly cites B, and B cites an equivocator without accusing them as faulty, then B must be endorsed. That's a much stronger criterion than the LNC: It implies the LNC, but may redundantly require much more endorsements.
If that's intended, I'm okay with merging it that way, but maybe we should add a TODO that this should still be refined?
(Or shouldn't it? In that case we need to discuss and evaluate whether this criterion does the job, too.)

It was intended but I didn't realize it's that different. Let me think.

Superseded by #534


Variation track data

Outstanding: Colour variants by type if possible

In the ensembl browser.....different types of variation have different colour...

details of the SNP appear in the pop-up.
Not sure if these are available in JBrowse.
For  a while we might need to replace the Ensembl link... if we can't host this dataset easily in the next month or so we could tell people that it is still available.

it might not be a bad idea to have a permanent "external link" to variation data for a gene too.
the Ensembl view for this is quite good,
http://fungi.ensembl.org/Schizosaccharomyces_pombe/Location/View?db=core;g=SPCC18B5.03;r=III:721788-725835;t=SPCC18B5.03.1
each variant links to this info (which I resume is pulled from the variation database)

(although a link from JBrowse to the appropriate SNP in the DB will do  for this meta data I have no idea how this connection is made)

I've had a look at the variation data files found by Midori.  We'll need to fix the chromosome IDs in the VCF files before we can load them into JBrowse.  It's currently I, II, III etc.
Also there seem to be some tabs missing for the mitochrodrial data in both files.  It looks like there should be tabs after the "MTR" in the first column:
III     2431124 .       C       CTGCATAGCATCAAACCGAGAAT 106253  MEAN_ALLE
III     2431989 .       ACT     A       6982.84 DP;MEAN_ALLELE_BALANCE  A
III     2433201 .       T       TG      118279  MQ      AC=49;ACMI=8;ACMJ
MTR5053 UNION_BC_k31_var_139112_sub_indel_2     GT      G       .       P
MTR5065 .       A       AC      5808.74 MEAN_ALLELE_BALANCE;MQ  AC=2;ACMI
MTR5065 UNION_BC_k31_var_139112_sub_indel_3     A       AC      .       P
MTR5084 UNION_BC_k31_var_139112_sub_indel_4     AT      A       .       P
MTR5155 .       TAC     T       449.65  DP;MQ;QD        AC=3;ACMI=3;ACMJ=

Actually loading the data doesn't look too bad:
http://gmod.org/wiki/JBrowse_FAQ#How_do_I_set_up_a_VCF_file.3F

(although a link from JBrowse to the appropriate SNP in the DB will do for this meta data I have no idea how this connection is made)

I'm not sure how to do that either.  It will be easier to work out once we have a VCF track loaded.

We'll need to fix the chromosome IDs in the VCF files before we can load them into JBrowse. It's currently I, II, III etc.

Note to self, this issue has the new and old IDs:
https://github.com/pombase/website/issues/671

I have a feeling this will be the complicated part.... I * think*, but I am not sure, that there must be
IDs assigned for individual SNPs.

It looks like Ensembl assign an ID for each variant.  Those IDs aren't in the VCF file so won't be available in JBrowse.

Presumably Ensembl have an easy way to access these based on sequence

Could you explain that?  I don't understand.

I don't think Ensembl assigns them, I think the variation database assigns them.....and Ensembl use them...

I'll have a poke around

Maybe we just link to this?
https://www.ebi.ac.uk/eva/?Variant Browser
Although I can't find the pombe data?
In true EBI style they have found some interesting organisms  the lesser known "500 V1" and the  "5.0.1"

ffs from here they do have "fission yeast" as an option
https://www.ebi.ac.uk/eva/?Study-Browser&browserType=sgv
but  checking the boxes does not appear to filter anything????

Midori found lots of files on the EBI FTP site.  Which ones do we need to display?  Here's the list:
all.pi.wig
chr1.pi.wig
chr2.pi.wig
chr3.pi.wig
LTR-insertions.bed
mito.pi.wig
Spombe.2013-01-02.filt3c.nr57-final.snps.anno-snpeff3.vcf.gz
Spombe.indels.unique.nr57.final.annotated.2014-01-23.vcf.gz

We also have two other files that they sent in the big dump of all the Ensembl code and data:
DJeffares_Diversity.bw
LTR-insertions.PMID.25665008.bed

Midori found lots of files on the EBI FTP site. Which ones do we need to display? Here's the list:

I'm going to need help with this.

is this helpful? https://www.ebi.ac.uk/ena/data/view/PRJEB2733

wow, there is no indication what that data is whatsoever.
F no
A no
I no
R no
Great proliferation  of identifiers though..... (five per sample)

I clicked on the "analysis files" tab.
there are 6 files listed (VCF)
ERZ289252 S. pombe indels MTR
ERZ289253 S. pombe SNPs MTR
ERZ289254 S. pombe indels
ERZ289255 S. pombe SNPs
ERZ289256 S. pombe SNPs AB325691
ERZ289257 S. pombe indels AB325691
it looks like they bundled all indels into one track, and all snps into one track, and then had one track for everything

adding track descriptions for these.
@djeffares in the paper you report the accession numbers in dbSNP for the SNPs 974514578-974688138
but this ID is for a human gene?  https://www.ncbi.nlm.nih.gov/snp/?term=974688138
Also are you happy with these track labels for the 3 tracks or can they be tweaked?
Indels and SNPs in 57 non-clonal S. Pombe strains
Indels in 57 non-clonal S. Pombe strains
SNPs in 57 non-clonal S. Pombe strains

Ah, I also found this @kimrutherford from an email exchange April 17 between all of us
from Dan:
"Thanks for looking into this.  I put a new set of SNP, indel and structural variation calls in vcf format on figshare here: https://figshare.com/projects/fission_yeast_structural_variation/15798"

Hi all,
Yes, this is the best source. These are variant calls for the 161 strains though (not just the 57 non clonal). So its a better, more complete data set.
For @kimrutherford https://github.com/kimrutherford
Indels are here:
https://figshare.com/s/bf9075c15a9b481e2001 https://figshare.com/s/bf9075c15a9b481e2001
SNPs are here:
https://figshare.com/s/5e556e4698b128181cb8 https://figshare.com/s/5e556e4698b128181cb8
Thanks! Let me know if problems come up.
cheers,
Dan
On 15 Jun 2018, at 08:14, Antonia<EMAIL_ADDRESS>wrote:
Ah, I also found this @kimrutherford https://github.com/kimrutherford from an email exchange April 17 between all of us
from Dan:
"Thanks for looking into this. I put a new set of SNP, indel and structural variation calls in vcf format on figshare here: https://figshare.com/projects/fission_yeast_structural_variation/15798 https://figshare.com/projects/fission_yeast_structural_variation/15798"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/pombase/website/issues/729#issuecomment-397534577, or mute the thread https://github.com/notifications/unsubscribe-auth/APRzycu1Yhn0Hm8Mk9tTBOz2hhFIdvLIks5t817HgaJpZM4TNxeq.

Ok, removed the line for the combined track and updated the descriptions to
Indels in 161 S. pombe natural isolates
SNPs in 161 S. pombe natural isolates

The two new tracks are live:
https://www.pombase.org/jbrowse/?loc=II%3A2007265..2008224&tracks=PomBase forward strand features%2CDNA sequence%2CPomBase reverse strand features%2CIndels in 161 S. pombe natural isolates%2CSNPs in 161 S. pombe natural isolates&highlight=
Sorry for the delay Dan.  Please let us know if you spot any problems.

Looks great - thanks Kim!
cheers,
Dan
On 21 Jun 2018, at 12:42, Kim Rutherford<EMAIL_ADDRESS>wrote:
The two new tracks are live:
https://www.pombase.org/jbrowse/?loc=II%3A2007265..2008224&tracks=PomBase forward strand features%2CDNA sequence%2CPomBase reverse strand features%2CIndels in 161 S. pombe natural isolates%2CSNPs in 161 S. pombe natural isolates&highlight= https://www.pombase.org/jbrowse/?loc=II%3A2007265..2008224&tracks=PomBase forward strand features%2CDNA sequence%2CPomBase reverse strand features%2CIndels in 161 S. pombe natural isolates%2CSNPs in 161 S. pombe natural isolates&highlight=
Sorry for the delay Dan. Please let us know if you spot any problems.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/pombase/website/issues/729#issuecomment-399074674, or mute the thread https://github.com/notifications/unsubscribe-auth/APRzyU4qyeP62FOk2JfPCHT9C9QOFuahks5t-4ajgaJpZM4TNxeq.

In the ensembl browser.....different types of variant have different colour...

Sorry I didn't see that.  We can colour features in JBrowse but I don't know of a way to add a legend so it might not be useful.

Kim do you think this is possible?

I'm a bit confused.  We have "Type   insertion" in the details for the variation features.  Do we need more?
Perhaps we can chat about this at tomorrow's meeting?

Antonia can you open a new ticket for what is outstanding, and close this one? I'm a bit confused too.

Val said:

In the ensembl browser.....different types of variant have different colour...

Kim replied:

Sorry I didn't see that. We can colour features in JBrowse but I don't know of a way to add a legend so it might not be useful.

then I linked to this ticket with some screenshots:
GMOD/jbrowse#937

If we never announced the variation data availability in JBrowse, we should do that, maybe after this tweak

@Antonialock  what is to do here to announce this data?

Colour variants by type if possible
it seems like it will be a pain. Dataset was already announced, this was just a tweak.

ok if there is another variation dataset, can maybe implement then...


Enable running in Python mode

We should allow the user to run scripts in Python mode (perhaps as a driver_mode argument to start_single_node_cluster). In which case the code is equivalent to serial Python. It may be possible to do this by replacing each remote function call with just the regular (blocking) function call and replacing put/get with no-ops.

Addressed by #169.


Step may not be needed

Lab Name: Create a star schema model
Task: Set up Power BI
Step: 11
Issue Type: non-blocking
Description of issue: Step 11 may not be needed, I didn't have to log back in after step 10.

Updated :)


Deprecate the import for /lib?

The tree shaking feature is on his way with webpack 2 (http://www.2ality.com/2015/12/webpack-tree-shaking.html) and will probably soon be implemented by browserify.
I think that, at some point, we will have to deprecate the usage of import:
import ... from 'material-ui/lib/...';

And instead, advice people to use:
import {...} from 'material-ui';

@oliviertassinari How smart and reliable can it really be? How about nested imports? things like svg-icons? how can we address that? I mean we cant put all the icons in the main index file, right? can it actually shake those branches deep down the import tree? I couldn't find info whether it can or not. But there are other issues. Maybe people don't want to enable it for some reasons. I think it's best to support both rather than enforce one over the other.

@oliviertassinari @alitaheri I agree with both of you :smile: .
One of my first challenges with material-ui was knowing where to import components (especially icon components) and I often found myself looking at the source code to find what my import path needed to look like.  Put them all on the root material-ui makes it a lot easier for the user because you only need to know what the component name is. That makes looking into this webpack feature worth while in my opinion.
That being said, being dependent on the reliability of webpack and or browserify is risky and it might be worth while to continue supporting the "full" path for a while.
I'll throw another idea out there that may sound crazy. Material-ui is a big project, and maybe the components deserve their own packages. One possible approach is making material-ui a monorepo. We could put each component in their own directory (like we were planning on doing anyway) and give it its own package.json file with what other dependencies it has (like other material-ui components). That way each component instead of being in a deeply nested path will be flatter with it's own top level name. Like:
import TextField from 'material-ui-text-field'

/* In addition a top level 'material-ui' package could have all components as a dependency and re-export */
import {TextField} from 'material-ui'

Haven't thought through it much. It might be a really bad idea but think its worth mentioning since the React project even follows the monorepo approach.

@newoga Interesting idea. However, the dependency management will be really hard. unlike a library like lodash, this one is very tightly coupled. we'll need to make a huge number of packages (some internal) to include all. besides the svg-icons will prove difficult, they should all be in one package and yet support static imports so only the ones needed get through.
Supporting both styles:
import TextField from 'material-ui/TextField';

and
import {TextField} from 'material-ui';

is easy. ubt svg-icons will be harder to tackle. but having separate packages for each of them. will be really hard >.> npm will find us and kill up all :D :D

npm will find us and kill us all :D :D

@alitaheri Haha, indeed, they might. I don't want to die. :laughing:

Me neither xD xD

Also related to #1793.

@alitaheri @newoga this is on the 0.15.0 milestone, where are we going with this right now?

We still haven't discussed and decided on the best approach to implementing this. I think we've settled on putting all the components in directories (along with its internal components and tests, possibly docs).
I think we've also settled on the following path: material-ui/Component
To do this, we're going to have to change our build process to publish the package from within the conceptual lib folder rather than at the root level. To support backwards compatibility, for this version we will likely have to add a lib folder to src to maintain all the old paths and put deprecation warnings in them.
I'm open to better ideas but that's my current thought!

To do this, we're going to have to change our build process to publish the package from within the conceptual lib folder rather than at the root level.

It's a shame npm doesn't (appear to) support a way to put the published code in a subdirectory while still referencing it as if it is in the root. This could have been put to good use:

directories.lib
Tell people where the bulk of your library is. Nothing special is done with the lib folder in any way, but it's useful meta info.

To support backwards compatibility, for this version we will likely have to add a lib folder to src

Clever. How will backwards compatibility work for people importing from src? Or is that an inevitable breaking change?

It's a shame npm doesn't (appear to) support a way to put the published code in a subdirectory while still referencing it as if it is in the root. This could have been put to good use:

Yeah it'd be nice if it was supported :disappointed:. I've seen some projects that have a release script that simply copies the package.json to the lib folder then publishes from there. Other projects create  a commit and tag where the build is stored on the root with everything else removed and the pubish is still ran from the root. I'm open to either approach.

people importing from src? Or is that an inevitable breaking change?

Yeah, that's kind of what I had in mind. I consider importing from src the same as using internal API and not something we have to enforce backwards compatibility for. I'm hoping we can just use the lib in src trick for the 0.15.0 release, then when we're working on 0.16.0, we can just delete the lib folder.

Yeah it'd be nice if it was supported

Open a PR on npm? :smile:

Open a PR on npm? :smile:

Haha, possibly. Though, I've always got the impression that they try to keep their toolset really simple and allow more sophisticated tooling be developed in user space. Though as far as tooling goes, I think the directories.lib key you mentioned seems like a good use case for this.

I'll open an issue and see what they say.

I'll open an issue and see what they say.

Sounds good. :smile:

Just another note on this, I know we're already really busy/slammed but I think it would be nice/good form to create codemod to help users migrate their projects to the new paths.

this is on the 0.15.0 milestone

Yes indeed, the sooner we do it, the best it will be for everybody.

I think it would be nice/good form to create a codemod to help users migrate their projects to the new paths.

I completely agree and I volunteer to work on this codemod once we have moved the files.

I'll open an issue and see what they say.

I started to write the issue for this the other day, and realised this in't so much an issue for npm as for webpack, browserify, node etc to interpret directories.lib to be an alias for the package root dir. For np it's just a standards and docs issue.
I started poking through the code of the various tools to see if there's a shared lib used, but didn't have much luck finding anything.
So whilst It would still be nice to have - even if all parties agreed, it won't come soon enough for 0.15.0, and in the unlikely event it did, would force a dependancy on the latest versions of said tools.

@callemall/material-ui
Did we figure out where we are publishing from to make this happen?
Last I recall, we discussed publishing from the build folder.

@nathanmarks Yeah, let's get this sorted.
I don't think that that name of the folder matters a lot. I would rather focus on the process.
We need a script to copy the components from the src folder to the one we will use to publish.
We also need to filter the unit test components and the examples.

@oliviertassinari I already filter out the tests during build (so they don't end up in the lib folder). For the sake of simplicity then, why don't we just have the build process create a package.json and publish from the lib folder?


Would love example of CRUD

I'm sorry if an issue isn't the right place for this! I would personally implement and make a PR for this, but I don't know how — hence the request.
Anyway, I was hoping to see an example of a resource implementation. In the README for Vapor it mentions a "user" resource with a controller for it, but I would love to see how that actually works.
Thanks!

A CRUD example would be great. I will work on this during my next update to the example repo.
Or if you want to contribute, here's a guide for making a PR http://makeapullrequest.com


Add some internal functions for more extensible note creation

My use case is to be able to open the current days Journal entry or create it if it doesn't exist.
All of that is too specific, so I have that in my own functions outside of this plugin.
However, I have added the ability to create notes with titles and bodies instead of both of them being blank. Additionally, I added a function to look for a specific fullname/path to get its ID (primarily so I could script things without using telescope).
I've never written any Lua before, so I wouldn't be shocked if I did something dumb!

Nice ! I will look through the commits later and merge them in if there are no issue :)

I accidentally screw up the merge yesterday :/ so your branch might be off

All good. Thanks for looking that over.


Make article extractor with with dn.se

Problem with dn is that it's rss urls goes as following "dn.se/rss/om/fotboll" which doesn't work with our attempt at putting /rss, /feed, /feeds after a certain url. Make an exception in source.py in function set_feeds.

I could be wrong but it seems dn.se/rss/om/fotboll is just a subset of the items in dn.se/rss. If this is correct do you need the subset or can you work with the bigger set?

I could be wrong but it seems dn.se/rss/om/fotboll is just a subset of the items in dn.se/rss. If this is correct do you need the subset or can you work with the bigger set?

yes you are correct!


Some doubt about resumable.js

I found that to work with Resumable.js, the server side should:
First, save all trunks to files like file.trunk-1, 'file.trunk-2'...;
After all trunks received, the server side combines all trunk file to a single file to finish upload.
I am extremely worried that this combination will consume a plenty of time which will slow down the file-upload progress.
Any idea to solve this problem?

thanks @steffentchr , this help me a lot
"Finally, a number of reverse proxies (nginx, pound from my own experience) do not offer steaming uploads and instead buffers the entire request before forwarding it (again, a good motivation for node). In those cases, you'll have full data immediately upon request -- and the big file + seek approach is probably good."


SplitButton should have HotKey property like Button and DropDownButton

Is your feature request related to a problem? Please describe.
Currently SplitButton is the only Button that does not support the HotKey property, and setting the HotKeyManager.HotKey attached property does not work when the view is loaded and unloaded (Button handles this by resetting the property in Button.OnAttachedToLogicalTree)
Describe the solution you'd like
Copy over the HotKeyProperty and OnAttachedToLogicalTree handling from Button to SplitButton, linking the HotKey to the main SplitButton.Command

Sounds like an oversight to me. A PR is welcome. Use AddOwner when adding the StyledProperty

Yea, I missed this with the initial implementation. I'll check WinUI as well but this shouldn't be an issue to add. I'll add it to the list.


FileNotFoundError on 2_download_data_train.py

Tried following technical deployment to test the model, on step covering download training data, I ran into an error:
Traceback (most recent call last):
File "2_download_data_train.py", line 72, in
with open(fname_box, 'w') as f_box,
FileNotFoundError: [Errno 2] No such file or directory: 'technical_deployment\data_management\..\train_model\data\grocery\negative\d39fd583-8e4b-4c59-b12e-c4f00acd8584.bboxes.tsv'
After looking at 2_download_data_train.py, I think the call to file open is not correct.

Can you provide more details on the error message? The file "2_download_data_train.py" is used to download annotations from DocDB and images from Azure Blob, not from a local path.

I was using Windows DLVM on Azure to download training data and train the model. And as the error message shown, the traceback happened at line 72 of the 2_download_data_train.py. That line was to open files and  write to them the rois.

Did you follow the instructions to create DocDB account and Azure Storage account first? After you've done that, you can update config.py and run 1_upload_historical_data.py to upload data. Only after these steps can 2_download_data_train.py be run.

Yes, I did follow the instructions. DocDB and storage account are all
created. That is why in the error message the not-found filename has an
image id in the name.
I tried changing the call to open(fname_box,'w+') (create file if not
exist), but still got the same error.
On Sat, Mar 10, 2018 at 10:28 PM, Lixun<EMAIL_ADDRESS>wrote:

Did you follow the instructions to create DocDB account and Azure Storage
account first? After you've done that, you can update config.py and run
1_upload_historical_data.py to upload data. Only after these steps can
2_download_data_train.py be run.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure/cortana-intelligence-product-detection-from-images/issues/4#issuecomment-372086522,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgHfBH37KG4AMFV6HuOUr_GcxQcU6trks5tdJn6gaJpZM4Sk3C1
.

Hi, Lixin,
With some trial and errors, I figured it out why the error happens. The error is caused by path notation difference between windows (ex. C:\root\subdir) vs python way of path notation (ex. C:/root/subdir). I made some changes in config.py under data-management folder to make python happy about path to various folders and the errors went away.
I will make it do for now, and will come back and make a more general fix and will generate a PR.
For now I will close this issue.
Thanks

Good to know you found the fix. Yes, the paths in the config.py file need to be using "" instead of "/".


The code doesn't work

In the code of requests.post().
I don't why you code can only print('Hello there,  !')  even change Chinese into English.
data = {'firstname': '莫烦', 'lastname': '周'}
r = requests.post('http://pythonscraping.com/files/processing.php', data=data)
print(r.text)
# Hello there, 莫烦 周!

Best Wish!

try this url
http://pythonscraping.com/pages/files/processing.php
the code on github is not correct


Productize current work toward OpenAPITools enhancement for Helidon SE WebClient generation

Environment Details

Helidon Version: 2.x, 3.x
Helidon SE or Helidon MP SE
JDK version:
OS:
Docker version (if applicable):

Problem Description
Tim has already done some work for adding Helidon SE web client generation to the OpenAPITools generator.
This issue is for the work to finish that effort and merge it into the tool.

This will be covered by our Helidon specific generator.


ci(bindings/java): Add features support

We should verify build per PR which will be delivered in the release. Otherwise we should not release them or we "postpone" all the potential issues just before the release, which is not a good idea.

Service related staff should be covered by opendal rust core. So the real problem is we don't build opendal rust on all platforms. How about adding jobs to cover them instead so that we can aovid repeat those job at binding side?

TODO: move redis related tests to a separate job

I add this since I prefer to remove testcontainer in java binding. It's not scalable for either bindings nor services. I'm seeking ways to integrate existing behavior tests with binding features so binding developers don't need to test services one by one.

Service related staff should be covered by opendal rust core. So the real problem is we don't build opendal rust on all platforms. How about adding jobs to cover them instead so that we can aovid repeat those job at binding side?

Make sense.

It's not scalable for neither bindings nor services.

I don't think we should "keep consistency" among bindings, each binding can have its own test suite for coverage. If there is a better solution, we can migrate. If if we don't find a possible way yet, let's keep it as is.

If there is a better solution, we can migrate. But if we don't find a possible way yet, let's keep it as is.

LGTM.

The TODO for redis has been removed. And I'm working on to make sure opendal rust covered on all platform.

cc @tisonkun, would you like to take a look again, thanks!


AudioLink: As Local regressed in 1.8

Confirmed in 1.8.4 and 1.8.3. Shader responds to environment sound even if As Local is checked.

Probably related: https://github.com/lilxyzw/lilToon/issues/233


Info: Invalid argument(s): Failed to lookup symbol 'FPDF_InitLibraryWithConfig': undefined symbol: FPDF_InitLibraryWithConfig

Info: Invalid argument(s): Failed to lookup symbol 'FPDF_InitLibraryWithConfig': undefined symbol: FPDF_InitLibraryWithConfig
when use PdfViewer.asset

Please explain your environment; flutter doctor -v and your device.
And, if you use any PDF related library with pdfrx, please list them.

[✓] Flutter (Channel stable, 3.22.2, on Mac OS X 10.15.7 19H2026 darwin-x64, locale en-ID)
• Flutter version 3.22.2 on channel stable at /Users/mmuttaqin/Documents/DEVELOPMENT/FLUTTER/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 761747bfc5 (4 weeks ago), 2024-06-05 22:15:13 +0200
• Engine revision edd8546116
• Dart version 3.4.3
• DevTools version 2.34.3
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /Users/mmuttaqin/Library/Android/sdk
• Platform android-34, build-tools 34.0.0
• ANDROID_HOME = /Users/mmuttaqin/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
• All Android licenses accepted.
[!] Xcode - develop for iOS and macOS
✗ Xcode installation is incomplete; a full installation is necessary for iOS and macOS development.
Download at: https://developer.apple.com/xcode/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
sudo xcodebuild -runFirstLaunch
! CocoaPods 1.10.2 out of date (1.13.0 is recommended).
CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without CocoaPods, plugins will not work on iOS or macOS.
For more info, see https://flutter.dev/platform-plugins
To upgrade see https://guides.cocoapods.org/using/getting-started.html#updating-cocoapods for instructions.
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2023.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
[✓] VS Code (version 1.90.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.90.0
[✓] Connected device (4 available)
• Nexus 10 (mobile)         • R32D801FR5X      • android-arm    • Android 5.1.1 (API 22)
• Redmi Note 8 Pro (mobile) • eqivrstocyzdkrpf • android-arm64  • Android 11 (API 30)
• macOS (desktop)           • macos            • darwin-x64     • Mac OS X 10.15.7 19H2026 darwin-x64
• Chrome (web)              • chrome           • web-javascript • Google Chrome 126.0.6478.127
[✓] Network resources
• All expected network resources are available.
not working at old android Nexus 10 (mobile) API 22

Exception: Invalid argument(s): Failed to lookup symbol 'FPDF_InitLibraryWithConfig': undefined symbol: FPDF_InitLibraryWithConfig
I/flutter (16691): Stack trace: #0      DynamicLibrary.lookup (dart:ffi-patch/ffi_dynamic_library_patch.dart:33:70)
I/flutter (16691): #1      pdfium._FPDF_InitLibraryWithConfigPtr (package:pdfrx/src/pdfium/pdfium_bindings.dart:33:80)
I/flutter (16691): #2      pdfium._FPDF_InitLibraryWithConfigPtr (package:pdfrx/src/pdfium/pdfium_bindings.dart)
I/flutter (16691): #3      pdfium._FPDF_InitLibraryWithConfig (package:pdfrx/src/pdfium/pdfium_bindings.dart:35:44)
I/flutter (16691): #4      pdfium._FPDF_InitLibraryWithConfig (package:pdfrx/src/pdfium/pdfium_bindings.dart)
I/flutter (16691): #5      pdfium.FPDF_InitLibraryWithConfig (package:pdfrx/src/pdfium/pdfium_bindings.dart:26:12)
I/flutter (16691): #6      _init. (package:pdfrx/src/pdfium/pdfrx_pdfium.dart:46:14)
I/flutter (16691): #7      using (package:ffi/src/arena.dart:124:31)
I/flutter (16691): #8      _init (package:pdfrx/src/pdfium/pdfrx_pdfium.dart:38:3)
I/flutter (16691): #9      PdfDocumentFactoryImpl.openCustom (package:pdfrx/src/pdfium/pdfrx_pdfium.dart:150:5)
I/flutter (16691): #10     PdfDocumentFactoryImpl._openData (package:pdfrx/src/pdfium/pdfrx_pdfium.dart:119:12)
I/flutter (16691): #11     PdfDocumentFactoryImpl.openAsset (package:pdfrx/src/pdfium/pdfrx_pdfium.dart:62:18)
I/flutter (16691):
I/flutter (16691): #12     PdfDocumentListenable.load. (package:pdfrx/src/pdf_document_ref.dart:401:22)
I/flutter (16691):
I/flutter (16691): #13     objectSynchronized. (package:synchronized/src/extension_impl.dart:37:18)
I/flutter (16691):
I/flutter (16691): #14     BasicLock.synchronized (package:synchronized/src/basic_lock.dart:33:16)
I/flutter (16691):
I/flutter (16691): #15     objectSynchronized (package:synchronized/src/extension_impl.dart:34:12)
I/flutter (16691):
I/flutter (16691): #16     PdfDocumentListenable.load (package:pdfrx/src/pdf_document_ref.dart:395:12)
I/flutter (16691):

Thanks for the info.
Recentlty I updated libpdfium.so on 1.0.72 and it "might" introduce some incompatibility with older SDK version (I didn't confirm that).
Could you test the following versions one by one?

1.0.60 chromium/6406
1.0.66 chromium/6517
1.0.72 chromium/6555

FYI: I've confirmed the existances of FPDF_InitLibraryWithConfig on the latest libpdfium.so binaries used by pdfrx:
kawasaki@red:/mnt/d/pdfrx/android/.lib/chromium%2F6555$ nm -D x86/libpdfium.so | grep FPDF_InitLibraryWithConfig
00318b79 T FPDF_InitLibraryWithConfig
kawasaki@red:/mnt/d/pdfrx/android/.lib/chromium%2F6555$ nm -D x86_64/libpdfium.so | grep FPDF_InitLibraryWithConfig
00000000003157f1 T FPDF_InitLibraryWithConfig
kawasaki@red:/mnt/d/pdfrx/android/.lib/chromium%2F6555$ nm -D arm64-v8a/libpdfium.so | grep FPDF_InitLibraryWithConfig
0000000000333464 T FPDF_InitLibraryWithConfig
kawasaki@red:/mnt/d/pdfrx/android/.lib/chromium%2F6555$ nm -D armeabi-v7a/libpdfium.so | grep FPDF_InitLibraryWithConfig
0024285d T FPDF_InitLibraryWithConfig

Every so files for Android correctly contains FPDF_InitLibraryWithConfig.
I don't have any Android devices of API 22. So it's very difficult to check the actual loading behavior :(

@mmuttaqin Android 22 is too old to test on my side. It's up to you (or someone else who also need Android 22 support) to test the things on Android 22 or I can only to close the issue without fixing it.


Quality of test files

Lets get good automated tests by always having a assert before done()
Would you be able to extend test360.js and test360.js to assert the result?

Yes, sorry, I have not finished it yet.
Unfortunately, there couple of tests more without asserts.


config file need extra parameter

hi, because of testing the code, we need e a config file with 'init_prior' parameter,thanks!

hi, because of testing the code, we need e a config file with 'init_prior' parameter,thanks!

We add the missing 'init_prior' into config in the latest commit.


Fix typos

Fix typos

Thank you very much! As mostly non-native speakers we do make a lot of spelling mistakes.

I perfectly understand because mee too I'm non-native (Italian). Have a nice day ;-)


ui: Add confirm dialog components

Error: 'UnmodifiableUint8ListView' is restricted and can't be extended or implemented.


Odd discover failures with wrong post id for Reddit synd link

https://brid.gy/reddit/grich12 ran discover on a bunch of their original post URLs last night. Many found Reddit syndication links and enqueued background discover tasks for those, and many of those failed oddly. The background tasks got post_id=status, where it should have been the Reddit comment id, eg post_id=y0g1p3. They then duly tried to fetch https://oauth.reddit.com/comments/status/ from the Reddit API, which 403ed.
Looks like this is because we enqueue background discover tasks for all synd links we find, not just ones that match the current source's silo:
https://github.com/snarfed/bridgy/blob/24ec10ca2e34d42dc7aba03b4da34d7d25b4b7b5/pages.py#L495-L496
Example full /discover log:
Params: [('url', 'https://geoffrich.net/posts/4-things-i-miss-svelte/'), ('source_key', 'agdicmlkLWd5chMLEgZSZWRkaXQiB2dyaWNoMTIM')]
Got source: Reddit(key=Key('Reddit', 'grich12'), ...
fetching post permalink https://geoffrich.net/posts/4-things-i-miss-svelte/
requests.get https://geoffrich.net/posts/4-things-i-miss-svelte/ {'gateway': False, 'headers': {}}
Received 200
rel-syndication links: ['https://twitter.com/geoffrich_/status/1579478166384816131', 'https://www.reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in/']
requests.head https://reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in {'headers': None}
Redirected to https://www.reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in
Received 200
Resolved https://reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in to https://www.reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in
saving discovered relationship https://reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in -> https://geoffrich.net/posts/4-things-i-miss-svelte/
requests.head https://twitter.com/geoffrich_/status/1579478166384816131 {'headers': None}
Received 200
saving discovered relationship https://twitter.com/geoffrich_/status/1579478166384816131 -> https://geoffrich.net/posts/4-things-i-miss-svelte/
discovered relationships {'https://reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in': [SyndicatedPost(key=Key('Reddit', 'grich12', 'SyndicatedPost',<PHONE_NUMBER>371072), created=datetime.datetime(2023, 2, 8, 4, 12, 16, 749234, tzinfo=datetime.timezone.utc), original='https://geoffrich.net/posts/4-things-i-miss-svelte/', syndication='https://reddit.com/r/sveltejs/comments/y0g1p3/4_things_i_miss_from_svelte_after_working_in', updated=datetime.datetime(2023, 2, 8, 4, 12, 16, 749240, tzinfo=datetime.timezone.utc))], 'https://twitter.com/geoffrich_/status/1579478166384816131': [SyndicatedPost(key=Key('Reddit', 'grich12', 'SyndicatedPost',<PHONE_NUMBER>793280), created=datetime.datetime(2023, 2, 8, 4, 12, 17, 186017, tzinfo=datetime.timezone.utc), original='https://geoffrich.net/posts/4-things-i-miss-svelte/', syndication='https://twitter.com/geoffrich_/status/1579478166384816131', updated=datetime.datetime(2023, 2, 8, 4, 12, 17, 186024, tzinfo=datetime.timezone.utc))]}
Added discover task projects/brid-gy/locations/us-central1/queues/discover/tasks/6800654323078936852 with ETA None
Added discover task projects/brid-gy/locations/us-central1/queues/discover/tasks/28892214793288639731 with ETA None
Updating grich12 (Reddit) /reddit/grich12 : {'last_syndication_url': datetime.datetime(2023, 2, 8, 4, 12, 17, 808748, tzinfo=datetime.timezone.utc)}
Flashed message: Discovering now. Refresh in a minute to see the results!
Redirecting to https://brid.gy/reddit/grich12

And the corresponding /_ah/queue/discover log:
Params: [('source_key', 'agdicmlkLWd5chMLEgZSZWRkaXQiB2dyaWNoMTIM'), ('post_id', 'status')]
Got source: Reddit(key=Key('Reddit', 'grich12'), ...
Source: grich12 (Reddit) grich12, https://brid.gy/reddit/grich12
Fetching: GET https://oauth.reddit.com/comments/status/
Data: None
Params: {'limit': 2048, 'sort': 'confidence', 'raw_json': 1}
Response: 403 (38 bytes)
Error 403, response body: '{"message": "Forbidden", "error": 403}'

Also tracked in https://console.cloud.google.com/errors/detail/CNu4z_eI17Kd9QE;time=P30D?project=brid-gy

Root cause here is evidently that the Reddit Bridgy class doesn't override URL_CANONICALIZER, which we use to determine whether a syndication link is in a given source:
https://github.com/snarfed/bridgy/blob/main/original_post_discovery.py#L578-L581


libocr.so not downloading

We're using Mobile Vision'OCR reader in our app and we've came across a problem: in some cases, the libocr.so library is not downloaded or downloaded after a couple of seconds to a couple of hours. For example, if we try installing our application on a Huawei Nexus 6P, the library is downloaded after 2 hours.
The logs that we get:

I/Vision: Loading library libocr.so
I/Vision: libocr.so library load status: false
I/Vision: Request download for engine ocr
W/TextNativeHandle: Native handle not yet available. Reverting to no-op handle.
W/DynamiteModule: Local module descriptor class for com.google.android.gms.vision.dynamite not found.
I/DynamiteModule: Considering local module com.google.android.gms.vision.dynamite:0 and remote module com.google.android.gms.vision.dynamite:1402
I/DynamiteModule: Selected remote version of com.google.android.gms.vision.dynamite, version >= 1402

We've tried adding the meta-data tags in the AndroidManifest.xml:

<meta-data android:name="com.google.android.gms.version"
android:value="@ integer/google_play_services_version"/>
<meta-data
android:name="com.google.android.gms.vision.DEPENDENCIES"
android:value="ocr" />

and adding the dependencies in the module-level build.gradle:

implementation 'com.android.support:support-v4:27.0.2'
implementation 'com.android.support:design:27.0.2'

but there was not success with these. We've also tried clearing the application's data and the Google Play Service app's data.
After researching for hours, we've came across someone's response that was saying to disable the camera permission from the Google Play Services app before trying to install the application. This is the only solution that we've found, but not a solution we can use, as we can't ask the user to do that so the OCR feature would work.
Does anyone know a better solution to handle this?

Same issue here, we were planning to use the vision library in an offline environment and this is a big problem for us. Did you find a solution to this? Thanks.


Revert "Improve the ScrollBar behavior when nested"

Reverts flutter/flutter#71843
Dartdoc error:
dartdoc:stderr:   error: unresolved doc reference [notificationPredicate], from material.Scrollbar: (file:///b/s/w/ir/k/flutter%20sdk/packages/flutter/lib/src/material/scrollbar.dart:48:7)

dartdoc:stderr: found 1 warning and 1 error
dartdoc:stdout: Documented 163 public libraries in 263.2 seconds
dartdoc:stderr:
dartdoc:stderr: dartdoc failed: dartdoc encountered 1 errors while processing..
dartdoc:stderr: #0      Dartdoc.generateDocs (package:dartdoc/dartdoc.dart:231:9)
dartdoc:stderr: <asynchronous suspension>
dartdoc:stderr: #1      Dartdoc.executeGuarded.<anonymous closure> (package:dartdoc/dartdoc.dart:500:9)
dartdoc:stderr: <asynchronous suspension>

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8857995126157483168/+/steps/prepare_environment/0/steps/Docs/0/steps/docs/0/stdout

Merging to see if this was the cause of that failure.

Merging to see if this was the cause of that failure.

The specific error is:
dartdoc:stderr:   error: unresolved doc reference [notificationPredicate], from material.Scrollbar: (file:///b/s/w/ir/k/flutter%20sdk/packages/flutter/lib/src/material/scrollbar.dart:48:7)

The specific error is:
dartdoc:stderr:   error: unresolved doc reference [notificationPredicate], from material.Scrollbar: (file:///b/s/w/ir/k/flutter%20sdk/packages/flutter/lib/src/material/scrollbar.dart:48:7)

LGTM

LGTM

if this was the cause of that failure.
scrollbar.dart:48:7

Sure seems it :-)

if this was the cause of that failure.
scrollbar.dart:48:7

Sure seems it :-)

The specific error is:
dartdoc:stderr:   error: unresolved doc reference [notificationPredicate], from material.Scrollbar: (file:///b/s/w/ir/k/flutter%20sdk/packages/flutter/lib/src/material/scrollbar.dart:48:7)

Thanks, I missed it in the middle.

The specific error is:
dartdoc:stderr:   error: unresolved doc reference [notificationPredicate], from material.Scrollbar: (file:///b/s/w/ir/k/flutter%20sdk/packages/flutter/lib/src/material/scrollbar.dart:48:7)

Thanks, I missed it in the middle.

How strange, this should have failed in presubmit. Thanks!

How strange, this should have failed in presubmit. Thanks!

How strange, this should have failed in presubmit. Thanks!

This is most likely because I did not synchronize this change #73899

How strange, this should have failed in presubmit. Thanks!

This is most likely because I did not synchronize this change #73899


Adding documentation for Hamilton, ON, Canada

Hamilton, ON, Canada uses RecycleCoach.com which is already available in the repo.
This PR only adds documentation to both README and recyclecoach_com.py to make it easier for users to know the Canadian city is supported by hacs_waste_collection_schedule.

Did not see this as this is not the main repository please open this PR in the main repository (https://github.com/mampfes/hacs_waste_collection_schedule) and not in my personal fork


Add support for RFC 6486

Add module and test for RFC6486

Codecov Report

Merging #90 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
+ Coverage   99.34%   99.34%   +<.01%
==========================================
  Files          79       80       +1
  Lines        5642     5659      +17
==========================================
+ Hits         5605     5622      +17
  Misses         37       37

Impacted Files
Coverage Δ

pyasn1_modules/rfc6486.py
100% <100%> (ø)

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 a080f0b...d2c7ab9. Read the comment docs.


Project > Language Engagement > Product > Getting "Product selection required" error even product type is selected

Roles: All
Describe the bug
Project > Language Engagement > Product > Getting Product selection required error even product type is selected
Steps to Reproduce

Login as a Project Manager
Create a new Project and go to the project details page
Create a language engagement
Click on the View details button
Click on an Add product button
Select Scripture product

Expected Result
The error message should not be displayed while the product type is selected
Actual Result
Getting Product selection required error even product type is selected
See video -
https://user-images.githubusercontent.com/5212585/118490868-0da88200-b73c-11eb-9592-d6b765be6d00.mp4
FYI: @ParkeBrown @michaelmarshall
cord-field# 1d6e8100

Project Manager gets "Product selection required" error despite the product type being selected already.

@sethmcknight we should probably evaluate priority for this when you get back

Duplicate of #907


Wisemonk should not accept negative values for meditation

time.ParseDuration accepts negative numbers, so it'd be better to validate user input here: https://github.com/dgraph-io/wisemonk/blob/master/main.go#L363
Otherwise it could introduce a potential DOS vector by disallowing wisemonk to meditate :-)

Thanks for reporting this @ghoseb. We will push in a fix soon.

Fixed!


Input position and image into DQN policy

Hi, I am currently trying to create an AI for a drone for my observation space I would like to add one box space for the image, and one for position and velocity. Is there any way I would be able to do this or is there a way I can combine them into one box space?

Duplicate of  #133
You can use this trick do feed the 1D data as part of the image. Alternatively stable-baselines3 has almost-done PR that will add support for dictionary observations, which you could try out using.
Closing as duplicate of #133.

@Miffyli I installed the forked version of the stable baselines as you mentioned, but I still received an error:

It appears that the VecTransposeImage only works with environments that have single box spaces. I have two box inputs but one is not an image and I'm not sure needs to be transposed.
Is there any solution for this?
Should I try to change the class to only take in single observations, or will that be added in the full release?
My knowledge of this is limited sorry if I am asking anything stupid

@CoolCoder54323
The PR is still work-in-progress so issues like this still persist. In the current form you need to change the other "box" observation to be either 1D or a 3D array, where the latter needs to have either 1, 3 or 4 channels (this is the current limitation in the PR). I can not guarantee when this would be fixed, but I try to contribute to it the next week.
If that is not possible, you can try the trick I linked above where you combine the two images into one, and then separate them inside the tensorflow code.

Ok I will try the trick thank, you for your help.

@Miffyli
I have a np.array for my depth image with the dimensions [88,88,1] do you know how I would add the 1d data to this?

@Miffyli I installed the forked version of the stable baselines as you mentioned, but I still received an error:

It appears that the VecTransposeImage only works with environments that have single box spaces. I have two box inputs but one is not an image and I'm not sure needs to be transposed.
Is there any solution for this?
Should I try to change the class to only take in single observations, or will that be added in the full release?
also if i do that will it hinder the training of my agent in any way. (I don't really know why it needs to be transposed in the first place)
My knowledge of this is limited sorry if I am asking anything stupid

so the real issue here is that you need to have channel first images (we are planning to support channel last too, hence the WIP)
you have a full example here: https://github.com/DLR-RM/stable-baselines3/pull/243/files#diff-56297865c9e2f36cfbb1b907fe7dea14b65054a680bf2916509b73c1119c54f0

@araffin
I made the image in my dictionary a channel first image, and it gave the same error I believe the function VecTransposeImage only works with single box image environments. Do you think I could just ignore this function or is it critical for the policy, I was told it was needed to allow stable baselines to make frame observations(I don't exactly know what that means). The other thing in my dict is a 2x3 box space representing position and velocity fyi.

@araffin Sorry to ping you again but I am kind of in a hurry as I am doing this for a school project that is due soon. I would really appreciate an answer, again sorry for double pinging you.

@araffin Sorry to ping you again but I am kind of in a hurry as I am doing this for a school project that is due soon. I would really appreciate an answer, again sorry for double pinging you.

Please take a close look at the working example I linked.

@araffin Ok I took a closer look at what you linked me and was able to din f what I needed, for the most part, but I have one more problem. I cannot figure out how to return a box space as it requests

I am inputting two NumPy arrays of the appropriate size and within the right bounds and it is still giving this error even though this is what I inputted before when it was just one box space. I even tried inputting box.sample like this:

but that still didnt work.

Look at the error message: it should contain gym.Spaces objects but you are sampling observations into there. This kind of tech support is not what we offer here, so in future such questions will go unanswered.

@Miffyli where can I get some tech support for this, it seems not many people online know about this

@CoolCoder54323 Your best bet could be to ask on RL Discord, or alternatively on stackoverflow.

Ok thank you for your time.


DROOLS-733 - improved UserTaskServiceIntegrationTest.testSkipUserTask test

Test improvement showing possible bug.
taskSummary.getSkipable() returns false although task has defined Skippable data input which is set to true and is possible to skip task by calling client.skipTask(...).
Please try to run this test.

merged as part of other changes and fixed problem with skipable property not being set by https://github.com/droolsjbpm/jbpm/commit/7df638d98c1f86f6c30fdc266041027e6e5a47b1


Modify some classes to functions and struct in DEM

Description

Some classes should not has been classes but functions or struct.
I modified those classes to a more coherent architechture.
I also gathered all the particle-object contact info structures in one file because it is more convenient that having 3 files for small structs.
Testing

Unit tests have been change according to that.
Miscellaneous (will be removed when merged)

I do not think that needs a change log.
The particle-wall contact info needs to be changed as other, but I will do it when modifying the particle-wall forces.
Checklist (will be removed when merged)
See this page for more information about the pull request process.
Code related list:

[x] All in-code documentation related to this PR is up to date (Doxygen format)
[x] Lethe documentation is up to date
[x] The branch is rebased onto master
[x] Code is indented with indent-all and .prm files (examples and tests) with prm-indent
[x] If parameters are modified, the tests and the documentation of examples are up to date
[x] Changelog (CHANGELOG.md) is up to date if the refactor affects the user experience or the codebase

Pull request related list:

[x] No other PR is open related to this refactoring
[x] Labels are applied
[x] There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
[x] If this PR closes an issue or is related to a project, it is linked in the "Projects" or "Development" section
[x] If any future works is planed, an issue is opened
[x] The PR description is cleaned and ready for merge

Some tests seem to be crashing but it seems to be due to a deal.II update. I'll try to work on this tomorrow AM

@acdaigneault can you rebase, this way we will be able to see what is breaking because of previous bugs and not

@acdaigneault I'm not sure what is this hang error you are getting, but it seems something is off since it happens in both release and debug?

@acdaigneault I'm not sure what is this hang error you are getting, but it seems something is off since it happens in both release and debug?

Yes this is related to my PR


Rewrite in rust

Rust is an elegant, blazing fast, memory safe language. C++ is what old people use, if you want ACTUAL contributers you should switch to rust, its what everyone uses now.

Rust is an elegant, blazing fast, memory safe language. C++ is what old people use, if you want ACTUAL contributers you should switch to rust, its what everyone uses now.

https://github.com/korewaChino/libpolymc

LOL


Remove editor folder from asset store releases

it is only used to package the asset for github

Decided to leave this in


Rate Limit Exceeded Exceptions for OpenAI when is_async=True

The current implementation of the asynchronous version does not enforce rate limits on OpenAI API calls, potentially leading to exceeded rate limit exceptions. What would be the most appropriate method to effectively manage and resolve this issue?

I have the same question. I am running into this issue every time I try to generate synthetic data. Did anyone find solutions ?

hey @un-certainty and @norbert-eke-ef
we have fixed this in the latest version - you will have to use the code to verify how to use it right now is not documented and I'll add that soon.
in the mean time this is the code section https://github.com/explodinggradients/ragas/blob/e6be5bbe44e1f1908c79e0cc36fdb73dc18676fb/src/ragas/run_config.py#L15
I'll close this once we have added the docs :)

Is this now updated in version == 0.1.1?

yes it is fixed @bdeck8317

Experiencing the same, also with is_async=False. See https://github.com/explodinggradients/ragas/issues/624#issuecomment-2009820626

Could you @jjmachan please explain how to set parameters in RunConfig to  avoid rate limit exceptions? I do not understand from the definition of the class how timeout, max_retries, max_wait, max_workers (in version 0.1.4) will affect the rate of requests: I want to be sure that I will lower the rate enough to avoid rate limit exception.


Model location

Thanks for your amazing work！
Can you explain in detail how to place the corresponding model?

@JasonSongPeng Hi, you can check the code and manually load it using the resume option.


count time taken metrics for each unique message

Below metrics need to count at each message level. Currently, the dimensions are only topic and subscription due to which we are seeing skewed metrics on the dashboard.
metro_subscriber_time_from_publish_to_consume_msg_seconds
metro_subscriber_time_to_ack_msg_seconds
metro_subscriber_time_to_mod_ack_msg_seconds

Codecov Report

Merging #190 (30634b3) into master (050f0e7) will decrease coverage by 3.75%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
- Coverage   31.81%   28.05%   -3.76%
==========================================
  Files          92       92
  Lines        3860     3860
==========================================
- Hits         1228     1083     -145
- Misses       2523     2691     +168
+ Partials      109       86      -23

Flag
Coverage Δ

integration
?

unittests
28.05% <50.00%> (ø)

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

Impacted Files
Coverage Δ

internal/subscriber/subscriber.go
0.00% <0.00%> (ø)

internal/subscriber/metrics.go
100.00% <100.00%> (ø)

pkg/messagebroker/kafka.go
0.00% <0.00%> (-35.51%)
:arrow_down:

pkg/messagebroker/factory.go
33.33% <0.00%> (-33.34%)
:arrow_down:

pkg/messagebroker/validations.go
35.71% <0.00%> (-7.15%)
:arrow_down:

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 050f0e7...30634b3. Read the comment docs.


Azure Batch CLI command --os-family parameter should support the string name of the os family rather than magic strings

Is your feature request related to a problem? Please describe.
Azure Batch CLI command --os-family parameter only supports 'magic strings' identifying the os family
https://docs.microsoft.com/en-us/cli/azure/batch/pool?view=azure-cli-latest#az_batch_pool_create
$az batch pool create --os-family {2, 3, 4, 5, 6}
Describe the solution you'd like
Azure Batch CLI command --os-family parameter should support the 'magic number' and string name of the os family, or probably more easily, by adding a new parameter, such as '--os-family-name' to avoid breaking backwards compatibility.
Describe alternatives you've considered
none
Additional context
none

route to appropriate team

The osFamily refers to the --os-family number for cloud services as referenced https://docs.microsoft.com/en-us/azure/cloud-services/cloud-services-guestos-update-matrix. For a VMSS Image you should make use of --image


Fix problem when self-assign notification

Is it possible there are more instances like these?

Should we also ignore them for sending mail notification?

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@6767798). Click here to learn what that means.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #18797   +/-   ##
=======================================
  Coverage        ?   46.61%
=======================================
  Files           ?      853
  Lines           ?   122463
  Branches        ?        0
=======================================
  Hits            ?    57080
  Misses          ?    58501
  Partials        ?     6882

Impacted Files
Coverage Δ

modules/notification/ui/ui.go
60.71% <100.00%> (ø)

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 6767798...8394f7d. Read the comment docs.


Span#set_attribute with strings & symbols

Span#set_attribute currently accepts both symbols and strings for both keys and values.
We had a similar discussion here: https://github.com/opentracing/opentracing-ruby/issues/35 and went with only strings. Currently we can still decide what's the best approach.
The problem with allowing both strings and symbols as attribute keys has two main problems:

memory allocation in case we need to convert one to other
what happens if somebody does span[:foo] = "bar"; span["foo"] = "baz". who wins?

As the person who opened that issue, I'll add a few comments here. The OpenTracing Issue is worth reading. I really want to be able to use symbols in methods like set_attribute. They are the most natural and idiomatic code to write as a Rubyist. However, this flexibility comes at a pretty big penalty, both in terms of API complexity (for the Tracer) and performance, by way of object allocations. Ultimately symbol keys will have to be turned into strings at serialization time. The best way to avoid object allocations in this scenario is for attributes to be frozen strings. Because of this, I would be in favor of changing Span#set_attribute to accept strings and not symbols.

We discussed this at the SIG meeting and I wanted to summarize what we talked about.
With the flexibility of accepting strings or symbols for attribute keys / values comes some complications:

string allocations - Symbols will need to be converted to strings at serialization time
key clashes - what happens when both a string and symbol key is specified with an equivalent representation, i.e. span[:foo] = "bar"; span["foo"] = "baz"

A tracer could work around these issues by maintaining a symbol to frozen string cache to reduce string allocations and establishing a rule that keys with equivalent string-symbol representations will be treated the same ('foo' would be handled the same as :foo).
The most performant and simple solution would be for the API to only accept strings, and for users to use frozen strings. In order to take advantage of this, users will have to careful to freeze their strings, and handle symbol to string conversion themselves.
There is a tradeoff between the flexibility of using strings and symbols in terms of simplicity and performance.
@luvtechno suggested that we could start by only accepting strings and it would be possible to relax the API to accept symbols in the future, if we decided we wanted to go that direction.

Implemented suggestion (remove Symbol support) in #45


UI API called on a background thread

Describe the bug
Whenever I open the image picker I am faced with a screen that freezes for a few moments before the Image picker opens up. These are the logs that I get.
Main Thread Checker: UI API called on a background thread: -[UIView init]
PID: 6370, TID: 1500704, Thread name: (none), Queue name: com.apple.mobileslideshow.accessCallbacks, QoS: 0
Backtrace:
4   BSImagePicker                       0x0000000103fa6030 $sSo6UIViewCABycfcTO + 28
5   BSImagePicker                       0x0000000103fa4024 $sSo6UIViewCABycfC + 32
6   BSImagePicker                       0x0000000103fa5050 $s13BSImagePicker8SettingsCACycfc + 160
7   BSImagePicker                       0x0000000103fa4f3c $s13BSImagePicker8SettingsCACycfC + 52
8   BSImagePicker                       0x0000000103f75ca4 $s13BSImagePicker0aB14ViewControllerCACycfc + 76
9   BSImagePicker                       0x0000000103f75f8c $s13BSImagePicker0aB14ViewControllerCACycfcTo + 28
10  BSImagePicker                       0x0000000103f75be8 $s13BSImagePicker0aB14ViewControllerCACycfC + 44

To Reproduce
Steps to reproduce the behaviour:

Open BSImagePicker View Controller
Check the logs

Smartphone (please complete the following information):

Device: Iphone 7 Plus
OS: iOS 12.3

I also get the following message in my log, if it is helpful...
[Assert] Cannot be called with asCopy = NO on non-main thread.

Works for me. Could you show the code you use to setup/present the image picker?

If you get it at startup then perhaps you are presenting it from a background thread..?

I first check if the permission to access images has been granted,
let galleryAction = UIAlertAction(title: "photoLibrary".localize(), style: .default, handler: { (action) -> Void in
            PHPhotoLibrary.requestAuthorization({
                (newStatus) in
                if newStatus ==  PHAuthorizationStatus.authorized {
                    self.pickFromGallery()
                } else {
                    self.alertPromptToAllowAccessViaSetting()
                }
            })

        })

And this is how I open the photo gallery
func pickFromGallery() {
        let vc = BSImagePickerViewController()
        vc.maxNumberOfSelections = MAX_IMAGES_COUNT
        bs_presentImagePickerController(vc, animated: false,
                                        select: { (asset: PHAsset) -> Void in

        }, deselect: { (asset: PHAsset) -> Void in

        }, cancel: { (assets: [PHAsset]) -> Void in

        }, finish: { (assets: [PHAsset]) -> Void in
            // I consume assets here
        }, completion: nil)
    }

https://developer.apple.com/documentation/photokit/phphotolibrary/1620736-requestauthorization
Photos may call your handler block on an arbitrary serial queue. If your handler needs to interact with user interface elements, dispatch such work to the main queue.
i.e: You should dispatch to main in the requestAuthorization handler

Thanks for the help, I've solved the issue based on your point.


update old book references to the second edition of the book

There is a bunch of book references in the hints within exercises that lead to the outdated version of the book (pages have a message "There is a new edition of the book and this is an old link." and require a bunch of digging to find the appropriate section of the book). This change updated the outdated links with the relevant ones, where possible. In one instance (primitive_types3.rs) the content is no longer available in the book, so I provided a direct hint on how to initialize an array of a specified size with values.

also fixes #86

thanks so much!


Handles no-change domain transfer policy

no-change domain transfer policies (TLD specific) do not renew domains on transfer, but are still charged for the transfer. This PR removes the "with a 1 year renewal copy for no-change domain transfers

Codecov Report

Merging #2570 into canary will decrease coverage by 0.85%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           canary    #2570      +/-   ##
==========================================
- Coverage   14.55%   13.69%   -0.86%
==========================================
  Files         257      257
  Lines        9420     9463      +43
  Branches     1093     1087       -6
==========================================
- Hits         1371     1296      -75
- Misses       7924     8042     +118
  Partials      125      125

Impacted Files
Coverage Δ

src/util/domains/check-transfer.ts
0% <ø> (ø)
:arrow_up:

src/commands/domains/transfer-in.ts
0% <0%> (ø)
:arrow_up:

src/util/metrics.ts
68.18% <0%> (-11.13%)
:arrow_down:

src/util/dev/builder-cache.ts
77.27% <0%> (-5.59%)
:arrow_down:

src/index.js
0% <0%> (ø)
:arrow_up:

src/commands/update.ts
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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