Inconsistent message formats?

Ok, so I have two programs, one in node, another in rust. Rust can use capnp over zmq to communicate with another rust process. Works great. Node can also communicate with another node process. Works great (so far).
Now the node process is supposed to communicate with a rust process. Node can successfully send a request to rust and rust can successfully decode this message and send back a response. But the response I get back from rust looks... wrong? And it throws this exception when I try to parse it:
{ [Error: expected array.size() >= offset + segmentSize; Message ends prematurely in first segment.] cppFile: 'src/capnp/serialize.c++', line: 56 }

Which comes from here:
https://github.com/sandstorm-io/capnproto/blob/master/c%2B%2B/src/capnp/serialize.c%2B%2B#L56
For testing I encoded the expected response in node and the bytes don't match the same response I receive from the rust process:
Response encoded in rust (encoding/decoding works in rust):
<SlowBuffer 00 00 00 00 01 00 01 00 00 00 00 00 00 00 00 00 00 00 00 00 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00>

Same response encoded in node (encoding/decoding works in node):
<SlowBuffer 00 00 00 00 05 00 00 00 00 00 00 00 01 00 01 00 00 00 00 00 00 00 00 00 00 00 00 00 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00>

The lengths are different and it looks like the response from rust is missing a header despite having the same encoded content. Both response schemas are identical, including capnp ids (but not the same files due to different repos).

capnp: v0.5.1
capnp-rust: v0.1.26
capnpc-rust: v0.1.26
node-capnp: master / v0.1.11
Both the request and response schemas make use of a single union field if that makes a difference.

Any ideas for reconciling the two? Is it a version mismatch somewhere? What is the extra header coming from node-capnp? How does rust work without it?
Thanks in advance for any help.
EDIT: If it would be helpful and its not something immediately obvious that I'm missing I'm happy to rig up a GitHub project to replicate the problem.

Looks like the message that was encoded in Rust does not include framing information. See https://capnproto.org/encoding.html#serialization-over-a-stream .
The capnp-zmq-rust stuff uses MessageBuilder::get_segments_for_output() directly because ZeroMQ handles framing. If you're just sending the message over a byte stream, you should use ::capnp::serialize::write_message().

Interesting. That sounds like the problem but in my case both ends (rust and node) are using zmq to pass messages. Can I tell node-capnp not to expect the framing information since, as you say, zeromq should handle it for me or is it just easier to include the framing information redundantly on the rust side? I don't really have any bandwidth constraints so I'm okay with that.

I don't think there's currently a way to tell node-capnp to parse an unframed message directly, though it would probably be straightforward to add such a feature. To work around this limitation, maybe you could construct the segment table yourself before passing the bytes to node-capnp, though it seems like this would necessarily involve copying the whole message.
You might also want to investigate @popham's javascript implementation: https://github.com/capnp-js/serialization

Huzzah! That makes a ton of sense and now everything is working. I actually was using capnp-js because it was supposed to be faster but was told (correctly) that it didn't follow the spec which would cause problems with other languages so I abandoned that approach. Turns out the part it wasn't following was the frames header which I don't want anyway.


[zh-cn] resync /concepts/storage/persistent-volumes.md

Updated some links and resync with the en version.
content/zh-cn/docs/concepts/storage/persistent-volumes.md

Preview: https://deploy-preview-35473--kubernetes-io-main-staging.netlify.app/zh-cn/docs/concepts/storage/persistent-volumes/

/lgtm
/approve
Thanks.


Click version locked to 7.x.x leads to version conflicts

I'm trying to use beanie and celery==5.2.1 inside single environment (Python 3.9) using pipenv.
beanie has requiremnet for click: click<8.0.0,>=7.1.2 (from beanie==1.4.0
celery==5.2.1 has requirement for click:  click<9.0,>=8.0 (from celery==5.2.1
Having both of them inside my Pipfile leads to conflict:
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.
 Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: Could not find a version that matches click<8.0.0,<9.0,>=4.0,>=7,>=7.*,>=7.1.2,>=8.0 (from celery==5.2.1->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 7))
Tried: 0.1, 0.1, 0.2, 0.2, 0.3, 0.3, 0.4, 0.4, 0.5, 0.5, 0.5.1, 0.5.1, 0.6, 0.6, 0.7, 0.7, 1.0, 1.0, 1.1, 1.1, 2.0, 2.0, 2.1, 2.1, 2.2, 2.2, 2.3, 2.3, 2.4, 2.4, 2.5, 2.5, 2.6, 2.6, 3.0, 3.0, 3.1, 3.1, 3.2, 3.2, 3.3, 3.3, 4.0, 4.0, 4.1, 4.1, 5.0,
 5.0, 5.1, 5.1, 6.0, 6.0, 6.1, 6.1, 6.2, 6.2, 6.3, 6.3, 6.4, 6.4, 6.5, 6.6, 6.6, 6.7, 6.7, 7.0, 7.0, 7.1, 7.1, 7.1.1, 7.1.1, 7.1.2, 7.1.2, 8.0.0, 8.0.0, 8.0.1, 8.0.1, 8.0.2, 8.0.2, 8.0.3, 8.0.3
Tried pre-versions: 6.7.dev0, 8.0.0a1, 8.0.0a1, 8.0.0rc1, 8.0.0rc1
There are incompatible versions in the resolved dependencies:
  click (from click-repl==0.2.0->celery==5.2.1->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 7))
  click<8.0.0,>=7.1.2 (from beanie==1.4.0->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 4))
  click<9.0,>=8.0 (from celery==5.2.1->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 7))
  click>=4.0 (from click-plugins==1.1.1->celery==5.2.1->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 7))
  click>=7 (from click-didyoumean==0.3.0->celery==5.2.1->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 7))
  click>=7.* (from uvicorn==0.14.0->-r /tmp/pipenvacpdgwk2requirements/pipenv-sgnizixh-constraints.txt (line 6))

Is there any reason preventing beanie from using click==8.x.x? As I can see, main change in click==8.0.0 was dropping support for Python2

Hey,
Thank you for the catch.
Should be fixed in 1.8.11, pls check


GroupPhotosObservation

Depends on:

[x] #483

This is going to be very similar to <ObsGridItem>, and it might actually be best to do this as a modification of that
component, or maybe it's better to keep it simple and separate from that more complicated component, not sure, but here are the rough requirements:
Props

[ ] photos: array of photo objects represented by this item
[x] selected: boolean for whether or not this "observation" is selected
[ ] width: int, width in pixels

Visual

[x] 15px border radius
[x] circular checkbox in top-end (selected controls checked state) with shadow (this might need to be its own component like CircularCheckbox or something; hoping it can be rendered with CSS and the existing checkmark icon)
[x] <PhotoCount /> in bottom-end
[ ] width and height should be width or 100%

Seems like an easy lift on top of the ObsPreviewImage component, which ObsGridItem just feeds data to anyway.
I can make a pr and see if it fits the requirements.

Closed via #525


Develop

Please consider merging the code. I've improved and optimized it a bit.

These are generally good changes. Thank you!


random_compat 1.x not insecure if on patched PHP version?

Not that it's so relevant anymore, but from the looks of it seems the version rule on random_compat might be a bit too strict, as the underlying lack of randomness with openssl issue seems to have been fixed in later version of PHP:
https://github.com/php/php-src/commit/0e2447cd11f4b72257e5d2609f923177e9736c3c

@andrerom If you think there is something to change, please submit a pull request. I'm mostly merging pull requests here, so any issue won't be fixed by "maintainers" of this repo :)

Posted the relevant question on https://github.com/paragonie/random_compat/issues/96#issuecomment-369504833, based on feedback ew can sew if there is anything we could change here.
Solution could also be a 1.5 which requries the relevant PHP versions so  >=1.5 can be whitelisted.


How to change player variables

What file i need to change gems, coins, unlocked brawlers. I need to remove a lot of gems.
And where i can see skinds ids brawlers ids pins ids

If you want to change the players variables you need to connect to the database or just use an offer and change the player variables in there and for the ids i would say in the game files


Another go on "Solve renaming issues with special characters"

It's a challenge to create a regex that matches all possible names of objects.
Now I've added all printable ASCII characters except ". That should work for files converted from C/AL, but could still miss some Unicode characters if used... Don't really know how to solve that properly.
Anyway, I added a bit to your new test to also check that some characters as µ and åäö still exists after fixing filename.
The test runs fine for me now.
What do you think?

It does error out for me :(.

I'm having a closer look..

Strange. Let me know what your debugging comes up with.

Seems that the  ObjectNamePattern doesn't mach with the object name.  I tested that with regexr.com

This results in an "no match", so it's not able to "decompile" into a NAVObject.  So my guess is to extend that pattern to allow any character... So something like this:

This seems to fix my test, but I'm not sure this is what you agree with :-).
I'll have to hop on a plane now - can you check?
I like your approach better with these variables though - so I really want this in ;-).

It must have been some issues with your merge, I changed the ObjectNamePattern in this PR compared to the first one, but your screenshot looks like the first PR. Check out the "Changes" on this PR and you'll see. I'm mobile right now, so not able to add a decent screenshot. 😉
I don't think that a .* pattern results in any good. It passes this test, but I'm sure that it will match a lot of unwanted results as well. I created a pattern that matches all printable ASCII characters except ".
Fly safe, and let me know when you've tested with this new pattern. 🙂

Now I'm at a computer again...
I changed the ObjectNamePattern in NAVObject.ts in this PR:

And a test on regexr.com show that it works pretty well on the test object:

At least one scenario that might fail is if someone got some a couple of " in the object names...
If a name is not matched, we might want to throw a more user friendly error... Ie. if currObject = null -> Error(The name xxxx is not allowed) or something similar. The "null reference"-ish error is a bit too non informative... The error should point the user to what file is causing issues.

@waldo1001 I think I have done the object name pattern a bit too complicated!
What about this simple pattern:

Sorry to spam this PR, @waldo1001.
But now I'm happy with how it works for me anyway.
Added test for object names containing ".
Also shows user friendly messages/errormessages for the "Reorganize All" and "Rename All" actions.
I will not do more on this now, if you don't say otherwise.

No problem :) - sorry for not having time to look at it - quite busy in Dubai at the moment.. .  And I want to give it the attention it deserves - one of these days, I'm certainly going to check it out!

No worries.
I've been able to reorganize my project now, so I'm happy. Look into this when you've got time to, no rush 🙂
Enjoy Dubai!


Implement RethinkDB cluster

Needs to be done in conjunction with Kubernetes support for replicas and persistent storage (i.e. PetSet proposal).

Did it with a separate CoreOS cluster.


Largest Size Always Loaded In Chrome

Firstly, apologies if I'm completely misunderstanding something, but in switching from Picturefill to Lazysizes/respimage, I noticed the images with the largest width in the srcset would always be used, regardless of the initial screen or image size. I assumed I had made a mistake, but looking at the examples on your demo page, I see the same thing happening.
The markup for your third example – responsive image with srcset and sizes attribute  – as rendered in Chrome 44.0.2403.125 (64-bit) OSX:
<img sizes="245px" data-sizes="auto" data-srcset="https://farm5.staticflickr.com/4078/5441060528_31db7838ba_m.jpg 240w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_n.jpg 320w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba.jpg 500w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_z.jpg 640w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_b.jpg 1024w" class=" no-src lazyautosizes lazyloaded" srcset="https://farm5.staticflickr.com/4078/5441060528_31db7838ba_m.jpg 240w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_n.jpg 320w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba.jpg 500w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_z.jpg 640w,
    https://farm5.staticflickr.com/4078/5441060528_31db7838ba_b.jpg 1024w">

Setting the browser width to 300px wide (meaning the image is 265px wide) and refreshing the page results in a single image download - of the largest image: https://farm5.staticflickr.com/4078/5441060528_31db7838ba_b.jpg 1024w

Surely in this situation, the smallest image should be served given the window / image dimensions.

It's a feature not a bug. Chrome does not switch the size because Chrome already has a larger image in cache, so there is no need to download new data of the same image.
You can test whether the right size would be chosen, by disableing the cache.

Ah. Of course. That makes perfect sense. Thanks a lot for this. It's really nicely thought out.

This is a native Chrome feature. It is impossible to replicate with JS. In my opinion this feature should be further optimized by Chrome. If the "perfect" but smaller sized candidate is also cached Chrome should decode the smaller image candidate. This would be better for decoding and resize performance and also for memory.

This is a sh**, chrome always loads the max size even for the first time
i removed all the cache, and i refreshed the page, chrome always loads the max size instead of small one!

@rafikalid: Reduce the pixel ratio to 1 in your mobile emulation view.


Android audio track

I've added the possibility to use custom audioTracks on android part. As you know that you cant add custom audio track in code part on Android. You just select audio track that whit in video chanel. But I have need custom mp3 file add in code part. So in IOS part @reime005 done it by using alternativeAudioSource. Thanks a lot for that. Developed android part by using audioTracks props by me. Now You can add custom audio file in array like textTracks. So just add like this:
 audioTracks={[
              {
                language: 'en',
                title: 'English CC',
                type: 'audio/mp4a-latm',
                uri: "http://webaudioapi.com/samples/audio-tag/chrono.mp3"
              },
              {
                language: 'en',
                title: 'English CC',
                type: 'audio/mp4a-latm',
                uri: "https://www.soundhelix.com/examples/mp3/SoundHelix-Song-1.mp3"
              }
            ]}

In summary, You can use alternativeAudioSource for custom audio in IOS and you can use audioTracks in Android.
instalition with yarn:
yarn add  yasinugrl/react-native-video#androidAudioTrack
with npm:
npm i reime005/react-native-video#audioTrack --save

Thanks for that! I've used the workaround of adding the 2nd audio track to the video(s), using ffmpeg. That's better now


Can it work postgres to postgres ?

I am using postgres only as low-cost data warehouse
Can I sync between postgres to postgres using this ?
I want to use incremental sync method using last_modified method

Unfortunately I haven't built a Postgres warehouse backend yet... Shouldn't be too hard. I'll update this ticket if I get a chance to work on it.


[Runtime] Assert that metadata mangled names successfully roundtrip.

This stress tests the mangler and demangler in builds with asserts enabled. When a new metadata object is created, it adds a verification step which gets the metadata's mangled name, runs the demangler to get the metadata back, and verifies it gets the same thing that went in.
This assertion failure is non-fatal and just logs a warning. A bunch of tests trigger it, so this avoids the need to fix all of those tests at once. When the tests pass again, this can be made into a fatal error.
It's also disabled entirely when the SWIFT_DISABLE_MANGLED_NAME_VERIFICATION environment variable is set. A bunch of tests trigger crashes in the mangler/demangler, and they use that to disable the check, again avoiding the need to fix them all at once. When these pass again, the environment variable can be removed.
rdar://problem/37551850

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - c72aeb09c267d011c0c355cb2620f2d941101192

Looks good to me! The setting-environment-variables-in-RUN-lines feels to me like there might be a more idiomatic solution available with lit, but I'm not the expert to say what that might be. Maybe @gottesmm has recommendations.

@swift-ci please test

@swift-ci please test

Looks like you caught something(s) on the source compat bot:
https://ci.swift.org/job/swift-master-source-compat-suite/1696
Assertion failed: (!isDigit(ident[Pos]) && "first char of sub-string may not be a digit"), function mangleIdentifier, file /Users/buildnode/jenkins/workspace-private/swift-master-source-compat-suite/swift/include/swift/Demangling/ManglingUtils.h, line 195.

We caught a similar-looking crash in rdar://problem/39826794 too.

I'm switching this to default to off in #16253. Rather than try to track down all the tests it breaks, we can turn it on in development and find stuff there.


genotype_reader_tables() error

I have recently update my local installation of limix. It is working through Anaconda 2 v. 4.4.0.
Limix version is: u'0.7.12'
The installation went fine trough conda but when I use genotype_reader_tables() to read an hdf5 genotype (the same file was working with previous version) it gives me this error:

geno_reader  = gr.genotype_reader_tables(geno_file)
Traceback (most recent call last):
File "", line 1, in
File "/g/furlong/forneris/software/anaconda2/lib/python2.7/site-packages/limix/io/genotype_reader.py", line 236, in init
self.load()
File "/g/furlong/forneris/software/anaconda2/lib/python2.7/site-packages/limix/io/genotype_reader.py", line 248, in load
self.f = tables.openFile(self.file_name,'r')
AttributeError: 'module' object has no attribute 'openFile'

Hi @mforneris . Limix is now on version 1.0.9: https://pypi.python.org/pypi/limix/1.0.9
Have a look at that more recent version. It will probably work better for you...

Thank you for taking care of it.
The update through pip worked and the package is loading properly.
I can see that some function changed their names though :/
For example
And all io packages work only if loaded from limix_legacy and not from limix
genotype_reader_tables now is genotype_reader_h5py
This is quiet annoying because is going to break all my scripts…
Is there any chance you will develop “backward compatibility" for the old functions? Can I return to an older limix version?
M.

On 7. Jun 2017, at 11:55, Danilo Horta<EMAIL_ADDRESS>wrote:
Hi @mforneris https://github.com/mforneris . Limix is now on version 1.0.9: https://pypi.python.org/pypi/limix/1.0.9 https://pypi.python.org/pypi/limix/1.0.9
Have a look at that more recent version. It will probably work better for you...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PMBio/limix/issues/15#issuecomment-306748167, or mute the thread https://github.com/notifications/unsubscribe-auth/Ab2PGZ5lGcpLVvqxdYYzdv9S0Y-rOZx1ks5sBnOMgaJpZM4Nv8_g.

Sorry to hear that. Limix 1.x is a major release and as such breaks some functionalities. However, there will always be the limix_legacy package (pip install limix-legacy) for old functionalities, or you can specify an older limix version to install: pip install limix=0.7.0 for example...


Missing documentation and example for cache behavior parameters

Description
In the module itself, you support the use of trusted_key_groups, trusted_signers, smooth_streaming and more for ordered and default cache behaviors. However the usage is missing from your "complete" example as well as the documentation .

I have submitted a PR for the same. The example uses use_forwaded_values and legacy settings

A complete example shows many arguments, but it would be nearly impossible to maintain and very hard to understand if there were all of them and supplementary resources. That is why we show the most common ones.
Adding a smaller subset of features into a separate example (like in #143) does not make it easier because there is no way to identify which feature to include.
Let's stick to the current complete example and maybe extend arguments if they are very important and used often. smooth_streaming - is not important, but trusted_key_groups and trusted_signersare good to have in the examples/complete with all required supplementary resources (such as aws_cloudfront_key_group, aws_cloudfront_public_key).


Hook: Create issues

seabird should be able to create additional issues for itself.
Example:
user: seabird: Issue: You're broke; seabird is broken and not properly working.

Put these in two different plugins probably.


undefined method `segment_keys' for nil:NilClass

on 0.1.4
visits the page, I get this error
Request parameters
{"action"=>"index", "controller"=>"sidekiq/monitor/jobs"}
block (2 levels) in define_generate_prefixactionpack (4.0.5) lib/action_dispatch/routing/mapper.rb
define_method :_generate_prefix do |options|
prefix_options = options.slice(*_route.segment_keys)
# we must actually delete prefix segment keys to avoid passing them to next url_for
_route.segment_keys.each { |k| options.delete(k) }
_routes.url_helpers.send("#{name}_path", prefix_options)
end
lib/action_dispatch/routing/mapper.rb 549
on rails 4.0.5

Any update/ideas on this?
Best,
Vinny

It's happening for me also. Anyone have an idea?


[WIP] Starting figuring out how to not remove wrappers

This is a work in progress to add an argument to not delete wrappers - I'm fairly new to the code base so it will also be a good practice in getting familiar. I'm currently trying to figure out all the points that we would want to check for the "don't do the delete" argument and then not do the cleanup, and I'm opening this PR (against my own branch) to get some help about where I need to be looking. I've added print debug statements, and then tested running a pipeline, but I don't see them being printed so I suspect I'm missing something!
Signed-off-by: Vanessa Sochat<EMAIL_ADDRESS>

@johanneskoester for your comment about where I opened the PR - I opened it intentionally here so that I could show you the diff between the current and my changes. I'll open the PR against the upstream (snakemake) when the PR is ready for review.

@johanneskoester I'm still trying to figure out how to pass the parameter to the script. It looks like a script is part of a rule, and so far I've traced:

we start with the snakemake function, that generates a workflow (with a subsnakemake partial) and then we execute it.
the execution generates rules

workflow._rules
Out[163]:
OrderedDict([('a', <snakemake.rules.Rule at 0x7f7b6ff8d208>),
             ('b', <snakemake.rules.Rule at 0x7f7b6fcc1198>),
             ('c', <snakemake.rules.Rule at 0x7f7b6fcc1c88>)])

and also a JobScheduler (I think this isn't related as you mentioned so I won't get distracted here). Each rule has a script that starts out empty:
 rule
<snakemake.rules.Rule at 0x7f7b6fcc1198>
rule.script

The rules are given to a DAG and a persistence.Persistence object (not sure what these are), there's also a RuleInfo object that seems to work via passing around this "decorate" function everywhere. I can see that script is imported into workflow.py and it gets passed from a ruleinfo.script
rule.script = ruleinfo.script

I've been interactively trying to walk through a dummy example and figure out how an input arg into snakemake even gets to the script.py file - two and a half hours now and I haven't had a clue, the logic of Snakemake is too complex for me to get my head around and I'm at a stopping point. How does the argument get passed to the script function, where we can designate for it not to be cleaned up?

It looks like the wrapper also has a script created during it's init, but rule.wrapper is None for all my rules in this test case.

Yeah, the route is complicated because of the way Snakemake implements it's DSL.

You need to add the additional parameter for the script function call here, and here.
You need to add the parameter to the run_wrapper here, and to the calls of the run function in that wrapper.
You need to add it where the arguments for the run_wrapper are prepared here.

And yes, this is cumbersome and should be refactored at some point, into a parameter class that can be passed around as a single argument. It has grown over time, and we should've done the refactoring after the fifth parameter was added or so. If you want you can also do that now.
class RunParams:
    def __init__(self):
        self.rulename = None
        self.input = None
        ...

class WrapperParams(RunParams):
    ...

And so on.

@johanneskoester I think we're missing one more spot for a rule maybe?
exception calling callback for <Future at 0x7f178fefe898 state=finished raised TypeError>
Traceback (most recent call last):
  File "/home/vanessa/Documents/Dropbox/Code/Python/snakemake/snakemake/executors.py", line 2035, in run_wrapper
    passed_shadow_dir
TypeError: __rule_b() takes 18 positional arguments but 19 were given

Woohoo found it! It's under Run
class Run(RuleKeywordState):
    def __init__(self, snakefile, rulename, base_indent=0, dedent=0, root=True):
        super().__init__(snakefile, base_indent=base_indent, dedent=dedent, root=root)
        self.rulename = rulename
        self.content = 0

    def start(self):
        yield<EMAIL_ADDRESS>        yield "\n"
        yield (
            "def __rule_{rulename}(input, output, params, wildcards, threads, "
            "resources, log, version, rule, conda_env, singularity_img, "
            "singularity_args, use_singularity, bench_record, jobid, "
            "is_shell, bench_iteration, cleanup_scripts, shadow_dir):".format(
                rulename=self.rulename
                if self.rulename is not None
                else self.snakefile.rulecount
            )
        )


Allow for camelCase type marshalling

Go-Jet by defaults doesn't provide json annotations for marshaling.
type Message struct {
	ID      *int32 `sql:"primary_key"`
	Content *string
}

Is it possible to auto generate json marshaling annotations by looking at the table names in the schema for example?
Is it possible to toggle between camelCase and TitleCase? Right now it uses TitleCase but camelCase is recommended for JSON (https://jsonapi.org/recommendations/)
Also, the table scheme calls the field id yet somehow Jet turns it into ID instead of Id.

Hi @tommysalt ,
Yeah, it is possible, but you would need to customize generator first. Take a look at this example.

Ok,


Automatically resolved path parameters are always marked as optional

I see that api_view.resolve_path_parameters is annotated to return Optional[Type[pydantic.BaseModel]] since it can return None if there are no path parameters defined.
This seems to have a side effect of making all path parameters as optional (required: false) in the openapi spec. Is there any way around this other than specifying path_parameters explicitly on the endpoint definition?
This endpoint:
    get_object = views.ReadView(
        path="/{external_id}",
        get_model=lambda request, path_parameters: ViewSet.model.objects.get(external_id=path_parameters.external_id),
        response_body=ObjectSchema,
    )

Results in this def:
"/api/me/obj/{external_id}": {
      "get": {
        "operationId": "get_object",
        "summary": "Get Object",
        "parameters": [
          {
            "in": "path",
            "name": "external_id",
            "schema": {
              "format": "uuid",
              "title": "External Id",
              "type": "string"
            },
            "required": false
          }
        ],
        "responses": {
          "200": {
            "description": "OK",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/ObjectSchema"
                }
              }
            }
          }
        },

Hi @jboutros, thanks for reporting this issue! While the issue isn't related to the signature of api_view.resolve_path_parameters, you're right that the behavior is not normal - since v1.2, Django Ninja has been wrapping pk types in Optional (see ninja/orm/fields.py), likely for automatic ModelSchema definition. This wasn't appropriate for path parameters, and I wasn't sure how to address it. I've now found a solution (cf. #475) to make all automatically inferred path parameters mandatory (which aligns with RESTful API conventions where optional parameters should be query parameters):
schema_fields: Dict[str, Any] = {}
for field_name in path_parameters_names:
    model_field = self.model._meta.get_field(field_name)
    schema_field = ninja.orm.fields.get_schema_field(field=model_field)[0]
    schema_fields[field_name] = (
        typing.get_args(schema_field)[0]
        if typing.get_origin(schema_field) is Union
        else schema_field,
        ...,
    )

This ensures that automatically resolved path parameters aren't wrapped in Optional anymore, correctly marking them as required in the OpenAPI spec.
PS: Unrelated to this issue but worth noting: your custom get_model implementation is actually the default behavior. You can simplify your code by removing this parameter, as ReadView already implements this logic:
def _default_get_model(
    self, request: HttpRequest, path_parameters: Optional[BaseModel]
) -> Model:
    return self.model.objects.get(
        **(path_parameters.model_dump() if path_parameters else {})
    )

I'll include this fix in the next release v0.6.2, which should be available tonight or tomorrow. Thanks for helping improve Django Ninja CRUD! 🌞

Hi @jboutros, great news! I've just released v0.6.2 with the fix. All path parameters should now be correctly marked as required in the OpenAPI spec. Check out the release notes here: https://github.com/hbakri/django-ninja-crud/releases/tag/v0.6.2
Thanks again for your help in improving Django Ninja CRUD! Please let me know if you encounter any issues with the new version or anything else.

thank you @hbakri I look forward to trying this out. And thank you for all of your hard work creating and maintaining such a useful library!


'exact' mode

I prefer to use --save-exact when doing npm install. Can we have an option to use that?

Duplicate of #50


Bump checkstyle from 9.3 to 10.17.0

Default version of checkstyle is 9.3
Latest release: https://mvnrepository.com/artifact/com.puppycrawl.tools/checkstyle

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Checklist

[ ] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

See #2754


Validation: Update README.md

Updated absolute links per @kushagraThapar

@v-emilypr - thanks a lot for creating this PR.
I looked at the builds and found that exactly what issues we are trying to fix in the docs, are not allowed to be fixed because of the CI - see this failure - https://dev.azure.com/azure-sdk/public/_build/results?buildId=1524389&view=logs&jobId=b70e5e73-bbb6-5567-0939-8415943fadb9&j=b70e5e73-bbb6-5567-0939-8415943fadb9&t=d155e9a2-7cbb-5b0e-f3c1-03b63fe55961
It is not allowing us to put relative links.
Also my concern is that these links will not work from the README.md page in the repo. For example, if you go to your branch here - https://github.com/v-emilypr/azure-sdk-for-java/tree/patch-1/sdk/cosmos/azure-cosmos-encryption
And open any links that you have changed, they will not work.
At this point, I am not sure what would be the way to fix these links just for the docs pages, and not for the repo and anywhere else this README.md file is used. Thoughts?

@kushagraThapar
Thank you for the follow up, let me chat with Andrea about how best to move forward with this.

@kushagraThapar
Thanks for sharing this! This seems like a situation where relative links just won't work. This is a new one for our team, so I'm sorry for all of the runaround, and I really appreciate you taking the time to figure this out. This PR can be closed and we will make a note that the absolute links shouldn't be changed.

@v-andreaco no worries at all, I understand :)
Thanks for the effort and your understanding, appreciate it.


originator is a bad choice for method name

This is really a point of feedback and not a bug.
Many of the papertrail objects that get patched onto a has_paper_trail object are very generic in namespace. I would strongly recommend a namespacing convention (like pt_) for all papertrail methods so that they don't conflict with other method names in the app.
"originator", in particular, makes this gem basically unusable with the Spree ecosystem because many objects in Spree have their own originator method.
I tried a monkey patch using alias_method chain and got a little bit of the way but still it's a bad pattern and I'd recommend changing originator to pt_originator which will surely not cause naming conflicts

Did you also make a suggestion to Spree to namespace their methods?

it's a relationship that describes a first-order relationship between two objects. it seems to me that papertrail is more of an add-on and not a first-class system within an app, and using the method 'originator' on all papertrail-tracked objects seems like it surely is asking for collision with anyone else's use of the (relatively generic) term "originator"
It's been removed in later versions of Spree anyway.

Ben, are you still planning on changing this method name?  If so, is this something you'd like help with, or is it too tricky and you want to do it yourself?  I'd like to help get a 4.0 release candidate out the door.

No this one is relatively straightforward. It's #518 I'm trying to work on, hopefully will have something by EOD with a fix for the test cases.


Test of await, language/await_future_test, times out on Chrome 41 on macos.

FAILED: dart2js-chrome release_ia32 language/await_future_test
Expected: Pass
Actual: Timeout
CommandOutput[dart2js]:
There are no delays or timers in the test, so it almost certainly is hanging when it times out.  This seems quite bad, and it is in the new await code in dart2js, so marking as high priority.
It failed (timed out) in 3 runs out of 10 on the buildbot.

I looked through the last 100 logs of the mac-chrome bot (with a script). And it seems to not time out anymore. Is it a newer Chrome on the bot?

Yes, the debug log (which can be clicked on in the buildbot) says it is on
version 43.  We can probably re-enable the tests.
https://chromegw.corp.google.com/i/client.dart/builders/dart2js-chrome-mac10.9-be/builds/942/steps/annotated_steps/logs/debuglog
On Thu, Jun 4, 2015 at 3:16 PM, Sigurd Meldgaard<EMAIL_ADDRESS>wrote:

I looked through the last 100 logs of the mac-chrome bot (with a script).
And it seems to not time out anymore. Is it a newer Chrome on the bot?
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/22695#issuecomment-108894425.

--
William Hesse

Yes, the debug log (which can be clicked on in the buildbot) says it is on version 43.  We can probably re-enable the tests.
http://build.chromium.org/p/client.dart/builders/dart2js-chrome-mac10.9-be/builds/942/steps/annotated_steps/logs/debuglog

Great! - I will reenable the tests.

Nope - my script was looking in the wrong places. The tests are still failing.

We are having lots of timeouts on many tests on Chrome on macos.  Here is a list of the most recent.  So it is not all async/await support.  And these are not tests that are taking a long time normally, when they do pass.
html/custom/attribute_changed_callback_test/fully_supported
html/custom/created_callback_test
html/custom/entered_left_view_test/disconnected_subtree
html/indexeddb_3_test
html/indexeddb_3_test
html/indexeddb_4_test
language/asyncstar_yield_test
lib/async/catch_errors18_test
lib/async/stream_event_transformed_test
lib/async/stream_event_transformed_test
This one does take 20s normally, but fails with 1 minute.
lib/convert/streamed_conversion_json_encode1_test

Tracked in meta-issue https://github.com/dart-lang/sdk/issues/23980

I am removing the two remaining timeout status lines for this issue (language/async_await_test and language/await_future_test), as the scattered timeouts are being tracked in #23980, and there is no evidence these timeouts have a different cause than the other, unsuppressed ones.  Closing this issue, leaving #23980 open.


Deprecate listenToFormUpdated

Replaced syncFormWithViewModel with deprecated listenToFormUpdated.

I need to figure out these failing CI tests

@ferrarafer the unit test can be fixed by updating the expected generator output for that test that's failing.

@ferrarafer the unit test can be fixed by updating the expected generator output for that test that's failing.

Fixed ✅

Thanks @ferrarafer merging in then I'll work on integration tests failing on separate PR.


Make catalogsource compatible with restricted SCC enforcement

Restricted SCC enforcement will be added with OCP 4.14
Updating the catalogsource to allow the operator to get deployed
Clusters that don't support the setting (<4.12) will ignore it

Jira: OSD-15618

/lgtm

Codecov Report

Merging #138 (4d69f34) into master (c4faba7) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #138   +/-   ##
=======================================
  Coverage   61.14%   61.14%
=======================================
  Files           8        8
  Lines         471      471
=======================================
  Hits          288      288
  Misses        176      176
  Partials        7        7


 doesn't work with v-show

Version
2.5.16
Reproduction link
https://jsfiddle.net/tucyeg82/
Steps to reproduce
Run the jsfiddle example
What is expected?
The documentation for  says it works with v-show, but only with a single element
It is expected to works in a similar fashion as v-if like this
What is actually happening?
Error is thrown saying needs transition-group

It does work for me try to put 2 seconds to your transition to see it. But yes you can have only one element in the transition tag else you need to use transition-group that works the same way.
For your example you can bind the value to the element like:
<transition name="fade" mode="out-in">
  <p :key='flag'>{{ flag }}</p>
</transition>

or use v-if  and v-else instead of v-show because the v-if can't render the 2 elements at the same time while v-show technically can (even if it's never going to happen with your condition).

What @maxirozay said

Hi, @maxirozay, @posva
As it stated  and observed, a  is used with v-if, if no mode is specified, there is clearly a moment there is more than one element exists.
The place I am trying to use v-show is with two different custom components, not as simple as my example.
Using transition-group has an apparent drawback of layout clashing because the mode attribute is not supported.
The reason I am trying to use v-show is to use v-show to prevent the component from being destroyed to keep the component state. (I tried to use keep-alive, but seems keep-alive is not designed for my case)
So, does it make sense to have a feature request for support of  with v-show?
Or do you have some other suggestion for my keep-alive needs?
Sincerely,

I don't see why keep-alive wouldn't work but that would rather be a discussion on the forum or the discord server 😄
I'm sure you will find your way 😉


Fix issue with k8s.io/docs/reference/command-line-tools-reference/fea…

…ture-gates/
Closes #23678
Problem:
Line 426: broken link
- `DisableAcceleratorUsageMetrics`: [Disable accelerator metrics collected by the kubelet](/docs/concepts/cluster-administration/monitoring.md).
Line 477: duplicated
- `LocalStorageCapacityIsolationFSQuotaMonitoring`: When `LocalStorageCapacityIsolation` is enabled for [local ephemeral storage](/docs/concepts/configuration/manage-compute-resources-container/) and the backing filesystem for [emptyDir volumes](/docs/concepts/storage/volumes/#emptydir) supports project quotas and they are enabled, use project quotas to monitor [emptyDir volume](/docs/concepts/storage/volumes/#emptydir) storage consumption rather than filesystem walk for better performance and accuracy.
[local ephemeral storage](/docs/concepts/configuration/manage-resources-containers/) and the backing filesystem for
[emptyDir volumes](/docs/concepts/storage/volumes/#emptydir) supports project quotas and they are enabled, use project quotas to monitor
[emptyDir volume](/docs/concepts/storage/volumes/#emptydir) storage consumption rather than filesystem walk for better performance and accuracy.
Proposed Solution:
Line 426:
- `DisableAcceleratorUsageMetrics`: [Disable accelerator metrics collected by the kubelet](/docs/concepts/cluster-administration/system-metrics/).
Line 477:
- `LocalStorageCapacityIsolationFSQuotaMonitoring`: When `LocalStorageCapacityIsolation` is enabled for [local ephemeral storage](/docs/concepts/configuration/manage-compute-resources-container/) and the backing filesystem for [emptyDir volumes](/docs/concepts/storage/volumes/#emptydir) supports project quotas and they are enabled, use project quotas to monitor [emptyDir volume](/docs/concepts/storage/volumes/#emptydir) storage consumption rather than filesystem walk for better performance and accuracy.
Page to Update:
https://kubernetes.io/docs/reference/command-line-tools-reference/feature-gates/

Preview page: https://deploy-preview-23679--kubernetes-io-master-staging.netlify.app/docs/reference/command-line-tools-reference/feature-gates/#feature-gates

/lgtm
/approve


Reivse FerGrid/FerAxis to support curvilinear grids

Reported by karl smith on 12 May 2016 19:14 UTC
The current FerGrid and FerAxis definitions only work for rectilinear grids (orthogonal axes).  Curvilinear grids need to be supported.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2414

Modified by karl.smith on 12 May 2016 19:14 UTC


x/tools/gopls: detect tools.go files, and suppress warning

I've been looking for a way to fix this for quite a bit of time and nothing I've tried has worked. I'm using oapi-codegen to generate boilerplate for my webserver. The authors of this library recommend the tools.go pattern for managing the dependency, so that go.mod can act as the single source of truth for versioning. For transparency, their readme recommends this here. Doing this actually works -- I can run go generate just fine and my code builds. However, VSCode insists on showing an error and warning in this tools.go file, marking the file and the module folder red:

This is driving me crazy. I'd like to at least suppress these, but I can't find any way to do so. I've tried using the lintFlags to exclude it, but it appears to be gopls and not staticcheck, so this doesn't do anything. I can't find a gopls configuration setting that will allow me to ignore this file or any issues with it, either.
For what it's worth, someone else noticed the same thing with a different library in this issue.
What version of Go, VS Code & VS Code Go extension are you using?
Version Information

Run go version to get version of Go from the VS Code integrated terminal.

go version go1.22.5 darwin/arm64

Run gopls -v version to get version of Gopls from the VS Code integrated terminal.

golang.org/x/tools/gopls v0.16.1

Run code -v or code-insiders -v to get version of VS Code or VS Code Insiders.

Version: 1.92.1

Check your installed extensions to get the version of the VS Code Go extension

v0.42.0

Run Ctrl+Shift+P (Cmd+Shift+P on Mac OS) > Go: Locate Configured Go Tools command.

# Tools Configuration

## Environment

GOBIN: undefined
toolsGopath:
gopath: /Users/[username]/go
GOROOT: /opt/homebrew/Cellar/go/1.22.5/libexec
PATH: /opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin:/Users/[username]/Library/Application Support/JetBrains/Toolbox/scripts:/Users/[username]/go/bin

## Tools

	go:	/opt/homebrew/bin/go: go version go1.22.5 darwin/arm64

	gopls:	/Users/[username]/go/bin/gopls	(version: v0.16.1 built with go: go1.22.5)
	gotests:	not installed
	gomodifytags:	not installed
	impl:	not installed
	goplay:	not installed
	dlv:	/Users/[username]/go/bin/dlv	(version: v1.23.0 built with go: go1.22.5)
	staticcheck:	/Users/[username]/go/bin/staticcheck	(version: v0.4.7 built with go: go1.22.5)

## Go env

Workspace Folder (point-system): /Users/[username]/Documents/Repositories/point-system

	GO111MODULE=''
	GOARCH='arm64'
	GOBIN=''
	GOCACHE='/Users/[username]/Library/Caches/go-build'
	GOENV='/Users/[username]/Library/Application Support/go/env'
	GOEXE=''
	GOEXPERIMENT=''
	GOFLAGS=''
	GOHOSTARCH='arm64'
	GOHOSTOS='darwin'
	GOINSECURE=''
	GOMODCACHE='/Users/[username]/go/pkg/mod'
	GONOPROXY=''
	GONOSUMDB=''
	GOOS='darwin'
	GOPATH='/Users/[username]/go'
	GOPRIVATE=''
	GOPROXY='https://proxy.golang.org,direct'
	GOROOT='/opt/homebrew/Cellar/go/1.22.5/libexec'
	GOSUMDB='sum.golang.org'
	GOTMPDIR=''
	GOTOOLCHAIN='auto'
	GOTOOLDIR='/opt/homebrew/Cellar/go/1.22.5/libexec/pkg/tool/darwin_arm64'
	GOVCS=''
	GOVERSION='go1.22.5'
	GCCGO='gccgo'
	AR='ar'
	CC='cc'
	CXX='c++'
	CGO_ENABLED='1'
	GOMOD='/dev/null'
	GOWORK=''
	CGO_CFLAGS='-O2 -g'
	CGO_CPPFLAGS=''
	CGO_CXXFLAGS='-O2 -g'
	CGO_FFLAGS='-O2 -g'
	CGO_LDFLAGS='-O2 -g'
	PKG_CONFIG='pkg-config'
	GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/by/cxzl4qc57c3c8dd9p9snhn680000gn/T/go-build3419525479=/tmp/go-build -gno-record-gcc-switches -fno-common'

Share the Go related settings you have added/edited
{
    "go.lintTool": "staticcheck",
    "go.lintFlags": [
        "-exclude=gateway/tools/tools.go"
    ]
}

I'm also seeing this when trying to import packages that make use of this pattern:

found packages crossplane (analyze.go) and tools (tools.go) in<EMAIL_ADDRESS>go list

Would love to see this get resolved!

With 1.24 adding first class support for tools in the go.mod file, this issue is a much lower priority.
https://tip.golang.org/doc/go1.24#go-command
We will probably not resolve the current issue, instead letting folks take advantage of the new tool directive. (We have extremely limited resources, and adding a workaround for an obsolete convention unfortunately ranks pretty low).


www not redirecting to non-www

Example URL(s)
When trying to access my website via www.jrroofinglancs.co.uk it is appearing as not secure.
Actual Behavior
It is appearing as not secure and not directing to non-www site.
Expected Behavior
It should direct to https://jrroofinglancs.co.uk/
Additional information
Page rule is set up in cloudflare. A records are both pointing to same IP. Your help would be much appreciated.

Howdy @jrroofinglancs, we can't provide technical support here (it's a repo for our docs).
You might want to take a look at our doc on this issue though.
If that doesn't help, I'd encourage you to instead reach out in the Cloudflare Community or Discord.


[Bug]: Redirection issue occurs when a user logs in for the first time using Auth0 OIDC

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Redirection issue occurs when a user logs in for the first time using Auth0 OIDC

Steps To Reproduce

go to nightly instance
In admin setting go to authentication
Set up OIDc with Auth0 idp
Now signout and try to login with OIDC
The user see the error as in the attached screenshot
Now Refresh the the page and sign in again
The user will be signed in without any issue

Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
Production

@nsarupr Will you be able to have a look on this issue ?


Upgrade ts 3.1

Use TS 3.1 as Angular now requires it (see https://github.com/angular/angular/pull/26151).
You can find a more simple solution to this in #12415. :warning: this PR embed the commit from this other PR.

Thanks for chipping in - as you saw we already started the update (we hoped to release on Friday along with the framework but ran out of time.)


295 lesson7

Summary
lesson 7 finished
Closes #

Screenshots

closes #295, #294, and #211

closes #295, #294, and #211

Move this into the top comment so it links to the tickets correctly

Ticketed the image issue but otherwise looks good! Good work!

Okay, I will work on the image issue. Thank you for all the help!


(doc) Add "Table of Contents" in ReadMe.md

This helps quickly find the content we need ;)

@PeterDaveHello thanks for this - hopefully you used a tool like Alan Walk's Markdown TOC - https://github.com/AlanWalk/Markdown-TOC#3-usage.

I'd pull it in, but it doesn't appear I have the good permissions yet - paging @gep13

Oh, I use vim, but maybe we can integrate preferred tool with CI if it's needed ;)

@PeterDaveHello your changes have been merged, thanks for your contribution 👍


Update Slack API

update forked repo if nlopes doesn't support channels.delete.
https://github.com/TimDurward/terraform-provider-slack/issues/11

Resolved
https://github.com/TimDurward/slack/pull/3/


[REQUEST] NS SWITCH : HIIRO NO KAKERA Tamayori Hime Kitan

https://vndb.org/r119577

I think this would probably be the same engine as https://github.com/0xDC00/scripts/issues/361
So maybe another one @Kalleo1 could figure out  (when they are back from their break) 🙇♂🙇♂🙇♂

Done 829880a


Request to merge TEAMNADO-4319 into "main-stable"

Codecov Report
Base: 89.53% // Head: 89.17% // Decreases project coverage by -0.36% :warning:

Coverage data is based on head (e54b77f) compared to base (fae6bb4).
Patch coverage: 77.77% of modified lines in pull request are covered.

:exclamation: Current head e54b77f differs from pull request most recent head 2a50ec7. Consider uploading reports for the commit 2a50ec7 to get more accurate results

Additional details and impacted files
@@               Coverage Diff               @@
##           main-stable     #157      +/-   ##
===============================================
- Coverage        89.53%   89.17%   -0.37%
===============================================
  Files               40       40
  Lines              669      702      +33
  Branches           127      134       +7
===============================================
+ Hits               599      626      +27
- Misses              70       76       +6

Impacted Files
Coverage Δ

...mponents/CreateManifestForm/CreateManifestForm.tsx
83.05% <77.27%> (-1.57%)
:arrow_down:

...firmationModal/DeleteManifestConfirmationModal.tsx
72.72% <100.00%> (ø)

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

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


Please upload a complete video tutorial of it who is successful in doing this URGENT HELP !!

I executed deploy.cmd command it seems that everything is fine there and it connected the pi (i guess ) i stopped with 3 second after that i checked my pi log files and no raspberrypi log file is there apart from booting.log in logs folder (ctdgtwy)
Please help urgently!!

Hello. Did you chaged /etc/rc.local as said in documentation?

Yes

@dinarisio yes I did

@dinarisio please help me dude

@shubhkmr could you list all files/folders you have in home/pi/staging directory?

a. Everything in the "GatewayService\WindowsService\bin\Release" folder
b. DeviceAdapters\SerialPort\bin\Release\Microsoft.ConnectTheDots.SerialPortAdapter.dll
c. DeviceAdapters\Socket\bin\Release\Microsoft.ConnectTheDots.SocketAdapter.dll
d. Scripts\RaspberryPi\Modified\certificate_update.sh
e. Scripts\RaspberryPi\Modified\autorun_install.sh
f.  Scripts\RaspberryPi\Modified\kill_all.sh


Add some more NonEmptyArray functions

Implement index, findFirst, findIndex, insertAt, updateAt, filter for NonEmptyArray
I may issue a PR.
@gcanti Ok with all of those?

@sledorze sure, thanks

Need that for:
https://github.com/gcanti/monocle-ts/issues/66
@gcanti , may it be possible to issue a release once accepted?


Black bar on right side of window for transparent themes

This plugins is exactly what I need for my workflow, and I'm excited to work with it. However, there appears to be a black horizontal bar to the right of the popup menu

Which is more noticeable when using a smaller width.
For context, I use a transparent-background theme based on catppuccin, so for many plugins I just override the highlights. I've taken a look into the highlights used with this plugin (at least those that started with Navbuddy, but couldn't find any that caused this to happen.
Happy to try various tweaks/updates to figure this out.

Ui drawing is completely handled by the nui library by @MunifTanjim.
Not sure what might be the issue, let me try experimenting with transparance settings to see if I can reproduce this.
Meanwhile, can you try setting border = "none" and see if this occurs in that case too?

Setting border to none doesn't get rid of the black bar

I'll also take a look into nui configuration, maybe there's something there

Just had a thought: could that be the scroll-bar?

Not sure nui has scroll bars, and if so I certainly didn't enable them. But if you are using some fancy GUI skin for neovim, that might actually be scroll bar added on.

I think I've narrowed it down to something with Catppuccin's implementation of transparent backgrounds. I know they have integration with navic, but not with this given how new it is.
Another thought I had was that it's a shadow, but I tried overriding the FloatShadow and FloatShadowThrough highlights to no changes

https://github.com/SmiteshP/nvim-navbuddy/blob/61afbf6b2e7a86ff0c88a8535b51ea192146dfc7/lua/nvim-navbuddy/display.lua#L268-L272
https://github.com/SmiteshP/nvim-navbuddy/blob/61afbf6b2e7a86ff0c88a8535b51ea192146dfc7/lua/nvim-navbuddy/init.lua#L8-L23
Percentage doesn't always translate to fill-up the whole area. Instead of using size in the last box, try doing grow = 1, that'll fill up the remaining space. i.e.
			nui_layout.Box(right_popup, { grow = 1 }),

I think I've narrowed it down to something with Catppuccin's implementation of transparent backgrounds.

Yeah, that's also the issue here. Normally that black empty space should've been transparent. But this particular theme is rendering the empty space as black.

@LittleTealeaf Added fix as per @MunifTanjim's suggestion, can you check if it is fixed


[1.13] Multipiston head not rendering

Known issue: The current 1.13 Forge version has not yet updated the submodel system for the Forge blockstate JSON format.
Multipiston rendering relies on submodels to render the piston heads, and until this is fixed the piston head is invisible.
This bug will fix itself once Forge 1.13 ports the submodel system.

Fixed in commit 9d418ab4be08243023f3d069aff36208bb900dff


[WIP] Notification-Backend

@rogeredthat You may close PR #187 as I will be pushing new code to my forked repository.

Get rid of the last commit here. https://github.com/crux-bphc/fest-manager/pull/201/commits/9d1f5af018052ac82d5e785bce68bd6dca6ee0f1
It has already been merged with some other PR.

Ok.
On Sun, 10 Sep 2017 at 8:08 PM, Abhilash Verma<EMAIL_ADDRESS>wrote:

Get rid of the last commit here. 9d1f5af
https://github.com/crux-bphc/fest-manager/commit/9d1f5af018052ac82d5e785bce68bd6dca6ee0f1
It has already been merged with some other PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crux-bphc/fest-manager/pull/201#issuecomment-328346835,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbfCtrTv7Ob4B0QKtsbvxnWj1WthZhnXks5sg_RjgaJpZM4PSVEg
.

@rogeredthat. Please review the latest commit 50344c6

Also, add an edit button routing to /dashboard/account and a cart button routing to /dashboard/cart  before merging.

Not sure what's the status on this one, but heads up @rogeredthat and @sohailrajdev97 that I've swapped enscroll with simplebar. Since this PR didn't seem to be doing that despite the commit message saying it, I didn't take that specific commit. Please make necessary changes while rebasing.

Commits from this PR were directly merged. Closing now.


Make lazy loading for modules at WebpackOptionApply.js

What kind of change does this PR introduce?
Make lazy loading for modules at WebpackOptionApply.js, so that no extra memory is allocated.

Did you add tests for your changes?
For these changes they are not needed.

Does this PR introduce a breaking change?
No.

What needs to be documented once your changes are merged?
Nothing.

Thanks


HSEARCH-3159 Upgrade to Hibernate ORM 5.3.0.Final

https://hibernate.atlassian.net//browse/HSEARCH-3159

looks great!
Just the readme needs a fix?

Right, done. At some point, we should probably replace this information with a link to the right page at hibernate.org, now that we have a compatibility matrix and all.


AD&D 1E: Added optional colors to the header of the General Template

I asked Vince if he minded the addition and he said it was great. =D

Actually I said...
" Absolutely not. (that I don't mind)  lol
That's cool.  I would like to see how you do that.  ;-) "
lol.  Nice addition to the sheet.  Thanks

thanks Phil

@Vince I was summarizing!  (It was confusing enough when you said it to me, I didn't want to bother Phil with it! =D )

lol
I was thinking of playing a "Confusionist" it my next game.  ;-P


HDDS-3438. Enable TestContainerServer test cases.

What changes were proposed in this pull request?
This PR is to allow running test cases of TestContainerServer class, as they were marked slow and some unhealthy.  In earlier written test case, some of the inputs to testClientServerWithContainerDispatcher like volumeSet and config like hdds.datanode.dir were not set correctly due to which the test was failing and it was failing after running and retrying for more than 2 mins. This PR has improved that corrected the test case.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3438
How was this patch tested?
Since this PR is to allow and enable running of test cases of TestContainerServer class, the test class has been tested using running multiple times and it passed successfully without any significant delay.

@adoroszlai pls review.

@devmadhuu can you please post link to repeated CI runs of TestContainerServer for reference?

@devmadhuu can you please post link to repeated CI runs of TestContainerServer for reference?

Thanks @adoroszlai for review. Here is the link for repeated CI runs of TestContainerServer.


Restore bundle analysis functionality

Working around danielholmes/relay-compiler-webpack-plugin#5 by avoiding stdout for JSON generation, using stats-webpack-plugin.
Related: #365.

#yolomerging because this doesn’t change much


Make sure transactions can't be started or finished when turn is in progress

Description

Transactions can be potentially performed inside observer callbacks, lift functions etc. Need to ensure it is handled correctly.

Done


Temperature sensor

Hi, is possible add a temperature sensor funcionality for sending to Home Assistant and control room temperature? For example DHT22 or analog to AI pin.
Thank you.

Closing as this repo is not being maintained.  Check here for the updated project!


DateEntry State Change

Desktop (please complete the following information):
ttkbootstrap Version 1.10.0
OS Linux
Describe the bug
The DateEntry widget appears to be able to be disabled but I cannot find a way to reenable it. I can set it to read-only but not fully enabled. The code below shows some of the things I have tried under the openSelect function.
To Reproduce
A minimal working example is as follows (maybe a bit more than a true mwe):
import tkinter as tk
import ttkbootstrap as tb

#----------------------------------------------------------

class MWE(tb.Window):

    #------------------------------------------------------
    def __init__(self):
        super().__init__()

        self.open_check = tb.IntVar(value=1)

        open_label = tb.Label(self, text='Open',)
        open_label.grid(row=0, column=0, padx=5, pady=5)

        self.open_entry = tb.widgets.DateEntry(self,
            firstweekday=6)
        self.open_entry.configure(state='disabled')
        self.open_entry.grid(row=0, column=1, padx=5, pady=5)

        self.open_unknown = tb.Checkbutton(self, text='Unknown',
            bootstyle='rounded toggle', variable=self.open_check,
            onvalue=1, offvalue=0, command=self.openSelect)
        self.open_unknown.grid(row=0, column=2, padx=5, pady=5)

    #------------------------------------------------------
    def openSelect(self):
        if self.open_check.get():
            self.open_entry.configure(state='disabled')
        else:
            self.open_entry.enable()
            #self.open_entry.configure(state='!disabled')
            #self.open_entry.configure(state='readonly')
            #self.open_entry.configure(state='normal')

#----------------------------------------------------------

if __name__ == '__main__':
    mwe = MWE()
    mwe.mainloop()

Expected behavior
I would expect that when the checkbutton is clicked to the off position, the DateEntry would become fully enabled via the openSelect function.
Screenshots
No response
Additional context
No response

I figured out a workaround to enable it by using
self.open_entry.entry.configure(state='normal')
self.open_entry.button.configure(state='normal')

under the else statement of openSelect. If this is the intended way to do this, please remove this bug. I will note that it would be nice to be able to just use something like self.open_entryconfigure(state='normal') if possible.


PW-forecast.php script set available using the Pirateweather.net API

This is not really an 'issue', just a heads-up and THANKS for providing a DarkSky-formatted API for weather enthusiasts worldwide.
I've done some mods to my existing DS-forecast.php script set (using the DarkSky API while it still is being offered to legacy users) to now use the pirateweather.net API instead and offer a multilingual forecast with icons for personal weather websites.
https://github.com/ktrue/PW-forecast
and
https://saratoga-weather.org/scripts-PWforecast.php
Best regards,
Ken

I have no background in PHP whatsoever, but I know it's widely used, so this is amazing! I'm tagging it as documentation to remind me to put a link to it in the docs, since I'm creating a "developer tools" section for these sorts of things.
Thanks again for creating this, and let me know if anything else comes up.


Remove BOM from .cs and csproj files

I noticed that my PR was getting polluted when my IDE automatically removed the BOM from the front of files just because I opened them, so I thought I'd solve that by removing them all in one go.

Codecov Report

Merging #439 (3b2adcb) into main (fe1e2e0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #439   +/-   ##
=======================================
  Coverage   62.65%   62.65%
=======================================
  Files         241      241
  Lines        7651     7651
  Branches      882      882
=======================================
  Hits         4794     4794
  Misses       2476     2476
  Partials      381      381

Files
Coverage Δ

...ers/ComponentDetection/CondaComponentExtensions.cs
94.11% <ø> (ø)

...mponentDetection/DockerImageComponentExtensions.cs
93.33% <ø> (ø)

...entDetection/DockerReferenceComponentExtensions.cs
0.00% <ø> (ø)

...pters/ComponentDetection/GitComponentExtensions.cs
93.75% <ø> (ø)

...apters/ComponentDetection/GoComponentExtensions.cs
0.00% <ø> (ø)

...ers/ComponentDetection/LinuxComponentExtensions.cs
0.00% <ø> (ø)

...apters/ComponentDetection/Logging/LoggingHelper.cs
0.00% <ø> (ø)

...ers/ComponentDetection/MavenComponentExtensions.cs
88.88% <ø> (ø)

...ers/ComponentDetection/OtherComponentExtensions.cs
0.00% <ø> (ø)

...ters/ComponentDetection/SpdxComponentExtensions.cs
0.00% <ø> (ø)

... and 171 more


chore: remove unused kokoro-readme generator

I believe this is all unused now, so I'm removing it. None of our repos have ci/kokoro/readme/ builds anymore. If it's still used/needed, I can drop this PR. Just cleaning stuff up.

If you're not planning to regenerate https://github.com/googleapis/google-cloud-cpp/pull/3918, I can drop this PR so it doesn't conflict w/ what you're doing. Let me know.


Fixed installation of ansible-lint

ansible-lint released version 5.0.0 what doesn't depend hard on ansible
any more. This PR installs ansible-lint[core] that does depend on
ansible.
See
https://github.com/ansible-community/ansible-lint/blob/650bc07fc4cbd05f9a98ea614589f907e965c61d/setup.cfg#L96

I don't have any idea why the test-run doen't use the code of my change. Maybe @bzhaoopenstack could help out?


RuntimeWarning: coroutine 'Command.call' was never awaited   else: func(*args, **kwargs)

Hello, I am very excited for the Button Menu and want to thank you a lot for implementing it, I am trying to integrate it right now but when using a caller button I always get this error:

C:\Users\user\AppData\Local\Programs\Python\Python39\lib\site-packages\reactionmenu\buttons_menu.py:570: RuntimeWarning: coroutine 'Command.call' was never awaited
else: func(*args, **kwargs)
RuntimeWarning: Enable tracemalloc to get the object allocation traceback

The Button was made in this way:

g_call_followup = ComponentsButton.Followup()
g_call_followup.set_caller_details(bridgebuy, ctx, "g", menu)
g_caller_button = ComponentsButton(style=ComponentsButton.style.grey, emoji='🏭',label='', custom_id=ComponentsButton.ID_CALLER, followup=g_call_followup)

And the Menu was made like this:

menu = ButtonsMenu(ctx, menu_type=ButtonsMenu.TypeEmbed, timeout=60, show_page_director=False, all_can_click=False,remove_buttons_on_timeout=True)

The Menu starts in the right way and shows the buttons fine, I just cannot use any of my caller buttons, my end session button works fine.
What do I need to change to get it to work >.>

Oh ye all my caller buttons call async functions, which is the main problem I guess... the normal emoji reaction menu had no issues with this :/

Is the function you're setting a discord.py command?

yes

Ahh, I see the issue. I forgot to add a line in the code. Update to the latest Github version and let me know if it's working now

I did the "pip install git+https://github.com/Defxult/reactionmenu.git --upgrade" am kinda new to this, hope that was right.
The issue sadly still occurs for me

Make sure you uninstall the current version first. And do you have "Git" installed?

Ok, uninstalled it first this time and it works fine now! Ty <3

Awesome! Thanks for reporting the issue! 😀 I appreciate it.


Add an entrypoint init process

You should have an ENTRYPOINT init process set for your docker image.  I recommend Yelp dumb-init.

Thanks for comment. I will consider adding it when I do the next release.


Add the link to the POH website on footer' left side.

Link to POH website for more information on footer:

Hi Plinio, how can I edit my profile pic?? Thanks

Done


Confusing error when calling function with wrong argument types

If I run this with dev_compiler:
try {
  dynamic fn = (int n) => null;
  fn("not int");
} catch (error) {
  print(error);
}

It correctly throws an error. Win! However, the error it throws is NoSuchMethod:
NoSuchMethodError : method not found: 'Symbol("call")'
Receiver: Instance of '(int) -> dynamic'
Arguments: ["not int"]

which is kind of confusing. The function does have a call method. It's just the argument type was wrong. I think a CastError would make more sense?
Heh, and now that I look in the source, I see:
  ...

  if ($_checkApply($ftype, $args)) {
    if ($typeArgs != null) {
      return $f.apply($obj, $typeArgs).apply($obj, $args);
    }
    return $f.apply($obj, $args);
  }

  // TODO(leafp): throw a type error (rather than NSM)
  // if the arity matches but the types are wrong.
  // TODO(jmesserly): nSM should include type args?
  return callNSM();
}

The fix should just be to add an else to that _checkApply() if and throw a better error.
cc @leafpetersen @vsmenon

The last commit should fix @munificent 's example, but it still doesn't always throw the 'right' exception when both a TypeError and an NoSuchMethodError apply:
void main() {
  Function f = (String str) => print(str);
  f(42, 38);
}

In checked mode, dart2js and the VM (and the spec) throw a NoSuchMethodError while DDC throws a TypeError.  We'll need to refactor _checkApply to check for these in the right order.

Fixed! this now prints: Type 'String' is not a subtype of type 'int'


discussion: async-graphql vs jupiter and diesel vs tokio-postgres vs sqlx

@clifinger did you know about async-graphql
At this moment this solution has more features and I feel better then use of jupiter.
I personally followed the implementation of twentyfive-stars and liked the interface more than Jupiter. Jupiter supports async, but not released yet and Jupiter lack of good documentation when async-graphql has excellent book
During the investigation, I came across sqlx and found this native rust driver feature to verify SQL during the build is awesome and helps to find bugs before the actual run.
There are actually 3 choices:

diesel
tokio-postgres
sqlx

I have tried all 3 and finally stopped on sqlx, diesel is crazy complex and not native driver, tokio-postgres is too basic and sqlx is a very good balanced generic solution and I feel performance would be better as it does not have a dependency on libpq (C driver)
I think, that for Bollerplate we need to pick the best solutions on the market and diesel and Jupiter are not the best

Not to be rude, but your info is wrong and presented as fact. Please refrain from making such sweeping statements.
Juniper has a book that covers just as much if not more:
https://graphql-rust.github.io/juniper/master/
async-graphql appears to have more features but many of them do not belong in a library and push async-graphql into framework territory. Juniper is firmly in the library camp. It's mainly preference and what you are looking for. The GraphQL featurset is the same for both (minus interfaces on Juniper master, which we are revamping). If there are missing GraphQL features, please file tasks on Juniper!


Data is not accessible

Possible bug

sorry, I never work with exceptions disabled, never catched that case.
although, not using the exceptions is dangeours as you can lose all the scene data if there is a crash during the parsing


dma_sam_xdmac - implemented dma device get_status()

added implementation for atmel sam xdmac - generic get_status() api

we can now read the current status of the dma through standard DMA API

Fixes #62003

pipeline stopped with:
The self-hosted runner: zephyr-runner-linux-x64-4xlarge-deployment-g2tcl-q2sdx lost communication with the server
is there something I can do?

tests seem to be complete.

modified commit message

author
removed second line

as requested per code-guideline-check

@fabiobaltieri @pdgendt thank your for your approval.
I am not authorized to merge - what is the typical procedure?

I am not authorized to merge - what is the typical procedure?

Someone from the release team will merge it for you, typically though we'd wait for assignee approval as well (that's @nandojve), keep in mind that many project members are on vacation these days so it may take another few days.

we are using the v3.0.0 because we found main not stable regarding network stack and network drivers for the MICROCHIP SAM cpu.
I will create a second pull request to fix this also in main, however we need this in v3.0.0, because that is the latest stable version, that we can use.

No need to open a new PR, you can edit this one and change the base. We don't accept new PRs for unsupported releases though so if you want it on top of v3.0.0 you'll have to maintain a branch yourself.

So I guess it still would be ok to change the pull request to branch-v2.7.5 (LTS) - as this is officially the long term support branch?

I guess its still ok to change the pull request to branch-v2.7.5 (LTS) - as this is officially the long term support branch?

Yes but for that change this PR to merge against main and then we can use the backport tag so that the backport gets filed automatically as this gets merged on main.

ok understand. thank you for clarification. I will change to main then.

We also noted that buffers needed to be increased. However regarding 24/7 stability we are not satisfied yet, maybe have to dig deeper. Thanks for your information.


feat: support aligning marble strings using leading spaces

This pull request contains a possible implementation for #81
The TestScheduler of RxJs, which is used under the hood, also implements leading spaces in marble strings. However, this only seems to work when the Observables are created within the callback to TestScheduler.run(helpers => { ... }). Outside of the run callback, the implementation treats spaces as a frame advancement (see this line). It didn't seem easy nor desirable to hack into this to trick the TestScheduler into 'run mode'.
One use case I can see for their conditional space treatment is to support their time progression syntax ('---- 4ms a 9ms b 9ms (c|)'), in which spaces need to advance a frame to maintain vertical alignment with other marble strings. However, this use case is still supported with this pull request because I only strip leading spaces.

Available in 2.2.0


Missing / soon to be missing implementations

First of: Thank you so so much for providing this library, it's amazing :)
I just wanted to let you know that there are a view strings missing currently. The title says "soon to be" because I am unsure weather they already are missing in the stable branch. I am using the beta branch.
Strings missing in MaterialLocalizationsEo:

MaterialLocalizations.lookUpButtonLabel
MaterialLocalizations.menuDismissLabel
MaterialLocalizations.searchWebButtonLabel
MaterialLocalizations.shareButtonLabel

Strings missing in CupertinoLocalizationsEo:

CupertinoLocalizations.lookUpButtonLabel
CupertinoLocalizations.menuDismissLabel
CupertinoLocalizations.searchWebButtonLabel
CupertinoLocalizations.shareButtonLabel

Probably not, I've just refreshed my Flutter to 3.13.8 and no warning yet. But let's be on our toes...

Yeah , my current (updated today) version is 3.16.0-0.4.pre
I think I'll just create a fork of it for now so it works with my project and then I guess if the translations are correct you can merge it, once the changes hit flutter stable?

Yes, sure.

I'm closing this as it already got released :)

Yes, I happened to be in an app that relies on it and it warned me, so I became aware of it.


ReduceLROnPlateau not logging LR to tensorboard anymore

I use the ReduceLROnPlateau callback and LR used to be logged to tensorboard. However since 2.3.0 it isn't showing up anymore. I tried to find a change that may explain the issue, but can't find anything. Was this some deliberate change?

@lminer
Please, fill issue template.
Please, share colab link or simple standalone code to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!


Should RACK_ENV be respected in production?

Hi @solnic. I've noticed that running ./bin/run db migrate on heroku where RACK_ENV is production still attempts to dump the schema, resulting in a prompt for the database password.
Is it right that RACK_ENV=production should mean that snowflakes assumes it's in production, or is there another intention? @timriley has noticed recently in test that he needs to pass -e test rather than RACK_ENV=test.
Some discussion here: https://icelab.slack.com/archives/C2BT34RK7/p1518490460000199
Thanks!

@andrewcroome this is not good, sorry about that. Snowflakes should respect whatever the application sets as its env, I need to make sure that this is what happens. I'll deal with it.

This is fixed in master


Add money in hand icon

Not sure if this really works, trying to create a icon based on helping hand for savings

Icon name: helping-hand-money
Use case: offers, banking

Similar icons

Things are a bit off grid:

I've recreated this in Figma as pixel perfect as I could manage:

Sorry my work was sloppy @karsa-mistmere  will make sure to watch that more next time I submit anything. I am still not convinced though my idea works in concept. As I am not sure it looks like money, however I am not sure how that could be rectified within the guidelines.

The current helping-hand icon does not feel awesome, not a fan of creating things based on that one even tho the use case is valid IMO.

The current helping-hand icon does not feel awesome, not a fan of creating things based on that one even tho the use case is valid IMO.

Maybe the solution is rethinking helping hand totally and creating a new one with different use cases in mind.

Or we could open another issue managing the helping-hand icon and come back when that's completed.

trying to create a icon based on helping hand for savings

The current helping-hand icon does not feel awesome, not a fan of creating things based on that one even tho the use case is valid IMO.

Maybe the solution is rethinking helping hand totally and creating a new one with different use cases in mind.

@KayleyHill @karsa-mistmere @jguddas @xnousnow See #1328.

@KayleyHill @karsa-mistmere @jguddas @xnousnow See #1328.

Really like the hand-platter and the new coin one.


CircleCI:Add a format_python3 test

Signed-off-by: cclauss<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

Closing in favor of #7388


Unable to build flexget

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)

python -m nuitka --version

<IP_ADDRESS>
Python: 3.9.6 (default, Jun 30 2021, 10:22:16)
Executable: /home/finkel/work_self/Flexget/bin/python
OS: Linux
Arch: x86_64

How did you install Nuitka and Python

I've install nuitka in pip and python from my distribution, working inside venv

The specific PyPI names and versions

python -m pip freeze

aniso8601==9.0.1
APScheduler==3.7.0
attrs==20.3.0
babelfish==0.5.5
beautifulsoup4==4.9.3
Brotli==1.0.9
certifi==2020.12.5
chardet==4.0.0
cheroot==8.5.2
CherryPy==18.6.0
click==7.1.2
colorama==0.4.4
colorclass==2.2.0
deluge-client==1.9.0
feedparser==6.0.8
Flask==1.1.2
Flask-Compress==1.10.1
Flask-Cors==3.0.10
Flask-Login==0.5.0
Flask-RESTful==0.3.9
flask-restx==0.4.0
-e git+git@github.com:Flexget/Flexget.git@ed41e678ac5e5e700cde7c2345b7004d2c1e03ef#egg=FlexGet
greenlet==1.0.0
guessit==3.2.0
html5lib==1.1
idna==2.10
importlib-metadata==4.6.0
itsdangerous==1.1.0
jaraco.classes==3.2.1
jaraco.collections==3.3.0
jaraco.functools==3.3.0
jaraco.text==3.5.0
Jinja2==3.0.1
jsonschema==3.2.0
loguru==0.5.3
MarkupSafe==2.0.0
more-itertools==8.8.0
Nuitka==<IP_ADDRESS>
plumbum==1.7.0
portend==2.7.1
progressbar==2.5
psutil==5.8.0
pynzb==0.1.0
pyparsing==2.4.7
pyrsistent==0.17.3
PyRSS2Gen==1.1
python-dateutil==2.8.1
pytz==2021.1
PyYAML==5.4.1
rebulk==3.0.1
requests==2.25.1
rpyc==5.0.1
SCons==4.1.0.post1
sgmllib3k==1.0.0
six==1.15.0
soupsieve==2.2.1
SQLAlchemy==1.4.20
tempora==4.0.2
terminaltables==3.1.0
typing-extensions==<IP_ADDRESS>
tzlocal==2.1
urllib3==1.26.4
webencodings==0.5.1
Werkzeug==1.0.1
zc.lockfile==2.0
zipp==3.4.1
zxcvbn-python==4.4.24

Many times when you get an error from Nuitka, your setup may be special

works with following output:
Nuitka-Options:INFO: Used command line options: --run some.py
Nuitka:INFO: Starting Python compilation with Nuitka '0.6.16' on Python '3.9' commercial None.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C level backend compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: gcc (gcc).
Nuitka-Scons:INFO: Compiled 12 C files using ccache.
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'disabled': 1
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'cache miss': 11
Nuitka:INFO: Keeping build directory 'some.build'.
Nuitka:INFO: Successfully created 'some.bin'.
Nuitka:INFO: Launching 'some.bin'.
hello world

Also supply a Short, Self Contained, Correct, Example

I'mtrying to build flexget with nuitka3-run --standalone --onefile --linux-onefile-icon=./256.png --plugin-enable=numpy --plugin-enable=pylint-warnings bin/flexget execute and getting following output
Nuitka-Options:INFO: Used command line options: --run --standalone --onefile --linux-onefile-icon=./256.png --plugin-enable=numpy --plugin-enable=pylint-warnings bin/flexget
Nuitka:INFO: Starting Python compilation with Nuitka '0.6.16' on Python '3.9' commercial None.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'tkinter' added '_tkinter'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'socket' added '_socket'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'ctypes' added '_ctypes'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'numpy' added 'numpy.compat,numpy.lib,numpy.linalg,numpy.fft,numpy.polynomial,numpy.random,numpy.random._bounded_integers,numpy.random._common,numpy.random._generator,numpy.random._mt19937,numpy.random._pcg64,numpy.random._philox,numpy.random._sfc64,numpy.random.bit_generator,numpy.random.mtrand,numpy.ctypeslib,numpy.ma,numpy.matrixlib'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'pycparser.c_parser' added 'pycparser.yacctab,pycparser.lextab'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'scipy._lib' added 'scipy._lib.messagestream'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'numpy.random' added 'numpy.random._bounded_integers,numpy.random._common,numpy.random._generator,numpy.random._mt19937,numpy.random._pcg64,numpy.random._philox,numpy.random._sfc64,numpy.random.bit_generator,numpy.random.mtrand'.
Nuitka-Plugins:INFO: implicit-imports: Implicit dependencies of module 'numpy.core' added 'numpy.core._dtype_ctypes,numpy.core._multiarray_tests'.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C level backend compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: gcc (gcc).
Nuitka-Scons:INFO: Compiled 160 C files using ccache.
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'disabled': 1
Nuitka-Scons:INFO: Cached C files (using ccache) with result 'cache hit': 159
Nuitka-Plugins:INFO: data-files: Included data file 'lib2to3/Grammar3.9.6.final.0.pickle' due to package data for 'lib2to3.pgen2'.
Nuitka-Plugins:INFO: data-files: Included data file 'lib2to3/PatternGrammar3.9.6.final.0.pickle' due to package data for 'lib2to3.pgen2'.
Nuitka-Postprocessing:INFO: Creating single file from dist folder, this may take a while.
Nuitka-Postprocessing:INFO: Completed onefile creation.
Nuitka:INFO: Keeping dist folder 'flexget.dist' for inspection, no need to use it.
Nuitka:INFO: Keeping build directory 'flexget.build'.
Nuitka:INFO: Successfully created 'flexget.bin'.
Nuitka:INFO: Launching 'flexget.bin'.
Traceback (most recent call last):
File "/tmp/finkel/.mount_flexgeAVwdY2/flexget", line 33, in <module>
File "/tmp/finkel/.mount_flexgeAVwdY2/flexget", line 25, in importlib_load_entry_point
File "/tmp/finkel/.mount_flexgeAVwdY2/importlib/metadata.py", line 77, in load
File "importlib.py", line 127, in import_module
File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
File "<frozen importlib._bootstrap_external>", line 850, in exec_module
File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
File "/home/finkel/work_self/Flexget/flexget/__init__.py", line 10, in <module>
from flexget import log  # noqa
File "/home/finkel/work_self/Flexget/flexget/log.py", line 14, in <module>
import loguru
ModuleNotFoundError: No module named 'loguru'

In your issue the options used
Ideally use the project feature, so options and example code go alone
Nuitka Options in the code

Note if this is a regression
If it used to work in an earlier version of Nuitka, please note what you know
about that.

Can you provide a link to the actual code too?

Oh, of course, I'm sorry
https://github.com/Flexget/Flexget

I did the cloning but I feel this is not minimal enough, since naturally there is no bin/flexget. The importing of loguru works. I am suspected, this is about the entry points created, which Nuitka will not see through. There is plenty of work done in this field in the pkg_resources plugin, but likely not enough yet. I have at least one other report about this kind of error.

Yeah, I'm sorry I can't provide you with a real minimal reproducer. Of course, flexget is big and complex. But I don't know how to minimize it — I just don't know enough about the whole field.

I am trying it out right now, I think importlib_load_entry_point is something that needs handling by nuitka. A naive build just includes nothing but the runner which in term won't find anything, not even the flexget top level module which it apparently did for you. What people can also do is python setup,.py bdist_nuitka which creates different runners. But I want to get the compilation from installation to work of course too.

So this is the trouble maker:
# EASY-INSTALL-ENTRY-SCRIPT: 'FlexGet==3.1.150.dev0','console_scripts','flexget'
import re
import sys

# for compatibility with easy_install; see #2198
__requires__ = 'FlexGet==3.1.150.dev0'

try:
    from importlib.metadata import distribution
except ImportError:
    try:
        from importlib_metadata import distribution
    except ImportError:
        from pkg_resources import load_entry_point

def importlib_load_entry_point(spec, group, name):
    dist_name, _, _ = spec.partition('==')
    matches = (
        entry_point
        for entry_point in distribution(dist_name).entry_points
        if entry_point.group == group and entry_point.name == name
    )
    return next(matches).load()

globals().setdefault('load_entry_point', importlib_load_entry_point)

if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw?|\.exe)?$', '', sys.argv[0])
    sys.exit(load_entry_point('FlexGet==3.1.150.dev0', 'console_scripts', 'flexget')())

Interestingly bdist_nuitka found an obscure bug that affected lexget.components.series.next_series_episodes, but it may also be because of recent optimization added, I fixed it for factory branch. Then bdist_nuitka works.

Unless I am mistaken, all it takes should be --include-package=flexget which I am currently trying out. Should that work, I will derive this from parsing # EASY-INSTALL-ENTRY-SCRIPT: in __main__ modules, that won't be hard and could be added to the pkg-resources plugin. Removing the useless distribution, etc. stuff might still be needed though, so maybe it's more a task for anti-bloat plugin.

The include is not enough, this is sort of a duplicate, and reproduces your issue. I now understand that I confused binary name and package name, where filename is in the traceback output
Traceback (most recent call last):
  File "/home/hayen/test/flexget-issue/flexget.dist/flexget", line 33, in <module>
  File "/home/hayen/test/flexget-issue/flexget.dist/flexget", line 22, in importlib_load_entry_point
  File "/home/hayen/test/flexget-issue/flexget.dist/importlib_metadata/__init__.py", line 967, in distribution
  File "/home/hayen/test/flexget-issue/flexget.dist/importlib_metadata/__init__.py", line 561, in from_name
importlib_metadata.PackageNotFoundError: No package metadata was found for FlexGet

So indeed, we need to replace the runner with a simpler version automatically.

Wow, it's a comprehensive analysis! Should we try to loop flexget authors/maintainers in?

I don't think it's anything special for flexget, I guess once it comes to data files, it will be needed for a command line to provide them, but bdist_nuitka ought to create a working wheel no problem with factory. It's just very unstable right now due to massive improvements in optimization.

Thank you for your deep dive into the problem!

So this is now working for me on factory branch, with working I mean "python setup.py install" into a virtualenv with Python3.7 (my self compiled Pythons don't do virtualenv without too much manual tweaking unfortunately). I need to checkout what this kind of thing generates for Python2 and what it puts into a bdist_wheel which may be yet another thing.

https://nuitka.net/doc/factory.html

Actually installed wheel do not have these strange runners, but the one that Nuitka now compile time generates with only a very minimal difference. For fun, somebody ought to create an entry point with the function named "sys", wheels wouldn't work with that, but it's unlikely to happen in real life.

This is on the develop branch now and will be part of the next stable release.

This is part of the 0.6.18 release.


[Feature-#5128]Support turning off sudo permissions

Purpose of the pull request
This closes #5128

Codecov Report

Merging #5129 (93135d4) into dev (f109a75) will decrease coverage by 0.00%.
The diff coverage is 22.22%.

@@             Coverage Diff              @@
##                dev    #5129      +/-   ##
============================================
- Coverage     47.29%   47.28%   -0.01%
  Complexity     3709     3709
============================================
  Files           570      570
  Lines         23943    23947       +4
  Branches       2759     2760       +1
============================================
  Hits          11324    11324
- Misses        11586    11590       +4
  Partials       1033     1033

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/dolphinscheduler/common/Constants.java
81.81% <ø> (ø)
1.00 <0.00> (ø)

.../server/worker/processor/TaskExecuteProcessor.java
78.75% <0.00%> (ø)
9.00 <0.00> (ø)

...er/server/worker/task/AbstractCommandExecutor.java
16.58% <0.00%> (-0.08%)
10.00 <0.00> (ø)

.../apache/dolphinscheduler/common/utils/OSUtils.java
43.95% <33.33%> (+0.06%)
25.00 <1.00> (ø)

...che/dolphinscheduler/common/utils/CommonUtils.java
65.30% <100.00%> (+0.72%)
13.00 <1.00> (+1.00)

...inscheduler/service/zk/CuratorZookeeperClient.java
70.73% <0.00%> (-4.88%)
9.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

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

Awesome feature!
Looking forward to the next version release!


Allow multiple compose files

Hi there,
I would like to use multiple compose file, so that I can override some parameters in my docker-compose.test.yml file wrt. the main docker-compose.yml file.
I seen now that the options for the docker compose environment, and in particolar this line do not allow for passing multiple compose files.
The underlying docker-compose library, however, accepts multiple compose files as string array string[].
Would it be possible to unlock this feature? Or is there any peculiar reason why this was somehow blocked?
Thanks

Maybe it is just a TypeScript definition and documentation issue, after all 😅
I just tried passing (in a .js file so that I don't get TS errors)
    const composeFilePath = path.resolve(__dirname, '../')
    const composeFile = ['docker-compose.yml', 'docker-compose.test.yml']

    environment = await new DockerComposeEnvironment(
      composeFilePath,
      composeFile
    )
      .withStartupTimeout(120000)
      .withBuild()
      .up()

And it worked pretty fine!
Thanks God nor your library nor docker-compose makes any runtime validation of the arguments! 😄

Thanks God nor your library nor docker-compose makes any runtime validation of the arguments! 😄

Haha that's great! I'm happy to change the API to allow for multiple files, will there be cases where we use files in different paths? If so I think I might inline composeFilePath and composeFIle, so composeFile becomes the full path including file

I think it could be, even though it's not that relevant for me, however the docker-compose library documentation says the files must be in the same cwd...

Cool, the cwd will be the same then. Check out the referenced PR, will merge unless you see any issue 🙂

Released in https://github.com/testcontainers/testcontainers-node/releases/tag/v5.1.0, thanks again!


GPU support

Thanks for this greate package! Is there a plan for adding GPU support for this package? I means, opencv has already added gpu support to many image operations.

Thanks for this greate package! Is there a plan for adding GPU support for this package? I means, opencv has already added gpu support to many image operations.

That's a good suggestion! however,  it's a pitty that, I am working as an intern in senseitme and do not have much time to build a GPU version. There is a GPU based lib https://github.com/NVIDIA/DALI which you may be interested in.

Thanks a lot whatever. This package is quite friendly to pytorchers, since we just need make little adjustment on our existed projects. DALI is a famous GPU based image process lib. However, it also has much more steep learning curve. Besides that, it is said that there is a bug on memory usage of DALI (the usage of memory will increase as the iteration grow.) which will limit the maximum batch size. This claim also escapes me from it. That's why I appreciate this package so much.


NotADirectoryError while loading the CNN/Dailymail dataset

Downloading and preparing dataset cnn_dailymail/3.0.0 (download: 558.32 MiB, generated: 1.28 GiB, post-processed: Unknown size, total: 1.82 GiB) to /root/.cache/huggingface/datasets/cnn_dailymail/3.0.0/3.0.0/0128610a44e10f25b4af6689441c72af86205282d26399642f7db38fa7535602...

NotADirectoryError                        Traceback (most recent call last)
 in ()
22
23
---> 24 train = load_dataset('cnn_dailymail', '3.0.0', split='train')
25 validation = load_dataset('cnn_dailymail', '3.0.0', split='validation')
26 test = load_dataset('cnn_dailymail', '3.0.0', split='test')
5 frames
/root/.cache/huggingface/modules/datasets_modules/datasets/cnn_dailymail/0128610a44e10f25b4af6689441c72af86205282d26399642f7db38fa7535602/cnn_dailymail.py in _find_files(dl_paths, publisher, url_dict)
132     else:
133         logging.fatal("Unsupported publisher: %s", publisher)
--> 134     files = sorted(os.listdir(top_dir))
135
136     ret_files = []
NotADirectoryError: [Errno 20] Not a directory: '/root/.cache/huggingface/datasets/downloads/1bc05d24fa6dda2468e83a73cf6dc207226e01e3c48a507ea716dc0421da583b/cnn/stories'

Looks like the google drive download failed.
I'm getting a Google Drive - Quota exceeded error while looking at the downloaded file.
We should consider finding a better host than google drive for this dataset imo
related : #873 #864

It is working now, thank you.
Should I leave this issue open to address the Quota-exceeded error?

Yes please. It's been happening several times, we definitely need to address it

Any updates on this one? I'm facing a similar issue trying to add CelebA.

I've looked into it and couldn't find a solution. This looks like a Google Drive limitation..
Please try to use other hosts when possible

The original links are google drive links.  Would it be feasible for HF to maintain their own servers for this?

It's possible to host data on our side but we should ask the authors. TFDS has the same issue and doesn't have a solution either afaik.
Otherwise you can use the google drive link, but it it's not that convenient because of this quota issue.

Okay. I imagine asking every author who shares their dataset on Google Drive will also be cumbersome.

I am getting this error as well. Is there a fix?

Not as long as the data is stored on GG drive unfortunately.
Maybe we can ask if there's a mirror ?
Hi @JafferWilson is there a download link to get cnn dailymail from another host than GG drive ?
To give you some context, this library provides tools to download and process datasets. For CNN DailyMail the data are downloaded from the link you provide on your github repository. Unfortunately because of GG drive quotas, many users are not able to load this dataset.

The following copy of CNN/DM dataset, fixed the problem for me:
https://huggingface.co/datasets/ccdv/cnn_dailymail

Thanks for the link @mrazizi !
Apparently the original authors don't host the dataset themselves ("for legal reasons", source here).