Problem with French IPTV

Which version are you using?
v0.13.3 (docker)
Which operating system are you using?

OS

[ x] Linux
[ ] Windows
[ ] macOS

Architecture

[x ] amd64
[ ] arm64
[ ] arm7
[ ] arm6

Describe the issue
My ISP (Free, France) is offering IPTV service based on RSTP/RTP. Rstp-simple-server, in proxy configuration, can't dectect this IPTV source: ERR: no UDP packets received recently (maybe there's a firewall/NAT in between)
debug_rtsp-simple-server.txt
This source is UDP/RTP with payload type 33 (MPEG II TS). It works with VLC.
Rstp-simple-server, same configuration, works in proxy mode with a different RSTP/RTP source (payload type 97 for ex).
Describe how to replicate the issue
You need this source to replicate. See attached dump with the ko source, and a good one.
Did you attach a network dump?
captures.zip

Hello @aler9 , thanks for your help.
Here the requested pcap dumps: captures2.zip
I hope it can help you.

Hello @aler9 , thanks for your help.
Here the requested pcap dumps: captures2.zip
I hope it can help you.

Hello @aler9,
FYI, I've tried last release with the undockerized version, same issue:
./rtsp-simple-server
2021/02/01 14:34:31 [I] [0/0/0] rtsp-simple-server v0.14.0
2021/02/01 14:34:31 [I] [0/0/0] [UDP/RTP listener] opened on <IP_ADDRESS>:8000
2021/02/01 14:34:31 [I] [0/0/0] [UDP/RTCP listener] opened on <IP_ADDRESS>:8001
2021/02/01 14:34:31 [I] [0/0/0] [TCP/RTSP listener] opened on <IP_ADDRESS>:8554
2021/02/01 14:34:31 [I] [0/0/0] [path proxied1] [rtsp source] started
2021/02/01 14:34:31 [I] [0/0/0] [path proxied1] [rtsp source] connecting
2021/02/01 14:34:31 [I] [0/0/0] [path proxied1] [rtsp source] ready
2021/02/01 14:34:41 [I] [0/0/0] [path proxied1] [rtsp source] ERR: no UDP packets received recently (maybe there's a firewall/NAT in between)

Hello, what happens is a very strange thing.
The server tells the client that the UDP ports are 32792-32793
RTSP/1.0 200 OK
Cseq: 3
Server: fbxrtspd/1.2 Freebox RTSP server
Public: DESCRIBE, OPTIONS, SETUP, TEARDOWN, PLAY, PAUSE
X-Freebox-ANPT-Range:<PHONE_NUMBER>/4004334049/4004334049/4004334049
Session: H20gKSxsvfwCX0i;timeout=30
Transport: RTP/AVP;unicast;mode=play;destination=<IP_ADDRESS>;client_port=48230-48231;server_port=32792-32793

But then the server uses two completely different ports to send packets (48230-48231)...
This happens with both VLC and rtsp-simple-server, but the former doesn't check the sender ports, while the latter does for security reasons.

In my opinion you can do 3 things:

check whether your NAT changes UDP source ports
if this is not the case, write to the TV provider and tell them to fix their RTSP server to send the right server port
anyway, use TCP

paths:
  mypath:
    source: rtsp://
    sourceProtocol: tcp

I'm sorry but i don't want to remove the security check of the UDP source ports.

Hello @aler9
Thank you very much for your help!
Indeed, I focus my troubleshooting on client_port and I never checked server_port. You are right, server_port doesn't match UDP/RTP source port. And it's not my NAT that changes source ports.
Based on RFC2326:
server_port:
        This parameter provides the unicast RTP/RTCP port pair on
        which the server has chosen to receive media data and control
        information.  It is specified as a range, e.g.,
        server_port=3456-3457.

My understanding is that these ports are not mandatory used for UDP/RTP source ports... but let's say that this part of the RFC is
ambiguous.
I've found a dicussion on the TV provider forum about this issue, based on RFC, they won't change the RTSP behavior :(
Is there any chance that you implement a disable "security check" option ?

Hello, in my opinion the best solution consists in hacking the code and compiling a custom executable by yourself.
Edit this file, go to this line: https://github.com/aler9/rtsp-simple-server/blob/2b4d22589ca05559afbbce2635ca0a4fa463286b/internal/sourcertsp/source.go#L124
Add
AnyPortEnable: true,

And recompile with Go (go build . ) or Docker (make release)

Hello, thanks for your answer. I'll give a try to compile the code myself but I'll loose the ability to easily deploy and maintain the app.
What specifications are you reffering to ? According to RFC2326, my IPTV provider is not breaking anything....


Added nutrition subtopic to allow nutrition fact-related queries

added nutrition values to mock product data and added match pattern and message handler for nutrition queries

New PR on product_store_query

added unit test
fixed database

@paulgray01 @SanaBob The other team wants to talk to you to migrate to use DialogFlow for NLP instead of regex. So we will have this PR as pending until the discussion is resolved.

Added endpoint for Diagflow integration


Add Grafana Dashboard

including information about the number of requests, runner and executions and their durations.
Closes #110
What do you think? Is all the important information visualized? Could the structure of the dashboard be improved?
One visualization I am missing is how much time it takes Nomad to start the runners. This information can be added in the runner manager but it is not bound to a request of CodeOcean. This problem applies also for the statistics idle_runner and prewarming_pool_size - we send the same data with all incoming requests but not at the moment the change happens. That's the Prometheus way but not the Influx'... Maybe we should reconsider our decision to restrict the access to the influx client and to not provide it as singleton?!
I have no idea how to normalise the histogram.

I really like the visualizations! They are so nice to look at (and contain much information), will definitely recommend them!
⭐ ⭐ ⭐ ⭐ ⭐
Some suggestions:

Length of the smallest bar for the bar charts should be increased
Naming of environments: could we add a mapping, even if done manually once?

Regarding your questions:

One visualization I am missing is how much time it takes Nomad to start the runners. This information can be added in the runner manager but it is not bound to a request of CodeOcean. This problem applies also for the statistics idle_runner and prewarming_pool_size - we send the same data with all incoming requests but not at the moment the change happens.

Wouldn't it be possible to send these events anyway as soon as they occur, even outside of the usual request-response cycle?

Maybe we should reconsider our decision to restrict the access to the influx client and to not provide it as singleton?!

Fine for me ;)

I have no idea how to normalise the histogram.

I think it is fine with the current visualization. I tried a few things in Excel, but unfortunately was unable to proceed. First, Excel does not allow multiple data sources in one histogram (as required for the multiple execution environments). Also, I was unable to find a good way of normalizing them individually. Therefore, and as the height also includes some information, I would suggest keeping it with the current visualization for now. We might only consider adding an overall gauge of the total usage of Poseidon, as discussed earlier today ;)

Wouldn't it be possible to send these events anyway as soon as they occur, even outside of the usual request-response cycle?

Yes, for example we could notify influx for every created or deleted object in our storage. This would include idle_runners, used_runners, aws_executions, nomad_executions, environments. To do so I would start #89 now.

Naming of environments: could we add a mapping, even if done manually once?

I could not find a Grafana feature for this (as we do not want to map values but labels), but we could map this via Flux and an additional data source (as found in some blog post). Should we therefore install the CSV plugin or use a database for the manual mapping between environment ID and environment name?

Yes, that might work. Alternatively, we could also wait for #89 and those additional events. Then, we could send those "static" information once, e.g., when synchronizing an execution environment and just send the name together with the desired prewarming pool size, RAM, ... to Influx. As they can only be changed with an update / synchronisation, this could also help us to build a corresponding data store.

Really great dashboards, I just used them again. One Two final questions:

Could we increase the aggregation for the line charts if a window larger >= 6hours is seen? In that case, we might want to switch to 5 min instead of 1 min.
Could we use the full page width for the first histogram "Execution Duration" in order to make that easier to use?

Then, we could send those "static" information once, e.g., when synchronizing an execution environment and just send the name together with the desired prewarming pool size, RAM, ... to Influx

Update Environment information in extra measurement with filter for one year.

Then, we could send those "static" information once, e.g., when synchronizing an execution environment and just send the name together with the desired prewarming pool size, RAM, ... to Influx. As they can only be changed with an update / synchronisation, this could also help us to build a corresponding data store.

With the new commit the data should be saved in another measurement. In the next pull request we could adjust the Grafana dashboard to create the mapping in order to display the environment image as name..
What else is an open ToDo here:

collect data to measure how long Nomad takes to start an allocation?


Symfony installer for 3.4 is broken

Hello,
there was a mistake on the 3.4.99 tag. The 3.4 branch is ok but the 3.4.99 tag targets the 5.2 version. Therefore symfony new and composer create-project with 3.4 version both create a project on Symfony 5.2.
Regards.

Fixed.

Hello,
It seems that this issue is reappeared.
Runnning symfony new my_project --version=4.0 (which runs composer create-project symfony/skeleton my_project 4.0.*) creates a Symfony 5.1 project

Indeed, /cc @nicolas-grekas

https://flex.symfony.com/versions.json is missing entries for 4.0, that's why.

Fixed

Note that 4.0 is not maintained anymore, you should use 5.1 or 4.4 instead.

Yes, but it was for a training :D thanks for fixing it


Possible Update Check on Portable Startup Connecting with MessengerforDesktop dot com instead of .ORG

Operating system: Windows 10
Messenger for Desktop version:  2.0.9

First startup of portable app initiated connection:
Domain: updates.messengerfordesktop.com
IP: <IP_ADDRESS> - California

Since messengerfordesktop.org is supposed to be the correct service, with messengerfordesktop dot com being the adware/spyware hijacking that was experienced, can you explain why that's still listing messengerfordesktop.com  in the connection?

I did a search through the repo, and I see src/package json still pointing to messengerfordesktop.com, and several other places referenced. Is this correct as I understood it should be .ORG now?

I'm no longer running due to this. To many issues with possible privacy or spyware. This is an open source project but by default captures some metrics for the original "company", so I'm not going to keep considering this truly as a community based open source tool. Too risky. I sure hope for more clarity on this soon, or might just have to eventually fork and modify if I can get the time.

There are no privacy or spyware threats. messengerfordesktop.com has installer ads, but other than that, both websites are the same.
There's only one updates server (mine, only I have access to it) and that's updates.messengerfordesktop.com.
The CI scripts and logs are public: when I push to brach deploy, the CIs build the app and then upload it to GitHub — that's all, and that's automatic and transparent. The updates server is also open source and is simply a node.js server that redirects to GitHub releases, but of course since only I control it, you can't say it's fully trusted / transparent.
If you require a higher level of privacy and security then I recommend that you fork the app, remove every connection to my servers (i.e. analytics and updates), and build it yourself. For personal use, you don't need to sign it or any hassles that the CIs do.


Move "Static asset delivery optimization" out of Blazor section

Description
The "Static asset delivery optimization" feature isn't Blazor specific. Any ASP.NET Core apps with static assets can benefit from this feature. Basically, if you use the old static files middleware you should consider using the new static assets support instead. So, let's move this section somewhere more top-level.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/release-notes/aspnetcore-9.0?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/release-notes/aspnetcore-9.0.md
Document ID
4e75ad25-2c3f-b28e-6a91-ac79a9c683b6
Article author
@Rick-Anderson

Oh, I see now that the Blazor bit is just a pointer to a different section. My bad!


updating docs for compiling

This may be a bit strong of an opinion for the documentation.  But  Debian10 is the only current working docker solution we have.  I'd like to focus on it during incubation so we can start releasing convenience binaries with future releases.  Please review the wording and make sure it makes sense and that you agree with it.
cc @thinker0

This may be a bit strong of an opinion for the documentation. But Debian10 is the only current working docker solution we have. I'd like to focus on it during incubation so we can start releasing convenience binaries with future releases. Please review the wording and make sure it makes sense and that you agree with it.
cc @thinker0

I believe ubuntu work, but it is ok if decide that debian 10 is our main platform.

@nwangtw I was under the impression ubuntu18.04 was not working.  I'm fine with either container.  I think we should pick one as the official released convenience binary.  It's too much to maintain multiple distros well.  I'll change the wording around before merging.

I've updated the documentation a bit.  Please give another review before we merge.

If there are no concerns with the changes I'll merge the changes after 24 hours passes.


add ability to ship remote only branch

@kevgo @allewun @ricmatsui
fixes #593

Nice! :ship: after the comments are addressed.
As a separate change, what do you think about making fetch something that is automatically done by the GT framework unless the command sets a variable to omit it? I think most commands need to fetch anyways.

I don't know about fetch becoming a part of the framework because each command can optimize when it should actually fetch in order to deal with checking preconditions.

Also was able to simplify the code and we already have the code in place to ensure fetch only happens once per command.

:ship:


Update AWS assets

Generated by:
./hack/assets.sh aws

/retest

Do we expect the assets job to pass on this PR now?

@JoelSpeed  No, we need to merge https://github.com/openshift/cluster-api-operator/pull/9 first to make assets work.
I'm going to put this PR on hold and regenerate PR after that.

/retest

/test e2e-aws-ovn-techpreview

/retest

/retest-required

/test e2e-gcp-ovn-techpreview

/hold cancel


Compress cached tiles in memory

We currently run with 500MB of raster cache by default, which fills up very very quickly. We could alleviate this a bit by compressing the tiles before adding them to the cache (and decompress upon reading).

That is a really good idea IMO, I have had problems with a gunicorn + tc deployment eating up a lot of memory, due to each worker thread having it's own cache. So anything that optimizes caching would be a good idea I think. I would also expect the overhead of decompressing the tiles to be insignificant compared to network I/O.

Yea should be fairly low-hanging. Alternatively we could have another stab at making some driver thread-safe, then you could throw some threads in the mix. Third option could be to leverage an external cache like memcached or redis.

Actually, if we are going to start doing sessions for authentication as per #21, we might want to use something like Redis to store the sessions and then we might as well use it for caching too.

An external cache would be nice, but would be a lot of work, since we also need to support the corresponding AWS service. So we could do both, in-memory compression for now, and explore some of the more sophisticated options.

Yes, it should also be easy to implement the compression, so we should do that as a first go. For the more sophisticated options on Lambda, there's Amazon Elasticache https://aws.amazon.com/elasticache/ which seems to be just a Redis instance managed by AWS.


Register with mail : the waiting view is killed when device is turned

To reproduce :

Open Riot, logout if necessary
Fill the register form using an invalid email
Touch the register button : the waiting view with the message "Please check your email to continue registration" appears
Turn the device horizontally
The waiting view is stopped

Not reproduced, state is now correctly restored on screen rotation


Random scramble

This code will provide a random scramble.
Cube.initSolver();
const cube = Cube.random();
const solution = cube.solve();
const scramble = Cube.inverse(solution);

For many platforms Cube.initSolver() takes several seconds in sync, which means screen is frozen for several times on load. I don't prefer it and am seeking for more lightweight solution.

Yea, I was bummed about that, too. Though you could put up a "Finding Scramble..." screen. Not great, though. Perhaps I can try to patch the library to make it async. In the meantime, here's a thousand scrambles to provide a bit more variety.

The TwistyTimer app also takes a couple of seconds to load the scramble, during which it shows some loading icon. It's okay, cubers expect the scramble generation to take some time.
I'm not sure how cstimer does it though, it seems to be in the background there somehow.

@hakatashi could you add more scrambles in the meantime? I'm using this a lot and don't want to start repeating scrambles :)

@tussosedan Okay. I'll add 1000. (currently 500)
Memo: scramble_333.js seems much faster than Cube.js. I got following scores on my Android.
console.time('scrambo.init');
scrambo.initialize(Math);
console.timeEnd('scrambo.init');
for (const i of Array(10).keys()) {
  console.time('scrambo.scramble');
  scrambo.getRandomScramble();
  console.timeEnd('scrambo.scramble');
}

console.time('cubejs.init');
Cube.initSolver();
console.timeEnd('cubejs.init');
for (const i of Array(10).keys()) {
  console.time('cubejs.scramble');
  Cube.scramble();
  console.timeEnd('cubejs.scramble');
}

scrambo.init: 4026.0478515625ms
scrambo.scramble:<PHONE_NUMBER>84375ms
scrambo.scramble: 98.74609375ms
scrambo.scramble: 7.961181640625ms
scrambo.scramble:<PHONE_NUMBER>25ms
scrambo.scramble: 38.23388671875ms
scrambo.scramble: 36.613037109375ms
scrambo.scramble: 5.102294921875ms
scrambo.scramble: 71.0361328125ms
scrambo.scramble: 59.135986328125ms
scrambo.scramble:<PHONE_NUMBER>40625ms
cubejs.init: 14741.630126953125ms
cubejs.scramble: 2374.02978515625ms
cubejs.scramble:<PHONE_NUMBER>4375ms
cubejs.scramble: 1218.901123046875ms
cubejs.scramble:<PHONE_NUMBER>46875ms
cubejs.scramble:<PHONE_NUMBER>21875ms
cubejs.scramble:<PHONE_NUMBER>625ms
cubejs.scramble:<PHONE_NUMBER>75ms
cubejs.scramble:<PHONE_NUMBER>625ms
cubejs.scramble:<PHONE_NUMBER>625ms
cubejs.scramble:<PHONE_NUMBER>5625ms

@tussosedan It seems much slower... plus, it's huge (700KB).
import './tnoodle.js';

console.time('tnoodlejs.init')
window.puzzlesLoaded = (puzzles) => {
  console.timeEnd('tnoodlejs.init')
  for (const i of Array(10).keys()) {
    console.time('tnoodlejs.scramble');
    puzzles['333'].generateScramble();
    console.timeEnd('tnoodlejs.scramble');
  }
}

tnoodlejs.init: 1010.31298828125ms
tnoodlejs.scramble: 3216.7578125ms
tnoodlejs.scramble:<PHONE_NUMBER>40625ms
tnoodlejs.scramble:<PHONE_NUMBER>59375ms
tnoodlejs.scramble:<PHONE_NUMBER>84375ms
tnoodlejs.scramble:<PHONE_NUMBER>28125ms
tnoodlejs.scramble:<PHONE_NUMBER>0625ms
tnoodlejs.scramble:<PHONE_NUMBER>65625ms
tnoodlejs.scramble:<PHONE_NUMBER>9375ms
tnoodlejs.scramble:<PHONE_NUMBER>9375ms
tnoodlejs.scramble:<PHONE_NUMBER>40625ms

Now we switched to random scrambles generated by scramble_333.js. Initialization is proceeded on background of giiker connection. Feel free to report any performance problems here.


Make Share page boxes clickable

Overview
Make entire boxes clickable for Github Star and Medium share options.
Action Items

[x] Make entire boxes clickable for Github Star and Medium share options
Entire box as opposed to just logo or icons should be clickable and take you to their corresponding URL's.

Resources/Instructions
http://civictechindex.org/support/share

@maxskewes to pick up once #1082 is closed

PR is open #1092

Should be good to go once Bruce's PR gets merged


theme: tag cleaning

remove non-Latin symbols
homogenize dark/light, dark-mode, dark_mode, Light/Dark Mode etc.
remove when the theme name is used as a tag
remove useless hexo-theme

I think this PR is outdated and we no longer use themes.yml


salt missing from encrypted output (regression)

SALT=aabbccddeeff0011
echo raininspain | openssl enc -aes256 -pbkdf2 -pass pass:hello -S $SALT > crypt.bin
openssl enc -aes256 -pbkdf2 -pass pass:hello -d -in crypt.bin
# bad magic number

It's unmagical because the salt is no longer included in the encrypted output.
Decryption is still possible, but only if the salt that was used is specified on the command line:
openssl enc -aes256 -pbkdf2 -pass pass:hello -d -in crypt.bin -S $SALT
# raininspain

However, that is not how a salt is supposed to work.
A salt is a salt, not a second password that needs to be recorded or remembered.
The salt should always be included in the encrypted output.
That is how it worked in earlier versions of openssl (and how it still works in the case of a random salt).
The manual workaround, if anyone needs it, is to prefix 16 bytes to the output, with the second 16 bytes being the salt:
((echo -n Salted__|xxd -p; echo $SALT)|xxd -p -r; cat crypt.bin) > crypt-salt.bin

Decryption then works as it should, without needing to specify the salt:
openssl enc -aes256 -pbkdf2 -pass pass:hello -d -in crypt-salt.bin
# raininspain

For openssl, the fix is the same, i.e., prepend "Salted__" and the salt, whether the salt is random or specified on the command line. That is what it used to do.
openssl version
# OpenSSL 3.1.2 1 Aug 2023 (Library: OpenSSL 3.1.2 1 Aug 2023)

Related: #19730

We cannot revert the behavior of -S to the 1.1.1 behavior.
We could add another option to force the salt to be written in the file.

Hi Tomáš, there is a break in compatibility no matter what you do. You prefer to keep compatibility within a minor version, whereas it appears preferable to break compatibility to fix a bug.
As far as I can see, the only intra-v3 compatibility break would arise where -S is specified on decrypt and a salt is already present, e.g., continuing the above,
openssl enc -aes256 -pbkdf2 -pass pass:hello -d -in crypt-salt.bin -S $SALT
bad decrypt
40D7003F7D7F0000:error:1C800064:Provider routines:ossl_cipher_unpadblock:bad decrypt:providers/implementations/ciphers/ciphercommon_block.c:124:
�gy�:�0����F

which is not an appropriate response in any case but also a bug that should be fixed - since it should at least work in the case where the salt given in the -S option is the same as the salt already present.
Adding another option would just make the interface messy.


Added onclick function binding to chart nodes

Now you can bind functions to your chart nodes by writing :$myFunction

Can you add a bit more documentation in the Readme?

Hi, I don't see much documentation in the Readme. I just see some inline comments in the sample code. Do you want me to put it there?

Yes, just to see what myFunction could be...

landed in v1.12.0


Fix eBPF agent's sampling: 0 value to disabled

An explicit sampling: 0 (disabled) value caused that the NOO interpreted this value as "unset", so it defaulted it to 50.

Just to clarify: "disabled" means "monitor all packets" or the opposite, "turn off monitoring"?
If the former (which I guess it is), it is equivalent than setting sampling to 1, so another approach could be to set 1 as a minimum, and update the documentation accordingly ... what would you prefer?

@jotak good appreciation. By "disabled" i mean "disable the sampling", this is "monitor all packets". Same as setting it to 1.
I have no strong preferences, but I'd be fine keeping both 0 and 1 as the same behavior.

/lgtm


chore: Improve coverage workflow

Few changes to tweak coverage workflow to occupy less disk space and run tests.

Based on tests in my fork original wf would finish successfully, but no coverage would render on codecov.io
https://github.com/romanzac/nomos-node/actions/runs/10845309954
https://app.codecov.io/gh/romanzac/nomos-node/commit/11f0a3b4b8aa7f651915e371e9b2c3e70f6549a9
Is the infrequent update on https://app.codecov.io/gh/logos-co/nomos-node/commits due to our license ?

Right now, coverage workflow takes around 26 minutes. If that is too long to have it after each push, I can change it as scheduled, once a day for example?
https://github.com/romanzac/nomos-node/actions/runs/10896693219

Right now, coverage workflow takes around 26 minutes. If that is too long to have it after each push, I can change it as scheduled, once a day for example?

Your suggestion to have coverage workflow less frequently is great, at least for the short term. I think once a day would be a good option. Regarding the changes in PR, I would keep grcov installed with the binstall, that would use even less space.


Implement EntityStore for async safe state management with per entity locking

I wrote this test to try and break destroying, and it sometimes seems to deadlock - not sure why. Probably my synchronisation in the test is deficient, as in debug it sometimes throws different exceptions, but it does seem to get stuck somewhat consistently.
        [Fact]
        public async Task TestDestroyingInConcurrentUsages()
        {
            ManualResetEventSlim secondGetStarted = new ManualResetEventSlim();

            var firstGet = await store.GetForUse(1, true);
            var secondGet = Task.Run(async () =>
            {
                secondGetStarted.Set();

                using (await store.GetForUse(1))
                {
                }
            });

            secondGetStarted.Wait(1000);
            using (firstGet)
                firstGet.Destroy();

            await Assert.ThrowsAsync<ArgumentException>(async () => await secondGet);
        }

I rewrote the above to not use Task.Run, as I though I was potentially wrecking myself with a task pool deadlock, and used a Thread instead - but it seems to still fail.
        [Fact]
        public async Task TestDestroyingInConcurrentUsages()
        {
            ManualResetEventSlim secondGetStarted = new ManualResetEventSlim();
            ManualResetEventSlim secondGetEnded = new ManualResetEventSlim();
            Exception? exception = null;

            var firstGet = await store.GetForUse(1, true);
            var secondGetThread = new Thread(async () =>
            {
                secondGetStarted.Set();

                try
                {
                    using (await store.GetForUse(1))
                    {
                    }
                }
                catch (Exception ex)
                {
                    exception = ex;
                }

                secondGetEnded.Set();
            });

            secondGetThread.Start();
            secondGetStarted.Wait(1000);

            using (firstGet)
                firstGet.Destroy();

            secondGetEnded.Wait(10_000);
            Assert.IsType<ArgumentException>(exception);
        }

What I'm seeing:

Place a single breakpoint at the inner using inside the thread action, and another right at the end
Run test in debug
Step over once
Execution is never resumed in the inner thread
At the final assert, secondGetEnded was never set, exception was never set, and the started thread seems to be missing from the list of threads

Again, likely that I messed something up, but I don't get what's happening here at all...

Added two other tests for operations that did not have coverage in this PR. Will now read through the other PR and maybe go back to my failing test above to see if I can figure out what's broken there.

Going to need to alter the test to either test each scenario separately (not sure how easy that is given hoe much of an edge case this is) or handle both types of exceptions.

On carefully re-reading the test again the type of exception that falls out of the second get comes down to who wins the race for the lock on entityMapping.
If the order is as follows:
FIRST									SECOND

secondGetStarted.Wait(1000);
										secondGetStarted.Set();
using (firstGet)
										using (await store.GetForUse(1)) [TAKES LOCK]
										{
										}
	firstGet.Destroy() [TAKES LOCK]

then the resultant exception is an InvalidOperationException. Conversely, if the order is
FIRST									SECOND

secondGetStarted.Wait(1000);
										secondGetStarted.Set();
using (firstGet)
	firstGet.Destroy() [TAKES LOCK]
										using (await store.GetForUse(1)) [TAKES LOCK]
										{
										}

then the resultant exception is an ArgumentException.
I don't think there's a sane way to test for this, as it's just contention on a lock. I'd say allowing both exceptions is OK to do here.

What I am going to do is group those two exceptions together. While updating the test I realised that every usage would need to handle both of these exceptions otherwise.

Check that now. Feels a lot better using KeyNotFoundException, i think.

I feel like this should be a sane enough state at this point to try merging and see if anything falls out. Still not completely 100% sure on the fine-grained locking (as in having separate locks for collection and item) - could potentially cause some weird ordering issues in extreme edge cases (i.e. a GetForUse(..., true) potentially not getting priority on the access after creating) - but we can deal with that as a follow-up, and get this in to get the ball rolling.


Data Classification - extended

while the current specifications include PII (Personally Identifiable Information) and classification designators, we should extend the design to include more detailed actions and attributes related to PII handling and compliance. Here are some suggestions for enhancing the schema:
PII Handling Attributes:

PII_Method: This attribute could be either a free-form string or an enumeration with values such as MASKED, ENCRYPTED, etc. It specifies the method used to protect the PII.
PII_Description: This is a descriptive field that details how the PII is handled. For example, if the PII is masked, it might specify that a name is replaced with five asterisks.
PII_Linked_Entity: This field references a masked or anonymized version of the PII field that should be used for analytics or other purposes by default.
Right to Be Forgotten (RTBF) Attributes:

RTBF_Eligible: A boolean attribute indicating whether this field qualifies under the right to be forgotten rules.
RTBF_Method: An enumeration describing the method used for RTBF, such as DELETE, ONE_WAY_HASH, ENCRYPT_ONE_TIME_KEY, or PATTERN_MASK.
RTBF_Description: A free-form string providing additional details on the RTBF method or specific rules. For example, it may state that the right to be forgotten applies only five years after the last customer activity.

{
  "fields": [
    {
      "Name": "field_name",
      "Type": "data_type",
      "PII": {
        "IsPii": true,
        "Method": "MASKED",
        "Description": "For a name, replace with 5 asterisks",
        "AnaliticsLinkedEntity": "Linked version of the entity in this record that is used for analytics premasked",
        "RTBF": {
          "Eligible": true,
          "Method": "DELETE",
          "Lifetime": 300,
          "Description": "RTBF applies 300 days after last customer activity"
        }
      },
      "Classification": "classification_string"
    }
  ]
}

Schema Explanation

fields: An array of field objects.
Name: The name of the field (string).
Type: The data type of the field (string).
PII: An object containing attributes related to Personally Identifiable Information (PII):
IsPii: Boolean indicating if the field contains PII.
Method: Enum specifying the PII protection method (e.g., MASKED, ENCRYPTED).
Description: A string providing details on how the PII is handled.
AnaliticsLinkedEntity: A string referencing the pre-masked version of the entity used for analytics.
RTBF: An object containing attributes related to the Right to Be Forgotten (RTBF):
Eligible: Boolean indicating if the field qualifies for RTBF.
Method: Enum specifying the RTBF method (e.g., DELETE, ONE_WAY_HASH, ENCRYPT_ONE_TIME_KEY, PATTERN_MASK).
Lifetime: An integer specifying the duration (in days) after which RTBF applies.
Description: A string providing additional details on the RTBF process or rules.
Classification: A string or ENUM indicating the data classification.

This structure ensures all necessary details are captured and provides a robust framework for managing PII and compliance with data protection regulations.

This probably should apply at the entity level mostly however it may be worth also allowing a whole dataset to also be defined as PII but generally I believe this should be at the entity level especially when you get to Right to be Forgotten, it may be that some bit can be forgotten straight away but other information needs to be kept longer.
e.g if you keep the last 5 addresses of a customer, at the point they stop being a customer if you do need to retain records for legal reasons you probably have no grounds to keep more than the current address.

Thanks, @pixie79 for this contribution.
I agree that PII and RTBF become more and more important and the data contracts should be the source of truth to configure the attributes for individual fields or models.
They also play nicely with some Policy as Code engines.
At the moment, however, I do not feel confident enough to defining a generic fieldset that fits for these engines.
So, for now, I'd propose to go with config fields.
E.g.:
fields:
  customer_email_address:
    description: The email address, as entered by the customer. The email address was not verified.
    type: text
    format: email
    pii: true
    classification: sensitive
    config:
      pii:
        method: MASKED
        description: For a name, replace with 5 asterisks
        analiticsLinkedEntity: Linked version of the entity in this record that is used for analytics premasked
        rightToBeForgottonPolicy:
          eligible: true
          method: DELETE
          lifetime: 300
          description: RTBF applies 300 days after last customer activity

I think that makes sense and meets what I was looking for from it.


Providing more than one language fails to load second language data

Environment

Tesseract Version: 5.0.0-rc1-24-g44690 (ubuntu ppa)
Platform:  Linux pop-os 5.13.0-7614-generic #14~1631647151~21.04~930e87c-Ubuntu SMP Fri Sep 17 00:24:58 UTC  x86_64 x86_64 x86_64 GNU/Linux

Current Behavior:
Providing more than one language fails to load second language data
$ tesseract --list-langs
List of available languages (2):
eng
lav

$ tesseract -l eng+lav ...
Error opening data file lav.traineddata
Please make sure the TESSDATA_PREFIX environment variable is set to your "tessdata" directory.

$ tesseract -l lav+eng ...
Please make sure the TESSDATA_PREFIX environment variable is set to your "tessdata" directory.

Single language works just fine
Expected Behavior:
This used to work just fine in 4.1.1

FWIW i built HEAD ebce8ab2eb97612929232d06b22b9b991a753b81 and 5.0.0-rc1 from source.
Bug is present in 5.0.0-rc1 but not HEAD.
Sorry, seems to be duplicate of  #3635.


Incomplete documentation

Sources section (https://docs.microsoft.com/en-us/azure/azure-monitor/essentials/diagnostic-settings?tabs=CMD#sources) does not provide helpful information. With the phrasing of the 5 word sentence I would expect at least a handful of example sources at a minimum. The past version of this section as found in GitHub was at least mildly more useful.
The intro of this article/page has a better definition/explanation on what sources is.

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

ID: 89743709-c3fa-9ce1-55fb-e9b7f6d2dba9
Version Independent ID: a4c9f0b7-5dde-d60c-1110-3ca53d904d61
Content: Create diagnostic settings to send Azure Monitor platform metrics and logs to different destinations - Azure Monitor
Content Source: articles/azure-monitor/essentials/diagnostic-settings.md
Service: azure-monitor
Sub-service: essentials
GitHub Login: @rboucher
Microsoft Alias: robb

@katiebrudos  - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

Work item 51273 created for this issue.

Tracking internnally already. Closing .  Now at [USER STORY 51273]. We are rewritting these sections so I'll make sure to integrate your feedback. #please-close

@mkbruhaas Following up on this. We now have source examples at https://learn.microsoft.com/en-us/azure/azure-monitor/essentials/create-diagnostic-settings?tabs=portal and https://learn.microsoft.com/en-us/azure/azure-monitor/essentials/diagnostic-settings-policy.  And we point to the sources of metrics, resource logs and the activity log.  Let me know if it's still missing something


Use encrypted service account JSON for uploading chart

Install gcloud for push to helm chart GCS bucket

I broke the push to vics copy of the repo, so now I'm just trying to see if this works.

seems to have gotten past decrypt. I'll move it to right before calling the sync-script and we can test next release.

I don't think there's anything else to do here but merge it and see what happens next release.
@viglesiasce can you check this?

LGTM

let's give it a shot for this weeks release then.
also, checking if I can:
/approve


My first commit to this project

I have added a Jupyter Notebook to compare the training and query images.

Hi @Sahiladiv,
Thank you for your patience in waiting for my feedback, and I appreciate your contribution! :)


Adding pt-BR/rss

When translating pages into Portuguese, I get an error saying that there is no rss in Portuguese, so I'm trying to create to follow the page translations
https://github.com/bitcoin-core/bitcoincore.org/pull/905 https://github.com/bitcoin-core/bitcoincore.org/pull/906
Ref:

internally linking to /pt-BR/rss/, which does not exist (line 197) RSS  htmlproofer 3.14.1 | Error:  HTML-Proofer found 1 failure! make: *** [Makefile:15: test-slow] Error 1

Use this page to open Pull requests for changes to the Bitcoin Core project
website at bitcoincore.org.
bitcoin.org is not connected with or maintained by the
Bitcoin Core project contributors. Pull requests for changes to that website
should be opened at the bitcoin.org
repository.
General bitcoin questions and/or support requests are best directed to the
Bitcoin StackExchange at https://bitcoin.stackexchange.com.

:(
what image?

what image?

image /assets/images/rss-24x24.png does not have an alt attribute (line 169)
image /assets/images/rss-24x24.png does not have an alt attribute (line 173)

Maybe it's related to t.rss_feed in:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_includes/pages/rss.html#L3
and:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_includes/pages/rss.html#L7
note:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_data/translations.yml#L11
and:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_data/translations.yml#L25
and:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_data/translations.yml#L39
and:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_data/translations.yml#L53
and:
https://github.com/bitcoin-core/bitcoincore.org/blob/dc06536f6d16d0407cb09c6a15787d01dd974a1c/_data/translations.yml#L67

Please consolidate your changes into a single PR. i.e #905. Rather than opening many different ones.


report doesn't work on SUSE 12

I installed sentinl 5.2.2 on kibana 5.2.2, and also I have installed phantomjs-prebuilt on the server. But when the report watcher is scheduled, sentinl throw an exception like "Unhandled rejection  HeadlessError: phantom immediately  exited whth: 126 .....".
The detail error info is at the attachment. I'm sorry that the photo may not be very clearly,  because my company do not allow us to connect to internet by work laptop, so I took the screen picture by my telephone ...

Hi @zxq1002  Could you please try the latest release? Let me know the result.

@zxq1002 In order to make it work for v5.2.2 change kibana version in package.json
~/dev/sentinl$ grep -rne "kibana\"" package.json -A 1
9:  "kibana": {
10-    "version": "5.4.0"

Do you mean sentinl v5.4.0 is compatible with kibana v5.2.2, not only for kibana v5.4.0?
I always thought that version must be consistent before, So I used https://github.com/sirensolutions/sentinl/releases/download/tag-5.2.2-1/sentinl.zip...
This version works on my macbook pro, so I think there is something wrong on my SUSE 12, but from the log info, I couldn't find the root reason, and doesn't know what to do... I hope someone can  give me some advice..
By the way, I compared the source code between v5.2.2-1 and v5.4.0 just now, the only difference is "package.json" and "README.md", so I think they are the same in fact.

@zxq1002 No, these releases are not identical. Even if you look at the .zip packages, v5.4.0 is approximately 52M bigger than v5.2.0.

I think your problem is that Sentinl couldn't locate phantomjs binary. It is used to create screenshots by report action.

I have linked phantomjs binary in node_modules directory to /usr/bin directory, but it still doesn't work.

I pulled the latest version b621d4 and packaged by command "gulp package --version=5.2.2", it works now. Thanks a lot!


fix(sync): Add an extra block retry, to speed up the initial sync

Motivation
As part of #4155, we want to speed up the zebrad initial sync.
If Zebra can't fetch a block, it restarts the sync, causing an expensive timeout. It's cheaper to just retry downloading the block again.
Solution

Retry each block download 3 times, rather than 2 times

Review
Anyone can review this PR. If it makes the full sync faster, it's an urgent CI fix.
This PR is based on PR #4183.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

I'm running a full sync test here:
https://github.com/ZcashFoundation/zebra/actions/runs/2218556843
This PR might be faster than the checkpoint PR, but it could be slower if the extra retries slow down a significant number of block downloads.

This PR might be faster than the checkpoint PR, but it could be slower if the extra retries slow down a significant number of block downloads.

This took 5h42m, an improvement of about 10 minutes from the checkpoint PR.

Didn't seem to improve sync time, and requires adjusting the ensure_timeouts_consistent test

@conradoplg I see an improvement of 10 minutes between this PR and the full sync test on #4183.
I fixed the test, the failing check was obsolete.


States - excludeFrom ignored

In the following app I expect the second trace statement to fail, but it doesn't
<?xml version="1.0" encoding="utf-8"?>
<mx:Application xmlns:fx="http://ns.adobe.com/mxml/2009"
                applicationComplete="init()"
                width="100%"
                height="100%"
                currentState="stateB"
                xmlns:s="library://ns.apache.org/royale/spark"
                xmlns:mx="library://ns.apache.org/royale/mx">

     <fx:Script>
        <![CDATA[
            private function init():void
            {
                trace(currentState);
                trace(b2);
            }
        ]]>
    </fx:Script>
    <mx:states>
        <mx:State name="StateB" />
        <mx:State name="StateA" />
    </mx:states>
    <mx:Button id="b1" label="yes"/>
    <mx:Button id="b2" label="no" excludeFrom="stateB"/>

</mx:Application>

Closing this, states were referenced with the wrong case (stateB instead of StateB).


samples: smp svr sample cleanup

CC @ccollins476ad

Codecov Report

Merging #6805 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6805      +/-   ##
==========================================
- Coverage   54.64%   54.64%   -0.01%
==========================================
  Files         447      447
  Lines       42598    42598
  Branches     8044     8044
==========================================
- Hits        23279    23278       -1
- Misses      16085    16086       +1
  Partials     3234     3234

Impacted Files
Coverage Δ

kernel/posix/pthread.c
69.5% <0%> (-0.5%)
:arrow_down:

Continue to review full report at Codecov.

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

recheck


Debugging invalid addresses on travis/nightly

Initial review: appears only to affect deterministic keys, and only on secp256k1. this narrows things down to secp256k1_ec_privkey_tweak_add.
Further review took place while working on the secp256k1-php repo, which actually never shown signs of the bug, which sucks. While the intention was to modify the zval in question, we only wanted to modify the variable passed into the function, not every reference to the zval. Added a regression test capturing this scenario.

Solved as per above, and fixed on nightly by using updated version.


ctrl-f and ctrl-b should extend selection

Expected: should extend selection
Actual : clears selection

I have already fixed this, it's on this PR https://github.com/atom/vim-mode/pull/616 :smiley:

Most excellent! Thank you.

Looks fixed on current Atom and vim-mode.  Time to close this issue?
I only see one difference to Vim: when near the bottom of the page, Vim's Ctrl-F extends the selection to the exact EOF (and scrolls everything off the top of the screen) while vim-mode's doesn't do anything.
However, Vim isn't consistent.  It doesn't extend the selection to the top of the page when hitting Ctrl-B, so it acts the same as vim-mode's Ctrl-F and Ctrl-B.  Personally, I'd rather see consistent behavior than have 100% bug-for-bug compatibility with Vim.

@bronson yeah, there are still some stuffs missing, but I think that will be better if someone open a new issue for this, and close this one.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


⚠ Samsung has degraded performance

In 95d536d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2851 ms

Resolved: Samsung performance has improved in 27bcc25.


Release 1.4.1 with relaxed faraday dependency

Hey Folks!
Any chance we could release 1.4.1 with relaxed faraday dependency as per this PR https://github.com/oauth-xx/oauth2/pull/397? Also, why lock the top version if we don't know that Faraday will change their API? What about locking it after the fact, i.e. if Faraday API changes?
I would have opened a patch, but there is no 1.4.0 stable branch.
Thanks!

We have already relaxed the version on master.  Please test master if you are able to switch.  We may release a pre-release version of the gem soon.  It will be a major version bump, and it will include breaking changes.
I was not involved in previous releases of oauth2, so this will be my first.  From my reading it seems that the reason for capping the faraday version is out of precaution, since faraday is known for changing their API and breaking lots of shit without warning, and without following semver.

Ok, thanks for the details.


For ivan to checkout and add

This fixes a few things with overrides not being present or empty.
Generated file still doesn't work because class names need to be mangled a little more to remove dashes and first part of name if it is prefixed by a number. Maybe add the number to the end with '_8' for example.

LGTM!
I will address the class name formatting next


fix(rust): remove hide default value attribute

Closes #3552
Current Behavior
As described in #3552: node subcommands have node_name as the first argument. The commands that act on existing nodes should default to the "default" node and should not hide this value.
Currently, this is only done correctly in the node show command.
Proposed Changes
remove attribute hide_default_value from node_name argument
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

@mergifyio rebase


Fix projection for union all queries with multiple aliases

Given two tables:
create table source1 (dt string, d1 int, d2 int) stored as orc;
create table source2 (dt string, d1 int, d2 int) stored as orc;
insert into source1 values ('20211107', 1, 2);
insert into source2 values ('20211108', 11, 22);

If you run this query with UNION ALL, the key column will be missing from the output:
select explode(map('D219', D219
,'D220', D220)) as (key, value)
from (
select '20211107' as date_key
,1 as D219
,2 as D220
) t
union all
select explode(map('D221', D221
,'D222', D222)) as (key, value)
from (
select '20211107' as date_key
,1 as D221
,2 as D222
) t

Result:
1
2
11
22

Correct result should be:
D219	1
D220	2
D221	11
D222	22

(Note: If you run the two subqueries individually, it works fine. The problem occurs with the UNION).
The problem seems to be related to parsing. Currently we assume that only the last column can be an alias column:
// Examine the last child. It could be an alias.
Tree child = selExpr.getChild(selExpr.getChildCount() - 1);

which ignores the fact that we could have multiple aliases defined, such as in this case (e.g. ... as (keyx, valuex))

@kgyrtkirk @kasakrisz As you're a lot more familiar with the parser than me, could you please take a look at this PR? This fixes an issue that came up in an escalation. Your guidance is appreciated. Thanks a lot!

@marton-bod
I think you are ok to merge this.


Added Themeselection

Themeselection
Here you will get selected high quality, modern design, professional and easy-to-use Free Bootstrap Admin Dashboard Template,
HTML Themes and UI Kits to create your applications faster
Website: https://themeselection.com/

Hey. Is it possible to get the templates without sign up?

Duplicate of #160.


Re-organize the LLVM build instructions

zig needs lib directory
llvm needs development headers
Keep installation instructions together
Group by platform
No longer split installation and debugging instructions

Thanks for the review @rtfeldman; I've updated the PR accordingly.


Workflow controller fails when dependencies are UUIDs

Summary
I set argo workflow dependencies to a UUID, which messes up the expr.eval https://github.com/antonmedv/expr package because it tries to parse the UUID as a number.
Diagnostics
What Kubernetes provider are you using? Local
What version of Argo Workflows are you running? 2.11.1
Name:         bje-5f7768fb5da4261220ffa1cc
Namespace:    bje-ns-5f7768fb5da4261220ffa1cc
Labels:       workflows.argoproj.io/completed=true
              workflows.argoproj.io/phase=Error
Annotations:  <none>
API Version:  argoproj.io/v1alpha1
Kind:         Workflow
Metadata:
  Creation Timestamp:  2020-10-02T17:52:59Z
  Generation:          6
  Resource Version:    174067
  Self Link:           /apis/argoproj.io/v1alpha1/namespaces/bje-ns-5f7768fb5da4261220ffa1cc/workflows/bje-5f7768fb5da4261220ffa1cc
  UID:                 71bf07f3-7937-489b-9f20-31951d965489
Spec:
  Arguments:
  Entrypoint:  bje-5f7768fb5da4261220ffa1cc-dag
  Image Pull Secrets:
    Name:                regcred-5f7768fb5da4261220ffa1cc
  Service Account Name:  argo-workflow
  Templates:
    Arguments:
    Container:
      Command:
        /opt/code/runme.sh
      Env:
        Name:   xxx
        Value:  http://<IP_ADDRESS>
      Image:    localhost:5000/managed-image:5f7768f75da4261220ffa1c5
      Name:     5f7768fb5da4261220ffa1c9
      Resources:
        Limits:
          Cpu:     1
          Memory:  104857600
        Requests:
          Cpu:     1
          Memory:  104857600
      Volume Mounts:
        Mount Path:  /mnt/output
        Name:        5f7768fb5da4261220ffa1c9-pvc
    Inputs:
    Metadata:
    Name:  5f7768fb5da4261220ffa1c9-container
    Outputs:
    Arguments:
    Container:
      Command:
        /opt/code/runme.sh
      Env:
        Name:   xxx
        Value:  http://<IP_ADDRESS>
      Image:    localhost:5000/managed-image:5f7768f75da4261220ffa1c5
      Name:     5f7768fb5da4261220ffa1cb
      Resources:
        Limits:
          Cpu:     1
          Memory:  104857600
        Requests:
          Cpu:     1
          Memory:  104857600
      Volume Mounts:
        Mount Path:  /mnt/output
        Name:        5f7768fb5da4261220ffa1cb-pvc
        Mount Path:  /mnt/input
        Name:        5f7768fb5da4261220ffa1c9-pvc
    Inputs:
    Metadata:
    Name:  5f7768fb5da4261220ffa1cb-container
    Outputs:
    Arguments:
    Dag:
      Tasks:
        Arguments:
        Name:      5f7768fb5da4261220ffa1c9
        Template:  5f7768fb5da4261220ffa1c9-container
        Arguments:
        Dependencies:
          5f7768fb5da4261220ffa1c9
        Name:      5f7768fb5da4261220ffa1cb
        Template:  5f7768fb5da4261220ffa1cb-container
    Inputs:
    Metadata:
    Name:  bje-5f7768fb5da4261220ffa1cc-dag
    Outputs:
  Volume Claim Templates:
    Metadata:
      Creation Timestamp:  <nil>
      Name:                5f7768fb5da4261220ffa1c9-pvc
    Spec:
      Access Modes:
        ReadWriteOnce
      Resources:
        Limits:
          Storage:  0
        Requests:
          Storage:<PHONE_NUMBER>
    Status:
    Metadata:
      Creation Timestamp:  <nil>
      Name:                5f7768fb5da4261220ffa1cb-pvc
    Spec:
      Access Modes:
        ReadWriteOnce
      Resources:
        Limits:
          Storage:  0
        Requests:
          Storage:<PHONE_NUMBER>
    Status:
Status:
  Conditions:
    Status:     True
    Type:       Completed
  Finished At:  2020-10-02T17:53:05Z
  Nodes:
    bje-5f7768fb5da4261220ffa1cc:
      Children:
        bje-5f7768fb5da4261220ffa1cc-1751792309
      Display Name:  bje-5f7768fb5da4261220ffa1cc
      Finished At:   2020-10-02T17:53:05Z
      Id:            bje-5f7768fb5da4261220ffa1cc
      Name:          bje-5f7768fb5da4261220ffa1cc
      Outbound Nodes:
        bje-5f7768fb5da4261220ffa1cc-2641006116
      Phase:           Error
      Started At:      2020-10-02T17:52:59Z
      Template Name:   bje-5f7768fb5da4261220ffa1cc-dag
      Template Scope:  local/bje-5f7768fb5da4261220ffa1cc
      Type:            DAG
    bje-5f7768fb5da4261220ffa1cc-1751792309:
      Boundary ID:  bje-5f7768fb5da4261220ffa1cc
      Children:
        bje-5f7768fb5da4261220ffa1cc-2641006116
      Display Name:    5f7768fb5da4261220ffa1c9
      Finished At:     2020-10-02T17:53:03Z
      Host Node Name:  node.localdev
      Id:              bje-5f7768fb5da4261220ffa1cc-1751792309
      Name:            bje-5f7768fb5da4261220ffa1cc.5f7768fb5da4261220ffa1c9
      Outputs:
        Exit Code:  0
      Phase:        Succeeded
      Resources Duration:
        Cpu:           3
        Memory:        3
      Started At:      2020-10-02T17:52:59Z
      Template Name:   5f7768fb5da4261220ffa1c9-container
      Template Scope:  local/bje-5f7768fb5da4261220ffa1cc
      Type:            Pod
    bje-5f7768fb5da4261220ffa1cc-2641006116:
      Boundary ID:   bje-5f7768fb5da4261220ffa1cc
      Display Name:  5f7768fb5da4261220ffa1cb
      Finished At:   2020-10-02T17:53:05Z
      Id:            bje-5f7768fb5da4261220ffa1cc-2641006116
      Message:       unable to evaluate expression '(5f7768fb5da4261220ffa1c9.Succeeded || 5f7768fb5da4261220ffa1c9.Skipped || 5f7768fb5da4261220ffa1c9.Daemoned)': bad number syntax: "5f" (1:4)
 | (5f7768fb5da4261220ffa1c9.Succeeded || 5f7768fb5da4261220ffa1c9.Skipped || 5f7768fb5da4261220ffa1c9.Daemoned)
 | ...^
      Name:            bje-5f7768fb5da4261220ffa1cc.5f7768fb5da4261220ffa1cb
      Phase:           Error
      Started At:      2020-10-02T17:53:05Z
      Template Name:   5f7768fb5da4261220ffa1cb-container
      Template Scope:  local/bje-5f7768fb5da4261220ffa1cc
      Type:            Skipped
  Phase:               Error
  Resources Duration:
    Cpu:       3
    Memory:    3
  Started At:  2020-10-02T17:52:59Z
Events:        <none>

time="2020-10-02T17:38:27Z" level=info msg="Skipped pod bje-5f7765735da42616dd60bae9.5f7765735da42616dd60bae3 (bje-5f7765735da42616dd60bae9-127836785) creation: already exists" namespace=bje-ns-5f7765735da42616dd60bae9 podPhase=Pending wor
kflow=bje-5f7765735da42616dd60bae9
time="2020-10-02T17:38:34Z" level=info msg="Processing workflow" namespace=bje-ns-5f7765735da42616dd60bae9 workflow=bje-5f7765735da42616dd60bae9
time="2020-10-02T17:38:34Z" level=info msg="Setting node bje-5f7765735da42616dd60bae9-127836785 outputs"
time="2020-10-02T17:38:34Z" level=info msg="Updating node bje-5f7765735da42616dd60bae9-127836785 status Pending -> Running"
time="2020-10-02T17:38:34Z" level=info msg="Workflow update successful" namespace=bje-ns-5f7765735da42616dd60bae9 phase=Running resourceVersion=172157 workflow=bje-5f7765735da42616dd60bae9
time="2020-10-02T17:38:35Z" level=info msg="Processing workflow" namespace=bje-ns-5f7765735da42616dd60bae9 workflow=bje-5f7765735da42616dd60bae9
time="2020-10-02T17:38:35Z" level=info msg="Updating node bje-5f7765735da42616dd60bae9-127836785 status Running -> Succeeded"
time="2020-10-02T17:38:35Z" level=info msg="Skipped node bje-5f7765735da42616dd60bae9-178169642 initialized Error (message: unable to evaluate expression '(5f7765735da42616dd60bae3.Succeeded || 5f7765735da42616dd60bae3.Skipped || 5f7765735da42616dd60bae3.Daemoned)': bad number syntax: \"5f\" (1:4)\n | (5f7765735da42616dd60bae3.Succeeded || 5f7765735da42616dd60bae3.Skipped || 5f7765735da42616dd60bae3.Daemoned)\n | ...^)" namespace=bje-ns-5f7765735da42616dd60bae9 workflow=bje-5f7765735da42616dd60bae9
time="2020-10-02T17:38:35Z" level=info msg="Skipped node bje-5f7765735da42616dd60bae9-194947261 initialized Error (message: unable to evaluate expression '(5f7765735da42616dd60bae6.Succeeded || 5f7765735da42616dd60bae6.Skipped || 5f7765735da42616dd60bae6.Daemoned)': bad number syntax: \"5f\" (1:4)\n | (5f7765735da42616dd60bae6.Succeeded || 5f7765735da42616dd60bae6.Skipped || 5f7765735da42616dd60bae6.Daemoned)\n | ...^)" namespace=bje-ns-5f7765735da42616dd60bae9 workflow=bje-5f7765735da42616dd60bae9

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@simster7

This is where the error is thrown https://github.com/argoproj/argo/blob/master/workflow/controller/dag.go#L696

The issue happens when the task name begins with a number, and then contains a non-number character. As a workaround, ensure that task names begin with a non-number character.
Working on a fix


improvement for option Local JAR

[+] support path with unix-like style on windows,
e.g. ~/.local/plantuml.jar
[+] search plantuml.jar from current vault path, instead of __dirname
exported from obsidian

probably, will need to do some more tests if this works as intented.

Your method does work for the most part, with one issue:
It does not search relative to the vault path, but relative to the currently opened file inside the vault.
I will merge this and fix that issue myself, since I already have that figured out.


Create deployment action

Create deployment action to implement continuous development.

Insanely useful resource: https://gist.github.com/coehne/caf0b3934455d842dfbfe1f4c1544348

Current status:

firebase deploy deploys code to firebase hosting
Deployed website does not work, gives a status of 500 internal error
Created (more accurately, generated) github actions to deploy on merge (to live channel) and PR (to presumably not live channel, not sure because it is not specified)

TODO:

figure out how to debug an application hosted on firebase hosting
validate github actions behaviors
i assume i will have to figure out how to get my ENV variables over

Fixed deployment action


[Crash // 1.18.2 - 1.2.0] Crash from opening the Tool Planner Menu from a Tinkers' Construct Anvil

Recently, my game started crashing just from opening the tool planner from an anvil. The crash thankfully didn't result in a temporary infinite loop crash affecting not just the world I tried this in but also every other world I tried to log into, forcing me to restart my game. Kinda weird because--despite not using it for quite a while--this mod was working perfectly fine.
I've already looked at someone else having this same or if not a similar issue [#8], but for their case they were using this mod for 1.16.5 and a new version was made available to them at that time. For my case, 1.2.0 is sadly the only version available and hasn't had any newer versions. Hopefully it doesn't become abandoned!
I know this probably might be because I need to update Forge or whatever (which I've been a bit lazy on, but if it's because of any of my mods interfering with it, then it'll be a very tedious process to go through all of them 1-by-1 to see what's causing it. Again, this crash doesn't force me to relaunch my game manually. It just kicks me out of my world.
Solutions for this crash might work for everyone who's having this same issue but are 100% not guaranteed to work for those with even a slightly different issue. Regardless, I appreciate your input(s)!
Version Info

Minecraft 1.18.2
Tinkers' Construct <IP_ADDRESS> [Forge]
Tinkers' Planner 1.2.0 [Forge]
Forge 40.2.17

Steps to Reproduce
This was tested with a Tinkers' Construct anvil but it probably happens with the tool station as well.

Place down any anvil from Tinkers' Construct.
Right-click the anvil to open its menu.
Click the blueprint icon.

Behaviors

Expected: Open the tool planning menu.
Actual: Crash my game.

Crash report
crash-2024-04-13_21.33.14-client.txt

Update 4.25.24: Tested it with the Modrinth version of this mod, this time on the tool station. The crash happens with the tool station as well.

It's safe to say that this mod has been abandoned as there hasn't been any response to this issue for 3 weeks, but I won't manually close this issue as either completed or not planned. However, if closed and you still have any inputs, please feel free to respond with them so that this issue can be opened again.

I don't know if this was either TConstructCompatible or Tinkers' Reforged causing this crash, but I disabled both and I'm able to open the planner. I'll report back once I figure out which of these two mods is the culprit. However, this issue will stay open for the dev to provide their input.

A'ight seems like these two mods were causing this crash, though other similar mods could cause this crash as well.

Had exact same crash. Don't have those 2 mods installed. :(

Had exact same crash. Don't have those 2 mods installed. :(

If you'd like to reopen this issue, feel free. Otherwise, we're not getting a response from @tiffit anytime soon or at all on this issue. This issue will be closed as not planned, having took 3 months for a response from the developer of tbis mod.

Having a similar crash that gives this error:
java.lang.NoSuchMethodError: 'int slimeknights.tconstruct.library.recipe.modifiers.ModifierRecipeLookup.getNeededPerLevel(slimeknights.tconstruct.library.modifiers.ModifierId)'
Neither of the TC submods I have installed seem to be responsible (Construct's Armory Port and Tinker's Rapier), nor do any of my other mods. I suspect that an update to the main Tinker's Construct mod made this one unstable, which is a shame.


index not updated properly

Hi!
I'm in the process of updating what @cibernox did here https://github.com/cibernox/ember-power-select-collection
In its newest version ember-power-select relies on the index yielded property of the #each helper.
It appears that this is not playing super well with ember-collection and when power-select filters the list the indexes are not updated properly.
ember-collection exposes an index but maybe I'm simply misunderstanding its functionality?
I'll try to create a reproduction repository as soon as I can, in the meantime here's a video from this happening in my company's app
https://gsnaps.s3.amazonaws.com/screencast_2016-05-01_18-02-50.mp4
I'm rendering only the indexes in the list instead of the full content and you can see they're not contiguous
thanks!

also to be noted, I have a workaround for ember-power-select that doesn't rely on the index property but I  still wonder if index is correct

Related: https://github.com/emberjs/ember-collection/pull/110
Sorry, just haven't had time to figure out why those tests are failing, but basically when reusing cells we aren't updating the index that gets exposed.

@raytiley no problem, I missed that #110 was the same.
As mentioned I have a workaround for now anyway.
Thanks for the quick response!


chore: Update the Github Action to set correct tag for the Docker Image

The PR adds support for adding the the release version tag without "v" { 0.10.0 instead of v0.10.0} to the docker images (including wolfi).
I have not removed the existing tagging logic as mentioned in the issue(needs to be keep for some time before complete migration to use the new format).
Closes #465
Note:
The specific approach suggested in here in the issue, doesn't work as Github Action doesn't allow inline string manipulation in expressions.
Testing
Github Action Run with correctly formatted tag: https://github.com/sp6370/wadm_/actions/runs/11905116268/job/33175286269

Thanks @sp6370!


ValueError, "Checkpoint {0} does not seem to exist".format( ckpt.model_checkpoint_path ) )

Hi,
Thanks for the code!!
I am trying to reproduce your results and get the hold of 3D reconstruction.
I am getting the following error when I tried to first train your model

and also still I tried further visualizing and i got these errors

Further Details:
OS:                                                            Windows 10
Tensor Flow version:                                 Tensorflow GPU 1.12
Python                                                       Python 3
Thanks,
Mritula
Thanks for your interest in our research!
If you have problems running our code, please include

Your operating system
Your tensorflow version
Your python version
The stack trace of the error that you see

This is research code, and its primary purpose is to reproduce the scientific results that you see in our paper.

Hi,
Can I have help on setting up the environment to use 3D_Pose_Estimation. My system info is as below
Details:
OS: Windows 10
Tensor Flow version: Tensorflow GPU 1.12
Python Python 3.7
Cuda 10.1
Thanks,
Mritula


Something wrong with update :/

hey, that's me again with 'the same' problem. After someone helped me and told I need to reinstall VS Code, I did it, but today I get a message to update a VS Code. I clicked 'Install now', because why not? And then started this...

vscode-inno-updater.log
here you are, error message and .log file

@RootKlawiter Just to confirm, Code was running fine after that, correct?

Yeah, it was working fine, but the problem still occurred 🤐

dupe of #44911


Array fields not cast as arrays

Example code:
fields: {
  availability: {
    type: [AbstractDuration],
    default: function() {
      return [];
    }
}

When I run Class.availability.push(new AbstractDuration), My document ends up like this:
availability: {
  '0' : {
    days: ''
  }
}

Where the nested object is the correct class but availability has been as an object, not an array

You are probably doing something wrong. Show me more code.

Volunteer = User.inherit({
  name: 'Volunteer',
  fields: {
    areasOfExpertiseIds: {
			type: [String],
			default: function() {
				return [];
			}
		},
    educationLevel: {
			type: String,//[what,what,what]
			default: ''
		},
    educationHistory: {
			type: String,
			default: ''
		},
    bio: {
			type: String,
			default: ''
		},
    volunteerExperience: {
			type: String,
			default: ''
		},
    employmentStatus: {
			type: String,//[employed,retired,independent]
			default: ''
		},
    certifications: {
			type: String,
			default: ''
		},
    resume: {
			type: Object,
			default: function() {
				return {};
			}
		},
    availability: {
			type: [AbstractDuration],
			default: function() {
				return [];
			}
		},
    associations: {
      type: Object,
      nested: {
        name: 'Associations',
        fields: {
          independent: {
						type: Object,
						default: function() {
							return {};
						}
					},
          organizations: {
						type: [Affiliation],
						default: function() {
							return [];
						}
					}
        }
      },
			default: function() {
				return {};
			}
    }
  },
	methods: {
		getImage: function() {},
		hoursVolunteered: function() {}
	}
  // relations: {
  //   requestsFor: {
  //     type: 'many',
  //     class: 'Request',
  //     local: '_id',
  //     foreign: 'volunteerId'
  //   }
  // }
});

and
Meteor.methods({
	'/volunteer/availability/add': function() {
		let volunteer = Volunteer.findOne({_id: Meteor.userId()});
		volunteer._isNew = false;
		volunteer.availability.push( new AbstractDuration() );
		volunteer.save();
	}
})

You're code is wrong. Why do you mix syntax for 1.0 and 2.0? Which version of Astronomy are you using? Why are you setting volounteer._isNew = false?

I'm using 2.0. I've commented out what I know to be 1.0 code. Where am I using breaking 1.0 code? Lacking documentation for 2.0, I lean on old syntax. I'm setting _isNew = false due to your comment here: https://github.com/jagi/meteor-astronomy/issues/236#issuecomment-187107552 because when I wasn't I got a duplicate ID error.

@tylershuster You haven't provided the definition of any of those nested types. Can you perhaps make a much simpler use case for this issue?

User = Astro.Class.create({
  name: 'User',
  collection: Meteor.users,
  fields: {
    name: String,
    phone: {
			type: String,
			default: ''
		},
    avatar: {
			type: Object,
			optional: true
		},
		createdAt: Date,
		services: Object,
		profile: Object
  },
	transform: function(doc) {
		let userType = doc.profile.role;
		if(userType == 'educator' || userType == 'school') {
			return new Educator(doc);
		} else if(userType == 'volunteer' || userType == 'organization') {
			return new Volunteer(doc);
		} else {
			return new User(doc);
		}
	}
});

Volunteer = User.inherit({
  name: 'Volunteer',
  fields: {
    availability: {
			type: [AbstractDuration],
			default: function() {
				return [];
			}
		}
  }
});

AbstractDuration = Astro.Class.create({
  name: 'Abstract Duration',
  fields: {
    days: {
			type: [String],
			default: function() {
				return [];
			}
		},
    times: {
      type: Object,
      nested: {
        name: 'Duration in Hours hh:mm format',
        start: {
					type: String,
					default: ''
				},
        end: {
					type: String,
					default: ''
				},
      },
			default: function() {
				return {};
			}
    }
  }
});

I run this code
var volunteer = Volunteer.findOne();
volunteer.availability.push(new AbstractDuration());

At this point everything is kosher:
{ _id: 'SxxXg9MSJpiB5NQeh',
  name: 'Tyler',
  phone: '3408734',
  avatar: undefined,
  createdAt: Thu Apr 07 2016 09:23:46 GMT-0700 (PDT),
  services:
   { password: { bcrypt: '$2a$10$KAOHeMbDDSEkm1fTJec4Q.UXSR4GCHJglViDsnUb4iJU2OcqQ/j5G' },
     resume: { loginTokens: [Object] } },
  profile:
   { role: 'volunteer',
     organization: '',
     name: 'Tyler' },
  availability: [ { days: [], times: {} } ] }

Now I run volunteer._isNew = false to avoid the duplicate id error and volunteer.save()
now my document looks like this:
{ _id: 'SxxXg9MSJpiB5NQeh',
  name: 'Tyler',
  phone: '3408734',
  avatar: undefined,
  createdAt: Thu Apr 07 2016 09:23:46 GMT-0700 (PDT),
  services:
   { password: { bcrypt: '$2a$10$KAOHeMbDDSEkm1fTJec4Q.UXSR4GCHJglViDsnUb4iJU2OcqQ/j5G' },
     resume: { loginTokens: [Object] } },
  profile:
   { role: 'volunteer',
     organization: '',
     name: 'Tyler' },
  availability: { '0': { days: [], times: {} } } }

Which rc version are you on?
I think this notation is no longer valid:
times: {
      type: Object,
      nested: {
        name: 'Duration in Hours hh:mm format',
        start: {
                    type: String,
                    default: ''
                },
        end: {
                    type: String,
                    default: ''
                },
      },
            default: function() {
                return {};
            }
    }

You should probably try creating another AstroClass for that nested schema.

I'm on rc8
Now my code looks like this:
AbstractTimespan = Astro.Class.create({
	name: "Abstract Timespan Duration in Hours hh:mm format",
	fields: {
		start: {
			type: String,
			default: ''
		},
		end: {
			type: String,
			default: ''
		}
	}
})

AbstractDuration = Astro.Class.create({
  name: 'Abstract Duration',
  fields: {
    days: {
			type: [String],
			default: function() {
				return [];
			}
		},
    times: {
			type: AbstractTimespan,
			default: function() {
				return new AbstractTimespan();
			}
		}
  }
});

But the result is the same.
I also added a default to the availability definition:
availability: {
			type: [AbstractDuration],
			default: function() {
				return [new AbstractDuration()];
			}
		}

but all that succeeded in doing was incrementing the "0" to "1", still being cast as an object.

You're code is too nested and in separate sections to reproduce what you are facing. Have a look at this:

Parents = new Mongo.Collection('parents');

Child = Astro.Class.create({
  name: 'Child',
  fields: {
    name: {type: String},
    age: {type: Number}
  }
});

Parent = Astro.Class.create({
  name: 'Parent',
  collection: Parents,
  fields: {
    name: {type: String},
    children: {type: [Child], default() { return []; }}
  }
});

Now do this in your browser console or the meteor shell:
var myParent = new Parent();
myParent.name = 'Testing';
myParent.save();

myParent.children.push(new Child({name: 'Sam', age: 15}));
myParent.save();

myParent.children.push(new Child({name: 'Max', age: 10}));
myParent.save();

Seems to work perfectly for me, the field remains an array.

@tylershuster syntax with the nested property was removed in 2.0. Try the example posted by @talha-asad

I got it to work once, but then I ran meteor reset to try it again, and it went back to the old behavior. I got it working now, though. It seems that if there's a default option set, it won't get cast as an array. So now the field is optional with no default. My method for adding to that field involves manually casting the field as an array if it's unset, and then pushing to that array, then saving.

Due to it being optional, the push method and all other Array methods will not be available. I think your method is correct.

I should note that it wasn’t optional before, and a Volunteer object that didn’t have its availability set in the document would show up as having an availability object cast as an array. It was only on saving that it was re-cast as an object upon insertion into the database. Regardless, I can move forward.

On Apr 8, 2016, at 9:00 AM, Talha Asad<EMAIL_ADDRESS>wrote:
Due to it being optional, the push method and all other Array methods will not be available. I think your method is correct.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/jagi/meteor-astronomy/issues/332#issuecomment-207493246


Update wash find examples to be more API-idiomatic

Some examples also have errors and are weird. E.g. In find --help syntax
find kubernetes -daystart -k '*pod' -m .status.startTime -{1d}
    Print out all the Kubernetes pods that started today.

doesn't make sense because -{1d} is a future time. That should be -1d.

This ticket also involves taking a final pass at the examples that are there to make sure they are correct.

Addressed by https://github.com/puppetlabs/wash/commit/7b5d811a0ae0562a869f87379445858bcca2250d.


How to categorize errors ?

Hi there,
I am trying to move to Monolog but I am wondering how I can categorize my errors with Monolog.
For example: I have several groups of errors that have different types and message and I need to save everything in one file. I can use channels instead of groups but how I can define my own types?
Group1 --> Type1, Message
--> Type2, Message
--> Type3, Message
Group2 --> Type4, Message
--> Type5, Message
--> Type6, Message
I appreciate your help in advance.

You could add the type in the context array when logging, and add %context.type% to the message format in the LineFormatter you use to log to the file, but I really have no idea what you're doing so maybe this is off.


Feature request: moving components in the hierarchy

It would be fantastic to be able to re-organize the component hierarchy in some way (e.g., via drag and drop in the Files pane).
This is a thing I've wanted to do many times since as a project matures, I've wanted to re-organize components for clearer presentation. I've also had multiple people ask me if it's possible to do it and I can sense their disappointment when I tell them it's not possible.  I've suggested a workaround of creating linked projects instead of components but it's not a clean solution.
Thanks!

Thanks for reaching out @zoidy! We do support this functionality. You can re-order components by dragging and dropping them in the "Components" section on the parent project. See this help guide for step-by-step instructions: http://help.osf.io/m/projects/l/814548-reorder-components

@BRosenblatt Thanks for the reply. I think you misunderstood my question though. I'm talking about re-organizing the hierarchy, not simply re-ordering components. I.e., if I have
Main project
-> Component 1
-> Component 2
I would like to be able to do something like
Main Project
-> Component 1
-> Component 2
without having to delete Component 2 and then re-create it under Component 1

I apologize for the misunderstanding! We are currently working on allowing components to be moved around in the hierarchy. I'll let you know when this gets released (it will be sometime in Q2).

I've recently started using OSF and had this exact same question. Is it now possible to move components around in the hierarchy?
Thanks for your help!

I've also recently started using OSF.  Is this feature now available?


Enable payloads for custom content types

Builds off of https://github.com/stfalcon-studio/ChatKit/commit/123f9ac2aef8176f56cb74328cb8c3a2b5c467a0 to enable passing custom data to custom content type view holders.

The exact same issue has been holding my project back. It'd be nice if this  is merged in as soon as possible

Hi @mweathers ! Nice work! But you changed publishing settings. Please revert it and I will merge this pull request.

Updating README to hint this will be great addition!


refactor: use pymmcore-widgets

ok, the main reason this is failing is that it's ending up with a different core instance.  So now, we get to really experience the joys of singleton management :joy:
it would be relatively easy to fix by not using a custom session core in our tests here, but I don't really want to do that yet.  (it's nice to be able to use an isolated little core instance, particularly for tests).  So we may have to either create a nice set_core context manager, that allows you to inject a given core instance (we could potentially use in-n-out here), or, we might need to laboriously pass the core instance around through all the widgets (ew)

see #194 instead


add flag to print out types

Hi,
I very much liked 'elm -p' to check the result of the type inference. This feature is no longer present in 0.14. It would be great if it could be reimplemented.
Regards,
Volker

Tracking from meta issue #98


change all imagePullPolicy to IfNotPresent

1s          Warning   Failed              pod/csi-lvm-controller-759f7b6c6b-dgp6v    Failed to pull image "metalstack/csi-lvm-controller:v0.6.1": rpc error: code = Unknown desc = Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

can this be closed ?

No... only done in Ansible for the control plane components.

fixed by https://github.com/metal-stack/gardener-extension-provider-metal/pull/318


ascii-to-table is buggy

e.g.
ibmcloud ks cluster-get mycluster

the output is truncated

another example:
When I clicked a bash script, I got run xxx.sh command issued (maybe a separate bug?), and the partial output is in the following screenshot


Add support for persistent subscription negative acknowledgments.

In my particular use case I require being able to send negative acknowledgments for persistent subscription event messages.  This code adds handling for sending PersistentSubscriptionNakEvents messages.
Some notes about the code.
I added a unit test which tests sending a Retry nak. The test is Try retry nak action in file extreme_test.exs line 720.
I added the ability to delete a persistent subscription. This allows for cleanup of created subscriptions in the test code.
In response.ex, the message SubscriptionDropped was commented out, but I wanted to handle this message for a persistent subscription, so I added it back. I'm not sure if this is a problem otherwise? All unit test pass and I haven't had problems in my system testing.
@slashdotdash could you please take a look at the code and let me know what you think? I really appreciate it.
If any changes are needed or I can answer any questions, please let me know.
Thank you,
Nathan Fox

Coverage increased (+0.3%) to 76.419% when pulling f3e5b2ccba83d53d3f2b5fa3dbb2a215d33d991f on nathanfox:add-persistent-subscription-naks into bc5e69bd55e50cc98731f230baf1cf1e6ff7a991 on exponentially:master.

The CI run failed for the merge request, but the tests run fine on my machine. I'm not sure why it failed :-(

The Travis CI test failure appears to be unrelated to your changes (listener test failure, not persistent subscriptions).
@nathanfox I would recommend adding the following nak/5 function to support persistent subscriptions on projection streams. You have to ack/nak the linked event in this scenario.
def nak(subscription, %{link: link}, correlation_id, nak_action, message \\ nil) when not is_nil(link) and is_atom(nak_action) do
  GenServer.call(subscription, {:ack, link.event_id, correlation_id, nak_action, message})
end

Otherwise looks good to me, thanks for adding naks.

I would only suggest to rename nak to nack, but overall good job!

Good point on the name change. I'll change that. I'll also work on @slashdotdash 's suggestion. I'm not sure if I can get to it this week. Hopefully by early next week I'll have the changes in place and I'll update the branch in the pull request.

Great, take your time :)

Coverage increased (+1.3%) to 77.489% when pulling 6aed52bc715788d2793fe5fb82351f47d3f1c04a on nathanfox:add-persistent-subscription-naks into bc5e69bd55e50cc98731f230baf1cf1e6ff7a991 on exponentially:master.

I've added the nak to nack name change as well as adding a nack function to handle linked events.
Please let me know how it looks.

@nathanfox The example in the README is using nak, not the renamed nack function.

Good catch. I ran grep against .ex and .exs :-) I updated the readme. I also corrected the terminology so nack means not acknowledged to match the EventStore documentation.

Coverage increased (+1.3%) to 77.489% when pulling 88f8969ce3542248581b00c6d94802a0cb6dab19 on nathanfox:add-persistent-subscription-naks into bc5e69bd55e50cc98731f230baf1cf1e6ff7a991 on exponentially:master.


Can't upload my app to heroku because of this dependency

I think the issue is coming from the fact that there's no requirements.txt file for this dependency, it would it great if you can provide one

@Walexero did the pypaystack==1.24 inclusion to the requirement file fix the issue ?

I was able to make it work but I didn't put it into the requirement.txt
file, I copied the files thats needed to initiate the paystack API and
added it as a folder into my project so heroku wouldn't have to install it
as a dependency
On Wed, 1 Dec 2021, 10:17 Nifemi SOLA-OJO, @.***> wrote:

@Walexero https://github.com/Walexero did the pypaystack==1.24
inclusion to the requirement file fix the issue ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edwardpopoola/pypaystack/issues/11#issuecomment-983442332,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVWZ3SZ6FQBHJH7V4WGJTSTUOXRZNANCNFSM5EO7X2MQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi! I've republished this package as pypaystack2
pip install pypaystack2

No major changes were made it still works like
pypaystack but you'll have to import from pypaystack2
more info at PyPaystack2

Just published a new version. I made some changes that aren't backwards compatible with it. But I've tried hard to create a documentation for it you can check it out here. https://pypi.org/project/pypaystack2/

Just added a simple tutorial to the documentation. you can find it out here https://gray-adeyi.github.io/pypaystack2/paystack-client-turorial.html

Merry Christmas developers! I'm glad to announce the release of version 1.0.2 of pypaystack2. Fixed bugs and included the the new Terminal API from paystack and a new way of using the API wrappers more info in the link
https://gray-adeyi.github.io/pypaystack2/how-to-guides/


Powershell Automation of Environment Setup and Scafolding.

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Enhancement / Suggestion
[ ] Additional article idea

I'm trying to build a powershell script to automate all the steps on https://dev.office.com/sharepoint/docs/spfx/set-up-your-development-environment and then run the yo scafolding from
https://dev.office.com/sharepoint/docs/spfx/web-parts/get-started/build-a-hello-world-web-part
I've created the PowerShell Script and It appears to work perfectly until it runs the "yo @microsoft/sharepoint" command. It then seems to hang forever. I've tested the command from a command prompt and Yeoman builds the project perfectly, so I know everything except the way I'm launching yeoman worked.
Here's the powershell script. Can you provide any guidance? I'd like to use the script to lead a usergroup orientation on the framework.
...write-host "Installing Windows Build Tools"
npm install -g --production windows-build-tools
write-host "Installing Yeoman and Gulp"
npm install -g yo gulp
write-host "Installing generator-sharepoint"
npm install -g @microsoft/generator-sharepoint
$ProjectRootDir = Read-Host 'Enter the Root Directory path to install project folder'
If(!(test-path $ProjectRootDir))
{
New-Item -ItemType Directory -Force -Path $ProjectRootDir
}
cd $ProjectRootDir
$ProjectName = Read-Host 'Enter a name for your project'
If(!(test-path $ProjectName))
{
New-Item -ItemType Directory -Force -Path $ProjectName
}
cd $ProjectName
write-host "Running Yeoman generator"
& cmd /c 'yo 2>&1' @microsoft/sharepoint ...

Are you sure its hanging or its just not showing the npm progress in the PowerShell?
I personally don't use PowerShell so not sure where the problem is. Have you tried a .bat file instead to see if that works?

It is hung when I run Yeoman in PowerShell or the Command Prompt (with elevated privileges)  I'm pretty sure it's a PowerShell removing issue when it attempts to load @Microsoft/sharepoint.  I think this one needs a PowerShell Ninja.

@DanKline The reason yo appears to hung is because it's waiting for you to type in the parameters required to build the project scafolding. since yo uses the command line it does not read the commands from a file, you have to type them.... :(

As @dcu-sharepoint noted, Yeoman generator takes the parameters from the prompt. We will leave the issue open as we consider adding a feature in the Yeoman generator in order to accept parameters.

@DanKline
See this issue:
Extension type parameter is not passed to extension generator in<EMAIL_ADDRESS>#623
You should be able to adapt the code to your needs.

@mpasarin one possible solution would be to modify the yo command to accept the parameters being passed in a command file.

We accept command-line parameters for the generator. Some general documentation is here. If this doesn't solve your issue, can you please open another issue.


Custom item is darkness in JEI

Custom item is darkness in JEI

This issue should be fixed in Mystical Agriculture version 3.0.4


Python keeps crashing every 10 - 15 minutes

I have updated Python on my Mac to 2.7.12 and ensured the latest version has precedence in my path but this app still uses 2.7.5 for some reason.
This is the crash report.

Process:         Python [4476]
Path:            /System/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python
Identifier:      Python
Version:         2.7.5 (2.7.5)
Code Type:       X86-64 (Native)
Parent Process:  pokemon-go-map [4417]
Responsible:     pokemon-go-map [4417]
User ID:         501
Date/Time:       2016-10-05 21:20:23.006 -0700
OS Version:      Mac OS X 10.9.5 (13F1911)
Report Version:  11

In Terminal

Mac-mini:~ chris$ python --version
Python 2.7.12

Any idea how to get the app to use 2.7.12 instead ?

Which version of OS X are you using? Python crashed regularly one until I realised I needed Yosemite or later. I was using Mavericks. Upgrading fixed it.

Thanks David. Yes, I'm using Mavericks on my mac mini which is the machine that I want to run this on. It's an older mac mini, so I don't want to upgrade it past Mavericks (performance).  I decided to install the main version of the Pokemon Go Map app (the command line version) and it's been working great.  It's definitely not as easy to install as this version but it has not crashed Python yet.


Sky texture isn't rotated properly

Dramatic Skies demo w/ FabricSkyboxes and FabricSkyboxes-interop

Just saw the new version with the fix, sorry

Dramatic Skies supports FSB out of the box. fabricskyboxes-interop will do nothing here.


[T4A4][T17-B2]Tay Ming Liang

Ready for review

@mlteh Some comments added. Please close the PR after reading comments.


textile: is ListPath blocked by PushPath?

This is likely to be fixed in https://github.com/textileio/textile/pull/402/commits/ab56a2609e00b7646b8a0f03866fdb93f14feaeb


gdal installation issues

UnsatisfiableError: The following specifications were found to be incompatible with a past
explicit spec that is not an explicit spec in this operation (libgcc-ng):

gdal -> libgcc-ng[version='>=7.3.0']
gdal -> libstdcxx-ng[version='>=7.3.0']
gdal -> numpy[version='>=1.14.6,<2.0a0']

The following specifications were found to be incompatible with each other:

I can install gdal successfully into non-binder environment, I think binder is just too big and gdals dependencies too many.


ec2.securitygroup: Avoid late init if it seems unsafe

If the structure of the ingress/egress rules are different, or any port
numbers/protocols do not match, avoid late init because we will not be
able to do it safely. The array does not have a unique key.
Also avoid late init of description because I have not seen that AWS
sets description.
Updates #618

Description of your changes

I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

kubectl create this object with description for one IP range:
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: SecurityGroup
metadata:
  name: sg-1
spec:
  forProvider:
    description: desc
    groupName: sg1
    region: eu-north-1
    ingress:
    - ipProtocol: "tcp"
      fromPort: 2
      toPort: 2
      ipRanges:
      - cidrIp: <IP_ADDRESS>/8
    - ipProtocol: "tcp"
      fromPort: 1
      toPort: 1
      ipRanges:
      - cidrIp: <IP_ADDRESS>/8
        description: "foo"
    vpcId: vpc-...

next, kubectl apply with a completely different rule, and without description:
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: SecurityGroup
metadata:
  name: sg-1
spec:
  forProvider:
    description: desc
    groupName: sg1
    region: eu-north-1
    ingress:
    - ipProtocol: "tcp"
      fromPort: 2
      toPort: 2
      ipRanges:
      - cidrIp: <IP_ADDRESS>/8
    - ipProtocol: "tcp"
      fromPort: 3
      toPort: 3
      ipRanges:
      - cidrIp: <IP_ADDRESS>/8
    vpcId: vpc-...

With the old code, using kubectl get -o yaml we see now that the description: "foo" from <IP_ADDRESS>/8:1 is now set in <IP_ADDRESS>/8:3. This is no longer the case.
    - fromPort: 3
      ipProtocol: tcp
      ipRanges:
      - cidrIp: <IP_ADDRESS>/8
        description: foo
      toPort: 3

what do you think @muvaf

I can write a unit test if you agree that this change is needed.

From what I can tell, we rely on protocol, fromPort and toPort as key in this other PR. Would we be able to rely to those as keys here, too?


[BUG] - Slider Thumb Outline not centered graphically.

NextUI Version
2.3.0
Describe the bug
Outline around the thumb of the slider is not symmetrical - the number of pixels offset varies around the circumference. I'm assuming this is a bug rather than an aesthetic choice, because it doesn't vibe with the otherwise neat and professional look-and-feel. The photos are some cropped screen grabs. The asymmetry seems to vary depending on my browser's zoom setting. E.g. on the larger screenshot the bulge is to the top right, while the 100% zoom setting has a bulge to the bottom/bottom right.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
For me, the asymmetry is visible anywhere the slider is rendered (e.g. the homepage for NextUI: https://nextui.org/docs/components/slider)
Expected behavior
A symmetrical thumb outline.
Screenshots or Videos

Operating System Version

Windows 10 Home V.22H2

Browser
Chrome

Hi @buchananwill @wingkwong
This issue arises due to subpixel rendering, which occurs at various zoom levels in the browser or at the monitor display settings, other than the default 100%. As shown in the image, the width and height are not integer values, causing the circle to be misaligned.

@wingkwong I think this can be solved using circle SVG instead of a div and pseudo-element. Can I give it a try?

@arindam1997007 Go ahead.


Fable integration

In theory, this library could be perfectly used in the Fable ecosystem.
Although it allows heavy use of SRTP, it also permits to manually specify a codec combinator, the same way ELM inspired Json libs do.
On top of that, the feature of specifying a single codec would also work without relying on SRTP overload, by specifying a codec for each field.
The last sample code in the readme https://github.com/mausch/Fleece#combinators illustrate this.
There shouldn't be any issue to use this in Fable.
I'm not sure what is required in order to by Fable ready, all I know is that there are no stoppers.
Is there anyone interested in take it there?
CC: @alfonsogarciacaro @wallymathieu

I had a quick look at the code, if you don't depend on reflection it should work (there's indeed some reflection support in Fable but it's limited). SRTP should work and has been improved in Fable 2, if you find an edge case we can try to fix it in Fable. Though it's the fist time I see the
GeneralizableValue attribute, not sure if it works in Fable.
If you're using Expecto for the tests I recommend just using some Mocha polyfills as Fable itself does, try to run them with Fable and see what fails.
BTW, there's already a similar JSON library for Fable, Thoth.Json. cc @mangelmaxime

At the moment there is zero reflection code in this library.
The generalizable attribute in that example is a bit exotic, the same effect could be achieved by making it a function accepting () instead. I should update that, it looks more complicated than what really is.
We're not using Expecto here, we use Fuchu, which is actually the original project Expecto was based on. So probably we can run those tests with Fuchu as well.
I had a look a Thoth.Json, it seems to be a minimalistic combinator based library which could be seen as a subset of Fleece if you compare them.
On one side, having a simple library could be convenient. The idea of adding more features to Fleece is that it could integrate better with different libraries, being suitable for different scenarios.
Maybe we can join efforts at some point.

I feel like I'm able to work in a flavor compatible with Fable.
But I wonder is there any user having the possibility to use it in such scenario?
So, we can test the design in a real project.

This library uses Fuchu for tests. We could adjust Fuchu to run on both Fable and .net.

IIRC Expecto uses the same API as Fuchu. What we usually do in that case is map the test declaration so they use a JS test runner like Mocha when compiled by Fable. See this and this.