Failed to determine active runtime file path for this environment

This is related to issue #222
I experienced the following when trying to launch the latest sdk's hello_xr on my quest.
$ adb shell am start -n "com.absurdist.nightmare.hello_xr.opengles/android.app.NativeActivity" -a android.intent.action.MAIN -c android.intent.category.LAUNCHER
Connected to process 5410 on device 'oculus-quest-1PASH9BTUF9411'.
Capturing and displaying logcat messages from application. This behavior can be disabled in the "Logcat output" section of the "Debugger" settings page.
W/main: type=1400 audit(0.0:98): avc: granted { read } for name="u:object_r:net_dns_prop:s0" dev="tmpfs" ino=9894 scontext=u:r:untrusted_app_25:s0:c512,c768 tcontext=u:object_r:net_dns_prop:s0 tclass=file
I/llo_xr.opengle: Late-enabling -Xcheck:jni
E/llo_xr.opengle: Unknown bits set in runtime_flags: 0x8000
W/System: ClassLoader referenced unknown path:
V/threaded_app: Creating: 0x7064d43dc0
V/threaded_app: Config: mcc=0 mnc=0 lang=en cnt=US orien=2 touch=1 dens=235 keys=2 nav=1 keysHid=1 navHid=0 sdk=29 size=4 long=2 modetype=1 modenight=2
V/OpenXR-Loader: Verbose [GENERAL | xrInitializeLoaderKHR | OpenXR-Loader] : Entering loader trampoline
V/OpenXR-Loader: Verbose [GENERAL | xrEnumerateInstanceExtensionProperties | OpenXR-Loader] : Entering loader trampoline
I/openxr_loader: getActiveRuntimeCursor: Querying URI: content://org.khronos.openxr.runtime_broker/openxr/1/abi/arm64-v8a/runtimes/active/0
V/threaded_app: Start: 0x7064d43dc0
E/ActivityThread: Failed to find provider info for org.khronos.openxr.runtime_broker
W/openxr_loader: Null cursor when querying installable content resolver.
I/openxr_loader: getActiveRuntimeCursor: Querying URI: content://org.khronos.openxr.system_runtime_broker/openxr/1/abi/arm64-v8a/runtimes/active/0
E/ActivityThread: Failed to find provider info for org.khronos.openxr.system_runtime_broker
W/openxr_loader: Null cursor when querying system content resolver.
E/openxr_loader: Could access neither the installable nor system runtime broker.
E/OpenXR-Loader: Error [GENERAL |  | OpenXR-Loader] : RuntimeManifestFile::FindManifestFiles - failed to determine active runtime file path for this environment
E/OpenXR-Loader: Error [GENERAL | xrEnumerateInstanceExtensionProperties | OpenXR-Loader] : RuntimeInterface::LoadRuntimes - unknown error
E/OpenXR-Loader: Error [GENERAL | xrEnumerateInstanceExtensionProperties | OpenXR-Loader] : RuntimeInterface::LoadRuntimes - failed to load a runtime
E/OpenXR-Loader: Error [GENERAL | xrEnumerateInstanceExtensionProperties | OpenXR-Loader] : Failed to find default runtime with RuntimeInterface::LoadRuntime()
E/OpenXR-Loader: Error [GENERAL | xrEnumerateInstanceExtensionProperties | OpenXR-Loader] : Failed querying extension properties
E/hello_xr: [13:09:21.206][Error  ] XrResult failure [XR_ERROR_RUNTIME_UNAVAILABLE]
        Origin: xrEnumerateInstanceExtensionProperties(layerName, 0, &instanceExtensionCount, nullptr)
        Source: ../../../../openxr_program.cpp:160
V/threaded_app: android_app_destroy!

Not sure what I missed!

I believe the quest does not yet support the standard loader, you have to use their loader for now.

I believe the quest does not yet support the standard loader, you have to use their loader for now.

This was definitely the issue, I had cmake problems with openxr_loader already being defined until I switched from 3.18 / 3.10 to 3.6 for some reason (just in case this is useful info to future seekers.
Thanks!


Identity support to menu create command switch

Control over menu Identity will help in setting menu identifier in menu items from the recipe data.

The code looks good, but I'd like to understand what the scenario is. Could you provide an example?

It helps to create menu and menu items interchangeably by using using command element of a recipe like below and after that menu items can be imported through data element of the same recipe. Otherwise both the menu and menu items should be created through command or both the menu and the menu items should be created from the recipe data.
Note:
Current implementation of the menu command uses menu name to locate a menu.
Current implementation of menu item import(MenuPartDriver.cs) uses identifier to locate a menu.
We do support setting the Identity in the widget command.
layer create Default /LayerRule:"true" /Description:"The widgets in this layer are displayed on all pages"

layer create Authenticated /LayerRule:"authenticated" /Description:"The widgets in this layer are displayed when the user is authenticated"

menu create /MenuName:"Main Menu" /Identity:"MainMenu1"

widget create MenuWidget /Title:"Main Menu" /RenderTitle:false /Zone:"Navigation" /Position:"1" /Layer:"Default" /Identity:"MenuWidget1" /MenuName:"Main Menu"


Windows Edge: selection is mapped incorrectly when selecting at the end of paragraph

Instead it gets mapped to a selection from begin of container to end of paragraph.

Similar happens when selecting a whole figure, i.e. adding the EOL of the caption.


is setting length broken on Linux?

I attempted to:
n = neuron.Neuron();

cell1 = n.Section("cell1");
cell2 = n.Section("cell2");

cell1.set_diameter(2);
cell2.set_diameter(2);
cell1.length = 10;
cell2.length = 10;

but this failed with:

Error using clib.neuron.set_dparam
MATLABCLibHost process for 'neuron' terminated unexpectedly. To reload interface library, first call
"unload(clibConfiguration('neuron'))" and then call function/class from interface library.

Error in neuron.Section/set.length (line 255)
            clib.neuron.set_dparam(self.sec, 2, val);

Error in indexing (line 95)
                self.(S(1).subs) = varargin{:};

I am, to be fair, not using the latest version, so this might already be fixed?

Not broken. Just ran with the wrong version of NEURON.


bump version in package.json to v2.27.2

release version v2.27.2

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


Add support for 'cursor'

My API supports pagination. In DB query the primary key 'ID' is used as cursor to get the next set of values after the nextToken ID. Its working fine in prisma but not getting next values from the prismock data.  @morintd

@morintd  Please review this issue.

Hello, I could definitely be interested in adding support for cursors, but will need to investigate, I'll add it to my to-do list but don't expect to have it done short-term


Gnuplot

#550
Existing package

[x] I am the maintainer of this package.

I think GitHub thinks this PR conflicts with itself?

I think I see whats going on, hold up.


nightly: add debian nightly build

Rename master tag to nightly-alpine
Add a debian-based nightly image
Make nightly and nightly-alpine multi-arch

This passes CI as per this run: https://github.com/willcl-ark/bitcoin-core-docker/actions/runs/11254254962


NTR: negative regulation by virus of cytolysis by virus of host cell

NTR: regulation by virus of cytolysis by virus of host cell
Definition: Any process that modulates the frequency, rate or extent of the killing by a virus of a cell in its host organism by means of the rupture of cell membranes and the loss of cytoplasm.
Ontology: Biological Process
Reference: PMID:14527655  Figure 1. in this paper shows loss of T4 rI results in a much earlier lysis time of its host.  This requested go term would be more accurate for this process.
Synonyms: regulation of lytic viral release
regulation of viral release by host cell lysis
regulation of viral release by cell lysis
The resulting Child-term would be:
NTR: negative regulation by virus of cytolysis by virus of host cell
Definition: Any process that stops, prevents or reduces the rate or extent of the killing by a virus of a cell in its host organism by means of the rupture of cell membranes and the loss of cytoplasm.
Ontology: Biological Process
Reference: PMID:14527655  Figure 1. in this paper shows loss of T4 rI results in a much earlier lysis time of its host.  This requested go term would be more accurate for this process.
Synonyms: negative regulation of lytic viral release
negative regulation of viral release by host cell lysis
negative regulation of viral release by cell lysis
Reported by: notjustin43
Original Ticket: geneontology/ontology-requests/11335

Increased  host cytolysis by loss of a viral protein does not seem to be strong enough evidence for a  biological process (seems like a phenotype)? It isn't demonstrating regulation.


[JOSS Review] - Community guidelines

Dear authors,
Thank you for including a CONTRIBUTING.md.
Can you also add a Contributing section somewhere in docs (so that it appears on readthedocs) to improve the visibility for third parties looking to contribute? This could paraphrase CONTRIBUTING.md or just provide a link to it. Often this appears in a FAQ page in readthedocs.
This issue is related to openjournals/joss-reviews#7291

Dear @ericfell,
we added a "Getting Involved" section in the documenation and put the contributiong guidelines and contact information there. Furthermore, we added invitations for contributions in the implementation tutorials in the readthedocs documentation.


Interact with items

Another thing that would really open things up for a lot of creativity would be the possibility to give the player items, that he can interact with like it is possible in the original app (for example in almost every szenario with the "Old journal" - evidence card where you are able to read the book if you spend an action).
But im not sure how hard it would be to implement that into valkyrie. Just something i had in mind while playing around with the app...

Yep, nice to have but this is post 1.0.

Alright ;-)

This could be temporarily handled by spawning an interact token outside the play space, then if you have multiple interactions within the journal just have it spawn them in order as new events. It's at least a thought for now if you're looking to write those kind of scenarios.

FYI that is how Descent: Road to Legend does it.


Incorrect Synonyms for GO:0042735 & GO:0004540

Hi,
Just bringing to your attention two GO terms with incorrect synonyms.

GO ID - GO:0042735
GO Term Name - endosperm protein body
Incorrect synonym - https://github.com/geneontology/go-ontology/issues/24425

GO ID - GO:0004540
GO Term Name - RNA nuclease activity
Incorrect synonym - https://github.com/geneontology/go-ontology/issues/24408

Thank you @SaverimuttuS; the property used should be changed to term tracker item.


Error when Deploying iOS app with Xcode 10.2

I'm trying to deploy an iOS app through Xcode by archiving it and then deploying it using the Development option. The archiving works fine but when I try to deploy, it returns me the follow error.
Failed to verify bitcode in Flutter.framework/Flutter:
error: Cannot extract bundle from /var/folders/vv/tzfmqkn92ss1xsc_pwfld4bm0000gn/T/XcodeDistPipeline.~~~k6v9M2/Root/Payload/Runner.app/Frameworks/Flutter.framework/Flutter (armv7)
But this archiving and deploying has worked for many months and I've been doing it for many versions of the app. But this error came about yesterday when I was trying to roll out a new update.

Duplicate of https://github.com/flutter/flutter/issues/39222.
@v-rohith Can you flutter upgrade and try again?


Allow re-registering consensus engines

To support a consensus engine restarting independently of the validator,
the consensus registry must be able to register a conensus engine again.
When this happens, the engine will register with the same name and
version, but a different connection ID. The registry will simply remove
any old entries associated with the engine name and version before
adding the engine to the registry.
Signed-off-by: Logan Seeley<EMAIL_ADDRESS>

There are other concerns when consensus engine restarts. For example, consensus engine is building a block and then it's restarted before finalizing the block. This will leave validator in unexpected state, and when consensus engine restarts and requests to build a new block again validator complains for the scenario.

There are other concerns when consensus engine restarts. For example, consensus engine is building a block and then it's restarted before finalizing the block. This will leave validator in unexpected state, and when consensus engine restarts and requests to build a new block again validator complains for the scenario.

This can be handled by the consensus engine; the validator will reply with an INVALID_STATE status if consensus attempts to initialize a block when one is already initialized, at which point consensus knows there's already a candidate block. A possible alternative would be to cancel any candidate blocks if the active consensus engine disconnects, but that's outside the scope of this PR.

Just a thought: Instead of only removing connection ID, should there be a session with respect to each consensus engine registration/activation, and then the entire session should be ignored when a new session is created?

Can you describe what you mean by a "session"? The EngineInfo object essentially represents a "session": it's a connection to a consensus engine with a given name/version. When a new one registers with that same name/version, the old EngineInfo is removed.

There are other concerns when consensus engine restarts. For example, consensus engine is building a block and then it's restarted before finalizing the block. This will leave validator in unexpected state, and when consensus engine restarts and requests to build a new block again validator complains for the scenario.

This can be handled by the consensus engine; the validator will reply with an INVALID_STATE status if consensus attempts to initialize a block when one is already initialized, at which point consensus knows there's already a candidate block. A possible alternative would be to cancel any candidate blocks if the active consensus engine disconnects, but that's outside the scope of this PR.

Right! Thanks. PoET doesn't make use of it currently.

Just a thought: Instead of only removing connection ID, should there be a session with respect to each consensus engine registration/activation, and then the entire session should be ignored when a new session is created?

Can you describe what you mean by a "session"? The EngineInfo object essentially represents a "session": it's a connection to a consensus engine with a given name/version. When a new one registers with that same name/version, the old EngineInfo is removed.

Your answer clarified my question.


Add character spacing & font kerning

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 12 (91.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 61.138%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/EasyPDF/ClassEasyPDF-Text.cs
10
11
90.91%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
591

Relevant Lines:
960

💛  - Coveralls


⚠ ClamAV has degraded performance

In f520a9b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 214 ms

Resolved: ClamAV performance has improved in 300149f.


[cmake] [c++] require CMake 3.18+

Fixes #5642.
Might help with the following:

#5775
#5785
#5955
#6249
#6252

Proposes the following:

moving this project's minimum CMake version up to v3.18
removing CMake version ranges from docs

Benefits of this change
This should reduce the risk of build failures with newer compilers, operating systems, architectures.
I'm especially hoping it'll help with M1/M2/M3 macOS support and supporting newer versions of CUDA.
Per the CMake docs (link):

The cmake_minimum_required(VERSION) command implicitly invokes the cmake_policy(VERSION) command to specify that the current project code is written for the given range of CMake versions. ... All policies introduced in later versions will be unset. This effectively requests behavior preferred as of a given CMake version and tells newer CMake versions to warn about their new policies.

(my emphasis)
So even on systems with newer versions of CMake installed, having cmake_minimum_required(VERSION 3.0) binds them to how CMake behaved as of that release. The last CMake 3.x release was 5+ years ago (v3.0.2).
This change resolves this warning currently seen in many of LightGBM's supported platforms:
  CMake Deprecation Warning at CMakeLists.txt:35 (cmake_minimum_required):
    Compatibility with CMake < 3.5 will be removed from a future version of
    CMake.

    Update the VERSION argument <min> value or use a ...<max> suffix to tell
    CMake that the project does not need compatibility with older versions.

(example macOS sdist build this was observed on)
Why 3.18?
From https://cliutils.gitlab.io/modern-cmake/chapters/intro/installing.html

Your CMake version should be newer than your compiler. It should be newer than the libraries you are using (especially Boost

And from https://cliutils.gitlab.io/modern-cmake/chapters/intro/dodonot.html

What minimum to choose

3.12: C++20
3.17/3.18: CUDA
3.19: First to support Apple Silicon

I'm mainly recommending v3.18 to support the CUDA builds here.
Other evidence it could be ok:

XGBoost has been using CMake >= 3.18.0 for about 9 months: https://github.com/dmlc/xgboost/pull/8853
XGBoost used CMake >= 3.14.0 for the 3 years prior to that: https://github.com/dmlc/xgboost/pull/7060
it should be very easy for Windows and macOS users to update to newer versions, via conda, pip, official binaries from KitWare, and many other options from https://cliutils.gitlab.io/modern-cmake/chapters/intro/installing.html
PyPI has CMake wheels for a wide range of platforms (link), and when building the Python package, scikit-build-core will automatically get one of those if the local CMake version isn't sufficiently new
the distro-provided CMake is v3.22.1 on Ubuntu 22.04 (link), so any downstream project building on GitHub Actions ubuntu-latest (GHA docs) will be unaffected

Thanks for the review @guolinke ! I think this will really help with the next batch of packaging fixes I'd like to make (especially around M1/M2/M3 macOS).


Close Dialog Through Script

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Currently, there is only way to advance dialogue. The player must press "A".
This limitation prevents the dialogue box from being a more useful, general-purpose tool.
Describe the solution you'd like
A clear and concise description of what you want to happen.
A new script event should be added to advance to the next piece of dialogue. It could be as simple as "Close Dialogue".
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
We could open this up even further by allowing player inputs to be simulated and triggered through script events. In that case, we could close the dialogue by simulating an "A" press.
Additional context
Add any other context or screenshots about the feature request here.
Programmatic control of the dialogue box opens up a lot of possibilities for dynamic content! I'm experimenting with animation in the dialogue box by combining instant animation speed with a custom ascii.png.

I've thought about this recently too.
The implementation i can think of that would work with our code,
Some kind of tick box for "Continue script", it wouldn't automatically close, but you could close it with move overlay, or a delay + new text.

Less related, but this may be helpful for you, i only just finished it.
UI Ascii table characters list, that you can copy paste from.
https://gist.github.com/RichardULZ/2920dca52989902dbb7225318256c545

I've thought about this recently too.
The implementation i can think of that would work with our code,
Some kind of tick box for "Continue script", it wouldn't automatically close, but you could close it with move overlay, or a delay + new text.

I feel like this would need some sort of Wait function, though. It would be nice to be able to trigger dialog, add a Wait, and then either add a "Close dialog" Event or add a new dialog Event.

@timsamoff Thats what i meant, said delay instead of wait.
Currently, a text box in code says "script continue = false", every time, so it can pause the script, and start checking if you've pressed A to close it.
If you change this to say  "script continue true", it will start writing and showing text, but then keep running the next script events, without closing the text box.
You can then use normal "wait" events,
"close dialogue" is the same code as "overlay move to" (it could be duplicated with a new name),
new dialogue would function as expected, and any other existing function you can think of would work too.

@RichardULZ Adding a "continue script" option sounds like a great idea to me.
Just to clarify, I would expect a script like this:
1. Display Dialogue "Text 1" (continue script)
2. Wait 5.0s
3. Display Dialogue "Text 2" (continue script)
...

to ignore all user input during the "Wait 5.0s", and only display "Text 2" when the 5.0s is over.
Is that what you had in mind?
--
Also, I didn't realize that the dialogue box is using the overlay internally. Does that mean we could potentially do full-screen dialogue boxes, or put dynamic content in an overlay? That might be worth another feature request.

@RichardULZ Any updates on this? Let me know if I can help out.

Messed about with it for a few hours today, something i'm doing is causing some text boxes to corrupt, but re creating them fixes it, so i'm rethinking my strategy.
Also, having the Continue Script tick box takes up space that otherwise makes the engine simple and clean.
Might putter about with a separate event as Continue next script, or Continue all scripts / Don't Continue all scripts.
Keep in mind messing with this stuff get's janky pretty quick. Later though, bed time.

Here's what I posted as a separate issue, before realizing this one existed:

It would be nice to have dialogue that goes away automatically after a specified amount of time, rather than waiting for input.
I personally hope to use this with a timer of 0, so I can continue the script immediately, with the dialogue kept visible by the overlay.

I think a "Timed Dialogue" event would be a fairly elegant method with more versatility, and there wouldn't be a need for a separate event to get rid of it.


Floating point math issue

We are having problems with floating point math.
File: int_adyen_overlay/cartridge/scripts/getPaymentMethodsSHA256.ds
Line: # 54

var paymentAmount: Number = basket.getTotalGrossPrice() ? AdyenHelper.getCurrencyValueForApi(basket.getTotalGrossPrice()) : 1000;
Ex: 288.15 * 100 = 28814.999999999996

There will be an error if requesting that value with Adyen's API.
I found the document on this issue here:
https://0.30000000000000004.com/
https://stackoverflow.com/questions/588004/is-floating-point-math-broken

Hi @khanhruby ,
I tried to reproduce this issue, but the return value for me is 28815 as expected.
Could you confirm this issue has been resolved?
Thanks,
Bas Maassen


Decide between iced and egui

Compare the two libraries to decide which one is actually easier to use.
I would like to file the following things we need to figure out:

API ease of use
Benefits

I think the most important aspect is the GUI mode that iced and egui implement.
iced implements retained GUI mode, while egui implements immediate GUI mode.
If I am understanding correctly, immediate GUI mode gives you more control on the order of rendering, and you essentially build the model of the screen, just passing to the library so that the library can actually render. This is like creating a game with SDL. If you want to know more about the advantages and disadvantages of immediate mode, check this.
With retained GUI mode, the library is in charge of explicitly building the model, and the only thing that you do is to declare to the library what you want (not how!), and you would pass callbacks that handle specific events.
In my opinion, I would prefer to go with the immediate mode, since it gives you more power in organising your layout and the like.

I would agree with you on that one I prefer  immediate mode, also I think for me egui looks much more readable.

@JerryHue  can you  close issue #12 and open a new one for setting up egui


Remove UTY-1578 workaround

Description
Now with 2018.3.5f1, we no longer need to force disable the protocolLogging.
Tests
Created a cloud deployment with logging enabled on GameLogic workers.
Didn't crash, saw nice protocol logs appear.
Documentation
Added changelog

FYI will need a corresponding docs PR to remove from the known issues - https://docs.improbable.io/unity/alpha/known-issues

Should also fix: https://github.com/spatialos/gdk-for-unity/issues/714


[Enhancement] HeaderTemplate in Expander control

Summary
Hi, I really miss HeaderTemplate in Expander control.
Expander itself already contains ContentTemplate and it would be really nice to have not only Expander.Header but to have Expander.HeaderTemplate as well.
Thanks 😊
API Changes
Same as we have Content and ContentTemplate we need Header and HeaderTemplate and we can use one of them:
<xct:Expander>
    <xct:Expander.Header>
        <!-- Grid or StackLayout or anything else like root of Header -->
    </xct:Expander.Header>
</xct:Expander>

and
<DataTemplate x:Key="CoolHeaderTemplate" x:DataType="models:SomeCoolModel">
    <!-- Grid or StackLayout or anything else like root of Header -->
</DataTemplate>

<xct:Expander HeaderTemplate="{StaticResource CoolHeaderTemplate}" />

<xct:Expander>
    <xct:Expander.HeaderTemplate>
        <DataTemplate x:Key="CoolHeaderTemplate" x:DataType="models:SomeCoolModel">
            <!-- Grid or StackLayout or anything else like root of Header -->
        </DataTemplate>
    </xct:Expander.HeaderTemplate>
</xct:Expander>

Who Will Do The Work?

[ ] I am willing to take this on myself
[x] Just putting this out there for someone to pick up

Hi @bbenetskyy and thank you for your interest in this project :)
Could you please show a use-case where it can be applied? I mean where it can be useful.
ContentTemplate exists due to lazy loading purpose. If expander is collapsed (by default) then we can "save" time for creating and rendering its content. Instead, we will render content "on the fly" once it is requested. That's why we use DataTemplate.
But since Header is always shown, I don't see any reason for adding data template. Because anyway a view will be created once we set the template.
Thanks

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


Puppet Hubflow

Hello,
I have created a module that will install hubflow for boxen users.
https://github.com/lazzurs/puppet-hubflow
Please let me know if you require any changes before forking under the boxen org.

@tarebyte @rafaelfranca can this be merged in? 8+ months waiting.

@lazzurs does this need to be it's own module? Just from reading over the code that it looks like all it does is install the homebrew package I found here: https://github.com/Homebrew/homebrew/blob/698a68313635ae0e986b7f3c2dd2a8812c80f3b8/Library/Formula/hubflow.rb

Agreed—if it's just a homebrew formula, just include it in a manifest. @lazzurs feel free to re-open if you have a good module use case.


Rate limiting API requests within multiple workers

My use case is :

A set of 5 (rabbitmq) workers consume messages, doing one ore more API request per message
No more than 10 API requests should be globally (throughout all workers) done for 1 second

Would this repo implementation be fine with this distributed constraint ?
Would this rate of 10 be an absolute limit or an average ?

If you use one of the DB or file Storage methods that all the workers can read from then it would work. As each would call from that. So decided where your workers are on same server, network, internet etc. I’ve used Redis for this before. (As each worker checks the same time left token, just think about how many workers and a time locking them if the timeout checking the worker kills it before it gets all its token use work done) — for this sometimes I have made more tokens with less quota for a group of workers, then say 2x5 workers with 10 token each might be better than 10 with 20 token. You need to test and see when all busy.


Connect Tale to Git repositoriy

As a user of Whole Tale, I want to be able to connect an existing Tale to a remote Github repository. I would be able to use standard Git tools in my tale environment to manage the local and remote repositories.
More details in the Git Integration design note.

Fixed by https://github.com/whole-tale/ngx-dashboard/pull/54 and https://github.com/whole-tale/girder_wholetale/pull/430

Fixed by https://github.com/whole-tale/ngx-dashboard/pull/54 and https://github.com/whole-tale/girder_wholetale/pull/430


define key_transform anywhere in serializer

So after looking at https://github.com/Netflix/fast_jsonapi/issues/41#issuecomment-366931614 I noticed that we cannot call set_key_transform just anywhere in the serializer definition, but it must be called before defining attributes.
Unfortunately, I'd like to DRY up my serializers by inheriting all from an ApplicationSerializer and call set_key_transform there, but alas, it does not work.
Is there any way for us to fix this so we can do:
class ApplicationSerializer
  include FastJsonapi::ObjectSerializer
  set_key_transform :camel
end

Hi @JohnBDonner ,
Here is my workaround
class ApplicationSerializer
  def self.inherited(klass)
    klass.send :include, FastJsonapi::ObjectSerializer
    klass.send :set_key_transform, :camel
  end
end

class MySerializer < ApplicationSerializer
  # include FastJsonapi::ObjectSerializer
  # set_key_transform :camel
end

Hey @fongfan999, thanks for the workaround! It works great!


hystrix-javanica-ctw artifact

Hi!
How can I get the hystrix-javanica-ctw?
I am trying this, but it didn't work...

com.netflix.hystrix
hystrix-javanica-ctw
1.5.10

I've already answered this here: https://github.com/Netflix/Hystrix/issues/907#issuecomment-286807723.  Closing this one as a duplicate


Updated readme and added ability to inform users that there are no agents available / online

Thanks Gary! I happily accept this contribution, but do note that the key functionality will move to https://github.com/tompaana/bot-message-routing/ leaving the command handling etc. here. I'll do my best to migrate all the changes from this there.

Great. Thanks. Look forward to contributing more :)

@tompaana Also, so I understand your strategy correctly. It looks like you are going with having the routing data manager in the new package, which can then use the app setting I have introduced, but then what you do with that command result will be up to you when you implement the message routing result handler in your own solution? Are you not planning to have something like a DefaultMessageRoutingResultHandler present in the new package?

Check out the development branch - the answer's there :) In short: No command handling nor result handling (implemented) in the core package. The latter will get an interface though (not sure if that is really required either). The plumbing (commands, result handling) will remain here, but we could figure out together (me, you, other contributors) how to best deal with that.


Normalization issues

[ ] number + preposition of time/place (in) mistakenly labeled as measurement:

ex. 1. Popular varieties in the wet season were Sahel 202 (65% of farmers) and Sahel 201 (30%), while 60% and 92% grew Sahel 108 in 2012DS and 2013DS, respectively.
ex. 2. released as Sahel 108 in Senegal in 1994
ex. 3 rice yield will increase from 3600 in 2000-2009 to 4500 kg ha-1 in 2090-2099 ( Fig. 4a ) .

[ ] unclear how to normalize the numbers in parens (may need to be addressed directly in habitus):

In plots receiving fertilizer, DAP was applied basally (19.3 and 21.5 kg N and P ha-1 )

[ ] pluses are included as B-MEASUREMENT: feature or a bug? (norms are correct, e.g., 100.0 kg/ha for + 100 kg ha-1):

das days after sowing, Fert fertilizer treatment, with F1: recommended dose (80 kg N ha−1), i.e., 200 kg ha−1 NPK (15.15.15) at sowing + 100 kg ha−1 urea at 20 das + 50 kg ha−1 urea at 50 das. F2: F1/4 (20 kg N ha−1)

[ ] range not extracted:

Ex. 1. Rice grain yield measured at maturity ranged from 2.7 t ha-1 to 7.1 t ha-1 , with an average of 4.8 t ha-1 .
Ex. 2 rice grain yields were between 8.8 t ha-1 and 9.2 t ha-1 ( i.e. about 1 t ha-1 more than in the 1998WS
Ex. 3 Average yields in SRV theoretically range between 5.0 and 6.0 t ha1 in the rainy season and between 6.5 and 7.5 t ha1 in the dry season

[ ] how to normalize two separate values with shared unit, e.g., 9 (t/ha) and 10 t/ha below?

Potential yield of all the varieties in the Senegal River delta was estimated at 9 and 10 t / ha in wet and dry seasons , respectively , and potential yield was taken as 8 t / ha for both seasons in the middle valley .
Ex. 2 rice yield will increase from 3600 in 2000-2009 to 4500 kg ha-1 in 2090-2099 ( Fig. 4a ) .

Probably similar to three values in the next example. Maybe extract as one unit and split with an action? (unsure if it will be easy to differentiate these from ranges---which will need to be stay unsplit):
Target yields on average were set to 6.4 , 7.9 , and 7.1 t / ha in 2011WS , 2012DS , and 2013DS , respectively ( Table 1 ) .

[ ] the needs to be added to the vague year date rule (probably here https://github.com/clulab/processors/blob/93c4dfec6549ce56d26b9fa71af057ffb32a4270/main/src/main/resources/org/clulab/numeric/dates.yml#L79) :

The planned timing for the first split was 23 days after sowing ( from 3 to 13 August in the 1999WS and from 14 to 25 August in the 2000WS , see Tab .

[ ] slash-separated dates:

The areas sown for this 2021/2022 wintering campaign are

Thanks @maxaalexeeva! I'll take a look at these soon.

@MihaiSurdeanu I started on these today. I will fiddle with them more tomorrow and will let you know if I run into issues.

Thanks @maxaalexeeva !

"pluses are included as B-MEASUREMENT" - this is a feature :) Let's keep it. Recognizing signs is useful, sometimes. And the number normalization handles this.
"more units come up split by the type of fertilizer" - let's ignore this. This requires a complicated fix. And I suspect this is not common: it is the first time I see a measurement reported this way.
-"units separated with a conjunction" - can you please add a rule for this? This should not be too complicated: V1 and V2 U1 and U2.

Thanks!

@MihaiSurdeanu, will try! Thanks! Another thing came up:
Would you actually also look at the first issue (number + preposition of time/place, including my previous updates to it). My suboptimal solution seemed to work but I now have found a case when it interferes with a different extraction:
I have an example where very similar spans of text get different pos tags:
200     kg     ha-1     NPK
CD     NN      JJ        NN

vs.
,       2000    kg     ha-1     NPK
,        CD    VBG    JJ        NNP

And because of that NNP on the last token, the rule below (the one I added the constraint to to solve the first issue) does not extract the complete unit (kg ha-1):
  - name: measurement-unit
    label: MeasurementUnit
    priority: ${ rulepriority }
    type: token
    pattern: |
      [entity=/B-MEASUREMENT-UNIT/ & !word = "DS"] [entity=/I-MEASUREMENT-UNIT/]* (?![tag = /NNP|CD/])

Any thoughts on what to do? Accept the incomplete unit? Or maybe you know a better solution for the first issue?

Hi Masha,
I think the "kg" issue can be fixed in the POS postProcessing method here:
https://github.com/clulab/processors/blob/master/main/src/main/scala/org/clulab/processors/clu/CluProcessor.scala#L595

@MihaiSurdeanu, the kg part actually does get extracted even with the wrong POS; the issue seems to be the very unique token NPK, which in the second case gets labeled as NNP. That interferes with my constraint in the measurement-unit rule. The constraint is supposed to disallow units immediately followed by a personal pronoun or a digit to solve the first issue in the original post (released as Sahel 108 in Senegal in 1994). Is it worth it adding this unique token to the post-processing method?

Probably Ok to ignore... This seems too specific.


Infinite Recursion in DxDateBox

To help us process the issue more efficiently, please provide the following information:

Specify the version of the devextreme-angular and devextreme packages you are using.
devextreme version: 17.2.4
devextreme-angular version: 17.2.4

Specify the type of the issue (check one with "x"):
[X] bug Report
[ ] feature request
[ ] support inquiry

There's a bug in ui.calendar.js, in the _moveCurrentDate function.  It is possible to go into infinite recursion if most dates are disabled, as the function keeps calling itself until it finds a date that is not disabled.  In our case, all dates in the future are disabled, so attempts to click into the future can cause the error.  This could be prevented by checking the min and max values of the DxDateBox, but the function does not check these values before recursing.

This issue is closed because of being inactive. If you need further assistance, leave a comment here.


This Old House changed URLs again

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.01.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
> youtube-dl -v https://www.thisoldhouse.com/21113884/s41-e13-paradise-lost
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.thisoldhouse.com/21113884/s41-e13-paradise-lost']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.01.24
[debug] Python version 3.7.2 (CPython) - Darwin-18.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.1, ffprobe 4.1
[debug] Proxy map: {}
[generic] s41-e13-paradise-lost: Requesting header
WARNING: Falling back on generic information extractor.
[generic] s41-e13-paradise-lost: Downloading webpage
[generic] s41-e13-paradise-lost: Extracting information
ERROR: Unsupported URL: https://www.thisoldhouse.com/21113884/s41-e13-paradise-lost
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3350, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.thisoldhouse.com/21113884/s41-e13-paradise-lost

Description
This Old House has changed their URLs again, I'm assuming this will require a change similar to the one in issue #11271. I haven't had a chance to dig into the code yet to see if it's that simple of a change. However from a look at a couple of URLs, it looks like the newer urls are now in the format of https://www.thisoldhouse.com/<serial_number>/s-e-<episode_title> (at least for This Old House and Ask This Old House). However, looking at older episodes, I'm seeing urls like https://www.thisoldhouse.com/<subseries_title>/<serial_number>/<episode_title>
Examples:

https://www.thisoldhouse.com/21113884/s41-e13-paradise-lost
https://www.thisoldhouse.com/21067015/pining-for-old-pine-the-westerly-project
https://www.thisoldhouse.com/charleston-houses/21053714/rainbow-row-the-charleston-houses-2018

Hi there,
This has fixed new links and broken the old links.
new sxx-exx links work
e.g. these work
https://www.thisoldhouse.com/21133625/s41-e15-no-pain-no-gain
https://www.thisoldhouse.com/21133650/s18-e15-indoor-garden-fireplace-hearth
but for example links like
https://www.thisoldhouse.com/21083431/seaside-transformation-the-westerly-project
and
https://www.thisoldhouse.com/season-18-ask-toh/21083433/ceiling-light-tool-storage-ask-toh
do not work
>youtube-dl.exe --get-id https://www.thisoldhouse.com/21083431/seaside-transformation-the-westerly-project -v [debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: ['--get-id', 'https://www.thisoldhouse.com/21083431/seaside-transformation-the-westerly-project', '-v'] [debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252 [debug] youtube-dl version 2020.02.16 [debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763 [debug] exe versions: ffmpeg git-2019-10-31-b414cff, ffprobe git-2019-10-31-b414cff [debug] Proxy map: {} ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\YoutubeDL.py", line 796, in extract_info   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\extractor\common.py", line 530, in extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\extractor\thisoldhouse.py", line 42, in _real_extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\extractor\common.py", line 1005, in _search_regex youtube_dl.utils.RegexNotFoundError: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update.

Yeah @remitamine, I can confirm @monty124's report.

Issue still exists...why was issue closed?

Any update to this issue?

@plk0507  (and anyone else who stumbles on this) I have not tried it on TOH, but you may get some luck with yt-dlp https://github.com/yt-dlp/yt-dlp


JSON parsing issues in various methods

Taking Swarm.Peers for example:
public async Task<ICollection<MultiAddress>> Peers(CancellationToken cancellationToken =
default(CancellationToken))
{
    HttpContent content = await ExecuteGetAsync("peers", cancellationToken);
    string json = await content.ReadAsStringAsync();
    var swarmPeers = _jsonSerializer.Deserialize<IDictionary<string, IList<string>>>(json);
    return swarmPeers
        .SelectMany(x => x.Value)
        .Select(x => new MultiAddress(x)).ToArray();
}

The JSON response (returned by 0.4.9) cannot
actually be deserialized as IDictionary<string, IList<string>>
Trying to do so results in the following exception:
Newtonsoft.Json.JsonReaderException: 'Unexpected character encountered while
parsing value: {. Path 'Peers', line 1, position 11.'

If, as a means to test this theory, you simply use the non-generic version of Deserialize:
var result = JsonConvert.DeserializeObject(json, _settings);

No exception will be thrown.
And another way to test the theory:
public class PeerCollection
{
    public PeerEntry[] Peers { get; set; }
}

public class PeerEntry
{
    public string Addr { get; set; }
    public string Peer { get; set; }
    public string Latency { get; set; }
    public string Muxer { get; set; }
    public object Streams { get; set; }
}

//Deserializing to the types above:
var result = JsonConvert.DeserializeObject<PeerCollection>(json, _settings);

This also completes without exceptions.

Hi @timothyparez. Do you know if this is caused by the new IPFS version? I don't remember this happening with previous versions.

Going to check that.
I'm also checking compatibility between the PeerEntry class above and the MultiAddress class
since the JSON can be directly converted to proper instances of PeerEntry I'm not sure why
it's currently relying on splitting strings (but I might not understand the full scope here):
    public MultiAddress(string uriString)
    {
        OriginalString = uriString;
        _components = OriginalString.Split('/');
    }

I tested 0.4.9 just now (was testing 0.4.10 before)
{"Version":"0.4.9","Commit":"","Repo":"5","System":"amd64/windows","Golang":"go1.8.1"}
The result is the same exception.
It returns the following JSON Pasted on Pastebin
If you paste the JSON in Visual Studio using Edit/Paste Special/Paste JSON as classes
you'll end up with something like the classes in the original post which can be used to
deserialize the JSON to show that it's actually valid.
I'm not sure about older versions because I only started using it when 0.4.9 was released.

Thanks for the JSON sample. I definitely don't remember that Muxer field but it's been a while since I looked at the raw output. I'll have a go at it over the weekend.

In case you don't like creating classes only to deserialize and would like to avoid working
with strings but rather let JsonConvert do the hard work you could use dynamic instead:
var json = "jsonstringhere";
var results = JsonConvert.DeserializeObject<dynamic>(json);
IEnumerable<dynamic> jsonPeers = results.Peers;

var peers = jsonPeers.Select(x => new MultiAddress
{
    Address = x.Addr.ToString(),
    Peer = x.Peer.ToString(),
    Latency = x.Latency.ToString(),
    Muxer = x.Muxer.ToString(),
    Streams = x.Streams.ToString()
});

The code above results in the following:

Note that I quickly created an empty MultiAddress class specifically for this example.
The idea would be you convert directly from the json to the real MultiAddress class in the project.
If they added fields in the future things would just keep working.


floating label and icon?

HI,
if you use:
<lx-text-field label="First and last name" fixed-label="true" icon="account">
    <input type="text" ng-model="textFields.fixedFirstName">
</lx-text-field>

then you get with the icon nicly in front of it

and when doing:
<lx-text-field label="Phone number" >
    <input type="text" ng-model="textFields.fixedLastName1">
</lx-text-field>

you get the floating label

now what would one have to do to get both? so floating label and icon.
Letting the icon="xx" doesn't work :/
and setting floating-label="false" doesn't work either

It might not be possible for the moment. I'll see with @matmkian, but I think it's not really "material design compliant" to have the label going up while the icon stay left. And moving the icon too will be disturbing. I'll get back to you.

As this is not "material design compliant" we won't provide this feature.


fix(simple psim): gear bug to update state in simple psim

Signed-off-by: Takayuki Murooka<EMAIL_ADDRESS>Related Issue(required)
https://github.com/autowarefoundation/autoware.universe/issues/371
Description(required)
Apply gear to current pose update (previously, only velocity and acceleration was updated)
Review Procedure(required)
run simple psim
Related PR(optional)

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Read commit-guidelines
[ ] Assign PR to reviewer

If you are adding new package following items are required:

[ ] Documentation with description of the package is available
[ ] A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

Could you fix ideal_steer_acc_geared too?

@tkimura4 Thanks for the comment. As you may know, Kataoka-san seems to create another PR for ideal one.

Could you fix ideal_steer_acc_geared too?

@tkimura4 I did, and confirmed it works well.


Multi-Select does not work on folders

So with the multi-select, there is quite an annoying limitation, which took me a while to figure out. Weirdly enough, Finder can select a file, multiple files, or a folder, but not multiple folders… m(
var wd = "/Users/chrisgrieser/Library/Mobile Documents/com~apple~CloudDocs/File Hub/"

var folder = Path(wd + "untitled folder")
var folder = Path(wd + "untitled folder copy")
var file = Path(wd + "untitled")
var file2 = Path(wd + "untitled 2")

var finder = Application("Finder")

// works
finder.select(folder);
finder.select(file);
finder.select([file]);
finder.select([file, file2]);

// does not work
finder.select([folder]);
finder.select([folder, folder2]);

The docs on the JXA Path object are pretty sparse. If anyone knows how to fix this, any tips are welcome

not planned anymore for now, since various other restrictions with Finder labels (e.g. not working in hidden directories) make it not very much worthwhile, even with this bug fixed 😕


Clean up

rm

nom I from ci
package-lock.json

and updates

Project.toml
template footer

Thanks!  Timing is not much different - 1m45s normal vs 1m57s on this PR, which is pretty OK - especially since it also tests the node installation etc.


manual sync asks for password

Staging template.
Committing changes of non blacklisted files.
Stashing and saving TEMPLATE branch changes!
Committed changes to TEMPLATE branch
Pushing TEMPLATE branch to remote: sc-autoencoder
Username for 'https://github.com': zethson
Password for<EMAIL_ADDRESS>
Checking out to new branch mlf_core_sync_v1.0.3
Pushing to remote branch mlf_core_sync_v1.0.3
Username for 'https://github.com': zethson
Password for<EMAIL_ADDRESS>
Successfully created PR!
Checking out original branch: development

I expect the token be sufficient and not be asked for the password. Else it worked well.

We suspect that this happens when a repository was cloned without the token. In any case the token should be used for push events.

As we discussed this, could this be closed now?


[samsungmobile] Add Galaxy Xcover 4

Released in April 2017
Official Product  homepage
Security maintenance release schedule
Samsung Security Updates
Current OS :  Android 9 "Pie"
Android Pie  final release : January 4, 2022

:+1:
eol samsung-mobile | grep Xcover


Image in FullTrack

Is it possible to include an Images field containing the track's song picture in the response for FullTrack? Other response types (ie FullAlbum) contain Images, and Spotify's Tracks API also contains this field as well.

Hi @Tengokuken
What do you mean with track song pictures? AFAIK and according to the documentation, the album object contains the images. You can also get those via track.Album.Images.
https://developer.spotify.com/documentation/web-api/reference/#object-trackobject
https://developer.spotify.com/documentation/web-api/reference/#object-simplifiedalbumobject
If you just see a IPlayableItem type, you can checkout this guide: https://johnnycrazy.github.io/SpotifyAPI-NET/docs/iplayableitem

You're absolutely right, I misread the API response in Spotify's documentation. Thank you!


Example project (exmaple-simple based on documentation usage)

Hi,
I am writing some example project based on your usage documentation.

example-simple will be exactly the use case from the documentation.
example-with-children will be almost the same but with comment document as children of blogpost document in elasticsearch

This is still a work in progress.
Be free to ask me some code modifications if needed to feet the purpose of documentation.
I will try to follow code recommendation using flake8 & I will try to add some test using pytest.
EDIT : Only the first project inside this PR, the other will come within another pull request.

Sounds good, thanks for the heads up. You can also the original files here,
for reference.
https://github.com/CommandrAvander/elasticsearch-django-tutorial
On Tue, Mar 29, 2016 at 5:29 AM, Benoit Brayer<EMAIL_ADDRESS>wrote:

Hi,
I am writing some example project based on your usage documentation.

example-simple will be exactly the use case from the documentation.
example-with-children will be almost the same but with comment
document as children of blogpost document in elasticsearch

This is still a work in progress.
Be free to ask me some code modifications if needed to feet the purpose of
documentation.
I will try to follow code recommendation using flake8 & I will try to add
some test using pytest.
You can view, comment on, or merge this pull request online at:
https://github.com/jaddison/django-simple-elasticsearch/pull/12
Commit Summary

Start of example projects (README & requirements)
Flake8 and autopep8 added to help for code linting.
Reorganized requirements in 3 categories : base / tools / test.

Reply to this email directly or view it on GitHub
https://github.com/jaddison/django-simple-elasticsearch/pull/12

--
Adam Vanderbush
<EMAIL_ADDRESS>@CommandrAvander https://twitter.com/CommandrAvander

Coverage : 95%
flake8 : ok
I need to add few more docs inside the README.MD file but I think it's almost done for the 'example-simple' project.
Maybe it's better if you merge this before I write the 'example-with-children' one ?

Hi @M0dM - I noticed that you were building against master; I'm currently in the midst of updating the code to a new version (see es2.0 branch), which includes some breaking changes.
I'm sure your PR will merge smoothly when ready, but will be out of date with the planned changes in es2.0. Just an FYI.
Feel free to go either way with your proposed PR(s) - one big one or two separate ones. Your call.

The thing is I never user this project with elasticsearch 2.x because I am actually using elasticsearch 1.7.5 provided by bonsai.io. So I don't really know what are the breaking change between elasticsearch yet.
Anyway, I will try to finish the other example with the same es version and see how to port these two example project on es 2.x after that.
I will edit the first message & title to match the 'example-simple' only PR and you can merge if it's ok for you. ;)

Sorry, I didn't mean Elasticsearch 2.x - I meant that the new version of django-simple-elasticsearch will be 2.0 (the es2.0 branch for this project has the changes). There are some changes that are NOT backwards compatible. I will see what I can do to minimize them, but I'm removing some functionality.
See https://github.com/jaddison/django-simple-elasticsearch/pull/13 for the current changes (likely more coming).

Could we merge this inside a new 0.9.16 tag before we figured out what will break with new version 2.0 ?

Hi jaddison,
Do you plan to merge this ?


Add bump2version to automate package versioning

Currently, we have to find the lines of code in different files and bump their versions manually. To make this process more automated, we can add bump2version.
bump2version is a Python package that automatically bumps the version of the software in the applicable files by running a simple command (e.g., bump2version <major | minor | patch>).
We just need to add the package to conda/e3sm_diags_env_dev.yml and update setup.cfg with these lines:
[bumpversion]
current_version = 2.4.0
commit = True
tag = True

[bumpversion:file:setup.py]
search = version="{current_version}"
replace = version="{new_version}"

[bumpversion:file:acme_diags/__init__.py]
search = __version__ = "{current_version}"
replace = __version__ = "{new_version}"

[bumpversion:file:conda/meta.yaml]
search = version = "{current_version}"
replace = version = "{new_version}"

I was thinking about how to automate this process in my packages.  It didn't occur to me that there would already be a package for that but it makes sense!

UPDATE: bump2version does not support prerelease candidates (or at least without extensive hacking) at the moment.
Related issue: https://github.com/c4urself/bump2version/issues/63
The alternative solution is using tbump.


Google Pay referer

One question about Google Pay and adyen plugin:
Is the plugin communicating with the google api directly from the shop server and the referer is the shop url or is it communicating of the adyen network and the referer is an adyen url?

Hello @andydandy80,
The shop server is used to initialize the google pay payment on the storefront.
Kind regards


Add initial fuzzing infrastructure

Hi Teemu,
as previously discussed here is the initial fuzzing infrastructure. Please modify the PR to your liking.
In the makefile, a new fuzzing target still needs to be added (which builds the library with fuzz.cpp instead of main.cpp), and the new target needs to be used in build.sh.

Looks good to me as is. :)

Note that it still needs the Makefile modification we talked about: In addition to the ancient target a fuzz target (or similar) needs to be added, and main.o need to be split from the rest of the OBJS list (because both fuzz.cpp and main.cpp have a main function, they cannot be compiled into the same executable as there will be a duplicate symbol).


[RFC] Support nullability for jax Array

Motivation
Nullability is supported in apache arrow, when converting arrow int array with nulls to numpy or jax, the type changed from pa.int8/16/32/64 to float automatically to use np.nan to express null. It's OK for int8/int16/int32 since it can be expressed using float64 precision bits, but when it comes to int64, converting from int64 to flaot64 will loss  precision.
Since jax is used for numeric computing widely, It would be great to support nullability natively in jax. For example, we can use a bitmap to expression nullability for every element in array, and propagate the nullability along the computing graph.
It may be not easy, since the XLA is developed for deep learning workload, and the int64 precision is not a priority.
Without the nullability support in jax, we have to define a higher-level expression on jax to propagate the nullability using a bool array.
Solution1

Add a bitmap region for every jax array.
Add extra XLA computing for bitmap along all jax arrays

Solution2
Solution1 may introduce extra cost for deeplearning since it doesn't have strict precision requirements.
To avoid introduce breaking changes or performance cost, we can introduce a new jax.Array type, i.e. jax.NullableArray , which carries a bitmap, and make all computing with jax.NullableArray` to progate the bitmaps: the result bitmaps are union of every bitmap
Checks

[x] Check for duplicate requests.
[ ] Describe your goal, and if possible provide a code snippet with a motivating example.

Thanks for the request! JAX previously included an experimental masking transformation that accomplished something like this, but it did not get enough use and was eventually deprecated and removed.


convert for darwin-arm64

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
To have convert tool available to run M1 Macs

@felipesere in the meantime you can clone this repo and run go run ./tools/convert/ --help.


Unable to install with<EMAIL_ADDRESS>

Describe the bug
Hey. I just started my new NextJS project and wanted to try NestJS-like decorators within my backend, so I followed documentation and tried to install this package, but got the following error:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/next
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@storyofams/next-api-decorators
npm ERR!   @storyofams/next-api-decorators@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/goodwin/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/goodwin/.npm/_logs/2021-06-13T22_07_17_838Z-debug.log

I see that you included next in your peerDependencies which I'm not really familiar with, but since you are using yarn and I actually remember running into an issue with it's installing dependencies in different maner than npm I can suggest that you meant to use "next": "^10.1.3" (10.1.3 is a minimum requirement) instead of "next": "10.1.3" (exactly 10.1.3). Please correct me if I'm wrong.

Hey @abriginets,
Thanks for reporting the issue. Just released a patch.


Use 764 for files created

In working with #5404 I notice that we create new files (in library helpers here and in Afero) with "0777 // before umask". I understand the need for the execute bit on directories, and while umask is great, it may be a little technical to most people, and we should consider making this something more strict for regular files.
https://chmodcommand.com/chmod-764/
Opinions @moorereason @kaushalmodi @anthonyfok ?

I think we should defer this to umask.  Don't fight against the host system; work within it.  I suspect that we'll have bug reports that Hugo isn't respecting umask if we make this change.

@moorereason never mind, I have re-read the code: We create directories with 0777 before umask (we need the execute bit here), but file creation is delegated to os.Create, which uses mode 0666 (before umask).


USQL Databricks driver fails when connection with Azure Databricks

I am facing an issue when trying to connect to Databricks using usql. Even when using the correct credentials, typing the command:
usql databricks://token:dapi*****@adb-*************.azuredatabricks.net:443/sql/protocolv1/o/*********/*******

The credentials and URL details have been omitted for security, but I assure you that the data is correct.
Error Logs:

What You Have Tried

Verified credentials and URLs.
Reinstalled usql.
Tried on different terminals and operating systems (MacOS and Linux).

Where it works
Using DBeaver and the same credentials, the connection works like a charm
Any help to resolve this issue would be highly appreciated.

@calebeaires this is likely due to how the URL is generated for the driver. I had based this on the reference code for AWS. Specifically, the URL is bad. Could you try opening an interactive interpreter session, and then doing this:
$ usql
(not connected)=> \c databricks token:dapi*****@adb-*************.azuredatabricks.net:443/sql/protocolv1/o/*********/*******

The issue is that the path generated from dburl is incorrect for Azure.
I will think about the best way to handle this in the future, but you should be able to connect using the direct connection syntax (see above's \c).
There's some difficultly here since Databricks' URLs do not conform to how other databases work.
Sadly, I don't have a Azure Databricks instance that I can test against, so will need your help with this. Thanks.

@calebeaires I have fixed defining connections in config.yaml to properly recognize YAML array syntax. You should be able to define a full connection string now in your config.yaml:
$ cat contrib/config.yaml
---
# named connections
connections:
  zzz: ["databricks", "token:dapi*****@adb-*************.azuredatabricks.net:443/sql/protocolv1/o/*********/*******"]

You can then use the named connection zzz:
$ usql
(not connected)=> \c zzz
^C6:00AM ERR error="Post \"https://adb-*************.azuredatabricks.net:443/sql/protocolv1/o/*********/*******\": context canceled" connId= corrId= queryId=
error: databricks: request error: error connecting: host=adb-*************.azuredatabricks.net port=443, httpPath=/sql/protocolv1/o/*********/*******: databricks: request error: open session request error: Post "https://adb-*************.azuredatabricks.net:443/sql/protocolv1/o/*********/*******": context canceled

Note in the above that the URL is now azuredatabricks.net.
I've tagged a new release and will publish it as soon as the build has finished.

@kenshaw
I use usql into linux ubuntu, and use just the cli with the parameters. Run through a terminal is our primary goal. I can offer you and Azure Databricks connection so you can test. How can I send it to you.

@calebeaires Sure, I can test it directly. My email is kenshaw  gmail. The latest release was published yesterday, it should work just fine though. Please check it out when you can.

@calebeaires Thank you for sending the credentials. I was able to connect and verify that usql will work with Databricks. You need to use this kind of syntax:
(not connected)=> \c databricks token:****@****.azuredatabricks.net:443/sql/protocolv1/o/****

Where the first ****  is your personal access token, the second is the assigned domain host, and the third is the remote resource/database name.


"not me" button should be "not them" when helping others

When helping others e.g.: https://bionomia.net/help-others/Q22113468 the button to indicate that specimens were not associated with a person should read "not them" rather than "not me"

Good point! I'm reusing bits and pieces throughout the site to minimize the codebase, but this one doesn't make sense.


alejandra: 0.6.0 -> 0.7.0

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

$ nix-build -A alejandra -A pkgsStatic.alejandra -A pkgsCross.aarch64-multiplatform.pkgsStatic.alejandra -A pkgsCross.armv7l-hf-multiplatform.pkgsStatic.alejandra -A pkgsCross.gnu32.pkgsStatic.alejandra -A pkgsCross.raspberryPi.pkgsStatic.alejandra -A vscode-extensions.kamadorueda.alejandra
/nix/store/s9294kpiyna4grv0qmjhsyyr7b0sf860-alejandra-0.7.0
/nix/store/lfvphi6zp72jlkhxac7l37vnhvvh98hc-alejandra-static-x86_64-unknown-linux-musl-0.7.0
/nix/store/wv27l4qy7wi3h59p0rlqp8abqzsmz99z-alejandra-static-aarch64-unknown-linux-musl-0.7.0
/nix/store/2xr3mdycjjzsik4flpfw2rqzx03l9s6f-alejandra-static-armv7l-unknown-linux-musleabihf-0.7.0
/nix/store/kki92zpxb42h6zz6xklvpbwd70jwq207-alejandra-static-i686-unknown-linux-musl-0.7.0
/nix/store/5hwf9c65wyzclnn8xq236zb7w1x9ivl2-alejandra-static-armv6l-unknown-linux-musleabihf-0.7.0
/nix/store/2351qcy4icxflq191qnrzpi1daf453ly-vscode-extension-kamadorueda-alejandra-1.0.0


Apply: added post about changing the question about location

Added a post to document the changes we made to the question in Apply about locations.

 Apply: changing the content of the location question


Rename $em to $entityManager

:+1: I like the change

@voronkovich thanks for making this change. Although I don't agree with it, I acknowledge that most developers prefer this variable name. Thanks!


f

Type of issue
Typo
Description
[Enter feedback here]
Page URL
https://learn.microsoft.com/en-us/vcpkg/get_started/overview
Content source URL
https://github.com/Microsoft/vcpkg-docs/blob/main/vcpkg/get_started/overview.md
Document Version Independent Id
c038c76a-550b-9b30-215f-a2747465590b
Article author
@AugP
Metadata

ID: c038c76a-550b-9b30-215f-a2747465590b
Service: vcpkg

Unclear where the typo is, so closing this issue. Feel free to respond though if you can provide specifics.

I never reported a problem. I don’t remember doing it.Sent from my iPhoneOn Oct 15, 2024, at 5:04 PM, Augustin Popa @.***> wrote:
Unclear where the typo is, so closing this issue. Feel free to respond though if you can provide specifics.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Cannot install

I can't seem to be able to install this hook. I tried this way:
php artisan hook:install voyager-bread-generator
But I get this nasty error message:
  [InvalidArgumentException]
  Could not find package voyager-bread-generator at any version for your minimum-stability (stable). Check the package spelling or your minimum-stability

I'm running Laravel 5.5 and Voyager 1.0.6 on PHP 7.1
Help?

I was able to install this hook by adding the following to my composer.json file:
{
  "minimum-stability": "dev",
  "prefer-stable": true,
}


Can't prove my github account.

Keep getting
""" \ "celestial-intelligence" on github failed: No server-given hint for sig=3f82cdb317ab4ffbeed147f85e1976a319ed80360f9447c81f04b27e09b38a330f (code=306) \ """ mistake, while i made everything by the instruction provided. See my proof https://gist.github.com/Celestial-intelligence/9d56ec296c52a2183cbf1c2569b8cf8c
my log id: f9f0ea1545fe8d1cda49721c

@Celestial-intelligence And now look how it should look.

Ok, didn't see this. Thank u 4 help.

@5h4d0ww0lf Hi, could you please explain what was @Celestial-intelligence doing wrong? For those who stumbles upon this issue...


InnerHit new implementation

Hi,
It's not a full pull request but more a implementation question that need some more work to be completed (tests).
InnerHits support mostly all parameters available for search (from, size, highlight, source, ...)
The current implementation as partial support for this parameters (from some via addParameters).
My proposal is to give a full search to innerhits, It's more user friendly.
In case you agree, you can close this one : https://github.com/ongr-io/ElasticsearchDSL/pull/153 that will be duplicate.
What do you think about it ?

Well, Search object scope is very wide. Are you sure that inner_hits needs child inner_hist or suggest or something other what's is in Search. That's why we added a type hint with builder interface, where you can add anything from query scope.

BuilderInterface does not help in our case, here is a example of valid query :
{
    "inner_hits": {
        "article": {
            "type": {
                "article": {
                    "sort": {
                        "article_id": {}
                    },
                    "size": 10
                }
            }
        }
    }
}

since you forced a query :
'query' => $this->getQuery() ? $this->getQuery()->toArray() : null,

It will send "query" instead of "sort"
to deal with, we must use this tricky syntax :
$query->addInnerHit($parent = (new ParentInnerHit('article', 'article')));
$parent->addParameter('size', 10);
$parent->addParameter('sort', (new FieldSort('article_id'))->toArray());

This works but seems tricky
Using search will allow ide autocompletion but you are true, not all search functionality is allow in innerHits.

Simple solution, in setQuery, add some control and throw exception.
Complexe one : make class Search use a lot of trait, and create a InnerHitsSearch class that use this trait

I see your point. Well in this particular case change BuilderInterface to Searchmakes sense. Then it's up to the user what he adds to the query. Moreover, elasticsearch will throw an exception if something is wrong.

Ok so you agree and I can go on the final implementation with test ?

Yes

I think it's ok now

Can you append changes in the docs as well according to change with Search object?

for the removing of return $this, my fault, failed merge.
for the setter public function setQuery(Search $query = null), same than earlier, query is optional
I rename setQuery for setSearch to be consistent

Yes, query is optional in inner hits, but to have a default value in setter doesn't make sense. I already fixed your previous implementation by removing default value. It's very simple, if you don't want to set query do not call setter.

Everything else looks good. Thanks for the support!

same debate than earlier, I need to change the construct to  :
    /**
     * Inner hits container init.
     *
     * @param string $name
     * @param string $path
     * @param Search $query
     */
    public function __construct($name, $path, Search $query = null)
    {
        $this->setName($name);
        $this->setPath($path);
        if ($query) {
            $this->setSearch($query);
        }
    }

But earlier, you tell me to put a null value on setter, so I do the same here :smile:

I believe there was a misunderstanding. In case the query is optional maybe it would be better to remove it from the constructor?

btw I have a doubt about setSearch and it's parameter $query naming. The function name look ok, maybe parameter name should be $search as well. In other way around maybe it's not good to say that we are setting here a search object, because main objective is to set a query.

to be consistent, I changed $query properties & parameters to $search
I remove the default value on settter and add a test on __construct

The search parameter is optional on construct but really needed to have a working innerHits.
In my case, I need to expend the size of inner hits (seems to be 3 by default) and I also need to filter innerHits.
I think only few people will use innerHits with empty search (meaning having top 3 child order by score) so IMO it's better to keep it nullable
Tell me what do you think about that

LGTM


[CI/CD] Add FTR workflow for gantt chart plugin test

Description
Add FTR workflow for gantt chart plugin test
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The FTR test is running and passing: https://github.com/opensearch-project/dashboards-visualizations/actions/runs/7267969235/job/19802907437?pr=306#step:20:102

This PR will also be manually backported into 2.x


💡 [FEAT]

Is your feature request related to a problem? 😢 Please describe.
For my use case, I find that having only one layer of nesting for child todos is very limiting (ie, I often need to do tasks 1, 2, and 3 in order to accomplish task A, which itself is one of multiple tasks required to complete todo foo)
Describe the solution you'd like   🤔
I would like A to create a child under any todo, including a child todo.

Hey @TOMES-TT ,
I had a discussion on this before here
The point about code complications seems vague now.
It was just bad design decisions that I plan to work on soon.
However, the first point still remains valid
I tried nesting too much which made the date column something like this:
The upper/parent tasks' date rendered ideally as they were supposed to be
But in the nested regions the letters got separated into different lines creating something like this:
No
 Due
Date

Idea :bulb::
What I can do now is to allow the users to tweak the UI and provide the respected ratios they want to assign to each column and the number of columns they want to see (for e.g. they don't want the urgency column they can remove that)
Example config:
tasks: 70
date: 20
urgency: 10

Where the number shows the ratio to divide the columns into ( according to the terminal's width ) where 0 means don't show the column
What are your thoughts on this?

Hi,
Thanks for the response! My mistake for not realizing there was already discussion on this topic, I thought I had checked everywhere but I'm still pretty new to github so I must have missed it. I think that would be a good solution.

My mistake for not realizing there was already discussion on this topic

No problemo!

I think that would be a good solution.

I'm a bit busy with my work and am not getting too much time for github stuff, so might be a while :disappointed: .
I'll keep this thread open and will update once implemented

Yeah, no worries; obviously I'm just requesting stuff from you, so I have no expectations about it :)
Something that just occurred to me; due dates could be made optional, which would let the todo column stretch out to basically fill the entire terminal window

Something that just occurred to me; due dates could be made optional, which would let the todo column stretch out to basically fill the entire terminal window

Yes, I did mention that:

where 0 means don't show the column

I am also thinking to make the Workspace or the left section toggleable (with animation).
The current code wouldn't allow that but I'll implement it soon.

Implemented in v1.0


Nodes intermittently loose their node addresses

Backlog tracking issue for bug report https://github.com/kubernetes/kubernetes/issues/68270
I had originally thought this was fixed in https://github.com/kubernetes/kubernetes/pull/65226 but recent reports suggest that this bug still exists. Should be p0 or p1 for v1.16 and we should cherry-pick it to previous versions if possible.

/close
Turns out this was already fixed in https://github.com/kubernetes/kubernetes/pull/65226


[bug/reopening of #165352] vscode markdown preview not showing images

Suppose we have a dir with a readme.md and some imgs
readme.md content is
# Hello Imaged World

![](foo.jpg)

![](subdir/bar.jpg)

![](../baz.jpg)

more precisely the files layout is
./readme.md
./foo.jpg
./subdir/bar.jpg
../baz.jpg

markdown preview is able to render foo and bar but not baz
while editor opens the whole three correctly if we ctrl click each link in markdown
#165352 has introduced  a getMdPath that seems somewhat misleading there
PS Change Preview Security Settings seems to not apply any distinctions

Hi o314, i tried to recreate the same scenario that you have specified but i am able to render all three images in the preview and i am even attaching the screenshot of it. Along with the Explorer window in the left. So i didn't face any issue here,

Hi @shashankrp  Thanks for the quick feedback !
I will have a look at your proposal as fast as i can (tonight ?)
BTW this was reported from a Windows 10 os

I tried to recreate the issue and i didn't face any issue related to that so here i am attaching the screenshot below. ok thanks but i tried it on windows 11 OS

That still puzzles me here !
A note about the env : vscode running on windows 10
may be useful too: the workspace file is on a different drive than those files

Hi o314, can you please recheck if the image is in the actual path specified because i think vs code would work similar in all the different operating systems.

@o314 What folder did you open? VS Code will only load images from the current workspace. You can't use .. to traverse out of it

Hi @shashankrp,

if your using ../ then you could traverse only one folder back

Well, it does not seem to fail anymore in my case. I use ![](../../whatever) that works well.
@mjbvz it seems to be related with an over narrowed workspace effectively.
I have not seen this point on the doc page @ https://code.visualstudio.com/docs/languages/markdown ; it is very interaction oriented (as too often) and skips this important point IMO.
I will close this issue since everything works as expected on my machine.
May be a mention in doc could be useful
Thanks for your feedback anyway gentlemen.


it was working now wont start windows 7

ive tried to reinstall clean to no availe i am pasting the winows crash report
Problem signature:
Problem Event Name:   APPCRASH
Application Name: emulationstation.exe
Application Version:  <IP_ADDRESS>
Application Timestamp:    54fcbb08
Fault Module Name:    emulationstation.exe
Fault Module Version: <IP_ADDRESS>
Fault Module Timestamp:   54fcbb08
Exception Code:   c0000005
Exception Offset: 00182cf4
OS Version:   6.1.76<IP_ADDRESS>.256.1
Locale ID:    1033
Additional Information 1: 0a9e
Additional Information 2: 0a9e372d3b4ad19135b953a78882e789
Additional Information 3: 0a9e
Additional Information 4: 0a9e372d3b4ad19135b953a78882e789

Exactly the same issue here.
The error message is : emulator frontend has stop working.
Try to reinstall emulationstation without any effect...
On Windows 7 too... Any idea ?


[Network] az network public-ip create: Expose a new argument tier

Description

#16818: Cannot create Global anycast IP.
Expose a new argument tier for az network public-ip create so that customer can create Global public ip (Default tier is Regional).
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network

/azp run


fix to build

I think this should do the trick.  Using
.\build.cmd protofx

builds using a .NET Framework proto compiler instead of a .NET Core proto compiler.

My first PR merged ever :+1:


How to change font-weight?

I see that you can change font family like so:
{
  --font-family-serif: courier new;
  font-family: courier new;
}

But how do you change font weight? Specifically for a wl-textfield? Thanks.

Nevermind, figured it out.


I want to add C++ Program to demonstrate class template

Hey,
I would like to add the stated program. Kindly assign it to me

@KarandeepSingh15 please go through this repo and put your programs in proper folder.


Cannot read .vtk file using meshio

I use meshio to convert .vtk files to .msh file format as below.
volumetric_mesh = meshio.Mesh.read('./ellipsoid_0.vtk', "vtk")
But unfortunately I cannot do that and gives below error.
ReadError: Unknown section 'OFFSETS'.
Any solution to that please ?

MWE please

ReadError                                 Traceback (most recent call last)
 in ()
1 #volumetric_mesh_7 = meshio.read("G:/DUMMY_DATA_GENERATION/reference_geometry/meshes/ellipsoid_0.vtk")
----> 2 volumetric_mesh_7 = meshio.Mesh.read('G:/DUMMY_DATA_GENERATION/reference_geometry/meshes/ellipsoid_0.vtk', "vtk")
c:\users\hp\appdata\local\programs\python\python36\lib\site-packages\meshio_mesh.py in read(cls, path_or_buf, file_format)
193         from ._helpers import read
194
--> 195         return read(path_or_buf, file_format)
196
197     def sets_to_int_data(self):
c:\users\hp\appdata\local\programs\python\python36\lib\site-packages\meshio_helpers.py in read(filename, file_format)
66         raise ReadError(msg)
67
---> 68     return reader_mapfile_format
69
70
c:\users\hp\appdata\local\programs\python\python36\lib\site-packages\meshio\vtk_vtk.py in read(filename)
144     """
145     with open_file(filename, "rb") as f:
--> 146         out = read_buffer(f)
147     return out
148
c:\users\hp\appdata\local\programs\python\python36\lib\site-packages\meshio\vtk_vtk.py in read_buffer(f)
177             _read_section(f, info)
178         else:
--> 179             _read_subsection(f, info)
180
181     _check_mesh(info)
c:\users\hp\appdata\local\programs\python\python36\lib\site-packages\meshio\vtk_vtk.py in _read_subsection(f, info)
270         d.update(_read_fields(f, int(info.split[2]), info.is_ascii))
271     else:
--> 272         raise ReadError("Unknown section '{}'.".format(info.section))
273
274
ReadError: Unknown section 'OFFSETS'.

See this attachment please.
error_trace.pdf
`

You just posted the issue, not a way to reproduce the error. Also, the PDF you attached does nothing to clarify the issue.

https://en.wikipedia.org/wiki/Minimal_working_example

volumetric_mesh = meshio.Mesh.read('./ellipsoid_0.vtk', "vtk")
Please extract this attached zip file and use that .vtk file to reproduce the error.
ellipsoid_0.zip

This should be fixed by https://github.com/nschloe/meshio/pull/885.
PS: Please make yourself familar with GitHub formatting, especially code fences.


Settings: allow communication between elevated PowerToysSettings and WebView

Allowing this communication fixes (partially) some cases from #243, namely when the elevated PowerToysSettings.exe process hangs while opening the WebView window.

Fix available in 0.15.0 https://github.com/microsoft/PowerToys/releases


Add big.js dep in marginfi-client-v2/package.json

The big.js dep is missing in marginfi-client-v2/package.json .
This causes error when only the client is installed via
npm i @mrgnlabs/marginfi-client-v2

@chambaz should I rebase this or do you have something else in mind ?

@chambaz should I rebase this or do you have something else in mind ?

hey thanks for this! looks like a bunch of files were modified with style changes, can you submit again with just the package json change and Ill merge today.
cheers.

@chambaz should I rebase this or do you have something else in mind ?

hey thanks for this! looks like a bunch of files were modified with style changes, can you submit again with just the package json change and Ill merge today.
cheers.

@sbs2001 ignore me, it was early morning lol. rebasing and merging.


fix: convert parameters so they can be serialized

Some parameter types (e.g. Path or np) are not serializable when requesting extended benchmarks (and maybe in other other places #1425):
$ snakemake test.out -j 1 --benchmark-extended -F
Assuming unrestricted shared filesystem usage.
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided cores: 1 (use --cores to define parallelism)
Rules claiming more threads will be scaled down.
Job stats:
job      count
-----  -------
1            1
total        1

Select jobs to execute...
Execute 1 jobs...

[Thu Jul 11 11:07:47 2024]
localrule 1:
    output: test.out
    jobid: 0
    benchmark: test.jsonl
    reason: Forced execution
    resources: tmpdir=/tmp

WorkflowError:
TypeError: Object of type int64 is not JSON serializable
[Thu Jul 11 11:07:50 2024]
Error in rule 1:
    jobid: 0
    output: test.out

Exiting because a job execution failed. Look above for error message
WorkflowError:
At least one job did not complete successfully.
[Thu Jul 11 11:07:50 2024]
Error in rule 1:
    jobid: 0
    output: test.out

Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: .snakemake/log/2024-07-11T110746.832532.snakemake.log
WorkflowError:
At least one job did not complete successfully.

Not sure why, but the tests all pass! :confused:
QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

I think this has become obselete with #3175. Please reopen if I am wrong.


Migrate app to boilerplate

closing - app will be moved to mature support at 6.1. Use Minimal Gallery app instead.


Refactor Emojis by removing redundant codes

Refs: https://github.com/mobu-of-the-world/mobu/pull/479
Updates
Refactor to remove redundant codes related to Emojis

@kachick
Please review and merge if there's no concern! 😋

@kachick
Thank you for your review! I addressed all your points.
Could you check again?


Add in support for multiple ANT+ devices of the same type

This is a big one 😄
My wife's device keeps on pairing and I have to run upstairs to pair my footpod 🤣. To support multiple ANT+ devices I needed to do a lot of refactoring. If you feel this should be a fork instead of a PR let me know. Normally I would do this in a series of smaller PRs but I was low on time for this one.
Motivations for this PR:

My wife and I workout at the same time and I wanted to be able to choose what device is broadcasted over bluetooth. I had to keep on restarting the app until I get lucky with timing and it selects my device
Now that the HR sensor and the RSC sensor are added the code had a lot of boilerplate. There is lots of copying and pasting to add new sensors, I figured a reorganization of the code would be better to scale. Someone will have to add much less code to add new sensors.
I work in Kotlin a lot at work and Android has been going in that direction, Kotlin made this much quicker for me
Right now the Service is killed when the Android task is killed. It felt more intuitive to have a Stop button in the notification
I find myself using this app multiple times a week so I wanted something that worked perfectly for my use case. Hopefully it is useful for the community 😄
It was fun, I learnt a bunch about BLE

Disclaimers:

I do not have a Bsd or a Bc sensor so I was unable to test this sensor. They should be coded correctly though 🤞.
I do not have a combined Bsd/Bc sensor so I was unable to test this sensor. They should be coded correctly though 🤞.
I plan on making some more UI tweaks (spinners when things are loading, better displaying of units on device)
Long press to search for a device doesn't work anymore, you'd just restart the BLE Bridge

New Code Organization:

Activities/Fragments/Service just manage lifecycle, UI and communication between components
service/ant - has all ant specific code
service/ble - has all ble specific code
AntToBleBridge - manages the communication between the two

Adding a new Sensor
To add a new sensor you should be able to:

Add a new AntDevice, Add a new AntDeviceConnector
Add a new BleServiceType with all the BLE specific stuff, add it to the list of serviceTypes
Hook the two up between AntToBleBridge
No need to add UI or worry about lifecycle of Pcc or BLE

ScreenShots
You can see only one of the Stride SDM devices is broadcasting to BLE right now:

The updated Stop button:

Awesome PR. I had to find some time to digest this but really appreciated your work!! Thank you!!

Do we know if this awesome PR is going to get accepted any time soon?

Oh my goodness I completely forgot this. Merged.


New gamut mapping approaches

Scale Achroma: which is like scale, but scales the color towards an achromatic version of itself based on the channel furthest out of gamut.
Scale LH Achroma: Like Scale LH, but uses Scale Achroma. Also clamps white and black if L >1 or L < 0.
Scale LH Achroma2: Like Scale LH Achroma, but works to improve results by taking 2 passes to approach the gamut (while preserving LH) and then one final pass to place the color in gamut. This helps to reduce any over correction.
All are performed in linear Display P3 and OkLCh.

I spent some time refining this approach to make it worth submitting. Generally, I tend to prefer Scale LH Achromatic 2. Generally, it is in the ∆E range of Scale LH, give or take some, but hue delta and lightness delta are lower. Obviously, there are different metrics that one may judge GMAs on, but I'll just give an overview of my observations. It at least fixes many of the things that bothered me about Scale LH.

Tone mapping seems to perform better using the achromatic scaling:

Gradients often seem less surprising using achromatic scaling:

Gamut mapping in relation to other colors seems more sane:
Scale LH

Scale LH Achromatic

Scale LH Achromatic 2

I'm actually pulling Scale LH Achromatic 2 for now. I think it needs a bit more time to bake as there were still a few corner cases.
I think Scale LH Achromatic is actually much more reliable in its current state and I'm generally pleased with how it performs.  It gives good tones, good gradients, etc. and is much faster than the CSS GMA.
It should be noted that the dark blue region is the one place where Scale LH keeps ∆h closer than Scale LH Achromatic, but it does so by sacrificing ∆L. This, for me, is why blues seem to gamut map so abruptly with Scale LH in some cases (images as an example). These changes in lightness seem more jarring (at least to me) when viewed in context with gradients and images.  When this is occurring though ∆E is still comparable and often even better for Scale LH Achromatic, which seems to reinforce my feelings about the issues with scaling to the midpoint. Again, this is just my personal opinion.
Anyway, I think I am done tinkering with this for now. I'm pretty pleased with the alternate scaling. Hopefully, others will find this interesting.

Closing in favor of #449 which improves the over-correction that could occur (particularly in the low light blue that I mentioned earlier).


graylog: 3.3.9 -> 4.0.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/graylog/versions.
meta.description for graylog is: "Open source log management solution"
meta.homepage for graylog is: "https://www.graylog.org/"
meta.changelog for graylog is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7/bin/graylogctl passed the binary check.

/nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7/bin/.graylogctl-wrapped passed the binary check.

2 of 2 passed binary check by having a zero exit code.

2 of 2 passed binary check by having the new version present in output.

found 4.0.7 with grep in /nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7

found 4.0.7 in filename of file in /nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7

directory tree listing: https://gist.github.com/480c80a187a7c5660322b2494a6394f2

du listing: https://gist.github.com/10e60aa8a8ebf55d2c8f3e52fa625b0e

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
graylog

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A graylog https://github.com/r-ryantm/nixpkgs/archive/dab4502cc759ffed295af7586ab2dd7a86917091.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7
ls -la /nix/store/cjij51hi1170if0dnj5r0laf8p9q3hm9-graylog-4.0.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

graylog

Maintainer pings
cc @fadenb for testing.

https://github.com/NixOS/nixpkgs/pull/105645#pullrequestreview-542565623