RenderCallbackRule ignores ::trustedCallbacks()

Bug report
(Now that all Entity Query problems seem to be solved, I looked a bit closer at the Trusted Callbacks, so expect some more issues on that)
Currently \mglaman\PHPStanDrupal\Rules\Drupal\RenderCallbackRule only checks if the class implements TrustedCallbackInterface.
However the callbacks should also be in the return array from trustedCallbacks() to be trusted.
Now I highly doubt that people "in the wild" go through the trouble of creating callbacks and making sure the containing class implements TrustedCallbackInterface and finally don't add the callback to trustedCallbacks(), so the impact seems low.
However what we're currently doing seems incorrect.
Code snippet that reproduces the problem
See testcases in (extended) tests/src/Rules/data/bug-543.php

Yep, this is a big gap. I thought I had documented it in the code with a @todo but I did not!

// @todo: Write @todo...

Problem: we would need to call the method to get its values. I think. Even though the method is static, I don't believe it's return values can be parsed that way.

Ideas:
                preg_match('#^([a-zA-Z_\\x7f-\\xff\\\\][a-zA-Z0-9_\\x7f-\\xff\\\\]*)::([a-zA-Z_\\x7f-\\xff][a-zA-Z0-9_\\x7f-\\xff]*)\\z#', $constantStringType->getValue(), $matches);
$foo = (new ObjectType($matches[1]))->getClassReflection()
                        ->getNativeReflection()->getMethod($matches[2])

We can then do \ReflectionMethod::invoke with null for first param.
So:
$foo = (new ObjectType($matches[1]))->getClassReflection()
    ->getNativeReflection()
    ->getMethod($matches[2])
    ->invoke(null)

$foo should have the method names, then.

The PR is almost ready, but it crashes on Drupal core due to the way a class fixture is defined.


For the project's future maybe some sort of offline persistence would be nice to have

Is your feature request related to a problem? Please describe.
A lot of time, windows's network is broken, or not  available.
Windows breaks a lot with it's network/adapter/firewall.
Describe the solution you'd like
If the project can have a bit of offline persistence it would be nice to have

There is an offline exe, but you can always download the ps1 script here and run it in your powershell with an unrestricted execution policy.


Level Order Traversal in Tree is missing!!

Is your feature request related to a problem? Please describe.
Given the root of the Binary Tree. The task is to print the Level order traversal of a tree is breadth-first traversal for the tree.
Describe the solution you'd like
Level Order Binary Tree Traversal Using Queue
For each node, first, the node is visited and then its child nodes are put in a FIFO queue. Then again the first node is popped out and then its child nodes are put in a FIFO queue and repeat until the queue becomes empty.

@Kumar-laxmi
Other traversal techniques like preorder, postorder and inorder are present in the tree folder, but Level Order Traversal is missing, so i want to work on this. Pls assign me this issue.

I work with trees quite frequently and I have a good knowledge of them. please assign me this level order traversal. I will code this in C++.

I have already resolved the issue, just assign me and i will generate the pr.


(BOLT-989) Add documentation for wrapping scripts with a task

@melamos

CLA signed by all contributors.


PRT Persistancy - ft-agilefreaks | ft-distributed-lab

replaced with this PR
https://github.com/lavanet/lava/pull/792


Gelf compatibility

This PR introduces breaking changes to pino-gelf, hence the major
version bump.
The GELF spec at http://docs.graylog.org/en/3.0/pages/gelf.html#gelf-payload-specification
specifies a flat JSON structure. Custom fields have to be prefixed with an underscore and can
only contain string values. We do that now. All tests pass.
Also, a passthrough mode is being added. That way we can pass pino messages to pino-gelf and then
pipe pino-gelf's output to another tool. That way we can both send to a graylog server and pretty-print.
This addresses https://github.com/pinojs/pino-gelf/issues/4
Any feedback is welcome!

@mcollina Ping :)

Bump. Can this be merged? This also fixes #3

I'm not familiar with gelf - would this be a new major version?

I think you are right, it could warrant a new major version because this PR changes the output which could potentially be a breaking change is some setups. I can go ahead and bump the version number.

@jfahrenkrug would you say this is ready to go?

@jfahrenkrug would you say this is ready to go?

@davidmarkclements I think so. The version in the package.json is currently set to ""2.0.0-beta.0". Do you want to leave it this way and release a beta first? I'm not sure how many people use this package. If it's a small number, we could probably skip the beta phase and go straight to "2.0.0".

happy to skip beta

happy to skip beta

@davidmarkclements Excellent! I just changed the version to "2.0.0", so it's ready to be merged now.

@davidmarkclements I don't have write access, would you be able to merge it?


Job did not get updated on heroku

I have written a job which runs every 1 min, deployed to heroku, it ran perfectly fine.
Then I updated that job to run in every 120 mins, when I deployed to heroku the job is still running every 1 min. (In my local env. I changed 1min->5min, it worked fine)
I am using RedisToGo addon on heroku.
The job creation script is in one of the initializers which is as follows:
#initializers/sidekiq.rb

hash = {
  'etl_job' => {
    'class' => 'EtlWorker',
    'cron'  => '*/120 * * * *'
  }
}

if Sidekiq.server?
  Sidekiq::Cron::Job.load_from_hash! hash
end

I tried restarting the worker, app and even RedisToGo server, still no change.
Am I doing something wrong?

Hi job settings is saved in redis so did you updated redis configuration? You can do it by Sidekiq::Cron::Job.load_from_hash! You do not need to restart sidekiq for changing cron.

BTW why do you do:
if Sidekiq.server?

try to remove that.

Thanks for reply.
I was just following the readme that's why used Sidekiq.server?
Anyways I got the problem.
It was invalid cron expression.
For 120 mins i.e. 2 hours, correct cron expression is like this '0 */2 * * *'

@manish-thinkbridge so everything work fine for you?

oh .... yeah :+1:


Days to rollover and Avg limit are wrong since month rollover

broken because I was using getMonth() will be resolved in next release or feel free to use dev branch as it's fixed.

should be resolve by commit https://github.com/lukealford/aussiebb_tray/commit/dc3c7d84d9620bb98e6942c69b1cb16d0905a60e in dev branch


"First cycle start" date does not work

Setting the "First Cycle Start" date in the advanced settings does not appear to do anything. ie - After setting this to a future date, the app still displays a reminder for "today" (missed or future dose).
Also: [feature request] It would be nice to display the First Cycle start date for each item in the medicine list, or in the reminder/dosage cards in the medicine edit view.

I can confirm that it does not work for future cycle start days, I found the cause and will fix it in the next release.
The feature request will also be part of the next release.


Feat/react flow

This PR changed the diagramming library from https://github.com/projectstorm/react-diagrams to https://reactflow.dev/
the biggest issue with the old library was around z-indexing of elements as well as issues with zoom. For example, try this demo, (http://projectstorm.cloud/react-diagrams/?path=/story/simple-usage--demo-simple), zoom in, and try to connect a new node. The front of the selector and the mouse cursor will be out of sync.
The replacement uses some of the same implementation as the old on in terms of tag filtering, so it may need to be tweaked a little.

LGTM!


add integration tests for rank, dense_rank, fix last_value evaluation with rank

Which issue does this PR close?
add integration tests for rank, dense_rank, fix last_value evaluation with rank
Closes follow ups on #555
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@alamb and @Dandandan this pull request is ready now


Fix Communication CallAutomation README Issue

Fixes #40920

API change check
API changes are not detected in this pull request.

@scottaddie , @jsquire , @lfraleigh , @mayurid and @acsdevx-msft Could you help review this PR and merge it if it looks good?

@scottaddie , @jsquire , @lfraleigh , @mayurid and @acsdevx-msft Could you help review this PR and merge it if it looks good?

Straightforward typo fix; as I don't expect any controversy for the change, I approved and merged.


[SPARK-24231][PYSPARK][ML]  Provide Python API for evaluateEachIteration for spark.ml GBTs

What changes were proposed in this pull request?
Add evaluateEachIteration for GBTClassification and GBTRegressionModel
How was this patch tested?
doctest
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #90655 has started for PR 21335 at commit 5765049.

Test build #90655 has finished for PR 21335 at commit 5765049.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90655/
Test FAILed.

Test build #90657 has started for PR 21335 at commit d02797c.

Test build #90657 has finished for PR 21335 at commit d02797c.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90657/
Test FAILed.

Merged build finished. Test FAILed.

Test build #90658 has started for PR 21335 at commit 17be21c.

Test build #90658 has finished for PR 21335 at commit 17be21c.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90658/
Test PASSed.

LGTM. Merged into master. Thanks!


Out of bounds value

Hi.
I'm currently using ResponsiveAnalogRead for a joystick projet. The joystick I use has two pots, X and Y. Very classic setup.
I use calibration phase, during which I push the joystick far in each direction. So far, it works fine. Once the calibration is done, I run a loop into which I read the values, and sends them through USB.
The problem happens right after the calibration.
Here's the code I use:
sketch.txt
And its output:

Calibration ended:
xMin = 198
xMax = 224
yMin = 216
yMax = 244
All is ok, acting like a real stick now!!
 But right after it, Y axis goes nuts and gives me out of bounds values. This is very strange, as I use the very same treatment for X, and X works perfectly.

BTW, I use Teensy 3.2 and Teensy LC


Add docs PCI passthrough

What this PR does / why we need it:
Added a document for PCI passthrough configuration on CAPK to avoid unnecessary trial and error.
Although we can find a clue from KubeVirt document, to make the entire steps clear, would like to suggest adding the document to CAPK.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #192
Special notes for your reviewer:
I think the release notes are not needed for this small change. If needed, please let me know.
Release notes:

NONE

/lgtm


UI - Threads become unresponsive to chat after a period of inactivity.

Electron build with commit id - e7bb77ab
Steps to reproduce the problem:

Have few assistants and chat with them so that you have few threads created.
Chat in one of the threads - say thread1
Leave the thread and continue to chat on other threads.
Revisit thread1 after sometime (In my case it was about 1 hour later).
Notice that there is no response for chat messages

Go to any other thread and re enter this thread.
Notice that the last 2 messages that did not get a response are gone from the thread.
We will be able to chat and get a response successfully.

@sangee2004 This issue should be fixed. The root cause is that socket lose functionality once it is reconnected back.
You should be able to easily reproduce by disconnecting network from your chrome browser tab and reconnect back after a while. The thread will stop functioning once it is reconnected back.


Add introduction

Hi Nico, can you add the following introduction:
This dashboard attempts to give an overview of all muskrat and coypu observations (both catches and field observations) in Flanders, the Netherlands and certain areas in Germany. This dashboard is part of the LIFE MICA project, in which innovative techniques are tested for a more efficient control of muskrat and coypu populations, both invasive species. This dashboard will strengthen cooperation between neighboring countries because information on the presence of muskrats and coypu will be readily shared and used to inform management decisions.
Be careful when interpreting the data! Not all data is updated regularly and more recent data may therefore be incomplete.
This dashboard was created as part of the MICA project, which received funding from the European Union’s LIFE Environment sub-programme under the grant agreement LIFE18 NAT/NL/001047. (this part can be a bit smaller at the bottom)

Sure, will do that soon!


KNL-1475 remove the unnecessary sub-select from getUsersIsAllowed

that can cause terrible MySQL query performance in sites with 150+ groups.
Primary change is towards bottom of PR look for getSelectRealmRoleUserIdSql. I remove the sub-select as it's unnecessary, can cause bad performance, requires 1+ temporary tables, and is confusing.

LGTM

Yeah, it doesn't look like it's a useful clause and this change looks good. I think the only way you'd know for 100% sure it to expand the very basic DbAuthzTest and actually test out the method. I don't see this method tested anywhere. Or just accept as-is and try in production like the rest of this code is

Yes we are now live in production with this with a large institution. I don't think this is a problem for 99% of institutions but a large institution using MySQL 5.6 with block_nested_loop=off and a site with 150+ groups may run into this issue.

Yeah, I don't even know why there was an override, probably something that was just never noticed for 10 years.


eMOU enhancements

If the product team enters a new account coding, display the message: "No eMOU exists for this account coding. We will initiate the process by sending an email to the EA for their signature. After the eMOU is signed, it will be reviewed and approved, which typically takes up to 2 business days."

Display the description and budget when the EA views the approval page.

eMOU Status Changes:

Current Step: Pending sign-off from the Ministry Expense Authority (EA's name)
Next Step: Sign-off by the OCIO Cloud Director
--> Sign-Off Complete

Enable eMOU download for other products with the same account coding.

Update the PDF to include the agreement paragraphs and redesign it to match the current PDF.

Check if the division is available via the MS Graph API.

Send emails to both the EA and the Cloud Director with the PDF:

For new account coding: After approval by the Cloud Director.
For existing account coding: After submitting the requests.

Update the existing message to: "Our records show that your team already has a signed MoU with OCIO for AWS/Azure use. This new product will be added to the existing MoU. A copy of the signed MoU for this product will be emailed to the Ministry Expense Authority."

OCIO and (TEAM NAME)- AWS (1).docx

If User uses for Azure product the Billing Code, that is already used for existing signed and approved AWS product, the message appears that says two things:

that MoU is signed for Azure, though it was signed for AWS
that account coding is currently undergoing an approval process, while it was already approved


👁 Show password eye on database form

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#218  👈
#217
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RTLS and the rest of your teammates on  Graphite


Clean up generated reference doc output

Description
The generated reference docs are very sloppy right now and need cleaning up. There must be a way to improve the parsing of the source material to make the output more readable.
Context
Example: https://awslabs.github.io/aws-sdk-swift/reference/0.x/AWSS3/S3Client#copyobjectinputcompletion
Everything's just one huge paragraph, more or less, with no breaks or layout changes.
Observed Behavior

Expected Behavior
At a minimum, paragraph breaks should be preserved. Ideally, subsections would be added when appropriate and feasible, and other formatting changes applied as appropriate.
Possible Solution

Closing as API reference docs have been revamped now.


Remove unused SV file that was used for testing

We have a more comprehensive test suite in the 'test' directory.

Closed #39


Pre-render R script issue with quarto preview

Running the _common.R script before rendering causes issues with quarto preview
project:
  type: book
  pre-render: _common.R

The preview window continuously refreshes and the script is run over and over and gets the following warning: WARNING: connection closed before message completed.
The issue is resolved when bbt_update_bib(ref_files, here("book.bib")) is removed from the script.
Temporary fix (only run script when project built to all formats):
if (!nzchar(Sys.getenv("QUARTO_PROJECT_RENDER_ALL"))) {
  quit()
}

No longer an issue.


How to change color for inactive status line?

It's getting a bit hard to see:

What color group is responsible for that?

doing some housecleaning.  please re-open if you need additional help setting colors for the inactive group.


Updated path not found error to be a bit more descriptive.

Simply changes "path not found" to something slightly more descriptive, anything would work here.
I ran into the issues today while deploying a docker container just receiving the error:
Error: path not found

Caused by:
    path not found

Often dotenvy is being loaded at the beginning of projects, and this error isn't super descriptive, I took (far too long) realizing what path was not being found, as I wasn't copying over my .env to my runtime environment.
I think a change like this could help avoid any silly mistakes for others in the future, but if not no problem, just a suggestion!

Thank you for your contribution :)


Update README.md

@AlbMata can this be merged?

@jamesgoodwin Yes. If Oleg didn't change anything in the integration process then the changes are only formatting and additional clarification lines to help the UX.


.*: initial staticcheck implementation

Changes

Verification

As suggested here https://github.com/improbable-eng/thanos/pull/1184#issuecomment-496614283, I think we should just use golangci, as we will inevitably add all of them.

Yes, it looks good from the onset however I have never used it. Could we discuss a bit about it in #thanos-dev when you'll have time?

Oh, that definitely looks good 👍 we should really probably migrate to it


Clear hooks in RemoteHook::Reset?

Should _hooks.clear() be called in RemoteHook::Reset?  Eventhough Restore( hook.second, hook.first ); is called, but _hooks still keeps it.

Yeap. Fixed it


Implement date related expression like to_date, date_sub, datediff

These date related expressions are very common in real-world application. Without these supported in gazelle, fallback is needed with overhead introduced.

@PHILO-HE  please take a look on this
https://github.com/oap-project/gazelle_plugin/pull/235

datediff is already supported.


Deis Workflow deis-builder component fails to start

known issue it should be fix in the new Deis Workflow v2.2 https://github.com/deis/builder/pull/382

the new Deis Workflow v2.2.0 works fine now, so closing the issue


Scroll to top on navigation

This doesn't handle scrolling to correct position on
back/forward. react-router-scroll does this but doesn't
support react-router v4.
Fixes #95

Seems to work fine!


fix(dashboard): add num pending transactions

fixes #48

:tada: This PR is included in version 1.0.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can someone help to run unittests locally ?

https://github.com/exacaster/lighter/blob/master/docs/CONTRIBUTING.md#server


Problem with Function which have the same name but different input Parameters

In Postgres you can have the same Function with multiple different input Parameters.
Example myFunction(id) and myFunction(id,date)
In this case those functions always shown as different as you just care for the function Name while compare the data.
If you could use also the Field proargnames as the identifier the Problem should be solved. With this you should be able to find the function for the param set corosponding function.

Confirmed here, makes using pgdiff super difficult when using extensions like postgis


chore: ab testing framework

Adds an A/B testing framework to Dendron. To demonstrate this framework, also adds an A/B test for displaying a toast with a link to the changelog, versus displaying the changelog page in a webview.
Docs: https://github.com/dendronhq/dendron-docs/pull/25
The webview looks like this:

Co-authored-by: jonathanyeung<EMAIL_ADDRESS>Dendron Extended PR Checklist
Code
Basics

[x] code should follow Code Conventions
[x] circular dependency check: make sure your code is not introducing new circular dependencies in plugin-core.  See Avoiding Circular Dependencies.
[x] sticking to existing conventions instead of creating new ones

eg: if configuration for utilities are already in one module or package, add future utilities there as well

Extended

General

[x] check whether code be simplified
[x] check if similar function already exist in the codebase. if so, can it be re-used?
[x] check if this change adversely impact performance

Operations

[x] when shipping this change, will it just work or will it introduce additional operational overhead due to complicated interface or known bugs?

Architecture

[x] check if code is introducing changes on a foundational class or interface. if so, call for design review if needed

eg: making changes to DNode

Instrumentation
Basics

[x] if you are adding analytics related changes, make sure you Document changes in airtable

Extended

[x] can we track the performance of this change to know if it is successful?

eg: see usage for export pod

Tests
Basics

[x] Write Tests
[x] Confirm existing tests pass
[x] Confirm manual testing
[x] Common cases tested
[x] 1-2 Edge cases tested
[x] If your tests changes an existing snapshot, snapshots have been updated

Extended

[ ] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), an example is included in the test workspace

CSS

[ ] display is correct for following dimensions

[ ] sm: screen ≥ 576px, eg. iphonex, (375x812)
[ ] lg: screen ≥ 992px
[ ] xxl: screen ≥ 1600px eg. mac (1600x900)

[ ] display is correct for following browsers (across the various dimensions)

[ ] safari
[ ] firefox
[ ] chrome

Docs
Basics

[x] if your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Basics

[x] does this change introduce a new or better way of doing things that others need to be aware of? if so, an async should be created and a process added in Development or Packages

Close the Loop
Basics
Extended

[ ]  is this a developer BREAKING change? if another person cloning from this branch will need to adjust their dependencies or mental model of the architecture, then it is. if this is the case, make sure this is communicated according to Close Loop

eg. breaking dev change due to new dependency

merging and pushing forward to v90


⚠ Bofrost.lu has degraded performance

In a92c916, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Bofrost.lu performance has improved in c51680e after 21 minutes.


Fix: Multiple Usage of Authenticators/Conditions in Authentication Sub-Flows

What this PR does / why we need it:
Which issue this PR fixes: fixes #978
Special notes for your reviewer:
See Linked Bug-Ticket for detailed description of the problem.
Subflow-Executions are created without special configuration and then modified according the definiton in the input file. This requires the config-tool to query keycloak for the created execution. If we have multiple executions with different configs within one subflow they could not be found distinguished.
This PR adds a filter to the repository method to return executions with corect "alias"/"configuration" which finally allows users to have sub flows with usage of same authenticators/conditions.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Hi @f11h,
I tested your PR locally, and all tests passed successfully. The changes work as intended!
However, the GitHub workflow is failing due to errors in ImportParallelImportIT during realm creation and updates. This might not be directly related to your changes, but it’s worth checking. Also, the line:
.filter((config) -> config.getAuthenticatorConfig().equals("config-1"))
    .collect(Collectors.toList());

seems to be missing the Collectors import, which might be causing the issue. Additionally, there are conflicts with the main branch that need resolving. If you need help with this or reviewing the failed tests, just let me know!
Thanks for your hard work!


<script>'s type attribute is a little odd with this change.
<script type="module"> means a JavaScript module, while <script type="webassembly"> also means a module, but a different type of module.  This is not super intuitive, but I don't really see a backwards-compatible workaround.  I guess this can be justified by saying that there is no such thing as a 'classic' WebAssembly script (is that right?) so the distinction is clear in context.
Same thing with Workers; https://github.com/littledan/proposal-module-attributes#worker-instantiation conflicts a bit with the current way that the worker options dict is used to decide whether a Worker should be classic or module: https://html.spec.whatwg.org/multipage/workers.html#worker-processing-model
const worker = new Worker('resource.json', {type:'module'});

or
const worker = new Worker('resource.json', {type:'classic'});

Now we would also have
const worker = new Worker('resource.json', {type:'webassembly'});

Which is a little confusing because it is also a module, just not a JS one.
I don't see a good way to address this as changing <script type="module"> is obviously a non-starter.  I thought I'd make a note of the issue though, as this is likely a criticism that we'll have to speak to.

Yes, I was thinking of using a secondary label, but I thought the overload was a good trade-off since those don't have many possible values anyway. Note that #7 points to more possible overloading.

@littledan the web has few values, but that is not necessarily true of other environments. Deno has typescript, node loaders allow arbitrary formats, etc.

@bmeck Maybe you meant to post that on #8 or #10?

@littledan it was in direct reply to:

since those don't have many possible values anyway

This isn't necessarily true in other environments especially with Worker APIs.

Cross-references would be useful here.

Node Loaders supporting arbitrary types: https://github.com/nodejs/modules/issues/115 , loader design work is ongoing but this is a first class feature
Workers

in Node: https://nodejs.org/dist/latest-v13.x/docs/api/worker_threads.html
in Deno: https://github.com/denoland/deno/pull/1993/files (no docs?, changes show loading typescript)

Reviewing these, I'm having trouble finding any currently shipping feature that would lead to compatibility/interoperability issues with using a { type: ... } option as a parameter to initialize workers. Am I missing something?

@littledan currently workers can load WASM via using type:module. I'm stating that the naming collision here doesn't make sense to try and overload.

Right, so, I was suggesting that we could join a namespace that has just a few entries, and work through the collisions. (And I imagine that, if you're talking about Node, you mean an experimental flag.) It still seems to me like that's a possible path.
But I'm also open to passing these options in a separate place. For example, the API for workers on the Web could be:
const worker = new Worker('resource.json', {with: {type:'webassembly'}});

I'm leaning towards using the syntax described above:
const worker = new Worker('resource.json', {with: {type:'webassembly'}});

This avoids all risk of namespace collisions. It's also used in import().
Also, for HTML tags, the withtype attribute would be used:
<script src="foo.wasm" type="module" withtype="webassembly"></script>

This is all checked into master.
I think these host integration things are something that we should work out by Stage 3.

I'm closing this issue, as the overloading has been removed.


Migrate portfolio to function

Changes

no visual changes
split Portfolio into multiple independent components
Portfolio is now function component using hooks instead of class component

Fixes

fixed a small issue when portfolio shows an empty state for a split second before rendering portfolio items
portfolio items now properly shows loading state when switching between portfolios

Codecov Report

Merging #224 into master will increase coverage by 0.58%.
The diff coverage is 81.13%.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
+ Coverage   84.33%   84.91%   +0.58%
==========================================
  Files          95       98       +3
  Lines         983      988       +5
  Branches       86       85       -1
==========================================
+ Hits          829      839      +10
+ Misses        144      139       -5
  Partials       10       10

Impacted Files
Coverage Δ

...portfolio-item-detail/copy-portfolio-item-modal.js
25% <0%> (+0.86%)
:arrow_up:

...c/smart-components/portfolio/portfolio-products.js
100% <100%> (ø)

...mart-components/portfolio/portfolio-empty-state.js
100% <100%> (ø)

src/helpers/share/share-helper.js
100% <100%> (ø)
:arrow_up:

src/helpers/shared/redux-mutators.js
100% <100%> (ø)

src/smart-components/portfolio/portfolio.js
78.94% <77.14%> (+3.94%)
:arrow_up:

src/toolbar/schemas/portfolio-toolbar.schema.js
92.3% <80%> (+8.97%)
:arrow_up:

Continue to review full report at Codecov.

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

@Hyperkid123 - looks good, only one issue:
on the Add Products page - the Filter is not displaying the input, but is filtering correctly:

@lgalis filtering should be OK

@Hyperkid123 - thank you - looks good, merging.


Hiding Table Columns for different screen sizes.

We are trying to figure out how to hide table columns when the screen size changes.  Is this possible?  For example if we have 4 table columns on the large screen we would like to only show 2 columns on the mobile screen.

This can be done with bootstrap 5 classes.  In your column definition just add a classes parameter using bootstrap 5 visibility classes.  Remember to use table-cell instead of block.  So for example to hide a column in every display bug lg you can do "d-none d-lg-table-cell".
https://getbootstrap.com/docs/5.0/utilities/display/

Ok, I see that now.  Thank you.
On Mon, Mar 6, 2023 at 4:32 PM Ian Starnes @.***> wrote:

This can be done with bootstrap 5 classes. In your column definition just
add a classes parameter using bootstrap 5 visibility classes. Remember to
use table-cell instead of block. So for example to hide a column in every
display bug lg you can do "d-none d-lg-table-cell".
https://getbootstrap.com/docs/5.0/utilities/display/
—
Reply to this email directly, view it on GitHub
https://github.com/311labs/swam/issues/6#issuecomment-1457276275, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A6GB3M6EOBCNXRG5OP5UDA3W2Z63FANCNFSM6AAAAAAVRZS6S4
.
You are receiving this because you authored the thread.Message ID:
@.***>


SSL validation of HTTPS requests

After updating to Chef 11.12.2 there is warning:
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
SSL validation of HTTPS requests is disabled. HTTPS connections are still
encrypted, but chef is not able to detect forged replies or man in the middle
attacks.

To fix this issue add an entry like this to your configuration file:

Verify all HTTPS connections (recommended)
ssl_verify_mode :verify_peer
OR, Verify only connections to chef-server
verify_api_cert true

To check your SSL configuration, or troubleshoot errors, you can use the
`knife ssl check` command like so:

knife ssl check -c /etc/chef/client.rb

* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

What is the best way to fix this, any suggestions?

You need to update your Chef client.rb to enable this. In Chef 12 it's on by default.  Check docs.chef.io for client.rb suggestions


Have to edit script to parse # baubles?

Would like to pass in the GUI / terminal how many baubles I have as they are used frequently and should not be edited in scripts (or am I doing this wrong).

I think this is a cool idea / I do believe it could be done.. It may be tricky trying to OCR the # of baubles? But I could also see it working by asking for a screenshot of 0 baubles and using them if SSIM doesn't match.
It may also work as a dumb function / trying to use baubles even if you have none.. but I haven't tested how this would work in the game.
In practice, I was being lazy and usually just set mine to 60/90 and always bought that many baubles before I left the bot going, bit I certainly think it should be moved to a config at the very least!

(Unfortunately I'm not sure if my WoW subscription is active anymore, so I don't know if I will be making any further improvements at this time -- I, however, welcome any improvements either submitted as Pull Requests, or if you just wanted to make your own fork and run with it! =D)

I would rather require the User to pass the # of baubles at the start of the script (heavily reduces complexity and OCR is really prone to errors).
I'll add that when I have time. Might provide a little GUI while I'm at it


ENH: add a query of Slicer extensions list

This is related to Slicer issue http://www.na-mic.org/Bug/view.php?id=4200
Note that this test succeeds

while executing the same query via Slicer is failing:

FYI @jcfr

Closing. This was superseded by refactoring (https://github.com/commontk/qRestAPI/pull/21) and transition to Girder (https://github.com/commontk/qRestAPI/pull/20)


Unclear/misleading README.md

Hi @chriselrod,
I think that the current README.md doesn't do a particularly good job at explaining how Polyester.jl works with respect to three (crucial) points:

How many "Polyester threads" are there in total (and does it vary dynamically)?
Are "Polyester threads" pinned to CPU-cores/CPU-threads?
How do "Polyester threads" relate to / cooperate with / interfere with Julia's task-based multithreading?

I think that sentences like the following are insufficient and potentially misleading:

Note that @batch defaults to using up to one thread per physical core, instead of 1 thread per CPU thread.

It suggests that Polyester potentially changes the number of threads (1) and also pins them to CPU-cores/threads (2). Of course, neither is the case. To be fair, that Polyester doesn't do (2) is mentioned in the README.md but only in a comment in a code example, which can easily be overlooked. The fact that Polyester just uses the already existing Julia threads (1) is also mentioned in the README (at a more prominent place) but is potentially misleading in the context of everything else that follows.
As for point (3) above, the README only compares to @threads which (as of now) defaults to :dynamic scheduling. This seems somewhat strange/misleading because (IIUC) @batch creates sticky tasks and and schedules the workload statically. Hence, it should really be compared to @threads :static. Of course, one can also compare to @threads :dynamic but the conceptual differences should be highlighted then IMHO.
Overall, IIUC, Polyester really just creates, manages, and re-uses a bunch of sticky tasks and just runs them on existing Julia threads (similar to @threads :static). However, IMHO, this isn't clear from the README.md which doesn't contain the words "sticky", "task(s)" or, most importantly, "static" at all.
If you're open to it, I might make a suggestion for an improved README.md in a PR later this week. Let me know what you think!
(BTW, apart from the stride option, Polyester.jl doesn't implement any form of load-balancing, right?)

Overall, IIUC, Polyester really just creates, manages, and re-uses a bunch of sticky tasks and just runs them on existing Julia threads (similar to @threads :static).

@threads :static does not reuse tasks, which is much of why Polyester should have lower overhead. I don't think the README examples have been rerun in a long time.
It might be that they haven't been re-run since before the :static to :dynamic switch.

If you're open to it, I might make a suggestion for an improved README.md in a PR later this week. Let me know what you think!

Certainly!
"3." is also definitely a major pain point.

BTW, apart from the stride option, Polyester.jl doesn't implement any form of load-balancing, right?

It does not. Static scheduling is much easier to implement. ;)
Perhaps it could be fun to try implementing something like a work stealing scheduler at some point, but at that point, it may also be worth looking at Base Julia's scheduler.

@threads :static does not reuse tasks, which is much of why Polyester should have lower overhead.

I know, but it isn't clear from the README :)

I don't think the README examples have been rerun in a long time.
It might be that they haven't been re-run since before the :static to :dynamic switch.

Makes sense, I will rerun the benchmarks when I update the README.

It does not. Static scheduling is much easier to implement. ;)
Perhaps it could be fun to try implementing something like a work stealing scheduler at some point, but at that point, it may also be worth looking at Base Julia's scheduler.

Yes, a work stealing scheduler would be nice. But I agree, it would be nice if we could just have one smart scheduler instead of multiple (FWIW, Dagger.jl also implements a scheduler etc.)

I'm rewriting the README.md and have a question about the following example
julia> let
           @batch threadlocal=rand(10:99) for i in 0:9
               println("index $i, thread $(Threads.threadid()), local storage $threadlocal")
               threadlocal += 1
           end
           println(threadlocal)
       end

index 8, thread 1, local storage 30
index 3, thread 3, local storage 49
index 9, thread 1, local storage 31
index 6, thread 4, local storage 33
index 0, thread 2, local storage 14
index 4, thread 3, local storage 50
index 7, thread 4, local storage 34
index 1, thread 2, local storage 15
index 5, thread 3, local storage 51
index 2, thread 2, local storage 16
Any[32, 17, 52, 35]

The storage doesn't seem to be thread-local: Julia thread 1 appears twice, once with local storage 30, another time with local storage 31. It also makes sense to me that it isn't thread-local. I would expect it to be "Polyester task"-local. Is that's what's happening here? If that's the case, I think threadlocal is a big misnomer.
BTW, is there a function like Polyester.taskid() that, similar to threadid() for Julia threads, returns a unique ID of a Polyester task. If not, would it make sense to add this?

Also, it doesn't work at all?
julia> let
           @batch threadlocal=rand(10:99) for i in 0:9
               println("index $i, thread $(Threads.threadid()), local storage $threadlocal")
               threadlocal += 1
           end
           println(threadlocal)
       end
index 0, thread 1, local storage 35
index 1, thread 1, local storage 36
index 2, thread 1, local storage 37
index 3, thread 1, local storage 38
index 4, thread 1, local storage 39
index 5, thread 1, local storage 40
index 6, thread 1, local storage 41
index 7, thread 1, local storage 42
index 8, thread 1, local storage 43
index 9, thread 1, local storage 44
Any[45, #undef, #undef, #undef, #undef, #undef, #undef, #undef]

julia> Threads.nthreads()
8

Eh, I suspect it's a bit broken.
Performance also isn't great.
Would be great if I (or someone else) can get around to reimplementing.
I described the approach I'd take a few times, which is to actually make it polyester-task, which is already how LoopVectorization.jl does it. This lets @tturbo can multithread accumulations like sums/dot products without allocations.
threadlocal in Polyester is implemented differently.
I'd have to check the source to see what it's actually doing (I should remember because I reviewed/approved the PR...alas), but I'd just discourage its use until it gets fixed.

@carstenbauer, I was just trying to make sense of that threadlocal example and I agree it’s a bit confusing, but it seems to be consistent: The thread-local storage is initialized to a random value for each thread (here 30, 14, 49, and 33) and each iteration prints & increments the local value, so thread 1 prints 30 & 31, thread 2 prints 14, 15 & 16, and so on.


Flatiron Jenkins continuous integration

With Dylan's help, we are setting up GPU testing w/ Jenkins at Flatiron Inst.

All pytest files run green on both gpu and cpu

Codecov Report

Merging #139 (3f01efa) into main (720b10c) will decrease coverage by 32.76%.
The diff coverage is 100.00%.

:exclamation: Current head 3f01efa differs from pull request most recent head 53e6389. Consider uploading reports for the commit 53e6389 to get more accurate results

@@             Coverage Diff             @@
##             main     #139       +/-   ##
===========================================
- Coverage   78.39%   45.62%   -32.77%
===========================================
  Files          33       33
  Lines        2971     2970        -1
  Branches      593      593
===========================================
- Hits         2329     1355      -974
- Misses        465     1494     +1029
+ Partials      177      121       -56

Impacted Files
Coverage Δ

plenoptic/tools/data.py
57.50% <ø> (-13.34%)
:arrow_down:

plenoptic/metric/perceptual_distance.py
77.96% <100.00%> (-3.39%)
:arrow_down:

...enoptic/simulate/canonical_computations/filters.py
97.05% <100.00%> (-0.09%)
:arrow_down:

plenoptic/simulate/models/naive.py
91.89% <100.00%> (-1.69%)
:arrow_down:

plenoptic/synthesize/eigendistortion.py
94.66% <100.00%> (ø)

plenoptic/tools/conv.py
57.81% <100.00%> (ø)

plenoptic/tools/signal.py
35.25% <100.00%> (ø)

plenoptic/synthesize/mad_competition.py
9.97% <0.00%> (-76.34%)
:arrow_down:

plenoptic/tools/display.py
10.93% <0.00%> (-58.99%)
:arrow_down:

plenoptic/synthesize/synthesis.py
21.83% <0.00%> (-53.17%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


regions plugin: ensure edgeScrollWidth is set (#2401)

Title: Fix edgescroll for regions created after region plugin
Short description of changes:
Looking for a value for edgeScrollWidth for a region, either from plugin params or a default value, was only happening for regions specified in the plugin params; it now happens for all regions. The consequence of it not being set was that the logic for doing edgescrolling was doing calculations with undefined, and so never doing any scrolling.
Breaking in the external API:
None
Breaking changes in the internal API:
None
Todos/Notes:
Related Issues and other PRs:
The bug was raised as issue #2401. The problem was noticed in the course of pull request #2294.

Coverage remained the same at 82.039% when pulling d285eb8ae821e6603c8aa083c740341df305b364 on rdn32:edge-scroll-width-param-fix into 00b9f7e4dcd04f66b5c7a3ae552aa9fb6ed588b4 on katspaugh:master.


Upgrade pgx and add go modules

Upgrade pgx to v4
Add go modules
Add docker-compose for postgres
Run test on Travis

Instead of #36
CI
https://travis-ci.org/github/gadelkareem/que/builds/736319827

@gadelkareem I saw your comment on the previous branch. I'm not the maintainer. Was it previously communicated this package is no longer being maintained? Sorry, just trying to determine how I should proceed with some changes I'd like to propose.

@grippy Actually I have no idea. I send the PR long ago but I did not get a reply to I just started to use my fork https://github.com/gadelkareem/que

up

Hi all, I recommend you check out my new project, River. River already does quite a bit more than que-go, uses a much more scalable design, and is designed for modern Go and pgx/v5.
I will be archiving this repository shortly.


AtomicPuppy does not support authentication

No support for authenticating against eventstore, which is required when default ACLs that don't grant read to $all have been applied

https://github.com/madedotcom/atomicpuppy/pull/63


Profiler doesn't work correctly in tensorflow 2.1

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): centos-release-7-6.1810.2.el7.centos.x86_64
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.1.0
Python version: 3.7.4
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory: None

Describe the current behavior
The code below, taken from https://www.tensorflow.org/tensorboard/graphs, and used to profile a function call, seems to behave differently in tf 2.0 and tf 2.1. In 2.0, the memory and CPU usage can be inspected with tensorboard; in 2.1, they cannot as the radio buttons are grayed out. I am filing this as a tensorflow issue rather a tensorboard one since it seems to be independent of the tensorboard version. I also suspect it's due to the tensorflow pip package now including GPU support by default (the problem appears on a machine without a GPU).
# The function to be traced.
@tf.function
def my_func(x, y):
  # A simple hand-rolled layer.
  return tf.nn.relu(tf.matmul(x, y))

# Set up logging.
stamp = datetime.now().strftime("%Y%m%d-%H%M%S")
logdir = 'logs/func/%s' % stamp
writer = tf.summary.create_file_writer(logdir)

# Sample data for your function.
x = tf.random.uniform((3, 3))
y = tf.random.uniform((3, 3))

# Bracket the function call with
# tf.summary.trace_on() and tf.summary.trace_export().
tf.summary.trace_on(graph=True, profiler=True)
# Call only one tf.function when tracing.
z = my_func(x, y)
with writer.as_default():
  tf.summary.trace_export(
      name="my_func_trace",
      step=0,
      profiler_outdir=logdir)

Describe the expected behavior
It should be the behaviour found in tf 2.0.
Other info / logs
Screenshots of the two tensorboard displays are shown below. Of note, running the code with tf 2.1 shows the errors below, which do not appear with tf 2.0. I do not have a GPU on my machine, but would still expect the profiler to work.
(...)
2020-02-11 10:26:17.256467: I tensorflow/core/profiler/lib/profiler_session.cc:225] Profiler session started.
2020-02-11 10:26:17.256643: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libcupti.so.10.1'; dlerror: libcupti.so.10.1: cannot open shared object file: No such file or directory
2020-02-11 10:26:17.256681: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1307] function cupti_interface_->Subscribe( &subscriber_, (CUpti_CallbackFunc)ApiCallback, this)failed with error CUPTI could not be loaded or symbol could not be found.
2020-02-11 10:26:17.256701: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1346] function cupti_interface_->ActivityRegisterCallbacks( AllocCuptiActivityBuffer, FreeCuptiActivityBuffer)failed with error CUPTI could not be loaded or symbol could not be found.
2020-02-11 10:26:17.393265: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1329] function cupti_interface_->EnableCallback( 0 , subscriber_, CUPTI_CB_DOMAIN_DRIVER_API, cbid)failed with error CUPTI could not be loaded or symbol could not be found.
2020-02-11 10:26:17.393327: I tensorflow/core/profiler/internal/gpu/device_tracer.cc:88]  GpuTracer has collected 0 callback api events and 0 activity events.

@ksabr,
I was able to run the code without any issues. Please find the Gist here. Thanks!

After some investigation, this feels more like a tensorboard graph plugin problem. Notably even with the above GPU errors (about CUPTI etc), the CPU profile is generated (it seems very similar to what tensorflow-cpu produces without any error) and it can be even viewed in the Profile tab (but not in Graph tab).
So my guess is maybe the TF 2.1 profile uses some new or different features which the Graph tab does not unserstand. And yes, profile generated by TF 2.0 is available both in Profile and Graph tab.

@ksabr,
Any updates regarding this issue? Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

After encountering the same problem today, I dug around a bit in the internals. Tensorboard has an endpoint that decodes the run metadata section of the protocol buffer containing the event logs. When running this on the logs of the above notebook, once with 2.1.0 and 2.0.0, I get different outputs:
run_metadata_v200.txt
run_metadata_v210.txt
It appears that the step_stats field in the run_metadata log is not filled out in 2.1, but is in 2.0.
https://github.com/tensorflow/tensorboard/blob/799c5cf6ae1f5f8d78a09382689dcb9d0ec5f561/tensorboard/compat/proto/config.proto#L637-L640

After digging deeper into the C++ code, it appears that between 2.0.0 and 2.1.0 the recording of this information was removed in this commit, so I don't think there is a "quick fix" for this.


Make BSTree searchable

I want to use the BSTree as an index. For instance:
// Assume I have the following function:
function sortByPos(left: {pos: number}, right: {pos: number}): number;
// And I use it to define the order in my BSTree

// First I add something in the tree:
const row = {
  pos: 1,
  value: 'lorem'
};
tree.add(row);

// Then later, I don't have a reference to row anymore
// I need to be able to retrieve the minimum entry as well as arbitrary entries:
tree.minimum();
tree.search({ pos: 1 }); // This is not possible

This is currently not possible because the BSTree data structure API assumes the whole state of objects in the tree is needed to define their order. I think this is only a special case of BSTrees. We can effectively generalize the definition with two types:

One type defines the total order in the tree: It is used for lookups, sorting, ...
The second type defines what the tree actually contains

This allows to effectively use the BSTree as a fast in-memory ordered index.
note: This pull request depends on #85

Hi @hmil . The idea is great but this is a breaking change.
I can't merge this change since it may break somebody's code.
May you introduce a new class instead of modifying the BSTree<T>?
I imagine BSTree<T> would be a special case of your class BSTreeKV where keys and values can be different types.

That's a fair point.

Released in 1.3.2


horizontal scroll on listening page

there is scroll on https://quranicaudio.com/quran/5 page please fix it by removing negative margin on line no 5354 and 5355 in public/main-c6367f480f98ef701275.css

Salam @ismailfarooq as you know this is an open source project. Is there anything I can do to help you contribute this fix?
Come join us on https://quranslack.herokuapp.com

w.salam @sabeurthabti i have joined the community but couldn't find any channel.


mpv-sponsorblock with pipe-viewer

I need to use URL instead of VIDEO to have mpv sponsorblock work.
With URL, pipe-viewer doesn't tell mpv the resolution to use so you would have to configure that manually.
--ytdl-format='bestvideo[height<=RESOLUTION]+bestaudio/best[height<=RESOLUTION]'
This would work if RESOLUTION returned just the vertical height of the video [720] and not the height with a p [720p].

See: https://github.com/trizen/pipe-viewer/issues/13

Yeah, that seems to work. This does not allow for changing the resolution inside of pipe-viewer though. If *RESOLUTION* returned just the number like i explained in my previous comment, you could add a --ytdl-format argument into the mpv section of the pipe-viewer config and have it play the desired format.

I now see what you're saying. Changed in: https://github.com/trizen/pipe-viewer/commit/169762ddce654368f3c503bf84a9aa6702d76358


Slick2D, Nifty GUI interaction problem, Nifty does not get any Input

Hey guys,
I'm currently trying to let my Game interact with my GUI, but somehow I'm encountering a problem. Slick is not forwarding any Input towards Nifty, or maybe it is but Nifty does not receive any incoming Input.
The only thing I could find on the Internet was this:
http://www.gamedev.net/topic/619545-slick2d-nifty-gui-interaction-problem/
I'm encountering except the same problem, my code is quite the same as stated in the thread above:
ScreenController:
    class MyScreenController implements ScreenController {
public void bind(Nifty nifty, Screen screen) {}
public void onEndScreen() {}
public void onStartScreen() {}
public void next() {
 System.out.println("next() called from MyScreenController");
   }
       }

XML File:

MainClass:
     public class MainGameState extends BasicGame {
       public Nifty nifty;
     public MainGame() {
       super("Test");
    }
      public void init(GameContainer container, StateBasedGame game) throws SlickException {
    nifty = new Nifty(new SlickRenderDevice(container), new NullSoundDevice(),                        new        plainSlickInputSystem(), new AccurateTimeProvider());
     nifty.fromXml("/xml/MainState.xml");
 nifty.gotoScreen("start");
      }

          public void update(GameContainer container, StateBasedGame game, int delta) throws                            SlickException {
    nifty.update();
     }

public void render(GameContainer container, StateBasedGame game, Graphics graphics)    throws  SlickException {
nifty.render(false);

}
   public static void main(String[] args) throws SlickException {
 AppGameContainer app = new AppGameContainer(new MainGame());
 app.setAlwaysRender(true);
 app.setDisplayMode( 1260 , 720, false); //window size
 app.start();
   }
  }

EDIT: xml code is not shown correctly somehow, I attached an image. But the xml file should be right anyway in my opinion, atleast I'm not encountering any Exceptions while validating the xml.

The problem may be with how you're calling nifty.update() and nifty.render(false) - I'm not 100% sure on that though, will need verification from @void256 on that.
Try lowering the Log level to fine in log.properties to see what Nifty is saying under the hood. It might give you an indication of whether or not any input is being received at all.

Am I right if I want to switch the Log level that I do it via:
Logger.getLogger("de.lessvoid.nifty").setLevel(Level.FINE); ?
I called it in my init method after initiliazing the nifty object, the console is not printing out any additional information tho.

I think you have to pass in a loggin configuration file to let the JVM know you want the logging system to submit log events at or below a certain level.
http://www.javapractices.com/topic/TopicAction.do?Id=143

Alright, it took me a while how to figure out how to load the propertie file exactly but I think I've done it right now.
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice disableClip
FEIN: disableClip()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice  setBlendMode
FEIN: setBlendMode()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderImage
FEIN: renderImage2()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice renderFont
FEIN: renderFont()
Mär 26, 2015 8:13:29 PM de.lessvoid.nifty.renderer.lwjgl.render.LwjglRenderDevice setBlendMode
FEIN: setBlendMode()
That's a bit of the stuff I get, doesn't look like any Input is recognized. :/

What exactly are you doing with the mouse? Just clicking on the element?

Nothing, I just started my programm. As soon as nifty update starts I'm getting this feedback.

No one has a clue or something I could try out? :(

The problem is that you are not using the facilities that come along with
the Slick renderer that do all the initialization for you. This means you
have to do the initialization yourself and you missed doing just that.
Now the Problem is that the PlainSlickInputSystem is forwarding the events
to Nifty, but it is not receiving any events from Slick. What you need to
do, is to add the instance of the PlainSlickInputSystem as a listener to
the Slick input provider that is accessible from the
GameContainer.getInput() function.
In any way I highly recommend you using the Game(state) implementations
that are provided with the SlickRenderer to get around problems like this,
as these classes do the proper initialization for you.
Regards
raLaaa<EMAIL_ADDRESS>schrieb am Mi., 1. Apr. 2015 um 12:22 Uhr:

No one has a clue or something I could try out? :(
—
Reply to this email directly or view it on GitHub
https://github.com/void256/nifty-gui/issues/323#issuecomment-88428792.

That worked :) Thanks a lot!


LOOP-1144 Refactor dosing decision errors to allow serialization

https://tidepool.atlassian.net/browse/LOOP-1144
Move StoredDosingDecision Codable from LoopKit to Loop to support error serialization
Update DosingDecisionStore to store and retrieve generalized data
Update decoding of optional properties to use decodeIfPresent
Use static encoder and decoder for StoredSettings
Add additional remote data service logging in MockService

@ps2, @nhamming I would like two reviews on this PR.

@rickpasetto Would you mind reviewing the latest coding test changes and re-approving?


Apple Silicon

Hi, I tried to build for apple silicon, so I changed all x64 to arm64, the build is success and when I try to run, here's the result:
% ./LlamaCppCli
USAGE:
    LlamaCppCli.dll <SampleIndex> <ModelPath> [TemplatePath] [GpuLayers]
SAMPLES:
    [0] = RawInterfaceSample
    [1] = WrappedInterfaceSampleWithoutSession
    [2] = WrappedInterfaceSampleWithSession
    [3] = WrappedInterfaceSampleWithSessionInteractive
    [4] = GetEmbeddings
edwardnone@Edwards-Mac-mini Release %

So the build is success, but when I put parameter :
 % ./LlamaCppCli 3 /Users/edwardnone/llama.cpp/models/Wizard-Vicuna-13B-Uncensored.ggml.q5_0.bin

this error occurs :
Running sample (WrappedInterfaceSampleWithSessionInteractive)...
Unhandled exception. System.DllNotFoundException: Unable to load shared library 'llama' or one of its dependencies. In order to help diagnose loading problems, consider setting the DYLD_PRINT_LIBRARIES environment variable:
dlopen(/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama.dylib, 0x0001): tried: '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama.dylib' (no such file), '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama.dylib' (no such file)
dlopen(/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama.dylib, 0x0001): tried: '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama.dylib' (no such file), '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama.dylib' (no such file)
dlopen(llama.dylib, 0x0001): tried: 'llama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSllama.dylib' (no such file), '/usr/lib/llama.dylib' (no such file, not in dyld cache), 'llama.dylib' (no such file), '/usr/local/lib/llama.dylib' (no such file), '/usr/lib/llama.dylib' (no such file, not in dyld cache)
dlopen(/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama.dylib, 0x0001): tried: '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama.dylib' (no such file), '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama.dylib' (no such file)
dlopen(/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama.dylib, 0x0001): tried: '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama.dylib' (no such file), '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama.dylib' (no such file)
dlopen(libllama.dylib, 0x0001): tried: 'libllama.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibllama.dylib' (no such file), '/usr/lib/libllama.dylib' (no such file, not in dyld cache), 'libllama.dylib' (no such file), '/usr/local/lib/libllama.dylib' (no such file), '/usr/lib/libllama.dylib' (no such file, not in dyld cache)
dlopen(/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama, 0x0001): tried: '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama' (no such file), '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/llama' (no such file)
dlopen(/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama, 0x0001): tried: '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama' (no such file), '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/llama' (no such file)
dlopen(llama, 0x0001): tried: 'llama' (no such file), '/System/Volumes/Preboot/Cryptexes/OSllama' (no such file), '/usr/lib/llama' (no such file, not in dyld cache), 'llama' (no such file), '/usr/local/lib/llama' (no such file), '/usr/lib/llama' (no such file, not in dyld cache)
dlopen(/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama, 0x0001): tried: '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama' (no such file), '/usr/local/share/dotnet/shared/Microsoft.NETCore.App/7.0.5/libllama' (no such file)
dlopen(/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama, 0x0001): tried: '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama' (no such file), '/Users/edwardnone/llama.cpp-dotnet/arm64/Release/libllama' (no such file)
dlopen(libllama, 0x0001): tried: 'libllama' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibllama' (no such file), '/usr/lib/libllama' (no such file, not in dyld cache), 'libllama' (no such file), '/usr/local/lib/libllama' (no such file), '/usr/lib/libllama' (no such file, not in dyld cache)

   at LlamaCppLib.LlamaCppInterop.llama_context_default_params()
   at LlamaCppLib.LlamaCpp.Load(String modelPath, String loraPath, String loraBaseModelPath) in /Users/edwardnone/llama.cpp-dotnet/LlamaCppLib/LlamaCpp.cs:line 74
   at LlamaCppCli.Program.WrappedInterfaceSampleWithSessionInteractive(String[] args) in /Users/edwardnone/llama.cpp-dotnet/LlamaCppCli/Program.cs:line 428
   at LlamaCppCli.Program.Main(String[] args) in /Users/edwardnone/llama.cpp-dotnet/LlamaCppCli/Program.cs:line 112
   at LlamaCppCli.Program.<Main>(String[] args)
zsh: abort      ./LlamaCppCli 3

It seems cant find the llama.cpp ? What did I do wrong ?
Thanks @dranger003

Never mind, I just copy file libllama.dylib to arm64/Release and all is working
Thanks for all your work, love it

BTW how to quit from CLI ? Tried Ctrl-C and not working 😄

Never mind, I just copy file libllama.dylib to arm64/Release and all is working
Thanks for all your work, love it

Where can I find that file please?

@amirvenus I don't use Apple Silicon but you should be able to get the file by compiling llama.cpp and specifying BUILD_SHARED_LIBS to generate the file.


[IoT] DPS track 2 update, access-policy deprecation, and linked-hub parameter changes

Description

Updates DPS  azure-mgmt-iothubprovisioningservices to track 2 SDK and adds convenience arguments to iot dps linked-hub create
Also begins the process of deprecating iot dps access-policy in favor of iot dps policy namespace for policy commands.
Testing Guide

Test new iot dps policy commands to replace all iot dps access-policy commands by simply replacing access-policy with policy in the existing commands.
Test using iot dps linked-hub create to link an IoT Hub to a DPS instance with new convenience arguments:
--connection_string, --hub_name, and --hub_resource_group_name
History Notes

[IoT] DEPRECATED: az iot dps access-policy: Being replaced with az iot dps policy
[IoT] az iot dps linked-hub create: Adds convenience arguments for linking hubs.

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

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

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

@zhoxing-ms Can we get this into the current milestone targeted for a January release?

Add @jsntcy to help with


Type 'NSAttributedStringKey' (aka 'NSString') has no member 'foregroundColor'

Just updated my Xcode and using swift 4.
Xcode = Version 9.0 (9A235)
iOS version 10.0
pod --version 1.3.1
When I try to change target swift version to 3.2 I get this error

Then changing it to swift 4 I get 6 errors mentioned in See #158

What integration method are you using?

Pod

Did you use the post_install hook from https://github.com/Skyscanner/SkyFloatingLabelTextField/issues/158 to change the version?

No I actually haven't figured out how to use. Can you please guide me

Sure, see this comment on the thread linked in #158 https://github.com/CocoaPods/CocoaPods/issues/6791#issuecomment-320028044

Just this in my pod file ?
post_install do |installer|
    installer.pods_project.targets.each do |target|
        if target.name == 'SkyFloatingLabelTextField'
            target.build_configurations.each do |config|
                config.build_settings['SWIFT_VERSION'] = '3.2'
            end
        end
    end
end

Yeah

This is what my pod looks like. Is that correct ?
pod 'OneSignal', '>= 2.5.2', '< 3.0'
pod 'RQShineLabel'
post_install do |installer|
    installer.pods_project.targets.each do |target|
        if target.name == 'SkyFloatingLabelTextField'
            target.build_configurations.each do |config|
                config.build_settings['SWIFT_VERSION'] = '3.2'
            end
        end
    end
end
end

target 'OneSignalNotificationServiceExtension' do
  use_frameworks!

  pod 'OneSignal', '>= 2.5.2', '< 3.0'
end

Yeah

I'm sorry but it's not installed with the other pods. Can you please check the pod structure for me. First time to use the hook :)

I've just pushed a Swift 3/4 compatible fix in a5fe9c9 can you try point directly to git for SkyFloatingLabelTextField per the comment in #158 ?

thank you

I take it that worked?

Yeah

Great stuff, I'll look at releasing this in the coming week. In the meantime you can keep running against master


Add Primitive/Neolithic tools

I think we should split the stone and metal tools in two. Primitive for only stone tools, and metal for the rest.
If so we can introduce more natural compatibility with other mods: E.g. Vanilla Weapons Expanded can replace the metallic tools and we patch survival tools properties to those.
The only thing is that we need some new graphics. Is anyone available for this?

Graphics aren't my strong suit, but I'll ask around. :)

I've asked Kasmex(Vladislav) over at Pelador's Discord for some graphics. @Jellypowered Can you implement them when he's made them?

Pretty sure!


ENA metadata: Proof of Concept

Pull Request
This PR is for a

[ ] New Publication(s)
[ ] Correction
[x] New Feature

For the following list(s):

[x] ancientmetagenome-anthropogenic (README)
[x] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[x] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[x] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Feature
Feature Information
This pull request is an example of automatically adding ENA metadata (ex. fastq ftp links) as mentioned in Issue #299. Rather than use the tool NCBImeta, I wrote a single python script that parses the existing TSV files for accession numbers and uses the ENA API to retrieve desired fields. The only python module required is the requests module.
I'm curious if the community is interested in this feature, and if it seems like a reasonable place to start.
An example of the script usage is:
python assets/scripts/ena-metadata.py \
  --tsv ancientsinglegenome-hostassociated/ancientsinglegenome-hostassociated.tsv \
  --output ancientsinglegenome-hostassociated/ancientsinglegenome-hostassociated-ena.tsv \
  --fields run_accession,library_layout,fastq_ftp

And an example of the script output:
ancientsinglegenome-hostassociated-ena .txt
Checklist
If there's interest in this feature, I think these would be relevant checklist items:

[ ] Code lints
[ ] Code documented
[ ] Unittests created
[ ] CI workflow created?

@ktmeaton I've made a new branch dev which I've changed this PR to merge into. Given our discussions on slack, it's up to you if you want to continue you on this PR or whether you want to to make a new one - but if the latter, make the PR into dev (which now has the current tables listed as -sample.tsv, so we can have the separate -library.tsv file.

Also I just saw your checklist - all sound great :muscle:

I think I'll make a new PR into dev with the changes we've discussed so far. You can feel free to close this PR if you think it's relevant, and I can create a new one when it's ready!


Bug with the selection of the java version at the Minecraft Images

The java version seems not to be correct everytime I create a new server using a Minecraft Image. I then have to change it from the Admin panel.

Is the minecraft17 docker image marked as the default image in your paper image?

Yes, it is!
20230624_114502.jpg

hmmm okay. i am trying to replicate this wehn i have time for it and i will notify you if i found anything

Should be fixed in the MinecraftVersions plugin


Correct approach to add plain .css files to the application

I am integrating a third party package that contains some .css files (not .scss) and I would prefer to leave them that way. What is the correct way to add the .css files into the application?

https://github.com/damienbod/AngularWebpackVisualStudio/blob/master/src/AngularWebpackVisualStudio/angularApp/vendor.ts

Ah, nice. Thanks!


ending $ followed by a -

Improving the #193, would it be possible to add the - character as authorized after the trailing $ character ?
For example, this one does not work without adding an extra space:
$n$-vector
Currently works:
$n$ -vector
Ty.

If that is part of the equation wouldn't it be better to include it in the xatex like $\text{$x$-vector}$?

@UziTech yes that would do it ;-)
Is there any reason not adding the - character as authorized after the trailing $ character ?

Ya then katex like I showed would not work.


Graph isn't drawn when max degree of separation is 0, i.e. when it should show all notes

When max degree of separation is 0, i.e. when the graph should show all notes, I don't see anything in the graph panel. The Redraw Graph button doesn't help. When I set a number other than 0, it works as expected, showing all notes within range that are linked downstream from the selected note; I believe that's the intended functionality. I'm using Joplin 2.6.2 on Linux with plugin version 1.1.3.

Just tried it again on a hunch, and now it's working. Even though I tried restarting multiple times before. Not sure what changed that "fixed" it, but there's still some bug somewhere because it didn't work before. :thinking:

Hmmm, let me know if you figure out how to reproduce it.  I am running the plugin primarily on Linux as well. My max degree of separation has always been set to zero and I am not sure I've run into this issue.
We can reopen this issue if it pops up again


Fix incorrect indent and missing double-quote in questions.yaml

Fix incorrect indent and missing double-quote in questions.yaml
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

Good catch @yangchiu


Resource manager fixes

A couple of fixes I had lying around for a while in my local copy of the repository.
This improves the behavior of the resource-manager in cases where the user requests more resources than available.

@sithhell Your comments have been addressed.


chore: allow transaction listing with get permission

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/server/interceptors/authorization.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9465

Relevant Lines:
97517

💛  - Coveralls


No notifications at all

Summary
There are no notifications at all when others post in threads I'm following or even in private messages. this is whether the app is open, in background or closed.
Environment Information

Device Name: moto Z Play
OS Version: 7.0
Mattermost App Version: 1.0 build 39
Mattermost Server Version: 3.10.0

Steps to reproduce

fresh install of mattermost on server and Android phone,
enabled all notifications in android app,
setup mattermost sso with seperate install of gitlab on same server

Expected behavior
Notification when a message is send, even when app is closed
Observed behavior
no notifications at all

Same here, with iPhone running iOS 10 or iOS 11. We did a custom build for our Mattermost server and notifications still work on desktop so I guess there is a bug in the app.

Does you server has push notifications enabled and configured? What about your user preferences?

Sorry about that, it was a configuration mistake on our side.

sorry, it was a configuration mistake on our end as well i followed the instructions in this documentation

Good to know


Move legacy dailies

Tidying up where we keep track of what preferences are daily - we have old prefs that aren't preceded with a _, it's neater to keep that in Preferences.

And our comments crossed.
Effectively all this does now is define isDaily() and use it instead of checking for a leading underscore, correct?  If so we can ignore my comment and I will approve.


Cleanup and explicit type hints

Follow up on #396

remove a leftover exception print
move the langchain import check to the langchain plugin definition
add explicit type hints on client public API methods
fix misc type errors, e.g. any -> Any
minor clarity refactors, e.g. x or y instead of x if x is not None else y

I can try to make an edit in the evening, but it's pretty busy here at the
moment and I'm OOO tomorrow.
You should be able to push here as well if I can't manage.
Hubert Bereś
On Tue, Feb 27, 2024 at 2:21 PM Max Deichmann @.***>
wrote:

@.**** commented on this pull request.
In langfuse/client.py
https://github.com/langfuse/langfuse-python/pull/405#discussion_r1504317274
:

@@ -480,24 +476,21 @@ def score(
id: typing.Optional[str] = None,
comment: typing.Optional[str] = None,
observation_id: typing.Optional[str] = None,

   kwargs=None,

Yes, this is a bug. Coudl you quickly add it?
—
Reply to this email directly, view it on GitHub
https://github.com/langfuse/langfuse-python/pull/405#discussion_r1504317274,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3OU3IMEDB7OOHXKNVNHE23YVXTWHAVCNFSM6AAAAABDVW3YXSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMBTGY2TCNZUHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok, I got a minute now, the change is easy, have a look


Brief hang when typing < in ts

This seems to happen anywhere but here's a specific place I see this

Open SettingsEditor.ts
On line 50 after extends BaseEditor type < like to start a generic parameter
See a pause of 1s or so before the character appears

The profile shows a really long time tokenizing after the keypress
Profile-20181123T140051.json.txt

I can't repo this in the latest VS Code insiders:

@roblourens Can you please test again to confirm

Yeah looks good now


fix: align username/password form

CSS files are blocked by CSP policy in browser. Server should include the URL of CSS files in its Content-Security-Policy header: https://content-security-policy.com/examples/blocked-csp/


Allow setting active driver

The current driver logic currently enforces:

only one active driver at a time
only either amd or nvidia driver

To allow other third-party drivers, we need a way to set the currently active driver. This can be done by:
import triton
triton.runtime.driver = SomeDriver()

Unfortunately this doesn't quite work out because currently we import the driver using from runtime.driver import driver, which means all modules have their driver variables pointed to LazyProxy(_create_driver) when we first import triton.
This means even if we set runtime.driver to something else after, other importing modules won't see the changes.
My proposed solution is to instead wrap the driver inside a class instance and export that:
@dataclass
class Driver:
    active = LazyProxy(_create_driver)

driver = Driver()

Then for other importing modules, instead of doing driver.whatever(), we will do driver.active.whatever().
Now we can do this to force setting an active driver if necessary:
import triton
from triton.backends.triton_shared.driver import CPUDriver

triton.runtime.driver.active = CPUDriver()

Since we are allowing setting the active driver, I have also made the nvidia driver to always use the CUDADriver to prevent misuses when the user decides to change the settings.
Alternative
Another approach that may work is to expose an environment variable to indicate what the active driver should be, then the is_active() methods of 3rd party backends will have to honor the value. Something like:
export TRITON_ACTIVE_DRIVER="cpu" python some_triton_program.py

# nvidia/driver.py

def is_active():
   force_active = os.environ.get('TRITON_ACTIVE_DRIVER')
   if force_active is not None:
      return force_active == 'nvidia'
   return actual conditions

There are less code changes required.
I have also created a draft PR for this approach here: https://github.com/openai/triton/pull/2974/files

The A100/H100 pipelines seem to be automatically cancelled after some time for me. Not sure what's going on.

This turns out to have surprising behaviours when running with pytest on a folder. If one test module sets the active driver, all other test modules have that same module as well. Closing this in favour of #2974

could we have set_active_driver instead?

@ptillet I added two methods, set_active_driver and reset_active_driver:
triton.runtime.driver.set_active_driver(CPUDriver())
triton.runtime.driver.reset_active_driver()

Let me know what you think. Thanks!

Yes, this LGTM, though minor nit: can we have driver.set_active(..) and driver.reset_active(..) instead?

Yes, this LGTM, though minor nit: can we have driver.set_active(..) and driver.reset_active(..) instead?

Thanks Phil, updated! These definitely sound cleaner.


ADOT docs revamp

Support versioned documentation
Move agnostic content upstream (docs consolidation project in OpenTelemetry)
Simplify and improve the navigation (fewer top-level items, better search and discovery)
Move all product stuff to PDP

Customer feedback:

https://github.com/aws-observability/aws-otel-collector/issues/1480

https://aws-otel.github.io/docs/getting-started/adot-eks-add-on/requirements  > Don't know if this is right spot to comment > The cert-manager version being documented is also wrong. Please revamp that too as per latest shared at: https://cert-manager.io/docs/installation/

@mhausenblas Let me know if we are stepping ahead on this feedback.

Kudos to author, will raise this as well in EKS Enterprise support. We want to introduce ADOT into our EKS cluster to send metrics from Prometheus and Cloudwatch alongisde Xray Traces.
Right now there is no way to see the most current AWS EKS Addon other than stumbling on AWS Blog post. For anyone interested I found a blog that mentioned "v0.102.0-eksbuild.1" to be latest.
Docs are designed in a way that me trying to configure version "v0.102.0-eksbuild.1" went to https://aws-otel.github.io/docs/getting-started/adot-eks-add-on/add-on-configuration which:

doesnt mention XRay (version lesser than 0.88 has outdated files)
mentions Prometheus but is not descriptive
says to check configuration_values for addon in schema in GUI, whereas guide for versions lesser than 0.88 had a dynamically generated table.
sample App for generating XRay traces does not lead to anywhere with something that could be deployed (https://aws-otel.github.io/docs/getting-started/adot-eks-add-on/config-xray#utilize-a-sample-application-to-generate-trace-data under "Sample Application for Trace Data](https://docs.aws.amazon.com/eks/latest/userguide/sample-app.html")

In order to really understand what I needed to do I had to read all the docs for previous version, review migration guide and then get to current version doc. This is a tedious process, please someone improve this!
Please update the docs with docs for XRay, Prometheus(section on AMP welcome), Cloudwatch setups that would work out of the box.

@simonloach thanks for your feedback and we will take this on board!


Voting Locking-Threshold Slider Not Working

This slider on the voting GUI is not working.

This is now fixed.


numpy_groupies is now required

It seems as though numpy_groupies is now required in order to run import xrft (rather than an extra), so I suggest moving it to INSTALL_REQUIRES.
(suggestions for other possibilities are welcome)

Interesting. Yes, I guess @roxyboy added this dependency in d6cbc0ab383ab2c6406b2145601eb82c016b448f but never added the dependency.

Interesting. Yes, I guess @roxyboy added this dependency in d6cbc0ab383ab2c6406b2145601eb82c016b448f but never added the dependency.

Yea, I've been thinking about moving the call import numpy_groupies inside the isotropic functions as it's only there where it's used

Yea, I've been thinking about moving the call import numpy_groupies inside the isotropic functions as it's only there where it's used

I think I'd like to keep it as extra requirement as numpy_groupies has quite picky dependencies.
I'll work on a PR.

I think I'd like to keep it as extra requirement as numpy_groupies has quite picky dependencies.
I'll work on a PR.

PR #133 should fix this ;)

PR #133 should fix this ;)

OK, no worries, I'll close this.

OK, no worries, I'll close this.


ensure binderhub has pycurl

using pycurl solved timeout issues with jupyterhub in the past when there are many concurrent requests, so hopefully it will help us out with current timeout issues

This may help with timeout errors we are seeing now that binder is under load

Looks good. Thanks @minrk


Add French holidays

Coverage decreased (-0.3%) to 98.68% when pulling 4aac70fc7f4fb722a1a66f6d98a01abafba42749 on oiffrig:master into 70c819a06ee1cfec54ebec98592d22406c872c2a on ryanss:master.

Coverage decreased (-0.3%) to 98.68% when pulling 5913ddd29b0e083d52d2bc3ac60006aac380b0ef on oiffrig:master into 70c819a06ee1cfec54ebec98592d22406c872c2a on ryanss:master.

I was looking for this, you beat me to it. Nice work.

Would love to see this merged!

@oiffrig is there any progress on this pull request ?
Thanks

@ryanss could you merge this pull request?

Up ?

Up ?


⚠ DesiTorrents has degraded performance

In 459cb8c, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 10492 ms

Resolved: DesiTorrents performance has improved in 67f310a after 8 minutes.


Implement validation for dropIndexes command

What should be done?
What should be done?
This issue is similar to #3320. Currently, we don't have much validation for dropIndexes. Because of this, some tests are marked failed.
Where?
See TODOs in the code.
Definition of Done

validation implemented;
integration/compatibility tests added/updated;
spot refactorings done.

Duplicate of #3320

@AlekSi let's keep them separate to have smaller PRs?


StepCA plugin with Yubikey

StepCA is a Certificate Authority. It can issue certificates via ACME protocol. (The server-side of the ACME Client plugin)
This is quite ambitious 'first PR', but I wanted to know 'What Now?'. That is, I am at a point as described in https://forum.opnsense.org/index.php?topic=38819.msg190661#msg190661 and need to figure out how I will 'distribute' the plugin (and its port dependencies) to my 'production' router, so that it survives updates and reinstalls.
This is a personal project for my homelab. Is this something (the community? maintainers?) might be interested in? Does anyone even have the time to review something 'so big'? Alternatively, I believe there is a community repo.. or ultimately, I might have to push all the packages (and port dependencies) to github, just for my own build..
For a few years, I have been running StepCA on raspberypi (next to dnsmasq with dhcp) almost verbatim as described in this article: https://smallstep.com/blog/build-a-tiny-ca-with-raspberry-pi-yubikey/. I am attempting to consolidate and simplify (my homelab).

This plugin also requires ports

security/step-certificates
security/step-kms (https://github.com/opnsense/plugins/compare/master...vpaprots:plugins:stepca?expand=1)
devel/pcsc-lite (pulls in quite a few other dependencies)

Whats missing. StepCA has a lot of features. This plugin (currently) only supports ACME provisioner, and then only a subset. No attestation, no SSH certificates and no other provisioners. This is currently sufficient to issue x509 certificates with root and intermediate keys stored inside the yubikey.
(I am not sure if I will get to it, but.. future improvements.. SSH certificates are great when working correctly link, TPM attestation looks like an interesting security model and StepCA+Radius+OpenWRT might be another project. Especially if I can reuse those certificates for VPN)
PS: If you are reviewing this code, thank you! This is my first plugin, first time on FreeBSD and I haven't done much meaningful PHP development in more then a decade. I tried to follow the style as best as I could.

Looking into hosting smallstep myself, I came upon your PR. I would also love to see this feature added to opnsense!
Lets hope a maintainer finds time to review this (arguably gigantic) PR 😁

Looking into hosting smallstep myself, I came upon your PR. I would also love to see this feature added to opnsense! Lets hope a maintainer finds time to review this (arguably gigantic) PR 😁

It should be possible to install the pluggin (i.e. for testing) with a 'one liner pkg command'. I have it somewhere in my notes, but I haven't touched this in a while. With Christmas break, might get back and add some of those notes here.. I never figured how to run my own repo, but pkg can pick things up off github iirc.


When run as a GitHub action on Ubuntu-latest, it fails to install/locate Prettier

My package.json contains prettier in devDependencies, version 2.8.4. It runs on ubuntu-latest. At some point in the last week the action started failing with the message below. I tested it by forcing a rerun of the action on a PR that previously worked fine and it failed there too despite there being no subsequent commits.
❌ Unable to locate or install prettier. Did you provide a workdir which contains a valid package.json?
I then tested it further by putting a 'run' step in before the call to uses: reviewdog-action-prettier@v1 in order to force an install of prettier. That installed OK, but the action was still unable to locate it.
It seems that something perhaps in Github or on ubuntu-latest with the "if [-f...]" test has changed that stops this action from functioning correctly?

Here's the relevant section from my github workflow:
  Prettier:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: EPMatt/reviewdog-action-prettier@v1
        with:
          github_token: ${{ secrets.github_token }}
          reporter: github-pr-check
          level: warning

And here is a sample package.json that triggers the issue:
{
  "name": "myproject",
  "version": "0.1.0",
  "description": "myproject",
  "private": true,
  "repository": {
    "type": "git",
    "url": "<censored>"
  },
  "dependencies": {
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-router-dom": "^6.4.2",
    "react-scripts": "5.0.1",
    "typescript": "^4.7.4"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject",
    "pretty": "npx prettier src/** --write",
    "prettyver": "npx prettier --version"
  },
  "devDependencies": {
    "prettier": "2.8.4",
    "tailwindcss": "^3.2.4"
  }
}

This script is broken on npm version >= 9. It makes use of the npm bin script which has been removed.

https://github.com/npm/cli/pull/5459
https://github.com/npm/cli/blob/latest/CHANGELOG.md

Probably makes sense to update use of $(npm bin)/prettier to node_modules/.bin/prettier unless there is a need to really have dynamic resolution.

Hey there,
thank you so much for reporting here. I've just merged #36 (thanks @kazkansouh!) which includes a fix for this issue.
If the problem persists, please don't hesitate to open a new ticket.
Thanks!


Add a page with a list libraries using HTTPlug

As discussed briefly on Slack. We would like a page similar to http://supervisord.org/plugins.html
We maybe also could mention the implementation. Are they using plugins? Custom plugins? MultipartStreamBuilder? Or maybe that is overkill.
Here is a list of projects using HTTPlug:
https://github.com/Payum/Payum
https://github.com/mailgun/mailgun-php
https://github.com/Happyr/LinkedIn-API-client
https://github.com/FriendsOfSymfony/FOSHttpCache
https://github.com/KnpLabs/php-github-api
https://github.com/flint/Stampie
https://github.com/SparkPost

it would be nice to show the logos of the most used "clients" on the
home page of the doc, and maybe also in the README of Httplug for those
that browse github.
the full list could be on http://httplug.io/ maybe? its not really part
of the documentation to me.

As far as I can see, this is fixed, isn't it?

Yes it is


ORC-1575: Use ASF Archive URL instead Download URL for orc-format

What changes were proposed in this pull request?
This PR aims to update ORC Format download link to ASF Archive from ASF Download.
Why are the changes needed?
Since ASF Download keeps only the latest live releases, the links on Download are improper for old release branches.
How was this patch tested?
Pass the CIs.

Thank you, @mystic-lama .

Merged to main/branch-2.0.


[AIRFLOW-4107] instrument executor

Cherry pick https://github.com/apache/airflow/pull/4928 from upstream.

Instrument open slots, running tasks and queued tasks.

@lyft/dp-tools-viz


nrt: add optional kni-specific informer

The podinformer supplied by the scheduler framework filters out pods in terminal state. Changing this is nontrivial, and we need a fix on older versions.
In order to completely fix the non-running pod desync, we need to either get pods in terminal phase on the scheduler side or to filter out these pods on nodes.
The best option here is adding a separate independent dedicated informer on the scheduler side. This is the more self contained change AND the one which scales better, requiring "just" one additional subscription to the apiserver (vs N=len(workers)).
We still need to pursue options in the context of the scheduler framework to see which improvements we can land in there.

/hold
needs a rebase

/lgtm
The proposed changes look good to me.
I would emphasize in the PR description that the KNI informer can be enabled by specifying NRT_ENABLE_INFORMER environment variable to consider all pods (including terminal ones).
We will need to overcome this problem upstream as well which I understand is going to be tricky from scheduler-plugin side. Perhaps a good way forward would be identifying if kubelet's behavior of reporting terminal pods is expected (which I understand is under way).

Yes, I keep forgetting to start this conversation u/s. Will file a upstream bug/conversation ASAP. We need to do this, no questions. Unfortunately however this won't help us wrt backports, so we need some form of workaround like this one I'm proposing.

/hold cancel

/cherry-pick release-4.13

/cherry-pick release-4.13


Add contributions to PPP

#203

Codecov Report

Merging #222 (f8ecc66) into master (40d2d0b) will increase coverage by 1.95%.
The diff coverage is n/a.

:exclamation: Current head f8ecc66 differs from pull request most recent head 5915ef3. Consider uploading reports for the commit 5915ef3 to get more accurate results

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
+ Coverage     51.07%   53.02%   +1.95%
- Complexity      616      654      +38
============================================
  Files           118      118
  Lines          2708     2708
  Branches        324      324
============================================
+ Hits           1383     1436      +53
+ Misses         1225     1173      -52
+ Partials        100       99       -1

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/person/Person.java
89.58% <0.00%> (+1.04%)
:arrow_up:

src/main/java/seedu/address/model/tag/Tag.java
81.81% <0.00%> (+9.09%)
:arrow_up:

...rc/main/java/seedu/address/model/person/Email.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...rc/main/java/seedu/address/model/person/Phone.java
90.00% <0.00%> (+10.00%)
:arrow_up:

.../main/java/seedu/address/model/person/Address.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...du/address/logic/commands/meeting/MeetCommand.java
31.81% <0.00%> (+31.81%)
:arrow_up:

...s/model/person/FieldContainsKeywordsPredicate.java
60.00% <0.00%> (+45.45%)
:arrow_up:

...in/java/seedu/address/commons/util/StringUtil.java
97.14% <0.00%> (+45.71%)
:arrow_up:

Continue to review full report at Codecov.

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


Poor Memory Management

Suppose we need to show view controllers of One type only.
now if i have 100 tabs, then i am forced to create 100 instances. each with its title.
and returning this 100 instance array back.   very pathetic.
Instead i must be able to use one instance and update its title and model only and reload the stuff.
Many thanks.

your requirement is too special while this project is for problems that more common, so we have to return the 100 instances in an array


Random tiny improvements

In the last few months, I've been noticing things here and there and collating this branch. It's really simple changes, I recommend looking at the individual commits for details.

Changes looks generally good to me. I would extract https://github.com/detekt/detekt/pull/5562/commits/670435d071bb127d509a46f1c194f555dfd8582c to a separate PR as it will show up in the changelog as well


[5.5] Add tests

@edenreich please explain the changes in this PR

Sorry, we don't merge things without descriptions. Please re-submit with one.

my bad...was just about to explain it nvm

Please add your description to the OP. :)

I still can't really tell what we're going for here and the code isn't super obvious.


Object node can return arrays

Inspect the output type of the object node and if it's an array, return a more-appropriate object[] type rather than object. Fixes #305

Good to know, I really only care about Array arrays here though I think


Expose /list with swagger

@wyegelwel The /list endpoint is not exposed with swagger at the moment - is there any reason not to expose it?

No, none at all.
On Wed, Apr 26, 2017, 4:34 PM Daniel Posada<EMAIL_ADDRESS>wrote:

@wyegelwel https://github.com/wyegelwel The /list endpoint is not
exposed with swagger at the moment - is there any reason not to expose it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twosigma/Cook/issues/307, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZnHYQREM_xsiMfkwZg11kJXm8onaamks5rz6phgaJpZM4NJY63
.

Superseded by #797.