Collection of beverages

Icon name
beverages
Use cases
We are a delivery system for Beverage Shops and are using lucide-react as our Icon Source.
We do like the simplicity of usage and amount of icons a lot, but we miss an Icon for a collection of beverages.
It would be very helpful to add one of this similar i provided as design idea. As there are no bottles in this style at all, maybe would be also great to add a single Bottle icon too. It would help a lot and i think it may be also a great icon for many Use-Cases
Design ideas

Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

Maybe something like this (based on milk)?

This would be great!

So whats the path forward here? I think the suggestion looks great, can we merge them?

@ericfennis:

Maybe something like this (based on milk)?

@ericfennis: which of these would you be okay with adding? (don't want to put more work in PRs than necessary :D ).

The next to last called "bottles" would be great

@ericfennis pls give feedback

@vjung28 @karsa-mistmere 4th one looks nice to me.

@vjung28 @karsa-mistmere 4th one looks nice to me.

What about their composite parts?

I think they are nicely designed. But adding all of them not sure if the use cases are that solid.
I like the 4th because it can also represent "cosmetics" or "chemicals".
We can definitely consider adding the others to the "lab" repo to research the use cases.


Update the readme file for new tool

The newly added sentence in the Japanese guide needs some translation. I will update it once I get the translation.

For this PR I will just make the changes directly in the PR code instead of making review comments to save some time.


chore(dependabot): add dependabot config

:tada: This PR is included in version 1.20.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


RUM-5530 Add the core-it module in the test pyramid

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.01%. Comparing base (b36d867) to head (dd9db58).
Report is 6 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2137      +/-   ##
===========================================
+ Coverage    70.00%   70.01%   +0.01%
===========================================
  Files          726      726
  Lines        26972    26950      -22
  Branches      4520     4516       -4
===========================================
- Hits         18881    18869      -12
+ Misses        6826     6816      -10
  Partials      1265     1265

see 30 files with indirect coverage changes


⚠ Bofrost.be has degraded performance

In 8d98ecd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1418 ms

Resolved: Bofrost.be performance has improved in f8b2bbe after 23 minutes.


Transient "node not ready" condition logged as an error

Hello
Heapster treats the "node not ready" condition as an error and produces an error-level log here:
https://github.com/kubernetes/heapster/blob/9686c3073660ab7d7757617d81dbca1e88378833/metrics/sources/kubelet/kubelet.go#L316
This this is a frequent transient condition and Kubernetes itself doesn't treat it as an error, I think that Heapster shouldn't do so either.
It's very helpful to reserve error logs for conditions which really do indicate errors an may require some kind of operator intervention. That makes implementing alerting for large distributed systems much easier.
Thanks
-Ben

Heapster is now deprecated, so no new features or sinks will be added.  Please see https://github.com/kubernetes/heapster/blob/master/docs/deprecation.md#initial-deprecation-kubernetes-111 for more information.


[Getting Started with CSS] Images didn't responsive in small screen please fix this issue

How can we reproduce this bug?

Step one: i think you need indicate a class (img-responsive) of every responsive images (specially full width images)
Step two: edit/add your foundation.css file line: 1115 (@media max-width: 596px) like below:
table.body img.img-responsive {
width: 100% !important;
height: auto !important; }
Step three: Then when anyone write their code where they used full width images/ or when they used max 270px images they should be used img-responsive class on their image

What did you expect to happen?
I expect when template go to small screen, images will be auto resize.
What happened instead?

Test case
hero.zip
Give us a link to a CodePen or JSFiddle that recreates the issue.
http://codepen.io/bulbul84/pen/zqwbYN

Right now, the images are width auto on small screens so they shrink with the container and do not stretch out (which would distort them). Thanks for the example! I didn't see an issue with how the images are handled. Can you provide a screenshot for which you would change the behavior?

Hi...
Thank you for your reply....
I face this problem on Firefox, Internet Explorer Browser....
In this browser, images didn't resize with screen width...
I attached on a screenshot with html file that problem i face...
please check it on your firefox and internet explorer browser....
hero.zip

if you see their happen above problem, please give me one way to solve this problem.
Regards
Bulbul Hasan

Closing as outdated. The issue might be because of some element which has width and height set or wrong code. Or in cases where the style tag is missing in the head or inlining was not done.


Screen displaying tables the user can click on.

The following commit takes care of this issue: https://github.com/jayzamazing/Restaurant-POS/commit/836dd888db7fb45c0f4218e2aa3b9a2989e62378


Multi select

Really nice progress! One thing I noticed is that you can click "between" two dropdown options and the dropdown will close. It should probably not be possible to click between two options at all. Do you see what I mean?


Exec tachikoma update<PHONE_NUMBER>2101

:hamster::hamster::hamster:   Strategy bundler
Powered by Bundle Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Changes Unknown when pulling beccad464d3938c66405dbcb4a6a4a55c7675fdd on pazjacket:tachikoma/update-20170802092101 into ** on Protom512:master**.


ls: compute the correct exit code

Turns out, ls's exit code is actually quite complex, and convincing clap to do what we need is not trivial.
#6162 grew too much, and in particular ls is a big hassle. So let's deal with ls in isolation first, and I'll rebase #6162 once this PR is merged.

Changes since last push:

Rebased on current main
Added a short explanation how each command.try_get_matches_from match-arm is reached; thanks @sylvestre!


Update how-to-determine-which-versions-are-installed.md

Title Update: How to determine which versions are installed
Summary
I made two changes:

Changed hklm: to HKLM:  - the latter is how Get-PsDrive displays the drive name.
Replaced aliases to full cmdlet names - aliases are great but can confuse. A best practice is to spell out cmdlet names in full.

Details
The registry drive names are all upper case by default. The documentation should reflect that.
Also, while using aliases is convenient at the console, we should not be using them here unless really necessary - and in this case, it isn't. In general, we should be showing whatever PowerShell shows (eg using Get-PSDrives or using tab completion and the inbox help.

Closing and reopening to begin new build.

Thanks @BillWagner.
  I'm going to close and reopen again because they should be fixed.

There were a few other suggestions regarding this PowerShell example in our comments section:
https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/how-to-determine-which-versions-are-installed#comments-container
@doctordns have you seen those?


Powerbase: Tshwane

text: Presumably, the same rulings should apply to both PB: Tshwane and The Shard, London
link: https://www.vekn.net/forum/rules-questions/80691-procurer-recruiting-another-with-the-shard?start=12#108765

But Tshwane is only pool reduction, so I don't see how this applies?

There is an argument that this particular ruling implies that you can use the Tshwane discount to play a card that you could not otherwise play (e.g. a spiteful Archon Investigation when you are at only 2 pool)
I do agree that the ruling is for a very narrow topic and maybe requesting for a ruling specific to pool may be more useful as a reference


(bot) Update Helm release postgres to v0.2.11

This PR contains the following updates:

Package
Update
Change

postgres
patch
0.2.9 -> 0.2.11

Configuration
📅 Schedule: 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.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (0.2.11). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Delete survey after 30 days when end date is reached

Not necessary to keep all surveys after they are finished.

Not considered


Token environment variable not picked up

When trying to deploy the exporter via docker compose, the token environmental token is not getting picked up.
time="2023-01-04T00:54:51Z" level=fatal msg="invalid config" error="No token configured"
  -c, --config string   Config file to use
  -l, --listen string   Address for the exporter to listen on (default "[::]:9806")

This is using the below compose file.
version: '3.7'
services:
  unms-exporter:
    image: quay.io/ffddorf/unms-exporter
    container_name: unms-exporter
    environment:
      - UNMS_EXPORTER_TOKEN="instance.unmsapp.com=token"
      - TZ=Australia/Melbourne
    ports:
      - 9806:9806
    restart: unless-stopped

I was able to get around this by passing through a config file instead. Slight gotcha that stopped me for a while is there needs to be no space between the -c or --config declaration and the file path. Suggest updating the Dockerfile to use an entrypoint rather than a command so end users can specify parameters by adding to the command.
Working docker compose file below
version: '3.7'
services:
  unms-exporter:
    image: quay.io/ffddorf/unms-exporter
    container_name: unms-exporter
    environment:
      - TZ=Australia/Melbourne
    volumes:
      - /config/prometheus/unms-exporter.yaml:/config/unms-exporter.yaml
    ports:
      - 9806:9806
    restart: unless-stopped
    entrypoint: ["/usr/local/bin/unms-exporter"]
    command:
      - '-c/config/unms-exporter.yaml'
      - ```

This is likely because the latest changes on main had not been part of a release, so they were not present on the latest tag in the registry.
I just released v0.2.0 which pushed a new image to the registry?
Can you try again with the environment variable?

One note about your trouble with the config file parameter: shell args needs to be provided as lists, so these variants should work:
    command:
      - "-c"
      - "/config/unms-exporter.yaml"

OR
    command: "-c /config/unms-exporter.yaml" # not using a list will use shell mode which allows seperating args with whitespace

closing for lack of activity

@mraerino sorry, missed the notification. Putting the command as a list worked for me, however using the environment var for the token instead of using the config file did not.

Environment VARS are now working as intended.


docs: Add documentation with example for LocalStack DocumentDB

Fixes #111
Adds a documentation section for LocalStack DocumentDB.

Hi @snhardin,
Thank you for your contribution! Just wanted to let you know I used what you've started as a basis for our dedicated service page and added some details for DocumentDB.
I will now close this PR.


Bug with and a question about the codex web demo

Thanks for all of your hard work 💪
http://alasql.org/codex/ noticed that the "SQL Data Model" panel is not updating.  Here's what I did:

http://alasql.org/codex/
click data types, then javascript data types then boolean
click the sql data model tab then click sql query tab
click create tables, then create table with columns
click sql data model

It is empty; expected to see

CREATE TABLE Person (
	Id SERIAL,
	Name STRING
);
INSERT INTO Person(Name) VALUES ("Vyacheslav"), ("Peter"), ("Semen");

Ok now the question: what is the purpose of the javascript tab?  None of the examples seem to use the javascript tab

Im not sure.
@agershun any input to this?


Modal component individually tested

Alternative implementation for the integration tests of the modal component (the original one to be found here: #22). Instead of testing the component within the styleguide page, it loads the component individually, without the code of all the other components. Further, it uses the styleguide examples to test against. The pattern should eventually be applied to all the components. This will make sure we can include components into projects individually, instead of all-or-nothing.

:thumbsup: LGTM!


Add prerequisites to "Prerequisites to access the Azure Serial Console" section

Hi team,
please add few points in "Prerequisites to access the Azure Serial Console" section:
serial console is not supported when storage account has firewall enabled
serial console is not supported when storage account has "Allow storage account key access" disabled
Thanks!
[Enter feedback here]

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

ID: 3a4dfc75-25a4-ae83-8d6a-00f267bf85b9
Version Independent ID: faa41c37-7b66-671b-09ca-3fb076f701e3
Content: Azure Serial Console - Virtual Machines
Content Source: support/azure/virtual-machines/serial-console-overview.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@lorenzomsft Thanks for the feedback. I will add this information into the article.

@lorenzomsf The article was updated. Thanks again for your contribution.


Migration status returns struct.

References point 8 of the #41 issue.

Looks pretty good - one small point about Locked before we can merge this.


Insert tags quantity

I'm wondering if you all would agree on inserting the tags quantity. Ideally in the basesection.py level, but for now at this level
@ka-sarthak @hampusnasstrom @budschi

The functionality for it is already implemented on the basesection level so the users can already add a tags quantity anywhere. But you mean that we would force it in on the SampleDeposition level?

what I mean is adding it in BaseSection together to name, lab_id, datetime, so it can be readily used


How to unset a value

I want remove a attribute from the model
var model = Model.forge({prop: ...})

// ...

model.unset('prop')

You may.

Thank you

No probs. I understand the docs leave a bit to be desired at the moment.


Unable to get authentication URL

Here is the log I am getting from the application on an Axis M2035-LE
2023-10-20T12:11:45.218-06:00 axis-b8a44f5f42e2 [ INFO    ] Tailscale_VPN[2166]: Starting Service
2023-10-20T12:11:45.222-06:00 axis-b8a44f5f42e2 [ INFO    ] Tailscale_VPN[2166]: Scroll to Bottom for link
2023-10-20T12:11:45.222-06:00 axis-b8a44f5f42e2 [ INFO    ] Tailscale_VPN[2166]: Service Started
2023-10-20T12:11:45.276-06:00 axis-b8a44f5f42e2 [ INFO    ] Tailscale_VPN[2170]: /usr/local/packages/Tailscale_VPN/lib/tailscaled: line 1: syntax error: unexpected word (expecting ")")
2023-10-20T12:11:45.276-06:00 axis-b8a44f5f42e2 [ INFO    ] Tailscale_VPN[2171]: /usr/local/packages/Tailscale_VPN/lib/tailscale: line 1: syntax error: unexpected word (expecting ")")

You are using the wrong EAP file for your camera, please unistall the app and install the correct one.
That should resolve the issue otherwise let me know.

You are correct, I thought it was correct because other ACAP apps fail to install when using the wrong architecture and this one installed without error. Thank you for the support, I will close this issue.


[Feature]: remember last used URL in import

Is your feature request related to a problem? Please describe.
I'm always frustrated when Insomnia does not remember my last used URL importing routes from a URL
Describe the solution you'd like
Insomnia should remember the last used URL when importing routes. It's unlikely that it'll change often.
Describe alternatives you've considered
None
Additional context

This seems like a useful feature! I suggest following an approach similar to this PR: https://github.com/Kong/insomnia/pull/2586, which does the same thing but persists the last used export format.

This seems like a useful feature! I suggest following an approach similar to this PR: https://github.com/Kong/insomnia/pull/2586, which does the same thing but persists the last used export format.

I'm gonna submit a PR on this if it's ok @develohpanda

I'm gonna submit a PR on this if it's ok @develohpanda


I need to create trusted pfx certificate which is trusted all windows users not only me.

I cannot find any instruction step by step how to get trusted certificate.

you have a few options here

Self signing- the docs for that can be found here- https://docs.microsoft.com/en-us/windows/msix/package/create-certificate-package-signing
There are third party certificate authorities where you can acquire a certificate


Incorrect AWS Region

I currently use aws cli using the AWS_PROFILE variable and I have a default region set for my profile to ca-central-1.
For some reason my secrets are saved to us-west-2.

Fixed in 491be74cc7fc70edbf4b57d3b08486e7de559bd8.


Fast git

When using many plugins, vim-plug spawn many git processes for them.

get revision
get branch
get remote.origin.url

This is too heavy. especially on Windows. This change get revision, branch,
remote origin url directly from .git directory.
This idea is borrowed from @k-takata's commit for minpac devel branch.
Before

After

[UserCustom DeviceProfile]
+CVars=r.UserQualitySetting=0
+CVars=r.MobileContentScaleFactor=0.8691
+CVars=r.MaterialQualityLevel=0
+CVars=r.ShadowQuality=0
+CVars=r.Shadow.MaxCSMResolution=4
+CVars=r.Shadow.CSM.MaxMobileCascades=0
+CVars=r.Shadow.DistanceScale=0
+CVars=r.Mobile.DynamicObjectShadow=0
+CVars=r.DepthOfFieldQuality=0
+CVars=foliage.LODDistanceScale=0.6
+CVars=foliage.MinLOD=0
+CVars=r.DetailMode=0
+CVars=r.MaxAnisotropy=3
+CVars=r.Streaming.PoolSize=200
+CVars=r.EmitterSpawnRateScale=0.5
+CVars=r.ParticleLODBias=0
+CVars=r.MobileNumDynamicPointLights=0
+CVars=r.MSAACount=2.0
+CVars=r.RefractionQuality=0
+CVars=r.StaticMeshLODDistanceScale=1.1946946686
+CVars=r.PUBGVersion=5
+CVars=r.UserHDRSetting=2
+CVars=r.MobileHDR=1
+CVars=r.PUBGLDR=1
+CVars=r.Mobile.SceneColorFormat=0
+CVars=r.BloomQuality=0
+CVars=r.LightShaftQuality=0
+CVars=r.Mobile.TonemapperFilm=1
+CVars=r.Mobile.AlwaysResolveDepth=0
+CVars=r.UserMSAASetting=0
+CVars=r.MobileMSAA=2.0
+CVars=r.DefaultFeature.AntiAliasing=0
+CVars=r.Mobile.EarlyZPass=1
+CVars=r.MobileSimpleShader=0
+CVars=r.PUBGDeviceFPSLow=120
+CVars=r.PUBGDeviceFPSHDR=120
+CVars=r.PUBGDeviceFPSMid=120
+CVars=r.PUBGDeviceFPSHigh=120
+CVars=r.OpenGL.StripExtensions=GL_ARM_shader_framebuffer_fetch_depth_stencil

As far as I can see the output of travis CI, it seems a problem of tester.

Oh, OK.

As far as I can see the output of travis CI, it seems a problem of tester.

There are failing test cases. Look for (X) in https://travis-ci.org/github/junegunn/vim-plug/jobs/720180357

@junegunn One of tests passed successfully. But others are timed-out. (over 10min)

Anyway, I just finished fixing problems now.

@mattn

But others are timed-out. (over 10min)

Tests are passing only with vim-nightly. Looks like your code uses features that are not available on older Vim versions. (For example, there is no trim() in Vim 7.4), and we are not planning to drop support for Vim 7.4.
Also, we don't use l: prefix in our code. Please remove them for consistency.

ok. will fix tonight

Fixed

Thanks, but as you can see, tests are not passing. You can run the tests locally with test/run script.
Test hangs here with Delete all directories? (y/N) prompt.

Can I make sure where test hangs from CI output?

Unfortunately, no. Vader prints the report only after the whole test is finished.
Looks like :PlugClean is broken. Does it work as expected on your setup?

Local test stop at same.

@junegunn Thank you.

Merged, thanks for your patience. I can definitely feel the difference in the overall performance even though I'm not on Windows.

Thank you


Launch new website for wizzy

[ ] Command descriptions
[ ] Atleast 3 examples

Work in progress - https://utkarshcmu.github.io/wizzy-site/home/


Eliminate possible SetIdleTimeout deadlock

Make setTimeoutChan buffered
Ensure that attempting to call SetIdleTimeout after the idleAwareFramer's monitor function has
stopped is a no-op instead of a deadlock

Fixes #63

@dmcgowan I put this together pretty quickly and have to run out for several hours, but I wanted to get this in front of you for review before I left. I'll check back for feedback later tonight. Thanks!

@dmcgowan ping, have you had a chance to look at this?
cc @smarterclayton @liggitt

LGTM, sorry it took a little longer busy with 1.10 and wanted to make sure I had time to give you a good review and test. The test does deadlock without the code which is refreshing to see something reproducible.


[Question] 3 different schemes and their API

Looking at Rust binding, I couldn't find dedicated APIs for "basic", "message augmentation", and "proof-of-possession" variants of the schemes.
More confusingly, in benches/bls_benches.rs here: one of the ciphersuite specified indicate usage of POP, but there's no code related to proof-of-possession to be found (maybe I miss something?)
Also for "basic" scheme, there's no option to check uniqueness of messages (I saw an TODO comment tho) either.
Could you please clarify how to use the current APIs for the 3 schemes in the standard?

See discussion in #93. "It might be helpful to view [blst] as something that works with EC points and byte sequences, rather than keys and signatures."


Add a custom header to load a mpd.

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
If the question is related to FairPlay, have you read the tutorial?

Yes
What version of Shaka Player are you using?
4.5.0
What browser and OS are you using?
Windows 10
Please ask your question
Hi I'm quite a noob so I ask you sorry in advance for my silly question.
I have made a local html page which loads shaka player and reproduce a mpd url with its clearkey and everything was working flawlessly until the mpd manifest is now accessible only with a custom header.
My question is how can I add a custom header in shaka player so that it can access and reproduce the manifest as before?
Here is my html code:
<html>
<head>
<!-- for non-UI builds: -->
<script src="https://ajax.googleapis.com/ajax/libs/shaka-player/4.5.0/shaka-player.compiled.js"></script>
<!-- or, for UI builds: -->
<script src="https://ajax.googleapis.com/ajax/libs/shaka-player/4.5.0/shaka-player.ui.js"></script>
<link rel="stylesheet" href="https://ajax.googleapis.com/ajax/libs/shaka-player/4.5.0/controls.css">
  <style>
    body {
        width:100%;
        height: 100%;
        font-family: Poppins;
        display:block;
        position: absolute;
        padding: 0px;
        margin: 0px;
        background: #000;
    }
    video{
        width:100%;
        height: 100%;
        display:inline-block;
        -webkit-user-select:none;
        -moz-user-select:none;
        -ms-user-select:none;
        user-select:none;
    }
 a, a:hover, a:focus, a:active {
      text-decoration: none;
 }
</style>
    <style>
        div a img{visibility:hidden;}
body::before {
        position: absolute;
        top: 5px;
        right: 5px;
        z-index: 10;
        content: '';
        height: 70px;
        width: 110px;
        background: url(' ') no-repeat;
        background-size: 110px auto, auto;
        opacity: 0.4;
    }
</style>
</head>
<body>
<center>   <div data-shaka-player-container style="max-width:40em"
         data-shaka-player-cast-receiver-id="1BA79154">
       <!-- The data-shaka-player tag will make the UI library use this video element.
            If no video is provided, the UI will automatically make one inside the container div. -->

        <video autoplay data-shaka-player id="video" poster="" style="width:100%;height:100%"></video>
    </div>
    </div></center>
<script>

const manifestUri =
    'https://example/example.mpd';

async function init() {
  // When using the UI, the player is made automatically by the UI object.
  const video = document.getElementById('video');
  const ui = video['ui'];
  const controls = ui.getControls();
  const player = controls.getPlayer();

player.configure({
  drm: {
    clearKeys: {
      // 'key-id-in-hex': 'key-in-hex',
      'kid: 'key',
    }
  }
});

  // Attach player and ui to the window to make it easy to access in the JS console.
  window.player = player;
  window.ui = ui;

  // Listen for error events.
  player.addEventListener('error', onPlayerErrorEvent);
  controls.addEventListener('error', onUIErrorEvent);

  // Try to load a manifest.
  // This is an asynchronous process.
  try {
    await player.load(manifestUri);
    // This runs if the asynchronous load is successful.
    console.log('The video has now been loaded!');
  } catch (error) {
    onPlayerError(error);
  }
}

function onPlayerErrorEvent(errorEvent) {
  // Extract the shaka.util.Error object from the event.
  onPlayerError(event.detail);
}

function onPlayerError(error) {
  // Handle player error
  console.error('Error code', error.code, 'object', error);
}

function onUIErrorEvent(errorEvent) {
  // Extract the shaka.util.Error object from the event.
  onPlayerError(event.detail);
}

function initFailed(errorEvent) {
  // Handle the failure to load; errorEvent.detail.reasonCode has a
  // shaka.ui.FailReasonCode describing why.
  console.error('Unable to load the UI library!');
}

// Listen to the custom shaka-ui-loaded event, to wait until the UI is loaded.
document.addEventListener('shaka-ui-loaded', init);
// Listen to the custom shaka-ui-load-failed event, in case Shaka Player fails
// to load (e.g. due to lack of browser support).
document.addEventListener('shaka-ui-load-failed', initFailed);
   </script>
</body>
</html>

I apologize if this has been clarified before but I'm really willing to learn and understand more about this subject.
Kind regards.

You can use something like:
player.getNetworkingEngine().registerRequestFilter(function (type, request) {
  if (type != shaka.net.NetworkingEngine.RequestType.MANIFEST) {
    return;
  }
  const myHeaders = {
    a: 'b',
  };
  for (const header in myHeaders) {
    request.headers[header] = myHeadersheader];
  }
});

Thanks for your answer, I have tried already in all the possible ways but I couldn't make it work, may you please be more specific on how to use the code or where to place it exactly? I'm still learning and maybe some things that for you are obvious for me are not.
What I'm sure is that the mpd url works with the correct header but I can't make it work in shaka.

Ok, I will make you a small, more extensive code that will be helpful.
shaka.polyfill.installAll();

const video = document.getElementById('video');
const player = new shaka.Player();
await player.attach(video);

player.getNetworkingEngine().registerRequestFilter(function (type, request) {
  if (type != shaka.net.NetworkingEngine.RequestType.MANIFEST) {
    return;
  }
  const myHeaders = {
    a: 'b',
  };
  for (const header in myHeaders) {
    request.headers[header] = myHeaders[header];
  }
});

await player.load(manifestUri);


Change Firefox support for WebAuthn to partial

It only supports FIDO2 PINs on Windows, and doesn't work on Mac/Linux/Android.

https://bugzilla.mozilla.org/show_bug.cgi?id=1530370
https://bugzilla.mozilla.org/show_bug.cgi?id=1530373
https://github.com/mozilla/authenticator-rs/issues/33

Attempting to register or authenticate with the webauthn.io demo will fail with “user verification” required, but succeed with it disabled.

Should the note perhaps be updated to be more descriptive?

👍🏻
I'll also update it to note that FF doesn't recognize TouchID as a provider

I added a a bit about OS support to the description in 759d32509bc499cac83c9e98281aeaade8e53522. Let me know if there was anything else you had in mind.

Firefox for Android supports biometric authentication, that note should only apply to desktop (macOS).

Thanks, and have updated per @leo60228's comment.


Modularize TM:PE such that individual features are available as workshop items

Players could choose between individual features by subscribing to the respective workshop items. As a side effect, this would minimize impact on performance since unused features would not need to run along with the game.

mmmmh ... not sure if that is a good idea ;D

moved


WIP: Deduplicate code for syncing configmaps and secrets

Currently, the code for syncing target configmaps and secrets is separate - even if the logic should be the same. This doesn't make it tempting to add new features to bundle targets. In this PR I made an attempt to duplicate some of the code. We could probably do more, but let me know what you think before I spend more time on this! ☺

/test all

/cc @inteon

Thanks @erikgb. I would like you to continue making improvements, let's do it in separate PRs.
/approve
/lgtm


Better control of texts

Hello.
I like this widget, but I am implementing it in a danish web application, so I didn't like the english texts that were hard-coded into the element. So I made a few simple changes to help me set the texts. Here are my changes:
<iron-icon icon="autorenew" title="{{retryText}}" on-click="_retryUpload" hidden$="{{!item.error}}"></iron-icon>
<iron-icon icon="cancel" title="{{removeText}}" on-click="_cancelUpload" hidden$="{{item.complete}}"></iron-icon>
<iron-icon icon="check-circle" title="{{successText}}" hidden$="{{!item.complete}}"></iron-icon>

These should be self-explanatory. Instead of the hard-coded texts, I put in some property names. Same goes for the error text:

{{errorText}}

And then, of course I also need to add the properties to the polymer object (with default values matching the hard-coded values so you still have these if you don't plot in your own):
  retryText: {
      type: String,
      value: 'Retry Upload'
  },
  removeText: {
      type: String,
      value: 'Remove'
  },
  successText: {
      type: String,
      value: 'Success'
  },
  errorText: {
      type: String,
      value: 'Error uploading file...'
  }

Another upside of this change, is that I can dynamically change the error message. Which means that by handling the on-error event, I can parse the response from server with a more specific error message and display it to the user.
And then a usage example:
<file-upload droppable drop-text="Or drag file here" retry-text="Upload Again" remove-text="Delete" success-text="Success!" error-text="{{uploadErrorText}}" on-error="fileUploadError" raised>Click to choose a file</file-upload>

And handling of the error, setting the error message to more specific error sent from server:
fileUploadError: function(error, obj) {
    var errObj = JSON.parse(obj.xhr.responseText);
    this.uploadErrorText = errObj.Message;
}

Great feedback! In the future if you could make a pull request that'd be awesome. I've gone ahead and implemented this and pushed a new version out through bower :)

I'm kinda new to the whole Git thing. But I'll try! (I gotta learn how to, anyway)


Transfer Legs and local transport

How would local transport with separate ticket (purchaseable locally) be modelled?
(e.g. Metro in Paris - Leg 1: Frankfurt-Paris, Leg 2: Metro from Gare de l'Est to Gare Montparnasse, Leg 3: Paris-Rennes)
(Note: when we send a Transfer Leg as part of the journey, we don't get an offer from Sqills S3)
I remember we previously had a flag somewhere in the offer response which said "ticket only available locally".

It seems, Sqills is not expecting transferLegs. If a transferLeg is part of a tripSpecification, no offers are returned.
One the hand, it is clear that a transfer is not priced. On the other hand, it is part of a trip.
That's why we have the following question:
How should we handle transferLegs in the tripSpecification for a offer request?
Should we include them, or not?

Add to LegAttribute structure documentation that the attributes are actually the "Service Attributes" from the Catalog of Code Lists. --> Ralf to provide patch. The Metro case would have the attribute "BOOKING_ON_LOCAL_STATION_ONLY".
So, in some cases this is part of the trip information.
On the other hand, in some instances it depends on who makes the offer, so one distributor may return an offer for all legs, where another distributor may return an offer for legs 1 and 3 (where 2 is the Metro leg).


Fix bug with chained dependencies

Previously, the depdendency chain
A <- B <- C

could only be specified by adding an additional
dependency from C to A. Otherwise, it would give you
A <-- C
B

in the dispatcher.
Fixes slide-rs/specs#253

Updated PR to also bump version.

@WaDelma Done, requesting approval / further suggestions.

Thanks for the reviews!
bors r+


Merging development branch (0.4.1)

This PR simply merges all changes accumulated over time back to master. See CHANGELOG for detailed information.

Codecov Report

Merging #104 into master will decrease coverage by 1.04%.
The diff coverage is 81.62%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
- Coverage   86.53%   85.49%   -1.05%
==========================================
  Files          28       29       +1
  Lines        3803     3929     +126
==========================================
+ Hits         3291     3359      +68
- Misses        512      570      +58

Impacted Files
Coverage Δ

pyasn1/error.py
100% <ø> (ø)
:arrow_up:

pyasn1/compat/binary.py
16.66% <ø> (ø)
:arrow_up:

pyasn1/compat/integer.py
94.11% <ø> (ø)
:arrow_up:

pyasn1/type/error.py
100% <ø> (ø)
:arrow_up:

pyasn1/codec/native/decoder.py
82.53% <ø> (ø)
:arrow_up:

pyasn1/codec/ber/eoo.py
100% <ø> (ø)
:arrow_up:

pyasn1/compat/dateandtime.py
77.77% <ø> (ø)
:arrow_up:

pyasn1/debug.py
93.5% <ø> (ø)
:arrow_up:

pyasn1/compat/string.py
36.36% <ø> (ø)
:arrow_up:

pyasn1/codec/der/decoder.py
94.73% <ø> (ø)
:arrow_up:

... and 25 more

Continue to review full report at Codecov.

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


Auto sign in

I'm a newbie in Ruby
I try using the issue 40 approach
and this is the code
##get tickets from cas server
ticket = RestClient("localhost:4000/api/users/tickets")
##auto sign in
RestClient.post("localhost:4000/login", {:username => "xxx", :password => "123123123", :lt => ticket, :rememberMe => "1"})

what should i store in cookies[:tgt]? (a login ticket ticket ? ex: CASino::LoginTicket.create.ticket)
or any approach to use api to get authorize from cas server?
Thanks!

The API has been removed in the 4.0 release. Check this documentation to see how to do this with the "Auth Token Login" mechanism.


STORM-803: Better CI logs

The CI failures were both caused by kafka, and are not related to this change at all.

LGTM on adding all test reports to print out issues!
Btw, it seems to be regression.
I've thought about logging to file, but if we can't access log file or can't print part of logs which points to failed build, it isn't informative anymore.
As we can see, printing "system-out" and "system-err" doesn't work at least Kafka tests.
Could we upload log file to somewhere?
Travis CI has a plugin which uploads artifact to S3, so we can use it.
Just I don't know about "free storage" and it should be accessed via http for easy seeing.

It doesn't mean that I don't agree only WARN log is enough.
I strongly agree we need INFO log to trace what happened.
For example, multilang tests with Travis CI may run fine but just makes some noise at cleanup.
With WARN logs we can't determine Topology runs properly cause we relied on 'topology.debug: true' and it prints out debug messages to INFO.

@HeartSaVioR I agree that some things may be lost when we write the logs out to a file.  That is why I added in the cat on failure when compiling, and the tail -500 on a test failure.  So if everything passed there is no issue, but if something failed we can get something out, even if it is not everything.  If you want to adjust the amount output on failure I am happy to.
I have also been looking at the clojure test runner, but I am really not ready to update it yet.  In most cases for me I have found this more useful at finding errors than the WARN has been, but that is just me.

Yes, right. We need INFO log since Storm debug messages are printed out INFO.
Maybe your approach requires Maven to fail fast (--ff), which I didn't used it yet.
You can see build 20.1 has storm-kafka test error, but it only prints out storm-hive logs.
Btw, whatever if it works like a charm, we may want to run tests continuously, cause we can't retry build for now.
Actually collaborators - committers can retry build at Travis CI UI page, but it checks permission on github repo. Is it a rule that only asfgit has write access to github repo.?

Btw, restricting log level to WARN level is only applied to storm-core now, since I didn't add logback-test.xml on any other modules. Other modules don't print out much so it can be fully printed without issues.

@HeartSaVioR I agree. I have shifted my focus to looking at how to improve the clojure maven plugin so we can capture the test results and display them properly.  Once my pull request there goes in I'll try to update this one to take advantage of some of the changes.  After that we should be able to get stdout/stderr only for tests that failed.

@revans2 I didn't think about it yet, but if it is possible it'll be great!

OK The updated test runner should be ready for review now.

@revans2
Far more better! Really amazing job, I appreciate it.
Btw, I have some points to ask / report.

I excluded storm-kafka tests from Travis CI build (STORM-864), and github diff shows that you revived storm-kafka.
I'd like to exclude it again, since tests in storm-kafka fails often, and it leads that we can't believe Travis CI result. (Travis CI notifies us your PR is broken but it's all about test failures of storm-kafka) Or do you have some ideas to fix?
drpc test report seems broken. There's some text between testcase tag.
https://travis-ci.org/apache/storm/jobs/67249769

175150 [main] INFO  b.s.testing - Shutting down in process zookeeper
175159 [main] INFO  b.s.testing - Done shutting down in process zookeeper
175160 [main] INFO  b.s.testing - Deleting temporary path /tmp/c4a2492e-6759-42c8-80bc-a48f42551e61
175161 [main] INFO  b.s.testing - Deleting temporary path /tmp/b9c776d0-5e9f-41b4-9f4e-dfd97669ae05
175161 [main] INFO  b.s.testing - Deleting temporary path /tmp/29bfa7dd-1a51-4561-8519-9a5355886ab6
175165 [main] INFO  b.s.testing - Deleting temporary path /tmp/1884de6c-49ae-4ca3-9b1f-0b7975420b56
            </system-out>
        </testcase>
175166 [Thread-851] INFO  b.s.util - Async loop interrupted!
        <testcase name="test-drpc-flow" classname="backtype.storm.drpc-test">
            <system-out>
175168 [main] INFO  b.s.u.Utils - Using defaults.yaml from resources
175178 [main] INFO  b.s.zookeeper - Starting inprocess zookeeper at port 2002 and dir /tmp/ebe0c08f-cf7b-47c6-9b24-123c16519c82
175181 [main] INFO  b.s.u.Utils - Using defaults.yaml from resources
175190 [main] INFO  b.s.d.nimbus - Starting Nimbus with conf {"topology.builtin.metrics.bucket.size.secs" 60, "nimbus.childopts" "-Xmx1024m", "ui.filter.params" nil,

If we can apply CDATA to report XML it would be more fine. Escaped characters disturbs readability slightly.

I like the CDATA and will look at switching over to that.

I updated the code to use CDATA, and it is a lot better now.  I also found some race conditions where log lines could leak into the XML file from background threads, which would cause the parsing of the file to fail.  That is fixed now, and should be ready for review.

Oh I forgot the kafka change.

Should be good now with all of the changes.

Great! +1.

+1


⚠ Processors has degraded performance

In 7e8529a, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 46 ms

Resolved: Processors performance has improved in bd059d6.


The item you requested is not available for purchase with reserved product id

My environment:
Android: 6.0.1
react-native: 0.50.1
react-native-billing: 2.10.0

I tried to use reserved product id to test the flow, but got the Error from Android The item you requested is not available for purchase. The IAP is opened, but failed when calling subscribe(productId). The exception received from the bridged module seems:
Error: Purchase or subscribe failed with error: 1

I made sure the android.test.purchased is used, and followed the docs to set the license key to null in MainApplication.js
        @Override
        protected List<ReactPackage> getPackages() {
            return Arrays.<~>asList(
                    ...
                    new InAppBillingBridgePackage(null),

            );
        }

Use new InAppBillingBridgePackage(null, this) as suggested would cause error as it does not match the constructor.
What else am I missing?  I tried to comment on #7 but it is locked, so have to create a new issue.
Thanks in advance!

I tried to test with real product id by uploading the signed APK to Alpha channel, it is working now.


V8: index.php/dashboard/users/search on mobile

design is broken:

This has to go hand in hand with https://github.com/concrete5/concrete5/issues/4057

Mobile is being re-thought.


Not able to generate PDF

[Enter feedback here]

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

ID: 782ca112-0ce4-83d9-019d-bf5cdc9e5eeb
Version Independent ID: 782ca112-0ce4-83d9-019d-bf5cdc9e5eeb
Content: Add copilot for app users in model-driven apps - Power Apps
Content Source: powerapps-docs/maker/model-driven-apps/add-ai-copilot.md
Service: powerapps
Sub-service: mda-maker
GitHub Login: @Mattp123
Microsoft Alias: mijosh

After refreshing several times also doc fails to build.

Thanks for your comment, @AhmedSyeda. This issue has been assigned to the doc author for review.

@AhmedSyeda - Are you trying to build a PDF from the Download PDF available from the Learn articles? I was able to successfully create the PDF via the site. If you're still experiencing the issue, reopen this issue or create a new one.

Still same behavior - download fails.

steps4PDFDNLD.zip

Hello Matt.
The issue is with our internal firewall setup. We figured out and fixed the issue. Could you close the ticket? since the issue is resolved now.
Just so you know, if any of you are facing the same issue check your AI extensions and modify access. This helped us.


Add integration tests to CI

This PR activates integration tests in the CI.

I wish there was a way to nest steps in Github workflows. I've looked online and there doesn't seem to be support for this in the syntax.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (180e1f8) 72.67% compared to head (185e8b5) 63.97%.
 Consider uploading reports for the commit 3059363 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #18513      +/-   ##
==========================================
- Coverage   72.67%   63.97%   -8.70%
==========================================
  Files         331      332       +1
  Lines       31978    31999      +21
  Branches     6243     6248       +5
==========================================
- Hits        23239    20472    -2767
- Misses       7210    10093    +2883
+ Partials     1529     1434      -95

Flag
Coverage Δ

keras
63.97% <ø> (-8.65%)
:arrow_down:

keras-jax
?

keras-numpy
?

keras-torch
63.97% <ø> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 65 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

All passing now!


ssanitycheck --coverage -p mps2_an385: Some remaing test cases that are still failing

Describe the bug
There are two cases that generate Hardware failure, and one that generates an Assertion Failure.
sanitycheck (without coverage) passed for all the test case noted here.
To Reproduce
./scripts/sanitycheck --coverage --coverage-platform mps2_an385 -p mps2_an385  -T tests/kernel -T tests/subsys/logging

Screenshots or console output
37/214 mps2_an385                tests/kernel/mem_protect/stackprot/kernel.memory_protection FAILED: timeout
see: sanity-out/mps2_an385/tests/kernel/mem_protect/stackprot/kernel.memory_protection/handler.log
***** Booting Zephy***** USAGE FAULT *****
  Illegal load of EXC_RETURN into PC
***** Hardware exception *****
Current thread ID = 0x200143c0
Faulting instruction address = 0x2001cef8
Fatal fault in essential thread! Spinning...

56/214 mps2_an385                tests/kernel/profiling/profiling_api/kernel.profiling FAILED: timeout
see: sanity-out/mps2_an385/tests/kernel/profiling/profiling_api/kernel.profiling/handler.log
***** BUS FAULT *****
  Precise data bus error
  BFAR Address: 0xaaaaaaaa
***** Hardware exception *****
Current thread ID = 0x20010228
Faulting instruction address = 0xa226
Fatal fault in thread 0x20010228! Aborting.
***** USAGE FAULT *****
  Illegal load of EXC_RETURN into PC
***** Hardware exception *****
Current thread ID = 0x2000b120
Faulting instruction address = 0x7095
Fatal fault in thread 0x2000b120! Aborting.
***** MPU FAULT *****
  Data Access Violation
  MMFAR Address: 0x78
***** Hardware exception *****
Current thread ID = 0x00000000
Faulting instruction address = 0x70c0
Fatal fault in ISR! Spinning...

40/214 mps2_an385                tests/kernel/mem_protect/userspace/kernel.memory_protection.userspace FAILED: timeout
see: sanity-out/mps2_an385/tests/kernel/mem_protect/userspace/kernel.memory_protection.userspace/handler.log
Please note that we are expecting the "BUS Fault" because the test case is expected to generate and test this fault. It is the "Assertion failed" that is not expected and the reason for listing the testcase as having a bug.
***** BUS FAULT *****
  Precise data bus error
  BFAR Address: 0xaaaaaaaa
***** Hardware exception *****
Current thread ID = 0x2001ff0c
Faulting instruction address = 0x11d02
Caught system error -- reason 0

    Assertion failed at /home/cinlyooi/tests/zephyrproject/zephyr/tests/kernel/mem_protect/userspace/src/main.c:107: z_SysFatalErrorHandler: (Re
ached unreachable code)
Unexpected fault during test
ASSERTION FAIL [(u32_t)thread->stack_obj == guard_start] @ /home/cinlyooi/tests/zephyrproject/zephyr/arch/arm/core/cortex_m/mpu/arm_core_mpu.c:2
23
	Guard start (0xffffffe0) not beginning at stack object (0x0)

Environment (please complete the following information):

OS: fedora29
Toolchain : zephyr
Commit SHA or Version used: 06ab394f05c0d7561a815cbaf4e7131fe91510f1

I think the BUS FAULT error as below has already been fixed by PR https://github.com/zephyrproject-rtos/zephyr/pull/16099.
***** BUS FAULT *****
Precise data bus error
BFAR Address: 0xaaaaaaaa


UX: armor module not documented on docs.rs

What were you trying to do
Upgrade a test application from rage 0.4 to rage 0.5
What happened
ascii armoring was removed from the default feature set and the armor module is not documented on docs.rs. It would be nice to have the docs built with that feature so that users don't have to run cargo doc locally, since I imagine pretty much every user of the crate needs ascii armoring.

I assumed that docs.rs rendered with doc_cfg by default, but it turns out I need to manually enable it. Thanks for noticing!


Windows taskbar appearing on second screen when hovering mouse over it when running not maximized application

Describe the bug
A clear and concise description of what the bug is.
When hovering mouse over windows taskbar, it pops out on second screen, when application like Vivaldi browser is running not maximized. When it's maximized - Windows taskbar is hidden.
Steps to reproduce

Connect second screen to PC
Run application like Vivaldi browser on second screen not maximized
Hover mouse pointer where windows taskbar is, when Cairo shell is not running

Expected behavior
A clear and concise description of what you expected to happen.
It should not pop out on mouse hover.
System information:

Windows version: Windows 10 1809
Cairo version: M3.7015
32-bit or 64-bit: 64 bit
In Cairo Settings > Advanced, is Cairo set as shell?: No

Additional context
Add any other context about the problem here.
Installed without administrator rights, using 7zip extract and running CairoDesktop.exe from extracted folder.

I can't seem to reproduce this on my machine using Vivaldi either. Do you have to do anything in the browser to make this occur?

After restarting Cairo shell it works normally, but I had it when I downlaoded newer build of CairoShell, exited from Cairo Shell, updated files and launched again, then I had Vivaldi on second screen.

I wonder if explorer got started after cairo exited? This would have made explorer shell for the rest of that session


Input typing on Android devices does not reflect actual behavior of Android keyboard

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
Using Android device keyboard, the input does not get keyPress event but some other event. On the other hand TestCafe fires the keyPress event.
What is the expected behavior?
Writing to input using TestCafe should fire the same set of events as the manual tests.
How would you reproduce the current behavior (if this is a bug)?
Use Android device and try to write something to native HTML input. Put the event handlers of keyPress or keyDown events on this input and watch the difference between manual test and TestCafe test.
Specify your

operating system: Android
testcafe version: 0.19.0
node.js version: 8.9.4

Hi, I was able to reproduce the described behavior so I've marked the issue as a bug.


Citation - Soft validation

When pages on citation is outside range of Source provide a soft validation warning.

This exists displayed under Taxon Name but needs to be displayed under Citations.  Might be done, just a view issue.

added soft validation to citation


Add support for text colors

Coverage decreased (-0.7%) to 62.179% when pulling cc361cef75f4baaa23fce251a411688214597958 on trollodel:master into f39a3934e4be271f2cc061e44bee34c6d6cb4126 on felipeochoa:master.

Coverage decreased (-0.7%) to 62.179% when pulling cc361cef75f4baaa23fce251a411688214597958 on trollodel:master into f39a3934e4be271f2cc061e44bee34c6d6cb4126 on felipeochoa:master.

@trollodel Why did you close this PR? Would've been interested in merging

@trollodel Why did you close this PR? Would've been interested in merging


Resource Estimator: add smallest number of qubits and corresponding runtime to the summary table

Given that we have not received user feedback requesting this feature, closing as not planned.


How do I get an IVoiceRegion?

I've seen the method "CreateGuildAsync" and It requires an IVoiceRegion, but when I try to create one or get one, it doesn't work because it's null and a type. Can anyone help?
    [Command("create-newdiscord")]
    public async Task DoSo(bool GiveInvite = true)
    {
        IVoiceRegion region = null;
        if (GiveInvite)
        {
            var newguild = await Context.Client.CreateGuildAsync("Guildman", region);

            var channel = await newguild.GetChannelsAsync();

            var invite = await channel.FirstOrDefault().CreateInviteAsync();

            await ReplyAsync("Your Invite: " + invite.Url);
        }
        else
        {
            var newguild = await Context.Client.CreateGuildAsync("Guildman", region);

            var channel = await newguild.GetChannelsAsync();

            await ReplyAsync("DOONE");
        }
    }

Make a class that derrives from IVoiceRegion and it should contain a string that has to do with the region name. Assign that to the voice region name you want to use.

You don't need to create a class that implements that interface, there's a collection of voice regions available on DiscordSocketClient.

Apparently that ^^

Thanks so much for the help, I have managed to successfully created a guild using my bot 😄


Drop EventHandler

Resolves #339, for discussion.

@natefaubion using DOM events brought more simplifications! No need for existentials with the handlers now (although, they probably weren't needed before either). It does coerce Events to their subtypes, but that's not really any different than what we had before. It could be done "properly" using Foreign, but as long as we haven't messed up anywhere with the HTML DSL this shouldn't cause any trouble.


Add --ip to kubectl proxy

Needed to allow binding to a non-localhost port.
Use Case: running kubernetes in a VM and running a proxy as an upstart service in the VM that binds to the exposed IP so that the kube-ui can be made available outside of the VM.

The kubernetes mechanism for doing this is to make a service (of type LoadBalancer or NodePort). Or you can use the apiserver proxy. Why do those solutions not work for your use case?

I did eventually get the kube-ui exposed correctly. The problem turned out to be having a dev version of docker 1.7 installed. Once I upgraded to the latest release version of docker 1.7, the dns was properly exposed.
This could still be useful, if you wanted to bind to something other than localhost, but it's not required for the use case I previously had.

Sorry, my last comment was written because I misread this as changing kube-proxy, not kubectl proxy, which is why it probably didn't make any sense at all. :)
I think we probably don't want kubectl proxy to serve on anything but localhost, because it removes auth and would leave your cluster wide open. @stephenR does that sound reasonable to you?

In our case, we were using it to make a background daemon running on a local VM (within a network space unaccessible outside of the local host) to deliberately host and expose the static ui files to the local host. While not exactly why we did it, this turned out to be pretty useful if you wanted to make changes to the ui and see changes in real time, without rebuilding and redeploying kube-ui as a pod.
I would agree that the kube-proxy shouldn't allow you to expose yourself to the world, but the kubectl proxy seems like it's mostly for development anyway. Is there some other primary use case I'm missing?
I won't push this too hard. I'm not developing the kube-ui right now. It just seemed potentially useful and worth sharing.

I think we probably don't want kubectl proxy to serve on anything but localhost, because it removes auth and would leave your cluster wide open. @stephenR does that sound reasonable to you?

Yeah, I think it's dangerous because it will expose the API without any authentication. But if there's a good use case for it, I think it could be fine if it's behind a flag and people are made aware of the risk.

I won't push this too hard. I'm not developing the kube-ui right now. It just seemed potentially useful and worth sharing.

I understand. Thanks for the PR; I think for the moment we will not include this, but definitely let us know if you think of another use case.


Pass real radius during ping/pong

What was wrong?
Fix #619
How was it fixed?
Rremove any duplicate values of data_radius and pass all reads through to storage. Then pings/pongs should use the real value from our data store.
To-Do

[ ] Clean up commit history

Someday, we'll have to deal with different radii per network, but for today, at least we'll stop sending the MAX radius in all PING/PONG messages.


Error: json.decoder.JSONDecodeError: Expecting value: line 2 column 9 (char 9)

Thanks for the Great Work!
I'm having trouble running

python ./usecases/console_chat.py

The error is
Traceback (most recent call last):
  File "console_chat.py", line 33, in <module>
    main()
  File "console_chat.py", line 13, in main
    claude = Client(cookie)
  File "/home/paul/anaconda3/envs/TA/lib/python3.8/site-packages/claude_api.py", line 13, in __init__
    self.organization_id = self.get_organization_id()
  File "/home/paul/anaconda3/envs/TA/lib/python3.8/site-packages/claude_api.py", line 32, in get_organization_id
    res = json.loads(response.text)
  File "/home/paul/anaconda3/envs/TA/lib/python3.8/json/__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "/home/paul/anaconda3/envs/TA/lib/python3.8/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/paul/anaconda3/envs/TA/lib/python3.8/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 2 column 9 (char 9)

I read all issues and readme.md carefully, but still can't find a solution.
I noticed that there are a few oddities when getting cookies:

First, I can't find the exact same response as your readme.me in the Network Tab, such as 'File: organizations, Type: html'. My My screenshot is as follows (and I click the 1st row to copy the cookies):

Second, my cookies contains 5 values, they are:

cf_clearance=***;
sessionKey=sk-ant-sid01-****;
intercom-sessionupk8zyo=RE***;
intercom-device-id-lupk8zyo=***;
 __cf_bm=****=

The number of values is different from that in your readme.md (i.e., 4).
I also attach the console_chat.py for reference here:
import os
from claude_api import Client

def get_cookie():
    # cookie = os.environ.get('cookie')
    cookie = 'cf_clearance=***; sessionKey=sk-ant-sid01-****; intercom-sessionupk8zyo=RE***; intercom-device-id-lupk8zyo=***;
__cf_bm=****='
    if not cookie:
        raise ValueError("Please set the 'cookie' environment variable.")
    return cookie

def main():
    cookie = get_cookie()
    claude = Client(cookie)
    conversation_id = None

    print("Welcome to Claude AI Chat!")

    while True:
        user_input = input("You: ")

        if user_input.lower() == 'exit':
            print("Thank you!")
            break

        if not conversation_id:
            conversation = claude.create_new_chat()
            conversation_id = conversation['uuid']

        response = claude.send_message(user_input, conversation_id)
        print("Chatbot:", response)

if __name__ == "__main__":
    main()

Looking forward to your reply！

It seems to be problem with cookie.. placement, Please the whole cookie ,check for docs or readme file... .you can Also check in storage tab for required cookies.


Reorder rgd_image URLs

Closes https://github.com/ResonantGeoData/ResonantGeoData/issues/644
URLs are routed greedily. For example, in the following:
router = SimpleRouter(trailing_slash=False)
router.register(r'api/a', SerializerA)
router.register(r'api/a/b', SerializerAB)

... a POST to /api/a/b would actually use SerializerA since the route is a full match.
This PR corrects the orders of URLs in the rgd_image application to prevent this issue.

Thank you @mcovalt!! Merging....


Bug: Route /threadsynchronizationJava Does Not Render Content

Kindly assign it to me

assigned you can fix this issue with #75 make one pr


Added two new features

Now when you scroll the page navbar remains attached to the top.
Also when you click on the Nav element the page will move smoothly.
Apart from these I had also corrected a spelling mistake.

Thanks so much for this. I'll check it out in a while when I have time.

Sure!!!

Seems ok.... thanks for your help.


custom Slience Management URL when proxy enabled

Hi, I have karma deployed behind a proxy from Rancher.
So the karma URL is something like https://example.com/k8s/clusters/project/api/v1/namespaces/monitoring/services/http:karma:80/proxy/?q=
When the proxy mode is enabled, silence management request will be sent to https://example.com/proxy/alertmanager,
instead of https://example.com/k8s/clusters/project/api/v1/namespaces/monitoring/services/http:karma:80/proxy/alertmanager
Both list alerts and list silence work find, but silence management requests are sent to an invalid URL.
I found the code that returning InternalURI wrote:
https://github.com/prymitive/karma/blob/main/internal/alertmanager/models.go#L183-L195
// InternalURI is the URI of this Alertmanager that will be used for all request made by the UI
func (am *Alertmanager) InternalURI() string {
	if am.ProxyRequests {
		sub := fmt.Sprintf("/proxy/alertmanager/%s", am.Name)
		if strings.HasPrefix(config.Config.Listen.Prefix, "/") {
			return path.Join(config.Config.Listen.Prefix, sub)
		}
		return path.Join("/"+config.Config.Listen.Prefix, sub)
	}

	// strip all user/pass information, fetch() doesn't support it anyway
	return uri.WithoutUserinfo(am.PublicURI())
}

It would be nice if InternalURI can be customized. And if no customized URL is specified, fallback to this code, for backward-compatible

#3063

Here is how karma run in my K8S:
      containers:
      - args:
        - --alertmanager.proxy
        - --alertmanager.uri
        - http://alertmanager-operated.cattle-prometheus.svc:9093
        - --alertmanager.name
        - xxx
        - --alertmanager.cors.credentials
        - same-origin
        - --alertmanager.internal_uri_prefix
        - /k8s/clusters/project/api/v1/namespaces/monitoring/services/http:karma:80/proxy

Silent Management works as expected.
The URL sent to from the browser looks like
https://example.com/k8s/clusters/{project}/api/v1/namespaces/monitoring/services/http:karma:80/proxy/proxy/alertmanager/{alertmanager_name}/api/v2/silence/5adbc660-0a66-42f6-b54c-7385ef3ea3e8

When the proxy mode is enabled, silence management request will be sent to https://example.com/proxy/alertmanager,
instead of https://example.com/k8s/clusters/project/api/v1/namespaces/monitoring/services/http:karma:80/proxy/alertmanager

That's expected, /proxy/$name endpoint handles those requests and all it needs is the alertmanager name. It's not a proxy in a sense that it will forward any request to any endpoint keeping paths and everything else as is.

If you want different URI to be used between karma->alertmanager and ui->alertmanager then you can set external_uri, there's a breakdown of all combinations of values on CONFIGURATION.md#alertmanagers

Can you share your config? After re-reading it seems that you're saying that listen.prefix doesn't work for proxied silences.

Thanks for the quick reply.
In my case, https://example.com is my Rancher's homepage. https://example.com/proxy/alertmanager points to nothing.
https://example.com/k8s/clusters/{project}/api/v1/... is the way to access Prometheus, Kiali, Jaeger in our cluster, here is the rancher's page:

Authentication is done by Rancher. And this is the only way we access prometheus and other 3rd UI (except kubectl port-forward)
Here is how our Jaeger UI pass request via Browser:
https://example.com/k8s/clusters/{project}/api/v1/namespaces/istio-system/services/http:tracing:80/proxy/jaeger/search?end=1620039159261000&limit=20&lookback=1h&maxDuration=10s&minDuration&service=istio-ingressgateway&start=1620035559261000&tags=%7B%221%22%3A%22true%22%7D
Nginx is integrated into rancher, request is pass to backend host after rewrite_url.
I've tried listen.prefix, external_uri and other kinds of combination, none of those worked:

if listen.prefix is set, Karma UI can not be accessed. Since all uri between example.com/' and .../proxy` is rewrote to before sent to Karma.
If external_url is used and set to sth like /k8s/cluster/..., then List Silence is done via the Internet instead of cluster internal access. (I've tried, there was another problem but I can remind);
If external_url is used and set to http://alertmanager-operated.cattle-prometheus.svc:9093 (cluster internal access), there would be CROS problem

My exec config is list above:
      containers:
      - args:
        - --alertmanager.proxy
        - --alertmanager.uri
        - http://alertmanager-operated.cattle-prometheus.svc:9093
        - --alertmanager.name
        - xxx
        - --alertmanager.cors.credentials
        - same-origin
        - --alertmanager.internal_uri_prefix
        - /k8s/clusters/project/api/v1/namespaces/monitoring/services/http:karma:80/proxy

And the Karma UI URL is:
https://example.com/k8s/clusters/{project}/api/v1/namespaces/monitoring/services/http:karma:80/proxy/?q=
As I mentioned above, if add internal_uri_prefix, the POST request is sent to
https://example.com/k8s/clusters/{project}/api/v1/namespaces/monitoring/services/http:karma:80/proxy/proxy/alertmanager/{alertmanager_name}/api/v2/silence/5adbc660-0a66-42f6-b54c-7385ef3ea3e8
I believed that what Karma received would be https://example.com/proxy/alertmanager/{alertmanager_name}/..., since the uri /k8s/... is rewrote/removed by rancher/Nginx
I am not an English speaker, Hope I explain well why I did this :(

Goodness me that's complicated.
Luckily everything except silence proxying works, which means there's one special case we need to convert to non-special and I'd very much rather do that, then add more knobs. Most of the UI doesn't know much about root URI and listen prefix as it's all relative to the URL, so we just need to simplify the code so proxy URIs are also relative and that will be it, no extra flag needed.

#3068 should fix this for you

Try :latest docker image or main branch build and see if it fixes your issue

v0.85 includes this change, can you confirm if it works for you?

It works perfectly! So simple!


File parsers

We need more parsers for various 3D geometry formats.  Eventually, it would also be nice to build some generic module that takes a simplicial complex and serializes it to various standard formats.

Ping @hughsk @mattdesl
Following yesterday's discussion about file parser on #stackgl, i'd like to have input from contributors.
I started building a 3DS file parser (https://github.com/thibauts/node-parse-3ds and https://gist.github.com/thibauts/b18eaeea43ba66143c12) and wonder what would be a common scheme to build such parsers, regarding module organisation and interfaces.
As @mikolalysenko pointed out it's probably a good idea to build parsers that stay true to the data they parse, ie that provide a strict JSON representation of the data.
@mattdesl suggested having a 2 layers model and provide modules that take the raw JSON data and extract relevant data in a common format that will make things easier and interoperable.
Now what would the module interfaces look like ? Could we agree on a module naming sheme to make things discoverable and clear ?
As an aside I'm not a big fan of parsers that take a stream as an input as they put a hard dependency on browserify for client side usage and are not really streaming parser in the sense that you have to wait for the stream end before getting the parsed data back anyway.
Maybe we could have Buffer / ArrayBuffer => JSON tree for raw parsers and Buffer / ArrayBuffer => vertices, faces, normals, uvs, etc for common usage parsers.
In case we need to build real streaming parsers, adding "stream" in the module name could to the trick like in parse-ply parse-ply-stream.
Having a naming scheme that can include serializers would be nice too.
What do you think ?


Calls to static methods in Java interfaces are prohibited in JVM target 1.6

Description
I'm trying to make a plugin which gives a player a permission. In the docs, it says I should use PermissionNode.builder("my.permission").build() but when I try to build my plugin, gradle says:
Calls to static methods in Java interfaces are prohibited in JVM target 1.6. Recompile with '-jvm-target 1.8'
I don't want to change a version or something I'm using
Reproduction steps

Make a plugin with Kotlin
try to set a permission / build a PermissionNode with `PermissionNode.builder([...]).build()

Expected behaviour
It works
Environment details

Server type/version: Spigot running version 1.15.2 build git-Spigot-a99063f-be6aaf0
LuckPerms version: v5.2.96

Any other relevant details

LuckPerms only supports Java 8 and above - you're trying to compile using Java 6.
If you're using Gradle, you need to define
sourceCompatibility = 1.8
targetCompatibility = 1.8

in your buildscript, or for Maven:
<properties>
  <maven.compiler.source>1.8</maven.compiler.source>
  <maven.compiler.target>1.8</maven.compiler.target>
</properties>

LuckPerms only supports Java 8 and above - you're trying to compile using Java 6.
If you're using Gradle, you need to define
sourceCompatibility = 1.8
targetCompatibility = 1.8

in your buildscript, or for Maven:
<properties>
  <maven.compiler.source>1.8</maven.compiler.source>
  <maven.compiler.target>1.8</maven.compiler.target>
</properties>

I did that and it still doesn't work

I did that and it still doesn't work

Please use StackOverflow to ask "how-to" and "why-didn't-it-work" questions.
Or GoogleSearch

Please use StackOverflow to ask "how-to" and "why-didn't-it-work" questions.
Or GoogleSearch

Ok I got it working, for kotlin, I have to do
compileKotlin {
    kotlinOptions.jvmTarget = "1.8"
}


Error when compiling egl renderer

Hi,
I get the following error when compiling the egl renderer:

Have you encountered this error message before?

I think we didn't encounter such an error before.
Are you compiling this in a docker container? If possible, could you please share the container? Or could you try compiling it directly in the host machine environment?


BT controllers stopped working in 1.6.1

Hello, first thanks for the continuous work!
I have installed blueretro in my Wii internally to the 4 gcn ports, the first firmware was 1.2.2 (gc spiffs) everything working fine, but i updated to 1.6.1 (gc spiffs) and as the title says all the BT controllers stopped working for some reason idk.
My question is, can i go back or downgrade to 1.2.2 without problems? If the answer is yes I'm gonna try other releases after that one

You can downgrade no problem,
Probably 1.6 broke it? Try 1.6 to confirm that it fail too, try 1.5.1 that one should be fine.

Ok thanks, here is my report
As you said 1.6 broke it and 1.5.1 worked fine

Ok so the fix for #264 likely at fault here

Not sure why but new FW push-pull output doesn't work when there is a small resistance in line ~100ohm
Double check your soldering of the data pin to make sure its not a cold joint. Maybe take a picture too so I can see.

Not sure why but new FW push-pull output doesn't work when there is a small resistance in line ~100ohm
Double check your soldering of the data pin to make sure its not a cold joint. Maybe take a picture too so I can see.

Okay I'm gonna take a pic, but (in wii) i remember that there are some components too far (at least 1cm) for a cold joint

From right to left: gnd, data1, Vin, data 2, data 3, data 4

v1.6.3 should fix this:
https://github.com/darthcloud/BlueRetro/releases/tag/v1.6.3


Add Ecmp Static Route for centralized gateway

For multiple centralized gateway node, we need to detect the node healthy status and update the routes for high available @hongzhen-ma


Bundle changes and restructure

This is a combination of the commits in existing PRs:

https://github.com/wolffiex/shellbot/pull/7
https://github.com/wolffiex/shellbot/pull/8
https://github.com/wolffiex/shellbot/pull/9
https://github.com/wolffiex/shellbot/pull/10 ← see this one for screenshots
https://github.com/wolffiex/shellbot/pull/11

Plus a few extra commits:

d98d049 chore: try GPT-4o
d95b524 refactor: move Neovim files into lua/ subdirectory
0682fbd feat: add syntax file
3201989 feat: add :ChatGPT command
a089bc6 docs: add instructions for installation
0809fdb feat: add an ftplugin file
a24dafd refactor: move settings from module into ftplugin
321ea3e refactor: replace deprecated nvim_buf_set_option() call
c934b21 refactor: move mappings into ftplugin

Basically, moved a bunch a stuff of around to make the repo follow the standard directory layout for Vim plugins, which means that we can ship a syntax file, set up the :ChatGPT command for users automatically etc, as discussed in #10. I normally wouldn't bundle all this stuff into a single PR, but GitHub doesn't support stacked PRs, and the file moves will otherwise be in conflict with the previous PRs. I'm going to close those ones.

Superseded by:

https://github.com/wolffiex/shellbot/pull/12

Whoops. Accidentally closed this right after opening it. Reopening.


Documentation request: How to integrate randomExample

It would be nice to have an example of how to use randomExample in connection with the rest of the library. It would also be nice to have a setter for example in jsonRequest and jsonResponse

Hey, if I'm not mistaken, the addition of random example compiler was quite ad-hoc as I needed it in the effect-http. It's possible I'll remove it from this package and either make it part of the effect-http or create a completely separate lib. AFAIK, there is no direct connection or intended interop between randomExample and OpenApi combinators. Sorry for the confusion.


Add Liquid super-filter

I write a lot of GitHub Pages websites. It has been wonderful to have the metadata view (though it needs some design work) on markdown files and the like. However, it would be very useful to have a general liquid lexer of sorts that would sit above another lexer (like json or markdown) in order to highlight the liquid bits.
I have found @tpope's vim-liquid especially useful, as it applies this principle to my vim syntax highlighting.
Thoughts on adding this?

Liquid can be used in any language and may not have the .liquid file extension. It may just be unconscionable.

I afraid we just don't have a good way to address this right now. :crying_cat_face:


youtube-dl does not work on wifi?

This is on a rooted Android 5.1 device with Termux, python 3.6.5 and youtube-dl installed via pip.
When I use youtube-dl with mobile data, it works pretty good. But instead if I use wifi, then it gets stuck after this step Downloading webpage, it takes about 5 minutes to print the next message, then again a few more minutes to the next message and so on.
The url I am trying is a vimeo playlist.
Apart from that I have also tried coursera-dl and edx-dl which IIRC uses youtube-dl internally.
The last message I received on -v before it became slow again was -
Checking Original video URL

Also this happens only on the video urls.
Pdf downloads work fine.
Is there anything that I can do to debug it more?
Or any general reason that could cause the issue?
I am able to browse internet on the wifi just fine.

Most likely unstable connection is the reason.

No that is not the problem, I can very well download other stuff using the browser or ADM.
It gets stuck at the exact same place always.

ERROR: Unable to download webpage: <urlopen error [Errno 113] No route to host> (caused by URLError(OSError(113, 'No route to host'),))
  File "/data/data/com.termux/files/usr/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 519, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/data/data/com.termux/files/usr/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 2203, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/data/data/com.termux/files/usr/lib/python3.6/urllib/request.py", line 526, in open
    response = self._open(req, data)
  File "/data/data/com.termux/files/usr/lib/python3.6/urllib/request.py", line 544, in _open
    '_open', req)
  File "/data/data/com.termux/files/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
    result = func(*args)
  File "/data/data/com.termux/files/usr/lib/python3.6/site-packages/youtube_dl/utils.py", line 1101, in https_open
    req, **kwargs)
  File "/data/data/com.termux/files/usr/lib/python3.6/urllib/request.py", line 1320, in do_open
    raise URLError(err)

After letting the program run for about 10-15 minutes I got this error.

But I got this error only once, i.e. right now. Whereas I am trying to use this on wifi since 2-3 days.

It starts to work if i do --socket-timeout 5
:/

Definitely not a problem with the network as I am able to access the same url from the browser and download the video using IDM

It starts working on vpn. Unfortunately that won't work with my work wifi.

Once again: it's clearly an intermittent network issue as described in previous comment.


generalize code to support different feeds, instances

duplicate own deployment pipeline with second feed to test it

probably have a default feed, but try to load custom from a file before using default


[KYUUBI #5010] Make Kyuubi server's connection URL configurable

Why are the changes needed?

This PR resolves https://github.com/apache/kyuubi/issues/5010, by introducing a new configuration property that makes it possible to override the hostname or ip of the Kyuubi server's thrift frontend service that will be registered for the service discovery ensemble.
How was this patch tested?

[X] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[X] Add screenshots for manual tests if appropriate

Before applying the patch, when we set kyuubi.frontend.bind.host=<IP_ADDRESS>, the znode becomes as follows.
[zk: some_zk_host:some_zk_port(CONNECTED) 0] ls /some_kyuubi_zk_namespace
[serviceUri=<IP_ADDRESS>:10009;version=1.8.0-SNAPSHOT;sequence=0000000007]

After applying the patch, when we set kyuubi.frontend.bind.host=<IP_ADDRESS> and kyuubi.frontend.advertised.host=some_kyuubi_host, it becomes as follows.
[zk: some_zk_host:some_zk_port(CONNECTED) 0] ls /some_kyuubi_zk_namespace
[serviceUri=some_kyuubi_host:10009;version=1.8.0-SNAPSHOT;sequence=0000000009]

[X] Run test locally before make a pull request

./build/mvn clean install -pl kyuubi-common succeeded.

Codecov Report

Merging #5015 (7fd980b) into master (04ee20b) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #5015   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         563     563
  Lines       31167   31175    +8
  Branches     4070    4071    +1
======================================
- Misses      31167   31175    +8

Impacted Files
Coverage Δ

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
0.00% <0.00%> (ø)

...a/org/apache/kyuubi/service/TFrontendService.scala
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

The following failed test will not be related to my PR, and it succeeds in my local environment.
DockerizedZkServiceDiscoverySuite:
- publish instance to discovery service
- KYUUBI #304: Stop engine service gracefully when related node is deleted
- distribute lock *** FAILED ***
  lockLatch.await(5000L, MILLISECONDS) was false (DiscoveryClientTests.scala:146)
- getAndIncrement method test
- setData method test
- acl for zookeeper
- set up zookeeper auth
- stop engine in time while zk ensemble terminates
- watcher for zookeeper
Run completed in 47 seconds, 328 milliseconds.
Total number of tests run: 9
Suites: completed 2, aborted 0
Tests: succeeded 8, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***

@pan3793
Could you kindly review my PR again when you have time?

@pan3793
Thanks for your careful review!
Let me fix the points you suggested soon.

Thanks, merged to master

@pan3793 @cxzl25
Thanks for your review!


How to compare objects ParameterEncoding?

I need to do my unit tests make sure that we have set the correct ParameterEncoding, but when I try:

How can I do it?

As you can see in the error message, you can't use == on parameters of ParamterEncoding and ParamterEncoding?, due to one being optional. Unwrap the optional and it should work fine.

It isn't not working!!!

The ParameterEncoding enum does not conform to the Equatable protocol because it uses associated values. Creating Equatable conformance for a set of associated value cases is a slippery slope. Mainly because you need to decide whether or not to just compare the case, or the associated values inside the case.
With that said, you should be able to use a switch statement instead.
let isEqual: Bool

switch (encoding, expectedEncoding) {
case (.JSON, .JSON):
    isEqual = true
default:
    isEqual = false
}

Cheers. 🍻

@RafaelPlantard Sorry about that. You can try what @cnoon suggested or something like this:
let equalEncodings: Bool = {
    if case .JSON = encoding {
        return true
    }

    return false
}()

A bit more compact.

Thanks, I did exactly that:
//
//  ParameterEncoding.swift
//  iShopSDK
//
//  Created by Rafael Ferreira on 7/24/16.
//  Copyright © 2016 Swift Yah. All rights reserved.
//

import Alamofire

/// An way to compare two instances of `ParameterEncoding`.
extension ParameterEncoding: Equatable { }

/// Compares two instances of `ParameterEncoding`.
/// Already, it cannot compare two instances of `ParameterEncoding.Custom`.
///
/// - Parameters:
///     - lhs: The first `ParameterEncoding` instance.
///     - rhs: The `ParameterEncoding` parameter that it will be comparable to `lhs`.
///
/// - Returns: True when both instances are completely equal, otherwise false.
public func == (lhs: ParameterEncoding, rhs: ParameterEncoding) -> Bool {
	switch (lhs, rhs) {
	case (.JSON, .JSON):
		return true

	case (.URL, .URL):
		return true

	case (.URLEncodedInURL, .URLEncodedInURL):
		return true

	case (.PropertyList(let lFmt, let lWOpt), .PropertyList(let rFmt, let rWOpt)) where (lFmt == rFmt && lWOpt == rWOpt):
		return true

	default:
		return false
	}
}

@jshier man how can I compare ParameterEncoding objects, once now they are structs?

@RafaelPlantard If all you want to do is ensure that your encodings are the correct type, optionally casting them should work. i.e. if let encoding = someEncoding as? JSONEncoding. Otherwise you'd have to implement == for all of the types specifically.

good idea @jshier implement equatable is a very good solution, but just a cast solve my problem, thanks...


feat(windows): generate device ID and persist it on disk

Relating to #2697 and #2711

Had to move a bunch of files to hide the whole project from the Ubuntu CI. Otherwise it would have been a small change.


Alpha release

Codecov Report
Base: 30.44% // Head: 29.81% // Decreases project coverage by -0.62% :warning:

Coverage data is based on head (0f81d51) compared to base (50b6375).
Patch coverage: 12.26% of modified lines in pull request are covered.

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

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


Roll Flutter Engine from 56a0e2715dca to 3da552c714ce (6 revisions)

https://github.com/flutter/engine/compare/56a0e2715dca...3da552c714ce
2023-08-14<EMAIL_ADDRESS>Roll Dart SDK from 3295a353980a to d445f5a18876 (1 revision) (flutter/engine#44691)
2023-08-14<EMAIL_ADDRESS>Revert "Make run_tests.py assert that the ios test dylib is at least as new as libFlutter.dylib" (flutter/engine#44690)
2023-08-14<EMAIL_ADDRESS>Roll Skia from 1cf6f71c8120 to a4377099b25a (1 revision) (flutter/engine#44688)
2023-08-14<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from JKTVoxVrHjZjBaxG4... to uIGMbDkXoIcpqWjgR... (flutter/engine#44687)
2023-08-14<EMAIL_ADDRESS>[Impeller] Conditionally set command debug info. (flutter/engine#44274)
2023-08-14<EMAIL_ADDRESS>Migrate more GL calls of GrBackend* (flutter/engine#44682)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from JKTVoxVrHjZj to uIGMbDkXoIcp
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Move some info out of the milestone chapter such as how space is calculated

It's a bit too much new info for a milestone project

and it's annoying to scroll through the milestone project just to search for something that could exist independently from the milestone project


[fix][cms] captcha: add store_in_repl_master in SS::Captcha

Coverage increased (+0.007%) to 84.617% when pulling ce727a545e799f0f3d95eab0c3811f39cef3f2fe on fix-captcha_store_in_repl_master into 4764a339ee206f247df4f3b88a0bed9ee42af8a0 on master.


Expose as_words()

I personally find it annoying that num-bigint didn't expose its internal representation, it will be great if the internal buffer is accessible (even readonly) from outside. This will be useful if other algorithms are to be implemented on ibig (such as algorithms related to modular arithmetics, number theory, rational and float numbers)

It will be even better if UBig can accept Vec<Word> as an input of constructor, in that case the overhead of copying will be minimized for the external algorithm implementations.

I understand the desire. However, this would limit the implementation. For instance, idea #13 would be outright prevented by this. Plus, Word is different on different platforms which may be annoying to users.
There is to_le_bytes and from_le_bytes, which can easily be converted to Vec<u64> or whatnot, perhaps this is sufficient for most algorithms?
Do you have a specific example of an algorithm that would benefit from accessing Words?

I understand the desire. However, this would limit the implementation. For instance, idea #13 would be outright prevented by this. Plus, Word is different on different platforms which may be annoying to users.
There is to_le_bytes and from_le_bytes, which can easily be converted to Vec<u64> or whatnot, perhaps this is sufficient for most algorithms?
Do you have a specific example of an algorithm that would benefit from accessing Words?

If we implement inline double word as &[u64;2], then we can take reference from all three enum variants of Repr. This is still feasible even if more words are inlined, the difference is just whether the memory behind the reference is on stack or heap.
Indeed I can use to_le_bytes/from_le_bytes, but the idea is to avoid copying. For example, if I want to implement a exact division algorithm (useful for implementing rational number), it's pretty fast for small integers, and in that case I guess the overhead of copying is not negligible? Montgomery form is another example, which I mentioned in another issue.


[Model] Add moondream vision language model

This pull request adds support for the moondream vision language model, added similar to how the LLaVA implementation works.
Usage:
    llm = LLM(
        model="vikhyatk/moondream2",
        trust_remote_code=True,
        image_input_type="pixel_values",
        image_token_id=50256,
        image_input_shape="1,3,378,378",
        image_feature_size=729,
    )

   output = llm.generate(
        [("<|endoftext|>" * 729) + "\n\nQuestion: Describe this image.\n\nAnswer:"],
        multi_modal_data=MultiModalData(
            type=MultiModalData.Type.IMAGE, data=image_pixels
        ),
        sampling_params=sampling_params,
    )

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Could you add a test to ensure that the model is consistent with its HuggingFace implementation? (Similar to test_llava.py)
To facilitate #4194, it would be great if you could wrap the image preprocessing logic inside of a HuggingFace AutoImageProcessor.

#4197 is very close to getting merged. @vikhyat can you address the issues mentioned above?

Hello! @vikhyat Thank you for the contribution and I'm excited to support this model in vLLM. We have done a significant amount of refactoring on vLLM for supporting multimodal models in the last 2 months - please see updates in #4194.
If you don't mind, I'm happy to update this PR accordingly so we can get this merged. Let me know!

Hi @ywang96! I had this in my backlog but I'm currently swamped trying to get a new version of the model trained and released, so it has been hard to find the time. I would definitely hugely appreciate it if you can help out!

Ooops - Feel free to ignore the comment from the bot. (I do still need to spend some time working on this model...)

👀


ScrollView auto scroll to previous position when focused new TextInput

Hi. I have a problem on IOS when use KeyboardAvoidingView and ScrollView with multiple TextInput. When the keyboard is showing, i scroll until the TextInput is focusing no longer visible in the screen and choose new TextInput, ScrollView will auto scroll to position of the previous TextInput. I need a solution for this issue, please help me. Thanks everyone
https://user-images.githubusercontent.com/33460888/127734585-c69e465d-0db6-43e7-87a7-eef9454c36ee.mp4
Update: Same issue when only using ScrollView

I found same problem. I will follow this thread: #30736


Adding Favicon

I just checked your website and realized it did'nt had any favicon. Can you assign me this issue

can u assign me this issue @boradesanket13

I want to work on this issue. please assign it to me @boradesanket13

@AyushTripathiIITP,
Go ahead! Contribute well🚀

check this image for favicon

or this image for favicon
@boradesanket13

Go with 2nd


upadte

Coverage decreased (-6.7%) to 93.333% when pulling 10154cdf168a6163d7d56288acd789d3eff7365c on pre-build into d63c4ea35347d44fd5ce50f2915a29e98c9dbd1a on master.


Example in docs could be improved

Maybe it only my case, but the example shown here https://github.com/mgonto/restangular#removing-an-element-from-a-collection-keeping-the-collection-restangularized seems to be invalid. Specifically the line
var index = $scope.users.indexOf(userWithId);

would always return -1, because collection of object cannot be searched (using regular indexOf) by object property. I ended up with using lodash for this purpose:
var index = _.findIndex($rootScope.currentUser.photos, 'id', $scope.photo.id)

It would be worth to update the docs with this, unless I'm missing something?

The example provided should work properly. If it doesn't, please provide some code (Plunker) to reproduce the problem, thanks!


no certs showing

Error
Badge Certifications Please see Activity Log for T-Rex Error!.
This error comes up when I go to Admin Functions- Post/Print Transactions and it shows no certifications in the list.

fixed with 1e7b40f


refresh token

Codecov Report

Merging #27 into master will decrease coverage by 0.72%.
The diff coverage is 26.16%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage      37%   36.28%   -0.73%
==========================================
  Files          48       52       +4
  Lines        2748     2985     +237
==========================================
+ Hits         1017     1083      +66
- Misses       1624     1780     +156
- Partials      107      122      +15

Continue to review full report at Codecov.

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


Fix format parameters for strings

I believe this is a good change, but doesn't it have to be done on TranslateWiki.net? e.g. by editing https://translatewiki.net/wiki/Wikimedia:Commons-android-strings-upload_progress_notification_title_finishing/he
I fear the bot may later "update" and revert the changes done here.
That said, maybe we can just try. We just have to keep an eye on such possible revert.

I'm good with doing it on the website, though I've never used it before as I don't speak a second language. Does anybody mind updating it on the site?

Let's try this. If it doesn't work as expected, I can do it on the website later.

Yeah, sounds good to me.

https://translatewiki.net/w/i.php?title=Wikimedia:Commons-android-strings-upload_progress_notification_title_finishing/he&diff=7336512&oldid=4543668
It has been propagated to translatewiki.net, so we are all set.