If baseDir is set libedir has to be repointed too to make use of bundled native libs

Linux native dependency includes two bundled libraries libjemalloc and libeaio.
If we change BaseLib from default these libraries would not be found and in order to fix that LibDir should be set relative to BaseLib. To reproduce - run it on linux with basedir set to something other then SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base"
            configBuilder = DBConfigurationBuilder.newBuilder();
            configBuilder.setBaseDir(dbFolder.getAbsolutePath());
            configBuilder.setLibDir(new File(dbFolder, "libs").getAbsolutePath());  // bundled libs are not found if LibDir is not set to this relative path

It is due to code in DBConfigurationBuilder constructor
        this.baseDir = SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base";
        this.libDir = this.baseDir + "/libs";

If you leave libDir as null on initialization and change implementation of getLibDir I would this it would do the trick
public String getLibDir() {
      if (this.libDir == null)
          return this.baseDir+"/libs"
      else
         return this.libDir;
}

Would you be willing to create a Pull Request with what you've outlined
above?
On Thu, 14 Jul 2016, 22:56 anverus<EMAIL_ADDRESS>wrote:

Linux native dependency includes two bundled libraries libjemalloc and
libeaio.
If we change BaseLib from default these libraries would not be found and
in order to fix that LibDir should be set relative to BaseLib. To reproduce

run it on linux with basedir set to something other then
SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base"
      configBuilder = DBConfigurationBuilder.newBuilder();
      configBuilder.setBaseDir(dbFolder.getAbsolutePath());
      configBuilder.setLibDir(new File(dbFolder, "libs").getAbsolutePath());  // bundled libs are not found if LibDir is not set to this relative path

It is due to code in DBConfigurationBuilder constructor
    this.baseDir = SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base";
    this.libDir = this.baseDir + "/libs";

If you leave libDir as null on initialization and change implementation of
getLibDir I would this it would do the trick
public String getLibDir() {
if (this.libDir == null)
return this.baseDir+"/libs"
else
return this.libDir;
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vorburger/MariaDB4j/issues/39, or mute the thread
https://github.com/notifications/unsubscribe/AASOZsxOictxsv9Ea9EIJZYm39X7vjZtks5qVqJqgaJpZM4JM14s
.

OK. It is about time I started using github correctly :) I'll create it
over the weekend. Thank you for great product :)
On Fri, Jul 15, 2016 at 2:52 AM Michael Vorburger<EMAIL_ADDRESS>wrote:

Would you be willing to create a Pull Request with what you've outlined
above?
On Thu, 14 Jul 2016, 22:56 anverus<EMAIL_ADDRESS>wrote:

Linux native dependency includes two bundled libraries libjemalloc and
libeaio.
If we change BaseLib from default these libraries would not be found and
in order to fix that LibDir should be set relative to BaseLib. To
reproduce

run it on linux with basedir set to something other then
SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base"

configBuilder = DBConfigurationBuilder.newBuilder();
configBuilder.setBaseDir(dbFolder.getAbsolutePath());
configBuilder.setLibDir(new File(dbFolder, "libs").getAbsolutePath());
// bundled libs are not found if LibDir is not set to this relative path
It is due to code in DBConfigurationBuilder constructor
this.baseDir = SystemUtils.JAVA_IO_TMPDIR + "/MariaDB4j/base";
this.libDir = this.baseDir + "/libs";
If you leave libDir as null on initialization and change implementation
of
getLibDir I would this it would do the trick
public String getLibDir() {
if (this.libDir == null)
return this.baseDir+"/libs"
else
return this.libDir;
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vorburger/MariaDB4j/issues/39, or mute the thread
<
https://github.com/notifications/unsubscribe/AASOZsxOictxsv9Ea9EIJZYm39X7vjZtks5qVqJqgaJpZM4JM14s
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vorburger/MariaDB4j/issues/39#issuecomment-232876732,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH4P7cjnXJL_LbIfW9CIMRJGkIznByB9ks5qVy4mgaJpZM4JM14s
.

@anverus thanks for this bug report and contribution https://github.com/vorburger/MariaDB4j/pull/40/ ! I've just merged it to master now (and a deploy of a new 2.2.1 including this to Maven Central should follow within the hour or so, hopefully).
Please note that in https://github.com/vorburger/MariaDB4j/commit/3ed30237c00784c485ed4ebe6af4802a826b3f3c in addition to the 3 new Unit Tests you've added in DBConfigurationBuilderTest (thank you!), I've also added additional non-regression integration testing for this problem in the MariaDB4jSampleOtherTest and MariaDB4jSpringServiceOverrideBySetTest.
I hope that it is OK that I've also added you to the https://github.com/vorburger/MariaDB4j/blob/master/CONTRIBUTORS.md file, according to https://github.com/vorburger/MariaDB4j/blob/master/CONTRIBUTING.md. You're also more than welcome to add yourself and/or Trip Advisor to the https://github.com/vorburger/MariaDB4j/blob/master/USERS.md via a new PR if you like.

Hi Michael,
Thank you for managing such a useful product :)
Just to share a bit more information on how we use it maybe for some
insights and to share some workarounds
We use it to run integration tests against embedded MariaDB. We run it in
MacOSX machines (developers) and Linux (Ubuntu / CentOS).
Running parallel tests require separate DB per test - so we create separate
instance of mysql deploying it to separate temp directory for each test
using setBaseLib / setDataLib.
One minor problem we had on Mac: if Firewall is enabled and MariaDB
instance started, MacOSX pops a dialog to a tune of - "Do you want to allow
incoming connections to mysqld? Allow / Deny". If left unanswered it
disappears after instance is terminated but considering that we run
multiple instances it pops up quite a few of them and it makes using
machine pretty hard as these popups grab cursor.
Workaround was to add
this.configBuilder.addArg("--bind-address=localhost");
I was not sure where to put that info and if it is going to be helpful to
others :)
Thank you again for your great work.
On Sun, Jul 24, 2016 at 4:50 PM Michael Vorburger<EMAIL_ADDRESS>wrote:

Closed #39 https://github.com/vorburger/MariaDB4j/issues/39.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vorburger/MariaDB4j/issues/39#event-732892972, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH4P7QKdkGLs66vPYob5LFMsPBiG0FLgks5qY9AxgaJpZM4JM14s
.


Possible to see where your uploaded images are used

I would like to be able to see what images contributed to Wikimedia are used, and also the pages they are used at. It already lists the amount of contributions used, but not which ones and on what pages. This would be very helpful.

A link to a web page that has all of this information is probably the best way to implement this.
What page do you think would be best? I am sure I saw such a page some time ago.

Well yes, that could work. Some images are used on several pages though, so
perhaps the image used on ie. Wikipedia English (if present) followed by
'also used on...' below. Then next image. Also the ability to sort by time
of upload, so it can show the most recent uploads. I think, as a
contributor, the 'reward' of seeing your work being used should have some
significance.
Den man. 29. okt. 2018, 09:04 skrev Nicolas Raoul<EMAIL_ADDRESS>
:

A link to a web page that has all of this information is probably the best
way to implement this.
What page do you think would be best? I am sure I saw such a page some
time ago.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/issues/1958#issuecomment-433819872,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqZ5u_uaxYZEMe3gDVT1m1gjrmudS_gNks5uprZ8gaJpZM4X-n24
.

I think this would really encourage users, and a link would be a good addition to anything received in Notifications.
Maybe at the top of the "Contributions" tab, have a URL, with the text "See how your contributions are being used"
Chris

I guess this #6006 solves this issue.


[REQ] add a (GH-compliant) license file

Hi there, 1st of all thanks for this cool work !
Since we've "doxed" it in our HyMPS project (under the VIDEO category \ Pan Tilt Zoom section \ Generic subsection), can you please add the license to it ?
As you know, expliciting licensing terms is extremely important to let anyone better/faster understand how to reuse/adapt/modify sources (and not only) in other open projects and vice-versa.
Although it may sounds like a minor aspect, license file omission obviously causes an inconsistent generation of the relative badge too:

(badge-generator URL: https://flat.badgen.net/github/license/jeffmikels/ptz-server/?label=LICENSE)
Anyway you can easily set a standardized one through the GH's license wizard tool.
Last but not least, let us know how - in your opinion - we could improve our categorizations and links to resources in order to favor collaboration between developers (and therefore evolution) of listed projects.
Thanks in advance.

Bump (request updated).

Thanks for the reminder.


Fix add-template commands

Hello!
Just a little fix for getting started with templates (cf: https://docs-next.graph.cool/reference/service-definition/templates-zeiv8phail#adding-templates-with-the-cli)

thanks 🙏


Cherry pick Correct array memory usage calculation for dictionary arrays to active_release

Automatic cherry-pick of f1a831f

Originally appeared in https://github.com/apache/arrow-rs/pull/505: Correct array memory usage calculation for dictionary arrays

Codecov Report

Merging #515 (9d2e599) into active_release (32b835e) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@                Coverage Diff                 @@
##           active_release     #515      +/-   ##
==================================================
+ Coverage           82.68%   82.79%   +0.10%
==================================================
  Files                 163      163
  Lines               44957    44940      -17
==================================================
+ Hits                37172    37206      +34
+ Misses               7785     7734      -51

Impacted Files
Coverage Δ

arrow/src/array/array_binary.rs
92.23% <ø> (+2.10%)
:arrow_up:

arrow/src/array/array_boolean.rs
94.01% <ø> (+3.10%)
:arrow_up:

arrow/src/array/array_dictionary.rs
88.48% <ø> (+3.60%)
:arrow_up:

arrow/src/array/array_list.rs
94.88% <ø> (+2.06%)
:arrow_up:

arrow/src/array/array_primitive.rs
94.60% <ø> (-0.10%)
:arrow_down:

arrow/src/array/array_string.rs
97.76% <ø> (+1.71%)
:arrow_up:

arrow/src/array/array_struct.rs
89.24% <ø> (+1.39%)
:arrow_up:

arrow/src/array/array_union.rs
89.26% <ø> (+2.33%)
:arrow_up:

arrow/src/array/null.rs
83.78% <ø> (-2.89%)
:arrow_down:

arrow/src/array/array.rs
80.90% <100.00%> (+4.04%)
:arrow_up:

... and 3 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 32b835e...9d2e599. Read the comment docs.


Loading Pytorch model

It is currently not possible to load a pytorch model which was saved in the banchmarking  repsitory.
To load such a model it is necessary to have access to the original class, as discussed here and here.
Loading it without access to the class causes the error ModuleNotFound Error: No module named 'ML_Model'

Add respective tests for the pytorch model at test_mlmodel.py

I saw that people used torchscript / jitter to load models without the class definitions. Would that be wort exploring? Otherwise the we'd have to ship model code with our library for every model we use

https://stackoverflow.com/questions/59287728/saving-pytorch-model-with-no-access-to-model-class-code
https://discuss.pytorch.org/t/how-to-load-using-torch-load-without-source-class-using-which-model-was-created/75302/2
https://pytorch.org/docs/stable/jit.html


CreatePlugin: Change default docker image to grafana-enterprise

What this PR does / why we need it:
We got a question about why the GF_ENTERPRISE_* environment variables wasn't picked up properly after migrating to the create-plugin configuration files. The reason for this is that we are using the grafana/grafana docker image which doesn't contain the enerprise version of Grafana.
This PR will change so that all the docker related configuration that is scaffolded by create-plugin will use grafana/grafana-enterprise as the default docker image. It also adds a way to configure this without adding any changes to the ./config folder which might cause issues while updating.

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:shipit:


Clicking on 'Privacy' link gives /Privacy page with html

Description of bug
On a new dnn 9.6.0 I click on the 'Privacy' link and I go to the /privacy page where I can see:

Steps to reproduce

Install a fresh DNN 9.6.0
Click on the privacy link in the footer
See error

Current behavior
The text shows html tags
Expected behavior
A nicely formated text
Screenshots
See above
Affected version

[x]  9.6.0

Affected browser

[x] Chrome

please check the language pack applied, there was an issue in earlier versions of DNN 9, where the language tool encoded twice, when saving HTML content entered (and only decodes once, when rendering the content)

Yes, looks like it is a bug in the GlobalResources.nl-NL.resx
So the dutch language pack.

@trouble2 Marnix,
there is a Repo for the Dutch LP at https://github.com/DNNCommunity.
Please log the issue over there.


There are no tmax_over_time values.

sensor:
  - platform: prometheus_sensor
    url: http://localhost:9090
    queries:
      - name: in3_voltage_max
        expr: max_over_time(homeassistant_sensor_voltage_v{entity="sensor.pzem_004t_v3_in3_voltage"}[1y])
        unit_of_measurement: V
        device_class: voltage
      - name: in3_voltage_max_timestamp
        expr: tmax_over_time(homeassistant_sensor_voltage_v{entity="sensor.pzem_004t_v3_in3_voltage"}[1y])
        device_class: timestamp

The timestamp sensor is always unavailable. How to find the cause of the problem?

Can you check the home-assistant log for clues? If a metric is unavailable it will generally log the reason there. Not sure if there is a good way to propagate errors into the UI :thinking:

Looks to me like tmax_over_time is specific to VictoriaMetrics; https://docs.victoriametrics.com/metricsql/#tmax_over_time
What Prometheus implementation are you using?

Understood thanks. I don't have VictoriaMetrics. And I don’t know how to get the time and date of an event :-(
How to assign a unique identifier to a sensor?

Understood thanks. I don't have VictoriaMetrics. And I don’t know how to get the time and date of an event :-(

The integration is using the Query API of Prometheus, which will always return the latest value for your given expression. The time that Home Assistant records as the last update is probably the best you're going to get.

How to assign a unique identifier to a sensor?

Set the unique_id field.

The integration is using the Query API of Prometheus, which will always return the latest value for your given expression. The time that Home Assistant records as the last update is probably the best you're going to get.

I need to find the maximum value and time when the event occurred.

I would recommend taking that question to the Prometheus community.
They're on Slack, Matrix and IRC: https://prometheus.io/community/


Does not support Flake8 6+

Fixed in version 1.2.1, released today.
For what it's worth, the Flake8 dependency of Flake8-pyproject was pinned to < 6 before this release. So I don't see why Flake8 needed to be pinned separately (as in those mentions). This should have been handled automatically by Pip's dependency solver. And it was, in my experience. I had no issues with CI.
Flake8 v6 did introduce some breaking changes with the new major version (as it does, and which is why the pin was put in place), but they didn't affect the parsing of pyproject.toml, as far as I can tell, only of Flake8's INI-like config files.

I don't know why pip's dependency solver didn't work either, we got these failures when it was unpinned:
https://github.com/DiamondLightSource/python3-pip-skeleton/actions/runs/3541575749/jobs/5946003894
Either way, with the new release and flake8 unconstrained works:
https://github.com/DiamondLightSource/python3-pip-skeleton/actions/runs/3564433455
Thanks for the quick release!

Thanks for the feedback, @coretl. Yeah, that's strange. It installed flake8==2.3.0 (super old) and Flake8-pyproject==0.9.1 (quite old). No idea why it did that, but not surprised to see that combination break. Maybe I should just remove the version pin entirely if it does not have the intended effect. There are good arguments against it after all.

@ssbarnea Can you explain why the dependency resolution failed in the above case?


Support Python>=3.9

We want to follow NEP 29.
Currently, tests fail for Python==3.9. See comment in the CI YAML file.

Closing this issue as Python 3.9 is soon (Apr 05, 2024) going to be deprecated as per NEP 29.


RPPL-1875: Update fireboltCalls.js

RPPL-1875

As part of the above ticket, we want to launch third party apps like Prime Video, Apple TV+, Hulu, etc. apart from FCA.
I observed that, by default, the discovery.launch() method launches the app with appId CONSTANTS.THIRD_PARTY_APP_ID.
Hence, with the proposed change, we are trying to update the CONSTANTS.THIRD_PARTY_APP_ID with any other appId, if present.
I have verified that it is working fine while launching FCA or any other app.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Radhakrishnan, Rajanika seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Wrong user used for commit, hence closing the PR.


Error deploying a Neo4j app on Heroku

I am getting the following error when deploying my Neo4j app on Heroku -
NoMethodError: undefined method `zero?' for nil:NilClass

https://github.com/neo4jrb/neo4j-core/blob/ec1e02a09741c0ef3da12518527680d91caa30ec/lib/neo4j/core/cypher_session/adaptors/bolt.rb#L158
This is the neo4j.yml
staging:
  type: bolt
  url: <%= ENV['GRAPHENEDB_BOLT_URL'] %>

GRAPHENEDB_BOLT_URL = bolt://user:password@server:24786

Runtime information:
Neo4j database version: Neo4j Community Edition 3.4.4
neo4j gem version: 9.3.0
neo4j-core gem version: 8.1.4

UPDATE: when I switch to http as the protocol, I am able to start the Heroku app and everything works.

Hello @kapso.  The problem is likely the Bolt protocol over SSL.  That is supported in the master brand of this gem, but it hasn't been released yet because there was an issue found with it.  I'm not sure when it will be able to be released, though you're welcome to try it.
As you found you can still use http just fine

@cheerfulstoic thanks for the quick response, will use http for now. Look forward to the bolt release/fixes. Thanks.


FF not showing proper checked checkbox style

see https://enke.to/::widgets

Seems like it doesn't support background-image yet for the css appearance.

A workaround might be to use a dot instead of checkmark (same as radiobutton)

The workaround is not great. It shows the checked state as

The problem is that once FF fixes the bug, both the tick and the dot will be shown. So the workaround has to be removed once FF fixes the bug.

Very fancy! Thank you. Will make next release a point release, as it will require apps to update their build systems accordingly.


Doc: Markdown formatting

A drive-by documentation improvement:

block for citations

table for for JSON Fields

If #6 is merged first will be happy to rebase it.

@mpatsis it would be great if you can please review all these pull requests.


PXT-354: Bug: Indexed column for a view may not load correctly from persistence

Bug RCA:
When an embedding is created on a column, the indexed_col_id field of the
index metadata tracks the column on which this embedding was created.
For a view that can be ambiguous: it may be id of a column in the view itself,
or id of a column of any base table(s) that the view is derived from.
Fix:
Persisted the indexed column's table id in the index metadata to resolve the
ambiguity. Fixed the code in TableVerison._init_schema()→_init_idxs()  to
lookup the right Column based on both column id and table id combination,
so that when it is loaded back, it can map to the correct column for a view.
Since this changed the persistent metadata, bumped up the pixeltable metadata
version and added converters to upgrade existing persistence to include the
new information (indexed column's table id) in the index metadata, if it is missing.
Note: Since we cannot know the correct value of the new field for past views,
upgrade assigns it the table id of the view on which the index is created.
Thus, we can still run into the bug.
Testing:
New test_embedding_index_reload_view that reproduced the bug for
a fresh database works correctly with this commit.
Existing test_db_migration. that would fail to load an older database
dump due to the missing new field in the index metadata, passes with
this commit, it passes.

This change is

The CI failure looks unrelated to my changes.

ImportError while loading conftest '/Users/runner/work/pixeltable/pixeltable/tests/conftest.py'.
tests/conftest.py:19: in
from .utils import (
tests/utils.py:540: in
e5_embed = sentence_transformer.using(model_id='intfloat/e5-large-v2')
pixeltable/func/function.py:130: in using
call = exprs.FunctionCall(self, bindings)
pixeltable/exprs/function_call.py:53: in init
return_type = fn.call_return_type(bound_args)
pixeltable/func/function.py:94: in call_return_type
return self._conditional_return_type(**kw_args)
pixeltable/functions/huggingface.py:69: in _
model = _lookup_model(model_id, SentenceTransformer)
pixeltable/functions/huggingface.py:513: in _lookup_model
model = create(model_id)
../../../miniconda3/envs/pxt/lib/python3.9/site-packages/sentence_transformers/SentenceTransformer.py:194: in init
modules = self._load_sbert_model(
../../../miniconda3/envs/pxt/lib/python3.9/site-packages/sentence_transformers/SentenceTransformer.py:1073: in _load_sbert_model
module = module_class.load(module_path)
../../../miniconda3/envs/pxt/lib/python3.9/site-packages/sentence_transformers/models/Pooling.py:195: in load
with open(os.path.join(input_path, "config.json")) as fIn:
E   FileNotFoundError: [Errno 2] No such file or directory: '/Users/runner/.cache/huggingface/hub/models--intfloat--e5-large-v2/snapshots/b322e09026e4ea05f42beadf4d661fb4e101d311/1_Pooling/config.json'


USER frame with test

This has conflicts and I don't know what the recent updates are :) If you can summarize I know what to look at.

@lawik this PR is ready for review and added test where directly check with id3v2 cli to form the existence of USER frame in the binary.

Fixed conflicts
it checks the presence USER frame with ffmpeg and id3v2 cli tool.


Routing /:id/*

This is an issue related to routers.
The results when handling routes such as /:id/* vary between each router. If you have the following app:
app.get('/:id/*', (c) => {
  return c.text(`${c.req.param('id')}`)
})

What are the results when accessing /, /foo, and /foo/bar? The results are as follows:
GET /
TrieRouter: 404
RegExpRouter: 404
LinearRouter: Error
PatternRouter: 404

GET /foo
TrieRouter: 404
RegExpRouter: 200 foo
LinearRouter: Error
PatternRouter: 200 undefined

GET /foo/bar
TrieRouter: 200 foo
RegExpRouter: 200 foo
LinearRouter: Error
PatternRouter: 200 undefined

These are not expected. Therefore, we need to decide on the behavior for /:id/* and fix the implementation.
I believe we should follow the specification used by TrieRouter, which would result in:
GET / => 404
GET /foo => 404
GET /foo/bar => 200 foo

If we choose this specification, the routes that need fixing are:
RegExpRouter

GET /foo

PatternRouter

GET /foo
GET /foo/bar

LinearRouter throws UnsupportedPathError errors for all patterns. If we allow this, we will need to use it with the SmartRouter and TrieRouter for the preset hono/quick.
This is an edge case, but we must resolve it as issues like #1240 have been raised.
The script
import { Hono } from 'hono'
import { type Router } from 'hono/dist/types/router'
import { LinearRouter } from 'hono/router/linear-router'
import { PatternRouter } from 'hono/router/pattern-router'
import { RegExpRouter } from 'hono/router/reg-exp-router'
import { TrieRouter } from 'hono/router/trie-router'

const createHono = (router: Router<any>) => {
  const app = new Hono({
    router: router,
  })
  app.get('/:id/*', (c) => {
    return c.text(`${c.req.param('id')}`)
  })
  app.showRoutes()
  return app
}

const routers: Router<any>[] = [
  new TrieRouter(),
  new RegExpRouter(),
  new LinearRouter(),
  new PatternRouter(),
]

const apps: Record<string, Hono> = {}

for (const router of routers) {
  const app = createHono(router)
  apps[router.name] = app
}

const request = async (path: string) => {
  console.log(`GET ${path}`)
  for (const [name, app] of Object.entries(apps)) {
    try {
      const res = await app.request(path)
      if (res.ok) {
        console.log(`${name}: ${res.status} ${await res.text()}`)
      } else {
        console.log(`${name}: ${res.status}`)
      }
    } catch (e) {
      console.log(`${name}: Error`)
    }
  }
}

console.log('')
await request('/')
console.log('')
await request('/foo')
console.log('')
await request('/foo/bar')

Hi @usualoma !
What do you think about this issue?

Hi @yusukebe!
I believe the specification is that /path/to/* should match /path/to in hono.
This spec has been tested here at TrieRouter.
https://github.com/usualoma/hono/blob/6c0b358ba387995e0c7adda3702b9e1858725685/src/router/trie-router/router.test.ts#L77-L114
Therefore, the following are the expected results.
GET / => 404
GET /foo => 200 foo
GET /foo/bar => 200 foo

The PatternRouter issue can be fixed in #1244.

@usualoma
Thanks! I'll fix the TrieRouter and merge it.

This is fixed!


HTTP-GET on Policy Examples throws Exception

At https://international-data-spaces-association.github.io/DataspaceConnector/Documentation/v6/UsageControl it is said, that POST /api/examples/policy can be used for policy examples.
If using GET-Method (without parameter), an exeption is thrown on both endpoints: policy and validation
WARN - Failure in @ExceptionHandler io.dataspaceconnector.controller.exceptionhandler.GlobalExceptionHandler#handleAnyException(RuntimeException)
java.lang.IllegalStateException: Could not resolve parameter [0] in public org.springframework.http.ResponseEntity<net.minidev.json.JSONObject> io.dataspaceconnector.controller.exceptionhandler.GlobalExceptionHandler.handleAnyException(java.lang.RuntimeException): No suitable resolver

Expected behaviour: This application has no explicit mapping for /error, so you are seeing this as a fallback.

Hi,
i think there are two problems in the behaviour of the DSC.

The global exceptionhandler should not throw an exception.
The connector should respond with Method Not Allowed, when performing a GET on the endpoint.


Setoutfit in the Debug context menu option doesn't work

https://github.com/space-wizards/space-station-14/assets/106132477/f16b5c5c-9926-4621-90e1-b6a4be2ac5bc

It's picking the wrong entity ID for the command.


change/background_color

Hii @a-mabe
Issue #45 was resolved by changing the background color to the recommended colors

Appreciate the contribution @LOGESH-B, I will merge this in


In API mode not working

In API mode not working

The same API Key works with another extension

Links:
VSCode Marketplace
Github

@benzntech are you still facing this issue? Could you check the error notification for exception details?

After the new version update its working fine.

Working fine Now after the update.


WIP: Implement IsAlreadyMirrored & cmd/is-falce-ebpf-probe-uploaded

I have a few questions about implementing this ticket:

You mentioned I would need to use github actions on a feature branch instead of personal access token to test IsAlreadyMirrored, I'm not sure how that would work? Would we just add it in temporarily to test this ticket, then remove it during the next ticket where it gets used by the actual build action?
I don't understand how the //cmd part can possibly work without using
a) a personal access token
b) an extra arg for using the repo.IsAlreadyMirrored?
Even with a personal access token I'm getting a 401 with
plz run //cmd/is-falco-ebpf-probe-uploaded -- --verbose --falco_version=0.28.1 --github_releases_token=$GRT amazonlinux2 4.14.200-155.322.amzn2 error="could not list releases: GET https://api.github.com/repos/thought-machine/falco-probes/releases: 401 Bad credentials []" I'm not sure if this is expected or something I need to configure on my github account?
For IsAlreadyMirrored I added a returned error is that ok?

I have a few questions about implementing this ticket:

You mentioned I would need to use github actions on a feature branch instead of personal access token to test IsAlreadyMirrored, I'm not sure how that would work? Would we just add it in temporarily to test this ticket, then remove it during the next ticket where it gets used by the actual build action?

Yeah, that's exactly what I was thinking :) though, you're right this isn't a nice thing to do. Ideally we'd be writing unit tests to validate this which would get picked up by the "Test" github workflow. I started trying to get this working here: https://github.com/thought-machine/falco-probes/blob/unit-test-releases/pkg/repository/ghreleases/ghreleases_test.go#L31 but haven't quite got it yet. So i'm happy for any workarounds we have just to demonstrate that it works (i.e. pasting command output, or a temporary workflow).

I don't understand how the //cmd part can possibly work without using
a) a personal access token
b) an extra arg for using the repo.IsAlreadyMirrored?

I'm not sure that I understand your question here; the //cmd/is-falco-ebpf-probe-uploaded's job is to check if the probe has already been uploaded. This requires a and is calling b. Ah right, sorry the command i've given you in your ticket doesn't detail the inputs. What you've done is 💯  what I was expecting 👍  i.e. plz run //cmd/is-falco-ebpf-probe-uploaded -- --falco_version <falco_version> <operating_system> <kernel_package_name>

Even with a personal access token I'm getting a 401 with
plz run //cmd/is-falco-ebpf-probe-uploaded -- --verbose --falco_version=0.28.1 --github_releases_token=$GRT amazonlinux2 4.14.200-155.322.amzn2 error="could not list releases: GET https://api.github.com/repos/thought-machine/falco-probes/releases: 401 Bad credentials []" I'm not sure if this is expected or something I need to configure on my github account?

Strange, it's working for me:
# Create github personal token from https://github.com/settings/tokens with no scopes attached
$ vim .github_token
# paste github token into file (can use nano)
$ export GITHUB_TOKEN=$(cat .github_token)
$ plz run //cmd/is-falco-ebpf-probe-uploaded -- --verbose --falco_version=0.29.1 amazonlinux2 4.14.200-155.322.amzn2
...
11:05AM INF Identifying if falco ebpf probe uploaded for  driverVersion=17f5df52a7d9ed6bb12d3b1768460def8439936d probeName=falco_amazonlinux2_4.14.200-155.322.amzn2.x86_64_1.o
11:05AM INF Matching release found, now checking it's assets:  release=17f5df52a7d9ed6bb12d3b1768460def8439936d
11:05AM FTL Probe cannot be found: error="release/asset not found that matches 17f5df52a7d9ed6bb12d3b1768460def8439936d/falco_amazonlinux2_4.14.200-155.322.amzn2.x86_64_1.o"

For IsAlreadyMirrored I added a returned error is that ok?

Yup, that's okay! nice one 👍

thanks @VJftw! I'll work on these changes now!

Tested manually using:
`#   plz run //cmd/is-falco-ebpf-probe-uploaded -- --falco_version=0.29.1 amazonlinux2 4.14.200-155.322.amzn2 -> 1
plz run //cmd/is-falco-ebpf-probe-uploaded -- --falco_version=0.29.1 amazonlinux2 4.14.101-91.76.amzn2  -> 0 `

Added in a workflow & accompanying build script:

.github/workflows/test-is-falco-ebpf-probe-uploaded.yaml
build/github/test-is-falco-ebpf-probe-uploaded
These are a bit quick and dirty but we plan to delete them after the next ticket is completed where we will actually use IsAlreadyMirrored in the existing build script.

2 test cases are still failing when they're meant to be passing, so I'm looking into it.

Added in pagination, tested locally using:
./pleasew -p -v2 run //build/github/test-is-already-mirrored -- amazonlinux2

@VJftw one last question, I left the verifying input parts to the cmd not IsAlreadyMirrored. I thought it would be duplicating code when IsAlreadyMirrored is used properly, do you agree?

@VJftw one last question, I left the verifying input parts to the cmd not IsAlreadyMirrored. I thought it would be duplicating code when IsAlreadyMirrored is used properly, do you agree?

Yes, there's very little value to verifying inputs as there are no negative consequences for bad inputs in the tooling we're making here. i.e. doing plz run //cmd/is-falco-ebpf-probe-uploaded -- amazonlinux2 as;djfhsladjfhj will just return an error saying that the package could not be downloaded 👍


Display reserved rockets on profile page Pull request

I this part of activity, I implemented the following:

Render a list of all reserved rockets (use filter()) on the "My profile" page.

Thank you for approving my project work! 😊


Added a README

I took a stab at fleshing out the README as part of our larger documentation effort.
@mklingen @psigen Anything else we should add here?

@aaronjoh Can you take a look?

The out-of-process viewer example needs a little more detail. Running this:
import openravepy
env = openravepy.Environment()
env.SetViewer('InteractiveMarker')

as suggested by the readme, and then separately running:
rosrun rviz rviz

does not work and puts out this error:
[ERROR] [1423682423.086219712]: PluginlibFactory: The plugin for class 'or_interactivemarker::rviz::EnvironmentDisplay' failed to load.  Error: Could not find library corresponding to plugin or_interactivemarker::rviz::EnvironmentDisplay. Make sure the plugin description XML file has the correct name of the library and that the library actually exists.

I'm sure this is just me missing a step, but I'm working only from the Readme.md. In-process viewer launches, though for both a suggested action or test that actually does something would be nice.
It would similarly be nice to have scripts/test.py either be the same as what is indicated in the readme, or have the readme setup how to run the test script.

@aaronjoh I made some changes to address your comments. Can you take another look?

Ok, looks good. Might add a note that the scripts/test.py is a good place to start (though that pulls the model from /opt/pr so i guess we might drop that from the release?)

I updated the test.py script and now mention it in README.


[DO NOT MERGE] Nim work for 2.16

Last of the NIM work.
Update 2024-11-15 3:10pm ET: Seems there are some problems with the current makeshift deployment on the NIM cluster -- not sure this code will work on a RHOAI cluster. (thread)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 47.80876% with 131 lines in your changes missing coverage. Please review.

Project coverage is 85.42%. Comparing base (b09ef1b) to head (411cc1e).
Report is 4 commits behind head on rhoai-2.16.

Files with missing lines
Patch %
Lines

frontend/src/utilities/useEnableApplication.tsx
1.69%
58 Missing :warning:

...nd/src/utilities/useWatchIntegrationComponents.tsx
37.73%
33 Missing :warning:

frontend/src/services/integrationAppService.ts
13.33%
13 Missing :warning:

...rc/pages/modelServing/screens/projects/nimUtils.ts
60.00%
12 Missing :warning:

...ing/screens/global/DeleteInferenceServiceModal.tsx
77.77%
4 Missing :warning:

...d/src/pages/modelServing/screens/projects/utils.ts
60.00%
4 Missing :warning:

...projects/NIMServiceModal/ManageNIMServingModal.tsx
78.57%
3 Missing :warning:

...d/src/pages/exploreApplication/GetStartedPanel.tsx
60.00%
2 Missing :warning:

...pages/exploreApplication/useIntegratedAppStatus.ts
87.50%
1 Missing :warning:

...lServing/screens/projects/ModelServingPlatform.tsx
92.85%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@              Coverage Diff               @@
##           rhoai-2.16     #379      +/-   ##
==============================================
- Coverage       85.65%   85.42%   -0.23%
==============================================
  Files            1347     1350       +3
  Lines           30676    30827     +151
  Branches         8554     8592      +38
==============================================
+ Hits            26274    26335      +61
- Misses           4402     4492      +90

Files with missing lines
Coverage Δ

frontend/src/concepts/areas/const.ts
100.00% <ø> (ø)

.../pages/enabledApplications/EnabledApplications.tsx
100.00% <100.00%> (ø)

...ntend/src/pages/exploreApplication/EnableModal.tsx
32.07% <ø> (ø)

.../modelServing/screens/global/EmptyModelServing.tsx
78.57% <100.00%> (-4.77%)
:arrow_down:

...modelServing/screens/projects/useIsNIMAvailable.ts
100.00% <100.00%> (ø)

frontend/src/pages/modelServing/screens/types.ts
100.00% <ø> (ø)

...c/pages/modelServing/useServingPlatformStatuses.ts
100.00% <100.00%> (ø)

...il/overview/serverModels/PlatformSelectSection.tsx
100.00% <100.00%> (ø)

...etail/overview/serverModels/ServeModelsSection.tsx
100.00% <100.00%> (ø)

...verModels/deployedModels/DeployedModelsSection.tsx
94.64% <100.00%> (-0.10%)
:arrow_down:

... and 12 more

... and 4 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


HuggingFaceEndpoint Class Bug

System Info
the provided class
langchain/langchain/llms/huggingface_endpoint.py
class HuggingFaceEndpoint(LLM):
has a bug.
it should be
if self.task == "text-generation":
# Text generation return includes the starter text.
text = generated_text[0]["generated_text"]

not
text = generated_text[0]["generated_text"][len(prompt) :]

the current class will likely just return a 0.
Who can help?
No response
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
    llm= HuggingFaceEndpoint(endpoint_url=os.getenv('ENDPOINT_URL'),task="text-generation",
                              model_kwargs={"temperature":0.7, "max_length":512})
 conversation_chain = ConversationalRetrievalChain.from_llm(
        llm=llm,
        retriever=vectorstore.as_retriever(),
        memory=memory
    )

Expected behavior
the output is 0

Good catch. Related conversation on Hugging Face 2 weeks ago: https://huggingface.co/tiiuae/falcon-40b-instruct/discussions/51
I tested your code and received blank response using text-generation.
When using text2text-generation , 1871 is returned.


Clarify what always means

The comment for editor.codeActionsOnSave: always just says "Always trigger code actions on save" but that's not true because format on delay isn't a trigger reason. This should be mentioned explicitly because other folks are confused. Also, the #setting.name# syntax could be used to link to the required setting

Related: https://github.com/microsoft/vscode/issues/194437


Update power-bi-imported-query.md

Added additional download link for Power BI Desktop as the store link was not working, and the "try free" links did not work either.

@davidwassonmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@davidwassonmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@davidwassonmsft Thx for the suggestion. As an MSFT, please propose future changes using the private repo. Thx
#sign-off


Linux package not found

https://static.wilsonl.in/minify-js/cli/0.6.0/linux-x86_64/minify-js linked from front page is broken, but macos one downloads fine.

Building the CLI tool from source is also possible using;
cargo install minify-js-cli --git https://github.com/wilsonzlin/minify-js.git


Enhance CLI table reporter

TODO

[x] Render table
[x] Dump table as string
[ ] Responsive by terminal width
[ ] Add tests

Motivation / Reason

console.table is not responsive by terminal width

If terminal width is small, table format collapse.

I want to customize table grid etc...

Example


Commit 2a289 breaks convenience scripts

I noticed in CI the install script for artifacts-credprovider stopped working just now

https://github.com/microsoft/artifacts-credprovider/commit/2a2891c55c7fb01f39aba11c23bd083402ea7273#diff-a3d32842c27ade73028cfca0c3b9686dd84d09453a06b78d38829b88e9cac7d0

#526 should resolve this.

Verified works in CLI and CI, thanks @cobya


Error: There are no scenarios; must have at least one.

Submit a feature request or bug report

[X] I've read the guidelines for Contributing to Roots Projects
[ ] This is a feature request
[X] This is a bug report
[X] This request isn't a duplicate of an existing issue
[X] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[X] This is not a personal support request that should be posted on the Roots Discourse forums

What is the current behavior?
composer finished. Went through and created folders and cloned files from dev-master. After setup questions, running "yarn" produces the error message
What is the expected or desired behavior?
Yarn will run and install dependencies

Bug report

Please provide steps to reproduce, including full log output:
Run composer
Finish setup questions
Run yarn
linuxbastard@BakulawDev:~/websites/serenecommunities.dev/public_html/wp-content/themes/serene-sage$ yarn
ERROR: There are no scenarios; must have at least one.
Please describe your local environment:
WordPress version: 4.7.4
OS: Ubuntu 16.04
NPM/Node version: 4.2.0/7.10.0
Where did the bug happen? Development or remote servers?
local dev server (not VM/Vagrant)
Is there a related Discourse thread or were any utilized (please link them)?
X

Sorry guys, was running the wrong yarn.

I have the same problem, what does it mean, the wrong yarn ? I am on ubuntu and yarn --version says:
0.27
what should i do ?

Upgrade Yarn:
https://roots.io/getting-started/docs/ubuntu-linux-development-environment-sage/
https://yarnpkg.com/en/docs/install#debian-stable
Please use Roots Discourse for further support.


Fix grid usage in edit and update tearsheets

From the edit and update migration design review —

[ ] Fix grid usage in full page examples in Storybook (see #2813)

Flexbox or CSS grid are fine since consumer is responsible for actual content

@SimonFinney Might be fastest for you since you made the same fix for create tearsheets, but if you’re starting other work, maybe @jagathgj or @davidmenendez can help with this one.


lr_scheduler for language modeling task (cosine vs inverse_sqrt)

I see that in the example "wikitext-103" for the language modeling task
the lr_scheduler is set to cosine, while usually for translation task is set to inverse_sqrt.
Is there any reason for this?
Is the inverse_sqrt a reasonale scheduler?
Any comment is very welcome.

Inverse square root was used in the Attention is All You Need paper. Cosine was introduced in this paper: https://arxiv.org/abs/1608.03983. Both are reasonable, but they have different strategies for decreasing the learning rate. In practice a well-tuned cosine schedule typically performs better than inverse square root, but the downside it that you have to define the number of steps in advance.
Inverse square root is a good default that produces decent results.


Add option in menu for filtering already read manga out of the libary updates tab

Real simple idea(though likely harder in execution ), adding a filter option to the options menu that weeds out the chapters you have read from the chapters you have not read in the library updates tab. Something that can be turned on and off at leasure.
An alternative would be a filter option on the library updates page itself. Same solution just different location and possible different headache.
I say this because my library updates tab can get a little messy with that I do and don't read at one time and sometimes i forget or dont read all my new chapters exactly as they show up so they can get buried, and it would be nice to see the list shrink abit as i have a looooong list.

This has been in Neko for a few releases now.


[BEAM-9707] Hardcode Unified harness image for fixing dataflow VR 2 t…

…ests
The changes are only temporary till we have 2.21 out. The changes should be reverted before 2.21 is cut.
R: @ananvay @robertwb @lukecwik
cc: @ibzib
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Python Dataflow V2 ValidatesRunner

Thanks Ankur! LGTM.

Run Python Dataflow V2 ValidatesRunner

I can add the code to check if the version has dev in it to ensure that its not used in release.

I can add the code to check if the version has dev in it to ensure that its not used in release.

That makes more sense to me, especially if you were planning on undoing the revert after the cut (?).

Run Python Dataflow V2 ValidatesRunner

updated the PR to only set the image when version is dev.

Run Python Dataflow V2 ValidatesRunner

The failing metric test seems to be working on my local repo https://gradle.com/s/snptwrrxau5we
The issue might be the recent changes to beam metric proto cleanup.

Can we loop in Luke and/or sickbay the test to get this PR in?

Had offline discussion with Robert and going ahead with the merge. We will need to sickbay the test or publish a new UW image with the fix which can be done separately.


Update CI go version to 1.20

The podman tests in CI were failing because podman failing to build. Podman was failing to build beacuse of of its dependencies need atleast go 1.19. Updated all go install actions to install go 1.20 .

Codecov Report

Merging #2227 (aa75a18) into main (2d6fd76) will increase coverage by 0.03%.
Report is 6 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2227      +/-   ##
==========================================
+ Coverage   65.03%   65.06%   +0.03%
==========================================
  Files         129      129
  Lines       15139    15139
==========================================
+ Hits         9845     9850       +5
+ Misses       5294     5289       -5


[HACKTOBERFEST 2024] - JavaScript Fundamentals Quiz

[!IMPORTANT]
This issue is part of our 2024 Hacktoberfest initiative. If you are interested in working on this issue, refer to our top-level tasks to see if this issue is claimed, or to claim it if it is not.

This issue is to create the JavaScript Fundamentals quiz.
The file you should be working on is found in /curriculum/challenges/english/25-front-end-development/quiz-javascript-fundamentals.
All quiz questions need to relate to one of these topics:

Topics

[!CAUTION]
Questions which do not fall under these topics will be rejected.

You are welcome to include code blocks in your questions, but please keep them short (four lines or less).
Please do NOT open a pull request until you have completed all questions. Your quiz should have exactly 20 questions.

Thank you for your interest in contributing. This issue will be opened for contribution when Hacktoberfest starts on October 1st.

[!CAUTION]
Pull requests which are created before October 1st will not be eligible for Hacktoberfest. We appreciate enthusiasm, but encourage you to hold off on contributing to this issue until the event has begun.

If you wish to be notified when this issue is opened for contribution, subscribe to notifications:

This issue is now open
If you would like to work on this issue, please leave a comment so we can assign it to you.
Be sure to read our contributing guidelines.

I would like to work on this issue

Hey @shivam-bhushan,
you are already assigned to different issue - https://github.com/freeCodeCamp/freeCodeCamp/issues/56186.

This issue has been assigned a contributor. To prevent extraneous notifications, the issue is now locked. If you have any questions, please reach out to me in our Discord community.


Disabling compass doesn't work on iOS < 9

Setting showsCompass={false} on iOS < 9 causes a crash.
-[AIRMap setShowsCompass:]: unrecognized selector sent to instance 0x122280c40

setShowsCompass is only supported on iOS >= 9

@rops What version are you using?
Looks like this was merged and made available in 0.5.0
https://github.com/lelandrichardson/react-native-maps/commit/edfa2b42ce40369d19b27cc2a9c835491ba3793b
Can you try installing that version and see if the problem is fixed.


Assembly 'RJCP.SerialPortStream' in NuGet /netstandard1.5 folder misses PublicKeyToken; version 2.1.3

Hello!
I encountered a problem when using your library in a .NET Standard 2.0 library of myself that is then used in a .NET 4.7.1 console app. The issue arises from a PublicKeyToken mismatch in the different assemblies you deploy with your NuGet package.
For instance, your net40 and net45 assemblies have a PublicKeyToken of 5f5e7b70c6a74deb, whereas your netstandard1.5 assembly has none.
Now, a .NET Standard 2.0 library will generally bind to your netstandard1.5 assembly, which has no PublicKeyToken. When I then use this .NET Standard 2.0 library in another .NET 4.7.1 console application - which is by the way a common use case - then this console application will bind to the net45 version of RJCP.SerialPortStream, which has a PublicKeyToken.
In such situation, compilation works fine, but at runtime, I get a System.IO.FileLoadException: 'Could not load file or assembly 'RJCP.SerialPortStream, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)' exception, which is caused by the different PublicKeyTokens.
I would expect to have all assembly variants in a NuGet package to have the same PublicKeyToken, meaning your .NET Standard 1.5 assembly should be strong-signed too.

Thanks for the detailed report. I'll have to investigate this as using NET Core is not the primary target. If you have some experience with strong signing NET Core binaries, I'd be glad to here. Then I can investigate an update.

I'll have to investigate this as using NET Core is not the primary target. If you have some experience with strong signing NET Core binaries, I'd be glad to hear. Then I can investigate an update.

Well, I don't target .NET Core as runtime. I just use a .NET Standard class library because .NET Standard is intended to provide target-independent libraries that then can run on different runtimes like Mono, .NET Core or .NET FX. My primary target is .NET FX 4.7.1. I just have chosen .NET Standard in my current side-project to learn about this new stuff.
That said, I also a kind of newcomer to .NET Standard, but I have time to dig into it. Hopefully I'll get together a PR.

It also sounds like your NET 4.7.1 project also references the project.

Yes and no. Basically, my application (.NET 4.7.1) references your package indirectly as transitive dependency. And because of that it picks the net45 assembly from the package, because that matches best.
I also tried to explicitly add your package to my application, making it a direct dependency, but that has the same effect. I then tried to override package target resolution via <PackageTargetFallback/>, but also without luck. It's possible that there exist some other mechanism that I don't know yet.
Nevertheless, having assemblies with different identities (for different targets) in the same NuGet package, isn't going to work. So, I think that's the root cause of my troubles.

Please be aware, that the two assemblies are not the same (NetStandard 1.5 and NetFx 4.0). It might be that if I get them both signed, there might still be a type load error.

Indeed, that can be a problem. But, in practice it won't. First, because your net45 and netstandard15 assemblies seem to be compatible (at least the public API). In my app project I enforce loading of net45 library by handling AppDomain.CurrentDomain.AssemblyResolve event. That means, I use net45 assembly at runtime, although compilation was done against netstandard1.5.  And it works fine, but this is a hack, not working for unit tests, etc.
The net40 version of course might break, but net40 isn't compatible with .NET Standard at all and therefore I think this target won't be picked in scenarios similar to mine; you already provide better matching targets like net45 and netstandard1.5.
This all is a bit complicated. I still believe that fixing the strong-naming will make all problem go away. So, I'm going to investigate there.

My 2 Cents. Maybe this could be resolved by using the new  XML-based MSBuild/csproj format? It allows to combine all targets into one project ("netstandard1.5;net40;net45") and build, sign and pack all DLL's in one go from Visual Studio.

Ohh. My fault. As I tested my PR locally, I replaced you PFX with a SNK, because I didn't had a PFX at hand and it didn't come to my mind that PFX wouldn't be supported in .NET Core... but that's the case.
I would suggest to switch to SNK in your projects because they are supported by .NET FX and .NET Core toolset.
I found some resources for PFX to SNK conversion:

https://gist.github.com/baywet/4a25e3b4ee485b30258a
https://github.com/aarnott/pfx2Snk
https://social.msdn.microsoft.com/Forums/vstudio/en-US/8c706144-7fa9-487a-a06b-8a25c801c682/convert-a-pfx-file-to-snk-file?forum=clr
https://stackoverflow.com/questions/8174229/how-to-create-a-snk-from-pfx-cer

Hope this helps.

Alright. Got it working, but I'll have to update the patchset first. NuSpec needs updating, the SNK should be used instead. I'll see if I can use the SNK for everything first. Saves me having to carry around two forms of the same key.

Can you please test the attached nuget packages? I'll look at your other ticket before making a release.
serialportstream.zip

By the way, just saw Amarok.SerialPortStream. Are you sure that you solved the problem there?
Projects\SerialPortTest\packages\Amarok.SerialPortStream.2.1.4\lib>sn -T net40\RJCP.SerialPortStream.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

net40\RJCP.SerialPortStream.dll does not represent a strongly named assembly

SerialPortTest\packages\Amarok.SerialPortStream.2.1.4\lib>sn -T net45\RJCP.SerialPortStream.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

net45\RJCP.SerialPortStream.dll does not represent a strongly named assembly

SerialPortTest\packages\Amarok.SerialPortStream.2.1.4\lib>sn -T netstandard1.5\RJCP.SerialPortStream.dll

Microsoft (R) .NET Framework Strong Name Utility  Version 4.0.30319.0
Copyright (c) Microsoft Corporation.  All rights reserved.

netstandard1.5\RJCP.SerialPortStream.dll does not represent a strongly named assembly

Doesn't look like your library does any signing.

By the way, just saw Amarok.SerialPortStream. Are you sure that you solved the problem there?

This is just there to get going with my work. To keep it simple I removed signing entirely, as I don't need it at the moment.
The description isn't correct; it doesn't solve the strong-name conflict, it removes strong-name entirely.

Can you please test the attached nuget packages? I'll look at your other ticket before making a release.

Tested it. Works!

Integrated in branch v2.x


No ability to lock one new dependency

If attempting to vendor one or more packages explicitly with command line arguments along with the -l option to lock revisions, the intuitive behavior--to me--would be to vendor the new dependency and add just that dependency and its dependencies to the vendor.yml file. For example:
govend -v -l github.com/sha1sum/govend

This should simply add the package listed (and its dependencies) to the lock file. This is not the behavior, however, and the entire vendor directory is cleaned and then all dependencies are re-fetched and added to the lock file.

PR #49 resolves this issue.


Is there a 3.13 nogil container?

I want a 3.13 nogil container, also with all optimizations, etc. I assume that the current 3.13 containers use the GIL.

We don't have a "nogil" image currently.  For 3.13 we're using the default compilation settings, which AFAIK still results in the GIL.  Unfortunately, compiling Python is pretty heavy, so expanding our current support matrix to include new "nogil" variants is also going to be a tough sell. :see_no_evil:

I'm not sure how you got there from what I said -- perhaps I can clarify: we do compile Python (currently ~42 times per architecture, in fact, and we support ~10 architectures where compilation is necessary), and it's very heavy to do so, and adding more variants would require us to do so even more times, so is not something we're willing to consider unless/until it is an officially recommended configuration (ideally the default configuration, but we'd be willing to consider an official statement along the lines of "everyone should try this out").

I agree that nogil is the future of Python, and there's nothing strange about the request.  If I were completely opposed to the idea in general, I probably would've closed the issue already.  It's more a matter of resourcing/prioritization, not desire/understanding.  If the demand were actively higher, it would also be a lot easier to justify the added resources/maintenance spend -- the best way for folks to signal that is to react with :+1: on the top post in this thread.
To put that another way, my answer is more accurately "not yet", not "never".
If you'd like to help with the work, figuring out what changes are necessary to our current Dockerfiles would be a really useful first step.

An experimental JIT-enabled build would also be great to have

Instead of prodiving a whole second set of "nogil" Docker images the free-threaded executable could also be part of the normal images just like it is done with the official Windows and macOS installers:

CPython now has experimental support for running in a free-threaded mode, with the global interpreter lock (GIL) disabled. This is an experimental feature and therefore is not enabled by default. The free-threaded mode requires a different executable, usually called python3.13t or python3.13t.exe. Pre-built binaries marked as free-threaded can be installed as part of the official Windows and macOS installers, or CPython can be built from source with the --disable-gil option.

https://docs.python.org/3.13/whatsnew/3.13.html#free-threaded-cpython
So we would have /usr/local/bin/python3.13t in every image and call this to use the free-threaded mode.
Unfortunately that still means that all of Python has to be compiled twice.

But that would break a lot of tooling that expects Python to be at the prior file path.

The normal paths would still be there, but point to the normal GIL version. So if you don't adjust your tooling everything will be as before.

If anyone seeking for some basic image to play with, I've made one: https://github.com/abebus/free-threaded-python-docker-image

+1 can confirm; would love to use this both at work and at home!

Python images without GIL and JIT enabled should be part of docker images. I have been using docker as dev container and to experiement new version python. Thanks to all the work been done so far.

this is much needed.

I'm a bit surprised that you're going out of your way to provide images for alpha versions of Python (if I'm reading https://hub.docker.com/_/python correctly, there are images for Python 3.14.0a2), but not for free-threaded builds which are an official feature of released Pythons. While these builds are still marked experimental, many Python packages have started using them to ensure they are free-threading compatible.
For example in Apache Arrow we have our own image for CI until the official Python Docker images support free-threaded builds:
https://github.com/apache/arrow/blob/main/ci/docker/python-free-threaded-wheel-manylinux-test-imports.dockerfile
Many Python open source projects are probably using similar home-grown recipes due to lack of official Docker image availability.

@impredicative

noGIL and JIT
is not possible to build (yet)

Those are unrelated, perhaps open a separate issue for JIT builds?
(but I'd argue that JIT builds are unimportant for now, as they don't provide tangible benefits)


Maximizing/resizing the window should automatically snap grid into place (provided it wasn't adjusted)

VS Code version: Code - Insiders 1.24.0-insider (41a7c4976ebd1f6ced20c034485300f46ab1829f, 2018-05-29T06:09:34.618Z)
OS version: Windows_NT x64 10.0.16299

#50458
Split to right:

Maximize window:

When the terminal panel containing split panes is resized it defaults to resetting the proportional size of the split panes. If something is manually saved it will try to retain that:
https://github.com/Microsoft/vscode/blob/f5300ced7fbd0193babf05ee37860bb2d160beb7/src/vs/workbench/parts/terminal/browser/terminalTab.ts#L22-L25

Duplicate of https://github.com/Microsoft/vscode/issues/1447


Add cEUR support to payment requests

Description
Note: This isn't a big PR, most of the line changes are snapshot files.
Add the ability to make payment requests. When paying a request you can choose which currency to use from the SendConfirmation screen.

Other changes
Deleted packages/cloud-functions/yarn.lock because it was causing deployment issues. I did a full reinstall after and didn't run into any issues.
Tested
Manually and with automated tests.
How others should test

Make a payment request to another user.
See that you receive a push notification with the payment request.
Tap the push notification or open the app and choose the Pay option on the request.
Change the currency from the default to another stable token (cUSD -> cEUR or viceversa) by pressing the Edit button
Make the payment and check that it's made successfully.

Related issues

Fixes #215

Backwards compatibility
N/A

https://valora.testquality.com requirement R47.

https://valora.testquality.com requirement R54.

https://valora.testquality.com requirement R55.


Tuple of inner functions typing regression in 0.49rc1 due to first class functions feature

Reporting a bug

[ ] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

Type inference in 0.49rc1 infers a tuple of different inner functions as a unituple like UniTuple(UndefinedFunctionType[undefined()1] x 2).  Example:
import numba
from numba.extending import overload

def g(t):
    pass

@overload(g)
def overload_g(t):
    print(t)
    print(t.types[0].dispatchers)
    return lambda t: 0

@numba.njit
def f():
    def h1(x):
        return x

    def h2(x):
        return 2 * x

    return g((h1, h2))

f()

output in 0.48:
Tuple(type(CPUDispatcher(<function g.<locals>.f at 0x1258cb5f0>)), type(CPUDispatcher(<function g.<locals>.f at 0x1258cb9e0>)))

output in 0.49rc1:
UniTuple(UndefinedFunctionType[undefined()1] x 2)

Looks like there is an attribute with a set of dispatchers:
print(t.types[0].dispatchers)
# {CPUDispatcher(<function g.<locals>.f at 0x11f4d1dd0>), CPUDispatcher(<function g.<locals>.f at 0x11f4d1d40>)}

Somehow it doesn't recognize that the functions are actually different.

Thanks for the report. Does #5473 perhaps fix this?

Looks like it fixes it.

Duplicate cause as https://github.com/numba/numba/issues/5470

#5529 Merged. Closing.


add support for wildcard when searching album

Not sure if the Subsonic API would allow that. Is that even possible?
❯ ve/bin/pSub album *
Usage: pSub album [OPTIONS] SEARCH_TERM
Try "pSub album -h" for help.

Error: Got unexpected extra arguments (pSub.py README.md setup.py ve)

It should be possible. In my tests running something like psub album "*stone*" will return no results but then hitting enter and repeating the search term "*stone*" returns all albums with "stone" in the title. This feels like a string handling issue. I'll take a look and close this issue when I have a better answer.


Improve cascaded values/params content

Fixes #14102

Move coverage to the bottom of the section because the current content breaks right into the middle of the example without naming.
Flesh out the code example with additional detail. For example, include both the ancestor and descendant components.

Thanks @Legends! :rocket:

@Legends

If it's permissible to show both a prop and a field, I like it. An alternative tho is to call out in text that both work. Let's see what Dan says to do.
WRT SxS or nested, depends on what engineering meant by "within the same subtree.". I'll update that PR to the nested presentation if that turns out to be what they meant (if GH will let me! 🙈 ... it usually doesn't stop me from committing to my own PRs via the interface ... usually! lol).

@Legends

If it's permissible to show both a prop and a field, I like it. An alternative tho is to call out in the text that both work. Let's see what Dan says to do.
WRT SxS or nested, depends on what engineering meant by "within the same subtree." I'll update that PR to the nested presentation if that turns out to be what they meant (if GH will let me! 🙈 ... it usually doesn't stop me from committing to my own PRs via the interface ... usually! lol).

1.) Fields, Props, Props with Attribute, everything is legit.
2.) I guess it has to be nested, otherwise it would be in another side branch.


pip3 install of mako-1.2.1 fails

At linuxfromscratch we are trying to convert to the python recommended use of pip3 to install modules.  What we are doing is downloading the module source and after untarring:
pip3 wheel -w dist --no-build-isolation --no-deps $PWD
(as root) pip3 install --no-index --find-links dist --no-cache-dir Mako
This worked fine for mak0-1.2.0.  For make-1.2.1, creating the wheel works, but installing does not:
Making Mako-1.2.1
Fri Jul  1 02:18:14 PM CDT 2022
Processing /build/mako/Mako-1.2.1
Preparing metadata (pyproject.toml): started
Preparing metadata (pyproject.toml): finished with status 'done'
Building wheels for collected packages: Mako
Building wheel for Mako (pyproject.toml): started
Building wheel for Mako (pyproject.toml): finished with status 'done'
Created wheel for Mako: filename=Mako-1.2.1-py3-none-any.whl size=78629 sha256=271ea0d8835a00de1d02e787ef89c86eb22a8c802b599bdbd9593e9c7a48ca7e
Stored in directory: /home/bdubbs/.cache/pip/wheels/f9/60/1f/a506fe41b85daa187fb3d7e5002dab98bc750145d7f81825e8
Successfully built Mako
Looking in links: dist
Processing ./dist/Mako-1.2.1-py3-none-any.whl
ERROR: Could not find a version that satisfies the requirement MarkupSafe>=0.9.2 (from mako) (from versions: none)
ERROR: No matching distribution found for MarkupSafe>=0.9.2
However MarkupSafe-2.1.0 is installed.  Using the above pip3 commands works (and still works) for Mako-1.2.0.
The old method if installing Mako using "python3 setup.py install --optimize=1" does continue to work.

hi  -
I would ensure you have upgraded pip to the latest version and if further problems report this to https://github.com/pypa/pip, nothing has changed from mako 1.2.0 to 1.2.1 regarding setup or requirements.
1.2.1:
$ pip install mako
Collecting mako
  Downloading Mako-1.2.1-py3-none-any.whl (78 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 78.6/78.6 kB 3.6 MB/s eta 0:00:00
Collecting MarkupSafe>=0.9.2
  Using cached MarkupSafe-2.1.1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (25 kB)
Installing collected packages: MarkupSafe, mako
Successfully installed MarkupSafe-2.1.1 mako-1.2.1
(.venv) [classic@photon3 foo]$ pip3 install mako
Requirement already satisfied: mako in ./.venv/lib/python3.10/site-packages (1.2.1)
Requirement already satisfied: MarkupSafe>=0.9.2 in ./.venv/lib/python3.10/site-packages (from mako) (2.1.1)

My fault.  pip3 usage error. Closing.


ProxyExchange function for pass-through forwarding of HTTP methods

Currently for a catch-all proxy, it is required to have a block inspecting the incoming request, and forward using corresponding function on ProxyExchange
for example:
when (method) {
  HttpMethod.GET -> proxy.get()
  HttpMethod.POST -> proxy.post()
  ...
}

It would be convenient to have a call that would inspect the incoming request and do this automatically.

org.springframework.cloud.gateway.webflux.ProxyExchange#forward()

I guess this comment was a little confusing to me (MVC version): https://github.com/spring-cloud/spring-cloud-gateway/blob/main/spring-cloud-gateway-mvc/src/main/java/org/springframework/cloud/gateway/mvc/ProxyExchange.java#L131
So .forward(...) can be used to proxy requests to downstream services?

Just as a follow-up, it looks like .forward(...) has a void return type, and we'd still like to get a response from the method somehow.

This was a misunderstanding on my part - sorry for the noise.


Fetch polyfill Implementation

Hello @dop251,
first of all, need to say thank you for the project.
Just for your information.
The benchmarks were good enough to try to switch to goja from go-duktape. One critical thing I need to have is a fetch polyfill. So, now I have one written for goja, have a look. It's a window.fetch JavaScript polyfill. Would be good if you leave some feedback here.
Thanks

Hello,
I'm not sure what exactly your task is, but the first question you need to ask yourself is whether you need it to be non-blocking at all. If not, you could use a patched version of Promise that would call the function directly instead of calling setTimeout(fn, 0) and synchronous implementation of the Request. You could even make it work in both scenarios by checking for the presence of setTimeout(), if it's present, then we are running in the loop so the non-blocking version should be used, otherwise use the blocking version.
Maybe need to think of a better way of building and embedding native node modules. If a project uses fetch and some other module that also requires a matching version of 'when' package for example it would be nice to avoid duplication. But I don't know enough about node.js to suggest anything specific, just a thought.
A few other notes:
It's always a good practice to use strict mode. This would prevent the 'k' variable from leaking into the global context for example (https://github.com/olebedev/gojax/blob/master/fetch/fetch.js#L74)
Since your implementation of Response.prototype.json() is not async, it could be written as this for a slightly better performance:
Response.prototype.json = function() {

  var _this = this;

  return new Fetch.Promise(function(resolve, reject) {
    resolve(JSON.parse(_this.body.text));
  });

}

There is a race in TestDefine, v and err may not yet be set by the time the condition is checked.

I have the project wich is used goja to render React.js at server-side. Before you published your goja I have used go-duktape for the same purposes.
Regarding synchronous mode. I think it's not a good idea to add specific logic into fetch. Better if it would work as everyone expects.

If a project uses fetch and some other module that also requires a matching version of 'when' package for example it would be nice to avoid duplication.

A user can avoid duplication by adding one Promise library to use before enabling fetch. Take a look - https://github.com/olebedev/gojax/blob/master/fetch/fetch.js#L109
Yeah, I know that we include a lot of when code info result bundle. But as I could suppose, we compile this code once and don't use it if no needed.
What's the payment for this approach, in terms of goja's internal?
Thanks for picking typos, fixed.

It will be compiled and run. Depending on your usage profile it may not be a big deal if it only happens once at start, but in other cases it may be significant. You can do your own benchmarks but compiler is rather crude at the moment especially when it comes to memory allocations. Maybe one day I'll rewrite it if I have time...
For now I'm closing this one, feel free to open another issue if you have more questions.


Add option --ssh-store-settings

This flag allows users to run nixos-anywhere with extra settings for SSH store, by appending it to ssh store url.
If a user has fast connection between client and target, but slow internet connection on target, the user can use --no-substitute-on-destination to copy closures from client instead of downloading it from internet.
The copy speed (nix-copy to ssh store) can be greatly improved by enabling compression via --ssh-store-settings "compress=true".
Existing url parameters (especially the special characters) need to be encoded to avoid unexpected results. Otherwise we will get something like root evaluates to /mnt?compression=true.

The code intentionally have ? and & at the end of the URL even without store settings passed, which is still a valid URL.
This is done to avoid complexity and make the PR diff as small as possible.
Please let me know if we want to not include these characters when ssh store settings are not used.

@aabccd021 Sorry for the delayed response. Idea sounds good to me - could you rebase?


Fix issue #49: Retry rmdirSync on Windows for up to 1 second if files still exist.

See issue #49

I have an idea on how to make a test for this using https://github.com/cjohansen/Sinon.JS by stubbing fs.rmdirSync to return that error on first call, and process to return win32 but I don't know if you want to take a dependency for it. Thoughts?

Looks good, I don't think we need to a specific platform test for this.. I need a second pair of eyes though - @Schoonology ?

Looking over that issue again, I wouldn't retry on EPERM - getting that error is the best way to know you've got multiple processes contending for those files.

I've left all the comments I can think of for now. Thoughts, rebuttals, etc.? Though this is a relatively small amount of code, I feel like rm is somewhat important to get right...

@Schoonology I fixed the style issues you pointed out and re-pushed.
Regarding EPERM: It was one of the errors I was getting for which retry helped. For the EPERM case, it could be an antivirus related issue. I don't think 1 second of retrying is that much of a big deal if the file is actually in use.
Regarding checking for win32: on other plaforms the files should disappear even if they're in use, it's only Windows that locks files from being deleted under such circumstances. If an error occurs on Linux for instance, it's probably because of permissions, not because of quirks such as this one. (at least - I couldn't reproduce this error on Linux and MacOSX, it works fine there)

There's one more issue here: In my usage, after doing a rm -rf, I'm recreating the same directory I deleted.
This sometimes results in an error in mkdir because the directory didn't disappear yet. There needs to be an additional check on Windows whether the directory is gone before returning from the function. There won't be any error in this case in rmdirSync and nothing logged, but it just takes one extra milisecond for the directory to actually disappear from the file system.
To reiterate, shelljs is being used as part of a test suite for a library I'm working on and this was the final fix to all of the shelljs related problems I was getting running this test suite of 100+ tests on Windows.

@Schoonology need your input on https://github.com/arturadib/shelljs/pull/179/files#diff-d0eda960c284276e6b0b269826fec539R56
I thought this was the simplest, I don't think that line will ever throw on non-Windows, but if it ever does, I think it's a good thing. I assume the calling application would want to know an allegedly successful call to remove a directory somehow still left the directory behind. Thoughts?

I'm good with this, and I think @arturadib's concerns are satisfied with that outer catch, just as you pointed out.
Squash it and :shipit:.

bump. It'd be nice if this could get merged, I'm running into it a lot on Windows...

changes published to version 0.5.1, thanks everyone!

Coool ! :+1:


Total 0 usable reads were sampled

Activate in a conda environment, no prebuilt kallisto index
ran using this command:
check_strandedness --gtf /SAN/vyplab/vyplab_reference_genomes/annotation/human/GRCh38/gencode.v31.annotation.gtf --reads_1 /SAN/vyplab/alb_projects/data/ibm_geo/raw_data/SRR5888449_1.fastq --reads_2 /SAN/vyplab/alb_projects/data/ibm_geo/raw_data/SRR5888449_2.fastq --transcripts /SAN/vyplab/vyplab_reference_genomes/sequence/human/gencode/gencode.v31.transcripts.fa
Output:
Results stored in: stranded_test_SRR5888449_1
converting gtf to bed
generating kallisto index

[build] loading fasta file /SAN/vyplab/vyplab_reference_genomes/sequence/human/gencode/gencode.v31.transcripts.fa
[build] k-mer length: 31
[build] warning: clipped off poly-A tail (longer than 10)
        from 1767 target sequences
[build] warning: replaced 4 non-ACGUT characters in the input sequence
        with pseudorandom nucleotides
[build] counting k-mers ... done.
[build] building target de Bruijn graph ...  done
[build] creating equivalence classes ...  done
[build] target de Bruijn graph has 1468053 contigs and contains 139801986 k-mers

creating fastq files with first 200000 reads

gzip: stdin: not in gzip format

gzip: stdin: not in gzip format
quantifying with kallisto

[quant] fragment length distribution will be estimated from the data
[index] k-mer length: 31
[index] number of targets: 226,882
[index] number of k-mers: 139,801,986
[index] number of equivalence classes: 960,759
Warning: 226882 transcripts were defined in GTF file, but not in the index
[quant] running in paired-end mode
[quant] will process pair 1: stranded_test_SRR5888449_1/SRR5888449_1_sample.fq
                             stranded_test_SRR5888449_1/SRR5888449_2_sample.fq
[quant] finding pseudoalignments for the reads ... done
[quant] processed 0 reads, 0 reads pseudoaligned
[~warn] no reads pseudoaligned.
[quant] estimated average fragment length: 0
[   em] quantifying the abundances ... done
[   em] the Expectation-Maximization algorithm ran for 52 rounds
[  bam] writing pseudoalignments to BAM format .. done
[  bam] sorting BAM files .. done
[  bam] indexing BAM file .. done

checking strandedness
Reading reference gene model stranded_test_SRR5888449_1/gencode.v31.annotation.bed ... Done
Loading SAM/BAM file ...  Finished
Total 0 usable reads were sampled
Unknown Data type
Traceback (most recent call last):
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/bin/check_strandedness", line 8, in <module>
    sys.exit(main())
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/how_are_we_stranded_here/check_strandedness.py", line 142, in main
    print(result.iloc[i,0])
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/pandas/core/indexing.py", line 1418, in __getitem__
    return self._getitem_tuple(key)
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/pandas/core/indexing.py", line 2092, in _getitem_tuple
    self._has_valid_tuple(tup)
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/pandas/core/indexing.py", line 235, in _has_valid_tuple
    self._validate_key(k, i)
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/pandas/core/indexing.py", line 2014, in _validate_key
    self._validate_integer(key, axis)
  File "/SAN/vyplab/alb_projects/tools/.conda/envs/kallisto/lib/python3.7/site-packages/pandas/core/indexing.py", line 2088, in _validate_integer
    raise IndexError("single positional indexer is out-of-bounds")
IndexError: single positional indexer is out-of-bounds`

Solved by adding a check if the file is gzipped


Add computer difficulty levels

There should be an option to choose between several options for the difficulty of the AI. This could be as simple as just increasing the depth that is searched for each difficulty level.

Added in 0efcdf3.

Added in 0efcdf3.


go to start/end date on focus

a user selects a start date, navigates 6 months into the future and selects an end date
the user decides they want to modify the start date so they have to navigate all the way back to the start date
if focusing on the start date or end date provided a hook that would allow you to focus on the start/end date, this would somewhat simplify navigation

In a way this would be addressed with https://github.com/airbnb/react-dates/issues/25, but I think a user expects that clicking on a pre-selected date, or manually entering a date, would navigate to that date in the calendar popover.

Hmm I think actually https://github.com/airbnb/react-dates/issues/48 is more inline with this prompt. If the visible month itself was controlled, then we'd be able to set it based on whether start date or end date was in fact focused.
Unfortunately, there are a lot of open questions about the animation in particular that I would want to address.

ok, thanks - had a feeling that is the case - I will close this since it looks like #48 would provide the ability to introduce this behavior

I'm not sure if this is the same request or not, but I was able to solve this with:
[...]
                initialVisibleMonth={
                    () => {
                        if (this.state.focusedInput === 'endDate') {
                            return this.state.endDate;
                        } else {
                            return this.state.startDate;
                        }
                    }
                }

Before, it always focused on the startDate

After implementing initialVisibleMonth, it focuses on whatever date you selected. (This is particularly helpful if the endDate is years away from the startDate)

Huge thanks for the wonderful library ✨  !


How to use Tensorboard with LSTM cells

Hi
I am trying to use Tensorboard with your example code. Can you please suggest how to approach this using your code structure?

I just update the code to use tensorflow-1.1.0, and a library polyaxon, and should be outputing multiple tensorboard data.


Add Styles for Responsiveness

Purpose

Added styles for effect only for <1680 sizes.
Fixes #4874

This is how its looks in my local computer with different resolutions

Now its looks like this


programs.sqlite for darwin

Database for command-not-found is currently generated only for nixos (https://github.com/NixOS/nixos-channel-scripts/blob/master/mirror-nixos-branch.sh#L94).
Is it possible to generate it for nixpkgs and for darwin platform to use command-not-found there?

cc @copumpkin @vcunat

Well, that command currently outputs suggestions of type nix-env -iA nixos.foo, so I assume that's why it's only generated on the "nixos" channels. Note that darwin might have a nontrivially different set of executable names, so it should probably get a separate database file.
BTW, @edolstra seems to have rewritten the script in the meantime. I know rather little about these personally.

"nix-index" is now available and works on Linux and macOS systems. You have to generate the database yourself for now, but hopefully in the future we can bundle it in channels like programs.sqlite does now.
https://github.com/bennofs/nix-index


How do I keep the snackbar on screen without dismiss automatically?

I wanna show Get.snackbar for connectivity status and I wanna keep it on screen until the connection is ready again. But I could not found any property to do that. "isDismissable" is not working properly or I'm misundertanding its feature.

Do you need to be persistent between screens?

Do you need to be persistent between screens?

Yes! I think it should be persistent between screens.


[4.0.0-beta.17] backdropDismiss doesn't works

Bug Report
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
   ionic (Ionic CLI)             : 4.4.0 (/Users/stewan/.nvm/versions/node/v10.11.0/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.17
   @angular-devkit/build-angular : 0.11.0
   @angular-devkit/schematics    : 7.1.0
   @angular/cli                  : 7.1.0
   @ionic/angular-toolkit        : 1.2.0

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3,cordova-plugin-ionic-webview 2.2.5, (and 11 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/stewan/Library/Android/sdk/)
   ios-deploy        : 1.9.4
   ios-sim           : 7.0.0
   NodeJS            : v10.11.0 (/Users/stewan/.nvm/versions/node/v10.11.0/bin/node)
   npm               : 6.4.1
   OS                : macOS
   Xcode             : Xcode 10.1 Build version 10B61

Describe the Bug
looks like it doesn't even exists in typescript

On which element, just had a look in my app with bet.a17 and it still seems ok with alertController or popoverController

@stewwan yep agree, there isn't this option for Loading. Don't know if it's correct and then the doc should be modified or if it should be implemented
https://github.com/ionic-team/ionic/blob/0680fe925145fbff927f1b0a73b3dd6f3ccee46e/core/src/components/loading/loading-interface.ts
https://github.com/ionic-team/ionic/tree/0680fe925145fbff927f1b0a73b3dd6f3ccee46e/core/src/components/loading

Sad but true, it was working before. hope they can implement it sooner because I need this option for a killer feature

I think I found the problem, but I have no time to PR today :( if no one does until Monday, I will.
tappable={false} should be {this.backdropDismiss}
https://github.com/ionic-team/ionic/blob/0680fe925145fbff927f1b0a73b3dd6f3ccee46e/core/src/components/loading/loading.tsx#L204

Then please post a PR :)


Unable to open channels

Hi @Stadicus,
Thanks for the excellent guide.
I have followed your guide until the end and everything is ok. But then after funding my node, I see no channels opened, I have added several peers and lnd log says it's discovering channels.
For example trying to make the payment to you in Y'alls results in:
{ 	"payment_error": "unable to find a path to destination", 	"payment_preimage": "", 	"payment_route": null }
But the graph of nodes is being constructed correctly... any idea?
EDIT:
Also I have tried to open a channel with Y'alls in mainnet and lncli gets stuck indefinitely (until CTRL+C)...
Thanks.

Hi, @piraces, there was a previous issue that you might be able to check out to see if it helps out. It can be found here #73

Are you on testnet or mainnet? If I understand correctly, bitcoind and lnd are running without issues, the funds are present and you could discover several peers. But you cannot make any payments.
The first thing to check is if autopilot created payment channels for you, otherwise you can open a few yourself. Does the log show any strange issues, or is it chatting merrily?
$ sudo journalctl -f -u lnd

Can you post the following outputs?
$ lncli getinfo
$ lncli walletbalance
$ lncli channelbalance

If you open a new channel (you can find peers here: https://1ml.com), use the following commands:
admin ~ ฿ lncli connect 036b32ac6acf6d178f47c2139b7327ab85bd3d5f5c40681a9a48109ea21f53e1e5@<IP_ADDRESS>:9735
admin ~ ฿ lncli openchannel 036b32ac6acf6d178f47c2139b7327ab85bd3d5f5c40681a9a48109ea21f53e1e5 20000
{
        "funding_txid": "04709256a95279dbc273fbe8b04e2745729f253e61015d12fc0ddd2f36edb008"
}

You can check the log (use either mainnet or testnet) to search the log for activity regarding this peer:
$ sudo grep 036b32ac6acf6d178f47c2139b7327ab85bd3d5f5c40681a9a48109ea21f53e1e5 /home/bitcoin/.lnd/logs/bitcoin/mainnet/lnd.log

Hi @Stadicus and @Cees2daBees, thank you for your help.
Apparently restarting my node and about 10 hours after, 2 channels were automatically opened!
Now I have tested your commands and trying to open a channel works perfect and now I can make payments.
I was on mainnet.
You can close this issue 👍

Woohoo, thanks for the feedback, great to hear!


[type:bug] Fix http client register of spring mvc examples bug.

When using HTTP to register Springmvc-examples, the registration failed, because some modifications were left in the previous refactoring.
Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3990 (d231bdf) into master (aaad59f) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3990      +/-   ##
============================================
- Coverage     69.90%   69.85%   -0.06%
+ Complexity     6673     6667       -6
============================================
  Files           924      924
  Lines         25195    25187       -8
  Branches       2276     2273       -3
============================================
- Hits          17613    17594      -19
- Misses         6213     6221       +8
- Partials       1369     1372       +3

Impacted Files
Coverage Δ

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
85.71% <0.00%> (-4.09%)
:arrow_down:

...yu/common/dto/convert/rule/Resilience4JHandle.java
71.60% <0.00%> (-3.71%)
:arrow_down:

...he/shenyu/loadbalancer/spi/RandomLoadBalancer.java
87.87% <0.00%> (-3.04%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
62.66% <0.00%> (-1.34%)
:arrow_down:

...rg/apache/shenyu/plugin/hystrix/HystrixPlugin.java
75.86% <0.00%> (-0.81%)
:arrow_down:

...yu/admin/service/impl/PluginHandleServiceImpl.java
75.47% <0.00%> (-0.46%)
:arrow_down:

.../plugin/grpc/loadbalance/AbstractLoadBalancer.java
70.32% <0.00%> (-0.33%)
:arrow_down:

...enyu/plugin/sofa/cache/ApplicationConfigCache.java
69.44% <0.00%> (-0.29%)
:arrow_down:

.../plugin/base/cache/CommonPluginDataSubscriber.java
73.00% <0.00%> (-0.27%)
:arrow_down:

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dragon-zhang pls review.

@dragon-zhang pls review.

ok, I'll review later.


[SPARK-16044][SQL] input_file_name() returns empty strings in data sources based on NewHadoopRDD

What changes were proposed in this pull request?
This PR makes input_file_name() function return the file paths not empty strings for external data sources based on NewHadoopRDD, such as spark-redshift and spark-xml.
The codes with the external data sources below:
df.select(input_file_name).show()

will produce

Before

+-----------------+
|input_file_name()|
+-----------------+
|                 |
+-----------------+

- **After**

+--------------------+
|   input_file_name()|
+--------------------+
|file:/private/var...|
+--------------------+

## How was this patch tested?

Unit tests in `ColumnExpressionSuite`.

Test build #60769 has started for PR 13759 at commit 10dedc2.

cc @cloud-fan Could you please take a look maybe? I remember renaming SqlNewHadoopRDDState to InputFileNameHolder was reviewed by you.

Test build #60769 has finished for PR 13759 at commit 10dedc2.

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/60769/
Test PASSed.

LGTM - merging in master/2.0.

Thank you @rxin! Would this be sensible if this one is backported to 1.6.2?

Can you submit a pr for 1.6? Thanks.

@rxin Sure!


Potential namespace conflict with ts.el

Hi @ubolonton,
Following the recent discussions on emacs-devel, I saw you mention a ts-parse function.  I can't find that function's definition in the master branch, but I do see some calls to it.  So I thought I should mention that there is a potential namespace conflict with my library https://github.com/alphapapa/ts.el, which implements a ts-parse function.
Thanks for your work on this package.  I'm looking forward to the improvements it will bring to Emacs.

Thanks for the kind words!
ts-parse is defined by the dynamic module here.
I think I'll rename it to something else, maybe ts-parse-chunks. It may be better if my package switches to another prefix, but I can't think of a satisfactory one currently.

Naming things is hard.  :)  Thanks.

On a side note, does the tree-sitter-dyn.so that's mentioned here need to be updated to account for this change?

On a side note, does the tree-sitter-dyn.so that's mentioned here need to be updated to account for this change?

Yes, but I haven't uploaded the new version yet, since I intend to add a mechanism for the binaries to be versioned and downloaded separately first, so that the package as a whole can be published on MELPA.
Do you need this change soon?

Thanks for the status update.
Initially I didn't know how to build it and put the result in place, but that turned out to be doable, so I'm all set :)
Wanted to try the query builder and that now seems to work:

Wanted to try the query builder

I made most of that feature for several months, any feed back? :D

Below are some things I noticed about the version I tried at the time of the screenshot (I went back and tried it again just now -- and I haven't updated since that time):

I thought it was helpful in making it clear what matched.
I've used it on one of Clojure's larger source files (core.clj 263653 bytes) and it seemed to perform without any difficulty so that's definitely nice.
The echo area was a bit noisy while the query is being constructed and not complete (Error: (rust-panic "called ...) etc.), but I can understand at this stage that it would be helpful to have that information (at least if you are the one developing the query builder).

Overall I think it is nice for testing out queries as well as for helping to interactively develop one's mental model of how tree-sitter queries work.
Thanks for making it!

Two ideas for a unique namespace:
Short, practical -> "ets" for "emacs-tree-sitter"
Evocative -> "canopy" as in "where tree-sitters live"

Since https://github.com/ubolonton/emacs-tree-sitter/commit/8e17490f908751c08aa90c1523baa29a4f1c0c25, the core APIs' prefix has been changed from ts- to tsc-.


ResNet Benchmark

This PR adds the ResNet + ImageNet benchmark. This code should replicate our baseline numbers and includes the option to train using our three recipes. There is code for both local and streaming data. The code should be able to pull any ResNet specified by torchvision.
One major thing missing is FFCV support. I'm not sure if we want people to be using FFCV workflows and would add some complexity to the code base. It seems like decent effort to convert a standard dataset into an FFCV one. This means the mild  and possibly the medium and hot recipes will be dataloader bottlenecked. A potential alternative to FFCV would be NVIDIA's Dali, but that has yet to be scoped out.
I attempted to write tests that can be run using pytest within the resnet directory. Unfortunately, I wasn't able to make testing for streaming datasets since I'm not sure how to do this, is this necessary before merging? Otherwise, this works for the most part, but I had issues importing and using the main() function for the smoke test. The workaround was to add the current directory to sys.path, but I don't think this would be an issue if there was a setup.py. I think we need to set one up at some point
Baseline WandB result: 76.47% accuracy in 3.06 hours

@dblalock I believe I addressed most of your comments, but let me know if I missed anything! I wasn't sure about a few of the comments, so I left those open. Let me know what you think.


on incoming message error, remove conn from cpool and peers.

Motivation
We observed a bug in testnet where a connection is closed due to an error but protocols keep trying to send messages to this connection.
Closes #1865
Changes

When a connection is closed due to an processing error of an incoming message, remove the connection from the connection pool and the neighbors list.
Test Plan
added UT
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

bors merge

bors merge