App crashes upon adding synced-cron

I added synced-cron and my app immediately crashes. Removing the dependency resolves the issue. The stack trace doesn't provide me any clues. Can anyone else make sense of this? Is anyone else experiencing the same thing?
Stack trace:
W20160728-09:14:32.369(-4)? (STDERR)
W20160728-09:14:32.386(-4)? (STDERR) /Users/n8barr/.meteor/packages/meteor-tool/.<IP_ADDRESS>qgf5pl++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20160728-09:14:32.386(-4)? (STDERR) 						throw(ex);
W20160728-09:14:32.386(-4)? (STDERR) 						      ^
W20160728-09:14:32.390(-4)? (STDERR) ReferenceError: Tinytest is not defined
W20160728-09:14:32.390(-4)? (STDERR)     at Template.hello.greeting (packages/percolate_synced-cron.js:388:1)
W20160728-09:14:32.390(-4)? (STDERR)     at packages/percolate_synced-cron.js:606:4
W20160728-09:14:32.391(-4)? (STDERR)     at packages/percolate_synced-cron.js:674:3
W20160728-09:14:32.391(-4)? (STDERR)     at /Users/n8barr/Google Drive/Sites/panac.io/.meteor/local/build/programs/server/boot.js:222:10
W20160728-09:14:32.391(-4)? (STDERR)     at Array.forEach (native)
W20160728-09:14:32.391(-4)? (STDERR)     at Function..each..forEach (/Users/n8barr/.meteor/packages/meteor-tool/.<IP_ADDRESS>qgf5pl++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20160728-09:14:32.391(-4)? (STDERR)     at /Users/n8barr/Google Drive/Sites/panac.io/.meteor/local/build/programs/server/boot.js:117:5
=> Exited with code: 8
=> Your application is crashing. Waiting for file change.
My app's package versions:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I had the same issue when running on Meteor 1.2.1. Upgrading version of meteor fixed it.
Seems like the test file was loading when it shouldn't have been.

Yes, that does appear to be a solution. I updated to Meteor 1.4 and I can add this package without my app crashing.


Deleting links between LINK nodes doesn't activate DEPLOY button

I have a flow with link nodes that are interconnected.
Deleting the virtual links between the link nodes does not activate the DEPLOY button.
NR 0.20.8
Dashboard 2.16.3
node version 10.15.2
npm version 5.8.0
Ubuntu 18.4 LTS
Firefox 69.0.2

This is fixed in 1.0


[master] Bump Mesos to nightly master 2ecbe56

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/9932550e9632e7fbb9a45b217793c7f508f57001...2ecbe56a5b2d7a0cc99e5ac1ce5765815d94678a

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot label Work In Progress

@orsenthil could we close this PR?


[W11H03] fix getmaxFlow

Having different flow rates in a graph like a -> b -> c leads to miscalculated maximum flow because it is an invalid state.
An error of this nature lead to the calculated (and expected) flowrate being 8 instead of 6.

Looks good to me. Aber es ist nicht [W11H03] sondern ein Fix für [W11B01]! :)


Add TypeScript types

Background:

A lot of the community prefers using TypeScript for their apps. Having types would be a great help to them. This can be done without converting the entire library to TypeScript by way of just maintaining a .d.ts file that you ship in the npm package.

This is done in 8cb1e32


Feature/Update-of-HTS-indicators-on-aggregate-tables

I get it so that query was only to fetch for the Positives only, makes sense now , thanks.


Plugin init should not fail if a bootstrap peer is down unless it's the only one

Fixed by https://github.com/ZcashFoundation/dnsseeder/commit/2e9a4e1fd7bffd87a599cff8693df695f80b5e47


Low performance

Hey,
just tried out this library with the react-icons/bs icon pack. Since there are more then 1000 icons, the performance isn't really great.
I would suggest to add pagination or show only a limited amount of icons, so searching is required.
Also searching for icons is a bit slow. Maybe throttling the filter fn would be usefull.
A nice to have: case insensitive searching
Cheers,
Daniel

Thanks @dkirchhof for bringing these up, I agree on all three points and will look to implement them very shortly.

Kalon Robson

I've implemented a performance fix. I would like to do page pagination when I get a chance but for now this should fix any issues.
The search fn takes less than 1ms (~0.6ms avg) to complete (before and after the above patch) I think it was the rendering that made it feel longer, if that's the case the above patch will have resolved this.
Kind Regards,
Kalon Robson

Great. Will try it out tomorrow :)


Feature Request: Hide from page list

As the 'hide from navigation' meta option does not hide a sub-page from a page list, there needs to be an option to hide also there.
I want to prepare a page which should not be visible to visitors until it is finished. I can't take it offline as I already need the URL for testing in other cases.

not totally sure if I understand it correctly, so if you hide a folder, then it does not hide the content-pages inside that folder (sub-pages)? Or is the page not hidden from certain navigations on the page? The hide-feature turned out to be very complex in the background, so I want to avoid changing it too much.
By the way, I work on a pro-content-Feature where content is only accessible for certain user-roles like "members". This might be a solution for your problem and will probably be published with the next release.

Maybe I did something wrong. I tried to reproduce it and now the article does not appear in the page list. So it already works like expected. Sorry for opening this issue.


3.2.4. ChainId Breaking Change

Version 3.2.4 renamed SupportedChainId to ChainId causing projects that import SupportedChainId to break.
I'd suggest adding SupportedChainId as a type alias.

@cbachmeier

Hello @Florian-S-A-W , we've deliberately deprecated SupportedChainId in favor of the more general ChainId. If you need only chains specifically supported by the Uniswap protocol you can use the subset array SUPPORTED_CHAINS and the type based off it, SupportedChainsType. If you need any help refactoring feel free to use this PR where we upgrade this dependency in the Uniswap interface as a reference.

@cbachmeier There are about 10 libraries that depend on the core SDK, including the (basically) deprecated Uniswap V2 SDK that a lot of people are still using. Why not release it as a major version bump that at least people can continue using the libraries that depend on the core-sdk?

Hi @cbachmeier ,
I can understand why these changes where made, but this is still a breaking API change and should therefore require a major version bump as @koraykoska mentions.

@Florian-S-A-W @koraykoska Thank you for the feedback, accordingly we have deprecated 3.2.4 - 3.2.6 and have released a major version 4.0.1.


Forgotten Password results in HTTP 500 error

I created an account by signing in using Twitter.
I then unlinked my Twitter account from the profile and continued to update my profile by setting a gender. The email address was left as the default generated<EMAIL_ADDRESS>one
I then logged out.
I attempted to a do a forgotten password request on<EMAIL_ADDRESS>and got a 500 error possibly because the code wasn't expecting the 535 error from the Twitter API?
Connect
500 Error: Invalid login: 535 Authentication failed: account disabled
   at SMTPConnection._formatError (/var/lib/openshift/577472bf2d52711b0f0000f5/app-root/runtime/repo/node_modules/smtp-connection/lib/smtp-connection.js:484:15)
   at SMTPConnection._actionAUTHComplete (/var/lib/openshift/577472bf2d52711b0f0000f5/app-root/runtime/repo/node_modules/smtp-connection/lib/smtp-connection.js:1069:30)
   at SMTPConnection.<anonymous> (/var/lib/openshift/577472bf2d52711b0f0000f5/app-root/runtime/repo/node_modules/smtp-connection/lib/smtp-connection.js:298:22)
   at SMTPConnection._processResponse (/var/lib/openshift/577472bf2d52711b0f0000f5/app-root/runtime/repo/node_modules/smtp-connection/lib/smtp-connection.js:622:16)
   at SMTPConnection._onData (/var/lib/openshift/577472bf2d52711b0f0000f5/app-root/runtime/repo/node_modules/smtp-connection/lib/smtp-connection.js:449:10)
   at emitOne (events.js:96:13)
   at TLSSocket.emit (events.js:188:7)
   at readableAddChunk (_stream_readable.js:172:18)
   at TLSSocket.Readable.push (_stream_readable.js:130:10)
   at TLSWrap.onread (net.js:542:20)

Would be nice to cleanly handle the situation with an on page message or for security should it say something along the lines of "If your account was found login details will be emailed." message.

@budda It's an unhandled error from Nodemailer. Basically, it is trying to send you an email but my my current Mailgun credentials are suspended, that's why you are seeing that error. If you create and use your own API keys (as you should), the error message will go away.

Ah, i was testing it on your demo domain before doing anything local.


Privacy group removes duplicates correctly

Fix generation of the privacy group so that duplicate public keys are removed when determining the privacy group id. Without this change we get a different privacy group id to what Orion calculates.

No longer needed


Create a new command to show Pull Request details

We have for the moment a issue:show, I think having a pull-request:show is nice too. It's the same logic than for issue :

It display general details / description
A flag --with-comments allow to display comments
[x] Add tests

add description txt with a checkbox for add test

Hello,
I've just added a test to validate command execution. This command will be able to dump PullRequest details like :
Pull Request #10 - Write a behat test to launch strategy by weaverryan [open]
=============================================================================

 Org/Repo:         gushphp/gush
 Link:             https://github.com/gushphp/gush/pull/10
 Labels:           actionable, easy pick
 Milestone:        some_good_stuff
 Assignee:         cordoval
 Source => Target: cordoval/gush#head_ref => gushphp/gush#base_ref

Body
----

 Help me conquer the world. Teach them to use Gush.

This PR is ready for a review.

bump @shulard please rebase so i can review
thanks

I've juste rebased on master.


Integrate microservices in Target list

Services are in repository https://github.com/reconness/reconness

Bug

Define a default color for a Target if the color is empty

Working as expected. There is an open ticket with a suggestion #459


Small changes to "classnames" tip on "Styling and CSS" page

Just two small changes on "Styling and CSS" page:

tip regarding "classnames" has been placed in quote box to visually emphasize it
link to "classnames" package now points to React related section on the page

Looks good, thanks!


photos capture to react page

added the ability to save a picture to the react page, still need to find a save solution


Just fixing a comment typo

Thanks, but the EML standard itself contains this typing error, so we need to keep it -- until we upgrade to the next version of the standard.
I'll leave this open so we don't get more reports.
#1757


HF power meter

can I ask for HF calibration and power setting, in my country legal limit is 1500w. it must wonderful if I program the max limit and when the power pass max limit there is some warning in the screen
thank You,
Anton

Being VHF focused with all my big amps are on 50Mhz or higher, I used the HF position as a way to tell the band decoder to switch everything to the equivalent of OFF.  In my remote transverter/amp setup, they are all outside the house.  The band decoder sets the correct band on the transverter, powers on the correct amp, selects the correct bi-directional coupler SM switches, sets the correct antenna RF Coax switch lineup, and passes through PTT.  HF, or "Band Slot 0" I use to set all that to the safest state (OFF or open).
In theory you can put in cal values and all should work but I likely have some code to zero displays on Band 0.
I do have an external HF amp and would like to revisit how Band 0 is handled to be useful for HF readings.
That said, you can rename any band label.  So you can have several HF bands if you like, or just 1.  If using manual band selection on the meter no problem, it will cycle though all slots and display the name you program in.  IF using band decoder input cable, then you need to send a band value other than all 1s or all 0s.  Slot 0, the "HF" label today is all zeros.
The band decoder can have any input pattern you like.  Can be BCD format, just one of up to 6 input lines, or any random pattern you can send to it.  You could use a SP6T rotary switch and active 1 line of the 6 lines to get 6 bands, all can be renamed HF or more likely 20M, 30M,, 40M, etc.     Using a BCD type coding gets you more bands with fewer input lines.
The point here is if you can control the band input pattern sent to the decoder, you can rename any band slot as you like.
The 2nd ask you made is for a High RF power alert on any band, set per band.  I can see this would be handy.  Adding the new threshold control and alert indicator requires some UI rework to squeeze it in but can be done.   You can skip all of this if you only use the desktop app.  Then all changes are in the Python UI including band button labels.  No Band input cable required.  An easy temporary code workaround is to take over an existing alarm threshold control and alert indicator such as 13VDC and make it a global high power alarm.  Just substitute the Fwd Power value for the 13VDC value and rename the UI labels.
Thank you for the suggestions.  These days I am working mostly on my Teensy 4 SDR but will consider this feature request when I come back around to the RF wattmeter code again.

Will be revisiting the HF support situation.

You make Christmas come earlier this year to me, Thank You

Tonight I made changes to support 21 bands and preprogrammed the names for each 160M though 3cm with default Band input and Port A, B and C output patterns based on my K3 band decoder output.  THE first (index 0) was meant to be a catchall when an undefined input pattern match is encountered.  Probably needs more work.  The default calibration values are just examples.  Each band must be calibrated as usual.   Each time your set RESET_EEPROM =1 it wipes all stored EEPROM values and you must start over.  It is helpful to dump the cal table and copy and paste the table text to a file.  You can put those values in the default band table also.   Be sure to set RESET_EEPROM value back to 0 after the first boot or you will overwrite any new config/cal changes you make!  You need to reset EEPROM when making changes to the Band table size or layout, or want to overwrite the cal/config values with hard coded values.
The HF change was fairly simple, increased the NUM_BANDS and populated the structure with more bands by copying and editing the existing ones.  IN my K3 I changed KIO3 to HF-TRN and wired up DigOUt0 and DigOut1 to use all 6 input pins.  This results in a unique pattern on the input for every bang HF and VHF.  IF frequency comes in serially then that is easier.
The desktop App will need more work.  It has a limited number of buttons for a select number of VHF bands today.  The button labels are hard coded text. In the Python code for each button calls code which sends a band index number associated with that button.  It is not hard to change the button text and band number to other bands.  If your radio is controlling the decoder then the buttons are not required.  You can instead edit the band table structure for each band for the proper values of Band_Input, Port A, B and C outputs and then in the top of the .ino, set RESET_EEPROM to 1 and send up to the Teensy (set for Triple Serial).  Change bands with the radio (or simulate with a switch or wire jumpers on a breadboard) and for each band the Desktop App will display the correct text.  For each band you can then Edit the Config in the UI to calibrate and change any Ports patterns, PTT polarity, etc.
RE: Hi Amplifier Ouput Power Alarm: The Nextion touch screen flags thresholds for Voltage, Temp, and Current.  There is no threshold today for max power.  In a special 1296 Amplifier config I calculate SWR and output a 0 - 4.0VDC signal intended for an amplifier sequencer board HI SWR shutdown protection input.  The same approach could be used for trigger something or even set a port connected to TxInhibit.  Special UI code would need to be added.  This type limit is normally a function of the amplifier if they have a UI. You can add Python UI code to do anything you need.  It is not something I see getting around to anytime soon.  Changing the Nextion screen and related code is tedious.  In the future I would replace the Nextion with a graphics screen like the 4.3" or 7" I use in my Teensy SDR project.

Next steps are to test this for a while and ponder what to do about the band select button limitations.  It is most convenient to use the UI to call up a specific band to enter the config and cal and force a band change in case the serial or wired inputs are not connected or correct yet. As mentioned, you can edit the band table defaults as a workaround.

I created a new Edit Meter Table page and added support in the firmware and Desktop App (only in the Desktop_App_190 version for testing).  Now supports 20 bands 160M though 3cm (10GHz).  Nothing fancy.
The new "Edit Meter Table" page has scrolling list box where you can select any one band and that will be made active allowing you to edit the Decoder IO patterns desired.  This is useful when you first setup a new wired or serial input.  You need a way to make the band current to be able to edit it before you have a wired config defined, a chicken and the egg scenario.
Also the per band Power Cal/SWR calibration fields are only on the "Edit Meter Config" page.  Will update the main Desktop_App.  Note that you must RESET_EEPROM one time for the new band data in EEPROM, the structure has changed so the EEPROM must be initialized.
Any Feedback is welcome.

its very nice feautures I really like to try it with real hardware but using teensy is not a good thing for me.
far better if its use raspberry pi pico 2


Setting the time period shown on status pages

📑 I have found these related issues/pull requests
well I've been researching how to do this for 2 or 3 weeks now and there no option built in kuma for this issue unless i change the source code and this road usually ends with more problems for me.
🏷 Feature Request Type
Status-page
🔖 Feature description
So I have made a list of servers and added them to a status page and I added the URL of the said page on my site for the customers, so they can see the uptime of the servers we are selling them, but little did i know that the time period is 40 mins and there is no built in option for me to change the time period shown in status page e.g. 365 days.
✔ Solution
I don't know much but I think there should be a built in option to set the time period that is shown in status pages.
❓ Alternatives
Or a shorter way would be a way to export the uptimes shown to me to a URL witch I can use on my site. I think it would be easier.
📝 Additional Context
No response

Duplicate of

https://github.com/louislam/uptime-kuma/issues/986

change the source code and this road usually ends with more problems for me

=> Please discuss your approach (what you tried, what did not work) in said issue
Closing as a duplicate


Changelog needs an update

Just noticed your changelog was not updated as part of your most recent release (yesterday). I was able to tell the changes by looking at the Releases page, however.

Thanks for the note! We'll keep that updated in the future https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking/pull/216


Bugfix/fix husky upgrade

What kind of change does this PR introduce?
Follow-up of https://github.com/webpack/webpack/pull/12630.
husky 5 works quite different from the previous version:

Important Husky v5 brings a lot of improvements but is also very different from v4. Git hooks won't work if you only upgrade husky dependency, existing config needs to be migrated too.

And here's the migration guide https://typicode.github.io/husky/#/?id=migrate-from-v4-to-v5.
Or we just downgrade to v4 as it's still available.

Did you add tests for your changes?
No need.

Does this PR introduce a breaking change?
No.

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

ah, that explains why it's no longer working... I already wondered and thought it is only a local problem for me.
Sadly the pinst thing is a no-go, since I want github and npm version to be exactly the same and that would break it. It would also break installing from github directly since the postinstall script is executed.
Could we move husky install into the setup script instead?

There's an issue here https://github.com/typicode/husky/issues/868 regarding your concern of pinst. Maybe we can downgrade to v4 at the moment until they figure out a better way?


38 Add auto-generation of a draft prerelease when a v* tag is pushed

This change allows our pipeline to auto-run when a new tag is pushed,
ensuring that no manual action is needed other than publishing the
release is needed when we are cutting a release. We currently attach
only the release notes and changelog markdown notes to the release but
there may be more changes that we want later.

Issue #38


Update to hub-common 24.0.0

Make sure before you merge to master that you merge master into your branch, then build and run all tests. We've had a ton of API changes this week and I want to make sure we don't miss anything.


Best way to integrate ray as Distributed Backend

I want to use ray as a distributed backend, mainly to make use of their autoscaling capabilities. Since I'd like to upstream my solution afterwards, and also get some feedback, I'd like to check with y'all first.
After reading the docs on both sides and the current implementation of LightningDistributedDataParallel I think there are a two main options:

Follow this guide and implement a tweaked LightningDistributedDataParallel, letting pytorch-lightning handle the training
Use the LightningModules functions to instantiate a TorchTrainer instance and let ray handle the training loops.
Merge pytorch_lightning.Trainer and TorchTrainer?

Currently I'm leaning towards option 1, Option 2 or 3 would give ray.tune's hyperparameter search for free, but it seem like quite a lot work.

Alrighty, after a bunch more reading it turned out tweaking pytorch_lightning.Trainer and creating a ray wrapper was easier than expected. You can see a basic API here, it doesn't yet fit the current pytorch_lightning coding style though (I'm more used to component based development rather than mixins).
The changes can be seen here, basically I added a check for slurm wherever it was making use of a slurm based environment variable, then created a two stage wrapper, a "RemoteRayTrainer" which runs on the remote nodes and sets an "LIGHTNING_X" equivalent wherever there was a "SLURM_X" used before, and   a RayTrainer which spawns the remote workers. I have not tested this on an actual cluster so far though, I'll need to check if it plays nicely when actually going over the network and not only localhost.
One big-ish problem I see is that the decorator didn't like being used dynamically, which means I couldn't automatically set the num_gpus requirement

Hey, this is great! I'd be happy to move this upstream into Ray too, if we agree on a design.
You can use .options() to dynamically set resources for a Ray remote worker:
@ray.remote(num_gpus=1, max_calls=1, num_return_vals=2)
def f():
    return 1, 2
g = f.options(num_gpus=2, max_calls=None)
g.remote() # uses 2 gpus

Reference: https://ray.readthedocs.io/en/latest/package-ref.html#ray.remote
Happy to chat more offline!

this is awesome! would love upstream support for our lightning users!

Ray announced RaySGD
https://medium.com/distributed-computing-with-ray/faster-and-cheaper-pytorch-with-raysgd-a5a44d4fd220
It would be great to have an opportunity to train on preemptible instances.

A small update on this: my forked repo now contains a version of raytrainer that can directly connect to a ray cluster and use both ddp and ddp2 to train a model, I have to see how it interacts with callbacks now

hey @williamFalcon, @Borda, are you all still interested in this? any idea of what's needed to make this happen?
would be more than happy to help out if given some pointers!

hey! thanks for asking. Right now we have a feature freeze for new features until 1.0.0 (coming in the next few months).
We just want to stabilize the package before adding new things :)
We’ll ping you once that’s ready to discuss how to do this.

Hi @williamFalcon and co! Highly interested in this as a Ray user myself. Is there any movement in this direction assuming that version 1.0.0 is arriving soon?

we’re almost done with 1.0. We can look into this afterwards!
thanks for mentioning this.

@amogkam has been working on integrating LightningModules with RaySGD: https://github.com/ray-project/ray/pull/11042. After this is merged into Ray, this will provide a way to use Ray for training with PTL. If you need hparam optimization, take a look at Using PyTorch Lightning with Tune

Hi @williamFalcon, any update on this? We'd be more than happy to explore an integration!


The number of genomes retrieved from the pie charts on Genome Overview page shows different count on the pie chart

For example, this pie chart shows 28 genomes for Diarrhea, but it retrieves 114 genome after clicking on the pie.

Chunhong, can you please check if you still see this problem? If so, can you provide more details / url to reproduce it?
-Maulik

This is still a problem. Here is the example: From organisms list, choose Bartonella, go to Genomes tab, select all genomes (83) from the list. Click on the "GENOMES" icon on the green bar. It'll show you the genome list view from the 83 genomes. Go to the Overview tab, choose Peru from the pie chart (13 genome on the pie chart). The resulting genome list shows 189 genomes.

fixed and deployed on beta.


[ko] Update outdated files in dev-1.23-ko.1 (M38-M42)

Related issue: #30845 (dev-1.23-ko.1)
Related PR: #30851 (dev-1.22-ko.4)

[ ]  M38.  content/en/docs/concepts/workloads/controllers/ttlafterfinished.md  | 22(+S) 29(-)
[ ]  M39.  content/en/docs/concepts/workloads/pods/disruptions.md  | 2(+XS) 2(-)
[ ]  M40.  content/en/docs/concepts/workloads/pods/ephemeral-containers.md  | 1(+XS) 10(-)
[ ]  M41.  content/en/docs/concepts/workloads/pods/pod-lifecycle.md  | 72(+M) 42(-)
[ ]  M42.  content/en/docs/concepts/workloads/pods/pod-topology-spread-constraints.md  | 9(+XS) 5(-)

/language ko

/assign

@yoonian 리뷰 감사합니다. 리뷰 의견을 모두 반영하였습니다. 😊

/lgtm

/approve


Investigate call_raw cycles and notify

I think with_cycles/with_cycles128 is handled by the payment parameter for call_raw/call_raw128, but how do you do a notify with call_raw/call_raw128? I'm not sure this is implemented in Azle either.

I think it's just notify_raw...might be solved then


Barbs - Broken Geometry Widget

To Reproduce:

Spin up Katrina, create a barb renderer, enable it
Reduce the minimum X extent via the geometry to 50% of the domain
The barbs disappear

The BarbRenderer is making calls to Grid::GetValue() that are being returned as missing; but only after we've reduced the minimum X extent past a certain threshold.  In the Katrina dataset, a minimum X value of -580,000 will return valid values.  -570,000 will not.
The missing value is retrieved at BarbRenderer.cpp:514.  This area of the code is a little bit confusing.  The variable named variableData is a vector of Grid objects.  The first three elements are our X, Y, and Z vector variable for the barbs.
One thing I've noticed is that these X, Y, and Z Grid instances are of the type CurvilinearGrid.  Since we're looking at Katrina, I think they should be StructuredGrid instances.
I followed the trace until DataMgr::_get_grid_type, where they're getting assigned as CURVILINEAR, at DataMgr.cpp:3271.  Since I'm not clear on the conditionals that designate the different grid types in this function, I just tried forcing the grid_type to be LAYERED for the sake of testing.  This did not work.


inline dataview query didn't render correctly in reading mode

Expected Behavior
The following snippet should render the actual file name
- [ ] #task `=this.file.name`

Current behaviour
but instead it renders =this.file.name, i.e. it seems the query didn't get executed.
Steps to reproduce
Install minimal theme
create a line as
- [ ] #task `=this.file.name`

Switch to reading mode
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
0.15.9
Tasks Plugin Version
1.15.1
Checks

[ ] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

checked using default theme; same result. as long as there is #task (tag of a task) on the same line, inline dataview expression didn't get evaluated.

Hi, no, it shouldn’t show the file name. That is dataview syntax and It is not something that tasks has any code to render.
Tasks sees the description as the exact literal text that is on the task line. It is not a Tasks bug, it is the intended Tasks behaviour.
What is it you are trying to acihieve? Probably your best bet is to use a template when creating the file to paste in the initial file name. A limitation of that would be that the filename would not be updated if you renamed the file.

The thing that confuses me is that the following can render the file name.
- [ ] `=this.file.name`

Or with any other tag
- [ ] #foobar `=this.file.name`

I assume this means there's some special treatment in terms of rendering here ?

@wangmeng99 Hi, the topic of Tasks' rendering of tasks cam up in another issue recently.
Here is the question, and the following reply is the answer....
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1259#issuecomment-1299715050


Fixed #24971 -- Made startapp generate an apps.py

Adding support for App config on the default app creation.

Could also be mentioned in the 1.9 release notes under "Management Commands". Please squash commits and follow the commit message format when updating, thanks!

@timgraham I will update the PR with your notes. But I think the app config on the init.py will not work in case of nested apps and I need to find a simple way to be able to get the python path or use relative import if supported.

I think the user would need to move the files after running startapp in a case like that, correct? If so, I'd consider it their responsibility to also make the necessary edits as I don't see a way to make it work, but feel free to explore your ideas.

@timgraham I updated the code with your notes thanks.
This patch will work only for a top level app creation, if you create a new project named myproject, a new app named myapp, cd into myapp and run djangoadmin startapp mysubapp, the last command will generate a wrong path both for the __init__.py file default_app_config="mysubapp.apps.MysubappConfig" and also in the apps.py for the AppConfig name which will be name='mysubapp'
I didn't find a way to get the user project root to be abled to extract the right module name myapp.mysubapp
Something like this inside django/core/management/templates.py to pass this on template context but the project_path return the current directory like os.getcwd():
project_path = os.path.abspath(os.path.dirname(__name__))
module_path = top_dir.replace(project_path + os.path.sep, '').replace(os.path.sep, '.')

buildbot, add to whitelist.

merged in e6dd7f995a5a785a5aa12378df209314b1d77d93, thanks!


Feature area: Physics

ODE was investigated and prototyped as a backend physics engine. It works well.
However, API design around a physics feature is difficult. This was postponed to after 1.0

core API checked in: https://github.com/lyte2d/lyte2d/commit/fa611b15c8f3233084e98905879723a48652e26c

With lyte_core. namespace in place, designing APIs like this is now easier. Core API is shaped mostly like the C API and if needed, a nicer lua API is writtenin lyte. namespace.

0.7.11 partially has this now. Some tuning and missing APIs and this should be ready soon.

In addition to known physics API we also need timing needs. Per frame times will be important because most physics engines recommend using fixed time for most stable simulations.
So we need a way to get time (in secs) since the frame has begun. This + "dt" (which is time between previous frame's start time and current frame's start time) could allow an app to calculate full time. (Except when the app is stopped, for debugging or in REPL)


Legacy Portal no longer available

[Enter feedback here]
https://docs.microsoft.com/en-us/power-platform/admin/connect-exchange-online#verify-that-you-have-the-profile-microsoft-exchange-online refers to the "Legacy Web Client" however this is not visible and no obvious way to switch to it.

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

ID: e0d917db-1300-f212-067f-4ac688c18bec
Version Independent ID: d19b118a-eddc-dc4f-75ad-2640f02aaeb4
Content: Connect to Exchange Online - Power Platform
Content Source: power-platform/admin/connect-exchange-online.md
Service: power-platform
Sub-service: admin
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

Thanks for your comment, @jimmyeao. This issue has been assigned to the doc author for review.

@jimmyeao
For apps that have "Web" as the UI type, you'll see "Advanced Settings" when clicking the Settings gear.
This topic shows how to see the UI type.
https://docs.microsoft.com/en-us/power-apps/user/navigation#move-between-apps


Fix gcc build

Scope & Purpose

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.11: (Please link PR)
[ ] Backport for 3.10: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

@MBkkt : fixes the compilation issues with g++-11.x (at least for me).


(PUP-10145) Add :puppet service

Add serialization error and protected methods for serializing/deserializing.
Add a :puppet service to get nodes/catalogs and put facts.

CLA signed by all contributors.

nah, I think it's good

I'll merge once travis & appveyor finish :)


Test the compiled app

By testing the actual compiled app, we can prevent issues with the app not starting because some file depends on something that just happens to be present, but won't be available in the app binary (for instance, the client requiring something from the server codebase).

[x] Write a test to ensure that the app launches correctly
[x] Since master is failing to launch currently, the test should fail
[ ] Catch the alert dialog to report the actual failure in the test output
[ ] Fix the actual problem

Closing in favor of #75, #76, #77, and #78


Updated the (side) nav bar logo to link to the Index

@pavitrabhalla
I just made a single line change on each page to make it so that tapping/clicking on the (side) nav bar logo takes you to the Index. Had Damien's help in verifying this was done correctly.

:+1:


Incorrect type in google.maps.GeocoderResult

I'm not sure if this is the right place to fill this issue since I could not open issues on https://github.com/googlemaps/DefinitelyTyped
I was looking for a simple type for the Geocode API. Right now the API's type would look something like this:
type GoogleGeocodeApiResponse = {
  results: {
    address_components: {
      long_name: string
      short_name: string
      types: string[]
    }[]
    formatted_address: string
    geometry: {
      bounds: Record<string, { lat: number; lng: number }>
      location: { lat: number; lng: number }
      location_type: string
      viewport: Record<string, { lat: number; lng: number }>
    }
    place_id: string
    types: string[]
  }[]
  status: string

The closest thing I found to this type in https://github.com/googlemaps/DefinitelyTyped would be google.maps.GeocoderResult.
For the most part, the type looks good, until you start digging into the nested properties. I didn't check the entire type but I noticed that if you follow the type to: google.maps.GeocoderResult.geometry.location instead of returning lat and lng are number it returns them as function which would not work for a simple use case where you would like to parse an API's response.
If that type is not meant to be used like this, then I guess there are no types for API responses which is also not convenient if you want to integrate directly with the APIs using types.

If you would like to upvote the priority of this issue, please comment below or react with :+1: so we can see what is popular when we triage.
@nbouvrette Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.


Wrong include in LEDMatrixDriver.cpp#L8

The line reads
#include "LEDMatrixDriver.h"
Should be:
#include "LEDMatrixDriver.hpp"

Fixed.


request for rtl layout

i hope to  add rtl layout support for polymer project,there are many guys that are using rtl layut for own production so i want to some rt layout support for polymer project
thanks

This is a work-in-progress for each element in https://github.com/polymerelements


Wrong information for forest functional level

[Enter feedback here]
On https://learn.microsoft.com/en-us/windows-server/identity/ad-ds/active-directory-functional-levels it says Server 2016 introduced PAM and MIM and links to https://learn.microsoft.com/en-us/windows-server/identity/whats-new-active-directory-domain-services#privileged-access-management which says PAM and MIM can be used with Server 2012 R2.

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

ID: 36f4538a-b7ae-fd8d-9337-23dffe93c8f8
Version Independent ID: 0a0a4ea0-8afe-b021-ed19-26e8e720fe8a
Content: What's new in Active Directory Domain Services in Windows Server 2016
Content Source: WindowsServerDocs/identity/whats-new-active-directory-domain-services.md
Service: windows-server
Sub-service: security
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

There are parts that were available in 2012R2 that were enhanced with 2016.

#please-close


use sync methods while updating the cluster

In order to avoid logic duplication we should use the same methods while updating the cluster as we use in sync

Coverage decreased (-0.05%) to 7.84% when pulling cbd4d78f9cd36465953881933f18d48965a67049 on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.05%) to 7.84% when pulling 3c476408863dc6311facb4f0b49a9419e7a27bd8 on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.06%) to 7.835% when pulling 5dff071d1f95c4ce60e9988661f7d6102b39fd2c on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.06%) to 7.833% when pulling d111de228399f163aeec3112c305255fc2fdbc38 on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.08%) to 7.811% when pulling 8605d88f667f32a097c27fe544344ac1efcf782b on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.1%) to 7.79% when pulling f2d722726e6573b800a4e8451b7ea6cf8771d426 on sync into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on master.

Coverage decreased (-0.2%) to 7.636% when pulling 79288bea7c7267f6d3f52970a94359918eb118c3 on sync into ce960e892a97537a0dc4f08679766ff21d3054ef on master.

Coverage decreased (-0.1%) to 7.689% when pulling b652a30b17d5911d9306fd28193fb036e06cfacd on sync into ce960e892a97537a0dc4f08679766ff21d3054ef on master.

👍

Coverage decreased (-0.1%) to 7.694% when pulling b652a30b17d5911d9306fd28193fb036e06cfacd on sync into ce960e892a97537a0dc4f08679766ff21d3054ef on master.

👍


Remove beta from GFW logo

It looks like the "beta" was intended for the logging roads project logo, not GFW. Either way, I'll just remove it.


Create 0000-01-02-KateFerrandino.md

created a branch, created a file and made a commit, and opened a pull request

Yay!

Merge it


R2 compatibility: Header 'x-amz-acl' with value '' not implemented

Attempt to upload any file to Cloudflare R2 storage leads to the following error
botocore.exceptions.ClientError: An error occurred (NotImplemented) when calling the PutObject operation: Header 'x-amz-acl' with value '' not implemented

I believe, that's because s3fs always passes ACL option to botocore methods even when it is just an empty string.
The problem can be reproduced as follows:
with s3fs.open('hello.txt', 'w') as f:
    f.write('Hello!')

I suppose in places like this, if the ACL parameter is empty, it should either have a reasonable default value, or not be passed at all? I see that it is used in quite a number of places. Do you know what AWS S3 assumes if an empty string is passed?

I think your workaround would be to explicitly pass acl="private" (or whichever is best for your use-case).

The same error happens when uploading to StorageGrid.
@martindurant proposed workaround to pass acl='private' works:
with s3fs.open('hello.txt', 'w', acl='private') as f:
    f.write('Hello!')

Would be happy to have "private" become the default and not need any workaround, assuming this is what a blank ACL currently achieves on AWS.

Adding on to this, for workflows such as img2dataset which support many different filesystems (local, S3, gcs, etc.) I had to make some minor changes to the code. It would be really great if private was default, or else for every PUT operation I have to pass in acl=private which makes the code less clean and less agnostic to the filesystem
https://github.com/rom1504/img2dataset/issues/334#issue-1827075905

Would you like to put that suggestion in a PR?

Sure I can take a look this week. Is the best way to set the default to private from ''? I'll have to dig into what AWS does in case

Yes, I think so

Okay that ended up being a lot simpler than I thought it might be. Tests seem to pass locally on my end


Added more badges in docs

Fixes #78
@shreyasbapat

@hrishi32 rebase your branch with einsteinpy master

As pointed out by @ritzvik

It needs a rebase onto the current master.

A suggestion: Make a new branch for every pull request.

@hrishi32 rebase your branch with einsteinpy master

Done. Thank you for pointing it out.

Cool. Thanks for the contribution!


Figure is uncropped when exporting to pdf in MatlabR2014b (with the latest export_fig)

Hi,
First thanks a lot for providing such a wonderful toolbox!
However, here I've found a bug when I switch to Matlab 2014b. In Matlab R2014b, when I was using the toolbox to generate pdf figure as:
export_fig test -pdf -transparent
The margin of the figure is not cropped as desired. Could you have a look at it?
Thanks!
Example code:
figure(1)
plot(1:100,1:100)
xlabel('Frequency (Hz)')
ylabel('attenuation (Neper/m)')
set(gca,'fontsize',16)
export_fig test -pdf -transparent

It looks like you're not using the latest version of export_fig.

I am using MATLAB R2014b with the latest version of export_fig.
I could reproduce, but only after changing the above fontsize from 16 to either 20, 24, 28 or 29. Other problematic values may exist as well.
Here is another plot where I found this issue to arise:
figure
ylabel('$$\sqrt{1+\sqrt{1+\sqrt{1+\sqrt{1+\sqrt{1}}}}}$$','Interpreter','latex')
export_fig  test -pdf

Actually I'm using export_fig_16jan2015 with GS9.15 on 64bit Matlab R2014b. I believe it is the latest version..

I've just committed a fix to the problem - it was caused by the fact that sometimes Matlab's print() function decides to place the %%BoundingBox x y w h section at the end of the EPS file rather than at the top, and this creates havoc... Moving this section to the top of the file fixes everything. Oliver already had code that does this, but the regexp that he used did not catch all of the cases, specifically those that had negative values in x/y/w/h.

Thanks a lot for solving this problem!


[STORM-3826] commons-io 2.11.0

What is the purpose of the change
security issue
How was the change tested
CI build

Two versions of same jar?

@bipinprasad I think the license file is ok now - the build had some failures but I think they are related to travis internet issues


Release v0.24.2

Patch release on main.

Can we add the fix from https://github.com/griptape-ai/griptape/pull/729 into the changelog? Looks like it got missed.

@collindutter yep! That's in the diff.

Ah missed that, nice!


top of tallest stack not visible

How to size the height of the flamegraph so that the highest stack will be visible.

You can find an example on how to adjust the height of each stack item automatically here: http://bl.ocks.org/kjellmf/c0cefee8d9572bb99012

This example demonstrates how to adjust the height of the flame graph automatically based on the deepest stack and the cellHeight attribute: http://bl.ocks.org/kjellmf/c0cefee8d9572bb99012#index.html


IndexError: index -2147483648 is out of bounds for axis 0 with size 64

Hi, thanks for your excellent works! But when I try to train by run "./train.py -d /path/to/dataset -ac /config/arch/squeezesegV2.yaml -l /path/to/log", an IndexError was caught, see:

I have check the variable "proj_y", it does have a large number. I want to know what's the problem? Thanks a lots!

sorry for the late reply. it's hard to debug what it's happening there, but something in the projection does produce a strange y coordinate. I would try to check if all points are in bounds and if you need to modify the field of view (fov) of the sensor. I guess that you try to use your own dataset?

hello, I met the same problem before, -2147483648 occur may due to the NAN value in lidar points, you may try to add below code in line 116 and line 119 in common/laserscan.py
self.points = np.nan_to_num(self.points)
may it help you!

Hi, @Trexzhou , did you sovle your problem? I met the same problem like you, can you help me?
@dl19940602 ,thanks your reply, I tried your method, but it doesn't work

Did anyone solve this issue ? :)


[Dropdown] Dropdown reports wrong value if using 'get value' from onChange

Not a bug, comes from calling invoke on original <select> instead of ui dropdown.


Could you please add note_glide in the dataset

I think note_glide is better if it is done manually through SlurCutter so I have not considered this yet. I don't really have an idea on how to implement it either so I won't put it in for now, sorry!
Maybe OpenVPI's SOME has note_glide support though!


Nearly permanently brick the current session

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
This utilizes ctrl+scroll (font scaling) to make a session permanently(nearly) frozen or corrupt.  Steps to reproduce work most of the time.
Steps to Reproduce

Open about 5-10 files with at least 3000 lines.  The content of the files must have incorrect spelling variations with spell checking enabled. (For some reason, spell checking on some txt files enable upon font scaling and some txt files don't have spell checking at all.)
Select any tab and ctrl+scroll (font scaling) in any direction for like a second to create a lot of scroll events.
Do a save all.
Close and reopen Atom.

Expected behavior: Scale the font size.
Actual behavior: Session immediately freezes upon opening, which may be due to it still trying to resize the tabs.
Reproduces how often: 75%
Versions
atom --version --safe
Atom    : 1.14.3
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
Additional Information

I can't reproduce this using Atom v1.16.0-dev-2e72f39a2 on Mac OS X 10.12.3. I used this script to create ten files of 3,000 lines each and followed your repro steps. There was no freezing.
I'm also not sure what you mean by "resize the tabs". The font size is set when you do the Ctrl+Scroll action. This setting is saved immediately and when Atom next starts, whatever font size you ended on is the font size that everything is drawn with from the beginning ... there's no "resizing" that happens.

@lee-dohm I've updated the steps to produce.

I was also unable to reproduce using the new steps using both Atom v1.16.0-dev-2e72f39a2 and Atom v1.14.3. I updated my script to generate Lorem Ipsum text and customize the file extension to activate the spell-check package. It is definitely sluggish, but at no point does it "brick" the session. Scaling the font also is unnecessary and seems to not contribute to or ameliorate the sluggishness.

@lee-dohm Just tested your script with 1 file of 1000 length, and it has permanently froze my session.  Even reloading Atom leaves it in an unresponsive state.

@ungb Can you reproduce this?

Hi @Xembie, can you try disabling spell-check?
We currently have a issue https://github.com/atom/spell-check/issues/190.  @lee-dohm This only occurs on Windows if the issue is related to the spell-check package.
To disable the package, you can run apm disable spell-check from cmd/powershell.

@ungb Ahh yeah that fixed it.

Thanks @ungb 👍 Closing this as a duplicate of atom/spell-check#190.


fix: Add missing implements into connectivity's package

Signed-off-by: Sunbreak<EMAIL_ADDRESS>fix https://github.com/flutter/flutter/issues/88222
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Duplicated #4242


ice4j has a bug in the role conflict processing code

RFC5242, section 5.2 (determining role) describes the tie-breaker value as
follows:

      To resolve this, each agent MUST select a random number, called
      the tie-breaker, uniformly distributed between 0 and (2**64) - 1
      (that is, a 64-bit positive integer).  This number is used in
      connectivity checks to detect and repair this case, as described
      in Section <IP_ADDRESS>.

The tie-breaker is a 64 bit unsigned number.  Right now, ice4j uses the signed
'long' type to store this value.  Unfortunately, blind comparisons of
tie-breaker values cast as 'long' will fail in cases where a large tie breaker
value with the top bit set to '1' will be interpreted as negative and less than
a lower value.

For an example, try comparing 0xffff_ffff_ffff_ffff (or -1)  to 0 using longs.
For this case, 0xffff_ffff_ffff_ffff should win since it's the greater unsigned
value, but since it will be cast as '-1', it will lose the comparison.

Since java doesn't support unsigned long types, I added a helper function to
compare two long values as if they were unsigned long values and will correct
this bug.

The patch also adds another helper function to set the tieBreaker value in case
a user of the library needs to override the generated tieBreaker value to force
ice4j to assume the ICE-CONTROLLING role.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Apr 2015 at 7:29
Attachments:

ice4j_role_conflict_bug_and_override_tiebreaker.diff

Thanks for looking at this!  What about the API to allow overriding the tie breaker value?


feat(jest-message-util): render codeframe on failure

Summary
I had a bit of time this morning, so tried to throw something together for #3415. This is not ready to merge, but pushing to get some early feedback.

Test plan
Eventually green CI.

Can we get rid of one newline before and after the codeframe?

What would this look like if we replaced the expect(received).toEqual(expected) with the line of code from code frame? I'm assuming what you have here is much better, especially for multiline expects, but I'd be curious if show a single line there could work.

Ohhh, that's an interesting idea! I'll see what I can do (later, out of "free time" this morning :P)

Codecov Report

Merging #5087 into master will decrease coverage by 0.03%.
The diff coverage is 36.36%.

@@            Coverage Diff            @@
##           master   #5087      +/-   ##
=========================================
- Coverage   60.63%   60.6%   -0.04%
=========================================
  Files         201     199       -2
  Lines        6676    6653      -23
  Branches        4       4
=========================================
- Hits         4048    4032      -16
+ Misses       2628    2621       -7

Impacted Files
Coverage Δ

packages/jest-message-util/src/index.js
81.48% <36.36%> (-5.26%)
:arrow_down:

packages/jest-cli/src/constants.js
100% <0%> (ø)
:arrow_up:

packages/jest-config/src/index.js
23.8% <0%> (ø)
:arrow_up:

packages/jest-cli/src/get_no_test_found_failed.js

packages/jest-cli/src/failed_tests_cache.js

packages/jest-jasmine2/src/index.js
5.97% <0%> (+0.33%)
:arrow_up:

packages/jest-cli/src/run_jest.js
53.57% <0%> (+1.02%)
:arrow_up:

packages/jest-cli/src/watch.js
76.11% <0%> (+1.29%)
:arrow_up:

packages/jest-cli/src/lib/update_global_config.js
90.9% <0%> (+3.4%)
:arrow_up:

packages/jest-cli/src/get_no_test_found_message.js
66.66% <0%> (+6.66%)
:arrow_up:

Continue to review full report at Codecov.

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


[SPARK-8683] [BUILD] Depend on mockito-core instead of mockito-all

Spark's tests currently depend on mockito-all, which bundles Hamcrest and Objenesis classes. Instead, it should depend on mockito-core, which declares those libraries as Maven dependencies. This is necessary in order to fix a dependency conflict that leads to a NoSuchMethodError when using certain Hamcrest matchers.
See https://github.com/mockito/mockito/wiki/Declaring-mockito-dependency for more details.

Merged build triggered.

Merged build started.

Test build #35907 has started for   PR 7061 at commit 70eccbe.

Test build #35907 has finished for   PR 7061 at commit 70eccbe.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

+1

Alright, I'm going to merge this now.  I'll backport to branch-1.4 later if it turns out to be necessary for backporting regression tests.


parse the boto.rds.dbinstance.DBInstance object

Pack AWS
Status

bugfix

Description
parse the boto.rds.dbinstance.DBInstance object so that we can retrieve useful data.
Checklist (tick everything that applies)

[x] Metadata: pack.yaml, icon, structure (required for new packs)
[x] Version bump (required for changed packs)
[x] Code linting (required, can be done after the PR checks)
[ ] Tests (not required but really recommended)

Awesome, thanks for the contribution.
Changes look good to me so I will go ahead and merge them into master.
In the future if you get a chance it would also be great if you can add a test case for parsing (yeah, I know we don't have any existing test cases for parsing right now, but we need to start somewhere :)).


[FLINK-13045][table] Move Scala expression DSL to flink-table-api-scala

What is the purpose of the change
This PR move the Scala expression DSL to flink-table-api-scala.
Users of pure table programs should define there imports like:
import org.apache.flink.table.api._

TableEnvironment.create(...)

Users of the DataStream API should define their imports like:
import org.apache.flink.table.api._
import org.apache.flink.table.api.scala._

StreamTableEnvironment.create(...)

This PR did not split the package object org.apache.flink.table.api.scala._ into two parts yet because we want to give users the chance to update their imports.
Brief change log

Updated all Scala implicits
Move the implicits to their final destination

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? docs

Thanks for your thoughts. I think the purpose of this annotation is to mark code where people need to think twice before they change something. Unfortunately is not possible to annotate package objects. So I think having an annotation there does not harm.


Allow presets

For example:
lambo --preset=API FacebookForPets

I think that would read a separate shell script after after.sh in something like ~/.lambo/presets/API.sh.
Other examples:
lambo --preset=fractal FacebookForPets
lambo --preset=API FacebookForPets
lambo --preset=jsend FacebookForPets
lambo --preset=adminCrud FacebookForPets
lambo --preset=spark FacebookForPets

Perhaps enable running of multiple presets:
lambo --preset=adminCrud,mysql

@milose yah, i like that idea

What exactly presets would work ? I dont get it clearly :D

Hi @tucq88;
imagine you might want to run a certain set of commands for every project that was going to be an API
but a different set of commands for every project that was going to have an admin panel
and a different set of commands for each project that was going to use Spark.
Each of those would be a "preset". And like I referenced above, it would likely live in ~/.lambo/presents/:presetname:.sh or something like that.

Hey @mattstauffer , working on this now and have a couple of thoughts.
Will start with naive implementation just accepting any string and matching to a file in ~/.lambo/presets/API.sh following the current structure of option [space] string to keep it consistent.
lambo --preset API would try to match ~/.lambo/presets/API.sh or display warning that the preset does not exist.
For an abbreviation -p is already taken by path, could use -s but none fit great...
I think that will do as a first pass.
What are your thoughts about providing some presets for people like your examples? Should we create a preset folder with presets on the repo that people can download and use? Or just an example in the docs? Any need for a 'create preset' command? I'm thinking not and just keep it simple.
A subsequent update can be to switch all options from command [space] "string" to command="string". I like that format better and is consistent with other Laravel command options.
Next I'm thinking about the use case for the after file, with presets added. Being that it is commands that you want to run on 'every' project, maybe we create a 'default' or 'base' preset? At the very least it aligns the language and makes it more clear what the 'after' file is vs a 'preset'. Thoughts?

Work has started on this in the mes/presets branch as part of the port to PHP.
The API may not be exactly the same but the intention is.


updated the parse.js for faster parsing of data.

This PR increases the performance of the website with less time in the parsing of data.
Please verify the changes.

@manik96m this doesn't seem to provide any discernible performance benefit as such?
Moreover, can you resolve the merge conflicts before going forward?

@someshkar - I have resolved the merge conflicts.
Also, have you checked this issue.


Use idiomatic dict call

Description
Iterating over dictionary keys, using
key in dict is more readable and efficient than key in dict.keys(),
while having the same semantics.
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

/lgtm

/approve


Changing pin to layer

Pin Protocol rebranded to Layer Protocol and requested the logo and links be changed.

@micahalcorn changed the description per your note

added nucypher

Re-approved ✅


Release Google.Cloud.ContactCenterInsights.V1 version 2.7.0

Changes in this release:
New features

Support for generating summaries during conversation analysis (commit c70b046)

Created release for Google.Cloud.ContactCenterInsights.V1-2.7.0


OutOfMemoryException

Thrown on very large json file

Sorry, my bad, I accidentally was using 32bit mode.


Disconnect after goodbye is sent

Currently we don't disconnect once we receive goodbye rpc message as we would close connection before response is sent.
Solution:
"sendResponse" method should resolve promise once response is sent

I also see this error in my environment, can you confirm it's this issue or not @mpetrunic
2020-05-01 12:59:03  [NETWORK]          warn: Failed to send goodbye request. Error: undefined
2020-05-01 12:59:03  [SYNC]             info: initial sync stop
2020-05-01 12:59:03  [NETWORK]       verbose: peer disconnected 16Uiu2HAm6bmYWZLoB52NYw461xS8Yhk1HWtGziNQswCP2Zx8yJz1
(node:85534) UnhandledPromiseRejectionWarning: Error: stream ended before 1 bytes became available
    at /Users/tuyennguyen/Projects/workshop/lodestar/node_modules/it-reader/index.js:37:9
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at /Users/tuyennguyen/Projects/workshop/lodestar/node_modules/it-length-prefixed/src/decode.js:80:20
    at oneChunk (/Users/tuyennguyen/Projects/workshop/lodestar/node_modules/multistream-select/src/multistream.js:12:20)
    at Object.exports.read (/Users/tuyennguyen/Projects/workshop/lodestar/node_modules/multistream-select/src/multistream.js:34:15)
    at module.exports (/Users/tuyennguyen/Projects/workshop/lodestar/node_modules/multistream-select/src/select.js:21:19)
    at ClassIsWrapper.newStream [as _newStream] (/Users/tuyennguyen/Projects/workshop/lodestar/node_modules/libp2p/src/upgrader.js:251:40)
    at ClassIsWrapper.newStream (/Users/tuyennguyen/Projects/workshop/lodestar/node_modules/libp2p-interfaces/src/connection/connection.js:172:34)

Confirm that's the same issue, working on it ...

I think we should ignore this error silently, because that means other side already closed connection. Issue here is that we should disconnect only after we send all goodbye message bytes. But our send response doesn't notify us on that. Solution would be to refactor code to support this but request and response are in different modules because of dependencies so Im not really sure what's the best way to fix this

I see the issue, have to leave it as is right now. The good news is it only happens to goodbye request.

unstale

unstale

https://github.com/ChainSafe/lodestar/pull/2142 would ensure correct behaviour
Goodbyes are sent exclusively with this method ensuring the connection is disconnect after completing the goodbye request or on error
https://github.com/ChainSafe/lodestar/blob/fa5a188e6e58e6279fb86d5aff28e944912d8485/packages/lodestar/src/network/peers/peerManager.ts#L407-L416
When receiving goodbye requests, it ensures that the response handler completes before triggering a disconnection
https://github.com/ChainSafe/lodestar/blob/fa5a188e6e58e6279fb86d5aff28e944912d8485/packages/lodestar/src/network/reqresp/reqResp.ts#L203-L206


Issue with k8s.io/zh/docs/reference/command-line-tools-reference/kubelet/

I install kubelet via binary
when I set the kubelet args but not work
--log-dir=/var/log/k8s
--log-file=kubelet.log
the log still goes into /var/log/messages
my kublet version:
kubelet --version
Kubernetes v1.18.6

@spiritman1990, GitHub is not the right place for support requests.
For English-language help, you could check Stack Overflow
You can also post your question on the Kubernetes Slack or in the Discuss Kubernetes forum.
/triage support

@spiritman1990, GitHub is not the right place for support requests.
For English-language help, you could check Stack Overflow
You can also post your question on the Kubernetes Slack or in the Discuss Kubernetes forum.
/triage support

maybe this is the effectivest way .....

no answer


langchain4j integration

Switch the narration app to use the quarkus-langchain4j extension instead.
Fixes #421

Need to wait for https://github.com/quarkiverse/quarkus-langchain4j/issues/149 and https://github.com/quarkiverse/quarkus-langchain4j/issues/148 to be incorporated into a release before merging this.

Waiting for https://github.com/quarkiverse/quarkus-langchain4j/issues/149 and https://github.com/quarkiverse/quarkus-langchain4j/pull/145 to be released and incorporated into this before merging.

@geoand Would you mind taking a look? You can focus on changes just in the rest-narration app.


Grafana: extend network-related dashboard

ref: https://github.com/kubernetes/perf-tests/issues/615 & https://github.com/kubernetes/perf-tests/issues/597
/assign @mborsz

reg also https://github.com/kubernetes/perf-tests/issues/640

/lgtm

/assign @wojtek-t

/approve


Add pe.number_of_imported_functions and pe.number_of_imported_libraries

Add two variables to the PE module to distinguish between library and function imports.

number_of_imported_functions: the count of imported functions

number_of_imported_libraries: same as number_of_imports but with a more explicit name to avoid confusion

Pretty sure this can be closed as we have pe.number_of_imports and pe.number_of_imported_functions now.


Bunch of smaller fixes

Remove the logconverter since it's no longer used
Fix a null ref exception
Fix the icons in add new item dialog (#1545)

LGTM


Cloudevents dependency and up-l3 modules are now optional features

Re-created PR because of actions pipeline hiccup.

@AnotherDaniel:
Did we punt on testing the various combinations of features for now?

It’s a cargo subcommand: https://github.com/frewsxcv/cargo-all-featuresOn 12. Mar 2024, at 19:37, Pete LeVasseur @.***> wrote:
@PLeVasseur commented on this pull request.
In .github/workflows/build.yaml:

     run: |

     # enable nightly features so that we can also include Doctests

     RUSTC_BOOTSTRAP=1 cargo tarpaulin -o xml -o lcov -o html --doc --tests

     cargo check-all-features

 - name: cargo-test all possible feature combinations

   run: |

     cargo test-all-features

Ohhh nice. Is that this crate?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


Error running electron-builder via TravisCI: Cannot check wine version

Describe the issue / bug.
When running my build through TravisCI, the OSX version runs fine, but the Linux build fails with the following message:
Error: Cannot check wine version: Error: Exit code: ENOENT. spawn wine ENOENT
    at /home/travis/build/klamping/chester-desktop/node_modules/builder-util/src/wine.ts:100:13
    at Generator.throw (<anonymous>)
    at runCallback (timers.js:810:20)
    at tryOnImmediate (timers.js:768:5)
    at processImmediate [as _immediateCallback] (timers.js:745:5)

Full log output here:
https://travis-ci.org/klamping/chester-desktop/jobs/400527733#L793
How can I reproduce this problem?
Set up a basic project via the CLI, then add the project to Github/TravisCI. On the first build, you should see the issues.
My travisci.yml file is a straight copy of what's provided by default.
Tell me about your development environment.

Node version: 8
NPM version:
vue-cli version: 2.9.6
Operating System: trusty

Looks like this was caused by me changing something that I didn't realize i changed.  Disregard.

Looks like this was caused by me changing something that I didn't realize i changed. Disregard.

I have the same problem, but I have no idea to fix it...


Force the refresh of metadata when topic is not present in them

Fix #549

Thanks @lafeuil and sorry for the delay


Cow: Watch ssoidloginraj.com

Cow requests the watch of the watch_keyword ssoidloginraj\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ssoidloginraj\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Unable to Interact with WebView in WinUI 2.6

Describe the bug
Unable to interact with Contents of WebView. (Clicking Links, Scrolling etc.)
This does not exist when downgrading to 2.5.0
Steps to reproduce the bug
This from XAML Controls Gallery
<WebView Source="https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/web-view" />
you won't be able to scroll/click any of the links
Can observe this behavior in the XAML Controls Gallery App itself.
Expected behavior
Can Interact with WebView like 2.5.0
Version Info
NuGet package version:
[Microsoft.UI.Xaml 2.6.0]
Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

19043
Yes

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

Same issue here after upgrading to WinUI2.6.
Can't interact with the webpage with the mouse...
Using Tab to focus on Webview and then PAGEUP & PAGEDOWN key to scroll.
And again clicking on webview, it loses focus as if there's some transparent control above webview.

@ranjeshj and @codendone FYI

Currently blocking upgrade of React Native for Windows to 2.6 - can we get this tagged as RNW too?

we have the same issue here, but when I open it inside a new AppWindow it seems to work

@RosarioPulella did you notice issues with the XAML Editor WebView when you updated our Sample App?

In https://github.com/windows-toolkit/WindowsCommunityToolkit/pull/4092#issuecomment-872620417 @XAML-Knight mentions that the editor does not respond to mouse or keyboard input. Also locally I can see the rendered xaml in the editor but I can't seem to interact with it.

@ranjeshj @codendone I can confirm the XAML Controls Gallery app exhibits this behavior as well. I can tab into the WebView with the keyboard and interact with the page (scroll, navigate links, etc...), but all mouse input is ignored.
This is a blocker for the Windows Community Toolkit as it effects our Sample App experience. It will also prevent XAML Studio from updating to WinUI 2.6 as well.

I'm on Windows build 22000.51, and the WebView can be interacted with normally (XCG v <IP_ADDRESS> and a few other apps I tested with).
https://user-images.githubusercontent.com/59544401/124644245-e706ed80-de46-11eb-8c8a-47ba7b1538a3.mp4

I am on 19043.1081, XCG <IP_ADDRESS> and I cannot interact with the WebView

I cannot replicate this issue in an empty project yet I encounter the same problem with XCG <IP_ADDRESS>. I'm on build 19042.1052.
https://user-images.githubusercontent.com/31434093/124655053-2ee04180-de54-11eb-9d10-818a81f6db25.mp4

@huynhsontung interesting, I just created a new project as well (min 17763, target 19041) on my 19043.1052 windows version and it worked fine. Just added the WinUI 2.6 NuGet and resources, and just added a WebView within the Grid of the MainPage...
If I instead put the WebView in a new page, and add a NavigationView to my main page with a Frame:
    <muxc:NavigationView x:Name="nvSample" SelectionChanged="nvSample_SelectionChanged">
        <muxc:NavigationView.MenuItems>
            <muxc:NavigationViewItem Content="Menu Item1" Icon="Play" Tag="SamplePage1" />
            <muxc:NavigationViewItem Content="Menu Item2" Icon="Save" Tag="SamplePage2" />
        </muxc:NavigationView.MenuItems>
        <Frame x:Name="contentFrame" />
    </muxc:NavigationView>

        private void nvSample_SelectionChanged(Microsoft.UI.Xaml.Controls.NavigationView sender, Microsoft.UI.Xaml.Controls.NavigationViewSelectionChangedEventArgs args)
        {
            if (args.SelectedItemContainer.Tag.ToString() == "SamplePage2")
            {
                contentFrame.Navigate(typeof(WebPage));
            }
        }

And then re-run the app, I can click on the 2nd item, and enter the invalid state where the WebView won't respond to mouse input.

Looks like the rounding of the navigation view content area is causing this issue in certain OS versions.
As a workaround, setting NavigationViewContentGridCornerRadius to 0 might mitigate this issue.
https://docs.microsoft.com/en-us/windows/apps/design/style/mica#standard-pattern-in-top-navigationview

I would highly appreciate if this could be fixed soon.
Same bug occurs when adding a WebView into a ContentDialog.

Is there any update on this issue on Windows 10?  Using InkCanvas in several places, including the main posture of the app.  Is there another workaround than removing round corners in ContentDialogs and main root containers of InkCanvas elements?

From ranjeshj's comment, I couldn't tell if we knew that this wouldn't occur in WebView2 as well. If it's limited to version 2.6 or WebView1, I'll close it, I just didn't want to do that without giving someone the opportunity tell me that's not the case.

@krschau this is still an issue and not just related to WebView1 see #5855, #6021, and #6427. I don't think it should be closed and I don't think it's resolved. Please sync with @StephenLPeters and @ranjeshj on this.
I don't know if it also impacts WebView2 or not. My general understanding is that it's an issue in the OS that is the root cause with how rounded corners impact input events in these scenarios.

There are also devices that don't have WebView2 today (like Hub) so their only option is WebView1, not fixing this means those devices have no web view control available.

When will you fix this issue? WinUI 2.8 still has this issue!
This is a blocking issue: WinUI2 is unusable since the 2.6 version if your app uses a WebView control.
I forced to use the WinUI 2.5 version because of this issue. Highly waiting for the fix!

I'm having the same problem with WebView2 which, obviously, can't be used concurrently with WinUI 2.5.

I found how to fix it! You can't set BorderBush and CornerRadius in same time. Or the WebView will can't be clicked. You can just remove the BorderBush, then you can use WebView with CornerRadius now.

Hi,
I am also facing the same issue when I upgrade from 2.5 version to 2.8.2.
I can interact with WebView2 but not with WebView.
And the previous workarounds on SplitView root element in NavigationView or with Resources override doesn't works.
My app is also published on Xbox so I cannot replace all of WebView by WebView2...
Have-you some news ?
I can reproduce this issue on Windows 11

Just wanted to add context for closure - this is in the original WebView control, and unfortunately we are unable to investigate most WinUI2-only issues due to the current focus on WinUI3 (more on the policy here).

@Marim99
You can set the corner radius in the page resource where your NavigationView is present.
<Page.Resources>
    <CornerRadius x:Key="NavigationViewContentGridCornerRadius">0</CornerRadius>
</Page.Resources>

This should fix the issue.


Support for WiFi/Networking

This adds:

CYW43 gem (added functionality)

Connect to WiFi access points
This is only enabled when USE_WIFI is defined

Net gem (new)

DNS
Basic TCP Client with TLS
Basic HTTP Client
details in https://speakerdeck.com/sylph01/adding-security-to-microcontroller-ruby

Change to build configs

BLE only Pico W build, WiFi only Pico W build, BLE+WiFi Pico W build
This resulted in renaming build config files

Note on 5db596a and 559d241:

Pico SDK uses a sub-version of Mbed TLS 2
Originally PicoRuby's Mbed TLS gem used Mbed TLS 3
This caused R2P2 to run Mbed TLS 2 using Mbed TLS 3 defintions, causing SHA256 function to use implementation of SHA1 instead
As the Mbed TLS gem does not depend on Mbed TLS 3 functionality, I decided to revert to Mbed TLS 2, specifically the version that Pico SDK uses.

As far as conflicting files go, I was able to merge the wlan branch onto the current master with a simple git merge wlan, so the conflict message is showing up because of GitHub's conflict resolution strategy, and my observation is that there is nothing that needs to be changed.


:fire: added the hamburger menu button with onclick cancel animation

I raised an issue regarding adding a new button to the existing project and getting assigned. Here is the new hamburger button with animation on click. After clicking it shapes as a cross button.
@Spyware007
Hope you will merge it thank you.
I am gssoc23 contributor.

Hi, there seems to be a merge conflict, please resolve it

Mam actually these conflicts are coming because new pr are accepted then code in the main branch also gets changed then conflicts are coming.
By the time I am raising the pr there is no conflicts.
But I am resolving them mam.


fix(pacmak): go local replace do not respect --outdir in some cases

We first look for the local module under <outdir> and only then the default dist directory of the dependency but since we don't break the search after we found it, the dist will override the outdir location.
Adds a break from the inner loop if we found a replacement.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 0a21130044f6e1fd5ec72e647e9e16a48be1ded1
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 0eae80b3322bf9c7f3ef2a873ac46c1df0def821
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: d6b6bd6840d0bdc75c4bfae6cead27cda62535f4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 354d49288893c7e998a51387f80af7cbd1edc1c4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


fix: unnecessary scrollbars

Hey 👋🏻 , I noticed this trivial issue while running the app for the first time. I didn't know where to contact if it was worth solving, so I went ahead and created a PR.


I Pull requests but Refuse

What did you do?
I Pull requests Chinese readme，but Refuse For “The base branch restricts merging to authorized users.You’re not authorized to merge this pull request.”
What did you expect to happen?
Pull requests success

@DaivePan only maintainers of this repo are allowed to merge pull requests.

@DaivePan only maintainers of this repo are allowed to merge pull requests.

How to be a maintainer ? I want to be a maintainer.

Thanks for your contribution. Don't worry, your pull request #4633 has been successfully submitted, and it will be merged or rejected after reviewing by maintainers. Now, you just need to wait for the review.

Thanks for your contribution. Don't worry, your pull request #4633 has been successfully submitted, and it will be merged or rejected after reviewing by maintainers. Now, you just need to wait for the review.

Thanks

As explained, this isn't an open repo, so PRs must be reviewed.


Add unit tests for Felix

This constitutes a reasonable level of cover for Felix (86% is being reported to me), though requires some work to get it cleaned up and to 100% coverage (notably in test_felix, which is just too long but still does not have enough coverage).
The biggest issue with it is that where I have stubbed out code, I have often used global variables to track system state. That's ugly, but has no major downsides for now, so I proposing leaving as is for now (sorting it out would require some code refactoring, but is not a huge job).
Sample test output is the following.
C:\work\git\calico>nosetests --with-coverage 2>&1 | more
...............'iptables' is not recognized as an internal or external command, operable program or batch file.
'iptables' is not recognized as an internal or external command, operable program or batch file.
'ip6tables' is not recognized as an internal or external command, operable program or batch file.
'iptables' is not recognized as an internal or external command, operable program or batch file.
'ip6tables' is not recognized as an internal or external command, operable program or batch file.
.'iptables' is not recognized as an internal or external command, operable program or batch file.
..'iptables' is not recognized as an internal or external command, operable program or batch file.
...........
Name                    Stmts   Miss  Cover   Missing
-----------------------------------------------------
calico.felix                0      0   100%
calico.felix.config        76      0   100%
calico.felix.devices       40      0   100%
calico.felix.endpoint      85     18    79%   46, 130-132, 147-148, 159, 163-168, 173-175, 178-181, 209-214
calico.felix.felix        291     88    70%   173-176, 195, 220-227, 246-250, 262-267, 306-308, 338, 351-387, 402-404, 428-431, 448-452, 457-458, 481, 499-503,
508, 558-561, 564-565, 568-569, 572-574, 627-649, 657-658, 661-662, 668, 680, 701, 706-728, 731
calico.felix.frules       342     21    94%   103, 133, 412, 584, 603, 623-626,637-640, 648, 657-675
calico.felix.fsocket      105     10    90%   78-80, 122-124, 144-146, 163
calico.felix.futils        36      0   100%
calico.felix.ipsets        25      0   100%
-----------------------------------------------------
TOTAL                    1000    137    86%
----------------------------------------------------------------------
Ran 29 tests in 0.224s

OK

The tedious complaints about iptables not being present are benign, and are due to the hack in place to deal with issue #35, which hopefully we can retire shortly.

I think we should leave as is for now, and resolve when we resolve #35 - that hopefully can be soon (i.e. this week).
From: Cory Benfield<EMAIL_ADDRESS>Sent: 18 December 2014 08:47
To: Metaswitch/calico
Cc: Peter White
Subject: Re: [calico] Add unit tests for Felix (#69)
Before I even start reviewing, I disagree with your assessment that those warnings are benign. On a Windows system, sure, but on a Linux system those subprocess calls may succeed and potentially throw you down a different code path. That leads to test behaviour that is different on different platforms, and I think that's a bad idea.
We should either remove the fix to #35https://github.com/Metaswitch/calico/issues/35 or patch out the subprocess calls.
—
Reply to this email directly or view it on GitHubhttps://github.com/Metaswitch/calico/pull/69#issuecomment-67456932.

One small note.

Alright, I'm happy. Let's do this.


GetPlanetByCoord

Hello,
Am I doing something wrong, or does GetPlanetByCoord not return any data ?
Really basic but;
mp := bot.GetPlanets()
fmt.Println(mp)
p, err := bot.GetPlanetByCoord(ogame.Coordinate{Galaxy: int(1), System: int(207), Position: int(10)})
fmt.Println(p)
"fmt.Println(mp)" outputs my planets, so it's logged in and targetting the right universe, but the "fmt.Println(p)" outputs the below.
{  0  0 [0:0:0:0] {0 0} {0 0} }
But a planet exists in 1,207,10 in the galaxy.
I also tried the HTTP services and that also seems to find no planets but other functions work fine.

The documentation could be clearer.
GetPlanetByCoord is to get one of your planet, using it's coordinates.
You cannot get someone else planet info using this function.
On Sun, Nov 18, 2018 at 6:05 AM Nathan Flynn<EMAIL_ADDRESS>wrote:

Hello,
Am I doing something wrong, or does GetPlanetByCoord not return any data ?
Really basic but;
mp := bot.GetPlanets()
fmt.Println(mp)
p, err := bot.GetPlanetByCoord(ogame.Coordinate{Galaxy: int(1), System:
int(207), Position: int(10)})
fmt.Println(p)
"fmt.Println(mp)" outputs my planets, so it's logged in and targetting the
right universe, but the "fmt.Println(p)" outputs the below.
{ 0 0 [0:0:0:0] {0 0} {0 0} }
But a planet exists in 1,207,10 in the galaxy.
I also tried the HTTP services and that also seems to find no planets but
other functions work fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alaingilbert/ogame/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2Hm0KcAf1kvb-DAnlcz0wOLrFwDhOwks5uwWkYgaJpZM4Yn9KW
.

Ah, it all started from;
curl -v <IP_ADDRESS>:8080/bot/galaxy-infos/1/120
{"Status":"ok","Code":200,"Message":"","Result":{}}%
Adding debug to the services I see;
{1 120 [   0xc00035e210 0xc00035e2c0 0xc0001ec000 0xc0001ec0b0 0xc0001ec160  0xc0001ec210  0xc0001ec2c0   ]}
So I guess it's the services not returning things correctly as it finds galaxy-infos, I'll er go see if I can debug it.

Oh, interesting, I'll give a look

I'm curious what version of ogamed are you using ?
ogamed -v

master branch, compiled it myself to try and debug it.
➜  ogamed git:(master) ✗ ./main -v
ogamed version 0.0.0
Also just tried the binary edition;
➜  ogamed git:(master) ✗ ./ogamed --version
ogamed version 7.2.6
➜  ogamed git:(master) ✗ curl -v <IP_ADDRESS>:8080/bot/galaxy-infos/1/200

Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8080 (#0)

GET /bot/galaxy-infos/1/200 HTTP/1.1
Host: <IP_ADDRESS>:8080
User-Agent: curl/7.49.0
Accept: /

< HTTP/1.1 200 OK
< Content-Type: application/json; charset=UTF-8
< Date: Sun, 18 Nov 2018 14:51:07 GMT
< Content-Length: 51
<

Connection #0 to host <IP_ADDRESS> left intact
{"Status":"ok","Code":200,"Message":"","Result":{}}%

Other stuff works;
➜  ogamed git:(master) ✗ curl <IP_ADDRESS>:8080/bot/planets
{"Status":"ok","Code":200,"Message":"","Result":[{"Img":"https://gf2.geo.gfsrv.net/cdn46/dDD22k2o2.png","ID":33621869,"Name":"MrCows","Diameter":12800,"Coordinate":{"Galaxy":1,"System":123,"Position":6,"Type":1},"Fields":{"Bui

Tried several random versions, seems to work on ogamed_1.2.10_darwin_amd64 and did not work on ogamed_4.0.0_darwin_amd64+++ so something between them editions, I've not managed to narrow it down further yet.
It seems like maybe the GO objects not being converted to a JSON object would be my guess, I don't really do Go code at all.

I fixed the issue.
Basically, unexported fields are not json serialized in Go.
https://github.com/alaingilbert/ogame/commit/6119cca51d8a54937eb5b600252b380ae85de015
I also added a test to ensure it keep working.
https://github.com/alaingilbert/ogame/commit/be13117e96a7644e29fbaf448003768fb8a4bb9e

Looks good to me. Thank you for your help.


JarExplode() detour fails on the second plugin on Windows

Sounds like a DHooks quirk.
An upcoming attribute reuses the DHooks setup for JarExplode() (since I currently don't want to rewrite it as a shared plugin), and that seems to not play nice on Windows.  Linux loads both plugins without issue.

Could also be a sigscan quirk -- this really should be cached.

Current plan is to wildcard the first six bytes to mask out the jump.


server bug: navigator is not defined

Usecase:
import Swiper from "swiper"

export default React.createClass({
   componentDidMount: function(){
      const swiper = new Swiper( this.refs.container, this.props.settings );
   },
});

If you try to render this on server side you'll get: ReferenceError: navigator is not defined
Problem: Swiper is not isomorphic.

Swiper is client-side only library, there is no sense to init/render it on server side

There is a sense to require it in isomorphic app, that's not said anyone is going to init it on server. Swiper fails here import Swiper from "swiper"

Can we reopen? This is a legitimate complaint. It means if you're using Webpack or Browserify or similar your only real option is to use the CDN, as far as I can tell.

@nolimits4web
Please reopen this.
We are not trying to render Swiper on server.
Your lib is braking apps with server side rendering just by being imported.
You should just not touch the DOM before Swiper constructor is called.
It shouldn't be hard.

Here is pull request that fixes the problem
https://github.com/nolimits4web/Swiper/pull/1960

I was using react-id-swiper (a wrapper around swiper), and I was able to get around this by
  componentDidMount() {
    this.setState({
      Swiper: require('react-id-swiper')
    });
  },

  render() {
    const { Swiper } = this.state;
    if (!Swiper) return null;

    return (
      <Swiper>
        ...
      </Swiper>
    );
  }

thanks to the idea from https://github.com/kidjp85/react-id-swiper/issues/14#issuecomment-228673157

@GingerBear could u link here the file where you managed to put this to work properly?
I'm trying the same here and I'm still not being able to make it work.

@renatogalvones
this should work
import React, { Component } from 'react';
import 'swiper/dist/css/swiper.min.css'

class App extends Component {
  constructor(props) {
    super(props);
  }

  componentDidMount() {
    const Swiper = require('swiper');
    const swiper = new Swiper(this.refs.container, {});
  }

  render() {
    return (

    );
  }
}

export default App;

@GingerBear thank you so much!!!
Worked like a charm!
PS: My bad about taking this looooong to answer here.

Booooooo.
Not supporting server side in 2017 is so lame.
Will have to fork this probably.

@GingerBear your solution works.
It can be also extracted into a component which can be used instead of the original <Swiper/>:
import React, { Component } from 'react'

// https://github.com/nolimits4web/Swiper/issues/1541#issuecomment-271439046
export default class FixedSlider extends Component {
  state = {}

  componentDidMount() {
    this.setState({
      Swiper: require('react-id-swiper')
    })
  }

  render() {
    const { Swiper } = this.state

    if (!Swiper) {
      return null
    }

    return <Swiper {...this.props}>{this.props.children}</Swiper>
  }
}

Bump, hey this is half of 2017. Please keep the library relevant.

Please consider reopening this issue.


[BUG]: reset fails with a syntax error if using a - in pgTableCreator to prefix tables

Report hasn't been filed before.

[X] I have verified that the bug I'm about to report hasn't been filed before.

What version of drizzle-orm are you using?
0.36.4
What version of drizzle-kit are you using?
0.28.1
Other packages
drizzle-seed
0.1.2
typescript
5.7.2
@vercel/postgres
0.10.0
Describe the Bug
I created my tables using drizzle's  multi-project-schema feature and because my project's name was alecs-project I had the following
export const createTable = pgTableCreator((name) => `alecs-project${name}`)

export const users = createTable('user', {
  id: serial('serial').primaryKey(),
  name: varchar('name', { length: 256 }).notNull(),
  profilePhotoUrl: varchar('profile_photo_url', { length: 256 }),
})

in case it's helpful, I'm also using vercel-postgres
index.ts
import { sql } from '@vercel/postgres'
import { drizzle } from 'drizzle-orm/vercel-postgres'

import * as schema from './schema'

export const db = drizzle(sql, { schema })

and this is my seed file
seed.ts
import { db } from '@/server/db'
import { reset, seed } from 'drizzle-seed'

import * as schema from './schema'
import { users } from './schema'

async function main() {
  await reset(db, schema)

  await seed(db, {
    users,
  })
}

await main()

but when I try to run this I get
pnpm db:seed

><EMAIL_ADDRESS>db:seed /Users/ayyjohn/code/alecs-project
> tsx --env-file=.env src/server/db/seed.ts

file:///Users/ayyjohn/code/alecs-project/node_modules/.pnpm/@neondatabase+serverless@0.9.5/node_modules/@neondatabase/serverless/index.mjs:1345
o?t(o):n(u)},"cb"),s=new r(function(o,u){n=o,t=u}).catch(o=>{throw Error.captureStackTrace(
                                                                         ^

error: syntax error at or near "-"
    at file:///Users/ayyjohn/code/alecs-project/node_modules/.pnpm/@neondatabase+serverless@0.9.5/node_modules/@neondatabase/serverless/index.mjs:1345:74
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async resetPostgres (/Users/ayyjohn/code/alecs-project/node_modules/.pnpm/drizzle-seed@0.1.2_drizzle-orm@0.36.4_@neondatabase+serverless@0.10.4_@types+pg@8.11.10_@type_ixvei6wbxfrdyjlzua65xktgve/node_modules/src/index.ts:457:2)
    at async reset (/Users/ayyjohn/code/alecs-project/node_modules/.pnpm/drizzle-seed@0.1.2_drizzle-orm@0.36.4_@neondatabase+serverless@0.10.4_@types+pg@8.11.10_@type_ixvei6wbxfrdyjlzua65xktgve/node_modules/src/index.ts:424:4)
    at async main (/Users/ayyjohn/code/alecs-project/src/server/db/seed.ts:10:3)
    at async <anonymous> (/Users/ayyjohn/code/alecs-project/src/server/db/seed.ts:26:1) {
  length: 90,
  severity: 'ERROR',
  code: '42601',
  detail: undefined,
  hint: undefined,
  position: '21',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'scan.l',
  line: '1188',
  routine: 'scanner_yyerror'
}

Node.js v20.18.0
 ELIFECYCLE  Command failed with exit code 1.

this is fixed by changing my multi project schema to just use underscores
export const createTable = pgTableCreator((name) => `alecs_project_${name}`)

but there's no problem until I try and use reset. If I simply use seed it runs just fine.
I feel like If that syntax isn't supported by postgres, then pgTableCreator should warn about that, or more likely it's just a mistake and that syntax should be supported.
Let me know if there's anything else needed!

Fixed in<EMAIL_ADDRESS>


Need a way to remove the autogenerated right sidebar

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
I'm trying to disable/remove the docusaurus autogenerated sidebar on the right. From what I've found so far, there doesn't seem to be a way to.
Self-service

[ ] I'd be willing to address this documentation request myself.

Have you seen https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-content-docs#hide_table_of_contents?
We can discuss this further in the discussions.

@Josh-Cena this is "per markdown file", but any option to remove it globally?
According to the code it seems I need to override frontMatter.hide_table_of_contents inside the context, but didn't figure out yet.

Yes, swizzling is one option (and not nearly a bad one). Another one is a createFrontMatter config API that we are planning for a minor version after V3, which allows you to inject front matter en masse.

createFrontMatter will come very soon, top priority for me: https://github.com/facebook/docusaurus/issues/5568
It's not a breaking change so I'll avoid delaying v3.0 launch for it, but it should be in v3.1 asap


hashPassword in patch

This check https://github.com/feathersjs/feathers-authentication/blob/master/src/hooks/hash-password.js#L36-L39 is preventing patch methods if password is not defined in the data.

Yeah.  hashPassword should just hash a password if one is available.  I made a PR.
cc: @ekryski, @daffl

An alternative for now is:
const { hooks } = require('feathers-authentication');

// Your custom hashPassword hook
exports.myHashPassword = function(options) {
  const hashPassword = hooks.hashPassword(options);

  // Add any custom options
  return function(hook) {
    if(hook.data.password !== undefined) {
      // Call the hashPassword hook
      return hashPassword.call(this, hook);
    }
  };
}

Taken from http://docs.feathersjs.com/hooks/usage.html#customizing-built-in-hooks