Sequential composition in vertx from list of items

I am trying to execute sequential composition of futures in Vertx Java. I have an array (it could also be a list with unknown number) that I need to loop through and call the db server via reactive mysql (that is on vertx) This is the code:
`
DbTransferRecord[] arrayData = new DbTransferRecord[4];
arrayData[0].setID(1);
arrayData[0].setName("John");
arrayData[0].setLastName("Smith");
arrayData[1].setID(2);
arrayData[1].setName("Johnas");
arrayData[1].setLastName("Smith");
int arrayDataLength = arrayData.length;
for (int i = 0; i < arrayDataLength; i++) {
dbSaveData(arrayData[i].getId(), arrayData[i].getName(). arrayData[i].getLastName());
}
public Future dbSaveData (id, name, lastName) throws Exception{
Promise = Promise.promise();
....
return promise.future();
}
`
I need to chain every call from dbSaveData sequentially,so if one of the calls to db fails, a handler should be called to clean up (rollback) the data. The documentation is very limited. I could not find how to create sequential composition from list, and I could not understand it enough from the information I found on stackoverflow and the internet. Thank you.

Duplicates https://github.com/eclipse-vertx/vert.x/issues/3241


Summary:Tip issue when turn on or off maintainance and there is no ho…

Tip issue when turn on or off maintainance and there is no host in or off maintaince
Description:
Jira: #AMBARI-23558
What changes were proposed in this pull request?
Tip should be ''All hosts that you selected are not in Maintenance Mode. when turning on maintainance and all hosts are not in maintaince.
And Tip should be ''All hosts that you selected are already in Maintenance Mode.' when turn on maintainance and all hosts are in maintaince.
@geksilla @aBabiichuk @Zangr @omalley @adoroszlai @billierinaldi @zjffdu

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2407/
Test FAILed.
Test FAILured.

Closing PR as it is over 6 months old.


Input() text is squished

Current Behavior
Text is squished into a fixed width area:

Weirdly, placeholder text is unaffected (although it is awfully off-centre for 'center' aligned text):

The amount of squash/stretching appears to be linked to the size of the value field.

And finally when no value is provided (same behaviour, width seems to be ~half the width of a single character):

Expected Behavior
Text should appear normally, and not stretched out.
Context
This is the entire code (honestly):
    this.$_nameField = new Input({
      placeholder: 'Enter text',
      bg,
      align: 'center',
      maxLength: 120,
      textStyle: {
        fontFamily: ['Arial', 'sans-serif'],
        fontWeight: 'bold',
        whiteSpace: 'normal',
      },
      value: 'example',
      padding: 8,
    });
    this.$_nameField.zIndex = 1_000;
    this.$rootView.addChild(this.$_nameField);

Is anyone seeing something similar?

entire* code probably should include the bg Graphics object too:
    const borderColor = 0xcccccc;
    const backgroundColor = 0xeeeeee;
    const border = 3;
    const radius = 5;
    const width = 200;
    const height = 50;
    const bg = new Graphics()
      .beginFill(borderColor)
      .drawRoundedRect(0, 0, width, height, radius + border)
      .beginFill(backgroundColor)
      .drawRoundedRect(border, border, width - (border * 2), height - (border * 2), radius);

Hi, @reececomo
Unfortunately I was not able to reproduce this issue. I have also checked all the usages of the pixi Text instance used here, and there are no places in code where size or scale is changing. You can check it here: https://github.com/pixijs/ui/blob/e77935ade04f994581bf4d13a4f36b55f6d180e9/src/Input.ts#L164
this.inputField is only used to get it's size and never set.
So in order to try to help I need either codepen example that reproduces this issue, or link to a repo (maybe if it is opensource)

Yeah all good, I figure we (or a plugin) must be doing something somewhere that's causing this.
Thanks for looking, will close this ticket.


Changes to jQuery api

Caution: breaking changes

i love that we dont have to initialize any of these plugins with js, just with markup. no page glue
this is fucking beautiful

I'd update the issues page so we don't forget the wait for animation and the show/hide classes on components.
other than that I'd say :ship:

looking forward to getting this merged so we can get tabs working on /library

ping @wnadeau and @lifeiscontent - wait for animation + ft-show-class + ft-hide-class are now in, with plugins, and tested. Think this is ready to :ship: ? Should probably bump version after, too, since this changes the data-api

:ship: :ship: :ship:
bump bump bump

do you have gem push rights?


Item Entity not updating light bug

Minecraft Version
1.18.2
Shimmer version
Custom Shimmer Fork of Shimmer-1.18.2-0.4
Modpack info or mod list
N/A
The latest.log file
N/A
optifine , Rubidium , flywheel or any rendering related mods. mod versrion is required.
currently in a dev environment
Issue description
Unless a block is place only the item entities and entities from the resulting glowing pink light from the apple with show.
Steps to reproduce
Throw apple
let it start glowing
place mobs around it and see them pink
Other information

From github -> https://github.com/Mysticpasta1/Shimmer

I hope to PR this as soon as the lighting update bug is gone

update methods tried
level#sendBlockUpdated
levelRenderer#setBlocksDirty

I think I found out how to issue occur LightManager isn't firing on an item entering and leaving the world

nevermind that isn't it

It has nothing to do with MBD, you should learn how Shimmer works first.


Kafka feign death

When I used brooklin, there was a problem with the destination kafka, which was manifested as: after writing data from the source kafka, the destination kafka could not synchronize the data
Here's how I did it:
After creating several data streams, the destination kafka will not receive the data. I need to restart the destination kafka. Do you have any general optimization methods

urgency

@Bruce2jiang  is this resolved? If not, can you be more specific? What do you mean by "destination kafka could not synchronize the data". Did the problem go away after restarting destination kafka?

Closing the issue because of inactivity


updated diskmaker-x (5.0.3)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as 812e621a8f405da792a0bdae66f02cbbbe409740.


[Debug filter] Exclude line number from filtering

#105185
Filter now takes source into account but also line numbers which is unexpected:

Related to #105868

/assign isidorn


input function python tutorial

Populate common-built-in-functions/input.py with a tutorial and challenge.

I have sent in a pr.


Source code for Seek Whence?

I know it says these sources will be coming soon, but is it possible to speed up the process? I'd really like to be able to examine the earlier iterations in the series, to help myself understand the basics and how the pieces came about and fit together.

Thanks for the reminder.  I have had to move and things were stuck in a box.  Since what I have is a hard printed copy of the documents, gracefully sent by Marsha and Daniel Defays, I'll just try to scan them with minimum damage, so they'll be good for study, but they will need at least OCR before trying to run anything.
I'm now traveling, but I MUST get this done in January!  Thanks again!

Haha no problem, thank you. XD

Hallo, any progress on this yet? Also, I'm looking through the repo for Copycat here, and I think it's missing the code to add descriptions to the list of structures in the workspace. I don't know if that one is updated, or relevant - it doesn't have issues enabled, so probably not. But, I thought I'd let you know. :/
https://github.com/fargonauts/copycat

Hm; not yet.  We will be going to the house in which the materials are stored later this month, so I think by mid February we'll have some material posted.  We tried to find ways to scan the hardcopies without destroying them, but the best we could find was still not perfect.
I will post here on this issue, so you may get an alert, ok?

Hi; Numbo is now online, as a pdf only... which is the best we could do for now.  I hope to have a running version of it as soon as possible, but time constraints and all other priorities make it a gambler's bet as to when it will be available.
I have Marsha Meredith's here, and the code is pretty massive, in those old dot-matrix printers, so it should take a while for me to post --- though I do want to do it asap!
Thanks for the patience!  If you're skilled on those things, maybe you could help OCR'ing the Numbo scan, just perhaps?

I have absolutely zero experience with OCR'ing, but I am going to be looking at it and trying to code it into something working, though it'll be in python, not lisp. I'll see about posting it, if I think the code is reasonable? Gonna take a while, though.

That sounds awesome!
On Sat, 29 Feb 2020 at 21:04 Angular-Angel<EMAIL_ADDRESS>wrote:

I have absolutely zero experience with OCR'ing, but I am going to be
looking at it and trying to code it into something working, though it'll be
in python, not lisp. I'll see about posting it, if I think the code is
reasonable? Gonna take a while, though.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Alex-Linhares/FARGonautica/issues/8?email_source=notifications&email_token=AAMQZNAOGSLX4OXSHBYVNRLRFGQ7RA5CNFSM4JZJ3ZM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENMJKXY#issuecomment-593007967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQZNFBEU7X5AJSHK3ZNY3RFGQ7RANCNFSM4JZJ3ZMQ
.

Oh, what about Jumbo? You removed it from the title, but theres no mention of it in discussion, and it's not present in the project. Is it's source simply not available?

Yes, I don't think that is available, sadly.
On Mon, Apr 6, 2020 at 10:04 PM Angular-Angel<EMAIL_ADDRESS>wrote:

Oh, what about Jumbo? You removed it from the title, but theres no mention
of it in discussion, and it's not present in the project. Is it's source
simply not available?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Alex-Linhares/FARGonautica/issues/8#issuecomment-610115231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQZNHHGN4AOQL3ROYAPN3RLJ33FANCNFSM4JZJ3ZMQ
.


Contacts

Replicate Xamarin.Mobile functionality of querying each platforms contacts with a simple cross-platform API.

ITNOA
@jamesmontemagno Why this issue is closed? Did you not want to add ContactsPlugin to popular plugin.


Support a Joystick Hat-switch

Joystick hat switch is not supporting in GLFW-3.0.4 currently.
The hat switch is a popular gamepad feature. For example XBOX360 gamepad has hat switch at the left-side, PS3 controller has it in the same. And any other game controllers has the hat switch feature.
In commonly, say a "supporting a game controllers" then it will support hat switch, digital buttons and analog axes/triggers.
Please plan to support hat switch of joystick.
Related old time discussion?: http://sourceforge.net/p/glfw/discussion/247562/thread/d4f45381/

@elmindreda Hello! :)
I'd like to use gamecontrollerdb.txt, but there are a lot of hat switches. Do you have an idea how one could transform them to axis (or buttons, on OSes where that's the case)?
For example, the line for "Microsoft X-Box 360 pad" on Linux is correct regarding buttons and axis, but then there's this part:
dpleft:h0.8,dpdown:h0.4,dpright:h0.2,dpup:h0.1,
Using GLFW the hat switch / DPAD is reported as two axis:
DPAD_HORIZONTAL_AXIS = 6;
DPAD_VERTICAL_AXIS = 7;

Currently, I'd just use the last two axes for the DPAD but that will likely not work everywhere. (Especially on operating systems where hat switches are buttons.)

@CreamyCookie My plan is to add direct support for that database (#341) along the lines of the SDL2 interface.

Great! I'm looking forward to this. =)
FYI, the issue (#341) you mentioned is already closed. Is there an open report that I could subscribe to?

Oh.. and will this be for 3.3? I'd also be interested in how this would work.
This is how I'm imagining it currently: (in pseude code)
GLFWController c = LoadControllerFromMapping();
if (c.isUpPressed() || !c.wasXPressed()) {
    // ..
}

@elmindreda ?


Improve LYX Transfer guide

What does this PR introduce?
Improve the LYX transfer guide by adding the following content:

[x] add Introduction section to explain the basics of the execute(...) function on ERC725X + how interactions via the Key Manager work.
[x] replace image + link to faucet from L14 to L16
[x] proof-read content + grammar for better understanding and walk through of the guide
[x] add step to load EOA.

Hey @CJ42 sorry for the delay in merging this PR!
Please ping me next time so it doesn't stay open that long ;)


METRON-2168 Elasticsearch Updates Not Tested in Integration Test

The ElasticsearchUpdateIntegrationTest is not testing that the ElasticsearchDao can update and retrieve values in a manner similar to what would occur in production.
What?
Within the Elasticsearch index, the test fails to define the 'guid' field to be of type 'keyword', instead the type is defaulted to 'text'.  In a production setting this mistake would prevent any documents from being found by guid.  Unfortunately, the test passes despite this. The test needs to match the behavior of what a user would experience in production.
Why?
These problems arise because of the way the test is setup.  Instead of directly testing an ElasticsearchDao as you might expect this test runs against a MultiIndexDao initialized with both an ElasticseachDao and an HBaseDao. On retrievals the MultIndexDao will return the document from whichever index responds first.
With the current test setup, the underlying ElasticsearchDao will never retrieve the document that the test case is expected.  In all cases where the test passes, the document is actually being returned from the HBaseDao which is actually just interacting with a mock backend.  The test needs to actually test that we can update and retrieve documents from Elasticsearch.
Proof?
If you alter the test to run against just an ElasticsearchDao the test will fail as follows in the attached log file.
Testing
This only changes the integration tests.  Run the integration tests to validate these changes.
Pull Request Checklist

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?
[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
[x] Have you included steps or a guide to how the change may be verified and tested manually?
[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
[x] Have you written or updated unit tests and or integration tests to verify your changes?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

One point that reviewers should consider is do we have enough test coverage for the MultiIndexDao now?
I don't think the MultiIndexDao warrants an integration test.  It doesn't integrate with any external systems.  I think it can be tested sufficiently with unit testing.  But is what we have in  MultiIndexDaoTest enough?

I added more unit tests for MultiIndexDao.

lgtm, +1


Wrong line numbers in stack traces when using Node v20

Problem
I was writing a unit test, and noticed that stack traces have the wrong line numbers when using Node v20:
Switch to Node v20, run the test:
$ nvm use 20
$ node --loader tsx --no-warnings --test index.test.ts
  Error: is the line number right?
      at SomeLib.doSomething (file:///Users/smart/thesmart/repro-tsx-line-no/lib.ts:1:149)
      at TestContext.<anonymous> (file:///Users/smart/thesmart/repro-tsx-line-no/index.test.ts:1:121)

Switch to node v18, run the test:
$ nvm use 18
$ node --loader tsx --no-warnings --test index.test.ts
 Error: is the line number right?
     at SomeLib.doSomething (/Users/smart/thesmart/repro-tsx-line-no/lib.ts:3:11)
     at TestContext.<anonymous> (/Users/smart/thesmart/repro-tsx-line-no/index.test.ts:7:5)

The line numbers in v20 are wrong.
Expected behavior
I would expected Node v20 to work the same as Node v18 and to have the correct line numbers to match where the exception was thrown.
Minimal reproduction URL
https://github.com/thesmart/repro-tsx-line-no
Version
v3.12.7
Node.js version
v20.5.1
Package manager
yarn
Operating system
macOS
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

tsx hasn't changed, so sounds like a regression in Node.js? Can you file a bug report there?
FYI the reproduction can be more minimal because it's specifically related to how Node.js handles source maps—unrelated to the test runner or TypeScript.

Filed an issue in Node.js: https://github.com/nodejs/node/issues/49455

Should be fixed now. Get the update with npm update @esbuild-kit/esm-loader

wow, this was just a tiny bug but it makes such a big difference to the dev experience. Thanks for fixing!

It seems like this regressed with v3.14.0. Using Node.js 20 the line numbers look right with v3.13.0 but not with v3.14.0 any more.

@privatenumber Could you please reopen this issue or should I open a new one?

Unless the repro above is broken with the latest tsx, can you open a new issue with reproduction?

You are right, the repro above seems to be fine with v4.2.0 on Node.js 20. Very similar issue that I'm seeing with our code, though. I'll figure out a new repro and open a new issue.

I've filed #404.


[feat] Implement P2MS tests / features

Bitcoin has many different transaction types, one of them being P2MS.
In order to support these, we will need to make sure all edge cases and some mainnet P2MS transactions can successfully be validated by Shinigami.
The learnmeabitcoin link can be super helpful in understanding Bitcoin P2MS transactions in depth.
Then, using the learnmeabitcoin P2MS examples and existing Shinigami transaction tests, create a set of tests to ensure all P2MS transactions run properly.
Resources
learnmeabitcoin P2MS
Existing P2PK & P2PKH tests in Shinigami

I

@b-j-roberts am still available to work on this

@b-j-roberts May I work on this?

Hi @b-j-roberts, can i work on this?


Started UMatter Github Discussions!

Head over to discussions to connect with other fellow contributors, ask any questions or doubts you may have, and have some open-source tech talks!

I create many of the issues but why are you not assign me ...

Issues have not yet been assigned.

Please have some patience!

okk

can you help me . i could not understand that my changes were rejected or not and if not how would i came to know of it . what are checks in vercel ..

can you help me . i could not understand that my changes were rejected or not and if not how would i came to know of it . what are checks in vercel ..

Don't worry! Ignore that!


How can I get the internal exons in detected circRNA.

Dear Dr. Christina Huan Shi,
Thank you for your great work. This software is very helpful and easy to use.
We got a series of output files in our study as follows:

To obtain more detailed circrna information such as circRNA strand, the internal exons, the spilce signal, and so on,  we extract the exons which located in first-last exons' region as circexons. The coordinate  of first-last exons were been fetched from the bsj id of circRNA (candidate_circ_junctions.bed), and the strand  of circRNA was feteched according to Zma mays reference bed file.
However, comparing with the sequence in the file 'candidate_circ_junctions.fa'  I got different sequence.
Could you please tell me how can I get more detail information about circRNAs? I' ll appreciate your help vey much.
Yours sincerely,
Peng Liu.

Hi Peng,
The outputs in your screenshot seemed to have generated too fast and the file sizes look small - this might be due to the same issue another user reported. If you haven't already, can you please try the run again with "kallisto" being the official 0.43.1 version, i.e. https://github.com/pachterlab/kallisto/releases/download/v0.43.1/kallisto_linux-v0.43.1.tar.gz, and then see if the results are the same?
To extract the backsplice donor and acceptor exons, you can consider the _EwBx suffix information in candidate_circ_junctions.bed, column 4. The number after E is the donor exon number and the number after B is the acceptor exon number. Exon numbers are counted from left to right for + strand transcripts (i.e. exon 1 is left-most in terms of genome coordinates) and counted from right to left for - strand transcripts (i.e. exon 1 is right-most in coordinates).
The sequence of these two exons will also be found in the beginning and end parts of the corresponding fasta entry (transcriptID;geneName_EwBx) in full_length_isoforms.fa.
Let me know whether this makes sense or if you have further questions.
Regards,
Ken

Hi Peng,
Since you haven't posted any further questions, so I will close this issue. Hope everything goes well. Let me know whenever you have one, I will reopen this.
Best,
Christina


JSDisconnectedException in published project

Good afternoon!
Today I made some first experiments with this awesome project (nuget package). Thanks a lot for that.
Unfortunately, I cannot use it at all in "published" server-side projects.
(Debugging mode works without any issues.)
As soon, as I request a page containing the control, the following exception is thrown:
fail: Microsoft.AspNetCore.Components.Server.Circuits.CircuitHost[111]
      Unhandled exception in circuit '4u9IgHDmxj1lynbcrvGjWBZE0endyewjrgqRO-pfngQ'.
      Microsoft.JSInterop.JSDisconnectedException: JavaScript interop calls cannot be issued at this time. This is because the circuit has disconnected and is being disposed.
         at Microsoft.AspNetCore.Components.Server.Circuits.RemoteJSRuntime.BeginInvokeJS(Int64 asyncHandle, String identifier, String argsJson, JSCallResultType resultType, Int64 targetInstanceId)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, CancellationToken cancellationToken, Object[] args)
         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
         at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
         at BlazorBarcodeScanner.ZXing.JS.BarcodeReaderInterop.StopDecoding()
         at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
         at Microsoft.AspNetCore.Components.Rendering.RendererSynchronizationContext.ExecuteSynchronously(TaskCompletionSource`1 completion, SendOrPostCallback d, Object state)
         at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
      --- End of stack trace from previous location ---
         at Microsoft.AspNetCore.Components.Rendering.RendererSynchronizationContext.ExecuteBackground(WorkItem item)

I can "delay" the exception by making the control rendering conditional after pressing some button, but as soon as it is rendered, the application crashes and I cannot interact anymore.
Googling around did not provide any useful information so far, but what made me suspicious is the fact that the stack trace mentions "StopDecoding" although this happens directly after opening the page.
Feel free to ask for further details if needed.
Best regards!

Can you give a sample repo to show how you set it up, so we can try to reproduce it.

I will provide a sample project this evening. Thank you for your quick reply.Message ID: @.***>

I fixed it for you: https://github.com/sabitertan/BlazorBarcodeScanner/pull/47
Have a nice evening! :-)


Weaver and custom serializers for classes

I'm unsure whether this counts as a feature request or if I misunderstood the new Weaver but here it goes:
I recently updated Mirror to version 32.1.4 (from a much older version, probably from around mar/2020). Suddenly, a number of errors popped up about the custom types I have. They worked fine in the past, and they all have custom Read/Writes.
The first errors I got were these:

I found the issue strange, as both of the classes being complained about have custom Read/Writes. After asking around in the help channel of Discord, I was made aware that Weaver now expects only structs, and part of the reason for that was to discourage inexperienced developers from abusing of reference-types, which generate allocations. While this is a valid reason, I believe it greatly limits the potential of custom serializers.
I'm working on a rather complex Action/RTS game, and have been using custom serializers in many different convenient ways, which, as far as I understand it, are no longer supported since Weaver only expects structs. Below I will list some common use-cases on my project:
Serializing NetworkBehaviours
You can order groups around in the game. The order itself is an object (ScriptableObject to be exact). Here's a simplified example:
public abstract class AttackOrder : GroupOrder
{
        Group subject; // The group responsible for the order
        Group target;  // The target group for the attack

        //etc...
}

Note that the Group class is a NetworkBehaviour. The attack order has a subject Group (who will carry out the order), and a target Group (who will be attacked by the subject). Serializing this order was rather simple: I wrote custom readers/writers for the Group class:
 public static void WriteGroup(this NetworkWriter writer, Group group)
        {
            writer.WriteInt32(group.ID);
        }

        public static Group ReadGroup(this NetworkReader reader)
        {
            var __id = reader.ReadInt32();
            return StrategySerializationHelper.Instance.GetGroup(__id);
        }

As you can see, I serialize a NetworkBehaviour quite easily, and cheaply (an int32). It's very convenient to send an order to the server and execute it. The Groups come as direct references, not as a byte id or a struct GroupData that needs to be de-serialized into a Group reference by some other means. This method of serializing not-easily-serializable objects has been very convenient for the structure of the game code. I understand I could be sending the NetworkIdentity instead, and then running a GetComponent, but that was easily avoidable in earlier versions.
Interfaces:
Another use-case I have is serializing interfaces. The battles in my game have a replay system. The whole replay of a battle boils down to a struct:
    public struct BattleReplayData
    {
        public float endTime;
        public float startTime;
        public IActorReplayData[] actors;

        public BattleReplayData(float startTime, float endTime, IActorReplayData[] actors)
        {
            this.startTime = startTime;
            this.endTime = endTime;
            this.actors = actors;
        }
    }

As you can see, the struct contains an array of IActorReplayData (which is an interface). I wrote a custom serializer for that interface, and that allows me to almost-seamlessly add new types of actors to my replay system. I'm unsure if the new Weaver covers this use-case, but it complained when I tried:

Type Objects
If you're not familiar with the term, it's is a programming pattern where you basically use a class (or a class instance) as a type. I use it in my game for some things, but the example I'll use is Factions. The player can choose between 6 factions in the game, each with its own models, names and other particulars. In the very beginning, they were an Enum (easy to serialize). Later on, we decided to change a Faction into a ScriptableObject, where it could hold references to things (such as unit models, names, and other various things). I have a custom writer for a Faction:
        public static void WriteFactionType(this NetworkWriter writer, FactionType type)
        {
          writer.WriteByte(type.ID);
        }

        public static FactionType ReadFactionType(this NetworkReader reader)
        {
            var __id = reader.ReadByte();
            return FactionSerializationHelper.Instance.GetFactionTypeByID(__id);
        }

Using a simple byte, I'm able to serialize the reference to a FactionType. This allows me to use FactionType as a syncVar, for example. Whenever I create a Group, a Missile or a Planet in my game, I can just grab the right models/names/colors for them because FactionType is their SyncVar. Again, incredibly convenient thanks to the custom serializers.
To sum up
This may well be a misundestanding on my part (please let me know if it is), but in case it isn't:
I've been abusing the custom serialization system of Mirror to get a massive advantage in the structure of the code. The convenience of just receiving a NetworkBehaviour reference as a parameter to a Cmd is awesome. The flexibility of serializing interfaces is great too, and I'd hate to get stuck at a previous version of the plugin for it. I've been using that in my game for the past year, and it's worked beautifully so far.
As I understand it, simplifying Weaver has made the code more manageable for the team, and it also stops beginners from using the wrong data types, generating unnecessary amounts of garbage, but is it worth it? I'd be very reluctant to give up on the methods I'm using above if I could.

hey.  sorry didn’t have time to go through this yet. will do soon.

Just wrote up a quick test on a fresh project using the latest version from the package manager (40.0.9), and everything works as expected. I can write custom serializers for my NetworkBehaviours, interfaces and custom classes (without needing to supply a default constructor). Haven't tried it in my big project again. Will report back when I do, but this makes me think that my complaint was not necessary. Maybe I just came across some bugs while updating the plugin.

closing becuase according to previous message, it doesn't happen anymore


add sample code in README.md

It is difficult to understand that there is no use cases

I would not accept pull-request.
It is because sample code should be executable.
That is, it should include package main, func main etc.. like examples of logrus.
Additionally, "Such an example might produce a log record like this" looks curious to me.
So, I made another pull-request for example codes.
Thank you for your advice.


docs: add new cache method

Documentation update for new caching algorithm introduced in EgoVenture.

Thanks! All suggested changes have been applied.,


Add --use-vcs and --no-use-vcs flags for build commands

From the discussion on #522, this adds an option to not use the VCS to get the list of files to include in the sdist. So flit build --no-use-vcs is equivalent to python -m build with flit_core as a backend. The idea is that in a future version (4.0?), this will become the default, and you will need to use --use-vcs to explicitly choose the sdist-files-from-vcs behaviour.
I've also tried to improve the documentation about what gets included in the sdist by default.

+1 from me. The docs changes look good, and the plan for switching the default seems reasonable. I can't really comment on the code so I'll leave that for others.
One thing that I'd like which is missing, is a clear description of what files will be put in the wheel. Is it just "the stuff in the sdist that's from the package directory"? Having it explicitly stated would avoid any guessing.

@Carreau @pradyunsg I'm planning to merge this in a couple more days, so if you're not happy with this direction, now's the time to say so. :slightly_smiling_face:


[BUG] Cannot join server

Describe the bug
Cannot join a friend's server.
Desktop:

OS: Windows 10
Minecraft Version 1.18.2
Ngrok-LAN Version 1.4.0

Additional context
Tried to join a friend's server, got "Connection refused: no further information". Others had managed to connect but for some reason I couldn't.
Error log:

[03:56:02] [Render thread/INFO]: Connecting to 6.tcp.ngrok.io, 16435
[03:56:04] [Server Connector #1/ERROR]: Couldn't connect to server
io.netty.channel.AbstractChannel$AnnotatedConnectException: Connection refused: no further information: 6.tcp.ngrok.io/<IP_ADDRESS>:16435
Caused by: java.net.ConnectException: Connection refused: no further information
at sun.nio.ch.Net.pollConnect(Native Method) ~[?:?]
at sun.nio.ch.Net.pollConnectNow(Net.java:672) ~[?:?]
at sun.nio.ch.SocketChannelImpl.finishConnect(SocketChannelImpl.java:946) ~[?:?]
at io.netty.channel.socket.nio.NioSocketChannel.doFinishConnect(NioSocketChannel.java:330) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.channel.nio.AbstractNioChannel$AbstractNioUnsafe.finishConnect(AbstractNioChannel.java:334) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:707) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:655) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:581) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:493) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:986) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74) ~[netty-all-4.1.68.Final.jar:4.1.68.Final]
at java.lang.Thread.run(Thread.java:833) ~[?:?]
[03:56:25] [Render thread/INFO]: Stopping!

Thank you for the bug report, but this is likely not an issue with our mod.
Only the host needs to install the mod, and the server connection from the clients is the same as a regular vanilla server connection. It's unlikely that our mod is causing the issue.
We would still like to help resolve the issue, but to fully understand what is going on it would be useful to test these things first:

Obtain the minecraft logs from the server host to see why you cannot connect.
Have another friend host an Ngrok LAN world and connect to their world.
Used a cracked version of Minecraft or another account to connect.
Join the server by yourself (no other connected clients) as you may be reaching Ngrok's free account connection limit.

If you still cannot connect to any Ngrok LAN worlds, please let us know.
I will close this issue for now, but feel free to reopen it if you deem necessary.


Update rubocop, support files to new ext std

Update rubocop and other support files to match the new standard laid
out by solidus_dev_support and implement in upstream master.

This is now working and completes in less than an hour in CI.


Split cluster into chromosome-specific

Hi, @maickrau
I am using ribotin for a diploid plant genome, three chromosomes rDNA cluster was mixed together after the run after verkko. Since we didn't the specific unit of rDNA (without prior information), I use srf to assemble high freq kmer and get one 9kb unit which only distributed 3 chromosomes and the size was reasonable.
But the ribotin only gives a big cluster, could you give me some clues on how to find the chromosome-specific rDNA? I think in human 5 rDNA loci should have similar issues, right?

~/software/ribotin/bin/ribotin-verkko -i ../test --mbg ~/miniconda3/envs/verkko1.3/bin/MBG -o ./test_ribo --guess-clusters-using-reference test.rDNA.srf.fa

Since the rDNA node is very similar, how could we know which node should belong to which chroms? For manual mode, I think this is the required information

We can't know for sure. The semi-separated parts might be separate chromosomes, but it's not certain since they could also be heterozygosity within a chromosome. From the bandage plot it looks like there might be 3-5 tangles. I've circled the parts which might be either separate chromosomes or variants within chromosomes. It's also a bit unclear and subjective where the nodes between the tangles should be assigned. If you only include the nodes which are clearly inside one tangle and ignore the nodes between the tangles, you'll probably get reasonable consensuses of some of the major rDNA units, but it's not certain whether they really are chromosome specific.

Thanks for your kind and clear explanation, it helped me a lot!


Distance-based classification methods

It seems that in practice the most popular methods for time series classification are distance-based, typically using 1-NN with Euclidian or dynamic time warping distance. These don't exactly fit into our machine learning approach, but perhaps it would be useful to add these types of methods for the purpose of reproducing other studies / demonstrating improvements over these baselines?

We've had a few false starts on DTW, which may be a good place to begin:

https://gist.github.com/38e0bd9d41f95b5c3b43
https://github.com/scikit-image/scikit-image/pull/518/files


Backward incompatibility in API functions in 0.0.6

The latest updates to the API functions are affecting tskit development tests here.
tskit uses the following API functions in lshmm:

forwards
backwards
viterbi
path_ll

The positional arguments have changed, because the updated API functions now take num_alleles, which is needed to scale mutation rates based on the number of distinct alleles per site.
The keyword arguments have changed. In particular, p_mutation is now prob_mutation, and scale_mutation_based_on_n_alleles is now scale_mutation_rate.
Also, path_ll has been renamed to path_loglik.
Modifications should be made to the lshmm API functions and the tskit test suite to resolve this issue.

We don't need to update this repo I think, just make the necessary changes to the tskit tests.

The num_alleles was causing problems in the tskit tests. It is no longer an argument in the API functions. But ploidy is now a required argument in the API functions, and the reference panel is a set of reference haplotypes (not genotypes as before).

I'll update the tskit test suite to use 0.0.7 instead.


javascript: main script broken in Safari private browsing mode

In safaris private browser local storage isn't working, which results in the following error breaking our main script:
[Error] SyntaxError: Unexpected identifier 'h'
    (anonymous function) (prozent:421)
[Error] QuotaExceededError (DOM Exception 22): The quota has been exceeded.
    setItem (scripts.js:12:14465)
    h (scripts.js:12:14465)
    update (scripts.js:12:16319)
    addCurrent (scripts.js:12:16646)
    c (scripts.js:12:16275)
    (anonymous function) (scripts.js:12:17117)
    execCb (libs.js:9:12212)
    check (libs.js:9:6462)
    enable (libs.js:9:8976)
    enable (libs.js:9:11264)
    (anonymous function) (libs.js:9:8834)
    (anonymous function) (libs.js:9:716)
    each (libs.js:9:238)
    enable (libs.js:9:8424)
    enable (libs.js:9:11264)
    (anonymous function) (libs.js:9:8834)
    (anonymous function) (libs.js:9:716)
    each (libs.js:9:238)
    enable (libs.js:9:8424)
    enable (libs.js:9:11264)
    (anonymous function) (libs.js:9:8834)
    (anonymous function) (libs.js:9:716)
    each (libs.js:9:238)
    enable (libs.js:9:8424)
    init (libs.js:9:5567)
    (anonymous function) (libs.js:9:10667)

The only usage of setItem is here, so this should be the breaking line...

Safari 11 doesn't have this issue anymore


Problem creating new poject in VS2015

Hi,
I have just downloaded the template and installed it in VS 2015 Community Edition.
Following the Developer Guide and try to create a new Project with the Serene Template I get the error: "A problem was encountered creating sub project "MovieTutorial.web".
Any clues why?

Do you have any errors in VS console, log etc. except this message? I also use that edition.

Sorry for my late response ... Just tried to install it at home on VS2015 Enterprise and all is working. Must be the computer at work.
I'll try on my Laptop and see what will happen.


drop cython language_level on v3.0.0

📰 Custom Issue

Once cython 3.0.0 lands:

drop use of the cython language_level compiler directive (3str will be the default behaviour)

#358

add a min pin for cython>=3.0 for conda and PyPI deps
address warning: #warning "Using deprecated NumPy API, disable it with " "#define NPY_NO_DEPRECATED_API NPY_1_7_API_VERSION" [-Wcpp], see

https://cython.readthedocs.io/en/latest/src/userguide/migrating_to_cy30.html?highlight=NPY_NO_DEPRECATED_API#numpy-c-api
https://cython.readthedocs.io/en/latest/src/tutorial/numpy.html?highlight=import_array#adding-types

See https://github.com/cython/cython/milestone/58

For avoidance of doubt, the cython 3.0 milestone (linked above) was closed Nov 2023 (!)

Closed by #460


Can I serve a folder that contains subfolders and mp4 files?

Folder:
- media
  - 2024
    - 01
    - 02
      - 01.mp4
      - 02.mp4

I have the same problem.
My security camera will save the video to NFS server in 60s mp4 clips. And the folder looks like:
- camera
  - YYYYMMDD
    - hhmm.mp4

Is it possible to use this folder as a source?
Also, I tried to generate an m3u8 file by a script that refers those file, but it seems that these mp4 files does not have moov record before mdat block so it fails on VLS to play this source. so I think it maybe necessary to generate the m3u8 along with modify the mp4 file by ffmpeg, is it possible?

Outside the scope of this project


simplify build & release actions

ci: simplify build & release process

FIxes #23

It looks like the Required PR check workflow is out of sync, anyone has the permission to change required check?


Use before_action instead of before_filter

We should use before_action instead of before_filter in Rails v4.0 or later.

Every bit helps - thanks for the pull request

Why this PR was rejected?

Closed by accident!

Unfortunately the jruby test failed - do you want to merge in the latest master and recheck?

@yujideveloper @ianheggie
FYI: https://github.com/bundler/bundler/issues/4975

I updated this pr and passed jruby test.


Ability to complete reminders

Closed with https://github.com/interstateone/remind/commit/306674e64365abcd1a45be73fefa9152e3e7814b


Missing Font

I'm missing a font. Which font is in use?
And how do I install it?

type lang en-US and press enter


Adding assertj to test dependencies

Assertj is a great library for writing junit test.

AssertJ seems promising, but has few stars on GitHub. Probably Hamcrest matchers or probably I would introduce Spckframework or JUnit 5 which should be GA for Q1 of 2017 (practically tomorrow). What do you think?

I work for 2 years with assertj now. The collection methods and error checks in java 1.8 are nice. I'll take a look on your Suggestions. My preferred test library's are awaitility, mokito and assertj.

JUnit 5 is java 1.8.
I thing Hamcrest matcher can be use in assertj. They are not so nice to use, because you must know the methods autocompletion struggles with this.
Spockframework locks like python :D not my favorite
https://github.com/google/truth looks a bit like assertj.

I would move to Java 8 for release 0.10, surely version 1.0 is using Java
8, JUnit5 could be a good step  forward.
On Mon, Jan 30, 2017 at 6:24 PM, Andreas Mager<EMAIL_ADDRESS>wrote:

JUnit 5 is java 1.8.
I thing Hamcrest matcher can be use in assertj. They are not so nice to
use, because you must know the methods autocompletion struggles with this.
Spockframework locks like python :D not my favorite
https://github.com/google/truth looks a bit like assertj.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/andsel/moquette/pull/261#issuecomment-276127185, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA_Y0bE3R7TZccQQJDd2vgB80JKYYH3Yks5rXhzFgaJpZM4LvzkZ
.

Really Spock is Groovy ;-)  You convinced me, I'll give a try to AssertJ, if the project in future is abandoned (that's my unique trouble about it) we can switch to Google's Truth or fallback on Hamcrest


[TDP survey] Adds border on mobile progress dropdown

In TDP survey mobile view, the progress section in an expandable. This adds a missing bottom to the bottom.
Checklist

[x] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

I messed up a branch push... pls hold.


Split Sumcheck

Description
Sumcheck

Both parties now return a SumcheckOutput so that the evaluations and opening point do not need to be re-read from the transcript
Remove alpha and  zeta from RelationParameters
Remove constructors in favor of passing arguments directly to the prover/verifier.
Verifier is implemented completely in Sumcheck and no longer depends on SumcheckRound.
Sumcheck and SumcheckRound are static. Parameters are passed explicitly to functions to show what is modified and ensure constness.
renamed fold to partially_evaluate and make all Sumcheck rounds uniform for the prover. (no more special case for the first round)
temporary buffers for partially evaluated polynomials are now created in execute_prover
Added method for evaluating the full Honk identity

Honk Prover/Verifier

Clean up types to make things a bit more generic
Move challenge extraction in ways which will make the transcript work easier

Checklist:

[x] I have reviewed my diff in github, line by line.
[x] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[x] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[x] There are no circuit changes, OR specifications in /markdown/specs have been updated.
[x] There are no circuit changes, OR a cryptographer has been assigned for review.
[x] I've updated any terraform that needs updating (e.g. environment variables) for deployment.
[x] The branch has been rebased against the head of its merge target.
[x] I'm happy for the PR to be merged at the reviewer's next convenience.
[x] New functions, classes, etc. have been documented according to the doxygen comment format. Classes and structs must have @brief describing the intended functionality.
[x] If existing code has been modified, such documentation has been added or updated.

Some of this work was completed in #248. The rest is not currently planned.


Add a specific decoder for map[string][string]

Just an idea I had to save memory allocations via reflect, for this common case.
I looked for a way to specify a decoder like this outside the library, but it seems that Iterator doesn't expose enough.  Also I would guess map[string]string is a pretty common target, so worth adding to the library.
The code itself is patterned after mapDecoder.Decode().

you can add custom decoder using extension: https://github.com/json-iterator/go/blob/master/extension_tests/decoder_test.go
this should not be part of the core.

The problem I had was not with calling RegisterTypeDecoderFunc, but in writing the parser.
More specifically, writing the loop - it has to react to the next character being , or }, but I can't find any exposed function on Iterator that lets me do that.
Can you suggest a way?


Variables for Audio states

Hey everyone,
It would be great to have variables for the fairlight audio inputs to be able to use them with triggers (and for status info on the stream deck)
e.g. $(atem:audio_input_1) returns State on/off for input 1
$(atem:audio_ch_1) returns on/off/AFV for channel 1
Bonus $(atem:gain_input_1) returns db gain (+1.00)

I would also like variables for the input gain.

handled by #200


Searcher and its SearcherGeneration may reference a different set of segments

Bug
With tantivy @ d24f31f965543e7ad300b4ce9965b27aef8f3be1,
ran into a situation where Gen 1 below of the Searcher has an inconsistent set of segment IDs referenced in its SearcherGeneration object.
Gen 0: good
Searcher(
	[
		Seg("2b9d1870"),
		Seg("de008a03"),
		Seg("1443006d"),
		Seg("ce5a247e"),
		Seg("70af8500"),
		Seg("7c542ba6")
	]
)

SearcherGeneration {
	segments: {
		Seg("1443006d"): Some(103364896),
		Seg("2b9d1870"): Some(103364896),
		Seg("70af8500"): Some(103364896),
		Seg("7c542ba6"): None,
		Seg("ce5a247e"): Some(103364896),
		Seg("de008a03"): Some(103364896),
	},
	generation_id: 0
}

Gen 1: bad
Searcher(
	[
		Seg("2b9d1870"),
		Seg("de008a03"),
		Seg("1443006d"),
		Seg("ce5a247e"),
		Seg("70af8500"),
		Seg("68356e6c”),
	]
)

SearcherGeneration {
	segments: {
		Seg("1240c4ed"): None,
		Seg("1443006d"): Some(103367743),
		Seg("2b9d1870"): Some(103367743),
		Seg("5a475796"): None,
		Seg("7c542ba6"): Some(103367743),
		Seg("70af8500"): Some(103367743),
		Seg("ce5a247e"): Some(103367743),
		Seg("de008a03"): Some(103367743)
	},
	generation_id: 1,
}

Reason
Looks like this may have been a regression introduced in 23fe73a6c0a0a9f9b0a263f7fa31b278a9a6101f cc @PSeitz
In between the below calls to open_segment_readers() is potential for a race condition.
https://github.com/quickwit-oss/tantivy/blob/23fe73a6c0a0a9f9b0a263f7fa31b278a9a6101f/src/reader/mod.rs#L239
https://github.com/quickwit-oss/tantivy/blob/23fe73a6c0a0a9f9b0a263f7fa31b278a9a6101f/src/reader/mod.rs#L224
The solution seems to be to only call it once when creating a Searcher, this used to be the case before. And maybe we can add an assertion that the set of SegmentIds is identical.

@shikhar Thanks for the report, I think your analysis is correct. I created a PR, which is a little more minima than yoursl, by passing the segment_readers as argument instead of reloading.

Did you experience a crash and if yes can you post the error message so it's more discoverable if someone else has the same problem?

@PSeitz this was from a panic in our custom Query, no Tantivy-level error to share :-) It was expecting some segment-level state that is warmed at the SegmentId level and did not find it.
Warmer::warm() is provided Searcher, and Warmer::garbage_collect() is provided SearcherGenerations, so this was leading to a segment's state being warmed and then garbage-collected while the Searcher was still around.


flatpak: 1.12.2 -> 1.12.4

Motivation for this change
Fixes CVE-2021-43860 and CVE-2022-21682
Changes:
https://github.com/flatpak/flatpak/releases/tag/1.12.4
https://github.com/flatpak/flatpak/releases/tag/1.12.3
Security advisories:
https://github.com/flatpak/flatpak/security/advisories/GHSA-qpjc-vq3c-572j
https://github.com/flatpak/flatpak/security/advisories/GHSA-8ch7-5j3h-g4fx
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

See also the related PR #157045 (CVE-2022-21682 needs both to be fixed)


Example code from README doesn't compile

// Create subscription using the high level api
Subscription subscriptions = nakadiClient.subscription(applicationName, eventName).subscribe();

// Start streaming, the listen call will block and automatically reconnect on IOException
nakadiClient.stream(subscription)
        .listen(SalesOrderPlaced.class, listener);

Variable should be named subscription instead of subscriptions.

@mkulak Thanks, would you be interested in creating a pull request?


QXlsx as a DLL

We like to use QXlsx as a DLL in our project and found that QXlsx is arranged for a static LIB. By studying the code it occured to us that the macro Q_XLSX_EXPORT is missing in the header files compared to QtXlsx.
Is it possible to change the project, so that it is possible to create DLLs with little effort? Thanks in advance!

Dear @PMime
I intentionally deleted that macro.
Because DLL and SO have complex problems in distributing executables.
Of course, you can put that macro in the class you want to use. (such as doc, worksheet, cell, etc)
Because this project is open source.

What complex problems did you have?
I have several Qt/CMake projects that can be compiled as both things, and people use it without issues on any OS.
Cutelyst for example is even on VCPKG, changing BUILD_SHARED_LIBS to off will build it in static mode, pretty trivial.

Thanks for reporting.
For easy support of iPhone, Android, car software, etc., I was not supporting DLL.
I will register as an enhancement how to support DLL and LIB.

Well as I said you can have both, would you accept PR if I proper turn this into a CMake project?
I'm also working on getting rid of GUIPrivate as that would break things when used as dynamic library

Sorry for the late response.
Please upload a pull request.
This project is an open source project and help is always welcome.

Well as I said you can have both, would you accept PR if I proper turn this into a CMake project?
I'm also working on getting rid of GUIPrivate as that would break things when used as dynamic library

I suggest to reimplement QZipWriter in the project. See #55

yeah, sadly I got too busy with another project didn't manage to make all changes I wanted to, Qt upstream indeed suggests to just copy their implementation.

yeah, sadly I got too busy with another project didn't manage to make all changes I wanted to, Qt upstream indeed suggests to just copy their implementation.

OK. I copy it.

You don't need a macro
Cmake can handle for you in one line
Create dlls on Windows without declspec() using new CMake export all feature
July 24, 2015
https://blog.kitware.com/create-dlls-on-windows-without-declspec-using-new-cmake-export-all-feature/
Implemented here
support shared and static libraries #127

You don't need a macro
Cmake can handle for you in one line
Create dlls on Windows without declspec() using new CMake export all feature
July 24, 2015
https://blog.kitware.com/create-dlls-on-windows-without-declspec-using-new-cmake-export-all-feature/
Implemented here
support shared and static libraries #127

What about private classes that you don't want to export, from the name and a quick read it seems it exports ALL symbols which isn't something desired in most projects

What about private classes that you don't want to export, from the name and a quick read it seems it exports ALL symbols which isn't something desired in most projects

How do you handle that in llinking your static library -which is the only option now?

I simply don't mark the private classes to be exported.

I can understand limiting the number of symbols to prevent linker table overflow in Visual S.
Although not all systems would exceed this
and there are compiler and linker options to mitigate this effect
And dynamic loading "LoadLibrary()" on the DLLs rather than static linking an import library for the dll.
Thus its a judgement call on whether
annotating every large API by hand and then maintaining those annotations as the library evolves
is worth the effort.
One could consider WINDOWS_EXPORT_ALL to be the equivalent of static linking

any symbols in "obj references table" introspected by cmake would ALSO be seen by linker!
while still sharing the library across dlls.

Your question "how to limit export all" invites examining implementation detail.
Implementation in cmake about 80 lines
https://github.com/Kitware/CMake/blob/master/Source/cmGlobalVisualStudioGenerator.cxx#L868
Cmake queries the objs for symbols, writes them to "objects.txt" then combines with module *.def you specify to automate a module definition for shared library.
Exporting from a DLL Using DEF Files
https://docs.microsoft.com/en-us/cpp/build/exporting-from-a-dll-using-def-files?view=msvc-160
Q> you don't want to export
ANS: We could manipulate the def file (or create a different one) before linking to achieve "limiting auto exports".

There have been many good conversations.
I have read the article now.
It's in a place where the internet is not available on weekdays. 😭

Why did you drop modern cmake with exporting cmake config file and possibility to build shared lib https://github.com/QtExcel/QXlsx/commit/dddf63eb48e0a2812d8d559edc515a6e437163b7#diff-f2d2dde2d868d22fc4961cc942d539887b1147df237fe2016f24dd29b48d4daa and replace it with another cmakelists with configs and hardcoded only static lib?

Why did you drop modern cmake with exporting cmake config file and possibility to build shared lib dddf63e#diff-f2d2dde2d868d22fc4961cc942d539887b1147df237fe2016f24dd29b48d4daa and replace it with another cmakelists with configs and hardcoded only static lib?

Dear @Jihadist
I have modified the files to make cmake build on both Qt5 and Qt6.
This is because there is still a lot of code that uses Qt5 in commercial projects.

Sorry for the late reply. I work in a place where there is no internet on weekdays.

Why did you drop modern cmake with exporting cmake config file and possibility to build shared lib dddf63e#diff-f2d2dde2d868d22fc4961cc942d539887b1147df237fe2016f24dd29b48d4daa and replace it with another cmakelists with configs and hardcoded only static lib?

Dear @Jihadist
I have modified the files to make cmake build on both Qt5 and Qt6.
This is because there is still a lot of code that uses Qt5 in commercial projects.

Sorry for the late reply. I work in a place where there is no internet on weekdays.

I use old version of cmake script and it works ok with Qt5. You dropped support for dynamic linkage and correct cmake usage aka FindPackage in new cmake script.
In old script you only two fix two lines for support Qt6
find_package(Qt5 CONFIG REQUIRED COMPONENTS Gui)

and
target_link_libraries(QXlsx PUBLIC Qt5::Gui Qt5::GuiPrivate)

So i dont see any reason to rewrite it whole. I dont know why Qt Company doesnt describe the universal cmake script for Qt5 and Qt6 here https://doc.qt.io/qt-6/cmake-get-started.html but you can find it here https://doc-snapshots.qt.io/qt6-dev/cmake-qt5-and-qt6-compatibility.html#supporting-older-qt-5-versions

I don't know the exact solution right now.
I will keep this issue until a good solution is found.
Thanks for reporting.

@j2doll maybe the good solution is https://github.com/Jihadist/QXlsx/commit/1a078b664dec302677129099b342ae7ca77cc7b9 look please, I reword a bit old CMakeLists to provide compatibility with qt6

@j2doll maybe the good solution is Jihadist@1a078b6 look please, I reword a bit old CMakeLists to provide compatibility with qt6

committed to branch qt6-5.

@j2doll wow, thank you! Is there any info when it will be in release?

Dear @Jihadist
Sorry for the late reply.
I am currently working where there is no internet.
Sooner or later, when I can afford it, I'm trying to organize my preparations for release.

Ok so I finally got some time and created the PR that I wanted to #175.
By default it will keep building as static library, which is somehow convenient as you can then do:
FetchContent_Declare(
  QXlsx
  GIT_REPOSITORY https://github.com/QtExcel/QXlsx.git
  GIT_TAG        tag-or-sha-of-the-merge
  SOURCE_SUBDIR  QXlsx
)
FetchContent_MakeAvailable(QXlsx)

OR when shared built with -DBUILD_SHARED_LIBS=ON
find_package(QXlsx)

After that just link to it:
target_link_libraries(... QXlsx::Core)

We still need to mark which classes to export when using as shared lib, and I'll do that once this PR get's merged.


Machine mac-39 lost connection from the external iPhone device.

Machine mac-39 lost connection from the external iPhone device.

The external connection has been restored by rebooting machine mac-39.


realloc leaks

temporary fix for realloc leaks in nim
workaround for bug https://github.com/nim-lang/Nim/issues/4818

Thanks!


pgadmin4 1.11.0 - Cant login

Describe the bug
Deploy the helm chart with valid secret and email set but not able to log in with those settings
Which chart:
pgadmin4 1.11.0
What happened:
Unable to login using the set user/pass
What you expected to happen:
able to login using the set user/pass
Anything else we need to know:

+1

I've also encountered this problem but I could fix it if you specify an email (must be an email, even tho there is a comment stating it can be nickname too) in env.email and password in env.password. The default values somehow don't seem to work. After that, delete the content in the persistent volume and upgrade the chart (or completely reinstall it). This is important because otherwise it will use the old email and password from the pv. After that it worked for me

This should help https://github.com/rowanruseler/helm-charts/pull/177


Group::save() should not prune empty Groups

Several groups have said they have use cases for empty Groups in a saved data store, as structure for stuff that's not there yet.

done.


Provide apiary.io fetch & publishing

It would be a nice extension feature to implement a new functionality for:

Connect to an existent apiary.io account
Fetch and explore online api
Publish on apiary.io api changes

Hello, and thanks a lot for the issue.
Actually Apiary provides some tools to do the job here.
In particular, you can use apiary client to fetch and publish your API documents. If those are linked with Github, the integration will still happen.
Moreover, if you're using Github integration, you can use the normal Github flow to make sure everything stays up to date
Nevertheless, I still think not everybody is using GitHub, as well less you leave your editor, more happy you are.
This is something I'd might be interested working on, but not in the near future.
Would you be interested to submit a PR, eventually?

@liguori #23

@liguori This has been implemented and published in 0.2.0

I am satisfied about this implementation.
It's a very good wor and it work very well
Congratulation!!!!!!!!!!!

I totally agree with the PS.


Python crashed after importing Tensorflow

System information

OS Platform and Distribution: Windows 10 Enterprise
TensorFlow installed from (source or binary): pip install tensorflow
TensorFlow version: 2.1.0
Python version: 3.6.7
Installed using virtualenv? pip? conda?: Virtual environment created with Conda, tensorflow installed with pip
CUDA/cuDNN version: v10.1 / v7.6.5
GPU model and memory: Nvidia Titan Xp

Describe the problem
Following the instruction provided online, I have successfully installed Tensorflow. To verify my installation, I have tried to import tensorflow in python. However, Python crashed immediately after importing tensorflow library. I have also tried to install through:
pip install tensorflow-gpu==2.1.0
Unfortunately, I experienced the same problem.
Any other info / logs
Before the crashing of Python, such message has been printed:

2020-01-15 11:51:56.544389: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library cudart64_101.dll

Having the same issue in anaconda env. What worked it to
1 conda uninstall tensorflow
2 conda install tensorflow-gpu==2.1.0

I have a similar issue:


When using jquery 1.11, popups do not launch.

When I switched from a local copy of jquery (1) to a newer version (1.11) of jquery, it stopped launching the Popups
I made one change to the sample index.htm code:
switched from:
src="https://code.jquery.com/jquery-1.8.2.min.js">
to:
src="//ajax.googleapis.com/ajax/libs/jquery/1.11.3/jquery.min.js"
Is this a known issue?

Nevermind. Looks like it was the missing HTTPS:\  prefix  (I'm not sure how the site actually works without that, but that fixed it.)


add support for unix domain sockets

Redis supports unix domain sockets, which are slightly faster than a loopback tcp connection. The redis python bindings have direct support for this. So it should be very easy to add.
I propose the following syntax in order to keep the current syntax untouched.
CHANNEL_LAYERS = {
    "default": {
        "BACKEND": "asgi_redis.RedisChannelLayer",
        "CONFIG": {
            "unix": ["/path/to/redis.sock"],
        },
        "ROUTING": "my_project.routing.channel_routing",
    },
}

Have you tried using the URI form and passing a single host of "unix:///path/to/socket"? Connection strings like that get passed down to the python binding directly so if it has a URI form that can be used with no changes.

Using "hosts": ['unix:///tmp/redis.sock'] does work!
I did not try that because there is an extra keyword argument for it, i.e. redis.Redis(unix_socket_path='/tmp/redis.sock').
Thank you. Maybe you should add this example to the documentation.

Agreed, I'll change the ticket to reflect that.


Tenant configure menu does not allow to edit tenants created for organization management

Describe the issue:
Tenants are created to manage the organizations for accomplish the organization management feature. The created tenants does not have an admin user at the stage of their creation and the user who created the tenant will be added as the admin. Therefore in the process of tenant configuration menu of the management console, the operations failed with user not found exception.

How to reproduce:
1 - Create sub organization from the console app.
2 - Login to the management console and click configure tab and go to multi tenancy section to view tenants.
3 - Try to edit the tenant corresponds to the sub organization.
Expected behavior:
The tenant should be edited or it should be hidden from the UI as the tenant is created only to support organization management.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [IS 6.0.0-beta2]
OS: [Mac]
Database: [H2]
Userstore: [H2]

Tracked by https://github.com/wso2/product-is/issues/14259


dataman: split out file, ram, and ram_flash operations

This is a quick pass to split out the file, ram, and flash operations from the single monster dataman.cpp compilation unit.
This might make it easier (for me at least) to contemplate a larger refactor and more typical module class structure.

@dagar We need to find a way to be more responsive on reviews and bring in refactors. We need to stop having close to 200 open PRs. I'm concerned when seeing things like this one stale for over a week. Let's try to only take bites we can chew as a core dev team.

@dagar We need to find a way to be more responsive on reviews and bring in refactors. We need to stop having close to 200 open PRs. I'm concerned when seeing things like this one stale for over a week. Let's try to only take bites we can chew as a core dev team.

I largely agree, but we need to do a better job figuring out what those bites should be (metaphor breaking down?). This was supposed to be a quick incremental refactor on the side during discussion of navigator dataman issues (that haven't been resolved yet). We currently have thousands of unnecessary dataman reads for checking the safe point and fence point entries (https://github.com/PX4/Firmware/pull/13534#issuecomment-557805450). I was considering adding simple caching, but it's a bit difficult to fully understand the current sprawling dataman and I'd rather not pack more complexity into the current mess.
I'll revisit this when I have time to implement a few more steps of an internal redesign (dataman backend class hierarchy).


[EDA]  Lead Conversion Account Naming

Critical Changes
Changes
Issues Closed
[W-9021439]
New Metadata
Classes
ACCT_RenameLeadConvertedAccounts_TDTM
Unpackaged Metadata
Deleted Metadata
Testing Notes

@MatthewBlanski  Couple notes around this PR.  I didn't refactor UTIL_ACCT_Naming.defaultAccountName method only added 'theAccount' parm as we talked because turns out 'switch' statements are not available for record ID's in Apex.  This lessons the chance of breaking any existing code. Also, I wrote a bulk test in ACCT_RenameLeadConvertedAccounts_TEST.bulkConvertLeadAdminRecordType which uses a static 'leadCount' attribute but didn't it call anyplace in the unpackaged because I wasn't sure if we wanted to do this and where.  I should have time to add this and do any additional test methods you might suggest, just wanted to get the base PR to you for now.

@MatthewBlanski I just came across an interesting thing I hadn't noticed before.  The method ACCT_IndividualAccounts_TDTM.handleInsertProcessing has always been renaming Household accounts during lead conversion as it calls the UTIL_ACCT_Naming.updateHHAccountsName method every time.  This does not happen for Administrative accounts which is why we started this whole process.  So now the question becomes, if admins don't select Household accounts to rename, should we remove this logic and use the company name?  Obviously this will add more complexity as we will have to determine lead conversion context from this place.

@MatthewBlanski I just came across an interesting thing I hadn't noticed before. The method ACCT_IndividualAccounts_TDTM.handleInsertProcessing has always been renaming Household accounts during lead conversion as it calls the UTIL_ACCT_Naming.updateHHAccountsName method every time. This does not happen for Administrative accounts which is why we started this whole process. So now the question becomes, if admins don't select Household accounts to rename, should we remove this logic and use the company name? Obviously this will add more complexity as we will have to determine lead conversion context from this place.

@mrrigney  At this juncture in the process, altering any base functionality introduces a risk that I do not wish to introduce. That is further afield than refactoring code. I would rather go to production preserving and appending to existing behavior, even if one setting is slightly inaccurate. I am more comfortable logging a bug and moving on to readdress later.

@MatthewBlanski I just came across an interesting thing I hadn't noticed before. The method ACCT_IndividualAccounts_TDTM.handleInsertProcessing has always been renaming Household accounts during lead conversion as it calls the UTIL_ACCT_Naming.updateHHAccountsName method every time. This does not happen for Administrative accounts which is why we started this whole process. So now the question becomes, if admins don't select Household accounts to rename, should we remove this logic and use the company name? Obviously this will add more complexity as we will have to determine lead conversion context from this place.

@mrrigney At this juncture in the process, altering any base functionality introduces a risk that I do not wish to introduce. That is further afield than refactoring code. I would rather go to production preserving and appending to existing behavior, even if one setting is slightly inaccurate. I am more comfortable logging a bug and moving on to readdress later.

@MatthewBlanski agreed thank you, will just want to document this

@MatthewBlanski I refactored the defaultAccountName method and all of the test classes still pass. There was one question I wasn't sure about, you mentioned 'if the account naming format is not hhNameFormat' What does that mean? The settings will always have a hhNameFormat defined I believe and we know we are on a HH record type, so is there an additional check I am missing? Believe everything is ready for full code review

There is a scenario within the method itself that does not necessitate that we are using the hhNameFormat and the Household Account Record type at the same time.

AC testing has passed. There were some bugs that have been written up to be triaged and fixed in future sprints (or that need more dev research to determine level of effort).
@rhakeshj  and @spinkelman are doing some regression testing, will await their results before merging.

After confirming with Anne, QA Passes - Merging


Couldn't make it work when app is killed [android]

I am working on push notification for my app. I followed instructions from documentation. The app notifies me when app is on background but not when app is killed. Am I missing something?
import GCM from 'react-native-gcm-push-notification'
import Notification from 'react-native-system-notification'

var notification = GCM.popInitialNotification()
if (notification) {
  var info = JSON.parse(notification.info)
  Notification.create({
    subject: info.subject,
    message: info.message
  })
  GCM.stopService()
} else {
  class App extends Component {
    componentDidMount () {
      GCM.addEventListener('register', function (data) {
        if (!data.error) {
          console.log('send gcm token to server', data.registrationToken)
        }
      })
      GCM.addEventListener('notification', function (notification){
        console.log('receive gcm notification', notification)
        var info = JSON.parse(notification.data.info)
        if (!GCM.isInForeground) {
          Notification.create({
            subject: info.subject,
            message: info.message
          })
        }
      })

      GCM.requestPermissions()
    }
    render () {
      return <App />
    }
  }

  AppRegistry.registerComponent('App', () => App)
}

sorry to take it so long to respond. I was working on fcm project.
are you passing notification payload or data payload? If you are passing data payload, gcm will not wake up when app is killed

This is probably the same issue I pointed out in commit comment. popInitialNotification just never returns a notification in Android, It would be nice to find the cause and fix it.

change new GcmPackge() to new GcmPackage(getIntent() in MainActivity.java
this should fix it

Will try. It would be great if this helped.

App shows error when I supply getIntent() into new GcmPackage(). Menu stops working and after a while, app crashes.

It starts fine without getintent().

I don't have clue how. I am not developing in Java, I am just doing PN project in WebStorm.
But here is report from fabric:
Fatal Exception: java.lang.RuntimeException: Failed to initialize bridge
       at com.facebook.react.bridge.CatalystInstanceImpl.<init>(CatalystInstanceImpl.java:103)
       at com.facebook.react.bridge.CatalystInstanceImpl.<init>(CatalystInstanceImpl.java:39)
       at com.facebook.react.bridge.CatalystInstanceImpl$Builder.build(CatalystInstanceImpl.java:551)
       at com.facebook.react.ReactInstanceManagerImpl.createReactContext(ReactInstanceManagerImpl.java:867)
       at com.facebook.react.ReactInstanceManagerImpl.access$700(ReactInstanceManagerImpl.java:102)
       at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(ReactInstanceManagerImpl.java:198)
       at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(ReactInstanceManagerImpl.java:181)
       at android.os.AsyncTask$2.call(AsyncTask.java:287)
       at java.util.concurrent.FutureTask.run(FutureTask.java:234)
       at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
       at java.lang.Thread.run(Thread.java:838)
Caused by java.util.concurrent.ExecutionException: java.lang.NullPointerException
       at com.facebook.react.common.futures.SimpleSettableFuture.get(SimpleSettableFuture.java:68)
       at com.facebook.react.bridge.CatalystInstanceImpl.<init>(CatalystInstanceImpl.java:90)
       at com.facebook.react.bridge.CatalystInstanceImpl.<init>(CatalystInstanceImpl.java:39)
       at com.facebook.react.bridge.CatalystInstanceImpl$Builder.build(CatalystInstanceImpl.java:551)
       at com.facebook.react.ReactInstanceManagerImpl.createReactContext(ReactInstanceManagerImpl.java:867)
       at com.facebook.react.ReactInstanceManagerImpl.access$700(ReactInstanceManagerImpl.java:102)
       at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(ReactInstanceManagerImpl.java:198)
       at com.facebook.react.ReactInstanceManagerImpl$ReactContextInitAsyncTask.doInBackground(ReactInstanceManagerImpl.java:181)
       at android.os.AsyncTask$2.call(AsyncTask.java:287)
       at java.util.concurrent.FutureTask.run(FutureTask.java:234)
       at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
       at java.lang.Thread.run(Thread.java:838)
Caused by java.lang.NullPointerException
       at com.oney.gcm.GcmModule.convertJSON(GcmModule.java:118)
       at com.oney.gcm.GcmModule.getConstants(GcmModule.java:76)
       at com.facebook.react.bridge.BaseJavaModule.writeConstantsField(BaseJavaModule.java:441)
       at com.facebook.react.bridge.NativeModuleRegistry.writeModuleDescriptions(NativeModuleRegistry.java:82)
       at com.facebook.react.bridge.CatalystInstanceImpl.buildModulesConfigJSONProperty(CatalystInstanceImpl.java:368)
       at com.facebook.react.bridge.CatalystInstanceImpl.initializeBridge(CatalystInstanceImpl.java:127)
       at com.facebook.react.bridge.CatalystInstanceImpl.access$200(CatalystInstanceImpl.java:39)
       at com.facebook.react.bridge.CatalystInstanceImpl$1.call(CatalystInstanceImpl.java:96)
       at com.facebook.react.bridge.CatalystInstanceImpl$1.call(CatalystInstanceImpl.java:91)
       at com.facebook.react.bridge.queue.MessageQueueThreadImpl$1.run(MessageQueueThreadImpl.java:73)
       at android.os.Handler.handleCallback(Handler.java:725)
       at android.os.Handler.dispatchMessage(Handler.java:92)
       at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:31)
       at android.os.Looper.loop(Looper.java:153)
       at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:187)
       at java.lang.Thread.run(Thread.java:838)

at com.oney.gcm.GcmModule.convertJSON(GcmModule.java:118)
so here is the issue. seems like something null is passed into the function

try this:
in GcmModule.java file
change Bundle bundle = mIntent.getBundleExtra("bundle"); in line 75 to
Bundle bundle = mIntent.getExtras();

I use Android Studio to debug native code and it is convenient and has better logging

Yeah, that much I could conclude as well. ;-)
This happens at the normal start of the app. IIRC you have added the JSON parsing into Java part and it wasn't there before. I think the problem is that with plain start of the app, there is no initial notification (and therefore, null). Maybe it should be ifed properly to distiguish between normal start and start driven by offline notification arriving.
Working on something else, but will see at it later.

you can use mIntent.getAction() == "android.intent.action.MAIN" to check if it is normal start up

another solution is checking if mIntent.getBundleExtra("bundle") is null. I think this makes more sense here. Can you verify if "click_action" is passed into the bundle when you receive notification?

I changed the appropriate part to:
        if (mIntent != null) {
            Bundle bundle = mIntent.getBundleExtra("bundle");
            if (bundle != null) {
				String bundleString = convertJSON(bundle);
				Log.d(TAG, "bundleString: " + bundleString);
				constants.put("initialNotification", bundleString);
			}
        }

but the problem is that I cannot get any events in the app itself. Because of this in GcmModule constructor:
        if (mIntent == null) {
            listenGcmRegistration();
            listenGcmReceiveNotification();
            getReactApplicationContext().addLifecycleEventListener(this);
        }

I presume it all needs a bit of redesign,

That is, I think every mIntent check should discriminate not between null / non-null case but between "non-null and started because of offline notification" / "other". It is probably possible as I presume intent is different (and hopefully known) for the case of offline notification, and all the rest should be treated as standard startup.

you don't get any events because mIntent.getBundleExtra("bundle") is always null.
Here is working code for my fcm project. firebase cloud messaging is mento replace GCM. any reason you can't upgrade to newer google play service?
Map<String, Object> constants = new HashMap<>();
        if (mIntent != null) {
            if(mIntent.getExtras() != null) {
                Map<String, Object> extra = new HashMap<>();
                Bundle data = mIntent.getExtras();
                Set<String> keysIterator = data.keySet();
                for(String key: keysIterator){
                    extra.put(key, data.get(key));
                }
                constants.put("initialData", extra);
            }
            constants.put("initialAction", mIntent.getAction());
        }
        return constants;

I'm not getting any action because listenGcmRegistration(); was not called because I pass non-null mIntent. That thing needs redesign, isn't it? Now you either give me initialNotification but not giving me registration and subsequent ones, or the other way - I don't get initial one but get all the others.

Are you sure that your above code is able to get both inital notification, device id registration and subsequent push notification while app is live? Because that's what is not working - getting both. Now the if (mIntent == null) I posted above rules one of it out.

Unless I do not understand something and getIntent() returns null in normal appo startup, but it does not seem to be the case.

if you start app normally, getIntent() with return an Intent with getAction() == "android.xxxx.DEFAULT". It will never be null

the code above is just for inital notification. It is RN's getContants method. device id registration and subsequent push notification while app is live is handled correctly by this library.

But again, if I pass getIntent() (which is not null), the registrations is not working. Because of if in the constructor.
Shouldn't both be working at the same time?

oh man, the code needs so much refactoring... Are you sure you can't upgrade to firebase cloud messaging?

(that is, not at the same time, but, pls:

if I pass getIntent(), the registration and notification receiving is not started;
if I start app normally, that means it won't get registration nor notifications,
I would like app, when normally started, to receive registrations and notifications
so, could it be made, somehow, that even I getIntent() is passed, the registration and notification is still received?)

FCM is part of firebase3, which is not working with RN.

What I was suggesting, mIntent test should not check for null, but somehow otherwise to distiguish "started because offline notification" versus "started normally / null passed".

fcm is working for RN. It is GCM with another name. Check https://github.com/evollu/react-native-fcm.
I'm keeping the null check in case someone doesn't need initial notification and initiates the package without getIntent().

That's what I was suggesting, keep the null check but you need add smarter check to it:
Discriminate between two cases:

null or normal start w/ getIntent() (listen to reg and notif)
getIntent w/ offline notification (do not listen)

Now it discriminates between:

null (listen to reg and notif)
normal start w/ getIntent() or getIntent w/ offline notification (do not listen)

The "normal start w/ getIntent()" ending up with "do not listen" is incorrect.
Libin Lu wrote:

fcm is working for RN. It is GCM with another name. Check
https://github.com/evollu/react-native-fcm.
I'm keeping the null check in case someone doesn't need initial
notification and initiates the package without getIntent().
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/evollu/react-native-gcm-push-notification/issues/3#issuecomment-224928125,
or mute the thread
<https://github.com/notifications/unsub
scribe/AANjsVhYOkc224CvhfCjQrmz5Gui8qxaks5qKC5kgaJpZM4IpUDR>.

agreed. let me work on it later today or tomorrow and try to fix the code

BTW thanks for pointing to fcm working in RN, I had the impression it is part of firebase3, which is not RN compatible yet.

if you decide to use FCM, I will drop support for this library. It is just an headache to maintain an outdated library and code not written by me

Will look if it is usable tomorrow.

cool. I will wait for your updates then

Hi, I'm running into the same problem. From what I see in FCM repository, there seems to be a lot of issues to solve still. Is there a partial fix at least for the time being?

issues in FCM is mostly configuration issue and use questions
have you tried it?

not yet. I am going to, I'll keep you posted

From the documentation, it sounds like FCM does not support notifications sent when app is killed : is that really the case? If so, there's no point for me to switch just yet.

it should support as FCM runs on the same framework as GCM
can you share the reference that it doesn't support?

I meant from the react-native-fcm repo :
https://github.com/evollu/react-native-fcm#i-cant-get-notification-when-app-is-killed

oh, if you want to send notification and show banner, you need to use notification payload instead of data payload. It is exact same behavior as GCM.

Ok thanks, I will try this. May I suggest you to create a gitter chat room for this repository ? I'm sure it would make things easier for this type of conversation.

gitter chat created for fcm repo

thanks!


Removed focus-visible polyfill from menu component

@grant-cleary brought up that the focus-visible polyfill is causing performance issues by dispatching too many visibilitychange events. The polyfill was used since old versions of Safari do not support the focus-visible pseudo-selector. As of Safari 15.4 (March 2022 release), focus-visible is now supported.
This PR removes the focus-visible polyfill and replaces it with the pseudo-selector. This will cause focus issues with keyboard navigation on older version of Safari. The number of people affected by this change would be relatively small and will continue to decrease, so we decided that it was worth the tradeoff for performance improvements.

Associated Rally:
https://rally1.rallydev.com/#/?detail=/userstory/643401699741&fdp=true


Encountered some  issues when build with clang13

toolchain:
clang13
cmake 3.24.2
ninja 1.11.0
Most of them are caused by Werror  compile option.
-Wreserved-identifier
-Wunused-but-set-variable
-Wdeprecated-copy
-Wdeprecated-copy-with-dtor
-Wshadow
If we do not want to enable  warning as error on these warnings globally make be it is better to suppress it in warnings.cmake.
If want to  enable it globally, we may need to fix it  or just supress it locallly.

sorry we haven't supported clang 13 yet and it needs some patches from communities. If you have interest, please try it.
https://github.com/ClickHouse/ClickHouse/pull/27714
https://github.com/ClickHouse/ClickHouse/pull/27777
https://github.com/ClickHouse/ClickHouse/pull/28046

sorry we haven't supported clang 13 yet and it needs some patches from communities. If you have interest, please try it.
ClickHouse/ClickHouse#27714 ClickHouse/ClickHouse#27777 ClickHouse/ClickHouse#28046

OK， I'ill Have have a try in my spare time.

sorry we haven't supported clang 13 yet and it needs some patches from communities. If you have interest, please try it.
ClickHouse/ClickHouse#27714
ClickHouse/ClickHouse#27777
ClickHouse/ClickHouse#28046

@hustnn  Do you mind put it into the roadmap disucssion?
If these patches don't have side effect, it could be greate if we can merge these patch into the main branch.


[Issue-1535] TO golang -- Revert change to deal with nil db entries

#1549 was inadvertently merged before discussion could take place on the approach.   This PR reverses that merge until that discussion is complete.

SUCCESS
No test results found.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/616/


Include all package data

Include all package data in the wheel.
$ python setup.py bdist_wheel
...
copying build/lib/jupyter_server/static/favicons/favicon-file.ico -> build/bdist.macosx-10.9-x86_64/wheel/jupyter_server/static/favicons
...
copying build/lib/jupyter_server/i18n/zh_CN/LC_MESSAGES/nbui.po -> build/bdist.macosx-10.9-x86_64/wheel/jupyter_server/i18n/zh_CN/LC_MESSAGES

Published jupyter_server-1.0.0rc10

... and also jupyter_server-1.0.0rc11


Fixed mainnet network config

Where are the current mainnet values coming from? the default durations are expressed in seconds.
Why the defaultDynamicFeeMultiplier is 1000?
http://ngl-dual-101.testnet.symboldev.network:3000/network/properties
Removed hardcoded node list for the old testnet. I think this is a harvesting node list workaround not applied to mainnet?
https://github.com/nemgrouplimited/symbol-desktop-wallet/pull/1390 Part Deux

If we need to keep the static list, we could put it in the NetworkConfig for both mainnet and testnet, Network.ts can do the merging.


Using the DNS Name to find the device

Currently the code looks at the first name on the list sees if it is a match and discards anything else on the list. I would be good if it finds a match on any position on the list to allow multiple devices on the network.

I've just patched in the last commit e169266.
Thanks!


simulation: Hypocycloid Curve

Hypocycloid Curve
https://en.m.wikipedia.org/wiki/Hypocycloid
Category Proposed: Mathematics
Additional points/links:

I would love to work on this but first may I know which software do you prefer?

I am a GSSOC'21 member I would like to work on this issue.


Proposing a new Fluentd maintainer

Which issue(s) this PR fixes:
N/A
What this PR does / why we need it:
According to our governance policy, we need to vote for changing maintainership.
As @daipom contributing to fluentd a lot, so I'll propose @daipom as a new maintainer.
Currently, there are following organizations:

@sonots (ZOZO)
@hatake, @edsiper (Calyptia)
@toru-takahashi (Treasure Data)
@ashie, @kenhys (ClearCode)

And individual developers.

@okkez
@repeatedly
@tagomoris
@fujimotos

(We need to also update affilicated compaty information in MAINTAINERS.md later)
Thus, the total number of organization is 8 and satisfying the majority of 2/3 must be  not less than  6.
So every organization need to show :+1: or :-1:
Docs Changes:
N/A
Release Note:
N/A

Hi @kenhys
I think we need to put a plan in place to incorporate new maintainers, happy to support that and have a voting system. Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

Thanks for proposing me.
I am interested in processing Ruby and log data. I want to learn these things through this product, and I want to leverage the benefits and new features of Ruby for this product.

Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

Sure!
Would it be better to comment on this PR this time? Or is it better to open a new PR?

@hatake, @edsiper (Calyptia)

FYI: My GitHub account name is @cosmo0920 not @hatake...

Thanks for proposing me.
I describe myself below.
Please let me know if any explanations are missing.
Intentions
I deliver Fluentd enterprise services in ClearCode over the past year.
I want to make Fluentd a better software, not only within the framework of our service.
This is also the philosophy of ClearCode. Our goal is to develop and grow free softwares.
Personally, I am interested in processing Ruby and log data.
I want to learn these things through this product, and I want to leverage the benefits and new features of Ruby for this product.
I would be happy to contribute more to Fluentd, a great software that is used a lot in the world.
List of contributions
My contributions to this repository:

I have made 50 commits to this repositories in the past year. This is 2nd highest.

https://github.com/fluent/fluentd/graphs/contributors?from=2022-01-01&to=2023-02-27&type=c
https://github.com/fluent/fluentd/commits?author=daipom

My main PRs:

#3644
#3680
#3649

Fix retry specification with @ashie
I did some reviews and added tests in the following PR.

#3685

#3768

I also fixed ServerEngine to add a new feature to Fluentd.
https://github.com/treasure-data/serverengine/pull/120

#4030
I have recently been working on improving features related to buffer corruption: #3970

#4025
#4027
#4028

I also have been working on improving the initialization process of Fluentd: #4037

#4054
#4064
#4065

I also have been working on supporting Ruby 3.2 with @ashie.

#3842
#3987

My contributions to fluentd-website:

I have made 12 commits in the past year.  This is 3rd highest.

https://github.com/fluent/fluentd-website/graphs/contributors?from=2022-01-01&to=2023-02-27&type=c

My contributions to fluentd-docs-gitbook :

I have made 19 PRs in the past year or so.

https://github.com/fluent/fluentd-docs-gitbook/pulls?q=is%3Apr+author%3Adaipom

I participated in the development of some plugins:

https://github.com/fluent-plugins-nursery/fluent-plugin-sigdump
https://github.com/fluent-plugins-nursery/fluent-plugin-windows-exporter

In addition to these plugins, I also maintain the following plugin.

https://github.com/fluent-plugins-nursery/fluent-plugin-remote_syslog

I want to be able to maintain other plugins etc. little by little.

I think we need to put a plan in place to incorporate new maintainers, happy to support that and have a voting system. Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

It may be better to use a proper voting-system in the future, but anyway, daipom explains his contributions in this PR. it is good enough at the moment.
Current status:

@sonots (ZOZO) N/A
@cosmo0920 , @edsiper (Calyptia) (only @cosmo0920 agreed with proposal)
@toru-takahashi (Treasure Data) N/A
@ashie, @kenhys (ClearCode) :+1: (@ashie, @kenhys agreed with proposal)

And individual developers.

@okkez N/A
@repeatedly N/A
@tagomoris N/A
@fujimotos N/A

Current voting status
Companies:

ZOZO: N/A

@sonots (ZOZO) N/A

Calyptia: :+1:

@cosmo0920: :+1:
@edsiper : N/A
Although @edsiper doesn't vote yet, @cosmo0920 already voted so I treat the organization vote of that Calyptia as one vote.

Treasure Data:

@toru-takahashi: :+1:

ClearCode: :+1:

@ashie: :+1:
@kenhys: :+1:

Individual developers.

@okkez: :+1:
@repeatedly: :+1:
@tagomoris: :+1:
@fujimotos N/A

Result

New maintainers are proposed by an existing maintainer and are elected by a 2/3 majority organization vote.

Now we've got 6/8 vote, so we able to we are able to have @daipom as a new maintainer.
So I'll merge this.

Thanks!
I am honored to join the maintainers.
I’ll keep on trying to make Fluentd and the community better!


Connect to existing lobby on create new lobby

Steps to Reproduce

Run 2 clients
Have client A host a lobby
Have client B create another lobby
See how client B joins client A's lobby instead of creating a different lobby

Prevent client from joining another clients lobby if its the same ip and port but the try catch failed with "Server running on this port already"


boost155: revision to generate 10.11 bottle

@DomT4 I'm assuming there's some reason this is verboten.
CC @MikeMcQuaid

Well, there's no need to force everyone to rebuild the whole formula to just update the bottle. Just stick a comma or a comment in somewhere & then drop this commit when you pull the updated bottle.

Just stick a comma or a comment in somewhere & then drop this commit when you pull the updated bottle.

Actually: just use the Homebrew Testing job on the bot.

Did we fix that in the end? Probably. It was problematic for a while after the core/brew repo split happened.

Did we fix that in the end?

It should work again. If it doesn't, adding --tap homebrew/versions to the arguments for the Jenkins job will make sure none of the involved pieces get confused about the correct tap.

needs to be --tap=homebrew/versions but that seems to work. thanks @UniqMartin

Closed via https://github.com/Homebrew/homebrew-versions/commit/a01ff9d90bada17bb70fe35b51bd1c0f8cb5cbf5.
Surprisingly, I needed to use https://github.com/Homebrew/brew/pull/707 to actually pull the new bottles with
brew pull --bottle --tap=homebrew/versions https://bot.brew.sh/job/Homebrew%20Testing/1035/


feat: add resource status summary for environment

Problem:
need to show resource/service status in the environment list
Related Issue:
#964

How does it look with both resource & service now?

How does it look with both resource & service now?

Should like below, group by ready, transitioning and error


[KKT] Fix symmetric K3 formulation

@frapac

I rebased the PR @frapac

The error in the CI is not related to this PR. Ok to merge.