Test "buffers partial TS video segments in sequence mode" failing on Chromecast

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
main
Can you reproduce the issue with our latest release version?
No
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
N/A
If custom app, can you reproduce the issue using our demo app?
N/A
What browser and OS are you using?
Chromecast
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Chromecast Ultra, firmware 1.56, Chrome 90.0.4430.225
What are the manifest and license server URIs?

N/A
What configuration are you using?  What is the output of player.getConfiguration()?

N/A
What did you do?

Run all tests on Chromecast in the lab
What did you expect to happen?
Tests pass
What actually happened?

Test "buffers partial TS video segments in sequence mode" consistently fails with:

Shaka Error MEDIA.MEDIA_SOURCE_OPERATION_FAILED (4)
Error: Failed: video queue should be empty after destroy!

A git bisect shows that the issue began with revision 464f33c984a78aa7d72c3e389ca97c9b28dc5b38 (PR #5217).
This may be purely a problem with the tests, or it may be an actual bug impacting the Cast platform.  This is not yet clear.

Since that revision, the test triggers a codec switch, which triggers the failure.  The switch occurs via changeType.  Forcing isChangeTypeSupported() to false triggers a codec switch via reset(), which fails with a timeout.  Something about codec switching is fundamentally broken on Chromecast, which will block the v4.4 release.

I'm purely speculating at this point about what could be the problem.  It seems that changeType is detected, but doesn't work, which is not shocking to me for the Cast platform.  But blocking the use of changeType and resetting the pipeline doesn't work either.
I may see an issue with the reset process that might affect Cast.  It appears that a new MediaSource is created, with new SourceBuffers, then the video src attribute is set to the new one, without first shutting down the original MediaSource object as we would during destroy().  Perhaps having two overlapping MediaSource objects is a problem on Cast devices, which leads reset() to fail and the test to timeout.

Also, the failing test doesn't explicitly cover codec-switching, so the fact that this test discovered this problem is likely a coincidence.  The failing test should probably be adjusted so that codec-switching does not occur, and new, explicit tests should be created for codec-switching.

I found the issue, it's a issue with the test. Also, I think that codec-switching is broken on chromecast (changeType)


pd.read_csv throws error when used with parse_dates and index_col together

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.2 LTS
Modin installed from (source or binary): pip3 install modin
Modin version: 0.4.0
Python version: 3.6.5

Describe the problem
As the title says, ValueError is thrown when using pd.read_csv with parse_dates and index_col together.
Source code / logs

This is the line that throws an error:
pd.read_csv(cb_filepath, names=['timestamp', 'symbol', 'high', 'low', 'open', 'close', 'spread', 'volume'], header=0, index_col=0, parse_dates=['timestamp'], encoding='utf-8')
The csv file is formed like this:

timestamp,symbol,high,low,open,close,spread,volume
2010-04-01 00:00:00,USD/JPY,93.52600,93.36100,93.51800,93.38200,0.00500,3049
2010-04-01 00:30:00,USD/JPY,93.47500,93.35200,93.38500,93.39100,0.00600,2251
2010-04-01 01:00:00,USD/JPY,93.42100,93.32600,93.39100,93.38400,0.00600,1577

Error:

File "/home/censored/tensorflow/venv/lib/python3.6/site-packages/pandas/io/parsers.py", line 1122, in _make_engine
self._engine = CParserWrapper(self.f, **self.options)
File "/home/censored/tensorflow/venv/lib/python3.6/site-packages/pandas/io/parsers.py", line 1904, in init
self._set_noconvert_columns()
File "/home/censored/tensorflow/venv/lib/python3.6/site-packages/pandas/io/parsers.py", line 1973, in _set_noconvert_columns
_set(val)
File "/home/censored/tensorflow/venv/lib/python3.6/site-packages/pandas/io/parsers.py", line 1963, in _set
x = names.index(x)
ValueError: 'timestamp' is not in list

error.log

Thanks for posting @eralmansouri!
This looks like an issue with the way we are using BytesIO as a way to parallel read the file. This should be a pretty straightforward fix, we've had to use something similar for the Apache Arrow reader in #502. Great find, thanks!


BadDeviceId calling waveOutOpen

I am using naudio to play audio files and is working fine. But I have a scenario where I am depositing an app on a Windows VM - and connected to it via MS RDP. I connect with audio being sent to system that the RDP is done one (i.e. client of VM). Audio will not play on client system and getting the
naudio BadDeviceId calling waveOutOpen
error in event viewer
Should it be possible to run this scenario and get audio playback on local system?
Note: I play the audio manually and comes across ok - so file is a good audio file
Peter

Yes, unfortunately WaveOut doesn't seem to work too well over RDP. What device id did you use? Did you try -1 (for wave mapper)

didn't specify any device ID - configured to operate with defaults.
doing the following:
using (var audioFile = new AudioFileReader(fileName))
{
using (var outputDevice = new WaveOutEvent())
{
outputDevice.Init(audioFile);
outputDevice.Play();
}
}
Peter

Depending on the version of NAudio, the default is either 0, or -1. You can try setting them both explicitly, but as I've said, WaveOut doesn't seem to work well over RDP.

What you recommend for playback over RDP?

I unfortunately don't know why NAudio doesn't work over RDP. It's not something I've ever needed so I've not investigated it. Maybe its to do with the WaveFormat - perhaps you could try a few different formats to see if that's the issue. You could also try DirectSoundOut - maybe that works better

Cheers for the reply Mark. I've been developing an application for recording voice and sending for transcription. After seeing the comments I had a bit of a panic as all of our staff use RDP in ThinClients. But I tested it in the office and everything played back fine, just need to tweak a few bits as about half a second delay on playback to position bar.

@MartinHolmes82  Can you please show an example snippet on how you managed to make this work. I am also getting the same error when I deploy my application to IIS.
My method to stream from a URL looks like:
    public static void PlayMp3FromUrl(Stream stream)
    {
        using (Stream ms = new MemoryStream())
        {
            using (stream)
            {
                byte[] buffer = new byte[32768];
                int read;
                while ((read = stream.Read(buffer, 0, buffer.Length)) > 0)
                {
                    ms.Write(buffer, 0, read);
                }
            }

            ms.Position = 0;
            using (WaveStream blockAlignedStream =
                new BlockAlignReductionStream(
                    WaveFormatConversionStream.CreatePcmStream(
                        new WaveFileReader(ms))))
            {
                using (WaveOut waveOut = new WaveOut(WaveCallbackInfo.FunctionCallback()))
                {
                    waveOut.Init(blockAlignedStream);
                    waveOut.DesiredLatency = 100; // 50ms latency
                    waveOut.Play();
                    while (waveOut.PlaybackState == PlaybackState.Playing)
                    {
                        System.Threading.Thread.Sleep(100);
                    }
                }
            }
        }
    }

Any suggestions/comments would be greatly appreciated. Thanks in advance.

@MartinHolmes82 Can you please show an example snippet on how you managed to make this work. I am also getting the same error when I deploy my application to IIS.
My method to stream from a URL looks like:
    public static void PlayMp3FromUrl(Stream stream)
    {
        using (Stream ms = new MemoryStream())
        {
            using (stream)
            {
                byte[] buffer = new byte[32768];
                int read;
                while ((read = stream.Read(buffer, 0, buffer.Length)) > 0)
                {
                    ms.Write(buffer, 0, read);
                }
            }

            ms.Position = 0;
            using (WaveStream blockAlignedStream =
                new BlockAlignReductionStream(
                    WaveFormatConversionStream.CreatePcmStream(
                        new WaveFileReader(ms))))
            {
                using (WaveOut waveOut = new WaveOut(WaveCallbackInfo.FunctionCallback()))
                {
                    waveOut.Init(blockAlignedStream);
                    waveOut.DesiredLatency = 100;
                    waveOut.Play();
                    while (waveOut.PlaybackState == PlaybackState.Playing)
                    {
                        System.Threading.Thread.Sleep(100);
                    }
                }
            }
        }
    }

Any suggestions/comments would be greatly appreciated. Thanks in advance.

Hi @rahsharm77,
We are getting the same problem. Could you please help us how you solve this ?
Thanks

@MartinHolmes82 Can you please show an example snippet on how you managed to make this work. I am also getting the same error when I deploy my application to IIS.
My method to stream from a URL looks like:
    public static void PlayMp3FromUrl(Stream stream)
    {
        using (Stream ms = new MemoryStream())
        {
            using (stream)
            {
                byte[] buffer = new byte[32768];
                int read;
                while ((read = stream.Read(buffer, 0, buffer.Length)) > 0)
                {
                    ms.Write(buffer, 0, read);
                }
            }

            ms.Position = 0;
            using (WaveStream blockAlignedStream =
                new BlockAlignReductionStream(
                    WaveFormatConversionStream.CreatePcmStream(
                        new WaveFileReader(ms))))
            {
                using (WaveOut waveOut = new WaveOut(WaveCallbackInfo.FunctionCallback()))
                {
                    waveOut.Init(blockAlignedStream);
                    waveOut.DesiredLatency = 100;
                    waveOut.Play();
                    while (waveOut.PlaybackState == PlaybackState.Playing)
                    {
                        System.Threading.Thread.Sleep(100);
                    }
                }
            }
        }
    }

Any suggestions/comments would be greatly appreciated. Thanks in advance.

Hi @rahsharm77,
We are getting the same problem. Could you please help us how you solve this ?
Thanks

@deepakdubeydsys Is this still a problem or have you resolved it?

@MartinHolmes82 Can you please show an example snippet on how you managed to make this work. I am also getting the same error when I deploy my application to IIS.
My method to stream from a URL looks like:
    public static void PlayMp3FromUrl(Stream stream)
    {
        using (Stream ms = new MemoryStream())
        {
            using (stream)
            {
                byte[] buffer = new byte[32768];
                int read;
                while ((read = stream.Read(buffer, 0, buffer.Length)) > 0)
                {
                    ms.Write(buffer, 0, read);
                }
            }

            ms.Position = 0;
            using (WaveStream blockAlignedStream =
                new BlockAlignReductionStream(
                    WaveFormatConversionStream.CreatePcmStream(
                        new WaveFileReader(ms))))
            {
                using (WaveOut waveOut = new WaveOut(WaveCallbackInfo.FunctionCallback()))
                {
                    waveOut.Init(blockAlignedStream);
                    waveOut.DesiredLatency = 100;
                    waveOut.Play();
                    while (waveOut.PlaybackState == PlaybackState.Playing)
                    {
                        System.Threading.Thread.Sleep(100);
                    }
                }
            }
        }
    }

Any suggestions/comments would be greatly appreciated. Thanks in advance.

Hi @rahsharm77,
We are getting the same problem. Could you please help us how you solve this ?
Thanks

@deepakdubeydsys Is this still a problem at your end or have you resolved it?


[java client] Cannot integrate ProducerStats with external metrics systems

Describe the bug
In case of PartitionedProducerImpl the getStats() method modifies the state of the internal "stats" object
public synchronized ProducerStatsRecorderImpl getStats() {
        if (stats == null) {
            return null;
        }
        stats.reset();
        for (int i = 0; i < topicMetadata.numPartitions(); i++) {
            stats.updateCumulativeStats(producers.get(i).getStats());
        }
        return stats;
    }

The fact that the method is synchronized does not save the user from having corrupted stats, because the getStats() returns a reference to the internal stats object.
So if you call getStats() and the call some methods on the returned object from two different threads it can happen that the first thread sees corrupted values, because the second thread calls reset in the meantime.
ProducerStats stats = producer.getStats();
// accessing an object that is mutated from a different thread
long totalMsgsSent = stats.getTotalMsgsSent();

This behaviour prevents you from collecting the stats using an external system that does not provide a documented and consistent behaviour about concurrency.
For instance I would like to create "Gauges" and attach them to each of the exposed metrics, but it is actually not possible.
I can do it with this kind of code, by it is likely to break if we change the implementation of the PulsarProducer
Gauge gauge = () -> {
synchronized(producer) {
   ProducerStats stats = producer.getStats();
   long totalMsgsSent = stats.getTotalMsgsSent();
   return totalMsgsSent;
 }}

Suggestions for improvement
Create a new instance of ProducerStats in PartitionedProducerImpl instead of recycling the existing object

The issue had no activity for 30 days, mark with Stale label.


Improve project resolving for StatusReporter

Fixes #902
Also fix the error status code from GitLab for which we want to fallback
to commit comment.
Move the logic behind resolving of project, which contains commit where
we want to set commit status, from helpers to the StatusReporter itself
Current behaviour:

GitHub
can set commit status from the project where the PR resides
GitLab

MR: if present MR ID, grabs the MR and related source_project
branch push: resolves _project_with_commit to the project itself

Pagure
similar to GitHub

Signed-off-by: Matej Focko<EMAIL_ADDRESS>

this is getting pretty scary
can we cover this by a test case?

Yup, will do. Since I didn't need to touch the tests, we don't have it (TFT results for GitLab) covered yet :D


Removed typo in std.concurrency, line 1196

NB: Is the use of "analog" correct here? It sounds a bit odd to me.

Could you rebase so that we don't have four commits for one typo?

Yes, please squash, then this is good to go.

Yes, please squash, then this is good to go.

It rather should re-work this so that there is no merge commit, that needs at least reset --hard to some older commit then rebase.

Sorry, what went wrong? I had a problem with github yesterday.

Sorry, what went wrong? I had a problem with github yesterday.

I don't know what exactly happened, but of the four commits in this pull request only the last one seems to actually do anything. It would be nice if we could get rid of the others.
If you're using the typical GitHub setup with origin and upstream remote branches, but you're not well-versed in git, here's a way to fix this: git rebase --onto upstream/master master~1 master
This takes the latest commit and re-applies it on top of upstream/master. The other commits between upstream/master and your local master are discarded.
Then git push -f to update the pull request.

I forgot the last step: leave a comment here when you're done. GitHub doesn't send notifications for pushed commits, so you have to ping the reviewers manually (or hope that someone notices it).
Looks good to me now.

Ok. I did what you recommended. I hope it's fine now.

Auto-merge toggled on


Runtime update

Made it so that the runtime doesn't need to parse the AST each time a function is invoked.
Additionally

Updated documentation
Updated Examples
Bumped package's semantic version

Heyy, thank you @ethanwillis :)
LGTM
I do not want to bother you, so I'll merge this and I'll update the example in the ./example/js/index.jsin which there's still the old function call.
So I use this occasion to do some quick hotfix to another leftover I spotted!

thanks!


How to redirect with vue-router?

So i have a SPA where the base path has to redirect to a locale folder. Normally i would do this by defining two paths that are pointing to the same page in the router:
  routes: [
    {
      path: '/',
      redirect: { name: 'index' }
    },
    {
      path: '/en/',
      name: 'index',
      component: index
    }
  ]

E.g. http://test.com/ is redirected to http://test.com/en.
Following solved the issue, but i am not sure if this is best-practice?
<script>
export default {
  fetch ({ params, redirect }) {
    redirect(301, '/en')
  }
}
</script>

Hi @maziarz,
It's the recommended solution and the easiest one 👍

Would love to see this in the documentation!! Example using dynamic routes:
export default {
  fetch ({ params, redirect }) {
    redirect(301, `/v/${params.title}/${params.id}`)
  }
}

I used

 fetch ({ params, redirect }) {
    redirect(301, `/v/${params.title}/${params.id}`)
  }

but, I find I cant use this.$router.push().  When I use this.$router.push(),my browser will death .
Does it have any other way to redirect ?


[#151] Correction of big publication speed up

With PR #152 vocabularies were not published correctly on the test environment. This PR corrects that issue.
Related #151

This pull request tries solve the following issue:

if i try to publish workspace with a loaded but not modified vocabulary (e.g. pracovní prostor, the PR is created but vocabulary, model and attachment files are deleted in the PR.


copr: Add non-vectorize trim1Arg

UCP #6957
Signed-off-by: Poytr1<EMAIL_ADDRESS>
What have you changed?
Add non-vectorize trim1Arg
What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?

No
Does this PR affect tidb-ansible?

No
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/6957

Hi @sticnarf , could you please help me review the pr? Thanks.

The PR title needs correction.You are implementing the vectorized version :)

The PR title needs correction.You are implementing the vectorized version :)

Yeah..you are right, just corrected the title. Thanks😆

@Poytr1 Really sorry. My fault.
I think trim needs to support invalid UTF-8 bytes. You should take ltrim or rtrim as a reference.

@sticnarf No worries, I just re-implemented the trim_1_arg function, please take a look.

Hi @sticnarf , could you please help me review this change again?

@Poytr1 FYI:
https://github.com/tikv/tikv/blob/f6772aeaf6994ad37160f3b52d3e88e9e50ffb9b/components/tidb_query/src/rpn_expr/impl_string.rs#L934

/merge


Refactor to support symbolic expressions

Refactor to support symbolic expressions. It should have little distinction between data and code. It should use same structures to represent both.

Let's call it sym-expressions so as not be confused by lisp's s-expression

Above referenced  22521214bc93e4ee6a5eaaeca164f945bd02be9e closes this issue


Create a category list page.

Hi,
Thanks for creating a great theme!
I want to display the category list page, but it doesn't work.
For example,
novela.com/categories/orange/
novela.com/categories/red/
I want to display articles with each category when I access the permalink as above.
Do you plan to introduce category features in the future?
This is a great theme and I don't want to change it if possible.

Hi, glad you like it.
We don't plan to add any other feature from the original theme at https://novela.narative.co
I don't see why you wouldn't be able to override the theme and use a category list template.

/ Category / orange /
When accessed, the same article as the home page, that is, all articles will be displayed.
I'll give it a try.


microk8s not starting on ubuntu22.04.1 on odroid sbc

Summary
I have a brand new ODROID N2+ SBC (ARM64)
I installed UBUNTU on it.
I installed microk8s on it
microk8s refuses to run
What Should Happen Instead?
microk8s should run
Reproduction Steps

Install ubuntu 22.04.1
Install microk8s : sudo snap install microk8s --classic

Introspection Report
inspection-report-20230125_094845.tar.gz
Are you interested in contributing with a fix?
no

Hi @AllardKrings
From the inspection report, I can see lots of message that look like this:
Jan 25 09:48:27 odroid microk8s.daemon-containerd[3432]: time="2023-01-25T09:48:27.166575670Z" level=error msg="RunPodSandbox for &PodSandboxMetadata{Name:calico-node-ss4pg,Uid:0673a412-ebfd-4cd3-b7a3-f5f5c5870644,Namespace:kube-system,Attempt:1,} failed, error" error="failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error setting cgroup config for procHooks process: open /sys/fs/cgroup/kubepods/burstable/pod0673a412-ebfd-4cd3-b7a3-f5f5c5870644/96005811154644d4d9e30b42b1d1d914b344f7e617d2130aa94604e6d5f8f714/cpu.weight: no such file or directory: unknown"

Are cgroups enabled on the machine?

I tried to do that, however the ubuntu version on odroid apparantly is
different from the one I use on raspberry pi.
I added the option but it seems the ubuntu kernel is too old (4.9)
I will switch now to another linux-brand until canonical has a decent
distribution for the odroid that supports microk8s :-(
Op wo 25 jan. 2023 om 15:54 schreef Angelos Kolaitis <
@.***>

Hi @AllardKrings https://github.com/AllardKrings
From the inspection report, I can see lots of message that look like this:
Jan 25 09:48:27 odroid microk8s.daemon-containerd[3432]: time="2023-01-25T09:48:27.166575670Z" level=error msg="RunPodSandbox for &PodSandboxMetadata{Name:calico-node-ss4pg,Uid:0673a412-ebfd-4cd3-b7a3-f5f5c5870644,Namespace:kube-system,Attempt:1,} failed, error" error="failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error setting cgroup config for procHooks process: open /sys/fs/cgroup/kubepods/burstable/pod0673a412-ebfd-4cd3-b7a3-f5f5c5870644/96005811154644d4d9e30b42b1d1d914b344f7e617d2130aa94604e6d5f8f714/cpu.weight: no such file or directory: unknown"
Are cgroups enabled on the machine?
—
Reply to this email directly, view it on GitHub
https://github.com/canonical/microk8s/issues/3693#issuecomment-1403745199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AET5SGMDYPMUNOI7M6RVI7DWUE5C5ANCNFSM6AAAAAAUGDOO3E
.
You are receiving this because you were mentioned.Message ID:
@.***>

@neoaggelos
Same thing happening with Orange Pi 5. I'm getting cpu.weight: no such file or directory.
cgroups seem to be enabled:
$ findmnt -lo source,target,fstype,options -t cgroup,cgroup2
SOURCE  TARGET         FSTYPE  OPTIONS
cgroup2 /sys/fs/cgroup cgroup2 rw,nosuid,nodev,noexec,relatime,nsdelegate,memory_recursiveprot

$ ls -1 /sys/fs/cgroup/
cgroup.controllers
cgroup.max.depth
cgroup.max.descendants
cgroup.procs
cgroup.stat
cgroup.subtree_control
cgroup.threads
cpu.pressure
cpuset.cpus.effective
cpuset.mems.effective
cpu.stat
dev-hugepages.mount
dev-mqueue.mount
init.scope
io.pressure
io.stat
memory.pressure
memory.stat
proc-sys-fs-binfmt_misc.mount
sys-fs-fuse-connections.mount
sys-kernel-config.mount
sys-kernel-debug.mount
sys-kernel-tracing.mount
system.slice
user.slice

$ uname -a
Linux pi-03 5.10.110-orange-pi #1 SMP Sat Feb 25 03:45:09 GMT 2023 aarch64 aarch64 aarch64 GNU/Linux

I was using this linux build: https://github.com/Joshua-Riek/ubuntu-orange-pi5

Hi @adubkov
I'm not sure, I do not see a cpu.weight in the list. Could it be the particular cgroup implementation and/or systemd version and/or kernel version and/or architecture that does not support cpu.weight?
Can you refresh all system packages, reboot the OS and then test? I don't think there is much we can do from the MicroK8s side here.

It was an issue with kernel parameters. Once I set to:
cgroup_enable=cpuset cgroup_memory=1 cgroup_enable=memory swapaccount=1 systemd.unified_cgroup_hierarchy=0"

it start working.


Update 0929-unique-email-addresses.js

File(s) Modified: 0929-unique-email-addresses.js
Language(s) Used: javascript
Submission URL: _https://leetcode.com/problems/unique-email-addresses/submissions/1043783690_

@aestheticw3 thanks for the better solutions.

glad to help!


pixels should be in [0,255] range

Hey, thanks for your interest in TorchBeast!
I suppose that depends on the settings used (scale=True or not), but I agree for the specific settings we apply this would be correct.


Async optimization

What is the purpose of the change
Fixes https://github.com/apache/incubator-dubbo/issues/3287
Brief changelog
GenericService provides extra CompletableFuture method.
Invoker chain
Embedded Filter implementations.
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

This pull request is too large to review... Could you provide a guide to the reviewers?

All the UT are failing. Could you take a look at it?

This pull request is too large to review... Could you provide a guide to the reviewers?

Sorry, but I am afraid this one has to be this way. I tried to refactor the whole call process inside Dubbo to make it totally asynchronous, the changes in this pull request are all restricted to this goal and they are almost the minimum changesets to make the framework work again.
I will try to provide a reviewing guide to help anyone who's willing to review.

https://github.com/apache/incubator-dubbo/pull/3847

Although many files have been modified in the PR, there are only 3 key points, all other changes are made to adapt to these core designs.

Refactoring of internal Filters, making them work with the async pattern by registering Listener to each Filter.
Use AsyncRpcResult as the type return back to the Filter chain.

Add status interface to Result interface

public interface Result extends Serializable {

    Result thenApplyWithContext(Function<Result, Result> fn);

    <U> CompletableFuture<U> thenApply(Function<Result, ? extends U> fn);

    Result get() throws InterruptedException, ExecutionException;

}

AsyncRpcResult

public class AsyncRpcResult implements Result {
    private CompletableFuture<AppResponse> responseFuture;
    ...
}

Rename RpcResult to AppResponse with the meaning no changed

public class AppResponse implements Result, Serializable {
}

Add async method with CompletableFuture signature to GenericService

public interface GenericService {

    /**
     * Generic invocation
     *
     * @param method         Method name, e.g. findPerson. If there are overridden methods, parameter info is
     *                       required, e.g. findPerson(java.lang.String)
     * @param parameterTypes Parameter types
     * @param args           Arguments
     * @return invocation return value
     * @throws GenericException potential exception thrown from the invocation
     */
    Object $invoke(String method, String[] parameterTypes, Object[] args) throws GenericException;

    default CompletableFuture<Object> $invokeAsync(String method, String[] parameterTypes, Object[] args) throws GenericException {
        Object object = $invoke(method, parameterTypes, args);
        if (object instanceof CompletableFuture) {
            return (CompletableFuture<Object>) object;
        }
        return CompletableFuture.completedFuture(object);
    }

}

Below are the async and sync workflows respectively:


fix: test scripts and readme doc about uninstallation

fix #315

Just one small fix to make the lint check green ;)
 In ./test/run-e2e-ocp.sh line 67:
make: *** [Makefile:38: lint-shell] Error 123
	oc delete crds $(oc api-resources --api-group=monitoring.rhobs -o name)
                       ^-- SC2046 (warning): Quote this to prevent word splitting.

@simonpasquier @danielmellado Please help review and approve


fix(docs): link to contributing data from docs homepage

https://docs.clearlydefined.io/ Currently the Contribute Data link 404s

@nickvidal I've heard you might be the person to review this? 🙏 🙇


matio: 1.5.22 -> 1.5.23

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/matio/versions.
meta.description for matio is: "A C library for reading and writing Matlab MAT files"
meta.homepage for matio is: "http://matio.sourceforge.net/"
meta.changelog for matio is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.23 with grep in /nix/store/7kakhkh0gf3kgf6v2xvb3gw2dqqrh35c-matio-1.5.23
found 1.5.23 in filename of file in /nix/store/7kakhkh0gf3kgf6v2xvb3gw2dqqrh35c-matio-1.5.23

Rebuild report (if merged into master) (click to expand)

20 total rebuild path(s)

20 package rebuild(s)

First fifty rebuilds by attrpath
gnss-sdr
grafana-image-renderer
haskellPackages.gi-vips
imgproxy
joplin
mathematica
mathematica-cuda
mathematica10
mathematica11
mathematica9
matio
nip2
photoflow
python310Packages.pyvips
python39Packages.pyvips
rubyPackages.ruby-vips
rubyPackages_2_7.ruby-vips
rubyPackages_3_0.ruby-vips
rubyPackages_3_1.ruby-vips
vips

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7kakhkh0gf3kgf6v2xvb3gw2dqqrh35c-matio-1.5.23 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A matio https://github.com/r-ryantm/nixpkgs/archive/257363d5eea91f18d4d0026a26fa4a5a7f728b93.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7kakhkh0gf3kgf6v2xvb3gw2dqqrh35c-matio-1.5.23
ls -la /nix/store/7kakhkh0gf3kgf6v2xvb3gw2dqqrh35c-matio-1.5.23/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package marked as broken and skipped:

photoflow

5 packages failed to build:

mathematica
mathematica-cuda
mathematica10
mathematica11
mathematica9

12 packages built:

gnss-sdr
grafana-image-renderer
imgproxy
joplin
matio
nip2
python310Packages.pyvips
python39Packages.pyvips
rubyPackages.ruby-vips
rubyPackages_3_0.ruby-vips
rubyPackages_3_1.ruby-vips
vips

Maintainer pings
cc @vbgl for testing.

This package has been orphaned on upstream.

Cannot find if the package has CVE attached: https://sourceforge.net/p/matio/news/


Windows fixes

Trying to use jspm-git on windows and ran into issues which appear to be parameter related.
Switching from exec() to execFile() in the exec-git module seems to fix it.
It does require parameters passed into to execGit to be in an array, but it does remove the need to escape any string parameters with double quotes.

Hi @gerarde, looks good and awesome that you added some unit tests :+1: After some testing on my machine I'll merge it.

I wouldn't call them unit tests - it was just updating an existing script to make sure things work.


package the UI in docker, and include in docker-compose

needed for testing and development of other backend services.

Fixed


avoid SetFileCompletionNotificationModes crash

According to Microsoft's own documentation, there is a known bug where SetFileCompletionNotificationModes crashes on some systems (see https://support.microsoft.com/kb/2568167 for details) and a user of ours has experienced this with our software. This adapts the precautions added to Golang itself (https://go-review.googlesource.com/c/go/+/69870/).
@maxtaco

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: zanderz sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

any progress

Does this also replace #27, fixing #26? If I'm reading the Golang-side change, it also handles Windows XP, so I expect the answer is "yes". I'm not sure how it handles XP though.
I'm also curious why the second hunk from #27 wasn't needed here, the change in asyncIo, performing the result-wait even if err was not syscall.ERROR_IO_PENDING.

Does this also replace #27, fixing #26? If I'm reading the Golang-side change, it also handles Windows XP, so I expect the answer is "yes". I'm not sure how it handles XP though.
I'm also curious why the second hunk from #27 wasn't needed here, the change in asyncIo, performing the result-wait even if err was not syscall.ERROR_IO_PENDING.


Can't run on iOS device

Performing hot restart...
Restarted application in 2,422ms.
flutter: libcurl/7.88.1-DEV BoringSSL zlib/1.2.11 brotli/1.0.9 nghttp2/1.52.0 quiche/0.16.0
flutter: Error: SSL peer certificate or SSH remote key was not OK
flutter: Status: 0
flutter: HTTP: HTTPVersion.unknown
flutter:
code:
void main() async {
  // Initialize client
  final client = curl.Client(
    verbose: true,
    interceptors: [
      // HTTPCaching(),
    ],
  );
  await client.init();

// Send request
  final res = await client.send(curl.Request(
    method: "GET",
    url: "https://ajinasokan.com/",
  ));

// Read response
print("Error: ${res.errorMessage}");
  print("Status: ${res.statusCode}");
  print("HTTP: ${res.httpVersion}");
  res.headers.forEach((key, value) {
    print("$key: $value");
  });
  print(res.text());
}

flutter version: 3.13.7
But it's normal on Android, what can i do?

verifySSL: false solved


can't install gridencoder

Description
pip  install ./gridencoder
note: This error originates from a subprocess, and is likely not a problem with pip.   ERROR: Failed building wheel for shencoder   Running setup.py clean for shencoder Failed to build shencoder ERROR: Could not build wheels for shencoder, which is required to install pyproject.toml-based projects
Steps to Reproduce
pip  install ./gridencoder
Expected Behavior
success
Environment
torch2.0.1+cu117

upgrade your pip.

same bug for 4 parts

你解决了吗，我遇到了同样的问题

你解决了吗，我遇到了同样的问题

请问你解决了吗，same problem.


No License

The Cargo.toml already specifies the license (MIT). I'll add an explicit license file as well.

Ah, missed it, thanks! Great library.

Thanks :)


Feat(builder): starting builder

@kreuzerk I wanted to open this up to get an idea of what you think.
I have a test to prove out the builder piece, can also test with npm link, but I am only covering constants scenario right now.

adds sample icons
basic test, no errors, only constants
first pass at schema, only partially complete
forks the process to avoid blocking

The tests are passing now, or should be but the install is broken due to this in the builders.json:
  "svg-icons-builder": {
      "implementation": "./src/svg-icons-builder",
      "schema": "./src/svg-icons-builder/schema.json",
      "description": "Build SVG icons."
    }

to run needs to be:
  "svg-icons-builder": {
      "implementation": "./svg-icons-builder",
      "schema": "./svg-icons-builder/schema.json",
      "description": "Build SVG icons."
    }

There is a TODO or two with questions on type.

Wow @kaltepeter - this is amazing 👍  Epic contribution!

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enhancements to Time Series Graphs

Time Series:
This was a great opportunity to look at 6 years of data for an element for all the Counties in one or multiple States.

The points were hard to see. When I tried to select a point, I got the following error when looking at multiple States.

When I choose the legend and toggled off the “Points”, I really like this graphic. When I tried to select a bar graph to see what site it was, nothing came up.

I would like to be able to toggle off and on Counties in the legend.

It was hard for me sometimes to differentiate the orange colors and compare County to County. I would like to be able to make a bar graph with one to multiple counties with the years right next to each other. May have to use hatching to differentiate year to year. I can’t really tell in the graphic below how the different “orange” counties compared to each other between 2005 and 2010.

@jshourds-usgs can you pull down the most recent changes and see if you think this issue can be closed?

So here's the superweird thing, @ldecicco-USGS .  I did a git pull upstream master today and it said everything was up to date even though I was on AL yesterday and you have done a ton of updates yesterday and today.  Seems like I got nothing... yikes.  Any one else run into the same issue?
I also updated all my installed packages and seems like everything stayed the same...
I have to resolve this soon (ie before our training next week)-- although as long as I can still use the shiny app, no one expects you to have made Kim's enhancements so quickly anyway... so maybe this is not a cause for stressing about right now...

I pushed it out to GRAN...so you should be seeing a slightly updated app if you do this:
install.packages("wateRuse",
       repos=c("https://owi.usgs.gov/R",
                "https://cloud.r-project.org"))
library(wateRuse)
explore_wateRuse()

Let me know if you don't see a tweeked app.
As for your github issue?....not sure....what do you get when you run in the git shell:
git config -l

Ah now I see an updated app-- thanks for the reminder!
as for my git config screen here is a screen rip of what I see:

which seems okay to me-- not sure why I wouldn't be getting the updates you did yesterday...   Thanks for your help!

so after reviewing the updated version, Kim's changes were made.  Yay, thanks @ldecicco-USGS .  The only one I think you missed was the color scheme (and I think this is a comment elsewhere too)-- when you are plotting a lot of counties, it's virtually impossible to tell apart the greens or oranges or other colors as it's a scale of color spectrum.  Kim suggested hatching to differentiate?

Looks like your "upstream" is the same as your "origin". You need to change the upstream url to:
git@github.com:USGS-R/wateRuse.git
https://help.github.com/articles/changing-a-remote-s-url/

yeah, it's always going to be hard to get enough distinct colors...but now the hover should always work well, and you can zoom in to the data you are curious about.

I agree it's going to be hard to find distinct colors with that many variables.  The hatching was a great suggestion per Kim but perhaps not feasible.  Zooming in is great as is the labels working now... Great add.
As for changing my upstream, not sure how they became equal.  Not having a lot of luck there-- trying to follow the website you sent and entering: git remote set -url git@github.com:USGS-R/wateRuse.git and here's what I get back:

Clearly, I'm doing something wrong as when I git config -l it's still the same and master=origin.  Argh...

don't put a space between set and -url...just set-url

wait, that's not it....
git remote set-url upstream git@github.com:USGS-R/wateRuse.git

Ahhhh thank you I didn't know where to put the word upstream
Wasn't clear on the webpage-- thanks so much Laura!  All set now!

Great, so now I don't have rights to that repository???  all sorts of fatal errors

I keep getting myself deeper and deeper into a pile of :hankey:

You probably need to add your SSH key on whatever computer you are using now to your github account.
https://help.github.com/articles/adding-a-new-ssh-key-to-your-github-account/
This is because the DOI made some new security certificate thingies, and now we have to use SSH (at the workshop, we used https).

okay so I feel pretty helpless here.  I checked for ssh keys and here's what I get
not sure how to add them?

Yes I'm confused about this SSH key.  I have one created on github, but I
don't think it is on my computer.  How can I tell?   (Every time I wamt to
git pull or git push, it asks for my passphrase.)
Also, I have been unable to create a new project for wateRuse from
scratch.  Using the old instructions linking to https gives In the Cloning
Repository popup window:
"
Cloning into 'wateRuse'...
fatal: unable to access 'https://github.com/grrmartin-USGS/wateRuse.git/':
SSL certificate problem: unable to get local issuer certificate
To create a new project, should I instead be linking to<EMAIL_ADDRESS>grrmartin-USGS/wateRuse.git/
https://github.com/grrmartin-USGS/wateRuse.git/  ?
Thanks
-Gary
Gary Martin<EMAIL_ADDRESS>Hydrologist
U.S. Geological Survey
Kentucky Water Science Center
9818 Bluegrass Parkway
Louisville, Kentucky
40299-1906
(502) 609-1383 (cell)
(502) 493-1914 (phone)
(502) 493-1909 (fax)
On Fri, Oct 7, 2016 at 11:49 AM, jshourds-usgs<EMAIL_ADDRESS>wrote:

okay so I feel pretty helpless here. I checked for ssh keys and here's
what I get
not sure how to add them?
[image: ssh]
https://cloud.githubusercontent.com/assets/18489683/19196560/1679e74e-8c84-11e6-98f0-aa7e2f510f34.JPG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-R/wateRuse/issues/149#issuecomment-252288698,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARkrJdzTzo2vxT7HG4vVEk26niwUYUyeks5qxmoSgaJpZM4KNBt5
.

Here's how you check for your existing SSH key:
https://help.github.com/articles/checking-for-existing-ssh-keys/
@grrmartin-USGS : you have to change your remote to the SSH remote instead of the https remote (again, this is because of DOI's newer policy... we took a short-cut using https at the workshop and now it's biting us in the butt).
Look above at the instructions for @jshourds-usgs :
It's probably:
git remote set-url upstream git@github.com:USGS-R/wateRuse.git
git remote set-url origin git@github.com:grrmartin-USGS/wateRuse.git

Does this need to be conveyed to the RWUS for training next week?
On Fri, Oct 7, 2016 at 10:28 AM, Laura DeCicco<EMAIL_ADDRESS>wrote:

Here's how you check for your existing SSH key:
https://help.github.com/articles/checking-for-existing-ssh-keys/
@grrmartin-USGS https://github.com/grrmartin-USGS : you have to change
your remote to the SSH remote instead of the https remote (again, this is
because of DOI's newer policy... we took a short-cut using https at the
workshop and now it's biting us in the butt).
Look above at the instructions for @jshourds-usgs
https://github.com/jshourds-usgs :
It's probably:
git remote set-url upstream git@github.com:USGS-R/wateRuse.git
git remote set-url origin git@github.com:grrmartin-USGS/wateRuse.git
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-R/wateRuse/issues/149#issuecomment-252298242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARyLb3wG4WKD6lm4agBO7TelP7tokRT3ks5qxnMogaJpZM4KNBt5
.

--
Molly A. Maupin
Hydrologist
USGS Idaho Water Science Center
230 Collins Road
Boise, ID  83702
Office:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
<EMAIL_ADDRESS>

I would hope that "regular old users" aren't using the devtools or github (except to report issues). They should only use:
#only do this next line ONCE!!!!!
install.packages("wateRuse", repos = c("https://owi.usgs.gov/R","https://cloud.r-project.org"))

library(wateRuse)
explore_wateRuse()

github and SSH keys and devtools should only be for the developers of the package...ie anyone who went to the workshop. If someone new wants to start contributing to the code, then we can walk them through all that.

and, I'd prefer the new users follow the instructions on the readme (adding GRAN to their profile), and just click the update button vs what I pasted above.

thank you, I'm relieved.
On Fri, Oct 7, 2016 at 11:50 AM, Laura DeCicco<EMAIL_ADDRESS>wrote:

and, I'd prefer the new users follow the instructions on the readme
(adding GRAN to their profile), and just click the update button vs what I
pasted above.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/USGS-R/wateRuse/issues/149#issuecomment-252317262,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARyLb2FSn4gU3zZNa_cucjuBOAwg4EHnks5qxoZYgaJpZM4KNBt5
.

--
Molly A. Maupin
Hydrologist
USGS Idaho Water Science Center
230 Collins Road
Boise, ID  83702
Office:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
<EMAIL_ADDRESS>

I created a new project linking to my fork at
git@github.com:grrmartin-USGS/wateRuse.git
(with Open in new window checked).  However, the usual commands used in
the git shell to push/pull don't work now.  I can pull and push with the
button/arrows though.  Any idea why the shell would work differently with
SSH?
I have the SSH key and upstream and origin established:
[image: Inline image 1]
On Fri, Oct 7, 2016 at 1:54 PM, Molly Maupin<EMAIL_ADDRESS>wrote:

thank you, I'm relieved.
On Fri, Oct 7, 2016 at 11:50 AM, Laura DeCicco<EMAIL_ADDRESS>wrote:

and, I'd prefer the new users follow the instructions on the readme
(adding GRAN to their profile), and just click the update button vs what
I
pasted above.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/USGS-R/wateRuse/issues/149#issuecomment-252317262,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ARyLb2FSn4gU3zZNa_
cucjuBOAwg4EHnks5qxoZYgaJpZM4KNBt5>
.

--
Molly A. Maupin
Hydrologist
USGS Idaho Water Science Center
230 Collins Road
Boise, ID 83702
Office:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/USGS-R/wateRuse/issues/149#issuecomment-252318264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARkrJfe3XPZ-e81JidYQi6go2Gqtpbvnks5qxodLgaJpZM4KNBt5
.

okay so once I have checked my SSH key then I do what?
here is what I get when I check my SSH key:

@mamaupin-usgs not only do the RWUSes not have to worry about this, I can still run what I need to run for the training despite not being able to get the pull request updates and being "fatal" over here in NJ!  ;-)
Thank goodness for that too!!

Now I'm here: [https://help.github.com/articles/generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent/#adding-your-ssh-key-to-the-ssh-agent] at step 2 that says:
Add your SSH key to the ssh-agent. If you used an existing SSH key rather than generating a new SSH key, you'll need to replace id_rsa in the command with the name of your existing private key file.
Sooo the question is how do I know what the name of my existing private key file is??
I tried replacing the id_rsa with my 4 digit # and then I went into the setting up on my profile to get that long jibber jabber key (the one with all the 2 digit alpha numerics and the colons) and neither worked...

I don't know...we might have to talk this through a webex next week.

Ironically, I can get updates and all works well using gov equipment at home, logging in via SecurePulse.  No need for passcode either.  That equipment must not be up to DOI's new specs?

Hi Laura @ldecicco-USGS ,
When you have a chance, can you help me deal with this?  I am locked out from getting any updates via git shell and I think I need to get this resolved to be of any use to the team.  This all started last fall when DOI changed the security requirements.
Here's what I get when I try a git pull upstream master :

Thanks so much!
Jen


Readtimeout

Describe the bug
Select request timeout and timeout cannot be configured
Is there any way to configure the timeout?
To Reproduce
I don't know, unfortunately does not seem deterministic, scaling issue or something.
Expected behavior
Request execute most of the time properly at the same scale
Additional context
Linux, Cloud Run
pypi 1.0.2
FastAPI
Crashing here: https://github.com/different-ai/embedbase/blob/0478aeb949c07990fdefcde98b6894e6638df577/embedbase/database/supabase_db.py#L46

Traceback (most recent call last): File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/httptools_impl.py", line 436, in run_asgi result = await app( # type: ignore[func-returns-value] File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in call return await self.app(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 276, in call await super().call(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 333, in _sentry_patched_asgi_app return await middleware(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/asgi.py", line 139, in _run_asgi3 return await self._run_app(scope, lambda: self.app(scope, receive, send)) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/asgi.py", line 188, in _run_app raise exc from None File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/asgi.py", line 183, in _run_app return await callback() File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 122, in call await self.middleware_stack(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 130, in _create_span_call return await old_call(app, scope, new_receive, new_send, **kwargs) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in call raise exc File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in call await self.app(scope, receive, _send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 130, in _create_span_call return await old_call(app, scope, new_receive, new_send, **kwargs) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/cors.py", line 92, in call await self.simple_response(scope, receive, send, request_headers=headers) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/cors.py", line 147, in simple_response await self.app(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 130, in _create_span_call return await old_call(app, scope, new_receive, new_send, **kwargs) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 108, in call response = await self.dispatch_func(request, call_next) File "/app/middlewares/auth_api_key/auth_api_key.py", line 182, in dispatch response = await call_next(request) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 84, in call_next raise app_exc File "/usr/local/lib/python3.10/site-packages/starlette/middleware/base.py", line 70, in coro await self.app(scope, receive_or_disconnect, send_no_error) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 227, in _sentry_exceptionmiddleware_call await old_call(self, scope, receive, send) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 130, in _create_span_call return await old_call(app, scope, new_receive, new_send, **kwargs) File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in call raise exc File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in call await self.app(scope, receive, sender) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/starlette.py", line 130, in _create_span_call return await old_call(app, scope, new_receive, new_send, **kwargs) File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in call raise e File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in call await self.app(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in call await route.handle(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle await self.app(scope, receive, send) File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 66, in app response = await func(request) File "/usr/local/lib/python3.10/site-packages/sentry_sdk/integrations/fastapi.py", line 130, in _sentry_app return await old_app(args, **kwargs) File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 237, in app raw_response = await run_endpoint_function( File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 163, in run_endpoint_function return await dependant.call(**values) File "/app/embedbase/app.py", line 153, in add existing_documents = await batch_select( File "/app/embedbase/db_utils.py", line 36, in batch_select existing_documents = await asyncio.gather([_fetch(ids) for ids in hashes_to_fetch]) File "/app/embedbase/db_utils.py", line 34, in _fetch raise e File "/app/embedbase/db_utils.py", line 30, in _fetch return await vector_database.select( File "/app/embedbase/supabase_db.py", line 46, in select return req.execute().data File "/usr/local/lib/python3.10/site-packages/postgrest/_sync/request_builder.py", line 55, in execute r = self.session.request( File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line 821, in request return self.send(request, auth=auth, follow_redirects=follow_redirects) File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line 908, in send response = self._send_handling_auth( File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line 936, in _send_handling_auth response = self._send_handling_redirects( File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line 973, in _send_handling_redirects response = self._send_single_request(request) File "/usr/local/lib/python3.10/site-packages/httpx/_client.py", line 1009, in _send_single_request response = transport.handle_request(request) File "/usr/local/lib/python3.10/site-packages/httpx/_transports/default.py", line 217, in handle_request with map_httpcore_exceptions(): File "/usr/local/lib/python3.10/contextlib.py", line 153, in exit self.gen.throw(typ, value, traceback) File "/usr/local/lib/python3.10/site-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions raise mapped_exc(message) from exc httpx.ReadTimeout: The read operation timed out

hmm, nevermind, see where the issue is

@louis030195 could you please tell where the error was?


Latin Characters Break ParTool

"E:\Program Dosyaları" turns into "E:\Program Dosyalar─▒ " which causes ParTool to not find the file

It probably happens because the app uses codepage 1252 to encode the filenames inside the par file. That is because the original par files uses that encoding.
Anyway, you give very little information to identify the problem:

"E:\Program Dosyaları" is the ParTool path, the .par file path or the .par contents path?
What are you trying to do? Extract a par? Create it? Add files to it?
What is the exact error message?

It probably happens because the app uses codepage 1252 to encode the filenames inside the par file. That is because the original par files uses that encoding.
Anyway, you give very little information to identify the problem:

"E:\Program Dosyaları" is the ParTool path, the .par file path or the .par contents path?
What are you trying to do? Extract a par? Create it? Add files to it?
What is the exact error message?

-It's the broken part of the full path, which caused it to not find
-I am trying to extract
-It tells me the file is not found

I don't understand:

"E:\Program Dosyaları" is the ParTool path, the .par file path or the .par contents path?
-It's the broken part of the full path, which caused it to not find

"part of the full path" of what? The "ParTool.exe" path? The input .par path? Or the output directory path?

What is the exact error message?
-It tells me the file is not found

That's not an exact error message. I need you to copy and paste the output message of the app or make a window capture.

ERROR: "E:\Program Dosyalar─▒ (x86)\SteamLibrary\steamapps\common\Yakuza Like a Dragon\runtime\media\data\sound_en.par" not found!!!!
extract.bat
partool.exe extract "E:\Program Dosyaları (x86)\SteamLibrary\steamapps\common\Yakuza Like a Dragon\runtime\media\data\sound_en.par" "E:\Program Dosyaları (x86)\SteamLibrary\steamapps\common\Yakuza Like a Dragon\runtime\media\data\sound_out"

Thank you.
The problem are not the path characters, but Program Dosyaları (x86) is a shortcut to Program Files (x86)
Try with:
partool.exe extract "E:\Program Files (x86)\SteamLibrary\steamapps\common\Yakuza Like a Dragon\runtime\media\data\sound_en.par" "E:\Program Files (x86)\SteamLibrary\steamapps\common\Yakuza Like a Dragon\runtime\media\data\sound_out"

Doesn't seem to be working, even if it did, it would only solve this for Program Files, and not any other case where the path still has latin characters (for example, someone could have a SteamLibrary folder inside a folder that has a latin character)

I've created a directory named Program Dosyaları (x86) in my computer and the app extracts the .par files in it without problem. And I've also tried with some Spanish character in directory names and it works well, so the characters are not the problem… Maybe, you are missing some space in the path or something like that.

It's not missing because i have copy pasted the same path to my Windows Explorer and it asked me to open the file

I've only been able to reproduce the error when I wrote Archivos de programa (x86) (the Spanish alias to Program Files (x86)) at the command line, but using Program Files (x86) worked fine. All the other tests have worked too.
I'm sorry, but I can't fix a bug if can't reproduce it. If you drag&drop the .par file on ParTool.exe, does it works?

No error on that

Well, if drag&drop works, the problem must be in the command you are using.
My advise is to open a command line and write the command from scratch using tab to autocomplete the path instead of using a bat file.

Alright


New version: Tauri.create-tauri-app version 3.13.17

Created by 🥟 Dumplings in workflow run #3949.
Logs
Updated: 3.13.16 → 3.13.17
Submitting WinGet manifests

Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 874.)

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Tauri.create-tauri-app__DefaultSource\create-tauri-app.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\create-tauri-app.exe returned exit code: 1
Installation failed with exit code -1978335187
2024-03-26 08:22:19.097 [FAIL] Installer failed security check. Url: https://github.com/tauri-apps/create-tauri-app/releases/download/create-tauri-app-v3.13.17/create-tauri-app-i686-pc-windows-msvc.exe Result: 0x80004005

(Automated response - build 874.)


CircuitOperation has non-deterministic diagrams, making tests hard to write

I filled in a diagram test from output during one run:
    cirq.testing.assert_has_diagram(
        cirq.Circuit(
            cirq.Moment(
                cirq.CircuitOperation(
                    cirq.FrozenCircuit(cirq.X(cirq.LineQubit(0)))),
            ),
        ),
        """
0: ───Circuit_0x92933a3050ff70c4:───
      [ 0: ───X───              ]
        """)

But then it fails in the next run:
E       Diagram of actual circuit:
E       0: ───Circuit_0xd4e3300492f7877d:───
E             [ 0: ───X───              ]
E
E       Desired text diagram:
E       0: ───Circuit_0x92933a3050ff70c4:───
E             [ 0: ───X───              ]
E
E       Highlighted differences:
E       0: ───Circuit_0x███33█████f█████:───
E             [ 0: ───X───              ]

The issue is that the hash id is going into the diagram, but hash ids are not stable over time.
The hash is entirely based on the contents, which we can see anyways. I suggest just dropping it.

My reasoning behind including the hash:

It's easier to visually compare hash values than large CircuitOperations
Tests can embed hash(circuitop) of circuit.diagram_name() in the diagram string if necessary

An example from the CircuitOperation tests:
https://github.com/quantumlib/Cirq/blob/099fc740c53a3e54b2bece568f246c7324fd3e11/cirq-core/cirq/circuits/circuit_operation_test.py#L350-L361

Is https://github.com/quantumlib/Cirq/pull/4712 what you are thinking of?  I (in my ignorance) feel like CircuitOperation could benefit from:

Being a Gate instead of an Operation (CircuitGate?)
Getting a similar treatment as this: https://github.com/quantumlib/Cirq/pull/4201

Responded to (2) in #3605.
Your thoughts on CircuitGate mirror the early design stages for CircuitOperation - we strongly considered that option, but ultimately determined that CircuitOperation was the cleaner solution. This mostly boiled down to Circuits being more similar to Operations (both have defined qubits, while Gates do not) and Operations having better access to certain features (e.g. tags are defined at the Operation level).

I believe this issue is now fixed with merge of PR #4712


[zedd] SSL support

This is an issue related to the zedd branch which introduces a new way of remote/local editing.
Currently only HTTP + BasicAuth is supported for auth, SSL is required too

Implemented


Small README generator fix for footer

The footer appeared as a single line without full stops separating the sentences on some less flavoured Markdown generators (like Bitbucket's repository browser).
So I fixed by adding a couple of missing . and also added double spaces to the end of the lines so that line breaks are inserted.

Looks good, thanks :+1:


Option to force HTTPS in origin

References

Fixes #1721

Description
This PR adds a new configuration parameter forceHTTPSInOrigin to force to return the origin URL with the https:// protocol. This PR also uses the location.replace() function to redirect and make sure the browser can keep track of the browse history.
Instructions for Reviewers
In the environment variables, in the list of ui configurations set forceHTTPSInOrigin to true.
Test that the citation_pdf_url on an item page and make sure it always returns a URL that starts with https://.
To test the redirect change, download a file, and use the back button of your browser. You should end up back on the item page
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.

is this something that we should really manage on the dspace side? to force a redirect to https (that is a best practice that we always enforce with an HSTS header) there are specific directive on the web server / balancer / gateway that will be put in front of node in any production deployment. I'm just not sure that we should implement our own solution for area that are under the responsibility of other component. I would like to hear other opinions on that

The goal is not to force a redirect, the goal is to solve the problem that if there is something in front of the node server that redirects from a public https url to an internal http url, that the code we use to determine the origin server side as no way of knowing that. As a result, things that use that getCurrentOrigin method, such as the citation_pdf_url meta-tag get the wrong protocol.
We added this forceHTTPSInOrigin boolean rather than a whole separate "public" UI section with the entire URL, because I think two UI sections would be confusing, and we are able to automatically determine the host.
I considered defaulting it to true as using HTTPS is indeed best practice, but that would mean it wouldn't be automatically backwards compatible.

@atarix83 or @abollini : Have either of you had a chance to glance at this again?  I'd like to get this merged soon if possible, as it's already at +1.

@abollini : This issue is reproducible on the demo site right now.  Here's a page where the citation_pdf_url shows HTTP on the first load: https://demo7.dspace.org/entities/publication/1974d7d3-9648-4dbe-b083-56374f5314e4
The issue is specific to when the page is initialized on the server side.  If you look at client side generated content, this bug will never appear.  However, if the page is loaded via SSR (which is highly likely when search engines are indexing), then this bug will appear.

@artlowel  @tdonohue  we check further and indeed the issue comes when the SSR kick-in. I was initially fooled by the reload that happen automatically in the browser, so it is much better test via the command line.
That said, I'm still not convinced by the fix. The X-FORWARDED-FOR headers are used exactly to manage these scenarios and there is also a proto header here.
I have done a quick search and found this npm library that is used to abstract the access to the request to get into consideration these forward headers if available. Can you take a look to that? could this provide a general solution that would not require extra configuration and will work with a proper configuration of the reverse proxy?
https://www.npmjs.com/package/forwarded-for

@artlowel : @abollini 's latest comment reminded me about the X-FORWARDED-PROTO: https header, which is also required to be set on the backend (REST API) whenever it is run behind a proxy.  See our notes at https://wiki.lyrasis.org/display/DSDOC7x/Installing+DSpace#InstallingDSpace-MyRESTAPIisrunningunderHTTPS,butsomeofits"link"URLsareswitchingtoHTTP?
We may want to double check whether that header has any impact on the Angular/Node side as well.    As it might be that, if we require the proxy to set X-FORWARDED-PROTO: https (just like on the backend), then our Angular code could just access that header to determine if it needs to use HTTPS or HTTP URLs.

@abollini @tdonohue These x-forwarded headers are already present. This logged the request headers in the node server with the current httpd config on demo7.dspace.org and got the following:
{
  host: 'demo7.dspace.org',
  accept: 'text/html, application/rss+xml, application/atom+xml, text/xml, text/rss+xml, application/xhtml+xml',
  'accept-encoding': 'gzip,deflate',
  'user-agent': 'REDACTED',
  'x-forwarded-proto': 'https',
  'x-forwarded-for': 'REDACTED',
  'x-forwarded-host': 'demo7.dspace.org',
  'x-forwarded-server': 'dspace7-demo.atmire.com',
  connection: 'Keep-Alive'
}

The fact that they're present means they are not automatically taken into account when simply retrieving the protocol of the request.
@abollini The package you suggested wouldn't be a solution, it will give you the IP of the original requester, but says nothing about the origin or protocol of the public url
Something that could work is to test for the presence of those x-forwarded headers, and use their values instead of the default request properties for host and protocol if they exist. I've created https://github.com/DSpace/dspace-angular/pull/1850 which takes that approach. We should discuss which one to go ahead with in the meeting today.

Closing, as this is replaced by #1850 (see discussion in that ticket)


Skill tree generator does not open 2nd time, after its 1st run

Hello,
So, if after i run STG (skill tree generator), it will not open second time, information window will pop-up "the STG is already open". It is not opened though, there is no secondary window "hidden" under some PoESkillTree itself, or some other window.
The way to reproduce
1)Select tools -> nodes - STG. It does not matter whether its new build, or saved.
2)Press "Run". It does not matter which settings are chosen.
3)After generation complete, close progress bar, press X button to close STG window.
4)Select STG again -> information window will pop-up.
PS. On a completely side note - that design, when screen is divided in half (with 2 white areas on top and bottom) when opening settings for example, is really bad. Not only it takes a couple of second to play, it also flashes, which is really troublesome, considering whole colour theme is dark, and then, ther is suddenly a flash and 2 seconds later, 2 white patches.

First problem has been fixed, but it has not been released.
@brather1ng may have some more information on the 2nd issue.

The dialog opening and closing animations have been shortened, there is only 200ms of animation now and the background overlay doesn't fade in and out but just is there.
The animation isn't flashing for me and I don't think it is too light in contrast with the background. Maybe the flashing is caused by the animation not being completely smooth? Anyway, that fading animation will be gone with the next release, so that should hopefully resolve your issue. If it doesn't, there isn't much we can do as the overlay comes from MahApps.Metro and can't be adjusted by us.

I haven't really been in a situation where the dialog boxes were too light,
but I use f.lux. I could see how white on the dark would be a bit
off-putting though.
On Sep 22, 2016 2:17 AM, "brather1ng"<EMAIL_ADDRESS>wrote:

The dialog opening and closing animations have been shortened, there is
only 200ms of animation now and the background overlay doesn't fade in and
out but just is there.
The animation isn't flashing for me and I don't think it is too light in
contrast with the background. Maybe the flashing is caused by the animation
not being completely smooth? Anyway, that fading animation will be gone
with the next release, so that should hopefully resolve your issue. If it
doesn't, there isn't much we can do as the overlay comes from MahApps.Metro
and can't be adjusted by us.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/EmmittJ/PoESkillTree/issues/378#issuecomment-248851054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIbbaTPAWaLCbU8mCqCei9nUKkrVlogks5qskebgaJpZM4KDhY7
.

Thanks for answers.
Yeah, that second problem may not be PROBLEM, but it really bugged me out when i first saw it.
Thanks again.

Closing as the fix was released.


feat: supporting OR queries

Codecov Report

Merging #318 (08ee85b) into main (3745a85) will increase coverage by 0.87%.
The diff coverage is 60.81%.

@@            Coverage Diff             @@
##             main     #318      +/-   ##
==========================================
+ Coverage   27.93%   28.80%   +0.87%
==========================================
  Files          60       60
  Lines        6376     6408      +32
==========================================
+ Hits         1781     1846      +65
+ Misses       4382     4346      -36
- Partials      213      216       +3

Impacted Files
Coverage Δ

query/filter/comparison.go
28.57% <0.00%> (ø)

query/filter/filter.go
58.33% <0.00%> (+8.97%)
:arrow_up:

server/services/v1/search_reader.go
0.00% <0.00%> (ø)

query/filter/selector.go
30.23% <66.66%> (+18.60%)
:arrow_up:

query/filter/logical.go
56.77% <93.33%> (+29.50%)
:arrow_up:

query/search/search.go
29.62% <100.00%> (ø)

Continue to review full report at Codecov.

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


DEP: Remove old Yahoo code

We don't use this code anywhere that I'm aware of (grepped through Zipline in emacs)

Coverage increased (+0.04%) to 87.411% when pulling d3e86b075f6996b120b41d0a1c8c43af7245fe5e on remove-old-yahoo-code into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

Coverage increased (+0.04%) to 87.411% when pulling d3e86b075f6996b120b41d0a1c8c43af7245fe5e on remove-old-yahoo-code into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

Coverage increased (+0.2%) to 87.537% when pulling ce29cf5e937bfd5df4c46ece69fe0e17bc8e6a5f on remove-old-yahoo-code into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.

Coverage increased (+0.2%) to 87.537% when pulling ce29cf5e937bfd5df4c46ece69fe0e17bc8e6a5f on remove-old-yahoo-code into 42b32007dd6df8b2ff8d75bd1b90bf1a902da064 on master.


Corrupted ODF summary files

This has been doing my head in for ages now, for some reason SAS ODF summary files on Apollo keep getting broken, which means I can't generate new cifs. It hasn't happened in a while, but here we are again:

I had the bright idea to look at when the file was last modified:
(conda4-xcsim) dt237@kraken:~$ date -r /home/dt237/apollo_mnt/xmm_obs/data/0677670135/odf/2218_0677670135_SCX00000SUM.SAS
Fri 26 Mar 23:03:42 GMT 2021
Now, I sort of assumed that I was causing these problems somehow, that maybe some SAS process I use writes to these files and sometimes gets interupted, but I don't think I was running anything at that time on Friday. As far as I can recall I didn't set anything running on Friday, and by that time I wasn't in any state to do so...
So, I've asked Reese if we can see who last modified file, and failing that I'll ask XCS if anyone was running anything then. Beyond that I'm sort of out of ideas.

We don't know why this is happening, but I think to get rid of my dependance on existing ODF summary files I will figure out how to call cifbuild with just an observation date, that way I can do away with needing ODF summaries entirely.


WIP: AutoOpt Fusion

Enhance AutoOpt with SubgraphFusion. Includes all the tools necessary as well as extensions to Subgraph Fusion

Merged implicitly by adding it in parts.


Is mask a background image? I output the mask. It seems that the value of most positions is 1. Which positions should be 0?

Is mask a background image? I output the mask. It seems that the value of most positions is 1. Which positions should be 0?

Hi! Please check #231 about mask usage.


Featured Image in Scheduled Releases Doesn't show

Regression in the Scheduled Releases area.

Possible issue with the server I tested this on earlier. Needs more testing but it works on my main Development machine.

Chocking this up to an issue with the system I was testing it on.


Photoshop CS5 Extended not finding the plugin

I know this is a really old version but it performs ably for my needs. If there is a simple switch that can enable support for this version, I'd love it if you could fix the plugin?
Thanks,
B

I do not know what changes need to be made to support older versions and as stated in issue #6 I do not have access to anything but the latest Photoshop versions so I cannot adapt WebPShop to anything but these, sorry. However feel free to build and fix WebPShop if you are able to.

Thanks for the reply, I thought it would be the case :) My coding skills
are sub-optimal for doing my own version...
All the best,
B
On Tue, 18 Aug 2020 at 11:44, y-guyon<EMAIL_ADDRESS>wrote:

I do not know what changes need to be made to support older versions and
as stated in issue #6 https://github.com/webmproject/WebPShop/issues/6
I do not have access to anything but the latest Photoshop versions so I
cannot adapt WebPShop to anything but these, sorry. However feel free to
build and fix WebPShop if you are able to.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webmproject/WebPShop/issues/17#issuecomment-675377970,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABK73LKGA4JWSZPNZKT6RO3SBJEPVANCNFSM4QDDNPWA
.

--
Ben Vost
Skype: BeeVee_at_work
Mobile/Cell/Portable: +33 628 51 56 43

If you feel brave enough one day, here are a few leads you might try:

Add #define LOG_TO_FILE "path/to/debugging/log" in WebPShop.h to print what is going on and potential helpful error messages to a file.
Add tags that could improve backward compatibility in the resource file WebPShop.r.
See if the entry point is even called.
Take a look at Photoshop logs, in case there is something valuable in them (never happened to me).
Compare with a plugin that works fine (such as Telegraphics WebPFormat).

Well, I can confirm that the Telegraphics plug in for Photoshop CS4 works fine and creates a file.  However when the file is uploaded to a web server, the server is unable to serve it up in HTML pages.  Here is a link to the webp image https://www.dogsindanger.com/doggie/images/dogsindangerDog_mobile.webp.  What's the deal?

See the other thread.


RSS Feed Subscription Integration

RSS Subscription Integration
This Feature introduces the ability for a RAG workspace to subscribe to an RSS Feed, check for new posts daily and send new posts for Website Crawling for the RAG Workspace. This can showcase simple automation for consuming data for RAG / Semantic Search.

How it Works
Users can subscribe to a RSS Feed by navigating to a desired RAG workspace and navigating to the "RSS Feeds" tab on the workspace documents table.
Clicking the "Subscribe to RSS Feed" button on the table or by clicking "Add RSS Subscription" in the "Add Data" drop down will navigate the user to the RAG Workspace's "Add Data" view, with the RSS Feeds Tab selected.
The user can paste a link to an RSS Feed and provide a title for the feed.

Once the user successfully subscribes to the RSS Feed, the feed will be visible in the "RSS Feeds" tab of the RAG Workspace documents table.
The RSS Feed is then sent to be checked for posts, which are then added to the RSS Feed DynamoDB table.
An independent scheduled lambda checks the RSS Feed table for posts that are in a pending status and sends up to 10 pending posts for to be Website Crawled into the RAG Workspace. The RSS Feed Check & Post Crawling were designed to be decoupled to provide a fixed rate-limit for the number of websites being crawled. Additionally, the architecture leverages an EventBridge schedule for each post with a flexible timeframe for execution. This can also help distribute work over time if the user has subscribed to a high-volume of feeds or feeds that have a high-volume of posts. Since the posts are ingested using the existing crawl websites functionality, the crawled blog posts will show under the websites once they are consumed.
Additionally, a user can click on a RSS Feed subscription from the documents table on the RAG workspace, which takes the user to a RSS Feed details page. The page gives the details of the RSS Feed subscription and the posts that have been found from the feed. It also provides the user with the ability to enable/disable a subscription.

Finally, if a user deletes a RAG workspace, the RSS Subscription's EventBridge schedule is deleted and any RSS Feed or Post data relating to the workspace is removed from the Documents DynamoDB table.

High-Level Technical Changes/Additions
The goal with changes was to leverage as much of the existing design patterns/structures.

3 New lambdas are added.

The first lambda runs once daily by EventBridge rule. This lambda pulls all RSS Feeds that are currently enabled across any workspace. It loops through each and Invokes an RSS Ingestor Lambda function.

The second lambda is triggered by the previous lambda. This lambda will parse the RSS feed for the provided workspace_id/feed_id. The lambda will add newly detected posts to the documents table as rsspost type documents. The documents have a status of pending. If a post already exists, the lambda will not overwrite it to preserve the original data and ensure process RSS feed posts aren't re-processed.

The third lambda runs every 10 minutes. This lambda queries the documents table for documents with a status of pending and a document type of rsspost. It then submits each post for website crawling. The website crawling will be associated with the workspace the RSS feed/posts exist.

The design of the three lambda architecture was based on a few factors:

The first lambda, which triggers one or more of the second lambda was designed to be the single RSS post check trigger, keeping scheduling simple. The decision to have each RSS feed execute feed crawling in an isolated lambda was to ensure scalability. While feed parsing is relatively quick, if users subscribe to a high volume of feeds, an execution time out could occur.
With the second lambda executing feed parsing on one feed per execution, it can help mitigate failures due DDB capacity issues (in high volume scenarios) by allowing smaller retries w/ exponential back off. Additionally, debugging and tracing is easier with isolated feed parsing executions.
The third lambda is designed to ensure the Website Crawling SageMaker Inference endpoint isn't overwhelmed. Leveraging DDB Documents Table as a processing queue, it grabs a max of 10 websites every 10 minutes.

This PR is relating to #226.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Great contribution @flamingquaks! looking into it

@bigadsoleiman thanks!
I'm hoping to have finally tweaks done today. I had some over-engineering introduced which I'm yanking out. The final push should be today and I'll take the PR out of draft.

I've moved the PR back to ready!
From the original PR state to now, I pulled out the rss specific logic and leveraged document logic where possible, adding handling by the document_type of either rssfeed or rsspost depending on the case.
I've also simplified the scheduling/feed parsing/crawl queuing, which I updated in the PR description.
Feature-wise, I've also removed the original hard-coded "follow links" and "link limit" parameters the RSS logic was using when sending a website for crawling. The user can now, in the UI, set follow links and limit, just like on the Add Website to Crawl page.
On the RSS Feed details page, the user can toggle the RSS subscription on/off. The user can also modify the "follow links" and "link limit" values applied to the RSS feed, which will apply the settings to any newly found RSS feed posts.
The original requirement to bump the aws-cdk-lib version has been removed as I removed the aws-solutions-construct I was leveraging in the CDK stack.
Within the requirements.txt there's an added dependency for feedparser https://pypi.org/project/feedparser/

@massi-ang, I moved RSS Subscription to its own construct.


How to enable disqus comment?

In params.tom, I set the following parameters to enable Disqus but Disqus is not shown at the bottom of my pages.
enableComment = true
disqus_shortname = "xxxxxx"

Are there any other settings required?

Nevemind, it is working now. I don't know what happened.


Calling container.delete() then calling sync() results in an AttributeError

Example:
>>> new.delete()
>>> new.status
'Stopped'
>>> new.sync()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/paul/.virtualenvs/demoer/lib/python3.5/site-packages/pylxd/models/_model.py", line 151, in sync
    response = self.api.get()
  File "/home/paul/.virtualenvs/demoer/lib/python3.5/site-packages/pylxd/models/_model.py", line 119, in __getattribute__
    return super(Model, self).__getattribute__(name)
  File "/home/paul/.virtualenvs/demoer/lib/python3.5/site-packages/pylxd/models/container.py", line 71, in api
    return self.client.api.containers[self.name]
AttributeError: 'NoneType' object has no attribute 'api'

hmm, I guess pylxd could throw a nicer error than just AttributeError, maybe explicitly tell that the container was deleted.
On the code side whenever a container is deleted the client attribute is set to None. This could be fixed by just creating a boolean "deleted" and setting it to True. So the next time someone tries to do an operation on a deleted container, it could throw a nicer complain.
I can cook something up if the maintainers think that's an acceptable solution, though I am not fully sure if this project is being maintained or not. @stgraber do you know ?

@om26er yes, the project is definitely being maintained; sorry if we've been a bit slow to get around to this.
It's probably better to leave the client being set to None and catch the AttributeError as non recoverable, and provide something a bit more useful (such as adding something like a ModelDeleted exception to exceptions.py)
I'll tag this as an enhancement.  If you've already got something then please do submit a PR.


Resize unsplash image width query to reduce download for smaller devices

Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[X] There's a clear use-case for this code change
[ ] Commit message has a short title & references relevant issues
[ ] The build will pass (run yarn test and yarn lint)

More info can be found by clicking the "guidelines for contributing" link above.
I've seen this issue come up in a few places:
https://github.com/TryGhost/Casper/issues/644
https://forum.ghost.org/t/unsplash-support-for-img-url-size/7081
Apparently, it's been done, but I haven't seen it. This change is to apply the standard local size to external URLs that include a width parameter (such as Unsplash and Cloudinary).

Hi @isonaj, thank you for the code. It was a missing feature for years.
This update somehow brakes /amp pages on 3.35.2. The server sends 404 for "amp" pages. I'm not familiar with how the pages rendered in Ghost, however, it might be related to "&" replace code.

Any update on this, this would be really awesome to incorporate into the project?

I am so happy if we could merge this PR...I am newbie in Ghost, note sure I could help.

@isonaj Thank you for this PR! I apologise for the delay but this is on my list to review ASAP.

Hi, this issue has been closed, even though this feature is necessary to have a proper search engine ranking when using the Unsplash featured image.

I second that. Right now, the workaround is to download the image from Unsplash und upload it so image sizes will be applied properly.


common-update-scripts: fixup for current/latest nix hash output

Courtesy of @jtojnar, thanks!
See https://github.com/NixOS/nixpkgs/issues/54962#issuecomment-459429698
Motivation for this change
Fix common-update-scripts to work with latest Nix
(as packaged by nixpkgs).
See linked issue for discussion.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Thanks.


Updated image of APIM protocols pane to match current.

The current pic no longer matches what Azure shows. Updating the image to match.

@mikebudzynski Could you review the proposed update to your article? If approved, the pixelation should be replaced either with generic text or with whitespace (pixelating to obscure personal data doesn't align with the Microsoft standard for images). Images also should be no more than 1200 pixels for any dimension.
Thanks!

@dcrreynolds, thanks for your contribution. Could you edit the image as per @ktoliver 's  message?
Adding @dlepow
#sign-off

@mikebudzynski: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Hey! Happy to update the pic to match the standards. I will try to get to it in a bit.

Update the pic! Now with 100% more standards compliance (I hope).

Looks good! Thanks @dcrreynolds and @mikebudzynski


The final argument to magicString.overwrite(...) should be an options object. See https://github.com/rich-harris/magic-string

Hi! I tried to squash this file and got an error:

The final argument to magicString.overwrite(...) should be an options object. See https://github.com/rich-harris/magic-string

I tried to run squash dist/redom.js > dist/redom.min.js – did I do something wrong?

Nope, you're not doing anything wrong. The underlying library magic-string changed the arguments to one of its methods. The old way of calling it still works for now, but prints this warning. This has actually already been updated in the master branch of butternut, but it hasn't been released yet.

Oh, I didn't realize it still works – sorry 🤣

can not reproduce on master

when will master be released?

Any news on this..?

hello,
I still get this warning...
thanks

Nope, you're not doing anything wrong. The underlying library magic-string changed the arguments to one of its methods. The old way of calling it still works for now, but prints this warning. This has actually already been updated in the master branch of butternut, but it hasn't been released yet.

Still not released. Is this project still actively maintained including the release process?


e2e: Allow setting custom retry strategies

What does this PR do?
Allows changing the default retry strategy for e2e infra setup, including the possibility to change the options applied to the stack.
Motivation
KMT had its own specific retry logic on top of the e2e retry logic, this PR allows KMT to override the defaults.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Tested locally launching stacks with KMT, including the configuration changes on retries (see https://github.com/DataDog/datadog-agent/pull/28990)

/merge


Packaging Request: ocamlPackages.ppx_python

Project description
Generate functions to convert OCaml values to/from Python values.
ppx_python is a PPX syntax extension that generates code for
converting OCaml types to and from Python. This uses
the pyml OCaml bindings for Python
to start a Python runtime, create the Python objects or analyze
them.
Metadata

homepage URL: https://opam.ocaml.org/packages/ppx_python/ppx_python.v0.13.0/
source URL: https://github.com/janestreet/ppx_python
license: mit
platforms: uncertain

Addressed by https://github.com/NixOS/nixpkgs/pull/145220


Unable to reproduce jeep watercolor example in demos/paper

Thank you for the excellent codebase and really useful research contributions. I was trying to replicate the jeep watercolor example given in demos/paper using the same hyperparameters (jeep_watercolor.yaml), but seems a bit degraded compared to what’s reported. Can you verify these hyperparameters are correct in the yaml?
I am able to reproduce some other examples closely including jeep posche.

I run again. The result is correct on my 3090 GPU.
As you can reproduce the Porsche example, your code should be correct.
I am not sure what the problem is with your results, but I do observe some minor changes using different random seeds or image frames.
There is still randomness exist in the computation pipeline

Thanks for your quick response.
I am running your code w/o any changes on a V100 gpu. I use the same hyperparameters as in jeep_watercolor.yaml. Below are the results I get. On the left I set the strength watercolor to be 3, instead of default 10 (shown to the right) as in the config file. As you see, the details are missing when using the default config (right).

Also, I see some differences in the image you posted above and the main result of the demo/paper (here). I understand that there is randomness in generation, but the change in details is high is these different versions. I am trying to get a faithful baseline to what's reported in the paper. I would highly appreciate if you can post the exact config file that you used to generate those results. Thanks in advance!
Again, great work and excellent repo!


'Access-Control-Allow-Origin' wildcard is being set in the client - this seems backwards?

It might've been a result of cargo-cult coding on my part. I'm gonna research this and fix it if it's an actual problem.

This needs to be reflected into Egg Trainer.


Checkpoint: Record Collection

Challenge Checkpoint: Record Collection has an issue.
Considering secong condition of update function,
'value' is not pushed in to the array of prop, it saved as string instead.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
// Setup
var collection = {
    2548: {
      album: "Slippery When Wet",
      artist: "Bon Jovi",
      tracks: [
        "Let It Rock",
        "You Give Love a Bad Name"
      ]
    },
    2468: {
      album: "1999",
      artist: "Prince",
      tracks: [
        "1999",
        "Little Red Corvette"
      ]
    },
    1245: {
      artist: "Robert Palmer",
      tracks: [ ]
    },
    5439: {
      album: "ABBA Gold"
    }
};
// Keep a copy of the collection for tests
var collectionCopy = JSON.parse(JSON.stringify(collection));

// Only change code below this line
function update(id, prop, value) {
  //collection[i];
  if (value !== "" && collection[id][prop] !== "tracks") {
    collection[id][prop] = value;
  } else if (value !== "" && collection[id][prop] === "tracks") {
    collection[id][prop].push(value);
  } else if (value === "") {
    delete collection[id][prop];
  }
  return collection;

}

// Alter values below to test your code
update(1245, "tracks", "Addicted to Love");

You need to test prop to see if it is equal to "tracks".  Right now you're testing then CONTENT of whatever collection[id][prop] is pointing to, which will never be "tracks".
In the future you should ask for help in the Help Chat Room.  Issues are for bugs with the site.
Thanks and Happy Coding!

Thanks!! Got it!


Break list layout

I found that using WrappingHStack in List and reverse the order of the content dynamically will make my content padding disappear. Back to normal when I switch back to HStack. 😢

please post a simplified example

probably a duplicate of #15


func_80097E28

The function matches.

You need to rebase from MediEvilDecompilation's main branch. so that you only have 1 commit instead of 16.

You need to rebase from MediEvilDecompilation's main branch. so that you only have 1 commit instead of 16.

Ok.


feat: improve influxdb line document

In Table Management, there is an example of creating a monitor table. In the following Write Data/SQL section, there is an example of inserting data into the monitor table. On the other hand, in Write Data/InfluxDB Line section, there is an example of inserting data into the system_metrics table. I guess that users should read Write Data/InfluxDB Line after they read Table Management. I think this might confuse them.
So, this PR replaces system_metrics with monitor so that users can immediately execute a insert command after reading Table Management.
In addition, this PR appends a InfluxdDB Line protocol reference in the Write Data/InfluxDB Line section so that users could easily understand about the InfluxdDB Line protocol.

Thank you for pointing that out. You are correct, it's better to maintain consistency with the table name in the user guide.

@nicecui please take a look.