Refactor uses of mktempdir to use do block

If one runs the PackageAnalyzer on many packages, for example the entirity of the General Registry, it takes that JUlia session a really long time to exit because of all of the defered cleanup.
This also would release disk space sooner, giving a much smaller file system footprint for such large runs.

Thanks for the interest in the package! That's a good suggestion, but it'd require a bit of redesign which may add more complexity, because many mktempdira are used to create directories which are passed as arguments to other functions, so the life of those directories isn't confined to a single scope.

Thanks for your quick response.
I took a quick look at the code and appreciate the amount of refactoring that would be needed.  All of the mktempdir calls would need to be lifter to scopes that encompass the complete lifetime of their respective temporary directories and those directory paths passed down to where they're needed.
I might take a deeper look and maybe write a PR when I finish up with other projects.

I believe they can mostly or entirely be avoided by passing a root directory to analyze to store the package code in. If that's not the case, I would prefer PRs fixing that (i.e. letting those be configurable directories) rather than big refactorings


Add Existential Check

Just a check that subjectsOfNote isn't null.

This follows comments made in PR #93 . LGTM!


Add missing feasts to skSk.mjs

What I have changed:

fixed the celebration date of ourLordJesusChristTheEternalHighPriest;
added the celebration of saintGallAbbotAndMissionary, saintFlorianMartyr, saintHelena, and saintFaustinaKowalskaVirginAndReligious, dedicationOfAParticularChurch, saintMaurusBishop;
raised the celebration type of allSouls to memorial (from optional memorial in general.mjs);
updated saintEmeric with liturgical colour;
dropped saintJohnDamascenePriestAndDoctor in favour of saintJohnDamascenePriestAndDoctorOrSaintBarbaraVirginAndMartyr.

Note that these changes reflect differences between current version of src/locale/calendars/slovakia.mjs in romcal-es6 branch and the same file in the last commit in this pull request.

If you fixed the problem described in the first comment of issue #21, then this pull request fixes and closed issue #21.

I have finally finished the rebasing and fixing some issues. Bellow is the full list of changes in the latest commit. There are still two things I’d like do before merging.
Changes

remove duplicate key saintPatrick (it was defined only in en.js and used only in scotland.js);
merge the Slovak translation (tukusejssirs/ts_add_sk_l10n) into this one (tukusejssirs/sk_feasts), as I have found out that I haven’t rebased sk_feasts onto the other branch and it was missing the localised strings;
remove ourLadyOfMountCarmelMotherAndQueenOfChile key as a duplicate of ourLadyOfMountCarmel

currently, it is defined only in en.js, but it is not even used in chile.js (it was probably me who removed it some time ago from that calendar);

I have already localised the optional memorial of Our Lady of Loreto (as Pope Frances decreed, via the Congregation for Divine Worship and the Discipline; see #106) in en.js and sk.js; I think the celebration definition (and translation to other locales) should be a separate PR;
add missing blessedIstvanSandorMartyr key to en.js (this needs to be added to all other locales except for sk.js);
remove the executable permissions from it.js and italy.js (I don’t think it is necessary that *.js files are executable).

TODO

add Sts Cyril and Methodius celebrations (on 14 Feb as OptMem and on 5 Jul as Feast);
sort the calendar definitions in slovakia.js.

Okay, the celebrations are sorted now, but I have no idea what to do with the votive Masses (#110), though I this we should not include them in the calendar for now. If so, review the changes and you can merge it (fixes #21). :smiley:

Merging this as a first step to merge with es6 branch.

@jarosz I know we all want to release this new branch, but please don't merge too fast. This issue was mentioning there is still Todo and I didn't had time to check the question about votive mass.

@jarosz, I agree with @emagnier. What if I have introduced some bugs? The reviews are needed on each and every PR.
@emagnier, yes, there is only one last thing: the votive Masses, though those might not be necessary to be included in the PR.


Elementum 0.1.97 crashing kodi on android 13

Kodi 20.2 randomly crashing with Elementum 0.1.97 on Android 13.

kodi.log
kodi.old.log

Strange log. Try to uninstall addon the reinstall it.

kodi.log
kodi.old.log
Cleared kodi data and cache, reinstalled elementum arm64 0.1.97. Same result, kodi randomly crashes, either when buffering the file or it may even start playing, then kodi crashes, or  elementum is not even opened, but resumes downloading afrer start nd kodi crashes.  It all happens really fast and it's random.

@misterti Hard to day. Probably the only option is to connect with logcat (https://developer.android.com/tools/logcat), then you could see why system fails.
I have a feeling that it can be caused by signal in a library, but first you need to get system logs from that device.

Ok, i will do that

How should i grep the logcat? I have it with kodi filter. Is that enough?

Galaxy S23, Android 13, elementum 0.198
Attached test results, see test-info.txt
Hope it gives any useful info.
test-info.txt
kodi(note1).log
kodi(note2).log
logcat-kodi-elementum-note1.txt
logcat-kodi-elementum-note2.txt

Another note: On android box with Android 10, Elementum versions 0.197+  crash on start, kodi still runs

@misterti I thought you have problems starting Elementum, but it looks that you start fine, but then when you start a torrent - it crashes your Kodi?
Fatal signal 4 (SIGILL), code 2 (ILL_ILLOPN), fault addr 0x73620465c0 in tid 13875 (Thread-4), pid 13715 (org.xbmc.kodi)

This can be somehow related - https://github.com/android/ndk/issues/1848


Possible renaming of this project

I've been digging and there's a couple of different apns2 implementations for Rust using the same name. In crates.io the name's been taken by a curl-based implementation and then we have the library I forked this implementation from and which I rewrote, that is pretty high on Google results when you search for apns2 rust.
I think this project might need to be renamed, but into what? tokio-apns2 is kind of meh, and then there's the possibility to go grand and after some Greek gods such as Zeus or Zeus of Tokio...
Any ideas?

Done. Everybody using this crate should rename references from apns2 to a2. I don't know who else is evaluating this right now, but at least ping @dbrgn .

Nice :) Thanks for the heads-up.


Pathfinder - approximated paths when original destination unreachable

Mobs now move to a nearby place if no path to target
Fixed bugs when leaning
Temporarily fixed Attacking through walls by lowering attackDistance to 1.5 (Someone should check out why the trace at cAggressiveMonster::Tick is failing)
Fixed mob head shakes

Fixes #1963

Do not merge yet, I may have commited an old code. Checking.

Yes, this is an old code, I have no idea what I messed up /:

Recovered the right code and rebased.

The AttemptToFindAlternative is undergoing an overhaul, you may disregard the current version.

Travis failed due to parts of the code I didn't touch...

It's as if @worktycho is focusing AttemptToFindAlternative on purpose :P

printf will be eventually removed.

This is not a minor change, I tested it thoroughly but I may have missed things. Please double check me, especially on CHUNK querying stuff.

Is Coveralls.io stuck?

Yep.

Squashing to see if Coveralls was disabled.

Yes, Coveralls is now disabled.

@bearbin, Travis is complaining about you:
'__APPLE__' is not defined, evaluates to 0 [-Werror,-Wundef]

#elif __APPLE__

I think this was already fixed before my build started, will re-shake Travis and see if it helps.


BUG: Fix MetaCommand linking error

This commit adds ITKMetaIO as a dependency for the CompareCommand
modules from Base/Common.
This fixes the following error [1] thrown by the TextCompareCommand
and JsonCompareCommand modules.
[1] : undefined reference to `MetaCommand::MetaCommand()'

Looks good to me.

Thanks for reviewing!
Travis tests are failing but I can't find why. Should we merge ?

Hi,
That is the same bug that Deepak noted: sliding organ registration.
It is ok to merge, but please bump up the priority on getting CircleCL
working - it will then use a newer version of ITK, and perhaps this bug
will go away.
Thanks,
Stephen
On Mon, Oct 19, 2015 at 12:00 PM, Lucas Gandel<EMAIL_ADDRESS>wrote:

Thanks for reviewing!
Travis tests are failing but I can't find why. Should we merge ?
—
Reply to this email directly or view it on GitHub
https://github.com/KitwareMedical/TubeTK/pull/483#issuecomment-149258795
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Operations, North Carolina, Kitware, Inc.
http://www.kitware.com and http://www.aylward.org
(919) 969-6990 x300


SFX-333: Request products for SAYT

Closing for now due to the issue with rebasing (51 files changed). Please see #31 .


Logout should reset URL to the application root

currently, logout retains the last visited URL, which may not even be accessible to the user next to login.

fixed in v0.2-b7


added support for Github Enterprise

I just extended the Github module, so the functions used for Github Enterprise are exactly the same as the Github ones.
I've tested it against a trial version of Github Enterprise I have running and it works fine.
The tests work fine too as it's an identical copy of the Github v3 API.
Added docs as well

closing as there's an unexpected bug, will reopen later


Attempted import error: 'pick' is not exported from './object-utils' (imported as 'pick').

During build the following error would occur:
Attempted import error: 'pick' is not exported from './object-utils' (imported as 'pick').
These chances solve this issue by changing "exports" to "export" and making it compatible with the rest of the code.

It will break CLI scripts tho


Really odd issue with the whitelist entries in the minecraft chart

I have the following whitelist block:
    whitelist:
      - BlockyJackBauer
      - io42
      - SuperSammyDino
      - BennyPlayerX

io42 and SuperSammyDino both get looked up and added to the whitelist json file:
alexs-world-minecraft-5798b94dd7-ckthl:/data$ cat whitelist.json
[
  {
    "uuid": "9c1b56c6-d2e0-4479-9437-84c609f51f02",
    "name": "SuperSammyDino"
  },
  {
    "uuid": "e1a5cd65-eb7f-4d55-952b-c9812bc72b86",
    "name": "io42"
  }
]

It's adding square brackets around the names when looking up the other two for some reason. Extracted from the logs:
Failed to download https://playerdb.co/api/player/minecraft/%5BBlockyJackBauer%5D
Failed to download https://playerdb.co/api/player/minecraft/%5BBlockyJackBauer
https://playerdb.co/api/player/minecraft/BennyPlayerX%5D

None of those are valid, but you can find they do exist without the square brackets.

Just noticed this right above that part in the logs:
[init] Updating whitelist
grep: Unmatched [, [^, [:, [., or [=

Ok, forcing pull policy to always got the new image and I no longer get the original error. I do now get this, however, and it refuses to update it:
[mc-image-helper] 13:45:23.293 INFO  : The file /data/ops.json already exists, so no changes will be made
[mc-image-helper] 13:45:23.947 INFO  : The file /data/whitelist.json already exists, so no changes will be made

Ok, so I set those:
      EXISTING_OPS_FILE:              SYNCHRONIZE
      EXISTING_WHITELIST_FILE:        SYNCHRONIZE

Now I get this:
[mc-image-helper] 22:24:47.929 ERROR : Invalid parameter provided for 'manage-users' command: Could not resolve user from Playerdb: [BlockyJackBauer]

Thanks for the update. I'll have to investigate that since calling the API manually returns that player
https://playerdb.co/api/player/minecraft/BlockyJackBauer

...hmm, those square brackets in [BlockyJackBauer] are not supposed to be there. What does your whitelist.json look like? Also, the .minecraftServer.whitelist value is supposed to be a comma or whitelist delimited string rather than a list value. I'm wondering if Helm/Yaml has converted that into an array ([...]) as the string input.

So that makes perfect sense. A list would typically render to [...] so [BlockyJackBauer and BennyPlayerX] make sense as they are the first and last elements of the list respectively.
FYI whitespace (I'm assuming this is actually what you meant by whitelist) delimited doesn't work, only comma.
Sorry for the noise. I copied this values file from an older version of the helm chart which seems to work with a yaml list just fine.


upstream fixes

E2E also passes.
Run: https://github.com/kewalaka/terraform-azurerm-avm-res-storage-storageaccount/actions/runs/9047987976


Remote control HA using a keyboard

Description:
Recieve signals from a keyboard and use keyboard as a remote control
This component allows to use a keyboard as remote control. It will
fire ´keyboard_remote_command_received´ events witch can then be used
in automation rules.
The evdev package is used to interface with the keyboard and thus this
is Linux only. It also means you can't use your normal keyboard for this,
because evdev will block it.
Example entry for configuration.yaml (if applicable):
keyboard_remote:
  device_descriptor: '/dev/input/by-id/foo'
  key_value: 'key_up' # optional alternaive 'key_down' and 'key_hold'
  # be carefull, 'key_hold' fires a lot of events

# and an automation rule to bring breath live into it.

automation:
  alias: Keyboard All light on
  trigger:
    platform: event
    event_type: keyboard_remote_command_received
    event_data:
      key_code: 107 # inspect log to obtain desired keycode
  action:
    service: light.turn_on
    entity_id: light.all

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

Okay, pylint is talking BS here. There is a member evdev.ecodes.EV_KEY.
would it be okay to add:
[TYPECHECK]
ignored-modules = evdev

to pylintrc?

I think its ready for merge now.

Bueno 🐬


camera stream not working(there is no stream at all)

stream not working

tried ./Rpi_Cam_Control_interface_Installer.sh start------no resolve

tried ./install.sh------same result
tried ./start.sh-------same result
restarted the system----again same result

The recommended way to install as per wiki is to use the ./install.sh method. I don't support the older Rpi_Cam_Control_interface_Installer.sh any more and using it may prevent the ./install method working afterwards.
I recommend starting again with a clean raspbian, enabling camera interface and following wiki instructions using the ./install.sh method and using the defaults for the first install. You can always change settings afterwards.
If you still have a problem then check your camera connections and try using raspivid to make sure the camera is working OK.

@rajeshkrarora1  Have you tried sudo apt-get install motion then reboot?


Add checking the array has a specific number on NumericField class

Hi there,
https://redis.io/docs/interact/search-and-query/indexing/#index-json-arrays-as-numeric
According to the guide page, I can index the JSON array to NUMERIC.
With this feature, I can also check if the number in the array is in a specific range and if the array has a specific value.
However, the com.redis.om.spring.metamodel.index.NumericField class does not provide a way to verify that an array contains a specific number.
Since the above features are not supported, I need to put a string other than a number into an array to see if it is included using a TAG index, or use a native query using the RedisDocumentRepository.
However, I would like to implement these features using EntityStream and NumericField.
For instance, implement it like the code below or there seems to be a way to change it like TagField's in method.
public InPredicate<E, ?> containsLong(Long... values) {
    return new InPredicate<>(searchFieldAccessor, Arrays.asList(values));
}

I would like to ask for your opinion on this.

@chiyongs can you help me a concrete example using EntityStreams of what you think this might look like, that way I can start tinkering with the cleanest way to accomplish it. Cheers!


Fix flex shorthand transformation

Two issues were fixed

now suppports all shorthand variations (grow/shrink/basis, grow/shrink, and grow). Previously it didn't support grow/shrink
Also allows 0 as a value for grow/shrink. Because the logic checked token.match(NUMBER) directly, a zero value evaluated as false.

resolves #37

I've fixed this in another branch. In terms of the flex order, it should be spec compliant. I'm not too sure what cases you were trying to get to work, but if you think there's a bug, feel free to open another bug! But thanks anyway for helping!

I think your fix was good!


New enumeration, SlashCommandOptionType

This is the equivalent of ApplicationCommandOptionType in the Discord API.  Having this means the programmer doesnʼt have to remember which integer refers to which type.

Nice, but I think SlashCommandOptionType should go to model.py though.

Done!

Done!

Merging this pull request. Thank you for contributing!

Merging this pull request. Thank you for contributing!


Fix Issue #408

Added Icons and updated activity.js

That's a good start but you have to resize the board else no more space is available. Here you're just hiding the toolbar.

That's a good start but you have to resize the board else no more space is available. Here you're just hiding the toolbar.

Okay.
Do I have to increase the size buttons as well because doing so results in overlapping of buttons. Moreover some buttons get hidden.

Adding a sizeincrease and sizedecrease function is not the good way to do that.
My idea was more to rely on current resize method in the activity called when the window size is changed. See https://github.com/llaske/sugarizer/blob/dev/activities/Calculate.activity/js/activity.js#L45

Adding a sizeincrease and sizedecrease function is not the good way to do that.
My idea was more to rely on current resize method in the activity called when the window size is changed. See https://github.com/llaske/sugarizer/blob/dev/activities/Calculate.activity/js/activity.js#L45

So, do I have to call just the onresize function..

When the window resizes means when the width of the window is changed the board resizes but when the length is changed their is no change.
I guess as hiding toolbar on fullscreen changes the length of the window and not the width.... the use of CalculateApp.onResize(); function is not working.

But there is still a margin to the size of the toolbar in fullscreen mode. This margin is a lost of space, specifically for small screens.

That's nice like that.
By the way the screen is not responsive so unfortunately it's not possible to see the result zone in small screens.

But it's not related to the issue #408, I will fill another issue on that.


rename docs and order

I have a header line that starts with '#' so I want to do:
var df = readCsv(tsv, sep='\t').rename(f{"mode" <- "#mode"})

this works, but the docs say to use ~ which does not work.
Another issue is that it changes the column ordering by adding the new column at the end (as expected using OrderedTable), but I would expect it to retain the column order but keep a new name.
I know it's early days, but just wanted to flag this as I saw it.
Thanks for the dataframe lib!

Good catch, that it definitely a typo in the docs, should be <- as you noted :)

Another issue is that it changes the column ordering by adding the new column at the end (as expected using OrderedTable), but I would expect it to retain the column order but keep a new name.

It should be possible by reconstructing the OrderedTable with the replaced key(s), and the columns have reference semantics so there is no expensive copying of the data involved in recreating it.

Thanks for the dataframe lib!

I have a header line that starts with '#' so I want to do:

First of all, you can just use the header argument of readCsv:
var df = readCsv(tsv, sep='\t', header = "#")

(Note: it takes a string, but the implementation currently only works on the first char of it)

this works, but the docs say to use ~ which does not work.

@HugoGranstrom is almost right. It's not actually a typo, but a leftover from the first fully runtime based data frame implementation. That one had formulas based on ~ without any f{} macro, so one could write:
let fn = "mode" ~ "#mode"

and pass such a thing to the procs. I threw out the non scoped formulas when I rewrote the data frame implementation, because it seemed to foreign. Better to encapsulate it.
I really need to go through the datamancer code and fix up the docstrings and add runnable examples everywhere. Didn't do any announcements about a first datamancer release because the docs are still in the current state.

Another issue is that it changes the column ordering by adding the new column at the end (as expected using OrderedTable), but I would expect it to retain the column order but keep a new name.

Yup. I haven't paid much attention to the order of columns so far aside from making sure the order is as initially inserted. It feels like bad style to depend on the order of columns. It should only be important not to confuse people when printing / viewing them or writing them to file.
It seems misguided to provide access to columns based on indices like pandas allows, but maybe I'm missing something.
I do agree though, that the order for renaming and mutating existing columns (here I'm not sure from the top of my head if the order is actually kept, but it should even now) should stay the same.
Again @HugoGranstrom has a good point. In principle we can reconstruct a new table and assign the columns, as they are ref objects.
Maybe the standard library could grow a replace procedure for OrderedTable though. Seems like a useful thing to have in general.

This should hopefully addressed once #15 is merged. Feel free to provide feedback. Otherwise I'll close this issue in a couple of weeks. :)

As already mentioned on discord/matrix, could you please provide the first few lines of the CSV file so I can reproduce the problem?
(you could have opened a new issue for this, btw)


Make SlashingSpans Public

Offchain Applications will often need to inspect this type because it is directly used in staking election, thus worthy of being pub. Rest of the slashing api can remain private, only this and the fn last_non_zero_slash() of SlashingSpans are of interest.

Two questions here:

Why not make a pub fn getter?
Why not make all storage structs pub by default? (in general)

I ask the second question because I have run into issues before where I wanted to get something more simple like the hashed key of a storage item (for whitelisting benchmarks), and because the struct is private, I am not able to do so...
Given that all storage can be manipulated with lower level APIs, I don't personally see the advantage of having such limitations around...
But in the case of this PR, while we are following these practices, seems like a getter fn is more appropriate.

I also agree with making all storage public by default. I think it almost always is, this is one of  rare case where it isn't.

I think it almost always is

Nope: https://github.com/paritytech/substrate/blob/master/frame/system/src/lib.rs#L399
Would have to look at other pallets, but System is the one used by all runtimes, and has nearly none pub.

Given that all storage can be manipulated with lower level APIs, I don't personally see the advantage of having such limitations around...

I'm mixed for example if we make HistoryDepth public, and if a user modifies it then some era information might not be cleaned, but if we don't expose HistoryDepth but just get/set then API is better IMO.
But at the same making HistoryDepth public can ease migration.

also yes if offchain application just need to read it why not a getter ? get(fn slashing_span)

Updated to a getter 👍

bot merge

bot merge


Add Japanese translation of Steps from Apps documents

This pull request adds Japanese translation of "Steps from Apps" document pages.
Category (place an x in each of the [ ])

[ ] slack_bolt.App and/or its core components
[ ] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[x] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

I've reviewed all the content and verified the preview too. Let me merge this now.


npm install - couldn't read dependencies

here is a screenshot of the error in git shell http://puu.sh/nhqFU/f4d8d01673.png
I tried my best to follow every step. I have node.js 4.3.1 LTS installed

Have you cloned the repo to "Github" or did you not enter the fb-sleep-stats folder after cloning?

Have you cloned the repo to "Github" or did you not enter the fb-sleep-stats folder after cloning?

fb-sleep-stats in in  PC > Documents >GitHub
I did indeed go into the folder after cloning to do the step where you have to edit the config file

@HatSimulator Looking at your screenshot you didn't cd into the fb-sleep-stats folder before running npm install.
You need to do:
cd fb-sleep-stats
`

@sqren it got pretty far then i got the error to run it in admin mode, there isnt an option to do this on git shell

http://puu.sh/nhzhl/101f1713a0.png

You'll have to start powershell as Administrator. Simply search powershell in your start menu, right click and run as Administrator. Then try again.
What you see is an issue with permissions on your system. Neither nodejs, npm nor fb-sleep-stats.

You'll have to start powershell as Administrator. Simply search powershell in your start menu, right click and run as Administrator. Then try again.

What you see is an issue with permissions on your system. Neither nodejs, npm nor fb-sleep-stats.
ok now im running it as admin and it wont go to C:\Users\AlphaTwo\Documents\GitHub like the other powershell. Sorry im a nub at this. How do i point the powershell to that file

heres anther screenshot http://puu.sh/nhAPJ/95d2e63bbe.png

In your administrator powershell type cd C:\Users\AlphaTwo\Documents\fb-sleep-stats. Followed by npm install followed by npm run build-minified.

In your administrator powershell type cd C:\Users\AlphaTwo\Documents\fb-sleep-stats. Followed by npm install followed by npm run build-minified.

ok got through all of that and now im back to where I was before this http://puu.sh/nhD6t/f003fbe457.png where minified doesnt work

As according to the (readme file)[https://github.com/sqren/fb-sleep-stats/blob/master/README.md] you'll have to copy the default.json file to development.json in the config folder and fill it out with your info.
I'd advice you to read the readme file from end to other which is clear you haven't yet.

@bdoner I apologize. I didnt realize I had to create my own development.json file.
Ok i got all the way to scrape, it says 8 active users.
Now how do i type npm start now and see my results ? You have to open up a new shell and type in npm start I guess.
Thanks, Im guessing the program jsut runs in the background forever until i close out

Thats the way to do it. Run the scraper for as long as possible to gather most data :)
Run the server in a new powershell. That shouldn't need to be run as Administrator.

Run the server in a new powershell. That shouldn't need to be run as Administrator.

yup thats exactly what I did too. Thanks. My problem was solved.

Glad to see it solved. Thanks again @bdoner!


nixos/ddclient: remove invalid config keyword

Description of changes
Current version of this module logs a warning on startup:
file /run/ddclient/ddclient.conf, line 13: Invalid Value for keyword 'ipv6' = 'no'

ddclient --help output clearly shows that this should be an address:
-ipv6 <address>       : set the IPv6 address to <address>

IPv6 support is disabled by default (the usev6 option defaults to 'disabled' in the code) and if the user wishes to use it, they need to configure more options which this module currently does not directly support, but are easily configured with extraConfig (either ipv6, ifv6, webv6, fwv6 or cmdv6), therefore I removed the misleading option. Making it support an address would not make sense here as it would need to be hard-coded in the user's nix config, which would obviate the need for ddclient in the first place.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

As the use option is deprecated by ddclient as well, could we maybe go a step further and replace the use module option by usev4 and usev6 options that set the respective ddclient options?

As the use option is deprecated by ddclient as well, could we maybe go a step further and replace the use module option by usev4 and usev6 options that set the respective ddclient options? Then, usev4 could default to webv4 and usev6 to webv6.

i didn't get it to work with webv4 whereas just deleting the useipv6 option worked for me.

Apparently this was fixed by https://github.com/nixos/nixpkgs/commit/76cabe1644504ea4ae1cd8b99e3796cd9b5ddfc9 was merged instead.


Update Installation & Configuration Guide to use Secure Sites URL when using HTTPS

https://powerofus.force.com/0D58000001ztoql

done for version 3.76


(optionally) display data-flow graph

implement (switchable via a fwk.Property) an ASCII representation or DOT representation of the job data-flow dependency graph.

This issue was moved to go-hep/sw#35


Remove lock (_lockClients)

Problem description
Can "lock (_lockClients)" be omitted because it is always in the UI thread and called by lstClients.Invoke.
Proposal
Remove lock (_lockClients)
Additional context
No response

When two clients connect simultaneously it needs to be ensured via the lock that the clients list is only modified one at a time. But there may be better ways to handle this. Currently it is not safe to remove it.


how nodes can get new ip blocks after a ippool was added

Expected Behavior

I have a k8s cluster and have set the calico ip pool with CIDR /21 and blockSize /26 . So The maximum cluster size is 32 nodes. But now my cluster has 34 nodes with ip blocks exhausted. So I want to know how 33rd and 34th nodes can get the new ip block after I add a new ip pool.
Is the only way to delete the node and then add this node to k8s cluster again?
Current Behavior

IP blocks was exhausted at first and the 33rd and 34th node can not get the ip block. Then I add a new ip pool but the 33rd, 34th node still can not get the ip blocks.
Possible Solution

Steps to Reproduce (for bugs)

set the ip pool cidr /21 and ip blocks /26
scale the cluster to 34 nodes
check the 33rd and 34th node and cannot find the ip block
add a new ip pool
check the 33rd and 34th node and still cannot find the ip block

Context

Your Environment

Calico version: v3.8.4
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes
Operating System and version:
Link to your project (optional):

you can get ip block after you schedule a pod to the target node(the 33rd, 34th node in my case). Do not need to delete the target node and add this node again.
/close


oXygen framework improvements

See discussion between Marc and Tony about Marc's oXygen framework for XSpec,
at
http://groups.google.com/group/xspec-users/browse_thread/thread/59a779801e330f97
.

Original issue reported on code.google.com by fgeorges on 15 Sep 2011 at 9:58

oXygen has integrated XSpec for quite some time.  See http://www.oxygenxml.com/doc/versions/17.0/ug-editor/#topics/xslt-unit-test-xspec.html
oXygen has also supported add-on frameworks for quite some time.  For users who want the latest version, XSpec could include a zipped framework bundle as part of each release.  See http://inasmuch.as/2013/10/23/oxygen-add-on-hosted-on-github/ for how to do it.  This is working fine for the https://github.com/AntennaHouse/focheck project.


Calling getNode() on the ref of an Animated component is no longer necessary

Using react-native-anchor-carousel gives me the warning below
FlatList: Calling getNode() on the ref of an Animated component is no longer necessary. You can directly use the ref instead. The method will be removed in future release
Packages:

react: 16.11.0
react-native: 0.62.2
react-native-anchor-carousel: 3.1.2

Smartphone:

Device: Redmi Note 8 Pro
OS: Android 10

Thanks for your feedback. I will update soon @tanvoid0

This issue was fixed. Checkout new version 4.0.1. Thank you


Feature / App Question (Enterprise Deployment / Remove "xbar" default button at bottom)

We are wanting to use this for enterprise deployment on our macs managed through MDM with a custom plugin to pull down a list of shortcuts for our users.   This works perfectly in concept but we're wanting to hide the ability to add other plugins and/or containerize it so where we can update plugins in a more managed way (and not interfere with).
We certainly don't care that they use extra plugins but we don't want to per-se interfere w/ that or "support" an end user in doing so  (they're free to do so, but we're not staffed to support their stock ticker / weather update desires).
Anyhow - short of recompiling is there an option to hide the preferences/xbar default plugin option?

We would also be interested in a preference to control the visibility of the xbar menu in the xbar.config.json file.
Also related to deploying this for multiple users, it would be very helpful if Xbar also looked in the system-level path /Library/Application Support/xbar for a plugins folder and config file, not just the user's library.

BTW - Via re-compile and tweaks to app.go - its easy to achieve this.
Have forked a repo here that now works for our needs - https://github.com/Jiriteach/FCOVPN.
e.g. Via app.go the entire Preferences (which was xbar) could be removed so users cannot bring up the plugin browser etc. Also add Quit button as a default menu + it will auto disconnect if the user forgets to on quit.


Sdc gallery app, multiple choice (check boxes) component workflow

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1011
Description
Design multiple choice questionnaire json.
Write api to fetch multi choice questionnaire json file name.
Apply custom style at application side.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Feature
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)


Does Swagger Annotation Override JAX-RS?

I apologize if this has been asked before, but I couldn't find it anywhere.
If there's a discrepancy between a Swagger annotation and a JAX-RS annotation, which one will be used for generating the spec?  Consider the following method signature:
public Response search(@ApiParam(name = "term") @QueryParam(value = "keyword") String keyword)
Will the swagger.json show the parameter name as "term" or "keyword"?
I'm not so much interested in @ApiParam specifically, but rather in the overall philosophy of how swagger-core handles such conflicts.  I'm curious if this behavior (and the rationale behind it) is documented anywhere.  Thanks.

Generally speaking, the value in the Swagger annotations will be used over the JAX-RS ones. The rationale behind it is simple - if you don't need to change the value, you wouldn't specify it in the first place.


New documentation

convert to reST
review current functionality and make sure it is all documented
add a quickstart section with some easy examples.

See https://github.com/ddeboer/data-import/issues/148, https://github.com/ddeboer/data-import/issues/196.

ddeboer/data-import#208

It seems to me that RTD supports markdown as well. Are there any reasons why reStructuredText is better?

Yeah, I just found that out by accident because RTD defaulted to Markdown.
Actually, let's try Mkdocs instead of reST, as it's better to have one syntax for all documents (readme, docs, etc.). It seems cross references are possible. The only thing missing may be including other files, which may be useful for including tested .php (so no spelling errors or other mistakes) from the tests directory into the docs. However, as our tests will be divided over many repos and our docs available only in the main repo, I'm not so sure we will be needing this feature.

:+1: Actually, I am planning to use RTD for PHP HTTP as well. ;) reST was my only concern.

Please continue discussion in #12.


feat(workflow): add prologue and epilogue to WorkflowDefinition

Add the prologue and the epilogue properties to the sghi.etl.core.WorkflowDefinition interface. These properties return functions or callable objects that are executed at the beginning and the end of an SGHI ETL Workflow execution respectively.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
34

Relevant Lines:
34

💛  - Coveralls

:tada: This PR is included in version 1.2.0-rc.1 :tada:
The release is available on:

v1.2.0-rc.1
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

v1.2.0
GitHub release

Your semantic-release bot :package::rocket:


Enamorus spawning at the wrong time of day

Describe the bug
An enamorus spawned at night (around 15:00) in a sunflower plains even though it should only be a dawn/dusk spawn (18:30-0:00, 10:00-11:30).
To Reproduce
Steps to reproduce the behavior:
Not entirely sure how to reproduce a legend spawn so no idea
Expected behavior
Expected for enamorus to spawn during dawn/dusk, and not night as only cosmog spawns in sunflower plains at night.
Screenshots

These 2 are just so you can see the time that it spawned:

Username:
Oryso
Server name (all applicable):
Orion
Additional context
It also happened with a Throh that spawned during night when it should be a morning only spawn, so not sure how many pokemon this is effecting. Also not sure if its an actual bug or like a one time thing due to some kind of lag, even though I personally wasn't lagging.

We have a fix queued for tomorrow, hopefully this resolves. Please open a new ticket if issue persists after 26/02/2024.


Extend Confirmation History

to include confirmation time per block and Avg
example response
    "confirmation_stats": {
        "count": "18",
        "average": "18.657268277833328"
    },
    "confirmations": [
        {
            "hash": "862AB923DA184D85D5719D8222463E6DA6D6DB33F026B079B5F628C0598C4645",
            "duration": "7.6989772170000004",
            "tally": "67004984199081277794346622015067693764"
        },

...

urgh mixed branches give me a min to clean this up


Suggestion for a new code example

Fixes #

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Table: Allow translated strings for pagination and sorting

Description

References
Fixes #2108
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?

Checklist

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Testing has been completed. Please confirm whether we need to translate the 'Please select' also.

Testing has been completed. Please confirm whether we need to translate the 'Please select' also.

@jewel-shajan added translations for select component

Testing Completed and ready for PR Review.


Java Extractor: correctly handle symlinks in root directory [phab:D354]

Created by schroederc at 2015-06-22 22:21:19:
Fixes #577

shahms wrote at 2015-06-22 22:29:49:
This seems to warrant unittests to me.

schroederc wrote at 2015-06-23 18:54:20:
Add tests

schroederc wrote at 2015-06-23 18:54:45:

! In (earlier comment), shahms wrote:
This seems to warrant unittests to me.

Done

schroederc wrote at 2015-06-23 18:55:37:
Remove unneeded import

schroederc wrote at 2015-06-23 23:59:55:
Rebase

jdennett commented inline (context lost during migration) at 2015-06-24 00:03:06:
Can this be Collections.emptyList() ?

jdennett accepted this revision at 2015-06-24 00:03:06

shahms accepted this revision at 2015-06-24 00:04:48

schroederc wrote at 2015-06-24 00:45:47:
Use ImmutableList.of() for empty lists

schroederc commented inline (context lost during migration) at 2015-06-24 00:45:58:
ImmutableList.of() works too.  :-)

schroederc closed via commit cca5376f4ef2b2d0647c8cedc5fb012360bf7595 at 2015-06-24 00:46:40


training code

Wonderful work! Is there any plan to release the training code?

Thank you!
We will be releasing our DreamFL code in a few weeks. However, the training code for Reward3D may not be made available.


DP Singleton in C Programming

Need to make a readme file which should have the
Definition
Explanation in simple terms
Code related explanation.

Assign to me


Playback shortcuts

It would be really nice if you could implement custom keyboard shortcuts like Play/Pause/Skip/Thumbs UP/Down. Other players like Pandorita or PandaBar do have them, but Hermes seems more appealing to me.

You can script Hermes play/pause/thumbs up/down, and use a utility like FastScripts or Keyboard Maestro to trigger them.  (I wrote my own — https://github.com/nriley/StreamVision, but probably not for general use).


fix: renaming

fixe https://github.com/ovh/grafana-warp10/issues/25

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


Add OOM testing for release process

Description
Changes proposed in this pull request:

add testing lots of secrets and config maps to the stress test process in the release pipeline

Related issue(s)

running stress test sfor 0.4.5: https://github.com/piotrmiskiewicz/btp-manager/actions/runs/5750183882

stress tests for 0.4.7

/retest


Delay Format On Save

I have a setting the setting to Format On Save enabled. However, sometimes I don't want it to run it yet. Could a configurable delay be added before running the formatter again? So that I can Save -> Undo -> Save again (without formatting this time). This is how Visual Studio handles this functionality

I have never made a feature request before, so I feel a bit uncertain about the Netiquette.
Anyway, I have taken the liberty to open a new issue containing my suggestion, because it makes sense for me to do so.

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Implement Metal fences

Add hard fence support to Metal backend.

Consider adding to the release notes.
On Wed, May 1, 2019, 12:38 PM Ben Doherty<EMAIL_ADDRESS>wrote:

Add hard fence support to Metal backend.
You can view, comment on, or merge this pull request online at:
https://github.com/google/filament/pull/1162
Commit Summary

Implement Metal fences

File Changes

M filament/backend/src/metal/MetalContext.h
https://github.com/google/filament/pull/1162/files#diff-0 (4)
M filament/backend/src/metal/MetalDriver.mm
https://github.com/google/filament/pull/1162/files#diff-1 (31)
M filament/backend/src/metal/MetalHandles.h
https://github.com/google/filament/pull/1162/files#diff-2 (19)
M filament/backend/src/metal/MetalHandles.mm
https://github.com/google/filament/pull/1162/files#diff-3 (31)

Patch Links:

https://github.com/google/filament/pull/1162.patch
https://github.com/google/filament/pull/1162.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/filament/pull/1162, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJ2VSFDLGKRXCCWQ76P3YDPTHWR5ANCNFSM4HJWVRCA
.

This does not compile on any of our Mac build machines (iOS, mac and web):
../../filament/backend/src/metal/MetalContext.h:88:5: error: unknown type name 'MTLSharedEventListener'
    MTLSharedEventListener* eventListener = nil;


https://enisn-projects.io/ - documentation not available (HTTP ERROR 500)

Can you please point me to working docs

I also noticed that it doesn't work, thought it was something on my end.
Yesterday it was saying "host unreachable," now it's error 404

I was just about to report this myself :-)
I think you can access the documentation directly from github: https://github.com/enisn/UraniumUI/tree/develop/docs/en

It should be working now, sorry I couldn't take an action at weekend 👍

it's 404 now


added PR Template

@abhay-raj19 please review and merge this PR
Also, please don't add any labels as this commit was supposed to be done in my last merged PR but I forget to add it.

LGTM @Asymtode712


Polished documentation of Paddle Cloud installation.

Made some modification on details of Paddle Cloud installation.

Make sense to me, reverted it back.


2.0.3 SDK Build Error

$ npx nestia sdk
env: node: No such file or directory
Command failed: npx ts-node -C ttypescript /Users/hamin/Desktop/Workspace/platform-programming/backend/platform-api-nest/node_modules/nestia/src/bin/../executable/sdk

An error is generated that is presumed to be a problem with path resolve.

It occurs in MacOS, may be caused by using the absolute path with th __dirname.


Completed History Component

Description
Created a new History.js Component, as per the template provided in the issue description. The component imports a hard-coded json object from the src/shared/ folder and displays the object in tabular form.

Fixes #155
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] Any dependent changes have been merged and published in downstream modules
[x] I have added all the three reviewers: @vinitshahdeo, @kashish121 & @ishika1727

Completed the History.js component. Kindly go through the same.

I attached the screeshots. Kindly check

Hi, If everything is fine with this PULL REQUEST kindly merge the same

Hi, It has been more than 15 days. since I have completed the code for this module. Kindly check the same and merge the PR

@vinitshahdeo Been more than a month since I created this PR. If it is not ready to merge kindly tell me the changes , or else please merge the same. @ishika1727


Separate avatar velocity from avatar update loop

Right now we are updating the avatar velocity every frame.
The problem is that if we don't receive updated velocity information this frame we can get hiccups. The solution I have found is to update the velocity on remote avatars when the observer updates on transform calls, since any change in motion would necessitate a change in velocity. Also, this velocity is used to update the appearance of the avatar, nothing to do with position.

Updated to set isLocalPlayer on the avatar at construction time


Test Improvements

Now that the api test are merged, we still need to improve some parts, so that testing can be streamlined:

Github actions are not working correctly. This is because API tests needs ray, and Github CI machines are not powerful enough to host our ray container (actually, we even disable ray with --disable_ray and RAY_ENABLED=False in the configs for testing). The solution is to host our own ray service on our GPU machines.

During development, it should be possible to run the tests locally without any disturbance. Right now, I need to reset my database(s) to have fresh environments for tests.  Raffael once prepared a solution for this in run_tests.sh. and a corresponding env.testing.example file. We have to improve this solution, optimally so that everything is configured correctly in our launch.json. Not sure if we really need two new weaviate and ES containers, we just have to make sure that everything is cleaned, after the tests run. In conftest.py, there are already hooks for this (STARTUP_DONE:) and pyteset_sessionfinish().

this was the old script by raphael:
#!/bin/bash

set -euxo pipefail

# No matter what, before this script stops,
# always remove any docker containers it created to prevent
# them from lying around unused.
function cleanup() {
	docker stop "dats-weaviate-testing-$TEST_CONTAINER_ID" >/dev/null
	docker stop "dats-elasticsearch-testing-$TEST_CONTAINER_ID" >/dev/null
}
trap "cleanup" EXIT

SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")

# Read test-specific environment variables from configuration
set -o allexport
source $SCRIPT_DIR/../.env
[[ -f .env.testing ]] && source $SCRIPT_DIR/../.env.testing

# Find the absolute path for the elasticsearch config as that
# is the only thing docker will accept
ELASTICSEARCH_CONFIG=$(realpath -e "$SCRIPT_DIR/../../docker/elasticsearch.yml")

TEST_CONTAINER_ID=$(date +%s)

# Start a temporary elasticsearch database for testing
docker run --detach \
    --name "dats-elasticsearch-testing-$TEST_CONTAINER_ID" \
    --publish "$ELASTICSEARCH_TEST_PORT:9200" \
    --mount type=bind,source="$ELASTICSEARCH_CONFIG",destination=/usr/share/elasticsearch/config/elasticsearch.yml \
    --tmpfs /usr/share/elasticsearch/data \
    --env "xpack.security.enabled=false" \
    --env "discovery.type=single-node" \
    --env "ES_JAVA_OPTS=-Xms512m -Xmx512m" \
    docker.elastic.co/elasticsearch/elasticsearch:8.11.1

# Start a temporary weaviate database for testing
docker run --detach \
    --name "dats-weaviate-testing-$TEST_CONTAINER_ID" \
    --publish "$WEAVIATE_TEST_PORT:8080" \
    --tmpfs /var/lib/weaviate \
    --env "QUERY_DEFAULTS_LIMIT=25" \
    --env "AUTHENTICATION_ANONYMOUS_ACCESS_ENABLED=true" \
    --env "PERSISTENCE_DATA_PATH=/var/lib/weaviate" \
    --env "DEFAULT_VECTORIZER_MODULE=none" \
    --env "CLUSTER_HOSTNAME=node1" \
    --env "LOG_LEVEL=info" \
    semitechnologies/weaviate:1.21.3 \
    --host <IP_ADDRESS> --port 8080 --scheme http

# Override backend connection ports for the new containers
export WEAVIATE_PORT=$WEAVIATE_TEST_PORT
export ES_PORT=$ELASTICSEARCH_TEST_PORT

# Activate the conda environment for testing
ENV_NAME=dats source _activate_current_env.sh

# Tests will wipe all data; use a temporary repo root
# to protect our development files
REPO_ROOT=$(mktemp -d)
export REPO_ROOT

export PYTHONPATH=.

cd src

pytest "$@"


Fixed async accessors all named undefinedAsync

Really I didn't know how we miss this, because all tests pass and we test it I'm sure. Good job.

@maritaria - can you tell how you reproduce this bug, because I think that this case should be covered by tests

Agree with @tttmaximttt. We updated a lot of tests
https://github.com/dresende/node-orm2/pull/807/commits/907400531dcb4325a87a4cecda451f7a6eadc139
so those tests that related to accessors also passed without issues.
It will be very good if you could describe in what situation you find this error, so we can look if we have something similar in other part of async functionality. Thank you

I found this because I kept missing accessors being added to the original model when enabling reverse for hasMany relations. I added some print statements to the hasMany function to see what was actually happening.


Unable to import module in Typescript

Error
Error [ERR_REQUIRE_ESM]: require() of ES Module
Code
import {convert} from 'cashify';
const { convert }: any = await import('cashify');
const convert: any = (await import('cashify')).convert;
None of those examples above working.
Is there anyone facing this issue ?

Please see https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c

@krishnaTORQUE I am maintaining an ES2017 fork of this package at https://www.npmjs.com/package/cashify-es2017 , which means it can be used as any other package.


Validation Loss Exploding to 10^5

🐛 Bug
Validation loss  is exploding and I figured out that the problem is already in the forward method of my model.
During Validation , the input tensors after a batch norm or an activation function, does not change a lot, while during training it is rescaled.

I found something similar here in the Lightning Forum.
Are there some particular modes in the validation loss that I should turn on or off?
Environment

CUDA:

GPU:

NVIDIA RTX A6000

available:         True
version:           10.2

Packages:

numpy:             1.23.0
pyTorch_debug:     False
pyTorch_version:   1.11.0+cu102
pytorch-lightning: 1.7.0
tqdm:              4.64.0

System:

OS:                Linux
architecture:

64bit
ELF

processor:         x86_64
python:            3.8.13
version:           #44~20.04.1-Ubuntu SMP Fri Jun 24 13:27:29 UTC 2022

Thank you @awaelchli , indeed this  was the problem :

During Validation , the input tensors after a batch norm or an activation function, does not change a lot, while during training it is rescaled.

I solved it setting in the BatchNorm2D track_running_stats = False , but actually I don't know if this is a wrong work around, I will also try to do what you suggested. Thank you!

Let me know if you have further questions regarding this. Instead of setting track_running_stats = False, alternatively you could also try to tune the momentum parameter on the batchnorm layer.


Vertically display for Output panel

As a developer, I would like to ask for vertical panel of output panel
So, I could maximize using the width of the Go playground in big monitor.
I tried with inspect element to visualize it 😂

I assume this one could be a new setting item as well.

@vuon9 this can be considered as idea in future release.
Feel free to submit a PR :)

@vuon9 I was thinking about adding a button at the panel itself because looks like that not a lot of people actually using Settings menu.

I see. I would like to give it a try 🗡

@vuon9 see https://github.com/x1unix/go-playground/pull/147

oh wow u did it. that's amazing!! 🤩
i actually just did some draft code so it hasnt been completed as PR.
Thank you so much @x1unix 😍😍


[SPARK-12248][CORE] Adds limits per cpu for mesos coarse mode

Add spark.cores.mb.min for minimum memory per core
Add spark.cores.mb.max for max memory per core

Can one of the admins verify this patch?

Doesn't affect heap memory properly, closing until fixed

local fork of spark at https://github.com/metamx/spark/tree/v1.5.2-mmx is using this successfully, but since #10993 will probably be the way to go forward, I'm not going to re-raise this PR unless requested.

@drcrallen Did https://github.com/apache/spark/pull/4027 covered the memory constraints you were adding as part of this issue? I can't find another open or closed issue that addresses the issue raised here.

@PedroAlvarado I just looked at master and 2.0 and the limit of one task per slave seems to have been removed. I do not think this is valid anymore. Closing out ticket.


Support modern AWS EC2 instances like T4g

Is Solo5 the right platform for this?
Perhaps this could be a meta-issue describing the work required. My goal is to get Mirage running on T4g instances at AWS.

Hello @mbacarella, would you mind to extend a bit on what would be needed to run a VM on such a T4g instance?

Several questions are needed:

which assembler such processor uses? It seems a 64-bit ARM assembler (aarch64/arm64) which is good for us but experimental
how the AWS hypervizor talk to the operating system? We know that, on Google Cloud, we advises to use virtio but I never tried AWS, do you have some pointers for us?

Yes, it's ARM64. These run on AWS's Nitro virtualization platform, which is their replacement for Xen.
Poking around a Linux T4g instance:
# lscpu
Architecture:           aarch64
  CPU op-mode(s):       32-bit, 64-bit
  Byte Order:           Little Endian
CPU(s):                 2
  On-line CPU(s) list:  0,1
Vendor ID:              ARM
  Model name:           Neoverse-N1
    Model:              1
    Thread(s) per core: 1
    Core(s) per socket: 2
    Socket(s):          1
    Stepping:           r3p1
    BogoMIPS:           243.75
    Flags:              fp asimd evtstrm aes pmull sha1 sha2 crc32 atomics fphp asimdhp cpuid asimdr
                        dm lrcpc dcpop asimddp ssbs
Caches (sum of all):
  L1d:                  128 KiB (2 instances)
  L1i:                  128 KiB (2 instances)
  L2:                   2 MiB (2 instances)
  L3:                   32 MiB (1 instance)
NUMA:
  NUMA node(s):         1
  NUMA node0 CPU(s):    0,1
Vulnerabilities:
  Itlb multihit:        Not affected
  L1tf:                 Not affected
  Mds:                  Not affected
  Meltdown:             Not affected
  Mmio stale data:      Not affected
  Retbleed:             Not affected
  Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl
  Spectre v1:           Mitigation; __user pointer sanitization
  Spectre v2:           Mitigation; CSV2, BHB
  Srbds:                Not affected
  Tsx async abort:      Not affected

# lspci
00:00.0 Host bridge: Amazon.com, Inc. Device 0200
00:01.0 Serial controller: Amazon.com, Inc. Device 8250
00:04.0 Non-Volatile memory controller: Amazon.com, Inc. NVMe EBS Controller
00:05.0 Ethernet controller: Amazon.com, Inc. Elastic Network Adapter (ENA)

According to this doc, a Linux instance appears to need to be installed/configured with some Amazon specific drivers.
https://aws.amazon.com/premiumsupport/knowledge-center/boot-error-linux-nitro-instance/
The drivers are here:
https://github.com/amzn/amzn-drivers
So, one way forward with the fewest variables changed is select an x86-64 instance type that runs on Nitro (t3.nano?), follow the Mirage AWS EC2 on Xen instructions (https://mirage.io/docs/xen-boot) and maybe hope for screen output.
From there, we can explore adding those drivers to Solo5. Somewhere in there, also try ARM64.


Cannot remove sides from tournament match

#1061 causes the dialog to be much wider. Remove button is shown but is off screen


[desktop] Playlist song -> right click -> Add to playlist -> should show "Starred" as an option

CC: @votaguz

Hello gubatron,
I can help in fixing this issue as I have more than 100 hours of development experience with "Frostwire". Basically I am a freelancer and for one of my client I had done lot of modification
Thanks,
Unni M

CC: @votaguz
Done.


Add a --no-color CLI argument

When running tgenv in CI/CD pipelines the output is polluted with the color characters.
[2023-03-09T12:20:42.216Z] [INFO] Installation of terragrunt v0.43.2 successful
[2023-03-09T12:20:42.538Z] [INFO] Switching to v0.43.2
[2023-03-09T12:20:43.707Z] [INFO] Switching completed
[2023-03-09T12:20:44.438Z] [INFO] Switching to v0.43.2
[2023-03-09T12:20:45.159Z] [INFO] Switching completed

Having the ability for tgenv to not output the color codes would be a great feature add.

@davidjeddy out of interest, which CI/CD are you using? I use it in gitlab and the colors work fine.
I can see some merit to this request though. terraform and terragrunt both have a -no-color option
e.g
$ terragrunt plan --help | grep -no-color
  -no-color           If specified, output won't contain any color.

Since tgenv wraps the terragrunt bin with our own script we should be able to check if the -no-color argument has been specified and disable color in tgenv output as well.
For this I would recommend refactoring colored output into an extracted function and having that function check for a no color flag.
Cant promise when I will get onto pull request for it but feel free to raise one and I will promise to be responsive to reviewing.
Will tag as good first issue.

We use Jenkins to execute our pipelines.

Sorry this year has been... challenging. Finally got onto this. Pull request here: #29

Thank you @iokiwi; awesome you found time to address this. I will push the update to my teams today!


IndexError: list index out of range

Hello,
when I use the command 1 to identify strains,
command 1: python StrainScan.py -i E.coli-3lr.fastq -d ./download_database/DB_Ecoli -b 1 -o ./test
I got the errors below:

I don't know what's wrong
By the way, the code that I install StrainScan is:
git clone https://github.com/liaoherui/StrainScan.git
cd StrainScan
conda env create -f environment_candidate.yaml
conda activate strainscan
chmod 755 library/jellyfish-linux
chmod 755 library/dashing_s128

Thanks for any feedback you might have!

Hi, thanks for using StrainScan.
It seems this error could be a potential bug due to the identification of low-depth strains. Would you mind sending your test data to us for debugging? Then, we will check the reason and give the solution asap. Thanks!

The test data "E.coli-3lr.fastq" is already on Baidu Netdisk.
Baidu Netdisk link: https://pan.baidu.com/s/131vdOx97DERIJwYnK2lZhQ
Extraction code: khdy
Thanks!!

Hi, sorry for late reply.
We have fixed the bug in the latest GitHub version. Please re-install StrainScan from the GitHub. And apply it to your data to see whether it works (we have tested the program with the data you provided, and it runs successfully). Thanks!

Hi, sorry for late reply.
We have fixed the bug in the latest GitHub version. Please re-install StrainScan from the GitHub. And apply it to your data to see whether it works (we have tested the program with the data you provided, and it runs successfully). Thanks!

Thanks for your reply! After re-installing StrainScan, it runs successfully on the test data "Ecoli-3lr.fastq"!
running command: python StrainScan.py -i E.coli-3lr.fastq -d ./download_database/DB_Ecoli -b 1 -o ./test
However, I have another problem. After running strainscan, I got a warning: "Warning: No clusters can be detected!" (even I have used the parameter "-b 1") and only got one output file named "strain_prob.txt" as follows:

That means there is no strain identified, or did I run it wrong again?

Hi,
I have reviewed your output, and it appears to be reasonable. When you use -b 1, this means there could be low-depth strain(s) in your input data. Thus, it's normal to detect no strain clusters. However, when you use the -b 1 option, the generated "strain_prob.txt" file provides the probabilities of strain clusters existing in your input data. In the screenshot you shared, C33 has a probability of approximately 75%, suggesting that this cluster could potentially be present in your input data. To determine which strains are associated with this cluster, you can refer to the "<Database_Dir>/Cluster_Result/hclsMap_95_recls.txt" file in your database. Usually, the higher the probability, the greater the likelihood of the cluster's presence in your input data.
I hope this explanation clarifies the output for you. If you have any further questions or encounter any other issues, please don't hesitate to ask.

Thanks for your explanation! That helps me a lot!

Hi,
I have reviewed your output, and it appears to be reasonable. When you use -b 1, this means there could be low-depth strain(s) in your input data. Thus, it's normal to detect no strain clusters. However, when you use the -b 1 option, the generated "strain_prob.txt" file provides the probabilities of strain clusters existing in your input data. In the screenshot you shared, C33 has a probability of approximately 75%, suggesting that this cluster could potentially be present in your input data. The final column of this file refers to the strains associated with this cluster. Usually, the higher the probability, the greater the likelihood of the cluster's presence in your input data.
I hope this explanation clarifies the output for you. If you have any further questions or encounter any other issues, please don't hesitate to ask.

Thanks for your explanation! That helps me a lot!

Hi,
I have reviewed your output, and it appears to be reasonable. When you use -b 1, this means there could be low-depth strain(s) in your input data. Thus, it's normal to detect no strain clusters. However, when you use the -b 1 option, the generated "strain_prob.txt" file provides the probabilities of strain clusters existing in your input data. In the screenshot you shared, C33 has a probability of approximately 75%, suggesting that this cluster could potentially be present in your input data. The final column of this file refers to the strains associated with this cluster. Usually, the higher the probability, the greater the likelihood of the cluster's presence in your input data.
I hope this explanation clarifies the output for you. If you have any further questions or encounter any other issues, please don't hesitate to ask.

Thanks for your explanation! That helps me a lot!
By the way, can I specify the name of the output file? I only see that I can specify the directory where the output file is located

Hi,
I have reviewed your output, and it appears to be reasonable. When you use -b 1, this means there could be low-depth strain(s) in your input data. Thus, it's normal to detect no strain clusters. However, when you use the -b 1 option, the generated "strain_prob.txt" file provides the probabilities of strain clusters existing in your input data. In the screenshot you shared, C33 has a probability of approximately 75%, suggesting that this cluster could potentially be present in your input data. The final column of this file refers to the strains associated with this cluster. Usually, the higher the probability, the greater the likelihood of the cluster's presence in your input data.
I hope this explanation clarifies the output for you. If you have any further questions or encounter any other issues, please don't hesitate to ask.

By the way, can I specify the name of the output file? I only find that I can specify the directory where the output file is located
Thanks!!

Currently, we don't have that function. But a Python script can achieve that. If you need that function, please email me (heruiliao2-c@my.cityu.edu.hk). Then I can help you with that.


Do not treat insufficient temp space as failure

When a task fails because we couldn't free enough disk space for it,
that doesn't mean there is anything inherently wrong with that task, it
only mean that task cannot be run on this node at this point in time.
If we only set the tasks state to queued and dissociate it with this
agent, however, there is a high likelihood that the master's scheduler
will just immediately reassign that task to us.  To avoid that, we add
this agent to the failed_on_agents list of that task.

Looks good, saw a couple of places that could be improved but nothing major or worth making different than a few other places in the code base.  Merging.


class 'MyBB\Core\Database\Repositories\Eloquent\MessageFormatter' not found

when create topic
 FatalErrorException in PostRepository.php line 191:
Class 'MyBB\Core\Database\Repositories\Eloquent\MessageFormatter' not found

Hi,
This is a known error due to ongoing changes being made to the message parser.


bug: kubecontext flag has no effect

Checklist:

[x] I've searched for similar issues and couldn't find anything matching
[x] I've included steps to reproduce the bug.
[x] I've included the version of Kubernetes and k8sgpt.

Subject of the issue
Relevant to issue https://github.com/k8sgpt-ai/k8sgpt/issues/114 that was closed in PR https://github.com/k8sgpt-ai/k8sgpt/pull/155.
Primary issue: The kubecontext flag has no effect. Current context is always used despite any given input.
Secondary issue: The functionality isn't described in the docs - happy to submit chore PRs to document how this works if this is an issue from my end.
Your environment

Version of KubernetesClient Version: v1.26.1
Kustomize Version: v4.5.7
Server Version: v1.24.10-eks-48e63af

Host OS and its version / If windows, is it WSL?MacOS Ventura 13.2.1

Version of k8sgpt
v0.1.4

Steps to reproduce
Assumes you have a kubeconfig with a functional current context - e.g. you can run kubectl get nodes successfully.

Download k8sgpt v0.1.4
Run ./k8sgpt auth, put some random key as we wont use it anyway
Run ./k8sgpt analyze - works fine, uses your default context
Run ./k8sgpt analyze --kubecontext aContextThatDoesNotExist - runs without issue, as it uses your default context despite any given value

Expected behaviour
If I provide a context that exists, that context should be used for the analysis.
If I provide a context that does not exist, the tool should crash or fail (hopefully with a nice error message explaining why.)
Actual behaviour
Current context of your kubeconfig is always used. Providing a context that doesn't exist will still use the default, leading to misleading behaviour as shown below.
# without context flag - uses current context, this cluster is healthy
❯ ~/Desktop/k8sgpt_Darwin_arm64_0.1.4/k8sgpt analyze
{ "status": "OK" }

# with context flag - current context is used anyway
❯ ~/Desktop/k8sgpt_Darwin_arm64_0.1.4/k8sgpt analyze --kubecontext aContextThatDoesNotExist
{ "status": "OK" } # this is a lie - the given context doesn't even exist

Additional context / screenshots

Hello,
Indeed, I believe that we never utilize this section of the code : https://github.com/k8sgpt-ai/k8sgpt/blob/main/cmd/root.go#L65


RepositorySimulator.add_key() only works for top level roles

def add_key(self, role: str)
this assumes the delegating role is root. That's a fine default  but I want to add keys for non-top-level roles too
This should work:
def add_key(self, role: str, delegator: str | None)
(and then use Root.type as the real default value)

this is fixed in #129


How would i revoke access on signout outside this.setState

Hi,
@adamjmcgrath Thanks a lot for this awesome project.
I wanted to confirm is there was a way user could signout by revoking access outside this.setState?

Hi,
Not sure what "revoking access outside this.setState" means exactly but hopefully this should answer you question:
From the author of SimpleAuth https://github.com/calebd/SimpleAuth/issues/19#issuecomment-33102305
"Simple is designed to authenticate your users against a social service and give you their access tokens. What happens to those tokens after that is up to you. You can store them in the keychain, send them to your back end, make requests, etc. When you no longer need them, or when requested by the user, you should remove those tokens from your storage and update your interface."
Any other questions, let me know

@adamjmcgrath Thanks a lot for taking time to explaining. I know better now.


Enhancement on validation_docker script

Is your feature request related to a problem? Please describe
We want to make a few of enhancemens on the docker validation script including the following :

[x] Adding a flag/argument to do Only the digest validaiton without moving to run cluster, nor the API test
[x] Adding a flag/argument to use Only staging images ( in which, we will not check digest )

By doing this, we can validate the OpenSearch API prior of publishing to the product account at DockerHub

[x] Adding a max retry for testing if the OpenSearch cluster is ready for accepting HTTP request
[x] Using Curl to replace 'requests' library in Python

it was found that the requests lib did not response quick enough to catch the return status code

[x] Return 0 instead of True to the caller when the script ends successfully

it was satisfy Jenkins CI process

Describe the solution you'd like
We will update the script accordingly
Describe alternatives you've considered
No response
Additional context
No response

PR has been merged .. closing the issue


Added Imagick to the php runtimes

I am suggesting adding phpx.x-imagick to the dockerfile for each runtime. The reason this would help others is Imagick is quite common in its use when it comes to using Intervention, Spatie packages as well as general image manipulation. This would save a lot of time and stress for people having to customize their dockerfile. By preventing one-off customizations a lot of issues will be alleviated from various issue reports.
I did test this on each runtime and there are no errors or deployment issues.

Are you sure this is all that is needed. For example on Forge we also run:
apt-get install -y --force-yes libmagickwand-dev

yes you do not need that. Please see reference
https://packages.debian.org/buster/libmagickwand-dev
The main thing to know is Imagick will install whatever dependencies it needs during install but for a functional system without turning Sail into a massive one-off solution, all that is needed is imagick.
I have tested this on all php versions with Intervention, Spatie Media Library, Spatie Image and direct imagick manipulation just to make sure there were no standards deps missing.


Simulate all OS/arch combinations in brew readall

This PR updates brew readall to simulate other OS configurations. This will hopefully help us reduce errors in formulae and casks that don't appear in the OS versions we test with in CI.
I've also added a --no-simulate flag to brew readall to opt-out of this if desired, but if we think it's better just to force this everywhere then I can remove it.
Of course, this does make brew readall a lot slower (which might be a reason to include the --no-simulate flag):
$ hyperfine 'brew readall --eval-all --no-simulate' 'brew readall --eval-all'
Benchmark 1: brew readall --eval-all --no-simulate
  Time (mean ± σ):      7.690 s ±  0.074 s    [User: 6.970 s, System: 0.690 s]
  Range (min … max):    7.560 s …  7.824 s    10 runs

Benchmark 2: brew readall --eval-all
  Time (mean ± σ):     46.813 s ±  0.382 s    [User: 43.519 s, System: 2.881 s]
  Range (min … max):   46.240 s … 47.521 s    10 runs

Summary
  'brew readall --eval-all --no-simulate' ran
    6.09 ± 0.08 times faster than 'brew readall --eval-all'

Review period skipped due to critical label.

Great work @Rylan12!

brew prof doesn't seem to work on my machine for some reason. If I have time later I'll look into it.
$ HOMEBREW_EVAL_ALL=1 brew prof readall
/opt/homebrew/Library/Homebrew/brew.rb:10:in `<top (required)>': /opt/homebrew/Library/Homebrew/brew.rb must not be loaded via `require`. (RuntimeError)
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-prof-1.4.3/bin/ruby-prof:291:in `load'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-prof-1.4.3/bin/ruby-prof:291:in `block in run'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-prof-1.4.3/bin/ruby-prof:290:in `profile'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-prof-1.4.3/bin/ruby-prof:290:in `run'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-prof-1.4.3/bin/ruby-prof:328:in `<top (required)>'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/bin/ruby-prof:23:in `load'
        from /opt/homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/bin/ruby-prof:23:in `<main>'
Error: Failure while executing; `ruby-prof --printer=call_stack --file=prof/call_stack.html /opt/homebrew/Library/Homebrew/brew.rb -- readall` exited with 1.

@Bo98 Think the above is a regression from https://github.com/Homebrew/brew/pull/13825, can you take a look?

I suppose I could revert that line. Though it is a nice safety feature to avoid require "brew".
Alternative is to gate it behind some HOMEBREW_PROF env that brew prof would set.
(side note: I think --stackprof is unaffected if you need something to work now)

I suppose I could revert that line. Though it is a nice safety feature to avoid require "brew" and might still be needed to avoid brew typecheck --update executing abritrary code paths in that file.

Doesn't necessarily need reverted, just a handled however.

Alternative is to gate it behind some HOMEBREW_PROF env that brew prof would set. That's probably the safest way.

Yeh, works for me 👍🏻. May also be worth having an integration test for something quickish to ensure this doesn't regress in future.

(side note: I think --stackprof is unaffected if you need something to work now)

Yeh, it looks like it works, thanks 👍🏻. Maybe worth being the default behaviour?


Runtime error with Angular 9 RC 1

In Angular 9 project (RC1) ngx-emoji-mart crushes on the line https://github.com/TypeCtrl/ngx-emoji-mart/blob/master/src/lib/picker/picker.component.ts#L417
with error
ERROR TypeError: Cannot read property 'width' of null

when use tag without style attribute
<emoji-mart></emoji-mart>

with style attribute everything works fine
<emoji-mart [style]="{}"></emoji-mart>

Repo to reproduce can be find here
https://github.com/alexanderyarm/ngx-emoji-mart-angular9

:tada: This issue has been resolved in version 1.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add an "Auto UI scale" feature

As of #115, the default UI scale is unusable on macOS:

This PR adds a new feature, enabled by default, that will automatically increase the UI scale. I added a new checkbox to the settings menu. When unchecked, the UI scale number directly sets the UI scale. When checked, the UI scale number acts as a multiplier, and the actual scale is calculated from this number and the window size.

Thanks!


Topic/query param key

this PR adds a typeclass that will represent a query key. It may not be finished (maybe it is), but I wanted to get this up for feedback.
Changes

QueryParamEncoder[T].encode(T) : Seq[QueryParameterValue] instead of a single value. This is needed for the following point to work.
Varadiac methods were dropped in favor of a single value which also is a type class. Multiple values can be handled by a typeclass that takes a Seq[T]; typeclasses for other types of collections are needed.
Quite a few methods were dropped from the Uri class, but not as many as I'd hoped. I cant help but feel the Maybe and Option receiving methods are largely clutter: the same functionality can be done with cata and fold or other standard techniques. Perhaps I just haven't needed them yet and they are really awsome: its really a matter of convenience.

Where do we stand on this? I know @julien-truffaut would like to do a little cleanup based on this involving removing the internal _ prefixed methods, but I'm not entirely sure what he wants so I'm keen to leave that to him.
I'd like to get this in before it gets too stale, so I'd appreciate suggestions or the OK to merge it. After that, we should decide if we are going to change the representation from a String (I think this is a good idea, maybe we can preserve laziness like we did with headers, maybe its not an issue and we just eat a few CPU cycles).

I haven't followed this very closely, but it looks reasonable.
I still like the LinearSeqOptimized query string approach.

I like it too.

Cool, if @julien-truffaut doesn't object in the next while, I'll merge it. @rossabaker, by LinearSeqOptimized I assume you are referring to the form of the query after parsing the String.

Yes.
On Jan 10, 2015 1:34 PM, "Bryce Anderson"<EMAIL_ADDRESS>wrote:

Cool, if @julien-truffaut https://github.com/julien-truffaut doesn't
object in the next while, I'll merge it. @rossabaker
https://github.com/rossabaker, by LinearSeqOptimized I assume you are
referring to the form of the query after parsing the String.
--
Reply to this email directly or view it on GitHub
https://github.com/http4s/http4s/pull/161#issuecomment-69466449.

I think there is something wrong on QueryParamEncoder / QueryParamDecoder type class because this change makes them non "symmetrical". QueryParamEncoder produces a Seq[QueryParamValue] but QueryParamDecoder accepts a single QueryParamValue. I don't have an answer but I think we should take some time to think about the law of these 2 type classes.

@julien-truffaut, the break in symmetry comes from using nested typeclasses to encode list type structures. If we Make all methods take a Seq[T], I think it would restore symmetry at the cost of requiring all parameters be a Seq. On the plus side, the special case of Unit for when we don't want any params is dropped. I haven't attempted to see if that works, I'll give it a shot.

Well, the zero param case is still a problem because it cannot infer the type of a Nil which is needed for the typeclass, so its free to choose any, and thus has ambiguity problems. This could be cleared up by renaming a class of the +? methods, much like is done with the Option and Maybe versions. I'm going to rename a few of them and see how it goes.

@julien-truffaut, that last commit restores the symmetry which I think is a good thing. Sadly, a couple of the +? methods had to be renamed to +*? to avoid ambiguities. I think this is acceptable, and maybe there is a better name?

I was happy enough with this not to wait any longer to merge it so I can move on to other things. If there are other changes to be made (almost certainly are) lets do another PR.

+1. Any further bikeshedding of the query can be done when it goes
LinearSeqOptimized.
On Jan 10, 2015 5:23 PM, "Bryce Anderson"<EMAIL_ADDRESS>wrote:

I was happy enough with this not to wait any longer to merge it so I can
move on to other things. If there are other changes to be made (almost
certainly are) lets do another PR.
--
Reply to this email directly or view it on GitHub
https://github.com/http4s/http4s/pull/161#issuecomment-69474617.

I'm working on that now.

I don't think that the case with zero value is very common, so I believe it could be acceptable to use Seq.empty[String] or Seq.empty[Unit].


Redirect to code flow example from nullable types example leads to inconsistent behavior

Bug Report

🔎 Search Terms

code flow example documentation
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about "code flow link documentation"

⏯ Playground Link
This is the link of examples section in typescript documentation where the issue is happening and this link to the same page is where the issue is not occuring
💻 Code
Follow code flow example link at line 40
// ( see more at example:code-flow )

Hover over jon on if and else statement at line 17 and see how the type does not change as expected
// If you hover your mouse over the three following uses of 'jon' below,
// you'll see how the types change depending on where the word is located:

if (jon) {
  jon;
} else {
  jon;
}

🙁 Actual behavior
The type did not change on hover
🙂 Expected behavior

I expected the type to change as is happening correctly when redirected from the type guards page code flow example link on line 4
// Type Guarding is the term where you influence the code
// flow analysis via code. TypeScript uses existing JavaScript
// behavior which validates your objects at runtime to influence
// the code flow. This example assumes you've read example:code-flow

Pardon any mistakes above but if there is anything I should fix or I can do to help let me know and thanks for the great tool!

Yeah, interesting - I think the examples loader needs to make sure it sets all of the required params.
Re: length, the goal of each one is to describe a concept completely independently - I'm not wild on dropping the length of the comments because that probably relies on linking out, but I'm open to PRs which can keep the original goal with less comments 👍🏻


Travis fails with cython 0.27. Use cython 0.26.1 for a while

Fix https://github.com/cupy/cupy/issues/527

asking at https://github.com/cython/cython/issues/1885


"An attempt was made to break through the security policy of the user agent" when creating a new novel

Happens Windows 8.1, typing any name at the "entry" screen and clicking "start writing":
Uncaught node.js Error

Error: An attempt was made to break through the security policy of the user agent.
  at Future.eval (www\scripts\main.js:39:32)
  at Future.eval(node_modules\data.future\lib\future.js:136:35)
  at Future.eval(node_modules\data.future\lib\future.js:136:35)
  at Object.exports.run (www\scripts\utils:29:12)
  at React.createClass.reload (www\scripts\screens\entry.js:169:23)
  at wrapper [as _onTimeout] (timers:252:14)
  at Timer.listOnTimeout [as ontimeout] (timers:110:15)

Workaround for those who want to write: Create an empty .md file and "import" a book. :smile:

It happened to me too (OS X Yosemite).
Also, by clicking on the button instead of hitting 'Enter' the problem was solved.

And I miss the old icon. :disappointed:

@haskellcamargo old icon?

The book instead of this:

@haskellcamargo ah, that looks like a packaging error. Can you open a separate issue for this?

Confirmed on Ubuntu 22.04. Error displays, then app auto-closes (crashes) a few seconds later.
Only happens the first time. Creating the project again (at least this same title) doesn't trigger the issue.


Global mode string is not visible on the tab bar

What did you expect to happen?
When adding the tab-bar-format-global function to the tab-bar-format list, the global mode string should be visible in the tab bar.
What actually happened?
The global mode string is not visible.
Describe your attempts to resolve the issue
The issue can be resolved by changing the face definition of tab-line.
Currently, tab-line is define as:
(tab-line :background bg-alt :foreground bg-alt)

Changing the foreground color to a different color than the background one will make the global mode string visible.
Steps to reproduce
Run this while any doom emacs theme is loaded:
(setq tab-bar-format '(tab-bar-format-tabs-groups
                       tab-bar-separator
                       tab-bar-format-align-right
                       tab-bar-format-global))

(display-time-mode 1)

I expect the clock to be shown in the upper right corner of the frame.
Package commit
83fd9545c2823b4b2610947802fa3a52995517b4
System Information
Emacs 28.1, on all platforms.

I am also experiencing this issue, and in fact opened a duplicate issue before seeing this one. This can be temporarily resolved by adding
(custom-set-faces
      `(tab-bar
        ((t (:background ,(doom-color 'bg-alt) :foreground ,(doom-color 'fg-alt)))))))

To init.el, for anyone else experiencing this issue.


Error on UMLS connection

Hi Tim,
I was trying to configure following the steps on the tutorial, however UMLS replaced the username and password to SSO, is this tutorial still valid?
What should be the configuration for the following file?
https://github.com/tmills/ctakes-docker/blob/master/env_file_sample.txt
Thanks

Certificate was added to keystore
>>> Setting defaultBrokerURL to:https://<IP_ADDRESS>:61616
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/apache-ctakes-4.0.0/lib/slf4j-log4j12-1.6.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/apache-uima-as-2.10.3/apache-activemq/lib/slf4j-log4j12-1.7.13.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
  UMLS Account at https://uts-ws.nlm.nih.gov/restful/isValidUMLSUser is not valid for user <username> with <password>

org.apache.uima.resource.ResourceInitializationException: Initialization of annotator class "org.apache.ctakes.dictionary.lookup2.ae.DefaultJCasTermAnnotator" failed.  (Descriptor: file:/apache-ctakes-4.0.0/desc/ctakes-dictionary-lookup-fast/desc/analysis_engine/UmlsLookupAnnotator.xml)
	at org.apache.uima.aae.controller.PrimitiveAnalysisEngineController_impl.initializeAnalysisEngine(PrimitiveAnalysisEngineController_impl.java:376)
	at org.apache.uima.aae.UimaAsPriorityBasedThreadFactory$1.run(UimaAsPriorityBasedThreadFactory.java:141)
	at java.lang.Thread.run(Thread.java:748)
Caused by: org.apache.uima.resource.ResourceInitializationException: Initialization of annotator class "org.apache.ctakes.dictionary.lookup2.ae.DefaultJCasTermAnnotator" failed.  (Descriptor: file:/apache-ctakes-4.0.0/desc/ctakes-dictionary-lookup-fast/desc/analysis_engine/UmlsLookupAnnotator.xml)
	at org.apache.uima.analysis_engine.impl.PrimitiveAnalysisEngine_impl.initializeAnalysisComponent(PrimitiveAnalysisEngine_impl.java:274)
	at org.apache.uima.analysis_engine.impl.PrimitiveAnalysisEngine_impl.initialize(PrimitiveAnalysisEngine_impl.java:172)
	at org.apache.uima.impl.AnalysisEngineFactory_impl.produceResource(AnalysisEngineFactory_impl.java:94)
	at org.apache.uima.impl.CompositeResourceFactory_impl.produceResource(CompositeResourceFactory_impl.java:62)
	at org.apache.uima.UIMAFramework.produceResource(UIMAFramework.java:279)
	at org.apache.uima.UIMAFramework.produceAnalysisEngine(UIMAFramework.java:407)
	at org.apache.uima.analysis_engine.asb.impl.ASB_impl.setup(ASB_impl.java:256)
	at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initASB(AggregateAnalysisEngine_impl.java:435)
	at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initializeAggregateAnalysisEngine(AggregateAnalysisEngine_impl.java:379)
	at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initialize(AggregateAnalysisEngine_impl.java:192)
	at org.apache.uima.impl.AnalysisEngineFactory_impl.produceResource(AnalysisEngineFactory_impl.java:94)
	at org.apache.uima.impl.CompositeResourceFactory_impl.produceResource(CompositeResourceFactory_impl.java:62)

You're right, the tutorial is probably not valid anymore. I have a new
docker image I use for the SSO change, but it probably won't fit with the
rest of the tutorial.
On Mon, Dec 28, 2020 at 3:12 PM Pedro Santos<EMAIL_ADDRESS>wrote:

Hi Tim,
I was trying to configure following the steps on the tutorial, however
UMLS replaced the username and password to SSO, is this tutorial still
valid?
What should be the configuration for the following file?
https://github.com/tmills/ctakes-docker/blob/master/env_file_sample.txt
Thanks
Certificate was added to keystore

Setting defaultBrokerURL to:https://<IP_ADDRESS>:61616
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/apache-ctakes-4.0.0/lib/slf4j-log4j12-1.6.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/apache-uima-as-2.10.3/apache-activemq/lib/slf4j-log4j12-1.7.13.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
UMLS Account at https://uts-ws.nlm.nih.gov/restful/isValidUMLSUser is not valid for user  with

org.apache.uima.resource.ResourceInitializationException: Initialization of annotator class "org.apache.ctakes.dictionary.lookup2.ae.DefaultJCasTermAnnotator" failed.  (Descriptor: file:/apache-ctakes-4.0.0/desc/ctakes-dictionary-lookup-fast/desc/analysis_engine/UmlsLookupAnnotator.xml)
at org.apache.uima.aae.controller.PrimitiveAnalysisEngineController_impl.initializeAnalysisEngine(PrimitiveAnalysisEngineController_impl.java:376)
at org.apache.uima.aae.UimaAsPriorityBasedThreadFactory$1.run(UimaAsPriorityBasedThreadFactory.java:141)
at java.lang.Thread.run(Thread.java:748)
Caused by: org.apache.uima.resource.ResourceInitializationException: Initialization of annotator class "org.apache.ctakes.dictionary.lookup2.ae.DefaultJCasTermAnnotator" failed.  (Descriptor: file:/apache-ctakes-4.0.0/desc/ctakes-dictionary-lookup-fast/desc/analysis_engine/UmlsLookupAnnotator.xml)
at org.apache.uima.analysis_engine.impl.PrimitiveAnalysisEngine_impl.initializeAnalysisComponent(PrimitiveAnalysisEngine_impl.java:274)
at org.apache.uima.analysis_engine.impl.PrimitiveAnalysisEngine_impl.initialize(PrimitiveAnalysisEngine_impl.java:172)
at org.apache.uima.impl.AnalysisEngineFactory_impl.produceResource(AnalysisEngineFactory_impl.java:94)
at org.apache.uima.impl.CompositeResourceFactory_impl.produceResource(CompositeResourceFactory_impl.java:62)
at org.apache.uima.UIMAFramework.produceResource(UIMAFramework.java:279)
at org.apache.uima.UIMAFramework.produceAnalysisEngine(UIMAFramework.java:407)
at org.apache.uima.analysis_engine.asb.impl.ASB_impl.setup(ASB_impl.java:256)
at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initASB(AggregateAnalysisEngine_impl.java:435)
at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initializeAggregateAnalysisEngine(AggregateAnalysisEngine_impl.java:379)
at org.apache.uima.analysis_engine.impl.AggregateAnalysisEngine_impl.initialize(AggregateAnalysisEngine_impl.java:192)
at org.apache.uima.impl.AnalysisEngineFactory_impl.produceResource(AnalysisEngineFactory_impl.java:94)
at org.apache.uima.impl.CompositeResourceFactory_impl.produceResource(CompositeResourceFactory_impl.java:62)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tmills/ctakes-docker/issues/37, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABXODP743V6D53BDDILDI3SXDRCZANCNFSM4VMN3TMQ
.

Would you have the process to build the file with the SSO or the docker build file? And the new repo

Here is the repo:
https://github.com/Machine-Learning-for-Medical-Language/ctakes-covid-container
On Mon, Dec 28, 2020 at 3:23 PM Pedro Santos<EMAIL_ADDRESS>wrote:

Would you have the process to build the file with the SSO or the docker
build file? And the new repo
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tmills/ctakes-docker/issues/37#issuecomment-751853038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABXODLBHSU6F5JHQNZDOALSXDSNDANCNFSM4VMN3TMQ
.

Hi, is this repo still available anywhere? Thank you!

Hi @chen-yifu , I've re-created it so the link above should work now.

Thank you very much!


Announcement : Vault + Gopass

Not really an issue here.
Just to let you know I have created a vault token helper for gopass.
It's a kind of plugin to store your vault tokens in gopass.
https://github.com/frntn/vault-token-helper-gopass

Thanks for letting us know, that is pretty cool!


update release nodes and bump material version

Actually, it looks like the original commit 85135e60c7 made it into 1.20.5, which has already been released. So we should have bumped that version.


Add new output style for API overviews with PlantUML

Example:

Codecov Report

Merging #13 (a6b3249) into main (3090439) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        18
  Lines         1246      1242    -4
=========================================
- Hits          1246      1242    -4

Impacted Files
Coverage Δ

openapidocs/mk/jinja.py
100.00% <100.00%> (ø)

openapidocs/utils/web.py
100.00% <0.00%> (ø)

openapidocs/commands/docs.py
100.00% <0.00%> (ø)

openapidocs/mk/v3/__init__.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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