gitlab: race condition of restarting coupled DB services ends up in gitlab being inactive

Describe the bug
In the NixOS gitlab module, the fleet of Gitlab component services is strongly coupled to the (local) postgresql and – via sidekiq – to Redis. On a systemd unit dependency level, this happens via a bindsTo relationship and causes the other services to be stopped and (re)started when just one of them is instructed to do so.
Unfortunately, there can be several edge cases where the system ends up with not all of the components being in the desired state (mostly up).
Steps To Reproduce
Steps to reproduce the behavior:
This issue is a subtle race condition, so I choose to describe the general problematic setup instead of trying to engineer a reproducer:
I've encountered a situation where both redis-gitlab.service and postgresql.service where restarted (stop and start) at system switch time due to changed configuration. This triggered Gitlab to be stopped as well. Unfortunately the service(s) was in its startup (Activating) phase and stayed inactive.
Expected behavior
The expectation is that Gitlab comes back after the dependency restart.
Potential Fixes
Instead of continuing to tune the strongly-tied, evermore complex inter-service dependencies I propose to decouple postgresql and redis from the rest of the gitlab services. I'll open a WIP pr on this.
Notify maintainers

@globin @krav @talyz @yayayayaka @yu-re-ka
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.146, NixOS, 23.11 (Tapir), 23.11.dev3109.af2cc4b3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs`

Add a :+1: reaction to issues you find important.

Here's my proposal on reducing the coupling between DBs and the rest of gitlab, currently with failing tests unfortunately: 286532


Topic/v0.4.2/issue 118 add expl shuffix dict upd script

Fixes #118
やったこと

richcat/_ext2alias_dic_generator.py で定義された辞書の更新を、developブランチへのmerge時に自動で行うようにした

@ShotaroKataoka
https://github.com/richcat-dev/richcat/pull/119#discussion_r799115384 を反映しました。これで問題なければmergeしようかなと思います。

Codecov Report

Merging #119 (e0a6a08) into develop (f031c92) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #119   +/-   ##
========================================
  Coverage    40.06%   40.06%
========================================
  Files           12       12
  Lines          302      302
========================================
  Hits           121      121
  Misses         181      181

Flag
Coverage Δ

unittests
40.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

richcat/modules/consts/_ext2alias_dic_generator.py
13.33% <ø> (ø)

richcat/modules/help.py
66.66% <ø> (ø)

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 7d321e0...e0a6a08. Read the comment docs.

LGTM


[Question] How to disable linter red color on the sidebar

Hi, there is a way to disable the linters related red highlights from the sidebar?

There is a setting that affects sidebar problem coloring :
// Show Errors & Warnings on files and folder.
"problems.decorations.enabled": true,

Simply add this to your user settings and voilà.
Also, next time, follow this guideline for submitting issues (questions are supposed to go to stackoverflow).

Thank you.


[FIX] Make hsvs workflow work

Addresses #718 and numerous neurostars issues. This fixes the hsvs workflow (specifically the FreeSurfer ingression) which got very broken when we moved to the nipreps config model.

Codecov Report
Attention: Patch coverage is 31.25000% with 11 lines in your changes missing coverage. Please review.

Project coverage is 29.85%. Comparing base (6860e52) to head (6c9bbdf).

Files
Patch %
Lines

qsiprep/interfaces/bids.py
14.28%
4 Missing and 2 partials :warning:

qsiprep/workflows/recon/anatomical.py
0.00%
3 Missing :warning:

qsiprep/interfaces/freesurfer.py
33.33%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #784   +/-   ##
=======================================
  Coverage   29.84%   29.85%
=======================================
  Files          97       97
  Lines       14604    14613    +9
  Branches     1885     1887    +2
=======================================
+ Hits         4359     4362    +3
- Misses      10117    10121    +4
- Partials      128      130    +2

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


error codes

Tracking progress:

[ ] nutshell
[x] CDK
[ ] cashu-ts
[ ] nutmix
[ ] gonuts
[ ] ...

Error codes not defined in nutshell.
10002 => Self::BlindedMessageAlreadySigned,
10003 => Self::TokenNotVerified,
11002 => Self::InsufficientFee,
12002 => Self::KeysetInactive,
20002 => Self::TokensAlreadyIssued,
20003 => Self::MintingDisabled,
20005 => Self::QuotePending,
20006 => Self::InvoiceAlreadyPaid.

close for #161


Valkyrien Computers Crash

crash-2023-11-18_19.18.10-fml.txt
Minecraft crashed with Valkyrien Computers, crash report above ^

VS computers doesn't support the newest version of VS2 as far as I know.
Also please use the bug report template next time.


WIP Globally Unique FATE Transaction Ids

A WIP PR for #4044. The end goal is to have the stronger type FateId replace the current representation of a transaction id (which is just a long). This was brought about from the addition of the AccumuloStore class - there are now two fate instance types associated with a transaction - META (for ZooStore) or USER (for AccumuloStore). FateId is a new class which includes the FateInstanceType and the transaction id. Creating the PR now to get feedback as I go as this will require a lot of changes.
TODO:

[x] Create new class FateId to replace FateTxId
[x] Change the stores to use FateId. Start with ReadOnlyFateStores. Resolve issues stemming from these changes.
[x] Change Fate to use FateId. Resolve issues stemming from these changes.
[ ] Change Repo to use FateId. Resolve issues stemming from these changes.
[ ] AdminUtil and Admin need to use FateId. Deferred for now. Need #4168 completed first.
[ ] Delete FateTxId when all uses have been replaced with FateId

In ReadOnlyFateStore will need to change the following to
void runnable(AtomicBoolean keepWaiting, LongConsumer idConsumer);

void runnable(AtomicBoolean keepWaiting, Consumer<FateId> idConsumer);

In ReadOnlyFateStore will need to change the following to
void runnable(AtomicBoolean keepWaiting, LongConsumer idConsumer);

void runnable(AtomicBoolean keepWaiting, Consumer<FateId> idConsumer);

Ah okay. Yeah wasn't too sure exactly how I should go about changing this method. Will make suggested change.

These changes look good so far. Based on what you have done so far do you think it would be possible to break this work in multiple PRs?

Yes, this is my thinking of how it could be broken up:

A PR changing ReadOnlyFateStore.java and the issues stemming from this (current PR)
A PR resolving all the issues from deleting FateTxId.java (maybe combined with first)
A PR changing Fate.java and the issues stemming from this
A PR changing Repo.java and the issues stemming from this
Potentially more PRs

However, I'm not sure if this is the best way or if these PRs standalone would actually pass all the git checks/not break the project. Not sure how interlinked everything is.

Update:

New fromThrift() method in FateId.
Fate now uses FateId and resolved issues stemming from these changes.
AdminUtil and Admin do not have FateId fully incorporated yet. Deferred for now. Marked "TODO DEFERRED - ISSUE 4044" in code. Need #4168 completed first.
Resolved all compile errors.
Now that it compiles, I am able to run tests but have found that several hang indefinitely. Need to resolve this.
Updated the main PR comment to better break down overall goal into smaller sub goals.


aws: configure clocksource on instance first boot time

To use kvm-clock as clocksource on AWS Nitro VMs, run scylla_bootparam_setup on
instance first boot time, instead of AMI image creation time.
see: scylladb/scylla#7444

This is part of https://github.com/scylladb/scylla/pull/7468

Looks good to me, @avikivity can you also please have a look?

I think I need to close this due to https://github.com/scylladb/scylla/pull/7468#issuecomment-714575429, I'll make another PR


Save the curl error in a public var to check if everything is OK

Hello,
Thank you for contacting us.
We have improved the cURL error handling by throwing a runtime exception.
Please refer: https://github.com/mailin-api/mailin-api-php/blob/master/V2.0/Mailin.php#L50
Regards,
SendinBlue Team


Source ~/.shellrc automatically if it exists

Fixes #95.

Do you think this should only be done as an option in interactive mode? I'm not sure how useful it would be but we can also add something like
cargo r ./scripts/script.sh --withrc

Thanks for the review! I had to add an option --norc to skip sourcing it if requested (just like Bash allows), as I quickly found out that sourcing every time breaks my MSVC loading script, as my ~/.shellrc sets the $PATH variable by overriding it (until #90 is fixed), so I need to skip sourcing it.

It currently is an option in interactive mode only I think:
        execute(&script_text, &mut state).await?;
        if options.interact {                                                               interactive(Some(state), options.norc).await?;
        }

The script gets executed without sourcing .shellrc, and then if interactive mode is started, it will source it.
I think this takes care of most of the use cases, and we can since bash doesn't even have --withrc, we probably don't need it, but if we do, we can.


Static visitAny / visitAsync methods should be deprecated and foldAny / foldAsync alternatives provided

Create foldAny / foldSync equivalents, deprecate the original methods for removal in cyclops 12.

Released in 10.1.0


Increase default retry interval for file SD

The automatic refresh is a safety mechanism in case
file watches fail. As they seem to be working well the
interval can be increased.
@brian-brazil

3 minutes seems like an odd number, I'd bump to 5 or 10

5 still seems fine to me. 10 is a bit much if anything goes wrong.


Conditional continuous charging session

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I would like to be able set a minimum saved amount of money when switching to non-continuous charging.
Describe alternatives you've considered
None
Additional context
When ever a schedule is calculated and switch.ev_smart_charging_continuous_charging_preferred is off, the integration should do two schedules. One with continuous charging and one where the charging is split into different session, and then compare the amount of money saved by not use a continuous charging. This can be either the total amount (a bit harder) or the difference in average price for the selected hours. And only if the save is greater than the set minimum save, it should do non-continuous charging.
If you go for the total amount saved suggestion, there will be another nice feature for free. That is that there can be a sensor with the estimated total cost for the next planned charging period.
My thought is that there is no meaning in postponing the the final charging if there is not so much to save. As it is now there can be a gap in the charging and you only saves one or two cents in total. Then there is a question if you really did save those cents, because there is a small cost to start and stop the charging depending on battery temperatures. How much, though, is not known to me...

As you say, there is an extra cost to for each start and stop depending on the temperature. And without knowing what that cost is, I feel sceptical about implementing what you suggest. Instead, I would suggest something more simple. For example an automation that turns on and off switch.ev_smart_charging_continuous_charging_preferred depending on the outside temperature (assuming the car is parked outside). Turn on when it's cold, turn off when it's warm.

That is one valid approach. I think I have to do some more investigation in this...

I close this for the time being.


Unable to mute hashtags/create filters

Unable to mute a hashtag from Tusky.
The error just states "error muting hashtags"
I get the same problem creating a filter manually, but I think this might be what Tusky does internally for muting a hashtag anyway?

Yes, Takahē does not yet support server-side filtering, so this will not work until we implement it. I was hoping to do it at the same time as we do a plugin API for more advanced incoming post filtering, but we'll see.


Add "Login with GitHub" feature

Is your feature request related to a problem? Please describe.
Nowadays, logging into websites with your GitHub account has become common and convenient. So adding this feature will help in improving Login procedure and User Experience.
Describe the solution you'd like
Add Login with GitHub option.
Additional context
Connect your app to GitHub
Are you working on this? (Yes/No)
Nope. For GCI 2019

@sal2701 Could u make this a GCI task?


build(deps): bump fastlane deps

This PR resolves []
Description
bumps fastlane deps

#nochangelog

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against 026bdb524fd30a9f388036a678e87b92bcd39b0c


Consider a @react-three/jolt-addons package

@react-three/rapier went down the path of having two packages:

@react-three/rapier - which as much as possible just wraps rapier.js
@react-three/rapier-addons - opinionated helpers built using @react-three/rapier

I think this is a good split. It keeps the core library light, and makes more sense for those who want to roll their own Attractor or VehicleFourWheel, character controller, etc.

As discussed in discord I agree.
I think an addons package is fine for unique things like you mentioned but I also think we should have a controllers package that can hold not only the character controller but also the vehicle and camera controllers.
would you mind setting up the packages/npm flows?
I wonder if we can do that with GitHub actions…

Created and published @react-three/jolt-addons and @react-three/jolt-controllers` now 🙂


[now-init] Check for dir existence prior to download

This is a minor enhancement to the way now init works.
This PR will check if the folder exists prior to fetching the tar.gz from the server so that the command can fail fast.

Codecov Report

Merging #2379 into canary will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           canary   #2379   +/-   ##
======================================
  Coverage    10.9%   10.9%
======================================
  Files         250     250
  Lines        9031    9031
  Branches      991     991
======================================
  Hits          985     985
  Misses       7945    7945
  Partials      101     101

Impacted Files
Coverage Δ

src/commands/init/init.ts
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Not working with Ionic3 - (void 0) is not a function

I am able to get it working with a new Angular4 application, but cannot get it to work with a new Ionic3 applications (using Angular4). I've created a repo to demonstrate: https://github.com/butters16/google-chart-using-ionic3
Steps to reproduce

create blank project using ionic3 cli
install google-chart npm install angular2-google-chart
modify app.module.ts to import GoogleChart and add it to declarations
override home.ts compnent to look like angular2-google-chart README

Error looks like this
platform-browser-dynamic.es5.js:170 Uncaught TypeError: (void 0) is not a function
    at Object.<anonymous> (platform-browser-dynamic.es5.js:170)
    at __webpack_require__ (bootstrap 3cf6321…:19)
    at Object.<anonymous> (src async:7)
    at __webpack_require__ (bootstrap 3cf6321…:19)
    at Object.<anonymous> (main.js:111954)
    at __webpack_require__ (bootstrap 3cf6321…:19)
    at bootstrap 3cf6321…:65
    at bootstrap 3cf6321…:65

Same problem since Ionic 2.

could you solve this problem?

Short answer: no, I could not solve the problem so I am not using angular2-google-chart.
Long answer: I'm turned off to using projects that wrap existing javascript functionality for angular consumption. There seems to be this constant cat-and-mouse game of waiting for support after each release and then having to debug the inconsistencies myself. Even if everything works, libraries like Google Charts already have tons of great documentation with examples, but I can't use any of it until I figure out how to translate it for the wrapper library. I see some value in the wrappers, but if they don't work right away or impose too much deviation from the original library, I just fallback to using the straight javascript as I did in this case. I've included the snippets below for how to do that.
index.html

some-component.html

some-component.ts
declare var google;
let chart = new google.visualization.ComboChart(document.getElementById('some-unique-id'));
chart.draw(data, options);
Admittedly, this could use some error handling for if Google Charts does not load in time (or ever).


Bugfix/distance measurement ray3d

Temporarily fix a problem in core-geometry where Ray3d.createStartEnd captures the origin point instead of copying it. This solves an issue where the DistanceMeasurement's start point was being modified when it shouldn't have.
(This will eventually be fixed with PR #4012 on itwinjs-core.)

@a-gagnon you might want to backport that core fix pr to the release/3.3.x branch if you need it now, unless you want to wait til 3.4

@a-gagnon you might want to backport that core fix pr to the release/3.3.x branch if you need it now, unless you want to wait til 3.4

All good, I just cloned the point as a temporary workaround for the measure-tools package. We'll pick up on this change eventually.


Add tests for errors

Following this issue: https://github.com/meilisearch/meilisearch-rust/issues/159
Add tests to ensure that errors are parsed successfully

Some tests were added but more should be added: https://github.com/meilisearch/meilisearch-rust/blob/dbd0c17301faa0683bc6e7460e5a696c76864dd9/src/errors.rs#L231

Doesn't #356 close this? If not, what more is required?

Hey @romilpunetha . Thanks for the interest :)
I just checked, there seems to be a lot of new tests. If you don't find any relevant test to add, we can close this issue :)

I believe we can.
The only two error attributes that do not have a test case are InvalidTenantToken and Yaup because they were particularly hard to test. However, they are not used anywhere in the codebase. For all the other attributes tests exist.

They are thrown by the external libraries. If you manage to test them, you can open a PR :)

@bidoubiwa Done.


Process Data Packages - Step 12 - Incorrect Screenshot

The screenshot associated with step 12 is incorrect:
Step 12: "To create a better sequence, select the Customers entity, and then update the value of the Execution unit field from 1 to 2. This change helps guarantee that customer groups are imported before the Customers entity is run."
... Yet the screenshot shows Customers with EU1, and Customer Groups with EU2; also Customer Groups has changed Sequence to 0.

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

ID: d8efea16-9cb3-e281-e8bd-373f0cf37735
Version Independent ID: 9772f535-3b25-5818-733d-c094b79e3372
Content: Process and consume data packages in Finance and Operations solutions - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/lcs-solutions/process-data-packages-lcs-solutions.md
Service: dynamics-ax-platform
GitHub Login: @kfend
Microsoft Alias: omarc

Bug # 299938 assigned for investigation


github preseed download

line 105 needs to be changed to...
https://raw.githubusercontent.com/netson/ubuntu-unattended/raw/master/$seed_file

Done! sorry it took so long to fix, somehow I missed this issue... :+1:


Display analysis owner/creator for users the analysis is shared with

Hello - I hope you are doing well. It would be useful if users that an analysis is shared with are able to see the analysis creator/owner user email on the analysis page. It would also be helpful to have the analysis creator/owner user email in the analysis list (see previous analyses). Thanks in advance!

Thank you for the suggestion!


Type Hinting

Is Type Hinting still a no-go for NLTK? - I see in setup.py that the minimum Python version required is 3.5, which supports Type Hinting.
I have some type hints created for the classification files for better understanding them myself. If possible, I can create a PR.

Thanks @sarthakTUM, please go ahead


Design: Scholarships

Please use one of the following templates to create new issues:
For a new feature:
As a [VISITOR, ADMIN, ETC],
When I [CONTEXT/LOCATION FOR FEATURE],
It should [DESCRIBE FEATURE]
Some UX notes are captured by @jeremysb1 in #467.

Hi! I would like to take this design task.
From the #467 Jeremy's screencast, there is only one thing left in his request:
As Visitor/Student,
When I click on Scholarship page
The scholarships of coding schools should display right after the video.

All yours!

Hello!
I did some user test on the original design to find some pain points.
Here is the first draft: http://operation-code-scholarship.webflow.io/
What do you think? @hollomancer

@tpham045, will take a look!

This looks great! @rickr, what do you think?

Just talk to @rickr today. It seems that we plan to build/convert a new front end (repo operationcode_frontend) using React.
I think it is better to organize these Design tasks over there.
@hollomancer What do you think?

Agreed - we can recreate there.


Improve the sample of browser.location.href assertion.

To reduce misleading for newbies.
https://github.com/assaf/zombie/issues/903

That code just illustrates how hostname and path are merged together, it's not intent on being a Mocha test case.


Padding between windows should not be doubled

Describe the bug
Padding between windows should not be doubled, they should have same value with edge padding.
To Reproduce
Steps to reproduce the behavior:

Press Meta+T
Adjust padding > 5
Press Meta+T again
Open up some windows
Observe Padding between windows, appears larger, 2*padding value

Expected behavior
Padding between windows should be overlapped to maintain simetri with edge padding

This is actually standard Kwin tiling behavior and I cannot do anything about it. But now you made me see it and I'll never be able to unsee it. I don't like it either.

https://invent.kde.org/plasma/kwin/-/merge_requests/4113
I think that will be fixed soon :)


Missing funtions

isfunctionhooked is missing, also hooksignal and issignalhooked would be nice to have

As far as I know, hooksignal and issignalhooked are not currently possible within the Roblox API. isfunctionhooked is possible however. This is going to be included in xAPI v4.1.

You could wrap everything and replace the Connect method of every signal. Heres how I did it if u wanna take a look at it, but this code will cause eye cancer so proceed with caution:
Signal Hooking.txt

I could wrap, yes, but I want xAPI to actually do something. By wrapping you can technically do it, but this will only work when the connection is inside of the script. If you want that, check out xAPI v3.7.2. If you want things to actually work, then you must hook the connection and the hook must be activated through any script, not only the "exploit script". This cannot be done by simple Instance wrapping.


setImmediate not supported in most browsers

Prerequisites

[X] I have read the documentation.

Description
setImmediate function is causing validation to break because it is not implemented in most browsers (and looks like it will not become standard):
https://developer.mozilla.org/en-US/docs/Web/API/Window/setImmediate
Steps to Reproduce
I've tracked the error to this line: https://github.com/mozilla-services/react-jsonschema-form/blob/210a94421f4b034d46bac38cbc1f602358afc12a/src/utils.js#L381

Submit invalid form data or type in fields of a react-jsonschema-form
See error in console in any browser from above docs

Expected behavior
No error.
 I suspect you're building your own files from the package lib files, which doesn't expose a polyfill for it...
I know both browserify and webpack provide it by default, but maybe you're using something else? Or do use options like --bare or no-builtins? I'm really curious :)
Back to the issue, if we don't use setImmediate, what should we use instead in your opinion? setTimeout(fn, 0)? requestAnimationFrame?

Could something be good enough?
--- a/src/utils.js
+++ b/src/utils.js
@@ -19,6 +19,10 @@ import ColorWidget from "./components/widgets/ColorWidget";

 const RE_ERROR_ARRAY_PATH = /\[\d+]/g;

+const setImmediate = global.setImmediate ||
+                     global.requestAnimationFrame ||
+                     global.setTimeout;
+
 const altWidgetMap = {
   boolean: {
     radio: RadioWidget,

Also, there's this, and this article defending the setimmediate initiative. Hmm.

Yeah I was trying to avoid adding another dependency.  I like the solution you have above because it serves as a fallback if the global isn't available.  I personally don't have an opinion about using setImmediate, it just looks like there is a lot of friction about it and no guarantee it will be implemented.
Back to your earlier question.  I'm using browserify, but no options like --bare or no-builtins, so not sure why it is not being included.  I just used npm to grab react-jsonschema-form.

I am running npm rebuild --no-bin-links due to this issue: https://github.com/npm/npm/issues/9901

@rossnanop could you please try building your project using the branch from #190 and tell me if that works for you? Thanks.

No longer get the error, but I am unable to type anything into the form.  It looks like the callback is just resetting the state to the default state?  This may be related to my issue (or a separate issue possibly).
index.js:38370 Warning: `value` prop on `input` should not be null.
Consider using the empty string to clear the component or `undefined` for uncontrolled components.

React 15 now warns on null value props on input because they want to clarify intent.  In the BaseInput widget and a few other places, the check is:
typeof value === "undefined" ? "" : value

which doesn't cover value === null because typeof null === object.  I guess it just depends on if you want to allow people to pass null to react-jsonschema-form or not.  However, in some cases, you just want FormData to be empty if you don't have default values.

@rossnanop I've updated #188 to use a decent polyfill to setImmediate, please check it out.
For your other issue, it's probably worth filing a new separate one.

The fix has landed as part of v0.30.0.


WIP http api documentation

Uses https://apiblueprint.org, which is formatted markdown which
includes embedded URI templates and MSON to make testable,
language-agnostic api docs.

Closing this as the community is more interested in open api initiative/swagger v2


mkdocstring' setup_commands isn't supported by the new Python handler

Describe the bug
I was taking a quick glance at your repository (nice cookiecutter!) and noticed you're using setup_commands in the configuration of mkdocstrings in mkdocs.yml. This option isn't supported by the new Python handler (which isn't really new anymore).
Instead of using setup_commands to insert the right paths into sys.path, you should use the paths option, which is much more robust :smile:

Thanks for raising this! (and thanks for developing mkdocstrings). I resolved this in https://github.com/fpgmaas/cookiecutter-uv/pull/7


Update CONTRIBUTING.md

Windows command added

 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.

Hey @murtaza-ghadiali it looks like our test suite currently doesn't work correctly for forked PRs. We'll need to look into that, but definitely don't want to gate this PR on it when it has nothing to do with the test suite, so I will merge it in now! :)


[ISSUE#1890][Functional - Open Bot Dialog] The focus goes on the disabled controls and functioning by space/return.

Solves #1890
Description
With these changes, the PrimaryButton and LinkButton elements will be hidden to screen readers while disabled.
In the issue report, the tester says the disabled buttons close the dialogs when pressing space, this happens because the Voiceover navigation mode goes through the elements even if they are not focusable by the keyboard, so the keyboard only follows it when it lands in a focusable element, and as a disabled button is not focusable, the keyboard focus remains in the last focusable element, in the case shown by the tester is the cancel button, so when pressing space bar or enter, the cancel button is clicked. This won't keep happening as Voiceover can't navigate to the Connect button when it is disabled
Changes made
We added an attribute of aria-hidden='true' to the PrimaryButton and LinkButton elements when they have its disabled property set to true, this way the Voiceover screen reader skips them when navigating.
Testing
In the following recording, you can see an example of a LinkButton and a PrimaryButton being hidden from Voiceover while they are hidden.

Coverage decreased (-0.005%) to 68.524% when pulling 3bd1f68275c510c9ad8f0779afc0e4fb3838532c on fix/focus-disabled-buttons-voiceover into c2bee8bf2f68f790aee0a5ab18a66d4d7dc1b1d4 on master.

Hi @tonyanziano ,
What we understood that is happening is that Voiceover (navigation with VO + arrows), navigates through every visible element, even if it is not focusable by the keyboard. Because of this, the keyboard focus follows the VO navigation only when it lands on a place the keyboard can focus, resulting on this case, that when voiceover is placed in the save button, the keyboard focus remains in the cancel button, so when the user press space, he is actually pressing the cancel button.
We noticed that this behavior occurs everywhere, for example when navigating into a Checkbox and then to the text next to it, when pressing space, the Checkbox will be the one checked.
Because of this, to have disabled buttons still being visible to the user, we suggest making it focusable by the keyboard, so it can follow the voiceover navigation and if the user press space it won’t do any action.
Also, this behavior only happens with Voiceover (with VO + arrows). Other screen readers only navigate through visible objects focusable by keyboard, and disabling an element makes this object not focusable by default.


Response.prototype.text() returning empty string in some cases

Hi all,
I think i’ve found a bug with either Response or ReadableStream, can you check this out when you have a moment?
The following code doesn't work as expected. It should return the same string as the input, but instead it returns an empty string:
  const text = "foo";

  // Create a ReadableStream from string
  const stream1 = new Response(text).body;

  // Read from stream1 using Response.prototype.text();
  const textFromStream1 = await new Response(stream1).text();

  // textFromStream1 should be 'foo', but it's ''

I've put a repro of this up in a fiddle https://fiddle.fastly.dev/fiddle/750711db
This happens for me on the newest version of js-compute-runtime: 3.8.2

Something to note is that if I instead use a reader like this, I get the right result (so I'm currently using this as a workaround):
  const text = "foo";

  // Create a ReadableStream from string
  const stream2 = new Response(text).body;

  // Use a reader to read from stream2
  const stream2Reader = stream2.getReader();
  let textFromStream2 = '';
  const decoder = new TextDecoder();
  while(true) {
    const { done, value } = await stream2Reader.read();
    if (done) {
      break;
    }
    textFromStream2 += decoder.decode(value);
  }

  // textFromStream2 correctly contains 'foo'


Can this template add tags for every article?

if someone have many ariticles, he will need a tag to classify articles.

I have done implemented tags (I call them labels) in my blog. Take a look here: https://github.com/flexdinesh/dineshpandiyan.com/blob/dcea33e9fc20d4ef041127136e03c9cb28765c41/src/lib/markdoc/frontmatter.schema.ts#L3

Thanks, it's awesome.


Incompatibility with CraftTweaker

When running the most recent release of both CraftTweaker and CookingForBlockheads, CFB causes CT to crash out and shut down the server.
fml-server-latest.log.txt
When I disabled CFB, my server starts up with no issue.

Please see this issue ticket as well for additional information https://github.com/jaredlll08/CraftTweaker/issues/170

Looks like the problem lies with a furnace recipe that results in nothing. Cooking for Blockheads doesn't error-check in that case yet.

Specifically it seems to be some Refined Storage furnace recipes in your scripts (see minetweaker.log):
ERROR: Furnace.zs:16: Could not resolve <refinedstorage : quartz_enriched_iron>
ERROR: Furnace.zs:40: Could not resolve <refinedstorage : processor : 1>
ERROR: Furnace.zs:41: Could not resolve <refinedstorage : processor : 2>
ERROR: Furnace.zs:42: Could not resolve <refinedstorage : processor>
ERROR: Furnace.zs:43: Could not resolve <refinedstorage : processor : 6>


Add Dutch (nl_NL) job titles

This PR adds dutch job titles. I took them from this Wikipedia page.
The Wikipedia list had some job titles that ended in text between parentheses, this was usually either a female alternative (e.g.: Acteur (Actrice)), or a clarification (e.g.: Analist (laboratorium)). I've removed all the text between parentheses.

Please add a formatter and some documentation in the locale-specific section of the documentation.

@fzaninotto Done, let me know if there is anything else

Nice addition! Looks good to me

thanks!


v9 rootdir is invalid

Environment

Operating System: Linux
Node Version:     v18.20.3
Nuxt Version:     3.13.0
CLI Version:      3.13.1
Nitro Version:    -
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      future, compatibilityDate, modules, devtools, i18n
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/bobbiegoede-nuxt-i18n-starter-frlapt?file=nuxt.config.ts,package.json,app%2Fapp.vue
Describe the bug
nuxt.config.ts:
future: {
compatibilityVersion: 4,
},
compatibilityDate: '2024-08-08',
When using the new directory structure under nuxt 4, when the root directory of i18n is rootDir/i18n, the parsing error
[plugin:vite:import-analysis] Failed to resolve import "../app/locales/en.json" from ".nuxt/i18n.options.mjs". Does the file exist?
Additional context
No response
Logs
No response

According to v9 https://i18n.nuxtjs.org/docs/v9/getting-started There is no restructureDir attribute, 8.5.1 the latest stable version is there, I wonder if v9 is not merged completely

It seems like we haven't set the default value for restructureDir yet in v9, which is what it should be according to the docs https://i18n.nuxtjs.org/docs/v9/options/misc#restructuredir.
You can work around this temporarily by setting the options to its default: restructureDir: 'i18n' until we have fixed the default value 🙏 Let me know if this solves your issue! And let's keep this issue open while the default value has not been updated.

I have no problem downgrading to 8.5.1, because there is already a default value, because v9 does not currently set a default value, I have not tested whether it is possible to add it by myself, but nuxt.config.ts should also report a type error, and may indeed need to do it yourself

This should get resolved by #3088, you can try out the changes in the PR by installing its preview release with pnpm add https://pkg.pr.new/@nuxtjs/i18n@3088.


taboo.vim support

we agreed to put the plugins to the core folder instead of plugins for instance for:

into core/tabs/{tabs.py/talon}: taboo


Use ES6 destructuring for grabbing elements

OS : W10 X64
Zulip Version : 1.3.0-beta
Fixes #271

Please help me here, I hope there are no discrepancies in changing ipc to ipcMain in using destructuring.

Thanks for the fix @akashnimare

I don't think there would be any problem. Merging. Thanks, @Anshuman-Verma.


Add SO_PRIORITY support to setsockopt

Beyond need for more detailed commit message, looks good, thanks.

Bottom line: Never head about SO_PRIORITY. Please explain in the commit message what it does and what standard defines it.

You could just either do "man socket" or do some interweb seach.
http://man7.org/linux/man-pages/man7/socket.7.html and search SO_PRIORITY

You could just either do "man socket"

If it's not clear, it's not about "me", it's about having clear, self-describing messages for the changes.

Added more description to commit msg.

recheck

Rebased against latest upstream


Wrong stack frame file path for macOS universal app

Is there an existing issue for this?

[x] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
[x] I have reviewed the documentation https://docs.sentry.io/
[x] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?
Sentry Saas (sentry.io)
Electron SDK Version
4.17.0
Electron Version
30
What platform are you using?
MacOS
Link to Sentry event
No response
Steps to Reproduce
For errors coming from a macOS universal app, I get the full path in a stack trace:

Correct path from non-universal app:

app.getAppPath() returns the path to app.asar, but the files are located in app-arm64.asar or app-x64.asar, depending on the device architecture.
To build a universal app, I use @electron/universal with mergeASARs: false option.
Expected Result
...
Actual Result
...

Reading through this again, it sounds like app.getAppPath() is returning the incorrect path?

https://github.com/electron/universal/issues/113

Update
app.getAppPath() works fine. There is a simple js file in app.asar that imports the required files (located in app-arm64.asar or app-x64.asar, depending on the device architecture).
Here is a problem: https://github.com/getsentry/sentry-electron/blob/master/src/main/normalize.ts#L17, filename is not normalized.
Demo

// https://github.com/getsentry/sentry-javascript/blob/master/packages/utils/src/normalize.ts#L285
function normalizeUrlToBase(url: string, basePath: string): string {
    const escapedBase = basePath
      // Backslash to forward
      .replace(/\\/g, '/')
      // Escape RegExp special characters
      .replace(/[|\\{}()[\]^$+*?.]/g, '\\$&');

    let newUrl = url;
    try {
      newUrl = decodeURI(url);
    } catch (_Oo) {
      // Sometime this breaks
    }
    return (
      newUrl
        .replace(/\\/g, '/')
        .replace(/webpack:\/?/g, '') // Remove intermediate base path
        // eslint-disable-next-line @sentry-internal/sdk/no-regexp-constructor
        .replace(new RegExp(`(file://)?/*${escapedBase}/*`, 'ig'), 'app:///')
    );
  }

const APP_PATH = 'D:\\sentry_test_app\\app.asar' // app.getAppPath()

// 1. Non-universal app => path normalized
console.log(normalizeUrlToBase('D:\\sentry_test_app\\app.asar\\dist\\main\\index.js', APP_PATH))
// app:///dist/main/index.js

// 2. Universal app => path is not normalized
console.log(normalizeUrlToBase('D:\\sentry_test_app\\app-arm64.asar\\dist\\main\\index.js', APP_PATH))
// D:/sentry_test_app/app-arm64.asar/dist/main/index.js

The normalisation code relies on app.getAppPath() returning the correct app root path. The issue is that @electron/universal uses a stub app that routes to arch specific apps but this means that app.getAppPath() does not return the correct path.

This has already been fixed in the latest @electron/universal. Sorry, my mistake.


Italic text is not recognized when "/" is present inside

Normal case: *a*

[{"type":"italic","children":[{"type":"text","props":{"text":"a"}}]}]

Problematic case: *a/n*

[{"type":"text","props":{"text":"*a/n*"}}]

Expected: *a/n*

[{"type":"italic","children":[{"type":"text","props":{"text":"a/n"}}]}]

仕様通りの動作であり、バグではありません。
もし変更する場合は、MFM側の仕様変更が必要になります。

I think it would be okay if it is - as you said - per the MFM spec (please correct me if I had it wrong since I don't know much about Japanese). Also, I can wrap italic text with  which will not suffer from this problem.
I just feel it a little weird because it does not affect the bold notation:
// *a/b*
[{"type":"text","props":{"text":"*a/b*"}}]
// **a/b**
[{"type":"bold","children":[{"type":"text","props":{"text":"a/b"}}]}]

I agree that the current behaviour is very unexpected (and might thus be considered a bug). I personally consider the current behaviour for **bold** to be correct, so I think we should make the characters allowed inside *italic* the same as allowed in italic. (But *italic* should still require white space around it to be recognized.) Alternatively, we could remove the star-italic all together but that is not a good idea for backward compatibility.
(I did not quite understand; is the MFM specification somewhere else? Where would it have to be changed, since you said that "MFM itself has to be changed". Maybe that is just my translator not working correctly.)

@Johann150
以下にmfm.jsを実装するために作成された仕様があります。
https://github.com/misskey-dev/mfm.js/blob/develop/docs/syntax.md

Any update on this?

Any update on this?

@ThatOneCalculator Per marihachi, it seems to be expected rather than a bug:

仕様通りの動作です。これはバグではありません。

A workaround is to use text as instructed by the MFM docs.

I personally disagree, as this isn't the behavior of other standard markdown parsers and confuses users.

I join the discussion to say actual behaviour, I described into #134, it's confusing for people writing Latin / European languages.
Although in this repository MFM is never addressed as Markdown For Misskey in the source code, or documentation, neither when the repository was renamed #30 (2021/03).
I want to note that since 2021/09 the Misskey.io Wiki Pages are claiming MFM is Markdown For Misskey, and not a different thing, for example *Markup For Misskey.
This is the origin of confusion. Since the beginning, people expect to have markdown into Misskey. At least a basic subset, made of bold, oblique, links and quote. (No support for bullet / numbered lists)
So I end up asking for oblique text rendering to be fixed. Please.

The Firefish project will be addressing this in libmfm

That software can't be used in this project because is lacking of a free software license compatible with AGPLv3.
If your code incorporated misskey-dev/mfm.js code, it must be released at least under MIT license or kept private.


[bug]: Update from 3.1.0 to 3.3.0post3 fried the installation

Is there an existing issue for this?

[X] I have searched the existing issues

OS
macOS
GPU
mps
VRAM
32GB system
What version did you experience this issue on?
3.3.0post3
What happened?
When using commandline interface to update to latest (3.3.0post3), the installation is now crashing on all menu options. Output was difficult to capture so I recorded a movie in case it is of value.
https://github.com/invoke-ai/InvokeAI/assets/11711050/83ca3fb3-4502-4e86-8f22-f9dc06ac6bc6
Screenshots
No response
Additional context
Log this and perhaps with enough data the AI can sense a pattern:

Contact Details
No response

Thanks for the star chart :D
It looks like the startup is getting hung up on a checkpoint safetensors model - or something it thinks is a safetensors model. Did you make any changes to models?

No, I hadn't. I just sort of leave it running on my machine and connect to the web interface when I want to render something. I wanted to see if there were new models to play with today so hit ctrl-c to go back to the menu, got a crash in the add models interface (which I didn't record), then I the updated to the latest version of invokeai - and that's the state it's left in.  Previously I had been on civitai and huggingface installing a few Loras to play with, so I definitely had some extras installed - perhaps thats the source of the crash.
Probably it seems like I'm blowing the directory away and re-installing. I haven't checked - do the docs have instructions as to what directory(ies) to save to keep ones data?

You can reinstall in the existing location without deleting any data - it should keep everything intact for you

Clearing the models was the ticket. :)


Consider using GitHub Releases in this repo to announce new releases.

@crd477 thanks for the suggestion. We can certainly consider using GitHub Releases in this repo to announce new releases. We will explore that for the January PSU if possible.
Originally posted by @brunoborges in https://github.com/microsoft/openjdk/discussions/34#discussioncomment-1951313

Here's the release page we will be using: https://github.com/microsoft/openjdk/releases/tag/untagged-6aed32b54f44432fccbf

Alright, release for the January updates has been submitted. Those watching this repo get a notification.
https://github.com/microsoft/openjdk/releases

I noticed that there have been subsequent releases that didn't go through this process.  As a result, I didn't get notified.  Was this unintentional?
Thank you!

@brunoborges - Possible slip in our process?


[Request for consideration/discussion] enhancement: Nicer current space indicator

These changes introduce two things:

Switch from an underline indicator for the current space to simply
colouring the icon
Provide a space_icon_color config option to set the colour of the
current space icon

Figured I'd submit these for consideration :) Quite opinionated, so I'm open to discussion on whether we might want to provide an option to configure either the current underline, or the coloured icon as the current space indicator. I personally find the underline indicator to be too hard to notice.

I like it.


add plugin custom class name

This plugin allows to replace Prism default class names (like .comment, .string, .property, etc) with your defined ones (like .editor__comment (BEM style) or ._7sh3a (CSS Modules hashed class)).
For more information, please see the plugin page: http://quoinex.github.io/prism/plugins/custom-class-name/
Issue: #947

@LeaVerou @zeitgeist87 I have some time to write the plugin as discussed previously with you two. Please take a look at it.
I also have one problem here: I create several separate files in my plugins folder to demo how this plugin works, but it seems that make uglify failed to run. Do you have any advice to fix this pull request?

Sorry for the late response. I was quite busy last month. Looking over your source code I had the following idea. Your plugin could also offer the option of a prefix to all classes. Something like a namespace:
Prism.plugins.customClassName.prefix("mynamespace-");

That way the user would not have to provide a map for every possible class. You can of course still support the map. What do you think?

Oh right, I should have that. Actually that is my first solution when trying to isolate Prism's class (before I meet CSS Modules)

Your example JavaScript files like example-2.js cause the minifier to crash. You cannot have any extra *.js files in that directory.
Also your examples are very small. It is probably best if you include them inline in index.html.

Your plugin currently only maps a single token type to a single class, which is a bit limiting. Prism supports aliases, so env.classes could look like this:
env.classes = ['token', 'equation', 'string', 'important'];

Wouldn't it be better to map classes to classes?
env.classes = env.classes.map(function(c){
	return classMap[c] || c;
});

Sorry for the late response. I was quite busy last month. Looking over your source code I had the following idea. Your plugin could also offer the option of a prefix to all classes. Something like a namespace:

Prism.plugins.customClassName.prefix("mynamespace-");

> That way the user would not have to provide a map for every possible class. You can of course still support the map. What do you think?

Accepting a function as a parameter that takes a class and returns the namespaced class would also be good for maximum customization (though I'm a bit worried it might be going towards feature creep :P)

Also, customClass would be a better I think, customClassName is a bit too verbose.

Hi @zeitgeist87 and @LeaVerou , I'm very sorry for the late response. I finally have time to enhance the pull request as your suggestions. Please take a look and tell me anything that I need to fix.
Summary changes:

Rename customClassName to className (https://github.com/PrismJS/prism/pull/950#issuecomment-224289704)
Inline example code (https://github.com/PrismJS/prism/pull/950#issuecomment-223747175)
Support prefix namespace (https://github.com/PrismJS/prism/pull/950#issuecomment-223519639)
Map classes to classes (instead of single class) (https://github.com/PrismJS/prism/pull/950#issuecomment-223748063)
Update docs (docs for prefix feature, add “Motivation”, simplify examples)

Hi @Golmote , would you mind taking a look at this? It seems @zeitgeist87 and @LeaVerou don't have time for this :(

@dvkndn I only had time to look through the code right now. It looks quite good to me. I'll find a moment this week to actually test your plugin.
I think I spotted some grammar errors in your plugin HTML page. I'm not an native English speaker myself, though, so I'm not really the best candidate to help fixing this.

Thank you @Golmote :D I fixed those grammar errors as you spotted. Sorry for my bad English :(

Ok, it looks good to me.
One last thing... why is components/prism-markup.min.js listed in the changed file? Can you revert the changes on this file?

Sure I reverted the file. That is an accident, sorry

Great! Merging, then!

Why was this meged into gh-pages? Will it be merged into master and published to npm any time soon?

Because everything is merged to gh-pages first. I'll try to make a release on master this month if nobody does it before.

I see. Thanks!


[Feature Proposal] DateRangePicker

I think it would be nice to have a date-range picker control.
It could work a standard way: two side by side calendars. The left calendar would be used to pick the 'from' date and the right calendar would be used to pick the 'to' date.

Related issue: #304

Duplicate to https://github.com/radzenhq/radzen-blazor/issues/157


Difficulties in hovering over histogram values

After entering an URI in the service and hitting Calculate, a histogram is displayed:

Hovering over a bar show the number of mementos available for the time slice, where each bar represents a month.
One must be very precise to move the mouse a certain number of pixels to hover over the bar. This is quite inaccessible. Hovering "above" the bar does not display the value, as done on IA's Wayback Machine toolbar and interface. Very small values make the "hotspot" to activate the hover as small as a single pixel or two.
Please change this effect to activate on hovering within the histogram anywhere in the vertical space for the respective bar.

The current ability to zoom for wider bars provides a nice solution to this.

Closing issue.


Publish new version

Thank you for merging my PR, would you be able to publish the update to NPM?

Hi, sorry about that. Didn't see this issue until just now. 0.0.3 has just been published, thank you for your work!


Add support for mocking Facebook passport login

https://github.com/AlexeyKhristov/passport-strategy-mock

Duplicate of #106

Resolved by #144


scripts: modules: support west group feature

West has introduced support for group tags in:
https://github.com/zephyrproject-rtos/west/pull/454
This means that manifest files might start containing groups.
Zephyr itself only requires west>=0.7.2 where groups are not supported
but other Zephyr based projects might start using the group feature.
When using a west version with group support, then only active projects
will be processed as Zephyr modules.
West versions without group support will consider all projects active.
Signed-off-by: Torsten Rasmussen<EMAIL_ADDRESS>

@nashif this should go in in order to properly support west 0.9.0 in Zephyr 2.5.

@nashif this should go in in order to properly support west 0.9.0 in Zephyr 2.5.


Multiple SCTP chunks in one packet

There are cases when a 5GC NF uses SCTP (for example AMF towards gNodeB), which means that one packet could contain more than one SCTP DATA chunk. These chunks contain NGAP messages.
The import_pdml function says:
    # For 5GC
    # Fix case (see free5GC trace) where there are several elements (not correct but well...)
    ngap_proto = packet.findall("proto[@name='ngap']")
    if len(ngap_proto) == 0:
        ngap_proto = None
    elif len(ngap_proto) == 1:
        ngap_proto = ngap_proto[0]

So this part (and perhaps more) would require an update to consider the fact of multiple SCTP chunks (meaning multiple NGAP messages) in one packet, which is entirely correct protocol behaviour.

It seems like the fix in #28 fixed this one. I checked by repeating a <proto name="ngap" showname="NG Application Protocol" [...] element in a PDML file and the SVG correctly displays a second NGAP PDU in the description.
Hopefully this fixes the issue :)

yes, it's okay for me too, not sure what I missed before. it's working as expected.


How to get bestpos and bestvel from ttyUSB0

After building from instructions using catkin I used $ roslaunch /home/andrew/novatel/src/novatel_gps_driver/novatel_gps_driver/launch/tester_for_usb.launch to launch the unedited launch file.  The error I received back was Failed to load nodelet [/novatel] of type [novatel_gps_driver/novatel_gps_nodelet]  with further details attached.
error.txt  Can I get some guidance on how to change the tester_for_usb.launch file?
tester_for_usb.txt
No doubt you realize I'm a noob and the training I'm taking at robotignite academy so far has had fairly simple launch files and the packages and folder structure are dissimilar to yours.  My goal at the moment is simply to get the binary format coming into USB0 into ROS so I can echo out the results of BESTPOSB and BESTVELB.  The reason I used the roslaunch command the way I did is because I could not get the 2-3 different source commands I tried to let me follow the typical structure of roslaunch (package) (launch file).  Thanks for your help and appreciate that you guys have provided the driver.
Edit: I needed to run $ sudo apt install ros-kinetic-novatel-gps-driver before the catkin build process.  Although I'm still not clear which source command to run I am able to use the full path and have the rostopic /bestpos echo results in another terminal.

Based on this line in your error log:
[ERROR] [1520799297.178989183]: The error before refreshing the cache was: According to the loaded plugin descriptions the class novatel_gps_driver/novatel_gps_nodelet with base class type nodelet::Nodelet does not exist. Declared types are  depth_image_proc/convert_metric depth_image_proc/crop_foremost depth_image_proc/disparity depth_image_proc/point_cloud_xyz depth_image_proc/point_cloud_xyz_radial depth_image_proc/point_cloud_xyzi depth_image_proc/point_cloud_xyzi_radial depth_image_proc/point_cloud_xyzrgb depth_image_proc/register image_proc/crop_decimate image_proc/crop_nonZero image_proc/crop_non_zero image_proc/debayer image_proc/rectify image_proc/resize image_publisher/image_publisher image_rotate/image_rotate image_view/disparity image_view/image nodelet_tutorial_math/Plus pcl/BAGReader pcl/BoundaryEstimation pcl/ConvexHull2D pcl/CropBox pcl/EuclideanClusterExtraction pcl/ExtractIndices pcl/ExtractPolygonalPrismData pcl/FPFHEstimation pcl/FPFHEstimationOMP pcl/MomentInvariantsEstimation pcl/MovingLeastSquares pcl/NodeletDEMUX pcl/NodeletMUX pcl/NormalEstimation pcl/NormalEstimationOMP pcl/NormalEstimationTBB pcl/PCDReader pcl/PCDWriter pcl/PFHEstimation pcl/PassThrough pcl/PointCloudConcatenateDataSynchronizer pcl/PointCloudConcatenateFieldsSynchronizer pcl/PrincipalCurvaturesEstimation pcl/ProjectInliers pcl/RadiusOutlierRemoval pcl/SACSegmentation pcl/SACSegmentationFromNormals pcl/SHOTEstimation pcl/SHOTEstimationOMP pcl/SegmentDifferences pcl/StatisticalOutlierRemoval pcl/VFHEstimation pcl/VoxelGrid stereo_image_proc/disparity stereo_image_proc/point_cloud2 swri_image_util/blend_images swri_image_util/contrast_stretch swri_image_util/crosshairs swri_image_util/draw_polygon swri_image_util/draw_text swri_image_util/image_pub swri_image_util/normalize_response swri_image_util/replace_colors swri_image_util/rotate_image swri_image_util/scale_image swri_image_util/warp_image swri_nodelet/TestNodelet swri_transform_util/dynamic_publisher swri_transform_util/gps_transform_publisher

It looks like ROS wasn't able to find the built nodelet library, which makes me suspect that you didn't source your setup file (. $HOME/novatel/devel/setup.bash) after building the workspace.  Installing the binary package (ros-kinetic-novatel-gps-driver) would "fix" that problem because now you have the driver nodelet installed in /opt/ros and would not need to source the workspace in order for ROS to find it.

I think this has been fixed now, so I'll close this issue.


Fixes [#25656] Stop displaying links in recent changes when user does not have view permission

Trac ticket number

Ticket: https://code.djangoproject.com/ticket/25656.
Branch description
Currently if a user in the admin portal has a add permission on a model, and they add it. They will see a clickable link in the log entries sidebar, which results in a 403 when clicked as the user does not have view permission on said object.
Checklist

[ ] This PR targets the main branch.
[ ] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[ ] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

Apologies @LilyFoote and @felixxm, I got myself into a spot of bother with the git history as it was an old stale branch. Link for the old PR here: https://github.com/django/django/pull/17149. I have now closed it.


Local dev

:tada: This PR is included in version 5.0.0-next.11 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Flip animation to modal popup

From filterable-landscape created by dankohn : cncf/filterable-landscape#13
Add a 3-D horizontal flip animation when clicking a card, such as:
http://xue2han.github.io/react-dynamic-modal/
https://www.npmjs.com/package/react-dynamic-modal

@alexkontsevoy in addition to making it work on close, could you please also try to have the card grow and flip from it's current location, as with: https://stackoverflow.com/a/35499596/1935918

I'll add 2 new states today : 'opening' and 'closing' so a css has a chance to properly display everything

To make it grow and flip, we 'll have to make some calculations

Now animation, although a different one, is supported from a framework, and that is good.
I want to try to also add a slide animation when a persons chooses a next or a previous item via click on left / right arrows

I appreciate the hardwork here, but don't like the flip after all.

You can close and delete branch.


broken on firefox and safari?

UPDATE:
looks like this is caused by minification (I'm using capstone.min.js, because couldn't compile from source):
https://github.com/vibornoff/asmcrypto.js/issues/82
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
I've got another issue for you ;)
This might be resolved, if as you said, the latest version gets recompiled with the latest capstone and emscripten, but for now I'm getting this localized error in FF:
TypeError: asm.js type error: expecting argument type declaration for 'a' of the form 'arg = arg|0' or 'arg = +arg' or 'arg = fround(arg)'
It also doesn't seem to run in safari, but no console errors.  This could be an issue with how I'm using the library however.
Anyway, something to be aware of, as for now this only seems to work on Chrome for me, which isn't so great.
I will update this with more information if I find some.

Confirmed. In the case of Firefox, this is just a warning and the library can still be used without further issues. Using the unminified version capstone.js it still throws different TypeError's, so there is nothing I can do to completely fix this. The case of Safari baffles me. Could you please try with the new unminified version linked above?
Probably, in the long-term, the most sane thing to do is wait for WebAssembly to be supported in all major browsers and can forget about this asm.js insanity and including 6 MB JavaScript files.

@AlexAltea :+1: WebAssembly.
Until that glorious day though, I'll test with the unminified version, specifically with safari.  But it could be a host of other unrelated issues on hindsight.
So right now silicon-disassembler is using the min version, but it's manually copied into the dir and distributed that way.  I might update the bower.json to use the published bower version as a dependency, as per #2 , but I'll have to think about how that would work, or if it's an advantage, etc.
But it looks like you updated to use the latest capstone version, etc.?  I assume there's no breaking changes. If not, i'll update silicon-disassembler with the latest capstone version.
Thanks for working on this :)

I apologize; it looks like this is running fine on safari, with or without the minified version.
Similarly for firefox (modulo assembly TypeError warnings); I'm closing this issue.


Scenes Package

As of now, only the Introductory Scenario scene is available as a DSA specific map. I did scan and import some scenes from e.g. "Hexe vom Schattenwasser", however, I had to do some crude photoshop work to delete room names, secret passages etc.
I'd happily pay for a package with DSA VTT scenes, maybe from the Dungeons GM screen or other non-adventure related sources.

We are working on that.


soc: riscv: riscv-privilege: telink_b91: dts: riscv: telink: telink_b…

…91: drivers: serial: uart_b91.c: Implement Telink B91 suspend mode
To reduce power consumption Telink B91 supports suspend mode.
This mode is involved when OS kernel enter idle process.
Co-authored-by: Andrii Bilynskyi<EMAIL_ADDRESS>Signed-off-by: Misha Tkachenko<EMAIL_ADDRESS>Tested manually using TLSR9518ADK80D by running sample projects with config PM.

@galak formatting of uart driver is back only PM relative stuffs are added.

Could someone proceed with review this PR. Because dependent PR into hal telink is already merged and it block to introduce its functionality.

@gmarull  Are you okay with our latest code? If so, could you approve this pull request?


Fix activate.sh for zsh

Apparently alias takes different flags under bash vs zsh.  In bash, alias -p is a valid flag, but the equivalent in zsh is alias -L.
Currently, this package emits this error when an environment is activated using zsh:
$ conda activate base
/opt/miniconda/etc/conda/activate.d/hub.sh:alias:1: bad option: -p

This PR fixes it.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy. (I did this locally.)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Oops, this is based on an old version of the feedstock.  Closing.


Proper parsing of Structured Annotations in OME-XML

Summary
Currently, bfio uses OmeXml.py, which is a modification of the OME XML parser found in the tifffile repository with some modifications made by the Allen Cell Institute. The current version does not properly handle the latest version of the OME XML spec, specifically things handled under the old structured annotation namespace. This makes it impossible to handle original metadata tags or metadata that does not have explicit definition under the ome namespace.
What is the current bug behavior?
bfio is not able to completely handle all OME XML metadata.
What is the expected correct behavior?
bfio should be able to modify all metadata, and sync the important pieces with the current file format.
Possible fixes
Possibly check tifffile to see if the OME XML handling class has been updated, or directly modify OmeXml.py to properly handle the metdata.

This has been handled in bfio 2.3 by using a package to handle OME metadata.


[indigo] verify joint limits for cob4 gripper

see https://github.com/ipa320/cob_common/pull/139#discussion_r24563453
@fmessmer FYI

During the preparation of HMI, we found some additional issues with the current cob4_gripper URDFs, i.e. joint orientation...see #151

not planned


[New] Covsirphy.line_plot() accept title is None to hide title

Summary of this new feature
Currently, we must set figure title for Covsirphy.line_plot() function, but this is not useful when we want to create a figure to check the content of dataframes quickly for exploratory data analysis.
Covsirphy.line_plot() should accept title=None to hide title.
(Optional) Solution with the current version
cs.line_plot(df, title="")

CovsirPhy version: 2.14.0-zeta

#527 was merged and we can use cs.line_plot(df) to remove title.

#527 was merged and we can use cs.line_plot(df) to remove title.


Quick fix to silence 1.3 warnings

Just a quick fix to get the compiler to be happy :)

Thank you! I just made a new release with the fix :smile:


TS2724: "@opentelemetry/semantic-conventions" has no exported member named ATTR_DEPLOYMENT_ENVIRONMENT.

What happened?
Steps to Reproduce
import {SEMRESATTRS_DEPLOYMENT_ENVIRONMENT} from '@opentelemetry/semantic-conventions';

Deprecated symbol used, consult docs for better alternative
TS6385: SEMRESATTRS_DEPLOYMENT_ENVIRONMENT is deprecated.
SemanticResourceAttributes. d. ts(140, 4): The declaration was marked as deprecated here.
Name of the deployment environment  (aka deployment tier).
Deprecated:
use ATTR_DEPLOYMENT_ENVIRONMENT

Then
import {ATTR_DEPLOYMENT_ENVIRONMENT} from '@opentelemetry/semantic-conventions';

TS2724: "@opentelemetry/ semantic-conventions" has no exported member named ATTR_DEPLOYMENT_ENVIRONMENT.

Expected Result
a non deprecated import
Actual Result
a Deprecated import
Additional Details
OpenTelemetry Setup Code
No response
package.json
"@opentelemetry/api": "^1.9.0",
    "@opentelemetry/auto-instrumentations-node": "^0.51.0",
    "@opentelemetry/exporter-logs-otlp-grpc": "^0.54.0",
    "@opentelemetry/exporter-metrics-otlp-grpc": "^0.54.0",
    "@opentelemetry/exporter-trace-otlp-grpc": "^0.54.0",
    "@opentelemetry/resources": "^1.27.0",
    "@opentelemetry/sdk-logs": "^0.54.0",
    "@opentelemetry/sdk-metrics": "^1.27.0",
    "@opentelemetry/sdk-node": "^0.54.0",
    "@opentelemetry/semantic-conventions": "^1.27.0",
    "@opentelemetry/winston-transport": "^0.6.0",

Relevant log output
No response

Hi @mpareja-godaddy, thanks for reaching out - I think this is a duplicate of https://github.com/open-telemetry/opentelemetry-js/issues/5025 so you can track progress there


Editor hover: markdown links should use their own custom scheme

Steps to Reproduce:

Set "enable-browser-code-loading": "bypassHeatCheck" in ~/argv.json and restart
Clone https://github.com/svsool/vscode-memo
yarn
comment out the line that reads Image preview is currently not supported
Run
open a markdown document with a reference to a PNG in the workspace
retype it as memo notation (e.g. [[images/1_54/macOS-universal-download.png]])
hover over it

=> 🐛 it looks like the resulting image tag uses file protocol and not vscode-file protocol:

We need to make sure to use the FileAccess utility to properly convert to a URL that the browser can handle.
Slightly related: @deepak1556 expressed some interest that the generated HTML from markdown uses a custom scheme so that we can distinguish markdown content from our core content. I had to push https://github.com/microsoft/vscode/commit/a34297a60858cf43f4bc92aae48b72e3ea37754a now as a workaround.

To clarify, I am proposing to keep vscode-file only deal with resources for the core, we create a separate scheme (ex: vscode-hover) to load resources for the hover html. That way we can keep the restrictions around vscode-file stricter and additionally there is no added cost to having multiple schemes in an app, offering cleaner separation for resource loading.

After looking a bit closer into what the extension does, I realised that the extension is actually returning file:// URIs as hover contents from here:
https://github.com/svsool/vscode-memo/blob/5dc4fe035b7c4474d29c6cd24afd33f5e13a1a6e/src/features/ReferenceHoverProvider.ts#L56
As such, we probably just take the contents as is and rely on the fact that we currently still support to embedd file URIs for rendering.
But going forward this will not be possible anymore, because once we set our origin to vscode-file://, Chrome disallows any file:// URIs to be loaded.
I think we almost have to detect absolute URIs that are file based and rewrite them to a custom scheme like Deepak suggests to resolve this going forward.

We already use the FileAccess-util for uris that have been transformed and it should be easy to use them for all, tho I am unsure how a unified scheme can be achieved because some need the special "this is from a remote" scheme already

// @roblourens unsure what the snippet below is for but it will be broken once we don't use the file-scheme anymore
https://github.com/microsoft/vscode/blob/55393229982b2ef32d68b766fc69f06585be596b/src/vs/base/browser/markdownRenderer.ts#L105-L108

Regarding FileAccess utility: the right method to call is FileAccess.asBrowserUri because this is a URI we want to embedd in the browser renderer. And that one already takes care of vscode-remote scheme:
https://github.com/microsoft/vscode/blob/7a9867c841e0222a08d29cbe6d0cc1512ba4c6e0/src/vs/base/common/network.ts#L159-L161
I think a good first step is to see whether FileAccess.asBrowserUri can be adopted for hovers or not. However, what Deepak is referring to with regards to a new custom scheme is the fact that yesterday I added this code to enable images to load even when their file paths point into the workspace:
https://github.com/microsoft/vscode/blob/a34297a60858cf43f4bc92aae48b72e3ea37754a/src/vs/code/electron-main/protocol.ts#L23-L23
Arguably this weakens our filtering and allows any images to load in the same context as the workbench. Hence the idea to introduce a new custom scheme specifically for images that the hover inlines through markdown rendering.
@jrieken though I think you mentioned yesterday in the standup that we have more places where images get inlined, not just the hover? I think you talked about editor decorations in the gutter as well?

I think a good first step is to see whether FileAccess.asBrowserUri can be adopted for hovers or not.

That's easy - we partially already do that (today only for uris that have been transformed): https://github.com/microsoft/vscode/blob/55393229982b2ef32d68b766fc69f06585be596b/src/vs/base/browser/markdownRenderer.ts#L79-L85

However, what Deepak is referring to with regards to a new custom scheme is the fact that yesterday I added this code to enable images to load even when their file paths point into the workspace:

Yeah, maybe. Pretty sure asBrowserUri returns different schemes depending on the input. We need to ensure that stays true. So a resource for a remote workspace will be having a different uri-scheme than one of a local workspace.

@jrieken though I think you mentioned yesterday in the standup that we have more places where images get inlined, not just the hover? I think you talked about editor decorations in the gutter as well?

Yeah, this place @jrieken though I think you mentioned yesterday in the standup that we have more places where images get inlined, not just the hover? I think you talked about editor decorations in the gutter as well?

Ok, so dom.asCssUrl has already adopted FileAccess a while ago:
https://github.com/microsoft/vscode/blob/f1d92e789602f9389114760636f21c1b6b91ec32/src/vs/base/browser/dom.ts#L1230-L1230

Yeah, maybe. Pretty sure asBrowserUri returns different schemes depending on the input.

FileAccess.asBrowserUri will leave any URI untouched that is not file:// scheme, the rule currently is:

vscode-remote: use RemoteAuthorities
file: rewrite to vscode-file
anything else: pass through

This is probably known, but just in case, this has become broken in the documentation shown in the completion/suggest widget too, not only hovers (I suspect they're using the same code though).
Is there any workaround extensions can use in the meantime to have images render again?

@bpasero is there any known workaround for this? I'm getting a lot of issues from users that are now seeing broken images in tooltips/completion documentation that used to work. It's not clear from this issue if this is likely to be fixed in the near-term so I'm wondering if I should remove some of the image/icon previews to avoid the reports.

This issue is affecting Dart Code's icon previewing, which is something really useful while developing a Flutter application. I'm sticking to an older version of VSCode just to prevent this from happening, as having to

This issue is affecting Dart Code's icon previewing, which is something really useful while developing a Flutter application.
I'm sticking to an older version of VSCode just to prevent this from happening, as having to open a separate documentation just to see what an icon looks like is a pain.
Is there any plan to fix this in the near-term?

@DanTup Can you please test out the next VS Code insiders build (1.60) and let me know if this still isn't working. The commit I pushed seems to fix this for markdown strings with links in my test extension

@DanTup Can you please test out the next VS Code insiders build (1.60) and let me know if this still isn't working. The commit I pushed seems to fix this for markdown strings with links in my test extension

It is working fine on insiders build.

Sorry, I thought I replied to this but remember that it was while GitHub was having issues so I guess it never sent. I also verified in Insiders it was good, thanks! :-)


title colour on views like 'Search for people' should not be green but match the tinting for that views

as green on orange is ugly

Fixed by #2419


data is vastly imcomplete

missing a lot from like
https://gist.github.com/adamloving/4401361
or even google temporary emails has tons of domains not been listed here

Hi James,
this repo only holds domain names from mailinator.com. It is an excerpt and appetizer from block-disposable-email.com where you can get access to an api currently filled with > 20k domains from several dea services. If you want to stick on github sources you might also find this repo with emailondeck.com domains useful.


How does semantic search work with encryption?

Hey VedankPurohit,
Cool project! Love hag you're doing. Kinda curious how you're managing to perform semantic search alongside encryption?
Would love to get a high level overview and pointed in the right direction in the code!

It's a great question.
When an image is captured, it's embedding is created and then it's instently encrypted by the key you provide. This embedding vector is saved unencrypted hence there is no need to encrypt it's embedding
The sementic search is performed on an index which only contained Vector of the image and image location.
Then the located images are temporarily decripted and show to you. Then the decripted image is deleted


how to disable tooltip in range date option

Hey guys, how do you disable these tooltips that pop up only during date-range option. See the attached.
to reproduce here -> https://uxsolutions.github.io/bootstrap-datepicker/?markup=range&format=&weekStart=&startDate=&endDate=&startView=0&minViewMode=0&maxViewMode=4&todayBtn=false&clearBtn=false&language=en&orientation=auto&multidate=&multidateSeparator=&keyboardNavigation=on&forceParse=on#sandbox
select range and click on the date input field to see, same date.

That may be a problem with your input. I am using v1.9 and I didn't get those popups and I tried with chrome, edge and firefox.
What version are you trying? These are the ones I'm using

![bootstrapdatepicker](https://github.com/uxsolutions/bootstrap-datepicker/assets/117616130/43d52504-cf47-42ba-8b84-3600a1a67264)

Ooops didn't send the image correctly
https://github.com/uxsolutions/bootstrap-datepicker/assets/117616130/43d52504-cf47-42ba-8b84-3600a1a67264

Also tried with the page you provided and still nothing. What browser are you using?

Also tried with the page you provided and still nothing. What browser are you using?

Hey, im using latest version "Datepicker for Bootstrap v1.9.0 (https://github.com/uxsolutions/bootstrap-datepicker)".
I tried in chrome, brave, firefox. To me it seems browser auto complete is doing something funny. If i use the demo page provided by the datepicker, i get the same issue.

just fixed it, add -> autocomplete="off" to form tag.

Glad you solved it bro

Glad you solved it bro

Thank you for your help :)


Wrong type types.path for nullable config field.

https://github.com/NixOS/nixpkgs/blob/4633a7c72337ea8fd23a4f2ba3972865e3ec685d/nixos/modules/services/monitoring/prometheus/exporters/mysqld.nix#L25
The module config should allow null path if you don't want to specify the config, so the type should probably be something like
types.path or types.null

Feel free to PR! Should be types.nullOr types.path


Update technical.md

New event from AVPlayer access log for bitrate measurements.

Hi this should be added to playback_playing to prevent overloading the analytics server.
Just use the latest entry when sending playback_playing.


Add some more utility lemmas in All_Forall

These will be useful to prove that we can weaken typing judgments past specific changes in the checker configuration.

LGTM


README.md should describe two expected src directory layouts

Please consider having README.md describe, early on, the
expected source directory layout.  This is especially
tricky for a reader because there two layouts are used.
Reading from the top down, the first encountered layout
is for the trio project. If I understand correctly, the
 .crossType(CrossType.Pure) // [Pure, Full, Dummy], default: CrossType.Full
causes the bar/.js, bar/.jvm, bar/native, shared, form to be used. Since
these are hidden directories, it makes it exceedingly hard to
figure out.
Since I like prove out my understanding, I usually start simple
and work my way to more complex.  I jumped into the second,
duo example before I attempted the trio example.  It took
more work than I expected, but I figured out the expected
bar/jvm, bar/native, shared directory structure.  This gives
nice, visible directories.
When I extended my duo project to the trio project
described in the README.md, sbt could no longer
find my source code.  After ratting around, I discovered
that this is because the duo project defaulted to 'CrossType.Full'.
The trio project had an explicit 'CrossType.Pure', which expected
a different source directory structure.
As both scala-cross & my understanding of it mature, I hope
to understand when one would use a Full & when one would
use a Pure.  For now, I am using Full seems to work, so I
am using it to allow me to see my directories.
The good ascii art in file CrossType.scala helped me get the
trio project up and running.  Adding that art to the README.md
would, I believe, ease the way of other early & potential adopters.
Thank you for considering this suggestion. As always, thank you
for sbt-cross. I have learned from following the README.md.
Lee

This guy gave it a shot. What do you think?
https://github.com/scala-native/sbt-cross/pull/19/commits/bb595ac71a5673a890398e1b9285a869e26f34ba

Thank you for asking my opinion.
First approximation:
I think bb595ac71a5673a890398e1b9285a869e26f34ba is a useful contribution and a good base for evolution. The perfect is the enemy of the done & useful. Thank you, @rom1dep
Second approximation

I would probably add a "NOTE WELL the leading dot" to the

   ├── .js
   ├── .jvm
   ├── .native

It would be nice to explain the intended purpose/use of the .mumble
directories.  I have no clue.

It would be nice to mention which cross type is the default & why.

If CrossType.Full is the default, I would introduce it first.

I am loath to reveal the limits of my understanding to the world, but
I do not understand what the section at the bottom is trying to say.
I will blame the hour for my shortcoming .,,

 `.crossType({/*custom*/})`
+
+One can easily extend CrossType and provide a custom tree structure.

Perhaps the author's intent can be realized by moving the last line way
earlier & providing an example at the bottom .crossType(CrossType.Custom)
+sbt-cross provides  3 implementations, described below, of the CrossType class that one can
+pass as `.crossType` parameter: CrossType.Full, CrossType.Pure, CrossType.Dummy.
CrossType.Full is the default if .crossType() is given no argument.

+One can easily extend CrossType to provide a custom directory tree structure.

Putting my actions where my mouth/typing is, I could do a PR  after bb595ac71a5673a890398e1b9285a869e26f34ba  is merged.
Third approximation, picky level.

I read for concept, not to as a copy editor, but the line

 This layout is preferred for codebases which do not contain any platform-specific code.

jumped out at me. Please consider something like the more direct
This layout is preferred for codebases which contain no platform-specific code.

Fixed in https://github.com/scala-native/sbt-crossproject/pull/39


Remove unused branches

@livarcocc , can you confirm that removing the MSBuild entry is ok?
@GrabYourPitchforks, can you confirm that removing the utf8string branches from Jenkins is ok?

Its ok. Confirmed for msbuild.

It's fine to remove them from CI since we're about to merge into master. Please don't delete the branches from source control just yet, though, since we're still cherry-picking commits.


Force 2nd instance of app to wait for lock release before starting

Hi,
I'm using mongobee in a microservice running in Kubernetes and several instances of my application can be started at the same time.
When the first instance starts, it sets the database lock and runs all the new changesets.
At the same time, a second instance starts and mongobee exits because of the lock but the application starts normally, which could be a problem since changesets are still being applied by the first instance.
I would like to be able to tell mongobee to wait for the lock to be released before continuing app execution. Obviously when the lock is released then mongobee will check the changesets to be executed and there will be nothing to do (since the first instance has just finished the job), so the application can now start.
This could be a new parameter in the Mongobee bean.

Also the lock is not cleaned up if the Mongobee execution fails / is interrupted.
If there is a lock from a old process, you are never able to recover.

#73


@import from node_modules does not allow omitting file extension

NPM version (npm -v): 6.9.0
Node version (node -v): v10.15.3
Node Process (node -p process.versions):
{ http_parser: '2.8.0',
node: '10.15.3',
v8: '6.8.275.32-node.51',
uv: '1.23.2',
zlib: '1.2.11',
ares: '1.15.0',
modules: '64',
nghttp2: '1.34.0',
napi: '3',
openssl: '1.1.0j',
icu: '62.1',
unicode: '11.0',
cldr: '33.1',
tz: '2018e' }
Node Platform (node -p process.platform): win32
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): 4.13.0
npm node-sass versions (npm ls node-sass):-- (empty)

What I try to do
I generate a single CSS output file from various SCSS partials, including a well established third-party sanitize lib.
what is my setup?
In my package.json I have "sanitize.css": "^8.0.0" in the devDependencies (because it is only needed during compile time, not during runtime)
As expected we will find the file in node_modules/sanitize.css/sanitize.css
According to the documentation (https://sass-lang.com/documentation/at-rules/import#importing-css) I should be able to import it from the css file (and have it inlined in the final output file) as if it were just any SCSS partial by omitting the file extension.
like so:
@import "~sanitize.css/sanitize";
But this results in an error:
{
  "status": 1,
  "file": "C:/workspace/<path_to_my_main_style.scss>",
  "line": 1,
  "column": 1,
  "message": "File to import not found or unreadable: ~sanitize.css/sanitize.",
  "formatted": "Error: File to import not found or unreadable: ~sanitize.css/sanitize.\n        on line 1 of src/main/path_to_my_main_style.scss\n>> @import \"~sanitize.css/sanitize\"; \n\n   ^\n"
}

When I change the import to also include the file extension:
@import "~sanitize.css/sanitize.css";
...it is treated like a plain CSS @import
as described in the documentation (https://sass-lang.com/documentation/at-rules/import#plain-css-imports)

~ in import os a sass-loader feature. Please open an issue with that
package.
On Fri, 29 Nov 2019, 1:59 am Martin<EMAIL_ADDRESS>wrote:

NPM version (npm -v): 6.9.0
Node version (node -v): v10.15.3
Node Process (node -p process.versions):
{ http_parser: '2.8.0',
node: '10.15.3',
v8: '6.8.275.32-node.51',
uv: '1.23.2',
zlib: '1.2.11',
ares: '1.15.0',
modules: '64',
nghttp2: '1.34.0',
napi: '3',
openssl: '1.1.0j',
icu: '62.1',
unicode: '11.0',
cldr: '33.1',
tz: '2018e' }
Node Platform (node -p process.platform): win32
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): 4.13.0
npm node-sass versions (npm ls node-sass):-- (empty)

What I try to do
I generate a single CSS output file from various SCSS partials, including
a well established third-party sanitize lib.
what is my setup?
In my package.json I have "sanitize.css": "^8.0.0" in the devDependencies
(because it is only needed during compile time, not during runtime)
As expected we will find the file in node_modules/sanitize.css/sanitize.css
According to the documentation (
https://sass-lang.com/documentation/at-rules/import#importing-css) I
should be able to import it from the css file (and have it inlined in the
final output file) as if it were just any SCSS partial by omitting the file
extension.
like so:
@import https://github.com/import "~sanitize.css/sanitize";
But this results in an error:
{
"status": 1,
"file": "C:/workspace/<path_to_my_main_style.scss>",
"line": 1,
"column": 1,
"message": "File to import not found or unreadable:
~sanitize.css/sanitize.",
"formatted": "Error: File to import not found or unreadable:
~sanitize.css/sanitize.\n on line 1 of
src/main/path_to_my_main_style.scss\n>> @import
https://github.com/import "~sanitize.css/sanitize"; \n\n ^\n"
}
When I change the import to also include the file extension:
@import https://github.com/import "~sanitize.css/sanitize.css";
...it is treated like a plain CSS @import https://github.com/import
as described in the documentation (
https://sass-lang.com/documentation/at-rules/import#plain-css-imports)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2796?email_source=notifications&email_token=AAENSWCSP5C7ECN47Y7SVM3QV7MEDA5CNFSM4JSVUL6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H4WRQLA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENSWFE6S22QUNRXIWWQIDQV7MEDANCNFSM4JSVUL6A
.

I'm not using sass-loader, only node-sass directly from the command line interface.
So, node-sass must have some mechanism of its own to resolve imports with an ~

It does not. However ~ is often an aliases the hike directory. That may be why the file is not found.

Sorry I have no idea what that is. I would be forever grateful if you could point me in the right direction.

The option if you are just using node-sass and not though a separate build tool is the includePaths https://github.com/sass/node-sass#includepaths

@nschonni Thanks. I got it working using the option you proposed.
Now an import from the node_modules like:
@import "sanitize.css/sanitize";
is inlined as expected into the output file.
I run node-sass like so:
node-sass --include-path node_modules <input_directory> -o <output_directory>


Add Transform Helper methods

Objective

The objective of this PR reflects the idea of adding some helper methods to Transform to avoid cumbersome way of providing direction and moving an object. The issue taken in consideration is the following: Helper methods on Transform

Solution

I decided to add a basic translate method in the Transform class translate(&mut self, direction: Vec3, speed: f32, delta_time_seconds: f32). This function simply provides a faster way of adding a translation to the Transform by providing a direction, a speed and a delta_seconds (or delta in general referring to time). This doesn't take in account the rotation of the Transform.
I also added a second function translate_with_local_rotation(&mut self, direction: Vec3, speed: f32, delta_time_seconds: f32). This function differs from the one above because it also utilizes the rotation of the Transform to provide a way of moving the object following a specific direction and taking in account also it's rotation.

Testing

I didn't test the new function, but I changed the example translation.rs using the newly implemented function translate to see if the result was the same.
I also tried the second method translate_with_local_rotation by modifing the example 3d_rotation and adding a new line just after the rotation. It moved as expected, but the addition of the line felt like out of scope of the example, so I removed it.

I see what you mean.
Maybe to make it more coherent might be better to only have a Vec3 as the signature of the method, calling it translation and let the user create the translation vector by multiplying it by the speed AND the delta time?

Overall I think the code is good, I'm not sure on the naming though.
I feel like translate should take an offset vector, and translate the Transform by that value.
This is more related to velocities and kinematics (moving objects over time).
Not sure what would be a better name.

I see what you mean.
Maybe to make it more coherent might be better to only have a Vec3 as the signature of the method, calling it translation and let the user create the translation vector by multiplying it by the speed AND the delta time?

I see what you mean.
Maybe to make it more coherent might be better to only have a Vec3 as the signature of the method, calling it translation and let the user create the translation vector by multiplying it by the speed AND the delta time?
Edit: going to change it later to only receive a Vec3

I think translate_to for a new absolute position and translate_by for an offset would be a good naming convention.

I think translate_to for a new absolute position and translate_by for an offset would be a good naming convention. It might also be worth wrapping the Vec3 in a newtype Offset and Position to prevent accidentally providing the wrong kind of value. Mojang actually just had this exact mixup earlier this month when reimplementing ender pearl teleportation!

I see your points, but isn't just easier to read and understand a single Vec3 as a parameter? Then it's the user's job to provide the correct calculation over that Vec3 I feel 🤔

I think translate_to for a new absolute position and translate_by for an offset would be a good naming convention. It might also be worth wrapping the Vec3 in a newtype Offset and Position to prevent accidentally providing the wrong kind of value. Mojang actually just had this exact mixup earlier this month when reimplementing ender pearl teleportation!

I see your points, but isn't just easier to read and understand a single Vec3 as a parameter? Then it's the user's job to provide the correct calculation over that Vec3 I feel 🤔

It's not wrong either way. I feel like it's the same difference between having a raw boolean parameter vs an enum with 2 values. There is a clippy lint against boolean parameters because it's easy to accidentally mix up which parameter or which value you meant to use. Also, a raw true doesn't mean much to a first time reader unless they also look at the name of the parameter. However, there's also a balance to be had, of course. We don't need to newtype everything. My suggestion can be decided against. It's just a little piece of input in case everyone likes the idea. :)

RE: Point & Offset newtype.
While mathematically I 100% agree that the semantics of vectors (as offsets) vs points are very different (and the reason we use Vec3 for both could be regarded as a "hack" based on arbitrarily fixing the origin) the codebase is generally not made for this and it would be a very controversial change.
(Feel free to make an issue to discuss this, but we would likely need to convince glam to cooperate and we would make a very big breaking change.)
(Note: There's also some API design issues with affine combinations (specific subset of linear combinations) being valid operations on both Points and Vectors, while general linear combinations being well-defined only on vectors, however there is no type-based way to decide this, so you either need to allow points to behave like vectors, or force users to convert a bunch, introducing a lot of friction.)
Relevant discussion @ glam: https://github.com/bitshifter/glam-rs/issues/133


Optimize Hash#select(Enumerable) and #merge!(Hash, &)

These two defs call has_key?(k) followed by [](k), performing a double lookup. This PR turns them into find_entry(k).
The optimization is quite noticeable when the keys are large structs, because their hash functions are more complex. I don't have a comprehensive benchmark but the speedup for #select is ~15% for UInt64 keys or ~60% for UInt8[128] keys.

By the way there is no public API for this when the value type is nilable, i.e. when #[]? cannot distinguish a nil value from the absence of a key. The closest thing is find { |k, _| k == key }, but it has much poorer performance and doesn't account for @compare_by_idenrity and @block.

Isn't that fetch with a block?

#fetch yields only when the key is not present. Here the desired behavior is to do something only when the key is present.


EOS signed message

We need to be able to create signed messages. This will prove ownership of an EOS Private key and by extension ownership of an EOS account. To do this we will need a standard prefix agreed upon.
For example on Ethereum, Geth prepends the string \x19Ethereum Signed Message:\n to all data before signing it (https://github.com/ethereum/wiki/wiki/JSON-RPC#eth_sign).
We could use a similar prefix \x19EOS Signed Message:\n
The ability for Ethereum holders to sign messages have been pretty useful on EOS with the Unregistered Process(Code) and the EOS Lost Key recovery process (Code).
Ethereum supports signing on Ledger, Metamask and numerous wallets all following the same standard which is added to the messages entered here. https://vintage.myetherwallet.com/signmsg.html

This was actually began by Bitcoin itself, but is not something I am that passionate about

ecc.sign does sign a message, however it is doing it without a standard EOS prefix.
https://github.com/EOSIO/eosjs-ecc#examples-5
Are you thinking a specific EOS blockchain prefix is needed? a signMessage function which prefixes should be fairly simple, although I think that'd belong in eosjs-ecc?

Cleaning up old issues so we can focus on more recent issues. Please reopen this ticket if needed. Thank you.


test: compatibility tests

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

1.1.0
GitHub release

Your semantic-release bot :package::rocket:


How to convert a container that already exists on the host machine to a WASM?

How do I convert a container that already exists on the host machine to a WASM?

@FunnyCorgi If you have a image of the container, you can just convert it into wasm using c2w command as described in README. If you don't have the image of the container, you need to acquire the image by pulling it from somewhere or by generating it using docker commit or something.

Thank you, this should help.


[WORKFLOWS-34] Onboarding Documentation

I've started drafting the onboarding documentation based on my work in #50. There are a few things that I need to update once #50 and Sage-Bionetworks-IT/organizations-infra#278 are merged.
Depends on #50.
Depends on Sage-Bionetworks-IT/organizations-infra#278

I decided to include the onboarding docs in the README PR to streamline things a bit.