Move "wiki" into repo

This PR moves the wiki into the repo. Wikis as implemented in GitHub do not allow PRs.
That means that after the change:

Contributors can make pull requests against these pages, e.g. which is the recommended way to link to external implementations.

We can make changes to the (currently) "wiki" file and the other files at the same time, which is needed to make a complete specification.

How I made this PR

Clone open-location-code and open-location-code.wiki
Move the wiki inside docs/ in this repo
Fix all links

I did NOT make other changes. This is a pure-play pull request. Existing typos, errors are all kept.
How to review and accept this PR

Review the files changed link above
Click some of the files, check some of the links
Merge PR
Disable the wiki feature for this repo in project setting (link at top of this page)

Follow on work

After this PR is accepted, I will contribute additional PRs to text that was previously in the "wiki".
I will invite the PHP implementation to add to our list of external implementations https://github.com/google/open-location-code/issues/427

Can you undo this please - I don't want to move the whole wiki into the repo without a discussion.

This change did not take effect yet. (It is already "undone".)
The place you are reading this is where we can discuss the topic.

As-is, so far, this current proposal is to simply move wiki contents.
This should be the simplest proposal, because after this step, we can then individually talk about other things that can be changed in the wiki and elsewhere.

I'm also able to implement/propose slightly more involved changes in this first pass. But then there's a risk that it slows everything down while negotiating it.

I'll close this as I have moved the wiki files into the repo in another pr.


IncompatibleClassChangeError: Couldn't find org.greenrobot.eventbus.Subscribe.threadMode

Fatal Exception: java.lang.IncompatibleClassChangeError: Couldn't find org.greenrobot.eventbus.Subscribe.threadMode
       at libcore.reflect.AnnotationAccess.toAnnotationInstance(AnnotationAccess.java:659)
       at libcore.reflect.AnnotationAccess.toAnnotationInstance(AnnotationAccess.java:641)
       at libcore.reflect.AnnotationAccess.getDeclaredAnnotation(AnnotationAccess.java:170)
       at java.lang.reflect.Method.getAnnotation(Method.java:301)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findUsingReflectionInSingleClass(SubscriberMethodFinder.java:165)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findUsingInfo(SubscriberMethodFinder.java:88)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findSubscriberMethods(SubscriberMethodFinder.java:64)
       at org.greenrobot.eventbus.EventBus.register(EventBus.java:136)
       at lt.lemonlabs.android.paysera.common.PayseraApplication.initializeEventBus(PayseraApplication.java:179)
       at lt.lemonlabs.android.paysera.common.PayseraApplication.onCreate(PayseraApplication.java:87)
       at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1020)
       at android.app.ActivityThread.handleBindApplication(ActivityThread.java:5007)
       at android.app.ActivityThread.access$1600(ActivityThread.java:172)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1483)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:145)
       at android.app.ActivityThread.main(ActivityThread.java:5832)
       at java.lang.reflect.Method.invoke(Method.java)
       at java.lang.reflect.Method.invoke(Method.java:372)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1399)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1194)

EventBus 3.0.0
Android 5.0 - 5.0.2
Hello, I get this crash randomly for some users. Is there any solution for this one or more information?

I assume this may happen if your code was not properly recompiled against the new EventBus. See this Stackoverflow post about IncompatibleClassChangeError.

Hi,
i'm quite sure that the project build was at least fully cleaned. Why it's happening only on 5.0 - 5.0.2 android? Maybe a link to Crashlytics will help: http://crashes.to/s/7a9918f4360?

Others see this problem for some Samsung devices as well.
Did you check that ProGuard does not remove the annotations?

We don't use proguard.

Well, then as linked above this is likely a bug with the firmware on those specific Samsung devices. So no change in EventBus can fix this, it will require a fix by Samsung.
Not aware of a work around either :(

Hello,
Could I quickly check that there's been no change on this stance? We're seeing this issue and it seems like there's nothing we could do on our end to mitigate it.
Cheers,
Matt

@etp404 Sorry, have no new info about this issue. -ut

@greenrobot-team I got this issue today. This is the logs on Crashlytics:
Fatal Exception: java.lang.IncompatibleClassChangeError: Couldn't find org.greenrobot.eventbus.a.threadMode
       at libcore.reflect.AnnotationAccess.toAnnotationInstance(AnnotationAccess.java:659)
       at libcore.reflect.AnnotationAccess.toAnnotationInstance(AnnotationAccess.java:641)
       at libcore.reflect.AnnotationAccess.getDeclaredAnnotation(AnnotationAccess.java:170)
       at java.lang.reflect.Method.getAnnotation(Method.java:301)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findUsingReflectionInSingleClass(SubscriberMethodFinder.java:175)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findUsingInfo(SubscriberMethodFinder.java:88)
       at org.greenrobot.eventbus.SubscriberMethodFinder.findSubscriberMethods(SubscriberMethodFinder.java:64)
       at org.greenrobot.eventbus.EventBus.register(EventBus.java:143)
...

And indeed it happens on Android 5.0 (Samsung Galaxy Note3).
I use this version of the library:
    implementation 'org.greenrobot:eventbus:3.2.0'

It's quite a rare bug though. I think I might have seen it in the past, but now it's a single case on Crashlytics.
Maybe it's only on Android 5 ?


"Outbound connectivity for IP address ranges" content format

There are user suggest to update "Outbound connectivity for IP address ranges" format according to the another article format(https://docs.microsoft.com/en-us/azure/site-recovery/azure-to-azure-architecture#outbound-connectivity-for-ip-address-ranges) which is more clear and user friendly.

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

ID: fa423fbe-759a-16f7-ce8a-c46eeb416037
Version Independent ID: 5e6271b8-0970-db08-7adf-c51e06d89ad8
Content: About networking in Azure to Azure disaster recovery using Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-about-networking.md
Service: site-recovery
GitHub Login: @sujayt
Microsoft Alias: sutalasi

Assigning @sujayt for visibility and document enhancement considerations.

We don't want to copy everything from networking article into the architecture section. Hence I have added a link into the document to make it more explicit.

https://github.com/MicrosoftDocs/azure-docs-pr/pull/86065

#please-close


longevity test: add step to delete/redeploy poet servers

Motivation

part of  #3441

Changes

systests: discover poet servers
systests: add step to delete and redeploy poet servers
activation: allow actual update of poet proof service and add logging

there will be a significant difference between current poet config and poet that runs for the whole epoch.
currently poet runs several hash iterations (1 << n) and starts a next round. it will run multiple rounds for the same epoch.
so there is a significant chance that node can register in one of them and succeed.
poet that runs for the whole epoch starts round only once and executes it for the whole epoch. if it fails and gets restarted later - it will finish round, but with less ticks. in order for it to work it needs to have a persistent state. in current version state is deleted with the poet.
so given the above, deleting a poet completely will guarantee that smeshers won't generate activations for one epoch. does it change anything in your tests?

so given the above, deleting a poet completely will guarantee that smeshers won't generate activations for one epoch. does it change anything in your tests?

yes. that's exactly what i want to achieve. i'd like to create such scenario (no ATXs for >1 epoch) and make sure the network moves forward despite it.
we should also test the case where poet's state persist and finish with less tick. but not the focus of this PR

bors merge

bors merge

bors merge


session not updating after you log in !

hi,
i just noticed, when you log in successfully , the session is going undefined and because of it the navigation options are not getting updated after history push , but when you reload the page it will update the ui and you can see the session variables filled with the data too.
.

Works for me. Hm, anyone else experiencing this issue?

@Gamezpedia which browser are you using and is there some error showing up in the developer tools for you?

hi i am using chrome -
i put some log checks and one in withSession - for the first time when page loads i can see empty session object every where - till the server response arrives in the form of an error, and after that session object get's undefined. and after that i reloads the page and sesiion object is again filled.

well i fixed it :D i was returning an error when the authorization header was not getting any token in my Graphql server , and the session var was getting undefined .


Better highlighting of archved labels

as followup of the not jet finished discussion at https://github.com/go-gitea/gitea/pull/29680#discussion_r1521867261
we enhance and chat about how best to highlight archived labels here :)

Could you post a screenshot?

Maybe opacity alone is enough, not sure. Some screenshots of the variants would help.

opacity .5:

opacity .5, greyscale .5:

opacity .5, greyscale .8:

I think I prefer the second, preserves a bit of the original label coloring.

better idea:  #30376


Badly formatted HTML causes PHP maximum nested calls reached.

I was trying to parse the HTML from a large collection of URLS, and this website was in the list.
http://www.weather.gov/
HTML validation shows that the <area> tag is not closed in the document.
https://validator.w3.org/check?uri=http%3A%2F%2Fwww.weather.gov%2F&charset=(detect+automatically)&doctype=Inline&group=0
I think this triggered this error.
Fatal error: Maximum function nesting level of '256' reached, aborting! in C:\work\tags\web\vendor\paquettg\php-html-parser\src\PHPHtmlParser\Dom\AbstractNode.php on line 212

I do pre-process the HTML with Tidy but it doesn't prevent this failure.
I'm not asking that the parser be able to handle invalid HTML, but the issue for me is that this is a critical PHP error. It can not be caught with a try/catch and causes PHP to just crash.
Is there any kind of fix that could be added to prevent this edge case from triggering a fatal error?

You can now change php settings to throw expections in cases like these so you should be able to catch these errors going forward.


bank selection

hi
there is something wrong with bank selection display

@piotrm73 please take a look at console if there are some JS errors.

yep there was a problem with the path to styles http://localhost:8000/src/View/Styles/style.css, I copied it to another location and form looks ok.
one more question, how to change language? I was trying (new Lang())->setLang('pl') but there is no effect
thanks

@piotrm73 you can change any frontend form language by adding
(new Util())->setLanguage('pl');
before method getting the form body.
Also if there are some problems with static files like style.css or JS, you can set any custom path to those files by running
$Util->setPath('path/to/static/files');
where $Util is the instance of Util class.

now it's working... I had to create directories View/Styles under path_to_static, now it's looks like this: path_to_static/View/Styles/style.css
thanks for your help


haskell.packages.ghc96: Remove unnecessary jailbreaks

Description of changes
I have manually check these.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

OfBorg is reporting that it can’t evaluate haskintex and profiteur?

Done. I am unsure if it should still be marked broken on GHC 9.8 somehow. Not sure how to do that.

We generally don’t bother about that.


Option to not open plugin browser every time app is started

I don't need to see the plugin browser every time xbar is started.
version: v2.0.39

It actually shouldn't be. Did you shutdown your laptop with the "reopen windows at startup" option?

Did you shutdown your laptop with the "reopen windows at startup" option?

No
I just confirmed again by quitting the app and restarted it and the browser opened again.

The browser does not open for me in this way. I wonder why it's opening for @grr?

Yeah I can't reproduce either.

Just noticed the README says minimum req is Catalina. I'm running Mojave, so that's probably the reason.

Nice pickup. I'm still interested to know why and now I know you're running Mojave,  I can test. Thanks!

This would be preferably managed by an entry in application prefs?
DoNotOpenPluginBrowserAtStartup = true
And app itself should also read prefs file in the global domain at /Library/Preferences/

It shouldn't be showing up at all until you open it. It seems that it's a quirk of Mojave which is being investigated

FYI, my Mojave machine died, so i won't be able to test if a solution becomes available.

I also have this problem. In my case internet is usually slow or not at all after waking up from sleep (Don't ask). For that reason my personal plugin connections fail and when I am trying to refresh entire xbar app fails (disaperar form menu bar). I have to run xbar from Launchpad and that's when I am seeing plugin gallery (too many times per day).
So I would appreciate an option not to show plugin gallery on xbar start up - just going straight to running my plugins.

@chew-z thanks for sharing your experience, that sounds very annoying. What OS version are you running?

BigSur 11.3

@leaanthony could this be some kind of error is occurring, which is causing the window the be shown?

I am running xbar on BigSur 11.3(.1) too and can second the problem of xbar opening the plugin browser on (macOS and/or manual app) startup. I'd be happy to help finding a solution - just not versed in GoLang at all

I'm having this problem as well on a MacBook Pro (13", 2016, with touch bar) - running Big Sur 11.3.1 and the xbar release 2.0.42. It looks to me like it's some kind of timing issue. It happens only when xbar is starting automatically at login, so while all the other apps are loading at the same time, not if i quit xbar and just restart it.


Make it possible to connect to a server using only one device.

I am on Ios and i run into a problem when trying to connect to a custom server. When i try to log in using a microsoft account it asks me to log in using a browser, but if i try to open a browser app and leave minecraft running in backround, minecraft kicks me out of the server so i cant connect with only my iphone, i have to use another device like my computer to complete login and not have minecraft kick me when switching apps. The mojang login does work when testing with my friends account, buti dont have a mojang mc account.

Related to https://github.com/GeyserMC/Geyser/issues/2496

Closing in favour of https://github.com/GeyserMC/Geyser/issues/2496 as mentioned since this isn't strictly a GeyserConnect issue


Bug fixes 202204

Separate console view logic from core model to address blinking menu bar issue.
Fix an issue which cause app unable to launch at startup.

Merged, thank you!


Tutorial Page fiori-tools-cap-modify-object-page.md Issue. PROD

Tutorial issue found: https://github.com/sap-tutorials/Tutorials/blob/master/tutorials/fiori-tools-cap-modify-object-page/fiori-tools-cap-modify-object-page.md contains invalid tags. Even though your tutorial was created, the invalid tags listed below were disregarded. Please double-check the following tags:

software-product>sap-fiori-elements
softrware-product>sap-business-technology-platform

Affected server: PROD

@cjbernal there is one typo that I missed in the review and the other tag should be software-product-function>sap-fiori-elements

@MichaelCzcz No worries. Here's the PR: https://github.com/sap-tutorials/Tutorials/pull/22749/

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Cant manually Install on D11, W11

I keep getting the following error when trying to install

Am i missing something?

Are you installing packages from Git-Hub or from GetIt?
If you are using Github repo, I've updated now the installation documentation: you must open the package group from \svgIconImageList\Packages\D11\SVGIconImageListGroupPackages.groupproj, then buil all run-time packages and install design time packages.

Installing from GitHub, sorry I am new to adding packages etc, normally use DELPHI in Vanilla, I take it that the run-time packages start with SVG and the design time packages start with dcl

Did you built the packages with my instructions? can i close this issue?


feat: initialize wasm in parallel

This improves the launch speed of games using multiple packages

Just to confirm how this works - each module is instantiated in a Tokio task, which is run on a worker thread somewhere, and then block_in_place is used to dedicate that worker thread to the task of instantiation so that other threads can process the other tasks. Is that right?
That seems reasonable to me; is there any risk of all of the worker threads being used on instantiation and blocking all async work in the meantime? My understanding was that you should generally use threads for compute-heavy work to avoid stalling the executor, which is why I explicitly used threads before.


Determine test plan

Existing deck integration tests generally rely on the CLI. This repo will not have that CLI readily available, and will need some changes to its testing strategy.
Offhand, I'd expect the simplest option is have the test actions here check out a copy of the deck repo main, update its go.mod to use the PR head commit as the current version of this library, and run deck's usual test suite. This approach will probably hit chicken-egg problems where coverage for this repo relies on features that aren't yet merged into deck main. Unsure how to best handle that, but we can maybe add a variant of tests that lets you choose the downstream branch and still satisfies the same Actions status.

https://github.com/Kong/go-database-reconciler/pull/10 is the most basic option for this. It reuses existing tests by importing deck as a test dependency here.
This generally works, though creates a (test only) dependency chain that's a bit silly. To avoid it we'd need to implement some sort of alternate to the client code, that invokes the sync/diff functions here without either of the downstream consumers (deck and KIC) of this library.


Is this implementation right for react native?

Can I use the 3dMap from this demo to use it in my react native impl?

No, this library wraps mapbox-gl-js, which is for the web. For native, see https://github.com/mapbox/mapbox-gl-native / there are also some react native wrappers unrelated to react-mapbox-gl

@esuarezz
For React Native, try https://github.com/react-native-mapbox-gl/maps .
This is the actively maintained succesor to https://github.com/nitaliano/react-native-mapbox-gl .


Route.fullPath is typed incorrectly when using path params

Describe the bug
The type of Route.fullPath is more or less the path that is passed to the createRoute function, but during runtime it is the current path of the page when Route,fullPath references a path with a paramater, which included the sub path, this makes it useless as a from parameter when used in a layout route.
Your Example Website or App
https://stackblitz.com/edit/github-jhyjxh?file=src%2Froutes%2F%24subsection.tsx
Steps to Reproduce the Bug or Issue
Click on sub, click on the links and see every navigation is relative to the current url instead of the expected Route.fullPath
Expected behavior
Not that, the same as for static routes
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Firefox
Version: 128a

Additional context
No response

The issue at hand is not that Route.fullPath is incorrectly typed, instead the following happens:
At runtime, router does not respect from, instead it always uses the current location when building relative links.
We had discussed this before, we even had a PR merged for this (see https://github.com/TanStack/router/pull/1595) but reverted since we were not confident about possible side effects.
cc @TkDodo

Wow, I wouldn't've guessed that

How is the behaviour of the demo I posted explained?  If I remove the path param it works fine

which path params? can you please post a separate working example?

I think the issue is simply that if you are on /$subsection/section-a and you make a link to ./section-b, you will wind up on /$subsection/section-a/section-b, no matter if you provide from or not. TypeScript will allow it because from is defined and points to $subsection, so we have a discrepancy between type-level and runtime behaviour.
This should be the same no matter if $subsection is used or something static. If that's not the case, please show a separate reproduction.
So I see a couple of ways forward:

make runtime behave like type-level and take from into account when building navigations. This is similar to what I've tried before but we reverted the merge.
don't allow relative routes with ./ or ../ if a from is supplied, as it's simply confusing. If I provide "from", I expect the link to navigate "from" that start. Since it always takes the current route at runtime (which we can't know at compile time), this would force us to use absolute routes when a from is supplied, like:

from="/$subsection"
to="/$subsection/section-b"

that will work on whichever page we are on, and it will still guarantee that you don't have to provide a param for $subsection on type level, because it will be narrowed by what we pass to "from".
Tbh, 2) is what I'm doing right now because I think navigations with ./ or ../ don't really work the way I'd expect them to. When I'm just reading a <Link> component in isolation, and it says:
from="/$subsection"
to="./section-b"

I'd expect the result to be "/$subsection/section-b", no matter on which page I'm currently on. But I also understand that isn't easy to achieve at runtime (I tried 😂), so maybe enforcing option 2) is a good compromise? @chorobin FYI

Let's try to fix the runtime. It can't be harder than the iterations we went through for the types for relative pathing

which path params? can you please post a separate working example?

Here is the version that works, https://stackblitz.com/edit/github-jhyjxh-4xvfsf?file=src%2Froutes%2Fsubsection.section-a.tsx only difference is $ removed

@Tiedye thank you. really interesting that this works and probably a good starting point to figure out the runtime differences


trianglevulkan13 example: invalid value is used in api call

In trianglevulkan13.cpp, the function render() contains
// Draw indexed triangle
vkCmdDrawIndexed(commandBuffer, indexCount, 1, 0, 0, 1);

The last argument to that function is firstInstance which should have been set to 0.

It actually doesn't matter, as instanceId isn't used anywhere. Will fix next time I change something about that sample.


Add the ability to add a header to the top of the comments

Feature
Would love a way to apply a header to the top of the comment when sent to the PR.   We do this internally so it shows the current version of the action was run, the latest version available and instructions inline on what commands can be taken in comments.
I can submit a PR on how we do it, but it's got some jank in how it does it.
Please notify me if you want me to submit a PR on this.


Compiler warnings for OTP 27

Some compiler warnings emitted by OTP 26:
$ ./make

/couchdb/src/recon/src/recon_alloc.erl:702:7: Warning: matching on the float 0.0 will no longer also match -0.0 in OTP 27. If you specifically intend to match 0.0 alone, write +0.0 instead.
/couchdb/src/recon/src/recon_alloc.erl:702:11: Warning: matching on the float 0.0 will no longer also match -0.0 in OTP 27. If you specifically intend to match 0.0 alone, write +0.0 instead.

/couchdb/src/couch/src/couch_ejson_size.erl:35:14: Warning: matching on the float 0.0 will no longer also match -0.0 in OTP 27. If you specifically intend to match 0.0 alone, write +0.0 instead.

This has been addressed in recon upstream: https://github.com/ferd/recon/commit/c7c07ad8193ec6470c775c213e15e0f3447114b7

This has been addressed in recon upstream: ferd/recon@c7c07ad

Thx for the hint Gábor. 🙏


[WIP] Adding histogram equalization and histogram matching

Description

Histogram equalization implementation.
Histogram matching (specification) implementation.
Histogram visualisation as .jpg image.

This is part of my competency test solution for GSoC 2020 and I would be happy to improve it according to feedback.
References

https://en.wikipedia.org/wiki/Histogram_equalization
https://en.wikipedia.org/wiki/Histogram_matching
The great book https://link.springer.com/book/10.1007/978-1-84800-191-6
Thank you for recommending the book it is very good.
Tasklist

[ ] Add test case(s)
[ ] Make a better generic implementation
[ ] Downscale histogram visualization image.
[ ] Ensure all CI builds pass
[ ] Review and approve

If this PR is related to GSoC and the competency test, it would be helpful if the description states that, for example, see #435.
As we already have received similar PRs, i.e. GSoC and competency tests, please find them and read through the comments to see what to expect w.r.t. to your PR For example, see https://github.com/boostorg/gil/pull/435#issuecomment-588371700 and #430

Thanks once again for your efforts, but this functionality has been developed as part of the [gsoc2020] Histogram Computation and Algorithms  project.
You are more than welcome to help us improve the histogram support.

Thanks once again for your efforts, but this functionality has been developed as part of the [gsoc2020] Histogram Computation and Algorithms  project.
You are more than welcome to help us improve the histogram support.


"Nicolești", "Obrănești", "Petecu" and "Vasileni" are listed in the drop-down menu

Description:
"Nicolești", "Obrănești", "Petecu" and "Vasileni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Nicolești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Nicolești, HARGHITA (Ulieș)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Obrănești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Obrănești, HARGHITA (Ulieș)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Petecu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Petecu, HARGHITA (Ulieș)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Vasileni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vasileni, HARGHITA (Ulieș)".


Add Integration tests

Description
Changes proposed in this pull request:

Add integration test scenarios

Related issue(s)

https://github.com/kyma-project/istio/issues/137

/test pull-istio-lint


Added validFamily function

This checks if a given device address belongs to the device families that the library supports.
This is useful for when you have other types of 1-wire sensors in a network, and don't want to send DS18xx specific commands
to those.

Indeed a usefull function.
It could be incorporated into validAddress() as that only checks if the CRC is OK but not if the address is indeed a DS18xxx
// returns true if address is valid
bool DallasTemperature::validAddress(const uint8_t* deviceAddress)
{
    if ( _wire->crc8(deviceAddress, 7) != deviceAddress[7]) return false;
    if ( !validFamily(deviceAddress) ) return false;
    return true;
}

That was my first idea, but this totally screws with the rest of the lib... so I kept away from intermixing it with any of the other code.
For instance if you have any other 1-wire device you cannot use requestTemperatures anyway, since it assume you can send the STARTCONV0 command to all devices on the bus. You have to loop through and use requestTemperaturesByAddress instead only for the correct devices.
What I do is to search for the devices:
typedef enum sensor_type_e
{
	SENSOR_DS18B20,
	SENSOR_DS2762
} sensor_type_t;
....
sensors.begin();
count = sensors.getDeviceCount();

for (int i = 0; i < count; i++)
{
  sensors.getAddress(mysensors[i].addr, i)
  if (sensors.validFamily(mysensors[i].addr))
  {
    mysensors[i].type = SENSOR_DS18B20;
  }
  else if (theOtherFamilyType(mysensors[i].addr))
  {
    mysensors[i].type = SENSOR_DS2762;
  }
}
...
// Then when getting temperatures.
...
if (s->type == SENSOR_DS18B20)
{
  sensors.requestTemperaturesByAddress(s->addr);
}
else if (s->type == SENSOR_DS2762)
{
   // Do other stuff....
}

The index searching and everything breaks if you simply add validFamily to validAddress, at least when I tried to do it like that. A lot more rewriting would have to be done to get that work properly, and I don't think it's worth it, because I guess 99% of people will only have one type of device on the bus anyaway.

This felt like a lot better solution, since I have 12 DS18B20, and 1 DS2762 sensor... To me it made more sense to do it very simply like this. Compared to screwing with the indexes and stuff, when this scenario is pretty much a corner case

Good argument.

Just started working with the library and thought it would be good to include validFamily() into validAddress(). Came across this nearly two-year-old thread. It looks to me like all the “…ByIndex()” methods call the getAddress() function which calls the validAddress() function. So, wouldn’t all the indexing still work but only relative to the DS18…. Devices?
Meaning, if there were four DS18… devices on the bus they would always be indexed 0-3 regardless of how many other devices were on the bus and in what address position. There could also be a function to let the user know if any other types of devices are on the bus. They would then know not to use the global requestTemperatures() function.
Will give it a try and report back.

I think it's a bit weird to "lie" about the index for this type of case. And I don't really see what you win by doing that? This is quite low level programming anyway.
It just becomes obfuscated what the index means if you do as you describe. And the index won't correspond to the 1-wire search algorithm as described here:
https://www.maximintegrated.com/en/app-notes/index.mvp/id/187
Which means it becomes harder to troubleshoot and understand which sensor is where physically and so on.
It would be better to then have that as some separate concept than Index in that case. And not mess with this current use case.

Understand what you’re saying. Don’t want to replace the concept of the OneWire index with a special “temperature sensor index” just for this device family. I’m working on a special purpose class for a project. It will inherit from DallasTemperature. I can deal with non-validFamily() devices there instead.


MacVim Window Resizing

Is it possible to override something in MacVim so that the Window can be freely resized regardless of columns/lines.
For example like this option in iTerm 2:

https://github.com/macvim-dev/macvim/wiki/FAQ

My question is not answered in this FAQ. Where can I ask for help?
After doing so you may post your question to the vim_mac(https://groups.google.com/forum/#!forum/vim_mac) Google Group if it is MacVim related

No, and MacVim will not support.
:set nowrap


Master Branch Pose completely off

Hi, I am currently experimenting with ICPCUDA.
When I checkout exp Branch, run it with rgbd_dataset_freiburg1_desk and feed my poses to the TUM evalutaion tool (off- or online) I get an error (rmse) from about 0.147 (about the same as stated in readme) and a plot which looks like this:

So far so good.

I haven't looked at the code but my first impression is that it's one of two things:

The two branches use a different unit (e.g. metres vs mm) to express the translation component of the pose.
One of the branches returns relative transformations between successive poses, and the other returns absolute transformations from the origin (possibly the first pose).

Hope that helps!

Works fine for me on master. Without more information about your OS, graphics card, graphics card driver, CUDA version, compiler, Eigen version etc... there's not much I can infer!

Hi,
OS is Kubuntu 14.04.1, Graphics Card is GTX960m, Cuda Toolkit 7.5 (nvcc 7.5.17) with nvidia driver 352.99, Eigen on current master branch

Any changes to the code to speak of? As mentioned in the README, you should be using Steven Lovegrove's branches of Eigen, Pangolin and Sophus

no changes to the code (clean checkout)
eigen, sophus, pangolin from the source in the readme

I'm afraid I can't reproduce. Tried on two different machines, one 14.04 and one 16.04

hmm bummer. Ok I will try a fresh install

Pulled/Build/Installed most recent version of eigen, pangolin and sophus. Now it works!
Thanks for your help!


cb never gets called if invoking download with no arguments.

When you invoke core.download(cb) without specifying a start and end, it never invokes your cb.
Here's a rough example with hyper-sdk:
const SDK = require('hyper-sdk')

run()

async function run () {
  const { Hypercore: H1, close: close1 } = await SDK({ persist: false })
  const { Hypercore: H2, close: close2 } = await SDK({ persist: false })

  try {
    const c1 = H1('example', {
      valueEncoding: 'json'
    })

    await c1.append('Hello World!')

    const url = `hyper://${c1.key.toString('hex')}`

    const c2 = H2(url, {
      valueEncoding: 'json'
    })

    await c2.ready()

    await new Promise((resolve) => c2.once('peer-open', resolve))

    await c2.update({ifAvailable: true})

    await c2.download()

    // This reaches, though
    // await c2.download({start: 0, end: c2.length})

    console.log('Should reach here but doesnt')

  } finally {
    await close1()
    await close2()
  }
}

I think this is due to end getting set to -1 here which means it'll keep downloading new bits of data and never be finished.
I propose changing the default range to this.length so that the download() completes after the current data has been downloaded. I think this would be a breaking change.
An alternative might be to document that the cb won't get invoked if you don't specify an end so that folks don't get tripped up by it (spent 20 mins adding debug statements all over the place wondering why I was stuck 😂 ).

Ya, I'm not happy with the current api here either. I think we should add a live flag instead of the -1 thing to make it super clear (defaults to false). wdyt about something like that?

I think a live flag would be great. Defaulting it to false sounds like it'd be more intuitive, too. I kinda got what -1 meant from looking at the sparse flag in the constructor, but it was a lucky guess. 😅

changed in 10


[Not for merge] MBR test.

See https://github.com/k2-fsa/snowfall/pull/84#issuecomment-772185201

OK. Well at least changing that weight has more effect on the MMI loss than on the MBR loss, but those loss values don't give me great confidence that the autograd is correct for the MBR loss. Can you try an experiment where in the 1st epoch it's mmi_loss + mbr_loss, and afterward it's just mbr_loss? (You can still plot the mmi loss). I want to see whether it just goes crazy.
I want to develop some automated testing for the autograd, have to think about it..

What this pull-request does is:

for epoch == 0, total_loss = mmi_loss + mbr_loss
for epoch >= 1, total_loss = mbr_loss

Loss curves
https://tensorboard.dev/experiment/jvlfa7oyRQiDxvO3hPTXZg/#scalars&_smoothingWeight=0
After epoch 0, mmi_loss increases as it does not contribute to the total loss. mbr_loss does not change too much after epoch 0.

OK, looks reasonable.

@danpovey
I have some doubts about mbr_loss. It only starts to decrease after epoch 7, when the learning rate starts to decrease.
From epoch 1 to epoch 6, mbr_loss is nearly kept the same.
Since the code tries to minimize mbr_loss, is there a reason why mbr_loss does not decrease
during epoch 1 to epoch 6?

The gradients for the MBR part of the loss can be a little spiky, and I think it's not as easy to optimize as the MMI part.

Could be gradient noise throwing it off.
We should create some randomized gradient checks for the whole procedure though.


[Docs] Update How to 8, label annotator

Closes: https://github.com/napari-threedee/napari-threedee/issues/211
In this PR:

rename the plane_labeler script and docs path to label_annotator
update the ToC
These changes update the naming to reflect the codebase and widget name
Within the How To I provide information how to download/run the script and make minor clarifications in the copy.
Note that the diff suggests a new file, but it was renamed and updated in place.

I updated the download and activate wording to match the suggestions from the other PR(s)


Improve the Readme of the repository

The readme should explain what the tool is providing:

features listing
sample to try
user guide
how to contribute

Screenshots and recordings would be great to have as well


1.6.1 build failed, 17 issues

1.5.0 worked fine.
1.6.1 gives 17 errors.
Environment

swift 4.0
target iOS 10.0

I guess I need to update to swift 4.2 too.
Thanks for your effort on this amazing photo viewer.

Hey @Hardtack1, v1.6.0 introduced Swift 4.2 support - in order to use the photo viewer past v1.6.0, you need to update your project to Swift 4.2.

Ok, thanks!


feat(lines): add 1 line

It was UI_MENU_PAGE_HITS_ELEMENT_CATEGORY_Elusive_Target_Hits with lowercase characters in the game's menu JSON files. The add_hashes.py script should probably uppercase paths automatically since I suspect that's why this one wasn't found before.

When did you make this!?!?!?


multipart/form-data request not supported?

I can't seem to upload files using a POST. Is this supported in the module?

Yes. I upload files via API. POST Content-Type: multipart/form-data; boundary=X-INSOMNIA-BOUNDARY
But if you need to download large files then you need to set AllowAjax variable
    $ul = wire(new WireUpload($formName));
    $ul->setValidExtensions(['mp4', 'avi', '3gp']);
    $ul->setMaxFiles(1);
    $ul->setMaxFileSize(100 * 1000000); // 100 MB
    $ul->setOverwrite(true);
    $ul->setDestinationPath($p_path);
    $ul->setLowercase(true);
    $ul->setAllowAjax(true);
    $files = $ul->execute();

@andy-scboy Thanks for stepping in, very much appreciated! :)
@spoetnik Generally speaking the module does not limit what you can do, basically it's just a router to your own functions, so anything you can do in PHP or in ProcessWire should be possible with this module aswell.

Thanks for the help @andy-scboy but can you please help me a bit further?
Looking at some examples, the use the $data variable to access the POST data. https://github.com/thomasaull/RestApi/blob/b628d6bed7160970bd4a472ceca98ad6bdb9ad8a/Auth.php#L46
My 'files'-field is not in the $data-array my function receives.
You mention in your code-example above '$formName', that's not part of the $data-array.
Can you please give me a code example of the function receiving the POST data?
Thanks in advance!!

As example add line in Routes.php
 ['POST', 'sp/{id:\d+}', Example::class, 'setPage', ["auth" => false]],
It give you POST request by address: Your_Domain/sp/page_id (as ex. localhost/sp/1821)
Next step edit Example.php and add function.
We will send different data:
id from routing (int);
field_digit (int);
field_name (string);
field_double (float);
photo (file).
public static function setPage($data) {
// Sanitize your fields to use
// but we don’t have a file here. we will take it the other way.
$data = RestApiHelper::checkAndSanitizeRequiredParameters($data, ['id|int'],['field_digit|int'],['field_name|string'],['field_double|float']);
// give a response
$response = new \StdClass();
// get page by id
$page = wire('pages')->get($data->id);
if(!isset($page->id)) throw new \Exception('page not set');
// change fields in page
$page->title = $data->field_name;
$page->number = $data->field_digit;
$page->temperature = $data->field_double;
$p->save();
Now let's talk about the file
// Use a global variable
	if($_FILES){
// PHP load errors with code
        $f_error = $_FILES["up_photo"]["error"];
// request header
        $fheader = getallheaders();
        $f_size = $fheader["Content-Length"];
// name label by reguest
        $formlabel = 'up_photo';
// name of file by request
	$formName = array_key_first($_FILES);
	$TempDir = $_FILES[$formName]['tmp_name'];
// path where this file is usually located in PW
	$p_path = wire('config')->paths->assets . "files/". $page->id . "/";

// next file manipulating

	    $ul = wire(new WireUpload($formName));
	    $ul->setValidExtensions(['jpg', 'jpeg', 'png']);
	    $ul->setMaxFiles(1);
	    $ul->setMaxFileSize(100 * 1000000); // 100MB
	    $ul->setOverwrite(true);
	    $ul->setDestinationPath($p_path);
	    $ul->setExtractArchives(false);
	    $ul->setLowercase(true);
	    $ul->setAllowAjax(true);
	    $files = $ul->execute();
	    $err_array = $ul->getErrors();

// WireUpload errors
	    if($ul->getErrors()) throw new \Exception("ErrUp:".$err_array[0]);

	    if(count($files)) {
// take the first file
	        $file = $p_path . reset($files);
		    // do something with your file
		    $resp = $page->photo->add($file);
		    if($page->photo->$resp->filesize == 0 ) throw new \Exception('Photo not correctly upload');
	    	$page->save();

	     $response = $files[0];
            }

Sorry if I wrote with errors. I just wanted to state the principles.

Thanks a lot @andy-scboy !!! This helps me a lot! I would have never figured this out myself. Thanks, thanks, thanks!!!


Switch Position Control

Create a place to control a switch driver and switch.

Switch control has been added.


Only color options

We should add special option to work only with colors, because it is more popular case.)

lol )) https://github.com/lutien/postcss-extract-value/blob/master/index.js#L12

Oops :)


Leaner frame invariants

To be tested if it has a measurable impact and if would make sense to pass around sequence/encoder-config as explicit arguments.

Coverage decreased (-3.09%) to 65.367% when pulling f10ce1cab2bf497f65c38f234d3f7ab10d079f21 on rust-av:leaner-frame-invariants into 3d10b4e623f4ef6a6c4609bcac144acc3362f0e8 on xiph:master.


Add ALB for search-api external access, use a CNAME for internal access

This PR removes the search-api internal service name, that should more properly be a CNAME pointing to the "search" hostname, as search-api lives on the app-search machines (just like rummager).
It also adds an ALB so that Carrenza IPs can talk to search-api over the public internet, which is needed for:

Traffic replay from rummager to search-api while we have both running in parallel
Direct communication from search-admin and whitehall when rummager is switched off

govuk-aws-data PR: https://github.com/alphagov/govuk-aws-data/pull/339

Trello card

Everything looks ok to me, but might be worth asking someone in RE to take a look, since they are far more knowledgable than me about the Carrenza/AWS link.


Backtracking Documentation

Description

Would like to contribute documentations for Backtracking

Programming language

[x] C++

Are you contributing under any open-source program ?

Hacktoberfest

could you please assign this issue to me

Description

Would like to contribute documentations for Backtracking

Programming language

[x]  C++

Are you contributing under any open-source program ?

Hacktoberfest

Assigning this to you🎊🎊🎊

could you please assign this issue to me

Find other issues

@siddhi-244 Actually, DSA Repositories are excluded from Hacktoberfest - according to the recent update.
So @nishajha629 can take this up

@siddhi-244 Actually, DSA Repositories are excluded from Hacktoberfest - according to the recent update. So @nishajha629 can take this up

Who said that?

@siddhi-244 Actually, DSA Repositories are excluded from Hacktoberfest - according to the recent update. So @nishajha629 can take this up

Who said that?

@hariketsheth it's not our repository.  Our repository is a documentation repo. And it counts once ur prs are merged

@siddhi-244 Actually, DSA Repositories are excluded from Hacktoberfest - according to the recent update. So @nishajha629 can take this up

Who said that?

The prs need to be merged first.

@hariketsheth it's not our repository. Our repository is a documentation repo. And it counts once ur prs are merged

Ohh Right, This repository is not maintaining codes alone.
Great ! Thank you for clarifying. :)
Can you re-assign this issue to me. Also, sorry for the inconvinience


skip_to_end in HTTP::Content#close: Useful or problematic?

HTTP::Content provides a single method, #close that delegates to super (IO#close) but ensures the IO is completely consumed before closing via skip_to_end.
I can't see how that is actually useful, and at least, it is confusing to the user. It doesn't just close the IO in it's current state but completely consumes it until the sender is done (a request from curl -T /dev/urandom example.com  would never actually close...).
This feature seems to be the only reason for the HTTP::Content type in the first place and was introduced in 5adc780f. Unfortunately, I can't find an explanation for the reasons behind that and there are no specs documenting a change in behaviour.
At the point of that commit, there was only a single HTTP::Response class and not yet separated between server and client. Not sure if that matters in any way.
I could only think of a possible explanation in the context of reusing a connection (HTTP keep-alive), where you would obviously need to completely consume the previously sent content before you could read the next request or response from the IO.
But that actually only matters in RequestProcessor and as far as I can see, wouldn't demand for an override of the close method. This would be it (and the special behaviour is even mentioned here):
--- i/src/http/server/request_processor.cr
+++ w/src/http/server/request_processor.cr
@@ -57,6 +57,7 @@ class HTTP::Server::RequestProcessor

         # Skip request body in case the handler
         # didn't read it all, for the next request
+        request.body.try &.skip_to_end
         request.body.try &.close
       end
     rescue ex : Errno

Without that, only spec/std/http/server/server_spec.cr:579 would fail.
With this change, we could remove HTTP::Content#closet. However, it would still provide a point of attack when a client just keeps writing to the socket.
Some light improvement could be to provide a means for the handler to force closing the connectio (by checking the response has Connection: keep-alive and Content-Length headers or closing the request IO in the handler directly). There could also be some limit to the amount of bytes that will be read to avoid never-closing connections.
But all in all, I think it might perhaps be best to simply close the entire connection when there is still content on the input stream after the handler has completed. The handler should make sure to consume the body if there should be one.
The consequence is only that you can't reuse the connection if the client sent a body that the server didn't expect. I don't think that's too much to worry about.
/cc @waj
Initially mentioned in https://github.com/crystal-lang/crystal/pull/6912#issuecomment-428293598

I could only think of a possible explanation in the context of reusing a connection (HTTP keep-alive), where you would obviously need to completely consume the previously sent content before you could read the next request or response from the IO.

That's the reason it was implemented. But alternative ways are probably fine too (I don't have much time to read all this issue now).

It doesn make me notice that you can totally break keep-alive by using ctx.request.body =  though.

@RX14 What?


Images - Disk Button

Please use a different icon. Looks like a Save Button. The disk icon is better for saving.

https://github.com/sibvisions/reactUIDesigner/issues/19 ???

Nope, the Images have a small button with a disk in it. The disk means for me: save
But if you press the button, a file chooser is shown. So, the icon should be changed to a "load" icon.

Now using File-Upload icon, is that ok?


pull request for e2e test at 01/01/2017 22:45:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


hidpp20: add suport for the G602

Signed-off-by: Filipe Laíns<EMAIL_ADDRESS>

Hello I'm using a G602 and found that for some reason restarting stoping and restarting ratbagd and then restarting piper makes piper detect the mouse but for some reason one of the keys is missing from the mapping
https://imgur.com/a/m78K3hd


update to latest version of rules_go

0.34.0 does not have https://github.com/bazelbuild/rules_go/pull/3272

@Wyverald @meteorcloudy can either of you help with the import?

/cc @oquenchil Should be able to help!

Can you sync? When trying to merge internally it complains about merge conflicts.

@comius Pedro seems to be OOO until next monday, could you please help me with the merge as this is blocking for our 6.0 release? Thank you


Feature/mm 37053

Summary
Test script for Billing History functionalities
Ticket
https://mattermost.atlassian.net/browse/MM-37053
Release Note
NONE

Hello @vinod-demansol,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@saturninoabril this would have to run with a server that's hooked up to the CWS or a mock of it. Is that currently set up?

@saturninoabril this would have to run with a server that's hooked up to the CWS or a mock of it. Is that currently set up?

@AGMETEOR Yeah, fully aware of that. It's not yet deployed to Cypress prod test, but let me know if this is passing on you locally, and also when using https://github.com/mattermost/customer-web-server/pull/533.

@hmhealey , @AGMETEOR and @stevemudie , I request all of you to review my PR (8567).
@hmhealey , you have requested for change and I have made those changes, please review those changes and provide your feedback on the same.

Thanks for the bump. Sorry about the delay since I meant to approve after the package.json changes were done.


chore(deps): update common-dev-assets digest to edde38e

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
56ffb8d -> edde38e

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


PlaceHolder text

Issue  📐

I Sohel have worked for <#6>

Guidelines 🔐
I accept the fact that i have followed the guidelines and have not copied the codes from around the internet

[x] Contribution Guidelines
[x] Code of Conduct

Issue to be closed 🛅

My pull request closes <#6>

Screenshots 📷
Here are the pictures of changes that i have made 🔽

✅LGTM


Interceptors are missing when using 'schema' property

Environment

Operating System: Linux
Node Version:     v22.3.0
Nuxt Version:     3.13.2
CLI Version:      3.14.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-hi8a9n?file=nuxt.config.ts
Describe the bug
If you add schema to your config and define a Swagger URL, the interceptors are not included on the endpoint.
apiParty: {
  endpoints: {
    myApi: {
      url: 'https://api.mysite.com',
      schema: 'https://api.mysite.com/swagger/v1/swagger.json'
    }
  }
}

Without schema:

With schema:

Additional context
Expected:
Access to interceptors as outlined here:
https://nuxt-api-party.byjohann.dev/guide/interceptors.html
Logs
No response

Possibly something missing from src/runtime/openapi.ts that is only included in https://github.com/johannschopplich/nuxt-api-party/blob/c22e27f8fc884ef1baa51221c5871e776ec0d427/src/runtime/composables/useApiData.ts


Add the Sliding Carousel and Featured Posts Highlight.

Riccardo and I like the sliding carousel and the featured posts highlight thing on the right. Can we add this to make our blog more snazzy?

For now, this seems to be unnecessary. Maybe if there will be more posts. Let's keep this issues open for later discussion.

@Belliusz with the @ character you can tag ppl so they will see notifications. Like @RiccardoMasutti

The blog looks nice now, I don't think we need this. Closing. Feel free to reopen if there is a reason to implement this.


Update 10year preso with correct pptx

This PR updates one of our 10year event presenter's slides with the correct pptx file. @georgiamoon or @pboothe  PTAL?
This change is previewable here: http://<IP_ADDRESS>/blog/mlab-10year-wrapup/#m-lab-10-year-anniversary-event

This change is

LGTM.


Add module word in usage help

Fixes #86

Pull Request Test Coverage Report for Build 212

37 of 76 (48.68%)  changed or added relevant lines in 29 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 64.303%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/logic/commands/calendars/ShowCommand.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/ContactsParser.java
11
12
91.67%

src/main/java/seedu/address/logic/parser/contacts/AssignCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/tasks/AddCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/tasks/AssignCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/tasks/DeleteCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/tasks/EditCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/parser/tasks/FindCommandParser.java
0
1
0.0%

src/main/java/seedu/address/logic/commands/contacts/AssignCommand.java
0
2
0.0%

src/main/java/seedu/address/logic/commands/tasks/AddCommand.java
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/logic/parser/tasks/EditCommandParser.java
1
0.0%

src/main/java/seedu/address/logic/parser/exceptions/ParseException.java
2
85.71%

Totals

Change from base Build 209:
-0.1%

Covered Lines:
1578

Relevant Lines:
2454

💛  - Coveralls


Untranslated Content In The Chinese Document

There is a seemingly untranslated English paragraph in the TypeScript Chinese document.
Link to the document page: https://www.tslang.cn/docs/handbook/functions.html

I would like to translate the document but wonder how can I contribute to the Chinese website

I don't know. We don't own tslang.cn


Chainable monad type

Do you want to request a feature or report a bug?
feature
What is the current behavior?
no simple monad type which boxes a value up, allows chaining and retrieving of the value (as far as I know - if this exists then this can become a documentation request!)
Doing this with vanilla js requires using an array in a hacky way:
const str = 'TheRainInSpain'

const sentence = [str]
  .map(toSnakeCase)
  .map(s => s.replace(/_/g, ' ')
  .map(upperFirst)
  [0]

console.log(sentence) // 'The rain in spain'

Or hard-to-read reversing of functions:
const str = 'TheRainInSpain'

const sentence = upperFirst(
  toSnakeCase(str)
    .replace(/_/g, ' ')
)

console.log(sentence) // 'The rain in spain'

What is the expected behavior?
it could look something like this:
const str = 'TheRainInSpain'

const sentence = box(str)
  .map(toSnakeCase)
  .map(s => s.replace(/_/g, ' ')
  .map(upperFirst)
  .value

console.log(sentence) // 'The rain in spain'

where box is defined something like
export interface Box<T> {
  value: T
  map<U>(f: (val: T) => U): Box<U>
}
export const box = <T>(value: T): Box<T> => ({
  value,
  map: f => box(f(value)),
})

Looks like the Identity functor


Minor issue with classpath scanning dependency

In this build.gradle file, https://github.com/crashinvaders/gdx-lml/blob/02e46856b0c011c895162f09d169bc039b360b51/autumn/natives/desktop/fcs/build.gradle#L9 , the dependency implementation "io.github.lukehutch:fast-classpath-scanner:2.21" uses implementation, but at least in the one place I use it, in gdx-liftoff, some types from the fast-classpath-scanner dependency are needed outside of this project. This is easily resolved in Liftoff by adding implementation "io.github.lukehutch:fast-classpath-scanner:2.21" to my Gradle dependencies (I could use api instead if I was making a library, but Liftoff is an application). I just wanted to point this out here in case you want to change to api, or for other people googling this issue to find a fix.
For posterity, the error I encountered when running a JAR built with the desktop fcs code was:
$ java -jar gdx-liftoff-<IP_ADDRESS>.jar
Exception in thread "main" java.lang.NoClassDefFoundError: io/github/lukehutch/fastclasspathscanner/matchprocessor/ClassAnnotationMatchProcessor
        at gdx.liftoff.MainKt.main(main.kt:123)
        at gdx.liftoff.MainKt.main(main.kt)
Caused by: java.lang.ClassNotFoundException: io.github.lukehutch.fastclasspathscanner.matchprocessor.ClassAnnotationMatchProcessor
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
        ... 2 more

Sounds logical, thanks.
I'll make a similar change for the most of the leaf modules.
Not sure thought if this requires a new release. At this point there's nothing else to include. I guess it will just sit and wait in the snapshot for now.


Remove 3bot name in recover process since this is redundant data

the 3bot name is still there

Should be done here

Verified on Android 14,
Threefold-Connect: 4.0.0-184
APK:12-12-2024
3Bot name has been removed


feat: matroid duality

This PR adds the definition of the dual of a matroid and co-independent sets, as well as some notation and API.

Once you get the linting errors fixed:
bors d+

bors

✌ apnelson1 can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

bors r+

bors r+


[Doc] Update copyright year to 2024

Search before asking

[X] I had searched in the issues and found no similar issues.

Documentation Related

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct *

Can I work on this issue? kindly assign it to me!

Can I work on this issue? kindly assign it to me!

Sure. Welcome to the Apache EventMesh community!

This issue is duplicate with https://github.com/apache/eventmesh/issues/4718. I will edit https://github.com/apache/eventmesh/issues/4718 after this issue is closed.


Code handling

Dear Lukas,
thanks for the awesome design.
I have tested it in my blog, however some code snippets flows outside the colored environment.
See here

What am I doing wrong?

Thank you!
It's not your fault, I simply haven't included code highlighting. I embed github gists instead.
In my other theme github.com/lukas-h/material-theme is a file called syntax.css, which does the highlighting, and there is (I think) a Jekyll plugin also required

https://github.com/lukas-h/material-theme

https://gist.github.com/demisx/025698a7b5e314a7a4b5

Amazing Lukas, but it is not a matter of adding these files to the css directory, or is it?
I might be doing something wrong.

ul{
    list-style-type: none;
    padding: 0;
}
li{
    display: list-item;
    text-align: -webkit-match-parent;
    list-style-type: none;
    margin: 0;
}

You could edit the list styles in assets/css/main.css
https://developer.mozilla.org/de/docs/Web/CSS/list-style

And you can set the line-height CSS attribute for the highlighted text (I think you mean the code blocks

@lf-araujo are all problems solved?

Yep, you helped a lot! Thank you very much!
You can close this issue if you want..
Best,
2017-03-26 21:30 GMT+11:00 Lukas Himsel<EMAIL_ADDRESS>
@lf-araujo https://github.com/lf-araujo are all problems solved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lukas-h/onepage/issues/1#issuecomment-289271306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJq5LpSlCbHOEKiNj-yiUmz6FJKJ_PMFks5rpj5RgaJpZM4MK7GR
.


fix safe-to-evict annotation

should be added in podTemplate instead of in DaemonSet

Makes perfect sense.  Thanks for your continued contributions!


[WIP] Cleanup

This is the definitive PR to fix the issues described in ControlDeAgua/bug_tracker#5. It is also the replacement to DiddiLeija/ControlDeAgua#1 (which was a PR oriented in the wrong way).
Closes ControlDeAgua/bug_tracker#5

My goal is to add black. It is a widely-accepted formatter.
Also, I think I made something wrong when setting up Python 3.10 in the CI.

Well, maybe it's time to inject Nox into this... :]

Actually... we could replace development.py with a Nox session, which is pretty much cleaner!

Though I fixed a large amount of issues, seems like there's still a lot of work to do...

Oh! Something even worse!
ERROR: Cannot install -r requirements-dev.txt (line 4) and -r requirements.txt (line 4) because these package versions have conflicting dependencies.

The conflict is caused by:

    flake8 4.0.1 depends on importlib-metadata<4.3; python_version < "3.8"

    cx-freeze 6.10 depends on importlib-metadata>=4.8.3; python_version < "3.10"

To fix this you could try to:

1. loosen the range of package versions you've specified

2. remove package versions to allow pip attempt to solve the dependency conflict

Agh! Even cx_Freeze==6.9 is a failure!
The conflict is caused by:

    flake8 4.0.1 depends on importlib-metadata<4.3; python_version < "3.8"

    cx-freeze 6.10 depends on importlib-metadata>=4.8.3; python_version < "3.10"

    flake8 4.0.1 depends on importlib-metadata<4.3; python_version < "3.8"

    cx-freeze 6.9 depends on importlib-metadata>=4.3.1

I'm now focusing on "killing" the issue, somehow. I hope flake8 will support new versions of importlib_metadata in the future...

However, this will only happen in Python 3.7, so... we could make a hack there.

I know this still has failures. Anyway, the 3.7 thing is gone! (Or that's what I think).

I'm experimenting to see where is the right dep for Python >= 3.7. I have to finish this before going ahead :/

I made a big mistake while trying to fix the conflicts. I'll close this PR, delete my fork and try again.


Bad Hive string in LogonUI configuration

"HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Authentication\LogonUI"  should be "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Authentication\LogonUI"

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

ID: 8a45c77e-0b22-27ef-8b4e-17db0ea60ddf
Version Independent ID: 7068c1b3-aa7b-6629-4ff4-45f522297a86
Content: Set up a single-app kiosk on Windows 10/11 - Configure Windows
Content Source: windows/configuration/kiosk-single-app.md
Product: windows-client
Technology: itpro-configure
GitHub Login: @lizgt2000
Microsoft Alias: lizlong

Thanks @AlphaXD001 for catching that. I'll fix it now, this issue will close automatically when the pull request merges, but given the US holiday, it may be a few days before the change is live.


Extra API call to fetch additional currency 🇳🇴

I make an additional call to bitcoinaverage.com to fetch the BTCNOK conversion rate. I made it so its easy to extend with additional currencies. However we need to make an extra API for each of these extra currencies so it does not scale very well.
If you agree I could start a greater rewrite where we fetch all available currencies first to populate the list to select from, but only update our selected currency.
Sample responses:
https://apiv2.bitcoinaverage.com/indices/global/ticker/BTCUSD
Edit: Found a better API that lets us fetch everything in on gulp
https://apiv2.bitcoinaverage.com/indices/global/ticker/short

I'll reopen soon

https://apiv2.bitcoinaverage.com/indices/global/ticker/short?crypto=BTC
you can also use a filter to only load BTC prices

Ah cool! That simplifies things!
Have you used something like Promises in Zap before?
I would like to fire both API-calls and wait for both to finish before merging their results:
Promise
    .all(fetchFromBlockchainInfo(), fetchFromBitcoinAverage())
    .ok((one, two) -> merge(one, two))

Nope, haven't used any promises yet.
For security reasons I try to use as few external dependencies as possible. (To avoid stuff like this this)
Bond has some reactive magic to merge two streams, but in my experience that makes things much harder to reason about.
For something similar to your problem I have used a DispatchGroup before (HistoryService line 67). That's also pretty simple.
Idk. If it is possible to come up with a short promise class without adding an external dependency that would be fine with me. Otherwise i'd just use Dispatch Groups.

Beautiful. 😍


No connection on windows VM & associated virt-launcher pod

What happened:
Followed this guide step by step, I managed to get the VM up and running. After setting it up and connecting to it with VNC I notice that though it doesn't have internet connection. When I go to the associated virt-launcher pod's compute container, I can't curl either.
What you expected to happen:
Windows vm should have internet.
How to reproduce it (as minimally and precisely as possible):
Followed the guide ( https://kubevirt.io/2020/KubeVirt-installing_Microsoft_Windows_from_an_iso.html ) on my minikube setup on WSL2, and upon completing it pod & vm had no connection.
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version: v0.57.1
Kubernetes version (use kubectl version): v1.24.0
VM or VMI specifications: VM is 2012 R2 windows
OS (e.g. from /etc/os-release): Windows WSL2
Kernel (e.g. uname -a): N/A
Install tools: N/A
Others: N/A

/triage needs-information
Could you post your vm yaml here so that we know what you have configured?

Thanks for replying. This was due to the missing Network driver installation, I have blindly followed the guide in https://kubevirt.io/2020/KubeVirt-installing_Microsoft_Windows_from_an_iso.html and because it doesn't install the network driver there was no connection: NetKVM\2k12R2\amd64. Also moving to an actual cloud vendor helped, trying to make this work in a minikube environment is painful.


404s in lighthouse audits

In the lighthouse audits listed below, the following URLs are returning non-200 results. Would you please take a look?
Lighthouse Audit Pages:

/web/tools/lighthouse/audits/aria-allowed-attributes
/web/tools/lighthouse/audits/valid-aria-attributes
/web/tools/lighthouse/audits/valid-aria-values
/web/tools/lighthouse/audits/required-aria-attributes
/web/tools/lighthouse/audits/offscreen-images

URLs that are 404'ing:

https://www.w3.org/TR/wai-aria/roles#role_definitions
https://www.w3.org/TR/wai-aria/roles#role_definitions
https://www.w3.org/TR/wai-aria/states_and_properties#state_prop_def
https://www.w3.org/TR/wai-aria/roles#role_definitions
https://github.com/WICG/IntersectionObserver/tree/gh-pages/polyfill

I ended up redirecting those a11y docs to the aXe docs because this is what the Lighthouse report UI does.


Use NamingThreadPool for parallel() statement

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@olivergondza - tests failed in the code path you changed.

@dnozay yes, it broke with code dependency bump. For now I failed to fix or even understand what ConcurrencyTest.testConcurrencyWithNonConcurrentDownstream does.

I am proposing an alternative that does not require core version bump...

thanks!


Only check type resolutions for cycles where every callable is type parameterized

It would be more efficient to just store whether a node in the call graph has type parameters along with the node.

/azp run

This seems like a good improvement for efficiency. Could we wait to merge this in until I can chat with you all tomorrow, and get properly caught up?

Sure thing! Disabled the auto merge so we can sync up tomorrow.

@swernli and I had a chat, and I'm all caught up. This looks ready to merge in. Thanks!


Please re-instate release creation

Hi when you release code to pypi, can you please create a numbered release in github? It would be great if the release in pypi could be tracked to a numbered release in git so that we can quickly check release notes.
Thanks.

Got it, thanks for the heads up @mckenzig I will do so from now on

Hi @codelucas. Great project!
Could you please keep github releases in sync with pypi.
Thank you!


cwd Doesn't seem to work (at least on Linux)

I tried:
println(Command("pwd").cwd("/tmp").spawn().waitWithOutput())

And didn't get /tmp but instead just my default cwd.

Fixed and will be released later

1.2.0 will released

Thank you! Verified that it works :)


Standardize the naming convention of base64-encoded binary data.

I am working on populating our internal (prod) authentication object onto wire in gRPC.  I am going to use Credentials, and because the authentication object is serialized as binary headers, I will base64-encode them.
The Credentials object will be passed to gRPC ClientAuthInterceptor and populated to gRPC Metadata, which accepts binary value (byte[]) when the key ends with "-bin", and string value otherwise.  Therefore, ClientAuthInterceptor needs to decode the base64-encoded binary headers returned by Crendentials, and pass through the others.
ClientAuthInterceptor could use "-bin" as the indicator. However, because it consumes any Crendentials implementation, it's not safe for it to rely on this indicator unless it's defined on the Credentials interface.
This is a documentation-only no-op change.  I have talked with @ejona86, and he is OK with this change.

@anthmgoogle, how do you feel about this?

I think at minimum the comment needs more clarify, and potentially belongs in a different location.

I have updated the PR per comment. PTAL.

Nit with the grammar, but other than that LGTM.


stdlib.h missing on build

I think its a known issue of the "#include_next <stdlib.h> : No such file or Directory"
Can anyone point me out to the fix cause I'm running to that here please
In file included from /usr/include/c++/8/ext/string_conversions.h:41,                  from /usr/include/c++/8/bits/basic_string.h:6400,                  from /usr/include/c++/8/string:52,                  from /usr/include/c++/8/bits/locale_classes.h:40,                  from /usr/include/c++/8/bits/ios_base.h:41,                  from /usr/include/c++/8/ios:42,                  from /usr/include/c++/8/istream:38,                  from /usr/include/c++/8/sstream:38,                  from /root/Downloads/openvdb/openvdb/Exceptions.h:9,                  from /root/Downloads/openvdb/openvdb/Grid.h:7,                  from /root/Downloads/openvdb/openvdb/Grid.cc:4: /usr/include/c++/8/cstdlib:75:15: fatal error: stdlib.h: No such file or directory  #include_next <stdlib.h>                ^~~~~~~~~~ compilation terminated.
This is the error i am getting.

I believe this is related to this issue: https://github.com/AcademySoftwareFoundation/openvdb/issues/144
They suggest building with -DCMAKE_NO_SYSTEM_FROM_IMPORTED:BOOL=TRUE
Let me know if that works

Ah yes thank you brother!
make function works now

I'm getting a CMake Error
Says that it can't find "FindIlmBase.cmake" in CMAKE_MODULE_PATG

The build and install was successful but when I try to include openvdb.h with #include <openvdb/openvdb.h> I get a big error.
/usr/bin/ld: /tmp/ccQFMAhD.o: in function openvdb::v7_1::math::AffineMap::preRotate(double, openvdb::v7_1::math::Axis) const':
test.cpp:(.text._ZNK7openvdb4v7_14math9AffineMap9preRotateEdNS1_4AxisE[_ZNK7openvdb4v7_14math9AffineMap9preRotateEdNS1_4AxisE]+0x7a): undefined reference to openvdb::v7_1::math::simplify(std::shared_ptr<openvdb::v7_1::math::AffineMap>)' /usr/bin/ld: /tmp/ccQFMAhD.o: in function openvdb::v7_1::math::AffineMap::preShear(double, openvdb::v7_1::math::Axis, openvdb::v7_1::math::Axis) const':
test.cpp:(.text.ZNK7openvdb4v7_14math9AffineMap8preShearEdNS1_4AxisES3[ZNK7openvdb4v7_14math9AffineMap8preShearEdNS1_4AxisES3]+0x80): undefined reference to openvdb::v7_1::math::simplify(std::shared_ptr<openvdb::v7_1::math::AffineMap>)' /usr/bin/ld: /tmp/ccQFMAhD.o: in function openvdb::v7_1::math::AffineMap::postRotate(double, openvdb::v7_1::math::Axis) const':
test.cpp:(.text._ZNK7openvdb4v7_14math9AffineMap10postRotateEdNS1_4AxisE[_ZNK7openvdb4v7_14math9AffineMap10postRotateEdNS1_4AxisE]+0x7a): undefined reference to openvdb::v7_1::math::simplify(std::shared_ptr<openvdb::v7_1::math::AffineMap>)' /usr/bin/ld: /tmp/ccQFMAhD.o: in function openvdb::v7_1::math::AffineMap::postShear(double, openvdb::v7_1::math::Axis, openvdb::v7_1::math::Axis) const':
test.cpp:(.text.ZNK7openvdb4v7_14math9AffineMap9postShearEdNS1_4AxisES3[ZNK7openvdb4v7_14math9AffineMap9postShearEdNS1_4AxisES3]+0x80): undefined reference to openvdb::v7_1::math::simplify(std::shared_ptr<openvdb::v7_1::math::AffineMap>)' /usr/bin/ld: /tmp/ccQFMAhD.o: in function openvdb::v7_1::math::ScaleMap::preRotate(double, openvdb::v7_1::math::Axis) const':
test.cpp:(.text._ZNK7openvdb4v7_14math8ScaleMap9preRotateEdNS1_4AxisE[_ZNK7openvdb4v7_14math8ScaleMap9preRotateEdNS1_4AxisE]+0x7a): undefined reference to openvdb::v7_1::math::simplify(std::shared_ptr<openvdb::v7_1::math::AffineMap>)' /usr/bin/ld: /tmp/ccQFMAhD.o:test.cpp:(.text._ZNK7openvdb4v7_14math8ScaleMap8preShearEdNS1_4AxisES3_[_ZNK7openvdb4v7_14math8ScaleMap8preShearEdNS1_4AxisES3_]+0x80): more undefined references to openvdb::v7_1::math::simplify(std::shared_ptropenvdb::v7_1::math::AffineMap)' follow
/usr/bin/ld: /tmp/ccQFMAhD.o: in function tbb::interface7::task_arena::current_thread_index()': test.cpp:(.text._ZN3tbb10interface710task_arena20current_thread_indexEv[_ZN3tbb10interface710task_arena20current_thread_indexEv]+0x5): undefined reference to tbb::interface7::internal::task_arena_base::internal_current_slot()'
collect2: error: ld returned 1 exit status

How are you building against VDB? If your project uses CMake, see the following:
https://www.openvdb.org/documentation/doxygen/build.html#buildUsingOpenVDB

Closing due to inactivity - please feel free to re-open or create a new issue if you're still encountering problems


Custom prompts for chain types that aren't "stuff" in RetrievalQA

Feature request
Consider the following example:
# All the dependencies being used
import openai
import os
from dotenv import load_dotenv

from langchain.chains import RetrievalQA
from langchain.document_loaders import TextLoader
from langchain.embeddings.openai import OpenAIEmbeddings
from langchain.chat_models import ChatOpenAI
from langchain.text_splitter import RecursiveCharacterTextSplitter
from langchain.vectorstores import Chroma
from langchain.prompts import PromptTemplate

load_dotenv()

openai.organization = os.getenv("OPENAI_ORG_ID_")
openai.api_key = os.getenv("OPENAI_API_KEY")

# Load up a text file
loader = TextLoader("foo.txt")
documents = loader.load()

# Split text into chunks
text_splitter = RecursiveCharacterTextSplitter(
    chunk_size=1000, chunk_overlap=0)
texts = text_splitter.split_documents(documents)

# Set up chroma
embeddings = OpenAIEmbeddings()
docsearch = Chroma.from_documents(texts, embeddings)

# I want a custom prompt that asks for the output in JSON format
prompt_template = """
Use the following pieces of context to answer the question at the end.
If you don't know the answer, output 'N/A', don't try to
make up an answer.

{context}

Question: {question}
Answer in JSON format:
"""

PROMPT = PromptTemplate(
    template=prompt_template, input_variables=["context", "question"]
)

llm = ChatOpenAI(model_name="gpt-3.5", temperature=0)

# This is what's done in the Python docs
chain_type_kwargs = {'prompt': PROMPT}
qa = RetrievalQA.from_chain_type(llm=llm, chain_type="stuff", retriever=docsearch.as_retriever(), chain_type_kwargs=chain_type_kwargs)

query = "Foo bar"
res = qa.run(query)

If we use anything other than "stuff" for the chain_type parameter in RetrievalQA.from_chain_type, we'll get the following error from that line:
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/langchain/chains/retrieval_qa/base.py", line 91, in from_chain_type
    combine_documents_chain = load_qa_chain(
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/langchain/chains/question_answering/__init__.py", line 238, in load_qa_chain
    return loader_mapping[chain_type](
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/langchain/chains/question_answering/__init__.py", line 196, in _load_refine_chain
    return RefineDocumentsChain(
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/langchain/load/serializable.py", line 61, in __init__
    super().__init__(**kwargs)
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for RefineDocumentsChain
prompt
  extra fields not permitted (type=value_error.extra)

Is there anything in particular that prevents custom prompts being used for different chain types? Am I missing something? Open to any help and/or guidance.
Motivation
I'm trying to perform QA on a large block of text and so using map_reduce or refine is preferable over stuff. I also want to perform the QA with a custom prompt as I need the chain's output to be in JSON format for parsing. When using stuff for text that doesn't surpass the token limit, it works as expected.
Your contribution
Happy to contribute via a PR if someone identifies that what I'm suggesting isn't impossible.

I’ve encountered the same problem.
This is my PR #6912
You can use the method in my description of PR to solve first.

were you able to get this working? This would be useful as a feature

were you able to get this working? This would be useful as a feature

+1

Hi everyone, unfortunately got caught up with work these past couple of weeks so didn't get to get very far with this. Curious to see if anyone else gave it a try

Here you get to read Langchain code, to figure out different keyworks used in different prompt templates in different chains. For example, in Refine chain, the input variables are question_prompt and refine_prompt. Good luck.


[evm] What about having 2 DenomUnits for UX?

Summary
Code4rena:
What about having 2 DenomUnits:

The smallest possible unit with Exponent = 0, universally called wei (i.e. Denom = "wei") for all such erc20 funtokens
The regular unit which uses the erc20 token's decimals, i.e. { Denom: bankDenom, Exponent: info.Decimals }

This first wei unit will be used as the coin's Base and the second unit as Display.
Originally posted by @berndartmueller in https://github.com/code-423n4/2024-10-nibiru-zenith/pull/21#discussion_r1794865216
@Unique-Divine
Yeah, I think that would improve the UX. That would bring about parity between MetaMask and Keplr/Leap.
The 0 exponent one's required and technically correct, but putting a second denom unit like you've described would make the behavior similar to NIBI's bank metadata:
    {
      "description": "The native staking token of the Nibiru network.",
      "denom_units": [
        {
          "denom": "unibi",
          "exponent": 0,
          "aliases": [
            "micronibi"
          ]
        },
        {
          "denom": "NIBI",
          "exponent": 6,
          "aliases": []
        }
      ],
      "base": "unibi",
      "display": "NIBI",
      "name": "NIBI",
      "symbol": "NIBI",
      "uri": "",
      "uri_hash": ""
    }

Integrating a change shortly.

Closed by https://github.com/NibiruChain/nibiru/pull/2074


Enable Labs to automatically upload corona test results

At the moment corona tests need to be scanned in by the person who was tested. This does not happen all the time - it would be desirable if the upload of the test result happened automatically after the test result is available. This would also speed up the detection of infection chains.
To enable labs to link the test with an instance of the corona app each test should have a unique  QR code which is (voluntarily) scanned by the app owner prior to submission of the sample to the lab. This way the lab can upload the test directly (instead of waiting for the test results being sent to the person who installed the app and this person needing to scan the QR code of the test).
If the default request is to please scan the QR code of the test prior to submission much more people will do it! This is the power of defaults! https://www.ted.com/talks/dan_ariely_are_we_in_control_of_our_own_decisions/up-next

The test result is already uploaded automatically by the lab when the result is available. The user can scan the QR code to access this saved result. If the test result is not available the user needs to wait, but the app will try to fetch the result every 2hours in the background.

To my understanding the test result cannot be used to identify infection chains before the user scans some additional QR code after the test result is available. Am I right here?
In the case that I am right, I propose that the QR code is scanned prior to testing instead of after the result of the test is available. This would speed up the detection of infection chains.

To identify infection chain we need the diagnoses keys from the user device. The user needs to share this manually because Apple and Google will show a popup to the user before we can access the diagnoses keys. This is not related to the QR Code or test result

So I suggest to share the "diagnoses keys from the user device" of the past with the corona test which is being sent to the lab and not wait with the sharing till the test result is available.

@ggjj11 actually, this is not what I want.
It means that I share all my contacts, just because I take a test, even if it turns out negative later.
This is worse than a centralized system, it violates the need-to-know principle and "Daten-Sparsamkeit".
cwa was intentionally designed as decentral system.
And even in a centalized system (which we don't have), it would be the health authorities (not the labs) that would be eligible to access the "social graph" of all contacts of all participating users.

Of course this sharing of user device data prior to test submission should be done voluntarily, but it should be the default with the ability to opt-out.

@ggjj11 such a feature has a very high potential to knock number of users down

https://www.atse.org.au/news-and-events/article/why-we-should-consider-making-the-coronavirus-tracking-app-opt-out/

@ggjj11 Thanks for your link.
Actually, the linked article proposes something very different. They propose to make usage of the app as default with opt-out on all supporting smartphones.
And also this I don't want.
If I look at the surveys about usage of the German app, I see not one peak of "undecided / don't-care" people, but clearly two peaks; people who want to use it, and people who don't want to. And IMO the adoption numbers in Germany are not so bad.
By the way, do you have any numbers that back-up your claim (e.g. that significant number of infected users are currently not sending their keys but wouldn't mind to do so)? Or is it just 'gut-feeling'?
I would love to see more numbers ...

Exact statistics are not available. These are some numbers (see also the links) which indicates that around 60% of corona positive people share their data. https://www.google.com/amp/s/www.connect.de/news/amp/corona-warn-app-gemeldete-faelle-infizierte-positiv-zahlen-schaetzung-3200916.html
Please, please if you have more accurate data share with us!

@ggjj11 Thanks for the links and the numbers.

only around 60% of corona positive people share their data

This number needs more explanation. What exactly is 100% here?
(And this is clearly not your fault, just to avoid any misunderstandings - I'll open a new question for it).

@ndegendogo

What exactly is 100% here?

The explanation on https://www.coronawarn.app/assets/documents/2020-10-23-cwa-facts-and-figures.pdf is:
"Erläuterung: Im Zeitraum vom 1. September bis 21. Oktober 2020
wurden insgesamt 25700 positive Testergebnisse verifiziert - via QR-Code oder teleTAN - dies ist die Anzahl der potenziell teilbaren positiven Ergebnisse. Anschließend haben sich 15453, d.h. 60 Prozent der Nutzerinnen und Nutzer dafür entschieden, ihr positives Testergebnis mit den anderen Nutzerinnen und Nutzern zu teilen. Nur so kann eine Risiko-Benachrichtigung der Nutzerinnen und Nutzer erfolgen, mit denen die Corona-positive Person epidemiologisch relevante Begegnungen hatte."
I think that is what you are looking for.


Drag and drop in explorer has visual artifacts on Linux

VSCode Version: Code - Insiders 1.6.0-insider (782e3ad7e504c308a5f1592ca6ae2574a13b0e73, 2016-10-05T16:59:53.654Z)
OS Version: Linux x64 4.4.0-38-generic
Extensions: EditorConfig.EditorConfig-0.3.3, Tyriar.lorem-ipsum-1.0.0, Tyriar.sort-lines-1.1.0, Tyriar.terminal-tabs-0.0.1, Tyriar.theme-glacier-0.0.10, be5invis.vscode-icontheme-nomo-dark-1.1.4, cssho.vscode-svgviewer-1.2.0, dbaeumer.vscode-eslint-1.0.7, felipecaputo.git-project-manager-0.1.15, gerane.Theme-Wombat-0.0.2, ms-vscode.csharp-1.4.1, ms-vscode.typescript-javascript-grammar-0.0.3, rbbit.typescript-hero-0.8.0, rebornix.nova-0.0.5, ryu1kn.annotator-0.10.0

Maybe we can tweak this better? seems introduced from https://github.com/Microsoft/vscode/commit/e058bde3ea1632edc1424dfe406e29c673c4aff5

Would probably go away if border-radius: 10px; was removed, I'm guessing this is just a chromium on Linux issue.

This seems to be an upstream issue: https://bugs.chromium.org/p/chromium/issues/detail?id=550999

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
Thanks for your understanding. Happy coding!


Vickel: Watch microsyringepump.com

Vickel requests the watch of the watch_keyword microsyringepump\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

microsyringepump\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Add Credits button needs better spacing

Fixed in https://github.com/macterra/artx-market/commit/69ec9224b3012bf04e729491f7d30e9078124d3f

tested


[FIXED] Server panic when poll for Varz and others concurrently

Resolves #327

Coverage increased (+0.05%) to 90.255% when pulling e6039e0a8b4f2cf77df6e672ea75aa193d7e38eb on fix_concurrent_monitor_poll into 638e249d2a9b33294d292d8b273cd60546831569 on master.


typo fix

Thank you @markgeek, could you see if this typo is present in the translations as well?

@doomspork  OK, let me check it later, and let you known the results

@doomspork I think I have fixed them all.

Thank you @markgeek 👍


import own package

Hello,
When I protoc a proto with multiple proto in the same package, the generated file import its own package. E.g.,
package ranking2

import proto "github.com/golang/protobuf/proto"
import fmt "fmt"
import math "math"
import _ "ranking2"
import ranking22 "ranking2"
import ranking23 "ranking2"
import ranking24 "ranking2"
import ranking25 "ranking2"
import ranking26 "ranking2"

https://github.com/golang/protobuf/blob/master/protoc-gen-go/generator/generator.go#L1328 seems to be for preventing this.
Is this a bug? Or how can I avoid this?

I found that this happens when I import other proto which defines MessageOptions.

Please provide more steps to reproduce this issue.  (Actual .proto files and the commands used to build them would be ideal.)

Sure.
Here it is: https://github.com/jhahn21/test
Output is: https://github.com/jhahn21/test/blob/master/aa/a.pb.go

Ok.  I think you're invoking it wrong?
You need to pass all of the .proto files for a given package to protoc at once:
protoc --go_out=. -I . aa/{a,b,o}.proto

not
protoc --go_out=. -I . aa/a.proto

If you do them one at a time, the protocol compiler assumes it's generating a separate package for each file.  (Not every build system for Go uses the same layout as the standard go tool, so protoc can't just assume that all of the files in a directory are intended to be in the same package.)

We use goprotowrap to automatically call protoc with one Go-package of files at a time.


Issue 4 console fix

PR for https://github.com/selvinortiz/craft-plugin-patrol/issues/4

Thanks for this PR @tmulry!

Thank @tmulry 👍


PHP: update examples

The PHP examples under the examples/php/echo directory had been heavily outdated.

should update to use cmake to build grpc_php_plugin
can remove an extra download of protoc
the build command for the node-server was a little outdated

php-update-example gh pr checkout 28159


Small Ci adjustments & cleanups

Minor CI cleanup prepping for landing of custom test images for windows in a subsequent PR.

suggesting we stop using explicit version for well-known actions, and use major versions instead
removing parameters that are actually default values

Not sure what is going on with #3719 (and variants).
Maybe these tests are somehow broken.
Will look into it as it becomes more aggravating.


reorganize BN-related "how it works" pages

Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[ ] Follows the developer docs style guide.
[ ] Follows the best practices and guidelines.
[ ] New documentation pages include document tags.
[ ] New documentation pages include SEO keywords.
[ ] New documents are in the .mdx file format to support the previous two components.
[ ] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[ ] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

@nathanosdev you might want to take a look at the webaccess/webserving part. This MR basically goes over this page https://internetcomputer.org/how-it-works/smart-contracts-serve-the-web/


Add config + documentation for apk signing

Closing this since I'm going to be looking into automating this build process this week. I'll definitely keep this around for reference though


[AutoRelease] t2-hybridaks-2022-09-08-03305(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3148
Live test success

issue link:https://github.com/Azure/sdk-release-request/issues/3148


Improve README.md

Look at (Kyle's) example https://github.com/CiscoDevNet/cognitive-intelligence-syslog-exporter

Re-written the README.md


Request to add Badge component

PropTypes draft as mentioned
const propTypes = {
	/**
	 * Text attribute for the badge container.
	 */
	badgeText: PropTypes.string,

	/**
	 * Class names to be applied to Badge component.
	 */
	className: PropTypes.string,

	/**
	 * Add icon to Badge component.
	 */
	icon: PropTypes.node,

	/**
	 * Specify icon position 'left' or 'right', by default it will be 'left'
	 */
	iconPosition: PropTypes.oneOf(['left', 'right']),

	/**
	 * Badge variants to apply relevant styling and icon rendering if applicable. Lighter and the Darker tones for badges. The default stays 'default'
	 */
	variant: PropTypes.oneOf(['default', 'inverse', 'lightest']),
};

I kickstarted working on this component

Hi @rakhi2104, I already had an open PR #1805 to add a Badge component.

Oh, I didn't see an issue, so I thought I'd open one. All the best.


Support plotting DHW and cyclone (mortality) scenarios

DHW and cyclone scenario plotting
Note: DHW loads the relevant trajectories dynamically, so if plotting after load_domain, it is necessary to load up the relevant DHWs manually:
dom = ADRIA.load_domain("<path to domain>")
dom2 = ADRIA.switch_RCPs!(dom, "26")

# Plot the 8th DHW scenario
ADRIA.viz.dhw_scenario(dom, 8)

# Plot the 8th cyclone mortality scenario
ADRIA.viz.cyclone_scenario(dom, 8)

Updated figure


Update to the new Windows API SDK

winsdk does not seem to be maintained anymore, please use the new alternative:
dependencies = [
    "winrt-windows-foundation-collections>=2.3.0",
    "winrt-windows-foundation>=2.3.0",
    "winrt-windows-globalization>=2.3.0",
    "winrt-windows-graphics-imaging>=2.3.0",
    "winrt-windows-media-ocr>=2.3.0",
    "winrt-windows-storage-streams>=2.3.0",
]

from winrt.windows.globalization import Language
from winrt.windows.graphics.imaging import BitmapPixelFormat, SoftwareBitmap
from winrt.windows.media.ocr import OcrEngine
from winrt.windows.storage.streams import DataWriter

@Sun-ZhenXing
Thank you! Would it be possible for you to make a PR?

Sure, please check https://github.com/GitHub30/winocr/pull/7

Thank you!


[ja] Localize loosely coupled architecture

Localize https://glossary.cncf.io/loosely-coupled-architecture/
Why loosely-coupled-architecture doesn't have "What it is", "Problem it addresses" and "How it helps"?

I would like to work on this issue if no one is working on.


The Divider widget has no width attribute。

The Divider widget has no width attribute。

Hi @gionchat
Can you elaborate what exactly you are trying to achieve?

This was also reported in February: https://github.com/flutter/flutter/issues/51009

Tanhks！


Added an idea for an "African Solution"

which is a simple solution which can be built
using scrap parts which are integrated into a
system to serve the purpose.

When I designed  the PCB  I used  Africa  in mind  for that reason  I used potentiometers that can be salvaged from FM radios  old style and a 2 layer PCB not 4 Layer that can be made by any maker in Africa.
If you send me your parts like switches I can adapt it for you  it is hard to know whats on your scrap pile. Note You Do need the TE sensor and a arduino . I only do electronic design motor and Mechanical is not my field.
I will supply files from my Github  not here  https://github.com/Arie001/ambovent_pcb_Rev_Arie
If you do not have Access to to arduino  Nano Uno mega  or at least some ATMEGA 324 or bigger part  Whatever   and a TE  Sensor I do not think I can help you.

@levush, I think this "African Solution" might be best on the AmboVent wiki, in part so it can be easily iterated and added to by anyone. I think the Ambovent wiki is public. Are you able to add a page and document it on the wiki here? https://github.com/AmboVent-1690-108/AmboVent/wiki


Add backend address pool associations to VM modules

Description
Add LB and AGW backend address pool associations to VM modules to support in-module handling of address pools.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings e.g. code analysis tooling or general use of the changed code

lgtm