compatibility with spree 3.0.1

I am trying out this spree extension. when I run bundle install I got this error:
Bundler could not find compatible versions for gem "spree_core":
In Gemfile:
spree_core (~> 3.0.0) ruby
spree_core (~> 3.0.0) ruby

spree (= 3.0.1) ruby depends on
  spree_core (= 3.0.1) ruby

spree_travel (>= 0) ruby depends on
  spree_core (~> 2.4.0) ruby

I would recommend you not to look at this gem, we have based all the development using directly SpreeTravelCore, SpreeTravelHotel, SpreeTravelSample. Please use those gems. They are not ported to 3.0, the complete buying ceicle is completed in 2-4-stable,
I´ve started the migration but for some reasons i had to stop the work.
If you have any problem in setting up please feel free to contact me.

I am actually interested to use https://github.com/openjaf/spree_travel_flight. Do you still do the development there?  And the readme says to use branch 2-3-stable, but I see that there is already branch 2-4-stable. Is it safe to use it?

also is there specific rails version ? I remember that spree 2.4 require 4.1.8 or 4.1.11

SpreeTravelFlight is not right now in our road map.
I am using right now Rails 4.1.9
please use 2-4-stable in all cases


Allow child components and give them access to chartist props

Added code to be compatible with react-chartist-plugin-accessibility.

Thanks @theslip , new version has published to v0.12.0


syk7.6homework

git commit -m syks7.6homework

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


Make FastFixtureTestCase work in Django 1.7.

See also issue #193.
I wish I had an idea on how to write tests for this.
I didn't look into the transaction stuff in detail yet, and I'm not exactly sure if it's easy to resolve those in a backwards compatible manner.

Yay, random red.
https://travis-ci.org/django-nose/django-nose/builds/60888480 and https://travis-ci.org/django-nose/django-nose/builds/60884305 show a green for each test at least once.

I consider this green. The reds are just ssl timeouts while downloading packages.

@Pike - This happens a lot - TravisCI, PyPi have occasional bad network days.  When this happens, I re-run the failing build, rather than close/open.
When you click through to the failing build (https://travis-ci.org/django-nose/django-nose/jobs/60884311), do you see a grey reload button in the upper right corner?  This might only be available to project owners.

Only find the rerun button on my own projects, but not here, sadly.

Thanks for doing my research :smile: I've hit the reload buttons, we'll get it green.  It's a busy weekend, so I'll have to look closer in a few days.

Wow, that was one busy weekend.
@Pike - Could you rebase this onto the new master? I've made quite a few changes and it won't merge cleanly anymore.  git rebase -i master might do the trick.
I've also added some basic database tests in testapp/tests.py, but none using FastFixtureTestCase.  I'd appreciate some tests, which may just be a copy of the UsesFixtureTestCase.

I'd like to see some tests for FastFixtureTestCase (issue #220) before I merge fixes.

I'm afraid I took this as far as I could. This works in django <= 1.7, but breaks because the enter_transaction_management (internal?) api was killed in 1.8 and above. How to write the code in a way that doesn't break is sadly beyond my understanding of that code :-(


Removes superfluous calls to res.WriteHeader()

Signed-off-by: Omer Yahud<EMAIL_ADDRESS>What this PR does / why we need it:
Removes the unnecessary calls to response.WriteHeader() to make virt-api and virt-handler logs cleaner.
The net/http package returns http.StatusOK as a default if a different status code was not explicitly
written, this means that every call to res.WriteHeader(http.StatusOK) writes an info log like the two below and floods the logs for virt-api and virt-handler.
Which issue(s) this PR fixes
virt-api and virt-handler logs are being flooded because of unnecessary calls to response.WriteHeader()
Notes for the reviewer
The logs that are flooding virt-api:
`{"component":"virt-api","level":"info","msg":"http: superfluous response.WriteHeader call from kubevirt.io/kubevirt/pkg/virt-api/webhooks/mutating-webhook.serve (mutating-webhook.go:68)\n","pos":"server.go:3012","timestamp":"2020-02-05T12:43:31.454691Z"}
{"component":"virt-api","level":"info","msg":"http: superfluous response.WriteHeader call from kubevirt.io/kubevirt/pkg/util/webhooks/validating-webhooks.Serve (validating-webhook.go:61)\n","pos":"server.go:3012","timestamp":"2020-02-05T12:43:31.458961Z"}`

Release note:

NONE

/lgtm

/hold
Will check if there are other superfluous calls to resp.WriteHeader()

/hold cancel

/retest

/approve

@omeryahud thx!


Increase page size

This increases the page size for requests to the API from the default of 25 to 100. This should hopefully decrease the assault on the BK api. /cc @toolmantim

Thanks Lox!


[db] Optimize bulk insertions using fixed-size statement caches and base expansion

This ticket tracks making bulk insertions (like at https://github.com/ncalexan/mentat/blob/db-tests/db/src/db.rs#L631) more efficient.  This is follow-up to https://github.com/mozilla/mentat/pull/214#discussion_r99420334.
I don't know the best way to do this, so there's some experimentation needed.  I can think of the following considerations:

average transaction size is > 1 assertion (since there's always a :db/tx :db/txInstant MILLISECONDS datom)
most transactions will be small, say < 5 assertions
some transactions will be huge, say 10,000 or 100,000 assertions
SQLite will accept only a limited number of bindings per statement handle (usually 1000, but possibly as many 500,000 on some platforms!)
there's a finite store of SQLite statement handles available

So we want to cache a certain number of insertion statements for some number of assertions, and then chunk our input into those insertion statements.  If we had unlimited bindings per statement and unlimited statement handles, we might do a base-2 expansion and lazily cache insertions of 1, 2, 4, 8, 16, assertions.  However, given our considerations, it might be better to cache insertions for 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, (999 / bindings-per-insertion) assertions, effectively dividing the parameter space into "little" and "huge" regimes.
If we wanted to be really fancy, we could determine our caching strategy dynamically at initialization time, or we could generate different strategies at compile time for different hardware profiles (like most FFI implementations do).
This seems to me similar to "Montgomery ladders" for fast multiplication and "base phi expansions" for doing arithmetic in algebraic groups that have non-trivial computable morphisms, with some additional restrictions thrown in.

This isn't a good first bug, and it's probably best to wait for more of the transactor to be implemented before even poking at it.  Correct first, performant second.

It's might be worth computing some of these repeated_value strings at compile time as well, although since SQLite will cache the handles eventually this might be wasted space.


Nested eloquent relations detection.

In my applications, sometimes I have the following scenario:
/**
 * @return \Illuminate\Database\Eloquent\Relations\HasMany
 */
public function tags()
{
    return $this->hasMany(Tag::class);
}

/**
 * @return \Illuminate\Database\Eloquent\Relations\HasMany
 */
public function redTags()
{
    return $this->tags()->where('color', 'red');
}

Currently redTags is not included in the generated output.
This pull request addresses that by checking the return type of the functions when scanning for relations on a model. (both native return type and fallback in the @return docblock)

Doesn't this break stuff like
public function relation(): HasOne
{
  return $this->hasOne(SomeModel::class, 'foreign_key', 'key');
}

public function anotherRelation(): HasOne
{
  return $this->relation->anotherHasOne();
}

$this->relation is null when ide-helper is trying to figure out the nested relation.

@emielmolenaar
The way you do it is not recommended, and I don't think it should be covered.
Notice the difference between:
return $this->relation()->anotherHasOne();

And:
return $this->relation->anotherHasOne();

The second one actually fetches the relation from the database. (If you eager load 5000 entities with this relation it will do at least 5000 queries)
IMHO: The idea of a relation function is to actually return a relation and not fetch data.
Just out of curiosity why do you need to do it like that ?

@edcoreweb Thanks for respondig. I understand the difference between relation and relation() and I agree with you for a large part. Your first example will not work by the way, the method anotherHasOne doest not exist one the HasOne class :smiley:
We have some convenience methods that return a relation based on some conditions. For example:
public function getRelation(): HasOne
{
   if($this->anotherRelation->isSomeValue()) {
      return $this->someHasOneRelation()->where('some_field', 'some_value')->where('another_field', 'another_value');
   }

   return $this->someHasOneRelation()->where('yet_another_field', 'some_value');
}

In this case anotherRelation has already been loaded (we are very aware of the N + 1 query problem). We sometimes use this kind of methods to prevent duplicate lines of code.

@emielmolenaar
I see, I usually use scopes for this. The code as it is now needs to call the relation function using reflection (to get the related model), so as long as the relation function can run without context, you should be fine.
The easiest way to solve this in your case would be to check for the relation existence:
if ($this->anotherRelation && $this->anotherRelation->isSomeValue()) {
    // do stuff
}

or use the Null Object pattern and make a default model, and use it without the extra check
public function anotherRelation()
{
    return $this->belongsTo(SomeModel::class)->withDefault();
}


Remove 'overweight'

This commit removes the word 'overweight' from the corpus. This is an
insensitive term that implies that there is a certain weight that one should
be, rather than 'over' or 'under'.

Thanks, I'll publish a new release soon.


Coverity 1518564: Out-of-bounds write in fq_pacing plugin

I believe this is a false-positive and TSfread shouldn't return a value larger than the buffer size.    However, we should just use open, read, close instead of our hacky buffered file reading in our APIs.  We are only reading 4 bytes anyways.

Cherry-picked to v10.0.x


Clause extends from empty class breaks type narrowing

Bug Report
🔎 Search Terms
type narrowing
extends breaks narrowing
empty class
🕗 Version & Regression Information

This is the behavior in every version I tried starting from 3.3.3 to nightly builds.

⏯ Playground Link
Playground link with relevant code
💻 Code

type Class<T> = new (...args: Array<any>) => T;

class ComponentA { public a: number = 0; }
class ComponentB { public b: string = ''; }
class ComponentC { public c: boolean = false; }
class EmptyComponent {}

abstract class Entity {
    abstract get<T>(componentClass: Class<T>): T | undefined;
}

class Query<T extends Entity = Entity> {
    public get entities(): T[] {
        return [];
    }
}

// This type is used to force order of type check for `Entity.get` method
export type HasComponent<T> = { get<R extends T>(type: Class<R>): R; }

class QueryBuilder<T extends Entity = Entity> {
    public has<U>(componentOrTag: Class<U>): QueryBuilder<HasComponent<U> & T> {
        return this as QueryBuilder<HasComponent<U> & T>;
    }
    public build(): Query<T> {
        return new Query<T>();
    }
}

const query1 = new QueryBuilder().has(ComponentA).has(ComponentB).build();
for (const entity of query1.entities) {
    entity.get(ComponentA).a = 1;
    entity.get(ComponentB).b = 'a';
    // works as expected, and shows an error that object is possibly undefined
    entity.get(ComponentC).c = false;
}

// Adding EmptyComponent breaks types narrowing
const query2 = new QueryBuilder().has(EmptyComponent).has(ComponentA).has(ComponentB).build();
for (const entity of query2.entities) {
    entity.get(ComponentA).a = 1;
    entity.get(ComponentB).b = 'a';
    // Now we can `get` anyhing. Every component somehow extending EmptyComponent now, and I won't get message 'object is possibly undefined'
    entity.get(ComponentC).c = false;
}

🙁 Actual behavior
I'm trying to force type checks priority and kind of 'override' Entity.get method for the components that exist in the Entity to return defined value instead of undefined.
Error message object is possibly undefined is missing when you try to access a field of a component that is not defined.
This happens when you add HasComponent to the union type (QueryBuilder constructs that.
🙂 Expected behavior
As the ComponentC or ComponentA/B not actually extends EmptyComponent I should get an error.

As the ComponentC or ComponentA/B not actually extends EmptyComponent I should get an error.

But they do extend it. Everything that the EmptyComponent defines is present in the ABC components.
See also: https://github.com/microsoft/TypeScript/wiki/FAQ#why-do-these-empty-classes-behave-strangely

Oh, that's why. Well, I have to check FAQ first. Thank you.


1.Can I update an image by Image_id?

Thank you very much for your contribution.
1.Can I update an image by Image_id?
2.Is it possible to specify image_id when an example is uploaded?

Hi, no both operations are impossible.
If you need to replace the image - you just delete it and upload a new one.
Could you tell more about the case where operations your mentioned are useful?

I want to replace the image ID with the user ID, so you don't have to store all the relationships between them.

Hi, no both operations are impossible. If you need to replace the image - you just delete it and upload a new one. Could you tell more about the case where operations your mentioned are useful?

I want to replace the image ID with the user ID, so i don't have to store  the relationships between them.


new TUI does not resize on window resize

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
not necessary, just create a monorepo and run turbo

I experience the same in iTerm on macOS

For me it's different.
With the old TUI, the TUI wasn't resizing along the terminal pane. Same as you are describing.
BUT with the new TUI i'm experiencing a different behaviour.
If i start a command with a little window, the TUI will not be present, it will just revert the turborepo process to behave as interactive: false (like, it's working exactly as before the TUI got introduced).
If i start the command with a taller window, the new TUI takes over and works perfectly.

Just noticed that actually this is an intended behavior made in https://github.com/vercel/turbo/pull/8457.
Why was this decision taken? This doesn't look right.

@AndreaPontrandolfo This decision was made due to the TUI requiring a certain minimum amount of area. Attempting to run the TUI in a terminal with less area results in an incorrect render or a crash.


Update interface of SigPro w/pipelines and primitives

Larger update:

Add Primitive and Pipeline base classes (Issue #51)
Added contributing_primitive and basic_primitives module to assisst with new primitive creation/usage
Added tree and layer pipeline creation (Issues #41 and #52)
Additional testing for pipelines
Additional documentation and demo notebooks for new pipelines

Additional modifications:

Fix some existing primitive JSONs (Issue #53)
Demo fix (Issue #35)

Made an additional slight modification to add a primitive_inputs argument. This is necessary so as to not break the FrequencyBand primitive, which has a non-standard input argument.


Add more medical staff

A real ER is teaming with doctors, residents, nurses, and techs. Right now we have only one of each.

Added functionality for multiple of each medical staff except for ct, radiology, and attending. CT I was just unsure if we want to have multiple cat scans able to occur at once, also it will likely require some more edits than the previous staff did. Radiology doesn't do really do much right now, and the attending currently doesn't do anything besides wait.

There are two CAT scan rooms. The current simulation places the CAT scan tech in the first room and the radiologist in the second room. What really happens is that there is a radiologist someone above the ER who reviews all the CAT scans before sending the to the doctor/resident.
To match current behavior, we need two CAT scans running in parallel. We need to put another CAT scan tech in the second CAT scan room and move the radiologist somewhere else. I initially placed the radiologist in a CAT scan room so that the radiologist's behavior would work like normal. However, if we can find a way to get rid of the radiologist and have CAT scans reviewed after a given time frame, then we can do that. If he need an embodiment of someone to have a behavior, then we should send the radiologist into a random room, into a hall, etc.

There are now two cat scans and ct's running simultaneously. I will leave the radiologist and attending as is until it becomes higher priority!


Integration tests against latest 0.3.6 version of the server

I will submit the PR in a moment. Create the issue just to link PR to it.

Closed with #48 and #49


Raidboss: Add e9n replace things

@Akurosia May you check the Deutsch part?

I was waiting for @Akurosia to chime in here, because I know they have some modified scripts that usually generate this.

sry i missed the mention here Q_Q
except the text sync, the rest looked fine

@quisquous I think it is safe to merge now.


Add a 16-bit 80{1,2,}86 compiler

Please add some 80{1,2,}86 16 bit compiler.
One of the uses for the compiler explorer is help in reverse engineering, for cross checking if some assembler is doing what you think it does it is convenient to put in the C[++] code and check if the assembler is about the same.
So far none of the compilers have support for the real segmented 16bit code model of pre 386 times.

I'd suggest Open Watcom C/C++ for this.

It's open-source
It doesn't require Wine or a DOS emulator
A single install comes with support for targeting every supported platform. (eg. one Linux install gets you to every legacy platform of note that can run on an 80x86 CPU, such as real-mode DOS, DPMI, NetWare, multiple types of OS/2 binaries, multiple types of Win16 binaries, etc.)
Anything DOS/4GW-based that you're trying to reverse-engineer would have been compiled using Watcom C/C++. (DOS/4GW = DOS/4G, Special Watcom Bundle Version)
Things like the Windows version of Sierra's SCI runtime are built using Watcom's Win386 extender. (It's basically a DOS/4GW equivalent for Win16, providing a bootstrap and API thunks for writing protected-mode Win16 executables.)
If you want to study what code compiles to for reverse-engineering purposes, you want a compiler which lets you choose between as many different calling conventions as possible.

the GCC IA 16 Project could be also added as possible DOS/16 bit compiler: https://github.com/tkchia/gcc-ia16

Huh. I thought the only DOS-targeting GCC build was DJGPP's DPMI target. Thanks for that. :)

old leacy 16bit code generated by up to date Optimizer :)

Once my Watcom hobby project's design has stabilized, I'll have to try rewriting the inline assembly (int 10h FFI wrappers) to see how much more size I can shave off with GCC IA 16's size optimizations, proper dead code elimination, and LTO.
I already know that a void main(void) {} test is smaller. (Open Watcom's real-mode CRT bootstrap produces a 996-byte .com file when configured for maximum size optimization, with an .exe header increasing that. ia16-elf-gcc test.c -Os -flto produces a 780-byte test .exe.)
(It's an installer builder for DOS akin to what InnoSetup is for Windows, which means it needs to not crowd out actual content if placed on floppy disks.)


MySQL scanning POLYGON WKB Always results in EMPTY

For example I used this and scanned it to a geo.PointSet
SET @g = 'POLYGON((50.866753 5.686455, 50.859819 5.708942, 50.851475 5.722675, 50.841611 5.720615, 50.834023 5.708427, 50.840744 5.689373, 50.858735 5.673923, 50.866753 5.686455))';
SELECT ST_AsBinary(PolyFromText(@g));

var p *geo.PointSet
row.Scan(&p)

// 	p == EMPTY

It's very likely that I'm way  off on this. What am I doing wrong?

I dug a little deeper, it seems PointSet is just []Point. So library doesn't actually support POLYGON or else there has to be some type with [][]Point. I don't know if this was intentional or not.
#L221-L223 looks like it wants to support POLYGON but then (#L225)[https://github.com/paulmach/go.geo/blob/e8994de4ca767ad1e416ce5e185f83538f0661a8/wkb.go#L225] will fail with POLYGON WKB because what follows is number of rings not number of points.
I think this is the most complete library for handling Geo-spatial data in Go and it would be nice to have POLYGON support, specially WKB support as it will make working with databases a breeze.

Yes, did the same digging :)   Here is the fix https://github.com/paulmach/go.geo/pull/34

Nice! :+1: Thanks!. Fortunately I don't need multi ring support.

@paulmach, if you want a load of WKB test data covering many geometry types, feel free to copy it from https://github.com/twpayne/go-geom/blob/master/encoding/wkb/wkb_test.go and https://github.com/twpayne/go-geom/blob/master/internal/testdata/random.go.

I think this is fixed, but there is more work to do. Thank @twpayne I'll look into adding more tests.


Add anchors for template arguments in declaration references. [phab:D1199]

Created by pcc at 2016-12-29 03:28:10:

zarko accepted this revision at 2017-01-03 22:48:26

zarko closed via commit 4b04f3475ac2f48215dfaa3a3693c53f534f1a16 at 2017-01-03 22:48:57


Give access to re-apply share button to consumers

Is your idea related to a problem? Please describe.
Currently, only a dataset owner/approver has access to fix a share by clicking the re-apply share button.
If a requestor finds that their share is in unhealthy state, they need to reach out to dataall admins or the dataset owners which causes an unnecessary wait cycle when a production system share could be broken.
Describe the solution you'd like
I propose giving access to the requestor team to the re-apply share button. A share would be a joint responsibility of a producer and consumer. Consumers should be able to hit the button and fix their own shares without relying on anyone else to fix it. We additionally would like the producers to get notified via email + UI that their share had broken and was in unhealthy state and fixed by a consumer. This would ensure that the error message from unhealthy status is also captured and sent to producers so they can fix any underlying issues that caused the problems instead of just bandaging them.
P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

Hi @anushka-singh - for the notification aspect of the above issue, let's use #1420 as the parent issue to improve all types of notifications on data.all
For reapply - it is true only Approvers are able to reapply a share at the moment intentionally so. Because re-apply is going through very similar motions to approve and potentially would be editing the permissioning on the data resources it felt safest to restrict re-applying only to the Approver persona in the share
That being said I do understand the alternative line of thought that once a share is approved it is a shared responsibility to manage the health of that share. I can not think of current risks of doing so - @anmolsgandhi @dlpzx  do you see any inherit risk here with this approach?
If not this could be a nice enhancement to the current functionality


Solve problem with different audio-formats on computers

Currently both computers need to have the same audio configuration (sample-rate, format, samples per package).
We need to find a way to either

allow for varying input and output configuration
or
agree on a common configuration.

Possible solutions:

use custom payload-format with the audio-configuration as parameter
write SIP stack to agree on configuration

See #35

As of now, audio-format and sample-rate can be set manually.
With the completion of passive configuration, there will also be a way to circumvent this problem automatically.

By using the passive configuration-mode, the audio-configuration will be synced between the two devices.


Bluetooth: Host: Raw API for HCI LTK REQ EVT

See test for usage.

Closing for update. Will reopen with new revision.


No Proper Steps

after downloading the source getting an error

Please make an easy and understandable documentation for wallet implementation.

Yes, I need to add that to the documentation...
You need to download two repositories from this account, EthersWallet-iOS and ethers.objc. Place them in the same folder, and then try building the Xcode project in EthersWallet-iOS.
I will leave this ticket open to remind me to update the documentation.
Thanks!

Hi, I've still unable to build it, can anyone provide the instruction?


docs: add aws plugin to executor plugin directory

Add a workflow executor plugin for AWS services.
Motivation

Provide ability to trigger the execution of AWS services.
Modifications

No changes.
Verification

N/A

duplicate of https://github.com/argoproj/argo-workflows/pull/12169
@agilgur5, this should be good to go.

@terrytangyuan , bump!


Fix comment in trace logging of blockchain module

Description
Fixes a trivial typo in the trace logging output of the blockchain module. Also squeezed in a minor code comment typo fix.
Tested locally with,
cargo build --all-targets
cargo test --workspace

All ✅  (my first attempts)

Screenshots (optional)
PR Checklist

[x] I've read the contributing guidelines.
[x] I've read "What is a Good PR?"
[x] I've included a good title and brief description along with how to review them.
[ ] I've linked any associated an issue.

Code Contributions

[x] I've built the code cargo build --all-targets successfully.
[x] I've run the unit tests cargo test --workspace and everything passes.
[x] I've made sure my rust toolchain is current rustup update.

Compare two following programs, the second does not compile.
1:
let x = String::new();
let _ = x;
println!("{}", x);
2:
let x = String::new();
x;
println!("{}", x);   // error[E0382]: borrow of moved value: x
"let _ =" still perform the right hand, just don't need the result.


fix maximum number of messages

Is the pull request based off the latest version?
Yes
What bugs did you fix?
If $number is higher than the number of $ids, $ids becomes null.
e.g. $number = 30, $ids = 21 items.
Is your code valid PSR-2?
Yes
Have you properly documented your code?
Yes
Has anything in your pull request already been fixed?
Don't know

Thanks


Angular Material with ui-mask

With this ....
//if ( ! originalPlaceholder) {
//    iElement.attr('placeholder', maskPlaceholder);
//}
I got this working fine!! Cause Angular Material input gets previous label to placehold the field with great style.

I'm not sure why you opened this issue. Is there an issue? If so, what is it?

Sorry ... i was not clear!
This is a incompatibility issue. If i download this plugin, and install in my modular app, this block breaks the modular layout ..
Modular uses label to create placeholder, but this block forces a creation of a poor placeholder with mask value.
I sugest that you change this in the new version, cause no one needs the mask as placeholder. :D

I'll look into providing an option for this functionality (ui-mask not adding the placeholder) but completely removing it could cause issues for people relying on this feature to have a placeholder.

The problem is that if i say to directive that want no placeholder, so i think the directive couldn't force this.
But i just wanted to help! thanks

@waldandrade Try out the new config option addDefaultPlaceholder and let me know if that works for you.


Upload result "Raw file: Unknown"

With API v2 I can upload with sample python script, and "Raw file" always same with file name.
But with API v3, "Raw file" always is "unknown".
I tried both sample python: https://github.com/youtube/api-samples/tree/master/python script and youtube-upload tool: https://github.com/tokland/youtube-upload, but no luck :"/
This property play important role in video SEO ranking.

Any workaround, please!

Triaging.
I verified that the description of the symptom is accurate via:
bin/python /home/chrism/projects/gcloud/youtube-api-samples/python/upload_video.py --file wickedgame.mp4 --title "Wicked Game" --description "Wicked game recording"

"Raw file" field is indeed "unknown".
Not sure about whether this is a feature or a bug.

The v2 API is gone, apparently:
https://youtube-eng.googleblog.com/2015/04/bye-bye-youtube-data-api-v2.html

Uploads via the API still produce a raw field name or "unknown". The raw filename does not play a role in SEO ranking. If you have another reason for wanting accurate filenames from API uploads, please try adding the 'Slug' header as a temporary workaround.
  query = service.videos().insert(
    body=resource,
    media_body=media_body,
    **kwargs
  )
  query.headers['Slug'] = 'raw_filename_here.flv'
  results = query.execute()


Top-level data source templates

Summary
Based on the discussion on https://github.com/graphprotocol/support/issues/19, and https://github.com/graphprotocol/graph-node/issues/1017, we want to move data source templates from being owned by a data source to being its own top-level field in the manifest, like this:
dataSources:
  - ...
templates:
  - ...

Requirements
The new requirement being covered is that any data source should be able to instantiate templates declared in the template section, otherwise this should behave exactly like templates do today.
We need to decide on a migration path.
For graph-node we need to maitain backwards-compatibility, I suggest we do this by supporting either the old form or the new form, but not a mix of both in the same manifest. We should log a deprecation warning when a subgraph uses the old form.
For graph-cli, a migration might not be a good idea, because if we simply move the nested data sources to the top level field, that might create an invalid manifest because of name conflicts. Also we haven't yet figured out how to keep yaml comments on complex migrations. To aviod a migration, we can either error or warn (which one should it be?) if the manifest uses the old form.
Proposed Implementation
graph-cli: Implement the first step of the migration path we decide on, accept and validate top-level templates. Codegen for top-level templates. Forbid mixing the old and new forms.
graph-node: Parse the new form. Detect whether the subgraph uses the old or the new form, error if it mixes both. Make sure all data sources can access all top-level templates. Make the code as generic as possible over both forms. Add the top-level templates to the subgraph of subgraphs.
graph-ts: No change.
Proposed Documentation Updates

Update the subgraph manifest reference in https://github.com/graphprotocol/graph-node/blob/master/docs/subgraph-manifest.md by marking the deprecated field as such and documenting the new field.
Update the data source templates section of the docs https://thegraph.com/docs/define-a-subgraph#data-source-templates.

Proposed Tests / Acceptance Criteria

Validation tests on graph-cli.
Manual testing of graph-node functionality.

Tasks
Task list so far:

[ ] (1d) graph-cli: Accept, validate and do codegen for templates in the top level field.
[ ] (0.5d) graph-cli: Forbid mixing the new and old form of templates.
[ ] (0.5d) graph-node: Parse top-level templates.
[ ] (1d) graph-node: Add an enum to support both forms, detect and error if they are mixed, log warning on use of the old form.
[ ] (0.5d) graph-node: Make the runtime take a list of templates instead of directly taking DataSource.
[ ] (0.5d) graph-node: Add top-level templates to the subgraph of subgraphs.

Open questions

We'll we do a graph-cli migration? If not, will using the current form be an error or a warning?

@Jannis Thanks, I've updated the plan.


Magic Connect doesn't load

Magic Connect keeps loading forever and doesn't show the interface to  make the login

@AN-Drew207 apologies for the confusion, atm MC isn't compatible with some of our extensions. In this case, our @magic-ext/bitcoin  extension. Try removing it and see if the issue is resolved. Thanks!

It worked, Thank you so much!


(PUP-4196) Remove "{}" from variables in systemd files

The "{}" prevent systemd from interpreting values correctly and causes
services to fail to start. {} indicates that the variable is a single
argument[1].
This is largely a cherry-pick of the fix for PUP-3228.
[1] - http://www.freedesktop.org/software/systemd/man/systemd.service.html

SYSTEMD

:+1:


Run tests with mdv and apply mdv only to versions directory

What?
I'm making npm test to run "tests" and mdv. Also, I'm limiting mdv to check for versions directory only.
Why?
So it doesn't bother on README with all-contributors bot.

I'm merging this one because I want to continue working on #122.


don't send ERROR:crowded in response to CLOSE

In https://github.com/magic-wormhole/magic-wormhole/issues/408 I think a client received ERROR(crowded)  (eventually, because of #19, so only after the server was rebooted and the client reconnected, and re-sent the OPEN), and then tried to release the mailbox by sending CLOSE, but was thwarted because the server sent back another ERROR(crowded). I think the client then got stuck, unable to exit (because it thinks it still has a hold on the mailbox), not sending a new CLOSE (because it already sent one and it's just waiting for CLOSED).
The fix is probably to have handle_close() (in the clause that does self._app.open_mailbox(), because this connection didn't have one already) react to CrowdedError by doing most of the rest of the function, instead of raising Error("crowded"). We can't do anything that touches self._mailbox, since we don't have one, but we can set self._did_close and (most importantly) do self.send("closed"). That should satisfy the unsettled ghost client and allow it to finally exit.

not sending a new CLOSE (because it already sent one and it's just waiting for CLOSED)

IMO that part is a bug on the client side: it sent a message and got an error response, thus it should not expect that message to have passed.
The relevant server side code appears to be this:
        if not self._mailbox:
            try:
                self._mailbox = self._app.open_mailbox(mailbox_id, self._side,
                                                       server_rx)
            except CrowdedError:
                raise Error("crowded")

I don't understand: why are we opening a new mailbox in the first place? If there's no mailbox on close, simply ignore it as already closed?

This seems like it could be related to what Im seeing right now on the public wormhole service.
I recent was reusing the same wormhole code during some testing. After about 12+ sends using the same code I got a waiting for confirmation and then after ctrl-c since it seemed to hang I got subsequent crowded errors. These go away if I just get rid of the reused code.
The following behavior is what Im see this morning:
$ wormhole send --code 2-businessman-pheasant openwrt-ath79-generic-netgear_wndr3800ch-squashfs-factory.img
Sending 12.0 MB file named 'openwrt-ath79-generic-netgear_wndr3800ch-squashfs-factory.img'
Wormhole code is: 2-businessman-pheasant
On the other computer, please run:

wormhole receive 2-businessman-pheasant

Key established, waiting for confirmation...

Then the following crowded error:
$ wormhole send --code 2-businessman-pheasant openwrt-ath79-generic-netgear_wndr3800ch-squashfs-factory.img
Sending 12.0 MB file named 'openwrt-ath79-generic-netgear_wndr3800ch-squashfs-factory.img'
Wormhole code is: 2-businessman-pheasant
On the other computer, please run:

wormhole receive 2-businessman-pheasant

Traceback (most recent call last):
--- <exception caught here> ---
  File "/nix/store/g9bwgzbj86sjyszyanzpxqjc0db3kbbp-python3.9-magic-wormhole-0.12.0/lib/python3.9/site-packages/wormhole/cli/cli.py", line 122, in _dispatch_command
    yield maybeDeferred(command)
  File "/nix/store/g9bwgzbj86sjyszyanzpxqjc0db3kbbp-python3.9-magic-wormhole-0.12.0/lib/python3.9/site-packages/wormhole/cli/cmd_send.py", line 93, in go
    yield d
  File "/nix/store/g9bwgzbj86sjyszyanzpxqjc0db3kbbp-python3.9-magic-wormhole-0.12.0/lib/python3.9/site-packages/wormhole/cli/cmd_send.py", line 139, in _go
    yield w.get_unverified_key()
wormhole.errors.ServerError: crowded
ERROR: crowded

Again I can just use a new code but figured it was worth reporting since its fresh. Happy to open up another issue if thats more appropriate.

I can confirm that reusing codes multiple times causes spurious "crowded" issues which could be related to this issue. I have no specific reproduction recipe, but I strongly suspect there is a race hazard involved.
@sarcasticadmin please make sure that you know the security consequences of using a code multiple times and don't use this in production without counter-measures.

@piegamesde thanks for the clarification

@sarcasticadmin please make sure that you know the security consequences of using a code multiple times and don't use this in production without counter-measures.

Yep, absolutely. Its was just for testing and pure laziness so I didnt have to keep updating inputs.


commander preflight delete individual sensor selftest

These checks are pretty worthless and implemented inconsistently (or not at all) in the drivers.

None of these self tests do anything with the actual device.


chore: add js file extension for js

Summary

Test plan

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

Project coverage is 100.00%. Comparing base (42f8a3b) to head (2d47bc2).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #80   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines          190       190
  Branches        42        42
=========================================
  Hits           190       190

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


How can I reset bigImageViewer after display an image?

How can I reset bigImageViewer after display an image?

You mean display another image?

How can I reset bigImageViewer after display an image?

You can use this method.
"imageView.cancel();"

Thank you for your reply. I wanted to reset bigImage to initiate state that bigImage does not display anything. I found a method bigImage.removeAllViews() that solved my problem.


Only use tx on #delete_insert for :atomic_write

Repositories that don't support :atomic_write shouldn't try to use a transaction for #delete_insert.
Repositories answering true to :atomic_write promise to deliver an atomic #apply_changeset.

Does this address the remaining issues on https://github.com/ruby-rdf/rdf/issues/190 ?
(I can't test the other repositories at the moment, but will try to circle back soon).

I was able to steal a moment to test SPARQL::Client::Repository, this fixes it, except for some largely unrelated issues. See: https://github.com/ruby-rdf/sparql-client/commit/dda25c78066398e6c8fab5f589849d1a1f1d8cbe

Looks like that should do it, but I remain a bit unclear on the underlying theory, and what a repository implementation should do. Why starting a transaction in #delete_insert is okay in one case (and doesn't lead to recursion), but not in another isn't clear to me.
Is there some guidance on how rdf-mongo, or rdf-do should implement this? Certainly, in the case of rdf-do, there is an underlying transaction model that could be use; how would a repository factor this in?

The base implementation is:

Mutable#delete_insert does #delete and #insert (non-atomic).
Mutable#apply_changeset calls #delete_insert (non-atomic)
Transaction#execute calls #apply_changeset

If a Repository supports :atomic_write it must provide both of these methods with atomic updates. #apply_changeset is the key method (see Transaction's YARD annotations; any repository that implements :atomic_write needs to have custom changeset application logic. #delete_insert must be transactional.
So, if :atomic_write is supported.

Repository#delete_insert should run a transaction

It falls back on Mutable#delete_insert (and avoids a transaction) for non-atomic repositories.

Repository#apply_changeset gets a custom implementation, and doesn't call #delete_insert.

That leaves Transaction#execute. Transaction subclasses may provide their own implementation of this, avoiding #apply_changeset altogether. For example, they might send an upstream request to commit (or do @data = tx_data as in Implementation). In this case, it is permissible to implement #apply_changeset in terms of transactions, as well. This would be up to the repository.
I'm open to suggestions on simplifying this. For the moment, the simple path to changeset-driven transactions is a one-step, atomic #apply_changeset.

No, I think that's fine. We should eventually update rdf-do to implement this transaction/atomic-write pattern to serve as an example for other repos. I'll see if we can do something similar with Mongo, but I doubt it.

I'll take a hack at some more thorough transaction docs targeted at Repository implementers later in the week.
I'm trying to use my limited time on this carefully. My todo, in order, is:

https://github.com/ruby-rdf/rdf/issues/269
https://github.com/ruby-rdf/rdf/issues/242
Setup github.io blog
Transactions/Repositories docs/blog posts.

Sure, updating rdf-do is not urgent, other than to make it basically work (which it should, now).
Regarding the blog, for this particular case, we should probably use the GitHub release notes on RDF.rb, once the release is tagged.
Even though it would be nice to get things out, we're not on a clock, and we're not being paid to do the work, so work on whatever schedule you're comfortable with.


Update to intro text

Here's new copy for the intro text
HOT CHOCOLATE STAND
The price of each cup of hot chocolate is decided by Fuzzy.ai. Try changing the number of people in line, the temperature or the weather and see how that affects the price. Check out the Profits chart below to see how well Fuzzy.ai learns to maximize profit.

closed via #a107aea


feat: removing atomics

Description

removing atomics

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Stop autostart

Hey,
I'm a bit new to this all. I've installed the deployment setup successfully, and got it all up and running. But how can I set it so it won't start up together with my PI every time? As this won't be something I'll need all the time, so I rather have it set that I need to enter a command to start it, instead of entering a command to stop it again.

Hey @justinkruit,
that depends on the linux distribution, I think Raspbian is based on Debian 7, so systemctl disable runit.service should disable runit which is used to start Huginn.

@dsander Yup, that did the trick, thank you!

Uhm.. Now that I tried to start it again, it gave some errors about runsv not running. Is there a way I can just start Huginn again with a command, without the need to enable and disable the service every time when needed?

Did you try systemctl start runit.service? That should start runit which then starts Huginn processes you configured in your Procfile.

Hi @justinkruit, is it okay if we close out this issue?

@cantino yeah sure, no problem.


Add subscription of mails for users

Feature description

/custom_mails/new - this route is only available to admins and can be used to send custom emails to subscribed users

Upon creation a preview of the email is shown with options to edit or send it right away

Users can unsubscribe to e-mails from the profile settings page

Notes -

Users are processed in batch sizes of 1000, currently, there are 7k users.
User table export is only 2MB in size hence there is no significant load on memory while loading the user data to send mails.

TODO -

[x] Add tests to increase test coverage
[x] Provide a way to unsubscribe from mails from with the sent mail

Coverage decreased (-2.5%) to 81.38% when pulling 794d7f5f6b8a4dc8deb37a26684c6e7740961f19 on ankitkataria:feature/mail-subscription into bbd0bc3c28e20afaab533b161bb174b921cc4e05 on CircuitVerse:master.

Amazing job!


crashing when playing songs with metadata

works perfectly, until i play a song that has metadata in it. more specifically the "title" section of the metadata filled out.

Already fixed :)

Just a thought, what is the error?

Fixed with latest release


Devilutionx error

Hello there just for your info devilutionx installation in not working,after a while it stops with an error saying:
home/pi/RetroPie-Setup/ext/RetroPie-Extra/scriptmodules/ports/devilutionx.sh line 31: cmake: command not found
home/pi/RetroPie-Setup/ext/RetroPie-Extra/scriptmodules/ports/devilutionx.sh line 32: cmake: command not found
Could not successfully build devilutionx diablo engine
(/home/pi/RetroPie-Setup/tmp/build/devilutionx/build/devilutionx not found)

What you posted doesn't tell me anything. That's the generic error that everything says when it fails
You need to post the log file.

cmake is missing from the depends you can understand that i think,when I
installed it manual everything worked fine.
On Sat, Oct 21, 2023, 2:05 AM Exarkuniv @.***> wrote:

What you posted doesn't tell me anything. That's the generic error that
everything says when it fails
You need to post the log file.
—
Reply to this email directly, view it on GitHub
https://github.com/Exarkuniv/RetroPie-Extra/issues/164#issuecomment-1773492184,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJCXONOGJBRGFK5AXKQE4OLYAL7SRAVCNFSM6AAAAAA6IUQZLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTGQ4TEMJYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Added AlphaVantage adapter

In fact of  Yahoo's decision to discontinue the Yahoo Finance API, I decided to search for another exchange rate service and found the free AlphaVantage API. You only have to obtain a free API key at https://www.alphavantage.co/support/#api-key

@cedric-g Do you need any information in order to merge this PR?

@lexikteam Any update on this?

@jdambacher
Hi,
Sorry for the delay it's difficult this time for us to maintain all theses bundles :smile:
I wanted to merge your PR, but I'm not sure about if we have to delete the yahoo adapter in the same PR. This could add some BC for users who still use YahooAdapter.
Maybe you could remove your last commit for this PR, and create an other PR for Yahoo, and I'll create a special tag for this deletion. What do you think?

@spike31 No worries!
I think this is the only option, because the yahoo API is completely disabled. I agree this i a BC, but the adapter is currently completely useless.


ci: use debug build for tests

Summary
This should:

reduce ci build time
have a debug build available to download from the uploaded artifacts

We can tune some of the dependencies to use optimization if things start to run slow.

🤖 Generated by Copilot at 26e239a
This pull request adds support for building the binding with different Rust profiles using a new profile parameter for the docker-build action and the workflows that use it. This allows for faster and more flexible builds of the binding for different purposes.

Walkthrough

🤖 Generated by Copilot at 26e239a

Add profile input parameter to docker-build action to allow specifying Rust profile for building the binding (link,link)
Add profile parameter to reusable-build workflow to pass it to docker-build action for different targets (link,link,link,link,link,link,link,link)
Set profile parameter to debug in ci workflow for faster builds and more debugging features (link,link,link)
Remove trailing whitespace from reusable-build workflow (link)

ping @IWANABETHATGUY


Sorting of source by path doesnt work

Issue Type: Bug
Hi Team ,
I see that with new version 1.66.2 sorting of source by path is not working correctly. Also not all the subprojects are shown as part of source control.
VS Code version: Code 1.66.2 (dfd34e8260c270da74b5c2d86d61aee4b6d56977, 2022-04-11T07:46:01.075Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz (8 x 1896)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.84GB (0.78GB free)

Process Argv
--crash-reporter-id ca8ce366-73c9-466c-b7c9-f5bc31d27c8c

Screen Reader
no

VM
0%

1.67 is likely to be released later today. I suggest you re-test with that, then paste screenshots (obfuscated if necessary) if you still have the problems.

I have (a version of) this bug in 1.67.0. It looks like all repos in a directory are listed before submodules of either of them. Example:
Is:
a
a/b
a/c
a/b/d
a/b/g
a/b/d/f
a/c/e
Expected:
a
a/b
a/b/d
a/b/d/f
a/b/g
a/c
a/c/e
I have not had an instance of missing submodules.

Hi ,
I tested today and I see the sorting issue is resolved with new 1.67
version.
I think I need to update the visible items of submodule from 10 to 20 in
user and workspace settings.
On Mon, 9 May 2022, 10:57 frank-weinberg, @.***> wrote:

I have (a version of) this bug in 1.67.0. It looks like all repos in a
directory are listed before submodules of either of them. Example:
Is:
a
a/b
a/c
a/b/d
a/b/g
a/b/d/f
a/c/e
Expected:
a
a/b
a/b/d
a/b/d/f
a/b/g
a/c
a/c/e
I have not had an instance of missing submodules.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/148784#issuecomment-1120831038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZA5XZM74KBW2UA5IKITBIDVJDHOTANCNFSM5VEPZPRA
.
You are receiving this because you authored the thread.Message ID:
@.***>


IOS: Cannot delete memory

Describe the bug
Cannot delete memory on IOS
To Reproduce
Steps to reproduce the behavior:

Create a memory
click on memory
select ... and delete
Taken back to memory list with memory still present.  refreshing screen does nothing, and you can click on the memory again

Expected behavior

Toast messages shown "memory deleted"
Return to memory list in same position as before (in case user wants to delete multiple memories in a large list
Memory is not present

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: IOS
Version 1.035+125

Smartphone (please complete the following information):

Device: iphone 15
OS: 17.6.1
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
https://github.com/user-attachments/assets/5527cfb0-e260-48d6-9986-c3cc505a3b6b

Hi there, the issue has been resolved in the latest version, which is currently under review by Apple. It will be available once the review is complete. Thank you for reporting it!


Height not calculated based on children

Thank you for this excellent library! I've run into a layout issue that seems like a bug to me. If I create a column with two rows of unspecified height and add a child to the first row, I would expect the height of the first row to be calculated from it's children and then the second row to be positioned below the first row, like so:

Instead I'm seeing the first row having a height of zero, causing the second row to be draw over the children of the first row, like so:

Here is a repro program:
package main

import (
	"flag"
	"fmt"
	"log"

	"github.com/hajimehoshi/ebiten/v2"
	"github.com/hajimehoshi/ebiten/v2/inpututil"
	"github.com/yohamta/furex/v2"
)

type Game struct {
	mainView *furex.View
}

func NewGame() *Game {
	return &Game{}
}

var showExpected bool

func BuildGameView(width, height int) *furex.View {
	var firstRowHeight int
	if showExpected {
		firstRowHeight = 100
	}
	firstRow := &furex.View{
		Direction: furex.Row,
		Grow:      0,
		Shrink:    0,
		// Based on flex box experience, I think omitting Height here should
		// cause the height to be calculated from it's children.
		Height: firstRowHeight,
	}
	firstRow.AddChild(&furex.View{
		Width:  100,
		Height: 100,
	})

	secondRow := &furex.View{
		Direction: furex.Row,
		Height:    200,
		Grow:      0,
		Shrink:    0,
	}
	secondRow.AddChild(&furex.View{
		Width:  50,
		Height: 50,
	})

	v := &furex.View{
		Direction:  furex.Column,
		Width:      width,
		Height:     height,
		Grow:       0,
		Shrink:     0,
		Justify:    furex.JustifyStart,
		AlignItems: furex.AlignItemCenter,
	}

	v.AddChild(firstRow)
	v.AddChild(secondRow)

	return v
}

func (g *Game) Layout(outsideWidth int, outsideHeight int) (int, int) {
	g.mainView = BuildGameView(outsideWidth, outsideHeight)
	return outsideWidth, outsideHeight
}

func (g *Game) Draw(screen *ebiten.Image) {
	if g.mainView != nil {
		g.mainView.Draw(screen)
	}
}

func (g *Game) Update() error {
	if inpututil.IsKeyJustReleased(ebiten.KeyEscape) || (inpututil.IsKeyJustReleased(ebiten.KeyQ) && inpututil.IsKeyJustReleased(ebiten.KeyControlLeft)) {
		return fmt.Errorf("Quitting...")
	}
	return nil
}

func main() {
	furex.Debug = true
	flag.BoolVar(&showExpected, "showExpected", false, "whether to show the expected layout")
	flag.Parse()

	ebiten.SetWindowSize(1280, 720)
	ebiten.SetWindowTitle("Repro")

	gameData := Game{}

	if err := ebiten.RunGame(&gameData); err != nil {
		log.Print(err)
	}
}

And a go.mod for version info:
module furex_repro

go 1.19

require (
	github.com/ebitengine/purego v0.1.1 // indirect
	github.com/go-gl/glfw/v3.3/glfw v0.0.0-20221017161538-93cebf72946b // indirect
	github.com/hajimehoshi/ebiten/v2 v2.4.13 // indirect
	github.com/hajimehoshi/file2byteslice v1.0.0 // indirect
	github.com/jezek/xgb v1.1.0 // indirect
	github.com/yohamta/furex/v2 v2.2.3 // indirect
	golang.org/x/exp/shiny v0.0.0-20221126150942-6ab00d035af9 // indirect
	golang.org/x/image v0.1.0 // indirect
	golang.org/x/mobile v0.0.0-20221110043201-43a038452099 // indirect
	golang.org/x/sys v0.2.0 // indirect
)

Hi @dvogel 👋
Thank you for raising the issue. Your provided reproducible code was very helpful. It seems that the issue was caused by a missing implementation of part of the Flexbox algorithm. I have just fixed it somehow, so please try the v2.2.4. Please be noted that not all specifications of the Flexbox algorithm are accurately implemented in this library.
Thanks again and let me know if you have any further questions or concerns!

Thanks for adding this functionality @yohamta! I just updated my game to use v2.2.4 and it works as expected there too.


HIVE-28616: OrcReader is not closed in OrcEncodedDataReader

What changes were proposed in this pull request?
OrcEncodedDataReader fully owns the OrcReader instance it creates, so it should be closed there too
Why are the changes needed?
It causes connection leaks in case of FileSystems like S3 (where an HTTP connection pool is utilized)
Does this PR introduce any user-facing change?
No.
Is the change a dependency upgrade?
No.
How was this patch tested?
Tested on cluster.

In private void cleanupReaders() {} the  rawDataReader is closed, but isRawDataReaderOpen is not set to false.

In private void cleanupReaders() {} the rawDataReader is closed, but isRawDataReaderOpen is not set to false.

i don't think it matters cause we basically destroying OrcEncodedDataReader there

latest comment are addressed in: https://github.com/apache/hive/pull/5534/commits/6da66f618cad2a08e1d3f2e97592f435dd324600
waiting for a green test


Can't resolve parameter set for Get-AzNetworkWatcherPacketCapture

When I try to get certain NetworkWatcherPacketCapture resource I get the following error
PS D:\azure-powershell\src\Network [Az.Network]> Get-AzNetworkWatcherPacketCapture -ResourceGroupName v-anevse -NetworkWatcherName nw -Name pc
Get-AzNetworkWatcherPacketCapture : Parameter set cannot be resolved using the specified named parameters. One or more parameters issued cannot be used together or an insufficient number of parameters were provided.

My guess is that it's because there are two parameter sets with the same required parameters:
GetStatus
Get-AzNetworkWatcherPacketCapture -Name <String> -NetworkWatcherName <String> -ResourceGroupName <String>
 -SubscriptionId <String[]> [-DefaultProfile <PSObject>] [-AsJob] [-NoWait] [-Confirm] [-WhatIf]
 [<CommonParameters>]

Get
Get-AzNetworkWatcherPacketCapture -Name <String> -NetworkWatcherName <String> -ResourceGroupName <String>
 -SubscriptionId <String[]> [-DefaultProfile <PSObject>] [-Confirm] [-WhatIf] [<CommonParameters>]

PowerShell was generated using the following extensions
Loading AutoRest core      'C:\Users\v-anevse\.autorest\@microsoft.azure_autorest-core<EMAIL_ADDRESS>(3.0.5464)
   Installing AutoRest extension<EMAIL_ADDRESS>(beta)
   Installed AutoRest extension<EMAIL_ADDRESS>(beta->2.0.552)
   Installing AutoRest extension<EMAIL_ADDRESS>(beta)
   Installed AutoRest extension<EMAIL_ADDRESS>(beta->2.0.318)
   Installing AutoRest extension<EMAIL_ADDRESS>(beta)
   Installed AutoRest extension<EMAIL_ADDRESS>(beta->2.0.352)

@number213 Good find. That's my fault. I can fix it.

Since the Status gives back a distinctly different object, I'm separating the Status part back into a separate cmdlet, Get-AzNetworkWatcherPacketCaptureStatus.

I've now separated this into Get-AzNetworkWatcherPacketCapture and Get-AzNetworkWatcherPacketCaptureStatus.


New feature that would allow environments to be created with R or Lua instead of Python

Currently, tox and tox-conda assume that they are being used for building and testing Python environments. However, Anaconda also supports R and Lua environments. It would be nice if it was possible to create an environment for those languages instead of Python. Currently, you can still create test environments, but Python always comes along for the ride even if you don't want it (which can sometimes cause dependency issues).
I'm thinking that maybe a language_base= option could be added to the top-level config that gives the base specification; something like python=3.8, r-base=3.4.3, mro-base=3.5.1, etc. That would be used for the initial environment creation rather than always just installing python.
There could be other niceties add too such as cran_deps= or lua_deps= to install from CRAN and LuaRocks repos, but I'd be happy with just being able to create environments for those other languages for now.

Actually, for my current use case, I really just need to specify something like language_base = none so that it creates an empty environment. Then conda_deps= can specify the language base to install.

Currently, tox and tox-conda assume that they are being used for building and testing Python environments.

There's a tox rewrite on-going under https://github.com/tox-dev/tox/tree/rewrite#rewrite-branch, which removes this assumption.


Add new school "Gymnasium Nepomucenum Rietberg"

The domain does not match the home page of the school as we are using a different system/domain for student email addresses. Please contact the administration "Frau Milsch" or "Frau Oesterschling" (see http://www.nepomucenum-rietberg.de/kontakt/) to verify the domain gnr.wwschool.de as student email domain. Thanks in advance.

@Nemobil There is no necessity to email your administration to check the domain validity, because the email is listed at the official website: http://www.nepomucenum-rietberg.de/medienscouts/
Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Configure pflag map/valued flags with option to set : as a delimiter

Currently, pflag requires the format of key1=val1,key2=val2 for map/valued flags.
We prefer to use the : delimiter (as in key1:val1;key2:val2 instead of =.
Would a PR to allow for delimiter customization be useful?

here at MongoDB we would benefit from this change as well