ERROR:    Error loading ASGI app. Could not import module "tools.api".

Self Checks

[X] This template is only for bug reports. For questions, please visit Discussions.
[X] I have thoroughly reviewed the project documentation (installation, training, inference) but couldn't find information to solve my problem. English 中文 日本語 Portuguese (Brazil)
[X] I have searched for existing issues, including closed ones. Search issues
[X] I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
[X] [FOR CHINESE USERS] 请务必使用英文提交 Issue，否则会被关闭。谢谢！:）
[X] Please do not modify this template and fill in all required fields.

Cloud or Self Hosted
Self Hosted (Source)
Environment Details
following your instructions,
and run
python tools/api.py --llama-checkpoint-path checkpoints/fish-agent-v0.1-3b/ --mode agent.
and the error happens without more infomations about what the error exactly is.
If the error is catched, I can solve myself quite easily. IMO however you are using kui which just swallow the traceback of python, then makes it difficult to debug.
BTW I run python tools/webui.py without errors
Steps to Reproduce
python tools/api.py --llama-checkpoint-path checkpoints/fish-agent-v0.1-3b/ --mode agent.
✔ Expected Behavior
run api
❌ Actual Behavior
nothing

please use
python -m tools.api --llama-checkpoint-path checkpoints/fish-agent-v0.1-3b/ --mode agent

to run the api

that's the line I run and fail before this. It throws an error says it can't find tools.api
---- Replied Message ----
| From | @.> |
| Date | 11/09/2024 16:00 |
| To | fishaudio/fish-speech @.> |
| Cc | neve33210 @.>,
Author @.> |
| Subject | Re: [fishaudio/fish-speech] ERROR: Error loading ASGI app. Could not import module "tools.api". (Issue #672) |
please use
python -m tools.api --llama-checkpoint-path checkpoints/fish-agent-v0.1-3b/ --mode agent
to run the api
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

export PYTHONPATH=$PWD


"Măriuța" is listed in the drop-down menu

Description:
"Măriuța" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Măriuța" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Măriuța, CĂLĂRAȘI (Belciugatele)".


[14.0] ISPN-15555 Update com.github.ben-manes.caffeine:caffeine to 3.1.8

https://issues.redhat.com/browse/ISPN-15555

The code doesn't compile

The code doesn't compile

I just cherry-picked the other commits, I will take a look.


Please include support for Python exception handling

Numba currently does not seem to support exception handling in Python. Please include the functionality so that Numba does not raise errors during try except blocks in Python code.

Thanks for the report. Basic support for this was added in Numba 0.47.0 http://numba.pydata.org/numba-doc/dev/reference/pysupported.html#exception-handling
If you would like to try out the latest releases of Numba they are available in the numba channel:
conda install -c numba numba=0.47.0


Create siemens.txt

@TheBolliwood Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


update site

It seems the update site is down (I get a 404 error). Can you please investigate?

:+1:

I don't think the site is down. If you hit the URL with a browser you'll get a 404, but it appears to work fine from the eclipse update manager.
Think it might need an index.html page adding to avoid the confusion.

I've updated the readme.  The preferred way to install is via Eclipse marketplace.  The update site is not viewable from a browser.
I have just tried the update site from an eclipse install locally and it seemed to work.   Can you confirm and close the issue?
Thanks

i already solve the problem.


Contact email

Hi,
I tried to contact you at<EMAIL_ADDRESS>but I get no answer.
Can you check your email please ?
Thanks,
Mikael

Hi,
Just a small update.
If you can check the email and contact me.
Thanks,
Mikael


Update/zio aws dynamodb <IP_ADDRESS>

Fixes https://github.com/zio/zio-dynamodb/pull/513
About this PR
📦 Updates

dev.zio:zio-aws-dynamodb
dev.zio:zio-aws-netty

from <IP_ADDRESS> to <IP_ADDRESS>
📜 GitHub Release Notes - Version Diff

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


Upgrade Snakeyml and Jackson

Signed-off-by: Sooraj Sinha<EMAIL_ADDRESS>Description
Bump Snakeyml to 1.32 and Jackson to 2.13.4 to address the CVE: https://nvd.nist.gov/vuln/detail/CVE-2022-25857
Issues Resolved
#569
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

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

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

Codecov Report

Merging #573 (ab4231b) into main (e1dc511) will decrease coverage by 4.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #573      +/-   ##
============================================
- Coverage     75.08%   70.88%   -4.21%
+ Complexity     1007      968      -39
============================================
  Files           141      141
  Lines          4595     4595
  Branches        506      506
============================================
- Hits           3450     3257     -193
- Misses          834     1010     +176
- Partials        311      328      +17

Impacted Files
Coverage Δ

...rch/replication/action/status/ShardInfoResponse.kt
35.29% <0.00%> (-56.31%)
:arrow_down:

...cation/action/changes/TransportGetChangesAction.kt
63.49% <0.00%> (-26.99%)
:arrow_down:

...arch/replication/metadata/UpdateMetadataRequest.kt
77.27% <0.00%> (-22.73%)
:arrow_down:

...lication/action/status/TranportShardsInfoAction.kt
80.00% <0.00%> (-20.00%)
:arrow_down:

.../action/status/TransportReplicationStatusAction.kt
79.34% <0.00%> (-18.48%)
:arrow_down:

...ch/replication/task/index/IndexReplicationState.kt
51.66% <0.00%> (-15.00%)
:arrow_down:

...ication/action/setup/TransportSetupChecksAction.kt
59.52% <0.00%> (-14.29%)
:arrow_down:

...ication/seqno/RemoteClusterRetentionLeaseHelper.kt
80.00% <0.00%> (-11.67%)
:arrow_down:

...tion/repository/RemoteClusterMultiChunkTransfer.kt
90.24% <0.00%> (-9.76%)
:arrow_down:

.../replication/metadata/store/ReplicationMetadata.kt
62.96% <0.00%> (-8.65%)
:arrow_down:

... and 16 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Print more helpful error when file does not exists for coverage check

How is ptr not doing what you expect?
If you add a file into require_coverage in setup.(cfg|py) you get a KeyError stack trace, which is not as helpful as it could be.
Traceback (most recent call last):
  File "/tmp/test_aioexa/bin/ptr", line 10, in <module>
    sys.exit(main())
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 1059, in main
    args.venv_timeout,
  File "/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
    return future.result()
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 972, in async_main
    venv_timeout,
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 915, in run_tests
    await asyncio.gather(*consumers)
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 674, in _test_runner
    force_black,
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 625, in _test_steps_runner
    stats,
  File "/private/tmp/test_aioexa/lib/python3.7/site-packages/ptr.py", line 191, in _analyze_coverage
    if coverage_lines[afile].cover < cov_req:
KeyError: 'aioexabgp/client/pipes.py'

What is your suggestion to make this better?
Catch the KeyError, check if file exists and LOG a friendly saying so.

I feel we should also error + exit so that the error does not get ignored

Code/Bug example?
See stack trace above
How can a developer reproduce this?
Add a non existent file into require_coverage

I'm currently working on this!

Thanks! Please put a PR up and add a unittest for this. If you get stuck feel free to put a PR up and I can comment on it :)

Fixed by #64


Twitter Login Broken

Attempting to login with Twitter gives a blank page at:
https://www.domain.com/ccm/system/authentication/oauth2/twitter/attempt_auth
Issue appears to be at /concrete/src/Authentication/Type/OAuth/OAuth1aGenericOauth1aTypeController.php Line 13
$token = $this->getService()->requestRequestToken();
The service returned is 'Concrete\Authentication\Twitter\Controller' but the requestRequestToken method does not appear to be defined there or anywhere else.

Nevermind. My Keys had a space in them.

I've been thinking about this some more and I think this actually is a bug that should be fixed. This is a case where there is for sure an issue but blame is split between multiple parties:

Twitter is at fault for including whitespace when you copy keys through their interface
The user is at fault for not noticing and removing those spaces
Concrete5 is at fault for allowing the user to enter invalid keys but then silently failing

Fortunately, the fix is as easy as a pair of trim commands.

https://github.com/concrete5/concrete5/pull/9479

Thanks!


Update copyright notice

/cc @kborchers

This may be easier and completely up to you but we don't have any requirement for you to have license/copyright headers in every file so you could just delete those and make sure the LICENSE file is updated.

Current coverage is 100% (diff: 100%)

Merging #1621 into master will not change coverage

@@           master   #1621   diff @@
=====================================
  Files           1       1
  Lines        3919    3919
  Methods         0       0
  Messages        0       0
  Branches      668     668
=====================================
  Hits         3919    3919
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update 5296b84...9506bbc

@kborchers Let's do that separately so that this step (copyright update) is memorialized first for the benefit of any future git log or git blame.


[FLINK-20938] Flink cos fs hadoop feature

What is the purpose of the change
This PR will add an implementation of a file system that read from & write to Tencent's COS  so that users can use COS with Flink without depending on Hadoop. In this way, users will find it is more easily to use COS with Flink now.
This implementation wraps CosNFileSystem and shade its dependencies. However, the wrapped jar is not in Flink's lib directory, users need to copy the jar which built in opt directory to plugins directory.
Brief change log

Adds flink-filesystems/flink-cos-fs-hadoop

Verifying this change
This implementation adds some tests to test instantiation and some reads & writes & lists operations(test communications with  COS). However, in order to run these tests, someone needs to have his own COS  secretId, secretKey and region. Then, set environment variables below：
export IT_COS_TEST_SECRET_ID=<Your access secretid>
export IT_COS_TEST_SECRET_KEY=<Your access secretkey>
export IT_COS_TEST_REGION=<Your region>
export IT_COS_TEST_BUCKET=<Your bucket for test>
export IT_COS_TEST_COSN_IMPL=org.apache.hadoop.fs.CosFileSystem
export IT_COS_TEST_AFS_COSN_IMPL=org.apache.hadoop.fs.CosN
some tests are skipped by default.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@yanghua  can you have a review, ths


get_market_history doesn't actually support count

Bittrex.get_market_history has a count parameter, however the Bittrex API doesn't support that parameter. The method always returns 200 results.
https://github.com/ericsomdahl/python-bittrex/blob/master/bittrex/bittrex.py#L236
https://bittrex.com/home/api , scroll to /public/getmarkethistory

@brousch agreed - the API used to support it but the parameters were removed.
A branch has started on a 2.0 API update.  In addition to 2.0, extraneous depth parameters are being removed from the 1.1 endpoints

there are several places where there is an extraneous depth parameter.  They are all being removed as part of the 2.0 support


"Status" output not reporting status (there is a workaround)

Hi @twocolors
Please remember I am willing to help with docs; I don't know how to reach you privately.
This is not a problem, I am simply reporting it.
When I use the ESP-IN node, the Status output does not send any message. That is OK, because I can still subscribe to the actual switches for their status, so my flows still work 100%. But if that output is supposed to send status data on everything, it's not working yet.
Thank you for your continued work on this project.

So, here I am using the ESPHome In node to get information about device status.
If I choose the last option (system - status), there is no data - nothing comes out.
To get status of switches, I have to select the switch itself.
Maybe I don't understand what Status is for.

yep, system - status is send any event when connect or disconnect node-red to esphome (this is status of connection for device)

system - status is set when change or fist connect, if you add after connect devie you must restart flow/full deploy (Node-Red) or reboot device and system - status will be read or wait when reboot device :)


Fix CI jobs on Kubernetes master

Kubernetes master changed the way that versions are determined, which requires a new kind build.
This PR has 3 changes:

Use a master commit of kind that includes the fix for Kubernetes
master.
Use git clone instead of git checkout (shallow) to source Kubernetes.
This lets kind correctly figure out the Kubernetes release tag.
Build kind with make install. The kind fix was not working correctly
when built with go build.

This has been tested in https://github.com/kubernetes-csi/csi-driver-host-path/pull/77

/release-note-none
/assign @pohly @jsafrane

/lgtm


DB-918 - Connector & Line Formatting

This PR handles new formatting for lines and for connectors.
Lines
For lines, we now offer a handful of different options, not all of which I am entirely sure what they do.  The main one we want is the head/tail formatting so we can add various arrow heads.  There's also some details about how lines join together and a few other properties that we don't need but I implemented while I was at it.
Connectors
Connectors were a bit awkward.  I ended up doing a bit of a workaround rather than a complete rewrite of how Steve handles autoshapes.  I'll leave a couple comments in the code about this but ultimately it works.

I'll come back and take a look at the failing tests first thing tomorrow.

Circle CI is now fixed and the few broken unit tests are also fixed.

That is handled by the validation built in to the simple type

@luna215 , just had to make another update before release.


Reindexer should be able to publish to a particular topic

So Harrison can run reindexes without triggering an entire pipeline reindex.

closing as superseded by infra updates in https://github.com/wellcometrust/platform/pull/2871


Index error: List of Index out of range

Help is requested please.

There seems to be a problem with the loading of the nuScenes dataset in your case. Verify that you have downloaded the dataset and set the config accordingly. For further information, refer to the nuScenes documentation directly.

Sorry to disturb you again. Actually I am new in that.
I downloaded mini (Subset of trainval, 10 scenes, used to explore the data without downloading the whole dataset.)
I think the dataset is loading properly (as shown in picture). But the error is coming in create_generators.

As you said, you are using nuscenes-mini. According to the loading message you are loading nuscnes-trainval.
If you want to try on nuscenes-mini, please try changing the "data_set" parameter in the config from nuscenes to nuscenes-mini.

Yeah you were right. the problem was due to loading the dataset. But it stuck in training after 2 epoch.


[RQ] general FFT interface?

I'm using the FFT in various dimensions, however I have boilerplate code that is copied over for each dimension because the FFT classes (*_1D, *_2D, *_3D) don't have a common interface.
Could it be possible to add an interface (called, perhaps "FFT") that all FFT classes could implement, so as to give generic access to the 1D versions of the methods, i.e.
{real|complex}{Forward|Inverse}({type}[] out) ?
If pull request needed, sure can do.
Cheers

To be consistent you would have to create such interfaces for all transforms. Please go ahead and create a pull request.


Reduce stack usage in loop(). no data duplication on stack

remove copy of topic string onto stack before giving it to callback()
Just move it one byte in buffer to add space for 'C' string end \0x00 same result, no stack needed.

Umh -1, payload isn't always a string

Moreover, see mqttDataCb() comments in https://github.com/Suxsem/pubsubclient/blob/master/examples/mqtt_esp8266/mqtt_esp8266.ino

Topic can not contain embedded '0x00' chars.
4.7.3 Topic semantic and usage
http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718109
The fix reduces the stack usage, it no longer copies the full topic onto the stack.
I am not touching any other payload.
Please read the code before commenting.

You're right I read payload instead of topic sorry.
I will review the proposal more carefully and report back here

Looks sensible to me


ssload doesn't work with USB Cart Dev

Currently I'm using ftx as a woarkaround

Solution is to either debug ssload D2XX drivers or reimplement with libftdi.

Implemented support. Can you please test and verify that it works? make clean followed by make should be enough provided you have libFTDI installed.

it compiles now... and works !

Thanks for the update. Closing. Open a new issue if a problem occurs.


Middleware poweredBy() causes error when using DurableObject

It happened to me with this simple code:
const app = new Hono<{ Bindings: Env }>()

app.use('*', poweredBy())

app.get('/', (c) => {
  return c.text('Hello Hono!')
})

app.get('/test', async (c) => {
  const id = c.env.AUTH.idFromName(c.env.DURABLE_AUTH_NAME)
  const stub = c.env.AUTH.get(id)
  return await stub.fetch(c.req)
})

export default app

When I run yarn dev --local and then access /test in browser, the console explodes with this error:
Trace: TypeError: immutable
    at Headers.append (/Users/polt/aces-workers/auth/node_modules/undici/lib/fetch/headers.js:227:13)
    at /Users/polt/aces-workers/auth/node_modules/hono/dist/middleware/powered-by/index.js:5:19
    at /Users/polt/aces-workers/auth/node_modules/hono/dist/hono.js:163:50
    at EventTarget.[kDispatchFetch] (/Users/polt/aces-workers/auth/node_modules/@miniflare/core/src/standards/event.ts:385:13)
    at Server.<anonymous> (/Users/polt/aces-workers/auth/node_modules/@miniflare/http-server/src/index.ts:298:20)
    at Hono.errorHandler (/Users/polt/aces-workers/auth/node_modules/hono/dist/hono.js:29:15)
    at /Users/polt/aces-workers/auth/node_modules/hono/dist/compose.js:52:27
    at /Users/polt/aces-workers/auth/node_modules/hono/dist/hono.js:163:50
    at EventTarget.[kDispatchFetch] (/Users/polt/aces-workers/auth/node_modules/@miniflare/core/src/standards/event.ts:385:
⎔ Shutting down local server.

The error goes away when I disable the middleware.
I am using hono-minimal template with Hono upgraded to version 2.7.2.

Update: Running yarn dev (without --local flag) produce this error:
Trace: TypeError: Can't modify immutable headers.
    at index.js:1238:19
    at async index.js:1213:50 {
  stack: TypeError: Can't modify immutable headers.
    at index.js:1238:19
    at async index.js:1213:50,
  message: Can't modify immutable headers.
}
    at logConsoleMessage (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:122442:25)
    at WebSocket2.<anonymous> (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:122216:13)
    at WebSocket2.onMessage (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:96544:22)
    at WebSocket2.emit (node:events:402:35)
    at Receiver2.receiverOnMessage (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:97432:24)
    at Receiver2.emit (node:events:390:28)
    at Receiver2.dataMessage (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:96096:18)
    at Receiver2.getData (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:96038:21)
    at Receiver2.startLoop (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:95815:26)
    at Receiver2._write (/Users/polt/aces-workers/auth/node_modules/wrangler/wrangler-dist/cli.js:95772:14)

Hi @muslax !
This is because resp is Reponse fetched from the external place.  The same error occurs in the following code as proof of this.
app.get('/', async (c) => {
  const res = await fetch('https://honojs.dev')
  res.headers.append('foo', 'bar')
  return res
})

To avoid it, we should clone it.
let res = await fetch('https://honojs.dev')
res = res.clone()

In practical terms, this only happens when Durable Objects or return fetched Responses directly. Therefore, we do not change powerd-by middleware to deal with this problem right now. So, please use clone in this case.

I hit a similar problem with etag middleware and cloudflare cache, but I solved it, so I'll leave you with some advice.
app.use("*", etag());

app.get("*", async (c) => {
  const req = c.req;

  const url = new URL(req.url);
  const cacheKey = new Request(
    url.toString(),
    req
  );
  const cache = caches.default;
  const cacheImage = await cache.match(cacheKey);
  if (cacheImage) {
    console.log(`Cache hit for: ${req.url}.`);
    //   ↓ Clone the cacheImage responce before returning it!!
    return cacheImage.clone();
  }

  console.log(
    `Cache none : generate for: ${req.url}.`
  );

  const response = await fetch(
    URL
  );

  const cacheResponse = new Response(
    response.body,
    {
      headers: {
        "Cache-Control": "public, max-age=100",
        "Content-Type":
          response.headers.get("Content-Type") ??
          "image/png",
      },
    }
  );

  c.executionCtx.waitUntil(
    cache.put(cacheKey, cacheResponse.clone())
  );

  return cacheResponse;
});

If you do not clone the cache when returning it, the next request will have a response rewrite error as shown in the issue, so be sure to clone the cache before returning it for safe handling.

Seeing this issue pop up when using cors() and trying to deal with the Upgrade header needed for durable objects + websockets. TypeError: Can't modify immutable headers. crashes the whole worker, and I'm not in a situation where clone()ing it would be the issue:
headers: {
upgrade: c.header('upgrade')
}});```

anyway, turning off the cors middleware fixes everything so, FYI :)


ConfigurationBinder missing extension on IConfiguration

In ASP.NET 5 RC1 I was able to do something like this:
var originUris = Configuration.GetSection("Cors:OriginUris").Get<List<string>>();
to retrieve a list of urls from my settings file
"Cors": {
    "OriginUris": [
      "http://localhost:8080",
      "http://localhost:60499",
      "http://localhost:53384"
    ]
  },

Now in RC2 this does not work and I have noticed that it's because Microsoft.Extensions.Configuration.ConfigurationBinder has no extension on IConfiguration to get a T from it.
Now it has things like public static T GetValue<T>(this IConfiguration configuration, string key); but it accepts a key.
How should I retrieve an array of values in a similar way as with the previous extension?

Try:
 var originUris = new List<string>();
 Configuration.GetSection("Cors:OriginUris").Bind(originUris);

Excellent @HaoK, it works well with your suggestion. Thanks.


Feature Request: Pagination

With a lot of ebooks, the book list (grid view or list view) takes a long time to display. It would be great to have pagination of some sort, or "infinite scroll".

I can probably do this along with #41 when I have the time.

Good evening,
Pagination would be awesome! Thanks in advance!

Yep. I'll implement it once I have more time.


highlight_hovered with letterboxing

Hi again!,
I am trying to implement letterboxing (black border for ultrawide wide resolution monitors).  Which is working I found an example here: letterboxing. I just use a rendertarget to an 1920x1080 image and black borders on the sides im using a 3440x1440 monitor. But now the cursor is no longer aligned with you highlight_hovered system. Do you a way that it renders cursor position relative to the cursor inside the image frame? How should I approach this?

Okay, after some inspection I found it out I had set an offset to the cursor position, it was really that simple sorry to bother you.


Allow Travis to fail when the result of the py.test run fails

Previously, Travis would report that the script passed when the py.test run
failed but the pych --testing run passed because the pych run happened last.
This allows Travis to tell us something went wrong in the first case.

For an example of this, look at the first Travis build result for pull #22

Good to go?

Yep, thanks!


Cannot Disable or unistall Minimap

Uninstalling minimap breaks the folder search functionality in atom. https://github.com/atom/atom/issues/5214

I'll need more details than just a screenshot of an empty pane...
What was the version of minimap you had installed, what version of atom are you running, etc.

Atom Version 0.175.0 and Minimap 0.2.0. Its not empty pane, you can see that in the summary it shows that 11010 results found but doesnt display it.

Current version is 3.5.6 and 4.0.0 is in the pipe. You really should update minimap, the Atom API has changed so much till that time that it's obvious it's gonna break somewhere (hence the updates...).

I am really sorry It was a wrong copy pasta. The version is actually 3.5.6.

Ok, can you try with the v4 preview setting enabled (needs a restart)?

Enabled V4 but issue still present after a restart.

I can't reproduce it (see below for animated gifs, warning, they're quite heavy)
Disabling the package:

Uninstalling the package:

And I won't see what in the minimap could affect the project search, my guess is that it has nothing to do with the minimap, but maybe another package or something else (custom styles, init, etc.).

You can Check this issue where there is one more person reproducing it. https://github.com/atom/atom/issues/5214.

Okay, this doesn't reflect my experience at all. I had this issue last night (see the original issue) and my search didn't work while minimap was installed. Removing minimap actually fixed my problem. The issue is, it isn't consistent. Everything works fine on my work computer though. Both have the latest version of atom (0.175 as of this posting) and minimap (3.5.6).

@zephraph So there must be something beside minimap. Maybe you have an old version of the minimap-find-and-replace plugin installed because there's just no code in the whole minimap source that works with find-and-replace. If it works on one computer and not the other it's likely that the configuration is different, maybe one has an obsolete version of a plugin while the other is up to date.

Maybe.
I freshly installed Atom last night on my home pc (where the issue occurred). I only had three packages enabled and I disabled those one by one. Minimap was last and after that was disabled my search results began populating. I reinstalled one of the other packages and installed several new packages and it still worked. Adding minimap back again caused it to break.
With that being said I have had atom installed on that computer before. There is a possibility that there were fragments from the last install that broke things. I'll clean everything out tonight and reinstall to see if I can reproduce the issue.

And there's no error in the console?

I stand corrected. It just started happening on my work PC as well.

After disabling minimap and reloading the window:

I've tested this and double tested. Removing all packages except minimap leaves me with the issue. Disabling minimap fixes the issue. Re-enabling minimap brings the issue back.

Hmm, what are the versions of highlight-selected and minimap-highlight-selected?
Now that I think about it I fixed something regarding mini editors in minimap-highlight-selected (see https://github.com/abe33/minimap-highlight-selected/commit/3d78a2f89d57ef29557b44bcc1608243a5133270).

It's the latest on both; 0.7.0 and 3.1.0 respectively. I've got them both uninstalled right now, with only minimap installed. I've been reloading the window in between tests, but I've also completely restarted Atom with the same net result: Having minimap enabled prevents me from using file search.
Strangely, I swear this was working yesterday. Maybe I'm going crazy, heh.

I don't think this is directly related to the minimap package. If I use a fresh installation of Atom 0.175 (linux) and only install the merge-conflicts package I get the same result. In the original issue on the atom/atom repo someone mentioned the command-logger package had a similar effect. So it is most likely a common component of the current API used by all of these packages that is causing this. I think the original issue on the atom/atom repo should be reopened and this should be closed.

I agree, this issue should be closed.


IOErr occured when client idle more than 300 seconds

I use pool. I found if a client idle more than 300 seconds(redis server default timeout is 300s), the Cmd() will fail. Someone suggest use the following code to avoid:
go func() {
    for {
        p.Cmd("PING")
        time.Sleep(1 * time.Second)
    }
}()

But can you move it to library? Because not everyone is observed this problem.

At the moment that little snippet is the best approach, unfortunately. My reasoning for this is that I'm trying to keep the pool package as simple as possible. Pools are a tricky thing where everyone wants something slightly different depending on their use-case, and it's very difficult to actually cover all use-cases with a single implementation. For example in this particular case, some users will have long running programs that need pinging, but others will have short-lived, performance critical ones where pinging would just slow things down. There's other concerns too. What if someone wants to write a log whenever the ping fails? What if they want to make a new connection vs just dropping the old? What if they want to do something crazy like re-make the entire pool when one connection fails?
The point is by keeping the implementation super simple I can accommodate all use-cases, although one some have to do more work than others. That said, I am working on a v3 of radix that will accommodate this particular use-case a bit better (since it does come up rather often). I'll link to it on the README of this project when it's ready for beta, should happen in the next month or so, so keep an eye out if you're interested in helping me test it.
Thanks for submitting the issue and PR, I'm sorry to shoot you down in both cases, but I appreciate your interest and you using the package. Let me know if there's anything else you have questions about! :)

I have also found this issue. Add some extra my case.
1 Dynamic connections pool according to the load. My service probably has more than 10 redis master-slave nodes. Each client will connect to all nodes and each pool will have 40~120 connections.So i want much more connections when there is a high peak and small connections with a low peak.
2 The PING is just sent for the idle connections.
3 I also expect the pool can do some performance improvemnt. It relies on channel. Channel is low performance compared to the mutex.
4 Pool is FIFO. But probably i want the LIFO(return the connection to the header of pool and the getting is from the head). Currently, if it's LIFO, it has a higier possiblity that the connection is unavaiable. Because the firewall probably cut the connection off if the connection is idle for 10 minutes. Redis server and client can't feel it otherwise sending message.
we probably have


The Machine Learning Landscape | Mashiro's Notes

https://mashiro.top/2018/04/27/ch-1-the-machine-learning-landscape/

Hello~


Not able to install community server

Hi Iam getting below error while installing only office community server , can u help me on this ?
ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: YES)

Hello
Please specify the type of installation and your operating system.
Are you trying to install the product using the instructions from our help center?

Hi, This issue has been fixed , I had  used wrong mysql password.I have reset mysql password to what I had given while installing only office community server .
Thanks for your response .


Why should x64-windows-static-md not be the preferred triplet on Windows ?

Windows (Win32) currently has the following triplets
x64-windows
x64-windows-static
x64-windows-static-md (community)
and then there's the following uwp triplet which maybe very well be considered separate platform but because of similiar build + consuming semantics I'm mentioning here.
x64-uwp (dyamic-lib, dynamic-crt)
The topic has been touched upon in another issue #15122  where the discussion originated but that issue was more targeted towards gathering enough consensus to at-least enable it as a tested-port. in the CI/CD Pipelines.
There's been some interest in carrying the discussion forward and documenting details around which triplet is best suited on windows
I think a whitepaper on this would go a long way to establish some knowledge base of the issues and debunking myths.
Based on my anecdotal evidence on ease of use for development, debugging, packaging
I would strongly recommend the following
x64-windows-static-md : preferred triplet
x64-uwp (static-lib dynamic-crt)
And would like to know why that is contrary (if at all) to what vcpkg recommends.
In both these cases, DLL management and packaging is a huge - nightmare and more so in x64-uwp which its is next to impossible getting the right ingredients to produce an msix/appx with multiple dlls from their libs.

I don't believe our position has changed from https://github.com/microsoft/vcpkg/issues/15122 to which you linked.

@BillyONeal , This is more about documenting why the position is what it is.
Most of us don't understand the reasoning behind that or the caveats around the inherent risks to make an informed decisions as to which of them apply.
A whitepaper by the vcpkg team on why it is what it is would help us understand it a better


Update calcite to fix handling of queries with nested group by

Depends on the linkedin-calcite release which includes: https://github.com/linkedin/linkedin-calcite/pull/38
Fixes: https://github.com/linkedin/coral/issues/72
cc: @wmoustafa @findepi

@losipiuk thank you!

Thanks @losipiuk! You can address the TODO item since version <IP_ADDRESS> is now published (thanks to @rzhang10).

Thanks for heads up.
PTAL @wmoustafa

Friendly ping @wmoustafa :)


Fix: Remove unused imports

This PR

[x] removes unused imports as they are unused

This has been corrected in the laster push to the master branch


Merge hardware and software in notebook examples

Description
Merge hardware and software in notebook examples
Something like
client = BlindAiClient()

# Comment this line for hardware mode
client.connect_server(addr="localhost", simulation=True)

# Comment this line for simulation mode
client.connect_server(
    addr="localhost",
    policy="policy.toml",
    certificate="host_server.pem"
)

Why this modification is needed?
Make the notebooks clearer, and less redundant
What documents need to be updated

[ ] Main README
[ ] CONTRIBUTING guidelines
[ ] BlindAI Client README
[ ] BlindAI Server README
[ ] BlindAI Client CHANGELOG
[ ] BlindAI Server CHANGELOG
[ ] Python Docstrings
[x] others => python examples

Additional Information
None
Checklist

[x] This issue concerns BlindAI Client
[ ] This issue concerns BlindAI Server

One person I talked to suggested we read information from an environment variable to know if the client should discuss with a simulation or hardware backend. I think default behavior should be simulation so that it always works but with a warning.
If the user wants to talk to a hardware backend, we could ask them to specify an environment variable like BLINDAI_CERTIFICATES which could point to a folder where there is a certificate and a policy file. If this environment variable does not exist or the folder is empty, it would go to simulation mode.
This way we can provide the same API everywhere.
What do you guys think @cchudant @JoFrost ?

I don't know whether this is a good idea.
This feels strange to me and not typical of library like that.
I think it's better to have simulation=True/False on connect?
As for the notebook examples, I think we should remove the hardware one and just have a comment where you need to change True to False?

Ok yeah true, maybe just remove hardware / software notebooks and put comments.
What do you think of getting the certificate and policy from env variable?

Closing since #56 solves the issue


Missing pack

The plugin info page shows a Project Miku pack, but I can't find it anywhere in the plugin settings nor the resource pack hub.  Is this missing from the plugin?  Or where can this pack be obtained?

Nevermind, found it here:
https://github.com/melkypie/resource-packs/tree/pack-project-miku
closing issue


Link to Regstr.h

To make it easier to use this property it would be nice to have a link to a copy of the regstr.h header. Or alternatively, the part of regstr.h related to this property should be listed on this page.

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

ID: ae9a626c-439a-0f73-27d1-40333c7fcc94
Version Independent ID: ec922c15-d2e3-9f9f-4b57-f3ad20759414
Content: DEVPKEY_Device_ConfigFlags - Windows drivers
Content Source: windows-driver-docs-pr/install/devpkey-device-configflags.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks @JonAxtell and apologies for the delay.  It turns out that those configuration flags are really just for internal use only.  I just updated the docs to reflect that.  That change should be live within the hour.


add a new config.yaml option gpus_are_scarce

High Level Description

introduce a new config.yaml option gpus_are_scarce with default values true
bump mesos to include --filter-gpu-resources master flag.

Related Issues

DCOS_OSS-876
CORE-1107

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
The PR does not include any integration test, since we do not have an infrastructure for integration tests with GPUs. @klueska has performed manual tests to make sure this change works as expected.
[X] Read the DC/OS contributing guidelines
[X] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[X] Change log from the last version integrated (this should be a link to commits for easy verification and review): mesos
[X] Test Results: jenkins
[X] Code Coverage (if available): jenkins

I performed a manual test for this PR as follows:
1) I spun up a cluster with:
    - 1 master
    - 3 private agents with 8 GPUs each
    - 0 public agents
    - the default config.yaml

2) I spun up a cluster with:
    - 1 master
    - 3 private agents with 8 GPUs each
    - 0 public agents
    - a config.yaml with gpus_are_scarce = false

For cluster (1), we expect any framework that doesn't register with the GPU_RESOURCES capability to never be able to run any jobs (because all agents have GPUs and it will never receive any offers).
For cluster (2), we expect all frameworks to be able to run jobs because the requirement for registering with the GPU_RESOURCES capability has been removed
Metronome is a framework that doesn't register with the GPU_RESOURCES capability, so we will use it to verify whether our new setting is working or not.
1) On cluster (1), I started a simple metronome job
    - sleep 9999
    - It never exited the starting state (i.e. it never started running)
2) On cluster (1), I started a simple marathon job which requested a GPU
    - sleep 9999
    - It started running immediately
3) On cluster (2), I started a simple metronome job
    - sleep 9999
    - It started running immediately
4) On cluster (2), I started a simple marathon job which requested a GPU
    - sleep 9999
    - It started running immediately

Verifying the settings in /opt/mesosphere/etc/mesos-master on cluster (1):
$ cat /opt/mesosphere/etc/mesos-master
...
MESOS_FAIR_SHARING_EXCLUDED_RESOURCE_NAMES=gpus
MESOS_FILTER_GPU_RESOURCES=true
...

Verifying the settings in /opt/mesosphere/etc/mesos-master on cluster (2):
$ cat /opt/mesosphere/etc/mesos-master
...
MESOS_FAIR_SHARING_EXCLUDED_RESOURCE_NAMES=
MESOS_FILTER_GPU_RESOURCES=false
...

@mesosphere-mergebot bump-ee

@darkonie Could you please comment on https://jira.mesosphere.com/browse/DCOS-13902 with the manual test case you performed above, so we can try to support such tests in an automated fashion in the future?

@adam-mesos commented

@karya0 Can you review this please?

LGTM 🚀

Is there something more I need to do to get this on a train?

No, once it has a ShipIt label, a train conductor will put it on the next
train.
In this case, I have put it on https://github.com/dcos/dcos/issues/1640
On Wed, Jun 7, 2017 at 8:09 PM, Kevin Klues<EMAIL_ADDRESS>wrote:

Is there something more I need to do to get this on a train?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dcos/dcos/pull/1615#issuecomment-306985908, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGKE0JJrkyRSiJC1DqsdNiz46ukynaUUks5sB2X7gaJpZM4Nu23S
.


Linux improvements

This doesn't quite work on Linux out-of-the-box.
The problem is that Unity Hub will install Unity into the user's home directory. To work-around this, a csproj / fsproj must have the following:
  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
    <UnityVersion>2019.4.13f1</UnityVersion>
    <UnityInstallRoot>/home/your_username_here/Unity/Hub/Editor/</UnityInstallRoot>
  </PropertyGroup>

The problem with this is that the install root depends on the username, meaning that the project file is not portable between developers.
Would it possible to add a search-path for the user's home directory by default?
Thanks for a great library!

The work-around is to add a Directory.Build.props file to root of the project folder like this:
<Project>
 <PropertyGroup>
  <TargetFramework>netstandard2.0</TargetFramework>
  <UnityVersion>2019.4.13f1</UnityVersion>
  <UnityInstallRoot>/home/your_username_here/Unity/Hub/Editor/</UnityInstallRoot>
 </PropertyGroup>
</Project>

This should be kept out of source-control, since it will vary by machine.

@njlr  Thanks for reporting this. Sorry for the delayed response, I just realized that I hadn't set GitHub to watch all activity in this repo!
Anyway, I'm glad you brought this up because I haven't actually tested this package out on Linux yet (the NuGet description technically says "on Mac or Windows", which is how I sleep at night 😆). Your Directory.Build.props approach is clever, but I disagree that all of those properties will vary by machine. Trying to collaborate on a project with different Unity versions is a nightmare, so I expect that teams will always want TargetFramework and UnityVersion to be versioned. Letting users override UnityInstallRoot in this way could work, but there may be a simpler way to get the current user's home directory, like I'm betting there's some MSBuild property to help with that. I'll have to do a little research...

@njlr Actually, we might be overthinking this. Couldn't you just use ~/Unity/Hub/Editor?

@njlr Actually, I might be overthinking this. Couldn't you just use ~/Unity/Hub/Editor? I'm not sure if MSBuild expands the ~ character correctly, but I don't see why it wouldn't.

Yeah that's the problem, MSBuild doesn't like the ~

@njlr Okay, how about this... If you set UnityInstallRoot in your csproj to the following, does the build succeed on Linux?
<UnityInstallRoot>$([System.Environment]::GetFolderPath('System.Environment+SpecialFolder.UserProfile'))/Unity/Hub/Editor/</UnityInstallRoot>

I've tested that the crazy GetFolderPath property function syntax resolves correctly on Windows, but I don't have a Linux machine on hand to test with (other than WSL distros, which I don't totally trust to represent your use case).

@njlr Okay, how about this... If you set UnityInstallRoot in your csproj to the following, does the build succeed on Linux?
<UnityInstallRoot>$([System.Environment]::GetFolderPath('System.Environment+SpecialFolder.UserProfile'))/Unity/Hub/Editor/</UnityInstallRoot>

I've tested that the crazy GetFolderPath property function syntax resolves correctly on Windows, but I don't have a Linux machine on hand to test with (other than WSL distros, which I don't totally trust to represent your use case).

Yes, seems to work!

Awesome! 🙂 Well that solves the versioned path issue. I should have a new version of the NuGet in a day or two with these changes. Please let me know if u find any other broken paths on Linux!

@njlr These Linux paths are now available in the latest package release.


Select "good" subtitle file on arte.tv

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.06.29 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hello,
Downloading subtitles on arte.tv videos dosn't work correctly.
For French language there are 2 version of subtitles:

Complete subtitle file, where everything is subtitled regardless of the language spoken
"Good" subtitle file where only languages other than French are subtitled

BTW when video is playing online it's "good" subtitle file that is selected.
You can dowload it using Firefox's "Web Development Tool" facilities: specify vtt in "Filter" while selecting "Network" tab ... link to vtt file of "good" subtitles appears.
But when using option write-subs and selecting French, it's Complete subtitle file that is downloaded.
Here is example:
https://www.arte.tv/fr/videos/104769-000-A/antonio-banderas-et-pedro-almodovar/
The size of "good" subtitles file is about 44kB, whereas size of "complete" subtitles (downloaded using option write-subs and selecting French) is about 98kB.
How to select a "Good" version of subtitles.
Thanks
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Seems related to #3302

I searched 3302, but couldn't find anything related to the problem, revealed in my 1st post.

When you ask for the subtitles for this particular video, you can see that apparently French and German have two subtitles. Otherwise how to explain two vtt symbols for French and German unlike only one for English and Italian.
Here is output of --list subs request (I did not understand why the lines are crossed out.)
(arte_env1) pavel@ALABAMA:~/arte_env1$ yt-dlp --list-subs --skip-download https://www.arte.tv/fr/videos/104769-000-A/antonio-banderas-et-pedro-almodovar/
DeprecationWarning: Support for Python version 3.6 has been deprecated and will break in future versions of yt-dlp! Please update to Python 3.7 or above
[ArteTV] 104769-000-A: Downloading JSON metadata
[ArteTV] 104769-000-A: Downloading m3u8 information
[ArteTV] 104769-000-A: Downloading m3u8 information
[ArteTV] 104769-000-A: Downloading m3u8 information
[ArteTV] 104769-000-A: Downloading m3u8 information
[ArteTV] 104769-000-A: Downloading m3u8 information
[ArteTV] 104769-000-A: Downloading m3u8 information
[info] Available subtitles for 104769-000-A:
Language Formats
fr       vtt, vtt
de       vtt, vtt
en       vtt
it       vtt
(arte_env1) pavel@ALABAMA:~/arte_env1$

Currently it is not possible to select subs (other than the lang ofc).
and yes ur assumption is likely correct that there are 2 vtt for ger and fr bcoz there are 2 types of subs available.
@fstirlitz can u take this feature in ur PR aswell. Use good- as a prefix for the subs.
So after the patch, --list subs will show:
fr vtt
good-fr vtt
de vtt
good-de vtt
en vtt
it vtt

What is a ‘good’ subtitle file?

The extractor could implement this in #3302 as Ashish said. But the general feature request is a duplicate of https://github.com/yt-dlp/yt-dlp/issues/946


Enhancement: template formatter for phone number

It would be nice to have some formatters available which I could use in the templates, like
$Tel.Nice

Hi Werner,
Yes, this sounds interesting. At the moment, the module only provides a form field storing the data in a text field. Happy to accept a PR to add a DBField with these formatters.
Cheers

This has now been implemented in version 4.0.0. See readme for formatting options.


fix: making crossFilter.chartsInScope optional

SUMMARY

It seems that these ton'g always exist in imported dashboards, which is causing trouble. Hopefully this allows imports to be upgraded to the latest code more smoothly.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

REPRO STEPS:

Roll back to a SHA earlier than https://github.com/apache/superset/pull/20743
Import a couple of dashboards
Update Superset to the above SHA (or newer)
Open imported dashboard(s) in workspace
See the error blocking dashboard loading

The above should work without error with this fix, if I'm lucky.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

i am not expert but my observations:

!chartConfig.crossFilters.scope.excluded.includes(chartId)
should this code be rollback ?


chore/DPDS-XXXX-workflow-manual-trigger

Not needed. The publish workflow automatically runs after tagging the latest master


@section and RenderSection()

@section Scripts {
@RenderSection("Scripts", required: false)
}
as far as I understand "@RenderSection" needs to be in Layout and @section needs to be in Razor Content Page. In this example they are together. Am I missing anything?

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

ID: b29734d0-2d2b-3b82-0456-785cc7f361fd
Version Independent ID: ab07129c-a4dd-44de-45f9-4b49eff0b54c
Content: Layout in ASP.NET Core
Content Source: aspnetcore/mvc/views/layout.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @ardalis
Microsoft Alias: riande

As you applied @RenderSection("Scripts", required: false)
then this section may or may not be to add in razor view or pages and depends on your need, but if we apply required: true in @RenderSection method then it will be mandatory to apply on every page where this layout page is used.

That is clear and it is not what I am asking. If you go to Sections part of the document (https://docs.microsoft.com/en-us/aspnet/core/mvc/views/layout?view=aspnetcore-2.2#sections) , the first example is what is confusing to me.

I think this is an error in the documentation that should be fixed. Having @RenderSection() inside @section doesn't seem to cause build errors, but as an example in the documentation it is at least confusing if not wrong.

as an example in the documentation it is at least confusing if not wrong
It's not wrong and might be even useful in certain cases, but it's defiantly confusing, especially in an introductory article, because it has both RenderSection and section definition.
As I said, it's not wrong, but may create a fallacious impression that RenderSection can be used only in a section.

@kosta-arnorsky  thanks for fixing. I can't think of good use case for what we had.

Fixed by #15804


Invalid subrecipient in Period 1 should not prevent uploading in Period 2

OH saw a case in which a subrecipient was part of an upload to the ARPA tool in Period 1.  Even though this subrecipient was never part of a "valid" upload, it was excluded (incorrectly) from subrecipient exports in Period 2.

If it is helpful to see which one this was - below is the upload that was not validated during a prior, closed period - but it appears Render considered the subrecipient reported and so it did not make it into the subrecipient output in the current period (period 3, ending June 30 2022)

ah the problem is in https://github.com/usdigitalresponse/arpa-reporter/blob/8fe00b86ce97e05ddcdc3b5dbe4b6f5c4cb65bc1/src/server/db/arpa-subrecipients.js#L73-L75 -- we maybe add a where('upload.validated_at IS NOT NULL') .

This fix is a bit more complicated that Igor's original diagnosis.
The issue is that we are creating records in the db for these subrecipients WHILE performing the validation. That means that we might create a db record even if the file is going to be rejected after it fails some validation later.
I think to solve this problem we need to come up with a better solution for when we add records to the database for a subrecipient. Should it happen when a file is validated successfully? Or only after the subrecipient has been included in a valid treasury report? I'll bring up this question in an upcoming partner meeting


phase visualization seems broken

This issues is coming from my attempt to answer this question https://stackoverflow.com/questions/66180030/qiskit-difference-on-statevector-no-difference-on-vizualization
import numpy as np
from qiskit.tools.visualization import plot_state_qsphere

display(plot_state_qsphere(np.array([1, 0])))
display(plot_state_qsphere(np.array([-1, 0])))

The color of the vector in the second draw should be different .

Same for plot_state_paulivec?
display(plot_state_paulivec(np.array([1, 0])))
display(plot_state_paulivec(np.array([-1, 0])))

It is bot broken per say, but implicitly normalizes the global phase via the largest magnitude component. It has done this since v1 of the qsphere code.

I found this too and figure out it was normalizing as indicated by @nonhermitian. Maybe we can deprecate this behavior in favor of not normalizing. I think the way it now produces unexpected results, following the implicit normalization, and also goes against the construction of the q-sphere.

At this point, the class DensityMatrix removes the global phase. Is it intended?
https://github.com/delapuente/qiskit-terra/blob/c3b2d7acb80fa89043e6f38efb501275ec296616/qiskit/quantum_info/states/densitymatrix.py#L98

Would like to also bump this up. In a course while teaching, people where confused why there wasnt a phase change in the QSphere visualization when Y gate was applied to the circuit. It indeed does normalize the phase which confuses a few people. Would be great if this was fixed!

Found a similar error. Y Gate not flip phase when run on circuit composer while GUI composer show the flip. Also on local computer, it doesn't flip phase.

Found a similar error. Y Gate not flip phase when run on circuit composer while GUI composer show the flip. Also on local computer, it doesn't flip phase.


Bind collider with a single IP address

is it possible to bind a collider service with a single IP address? If not, can we add such option?, for example:
/bin/collidermain -ip=<IP_ADDRESS> -port=443 -tls=true
In this case other services can be ran on the same server, same port, but different IP's

Pull requests are welcome ;) FYI You can listen on a different port and use port forwarding/mapping to map to a different external port.

With pleasure when my skills in Golang get to that level. I'll explore port forwarding/mapping options, thanks.

Closing due to age.


Lower disk usage on jenkins

Disable libnvvm API testing in numba (leave it for numbapro)
Remove cudatoolkit as test dependency

This doesn't affect the Numba-CUDA builder, right?

No, Numba-CUDA builder uses the builds from the test builds and install the CUDA dependency explicitly.  It does not use the conda recipe.


Remove some old files and updated a hive test case

Hi Chun,
Can you please review.
Thanks,
Krystal

LGTM. +1


feat(secretmanager): create update_secret_with_alias.go

Update samples for Access By Alias Launch

This is still failing with:
secretmanager_test.go:163: failed to update secret: rpc error: code = InvalidArgument desc = Aliases cannot be assigned to versions that don't exist

Looks like there's a test helper to create a secret version that we'll want to use here:
https://github.com/GoogleCloudPlatform/golang-samples/blob/main/secretmanager/secretmanager_test.go#L121-L128


MudTextField: Fix MudTextField label background

Description
Close #6457
Close #9894
How Has This Been Tested?
Add a viewer page named OutlineLabelBackgroundTest
Type of Changes

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

before

after

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

My concern is the lack of background contrast for Shrink Normal and Shrink Dense makes them hard to read.
Should probably keep the background color but take it from the field (or whatever's behind it)? I've tried to do that before though and don't have an answer.

I think this is better than it was before. Clearly before was bad:

And this is better after the change. An I consider this case as a simple "don't do that":

@charles7668 i found one bug. When Shrink Label is used or when the TextFields have a value they are always marked as focused even when they are not?
Left one is your branch, right one current mudblazor.

Fixed

@charles7668 i found one bug. When Shrink Label is used or when the TextFields have a value they are always marked as focused even when they are not?
Left one is your branch, right one current mudblazor.

Fixed

Awesome thanks! I'm just gonna double check with @danielchalmers before i merge.

I should be able to review on the weekend, thank you for the patience!

My concern that the contrast is not enough is worked around by picking appropriate colors from the start (the test page shows how important it is)

I believe MUI works the same way

LGTM


创建镜像报undefined

In GitLab by @cuiaq123 on Jul 31, 2020, 24:30

In GitLab by @luotian-github on Aug 11, 2020, 24:46
network configure error, please WangKun add more comments

In GitLab by @cuiaq123 on Aug 13, 2020, 06:22
closed


Is this opensource?

Will I need to purchase 'Handsontable Pro' license for using features like server side sorting/pagination etc?
Thanks.

No, the free version is sufficient for that. In fact, this Handsontable wrapper assumes you're using the free Handsontable version for now. If you wanted to change that, you would have to fork and adapt the code at the moment.


Fix some things

a couple of accessibility errors were fixed.
remove_car method removed from cars controller - forgot to remove it in Lra 576 lsa ride share deal with cars list in programs #65
added [data: { "turbo-frame": "_top" }] to some links get out from a turbo-frame these links are in

@rsmoke Empty tags are in the header. I did not add Titles to some columns because ... I did not ... And empty  gives an accessibility error and a suggestion

How to Fix It
If the table cell is a header, provide text within the cell that describes the column or row. If the cell is not a header or must remain empty (such as the top-left cell in a data table), make the cell a  rather than a .

I send Maria screenshots of the pages in question and my questions: do we need header titles for columns with Action links? where to put these columns: to the right or left?  Ananata suggested icons instead of links.
When I have all these answers I'll edit the PR.


Change the project name?

I've been thinking that it might be a good idea to change the name of your software. While the present name offers a reasonable suggestion of the software's function it's next to impossible to find 'When' with a search engine.
Maybe you could take inspiration from the equivalent software for Win and Mac. Both of which are named after famous butlers: Windows - Belvedere; Mac - Alfred.
My suggestion would be to twist this slightly by choosing the name Samuel . After Samuel Butler, the famous writer.

 Definitely. I don't even like the icon. I wouldn't change the name of the underlying applet, as it is somewhat self-contained, and it's more sort of a "power tool" if used alone. However, this more user-friendly interface could benefit of such a name change. I also like the clever twist and play on words: I'm seriously considering to follow your suggestion.
Thank you! :smile:
F.

If you need a new icon then what about something with a robot theme?
Robot One
Robot Two
Robot Three

My first option would be a "3D-cartoon" styled image of a butler, in case I find it, just to keep up with the joke. If not, then it could fall back to a robot(ish) thing.


Let fog-openstack compute the right keystone URL

fog-openstack version > v0.2.1 added logic to compute fog openstack
keystone url. That is why we remove our logic to compute the keystone url
fog-openstack doesn't expect any version in the url. The CPI removes now
the version from the auth url

#159296107

recheck stable/ocata


Setup Crowdin to manage translations

The way we manage translations is not friendly for external contributors that are not familiar with Git. Having a simple interface to do that could incentive non-developers to participate in this process
Another problem with the current system is that we can't easily say what percentage of translations are missing for a given locale. So even if we make the effort to fully translate the website for a given locale, there will probably be some new untranslated strings a few months later and we would have no way to directly know about that.
Crowdin could help us to fit this need by providing both a simple interface and statistics, and they are offering free licenses for open-source projects.
To do

[x] Talk with crowdin to see if we can get an opensource discount
[x] Bind the Crowdin project to Github to automatically import translations
[x] Team demo
[x] Update README and others appropriates places with the link so everyone can participate

All done :heavy_check_mark:


Fix incorrect visibility when compiling on cygwin with gcc

Whe compiling on cygwin using gcc, errors like this occur:
nb_error.h:30:17: error: 'dllexport' implies default visibility, but 'class nanobind::python_error' has already been declared with a different visibility
     30 | class NB_EXPORT python_error : public std::exception {
        |                 ^~~~~~~~~~~~

This PR should fix this, with formatting changes in nearby code.
I made reference with https://gcc.gnu.org/wiki/Visibility

Hmm, I have no plans to support CYGWIN officially as a compiler (This would require CI runners and then dealing with problems related to CYGWIN, which adds maintenance costs). I am also very hesitant to even mention its name anywhere  in the source code since that indicates to the reader that it is supported somehow.
Does the following also work? (i.e., does CYGWIN set _WIN32?)
#if defined(_WIN32) || defined __CYGWIN__
#  defined(__GNUC__)

Let's stick to the #if defined(condition) syntax which is used throughout the codebase.

Seems like newer version of cygwin no longer define _WIN32
Source: https://stackoverflow.com/questions/47150492/how-to-get-gcc-on-cygwin-to-define-win32

Then I don't understand the change you are making. If _WIN32 is not defined, the preprocessor will use the second GCC-specific part of the definitions. Which of these definitions is it that's causing a problem for CYGWIN?

I just copied boilerplate code without thinking... You are right about that, I have removed check for cygwin, please check!

Seems like newer version of cygwin no longer define _WIN32 (https://stackoverflow.com/questions/47150492/how-to-get-gcc-on-cygwin-to-define-win32)
MinGW has _WIN32 defined though (https://stackoverflow.com/questions/9025708/mingw-not-defining-win32-error-in-preprocessor-directives)

This makes sense. MinGW is supposed to be able to compile code written for the microsoft compiler, and Cygwin wants to behave more like stock GCC on Linux.
But then I don't understand the change you are making. It would seem that MinGW will parse the top set of definitions for the microsoft compiler, and the latest versions of Cygwin will use the bottom ones. You should not need to put __GNUC__ specific parts into the #if defined(_WIN32) block.

I think I made a mistake in the title, I actually discovered the problem in msys2 mingw64, not cygwin...

Great. This change is now something I can sign off on. Shall I merge it?

Sure, thank you!


add specs2 version 4

version 4 has been out for a while now is probably what we should go forward with as our only specs2 version in the 2.13 community build. I can embark on that plan right now, I think. (reasoning: any project unwilling to prioritize upgrading is a laggard project anyway and at least until later in the 2.13 cycle, we are comfortable cutting laggard projects.)
in the 2.12 build, I'm not sure. maybe start by having them both, confining specs2 4 to a separate space. then try getting rid of 3 and find out how much hell breaks loose.
fyi @etorreborre

I think this can stay the way it is for now (version 3 in 2.12 build, version 4 in 2.13 build)

if some 2.12 projects actually start requiring version 4, we'll cross that bridge when we come to it
if some 2.13 project doesn't work with version 4, we'll ask them to upgrade

we can reopen if the situation changes.


[docs, bugfix] Fix developers getting started broken links

Closes: https://github.com/napari-threedee/napari-threedee/issues/156
Fixes the broken links by linking to the markdown properly.
Also noticed a typo in the index, so I fixed that too!

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 54.46%. Comparing base (41322e0) to head (5744dfe).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #157      +/-   ##
==========================================
+ Coverage   54.43%   54.46%   +0.02%
==========================================
  Files          93       93
  Lines        3529     3529
==========================================
+ Hits         1921     1922       +1
+ Misses       1608     1607       -1

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


reduce verbiage to fit help text more efficiently

This one line of help text stands out as really wordy :-)

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Your commit needs to be signed-off (see CONTRIBUTING.md).
Also as you're modifying a translated string, you need to run "make i18n" to update the translation template and include that update in your commit.
Thanks


Plugin split of code-coverage-api plugin

Repository URL
https://github.com/uhafner/coverage-plugin
New Repository Name
coverage-plugin
Description
I am splitting the code-coverage-api into a legacy and new part. All old and deprecated code will stay in code-coverage-api, all new code will be moved to this new plugin: coverage. For details see dev mailing list thread.
Description of the plugin:
The Jenkins Coverage Plug-in collects reports of code coverage or mutation coverage tools. It has support for the following report formats:

JaCoCo
Cobertura
PIT

The plugin publishes a report of the code coverage and mutation coverage in your build, so you can navigate to a summary report from the main build page. From there you can also dive into the details:

tree charts that show the distribution of coverage by type (line, instruction, branch, method, class, etc.)
tabular listing of all files with their coverage
source code of the files with the coverage highlighted
trend charts of the coverage over time

GitHub users to have commit permission
@uhafner
Jenkins project users to have release permission
drulli
Issue tracker
Jira

Security audit, information and commands
The security team is auditing all the hosting requests, to ensure a better security by default.
This message informs you that a Jenkins Security Scan was triggered on your repository.
It takes ~10 minutes to complete.
Commands
The bot will parse all comments, and it will check if any line start with a command.
Security team only:

/audit-ok => the audit is complete, the hosting can continue :tada:.
/audit-skip => the audit is not necessary, the hosting can continue :tada:.
/audit-findings => the audit reveals some issues that require corrections :pencil2:.

Anyone:

/request-security-scan => the findings from the Jenkins Security Scan were corrected, this command will re-scan your repository :mag:.
/audit-review => the findings from the audit were corrected, this command will ping the security team to review the findings :eyes:.
	It's only applicable when the previous audit required changes.

Only one command can be requested per comment.

(automatically generated message, version: 1.26.14)

https://github.com/uhafner/coverage-plugin/blob/8cbba23f7d21511762db620ee0a1bb1097d22231/pom.xml#L7
Can you actually do that? 👀

Yes, this works in all my plugins. I have a parent module that itself has no version. Only the plug-in has a version. The hi tests do not have a version, too.

/hosting host

/hosting host

Ok, that's a good one, hosting failed because https://github.com/jenkinsci/coverage-plugin exists, and you want to have your plugin under jenkinsci/coverage.
If nobody objects, I'd rename the existing jenkinsci/coverage-plugin repository, freeing the repository namespace.
Given, GH handles renames with redirects, we don't break existing links if there are any.
It's worth noting that the existing coverage plugin is almost 15 years old, has no releases and is considered abandoned.

/hosting host

I wasn't aware that the same plugin name has been used before. I think it is ok to reuse the ID as the old plugin has been archived already.

I wasn't aware that the same plugin name has been used before. I think it is ok to reuse the ID as the old plugin has been archived already.

I think it is unwise to reuse the plugin identifier because it may confuse systems where the user has not yet removed the old plugin.  Plugin identifiers tend to be assumed to be unique in many places.  I think it is best to not risk discovering unexpected locations where reuse of a plugin identifier.

I think it is unwise to reuse the plugin identifier because it may confuse systems where the user has not yet removed the old plugin. Plugin identifiers tend to be assumed to be unique in many places. I think it is best to not risk discovering unexpected locations where reuse of a plugin identifier.

The old plugin was never released, hence safe to re-use


Support for Flat Config

Description

https://eslint.org/docs/latest/use/configure/configuration-files-new
https://eslint.org/docs/latest/use/configure/migration-guide#start-using-flat-config-files

I welcome pull requests

I'd be happy to see this issue move forward :smiley:
I'm a react dev, so I guess I could help if given some guidance, as I don't know how this project works :grimacing:

I managed to get it working, not sure if this is the best way to do it, though... here's the relevant portion of my .eslint.config.js:
// ... other eslint parsers/plugins ...
import ts from "@typescript-eslint/eslint-plugin";
import astroParser from "astro-eslint-parser";
import astro from "eslint-plugin-astro";

export default [
  // ...
  {
    files: ["**/*.astro"],
    plugins: {
      astro,
    },
    languageOptions: {
      globals: {
        // Enables global variables available in Astro components.
        node: true,
        "astro/astro": true,
        es2020: true,
      },
      parser: astroParser,
      parserOptions: {
        parser: "@typescript-eslint/parser",
        extraFileExtensions: [".astro"],
        // The script of Astro components uses ESM.
        sourceType: "module",
      },
    },
    rules: {
      ...astro.configs.all.rules,
    },
  },
  {
    // Define the configuration for `<script>` tag.
    // Script in `<script>` is assigned a virtual file name with the `.js` extension.
    files: ["**/*.astro/*.js", "*.astro/*.js"],
    languageOptions: {
      globals: {
        browser: true,
        es2020: true,
      },
    },
    parserOptions: {
      sourceType: "module",
    },
    rules: {
      // override/add rules settings here, such as:
      // "no-unused-vars": "error"

      // If you are using "prettier/prettier" rule,
      // you don't need to format inside <script> as it will be formatted as a `.astro` file.
      "prettier/prettier": "off",
    },
  },
  {
    // Define the configuration for `<script>` tag when using `client-side-ts` processor.
    // Script in `<script>` is assigned a virtual file name with the `.js` extension.
    files: ["**/*.astro/*.ts", "*.astro/*.ts"],
    languageOptions: {
      globals: {
        browser: true,
        es2020: true,
      },
    },
    parser: tsParser,
    parserOptions: {
      sourceType: "module",
      project: null,
    },
    rules: {
      // override/add rules settings here, such as:
      // "no-unused-vars": "error"

      // If you are using "prettier/prettier" rule,
      // you don't need to format inside <script> as it will be formatted as a `.astro` file.
      "prettier/prettier": "off",
    },
  },
];

ℹ the individual flat configuration are mostly copied wholesale from the user guide, with small tweaks to use the imported plugin and parser(s) - I'm sure the shareable configuration is possible, but I'm going with non-shareable.

sometimes i get this error with that implementation

@TheElegantCoding hmm, even with files set to ["**/*.astro"]? I would perhaps run eslint --debug src/ and see if it matches files?

my mistake was that i dont put ["**/*.astro"] thanks

Although the workaround works in some versions, using the new package from typescript eslint doesn't seem to work properly: https://typescript-eslint.io/packages/typescript-eslint.
It seems there's a mismatch between the TS code being evaluated vs the old typescript of astro eslint plugin while linting

Really thank you @ota-meshi, I will see the flat config support when I have time


Add then number of views in the proto list

See it here: https://beta.webots.cloud/proto

Nice. However, I feel we should invert the position of the view logo and the one of the counter like that:

Done!


feat: add ses identity notification

Assuming most projects are going to production setting up identity notifications is needed to mange complaints and bounce.
I tried to match the coding style if theres anything you need me to change just lmk and i'll try to do it as soon as i have time.

Oh interesting. Can you open an issue for this?
I also think we might want to take the ARN here instead of the ARN @fwang

Thanks @yyaskriloff!
Event destinations support more events types and destinations, and is preferred over identity notifications. I re-implemented what you had w/ event destinations here https://github.com/sst/ion/commit/ce169d8fdc29927989bb59e29461fed8cf835f2a
Will be in the next release.
^ feel free to open an issue if this doesn't work for you.


Current issues with autolamella example on v0.2-stable

I tried to run the autolamella example on commit https://github.com/DeMarcoLab/fibsem/commit/e3deb2b53360933115b87940daa556359a628735 (current head of v0.2-stable) and ran into the following issues:

state.ib_settings.resolution/state.eb_settings.resolution should be mapped back to the string f'[0]x[1]' before feeding it to the microscope, as our autoscript install will only process strings and not lists as valid values to set.
protocol_autolamella.yaml is missing a key depth for each of the lamella protocol stages. Is this a rename of milling_depth? (copying those values seemed to work)
During the milling, the example is is not moving the stage: The line microscope.set_microscope_state(lamella.state) does not update the positions, so I had to add  microscope.move_stage_absolute(lamella.state.absolute_position) before that line.

I am sorry if any/all of these issues are already fixed on the dev branch, I did not have time to check that out. Feel free to close if that is the case.
After some hacks to fix these issues, I got the example to run successfully.
Edit in case it matters: This is on a Thermo microscope

hey @sroet you are correct on all three, we've just pushed a fix onto 0.2-dev (https://github.com/DeMarcoLab/fibsem/commit/64a984f28fd063e9ad2290e9a10202b5e97bf471) if you want to try again.
We also have a more fully fleshed out autolamella if you are interested: https://github.com/DeMarcoLab/autolamella.
Thanks again


Mixed values on Y-Axis

I have implemented a chart with numerical values on the Y Axis:

The areas where the line appears to go below the X Axis is because I have added another value "5+" in there. This is because we don't capture the information if it is out of a certain range. The issue is that I need it to be above the 4, not pulling the line below the graph.
This seems to highlight a need for potentially mixed values in a single axis and a way of specifying their order.

Hi there. Can you tell more about the use-case? What do you mean with added another value 5+ ? Did you tried to use "holes in data" as shown on the examples, the low and high configuration and other things? This chart looks a bit broken to me.

Hi there, could probably do with explaining myself a little better!
If you imagine, for my labels I have a bunch of dates, as can be seen. Then in my dataset, I have a set of values for each date. Those values can be 1,2,3,4 or '?'. If I attempt to add '?' as a value, the behaviour above happens. So I tried replacing '?' with '5+' thinking that the number at the start would help the API order this naturally. Alas, that did not work either.
If you create a graph with these labels:  labels: ['Mon', 'Tues', 'Weds', 'Thurs', 'Fri', 'Sat', 'Sun']
And this series:
series: [
[1,2,3,4,'?',3,2]
]
You'll be able to replicate. I tried it on JSFiddle but you guys don't host your library on a CDN and Github doesn't allow JS fiddle to treat the file like an external resource. But I expected the '?' to be included in the Y-Axis. Instead, it seems to not be included and where the value is a '?', it does the above mentioned behaviour. Has that made things any clearer?

Chartist is hosted on most major CDN's. We propose the super awesome jsDeliver CDN http://gionkunz.github.io/chartist-js/getting-started.html#use-a-cdn-alternatively
I know this is not entiertly clear when looking at the documentation in the current state, but the series arrays need to consist of numbers only of values that can be converted to a number. '? will be coerced to NaN and you should actually even see errors on your console as Chartist doesn't like NaN :smile:
Isn't the holes in data feature exactly what you're looking for? Just use undefined for those values you're missing. The chart will break into segments like in this example: http://gionkunz.github.io/chartist-js/examples.html#example-line-data-holes

Perfect! Thanks I've got a JSFiddle for you: http://jsfiddle.net/vs8g0odo/
The reason I don't want to just ignore the '?' values is because they actually mean the number is not 1,2,3 or 4. Hence the '5+' instead. They're relevant because the numbers represent the finishing position in a race. Just because we didn't record their finishing position, doesn't mean they didn't run in the race, hence the need to include the data.

Then what about making them 5 and limit the range to max 5 by using the option high: 5 ?

Sorry high: 4

Well then the implication would be that they finished in 5th, which isn't true. They finished in at least 5th. Hence the "5+". I'm aware this is a very specific use case and part of me is thinking about forking the repo and just doing it myself, but I just wanted to see if you guys could offer any insight.

Actually if you cut off the chart at 4 with the high: 4 option, the user will not know if its 5, 6, 20 or 100.
The things I really don't get yet is how would you solve this with pen and paper? :-) Can you tell me how you'd like to present this?

That's how I'd solve it from all what I know from your usecase: http://jsbin.com/votipo/edit?html,js,output

Dude that's exactly, precisely what I had in mind. Cheers!


Add get_job_assets method

Add support for retrieving the list of assets associated with a job.

sorry I let thjis sit for so long.... merging now :+1:


⚠ Bofrost.es has degraded performance

In 78fb4ed, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1455 ms

Resolved: Bofrost.es performance has improved in 88445ba after 6 minutes.


Adding .env, and other changes

Hello guys.
Nice work. A very-complete set of docker images.
Inspecting a bit, I think/propose this PR. A short explanation about changes:

File "docker-compose.yml" renamed to "docker-compose.yml.default": Includes configurable variables from a .env file, where it should be easier and cleaner to change/manage several values. This ".env" file is created by copying the .env.default, and added to the .gitignore to allow end-user having its own, without losing the upstream version.

Makefile: Includes several commands by default. Same as previous point: A default one (Makefile) is provided with just creating another to place there user customizations.

.env.default: File where setting may be configured/stored.

Thanks!

I'm all for using .env file but the way you moved out only image name and a version is kinda weird. You either move repo + image so people understand that it's actually an image or you just move version and stability tag. I mean, do you really need MARIADB_IMAGE=mariadb:10.2-3.0.2 when you can have MARIADB_IMAGE=10.2-3.0.2, I would probably even separate them to:

MARIADB_VER=10.2
# MARIADB_VER=10.1
MARIADB_TAG=3.0.2

so in compose file you'll have:
mariadb
    image: wodby/mariadb:${MARIADB_VER}-${MARIADB_TAG}

We should not mix docker4drupal's Makefile and boilerplate for aliases. I suggest we keep the original Makefile and add boilerplate.mk for aliases (included in Makefile) that people will either rename to Makefile or include. No need to have default_ prefix for targets, it should be short and simple by default. The filename kinda suggests you can change things. Also, it's weird that init has down, do we really need this target?

What's the point of having front network? Why is it called front?

I don't think renaming to docker-compose.yml.default makes a lot of sense. It's not like people will use two compose files. And with .env file it will change very rarely.

Hello @csandanov. Thanks for suggestions. Some comments about your points:

I was thinking something like that. But then I saw (when doing it for PHP) that pattern not always the same. For example: For MariaDB works "wodby/mariadb:${MARIADB_IMAGE}"  or as you say, "wodby/mariadb:${MARIADB_VER}-${MARIADB_TAG}". But for PHP thinks are different, because we have as options: "drupal:VERSION" or "drupal-php:VERSION". So, in docker-compose.yml we should have an extra variable to know if vanilla Drupal, or just PHP for Drupal. At the end, it will introduce more variables into the .env. Then, if launched something like "wodby/mariadb-drupal:VERSION" at same time with "mariadb:VERSION", will make it more difficult to manage. That's the reaso I proposed to just keep it into a single variable :)

The "Makefile.override" (copied on the fly from Makefile.override.default) version provided is for allowing user to keep current target rules or edit them (or even add more) as needed without "changing" the upstream version. This allow, for example, a "git pull" for updates without loosing previous custom changes, witch will allow to inspect/compare new version with current one. Same applies to the docker-compose.yml.default: This is not used. The "real one" used is "docker-compose.yml", which is copied/created on the fly. But you're right. Maybe naming needs improvement. I'm not sure to understand what you mean with add a boilerplate.mk and include it or rename it. What I'ld like to propose is keeping the default Makefile allowing "secure pulls" (not overriding user changes).

PS: Sure. You're right with removing "default_" from target rules.

For example, at this PR by executing "make default_info" will show us assigned IPs, which maybe useful for quick-testing and/or easy debugging. Also, you watch the project into the web browser by just going to the IP given by that network.

It will be the "default-base file" that default "docker-compose.yml" will be copied from. Only this last will be used (or if another different added to the .env, it is possible, too). But this point is related to the previous 1: Keep changes from user and upstream.

For example, at this PR by executing "make default_info" will show us assigned IPs, which maybe useful for quick-testing and/or easy debugging. Also, you watch the project into the web browser by just going to the IP given by that network.

This "maybe useful" doesn't sound convincing to complicate things. Perhaps if you give me an example use case when it's super useful I will change my mind. Don't forget we have traefik to access everything you need in a browser. I highly appreciate your efforts but we should keep things as simple as possible, and we shouldn't cover specific cases, folks can always customize everything to their needs.

Thank you for your input, your suggestion have been accepted in most part, please see https://github.com/wodby/docker4drupal/releases/tag/5.0.0 and referenced commits.


Updating to 1.4.0 from 1.3.0 fails with with Unity 2020.1.17

Describe the bug
Updated my project from 1.3.0 to 1.4.0 with Unity 2020.1.17 and not all of my sources are compiled
To Reproduce
Can't really
Expected behavior
All sources compile
Screenshots

Environment (please complete the following information):

Version 1.4.0
Platform: Editor/Windows
Unity version: 2020.1.17
Build options: .Net 4.x

Additional context
The problem might come from the fact that we are using external .cs files added as unity packages. We have several projects included like:
[package.json]
"bimvisualizer.shared": "file:../../BimVisualizer.Shared",

These point to folders with .net standard 2.0 projects (which are using C#9.0) (also the bin and obj folders are renamed to .bin and .obj so unity will not try to use the dlls inside them). The errors seem to only come from one project where c# 9 and 8 are used, and not from the unity project.
Should I include another folder for assemblies in the settings?

@KuraiAndras
The asmdef files under the Packages directory are excluded by default.
(Prefix ! means "should be excluded.")

Solution 1:

Project Settings > C# Compiler > Compiler > Included Assemblies
Set Packages/your_package_name/ instead of !Packages/

e.g. Packages/com.bim.visualizer.shared/

Solution 2:

Select the asmdef file under the Packages directory
Turn on Enable C# Compiler Settings in inspector
Choose the compiler package and the language version
Click Apply

@KuraiAndras
The asmdef files under the Packages directory are excluded by default.
(Prefix ! means "should be excluded.")

Solution 1:

Project Settings > C# Compiler > Compiler > Included Assemblies
Set Packages/your_package_name/ instead of !Packages/

e.g. Packages/com.bim.visualizer.shared/

Solution 2:

Select the asmdef file under the Packages directory
Turn on Enable C# Compiler Settings in inspector
Choose the compiler package and the language version
Click Apply

To be precise, we have multiple projects, and they are not located in the packages folder. The structure is like:
bimvisualizer.shared
    |   bimvisualizer.shared.asmdef
    |   package.json
bimvisualizer.application
    |   bimvisualizer.application.asmdef
    |   package.json
*etc
bimvisualizer.unity
    |   Packages
        | manifest.json
        | package-lock.json
    |   Assets
        |   multiple asmdefs, assets, etc

In this context what does the Packages folder refer to? [UnityProject]/Packages or [UnityProject]/Library/PackageCache or just whatever folder the UPM resolves when reading the mainfest?

To be precise, we have multiple projects, and they are not located in the packages folder. The structure is like:
bimvisualizer.shared
    |   bimvisualizer.shared.asmdef
    |   package.json
bimvisualizer.application
    |   bimvisualizer.application.asmdef
    |   package.json
*etc
bimvisualizer.unity
    |   Packages
        | manifest.json
        | package-lock.json
    |   Assets
        |   multiple asmdefs, assets, etc

In this context what does the Packages folder refer to? [UnityProject]/Packages or [UnityProject]/Library/PackageCache or just whatever folder the UPM resolves when reading the mainfest?

@KuraiAndras
Check name field in bimvisualizer.shared/package.json and bimvisualizer.application/package.json.
For example:
[bimvisualizer.shared/package.json]
{
  "name": "com.bim.visualizer.shared",
  "version": "1.0.0",
  ...
}

and
[bimvisualizer.application/package.json]
{
  "name": "com.bim.visualizer.application",
  "version": "1.0.0",
  ...
}

Remove !Packages/
Add Packages/com.bim.visualizer.shared
Add Packages/com.bim.visualizer.application

Also why is the Packages folder excluded by default?

This is to prevent unintended compiler changes and to minimize the impact of compiler changes.
Packages and store assets should (essentially) be compiled with the default compiler.

@KuraiAndras
Check name field in bimvisualizer.shared/package.json and bimvisualizer.application/package.json.
For example:
[bimvisualizer.shared/package.json]
{
  "name": "com.bim.visualizer.shared",
  "version": "1.0.0",
  ...
}

and
[bimvisualizer.application/package.json]
{
  "name": "com.bim.visualizer.application",
  "version": "1.0.0",
  ...
}

Remove !Packages/
Add Packages/com.bim.visualizer.shared
Add Packages/com.bim.visualizer.application

Also why is the Packages folder excluded by default?

This is to prevent unintended compiler changes and to minimize the impact of compiler changes.
Packages and store assets should (essentially) be compiled with the default compiler.

I'll try this and get back to you soon

I'll try this and get back to you soon

Yupp, the project wors, after including the assemblies like : Packages/package.name.etc .
You can close this now if you want.
Also could you provide some documentation on this in the readme?

Yupp, the project wors, after including the assemblies like : Packages/package.name.etc .
You can close this now if you want.
Also could you provide some documentation on this in the readme?

👍

Also could you provide some documentation on this in the readme?

Sure.

👍

Also could you provide some documentation on this in the readme?

Sure.


After add waypoint with .and, it stop draw polyline after 1st waypoint in route

Could you please provide the latitude/longitude for the direction that I can reproduce your problem?

Facing same issue. Its not drawing polyline after first way point.

From lat/lng: (26.5507785,80.481009)
Ways Point lat/lng: (26.552678,80.479401),lat/lng: (26.536169,80.483978)
To lat/lng: (26.544338,80.4904724)

@mishragashish  I tested on sample project with your location. It's work fine
val start = LatLng(26.5507785, 80.481009)
val waypoint1 = LatLng(26.552678,80.479401)
val waypoint2 = LatLng(26.536169,80.483978)
val to = LatLng(26.544338,80.4904724)
GoogleDirection.withServerKey(serverKey)
    .from(start)
    .and(waypoint1)
    .and(waypoint2)
    .to(to)
    .transportMode(TransportMode.DRIVING)
    .execute(
        onDirectionSuccess = { direction -> onDirectionSuccess(direction) },
        onDirectionFailure = { t -> onDirectionFailure(t) }
    )

Please tell me more if there's incorrect in my test.

I close this issue because there's no response for a while. Feel free to reopen it if you still have an issue about this.


About Training

I ran this code on my lab computer .
but got the error like following:
UserWarning:
There is an imbalance between your GPUs. You may want to exclude GPU 4 which
has less than 75% of the memory or cores of GPU 0. You can do so by setting
the device_ids argument to DataParallel, or by setting the CUDA_VISIBLE_DEVICES
environment variable.
warnings.warn(imbalance_warn.format(device_ids[min_pos], device_ids[max_pos]))
terminate called after throwing an instance of 'std::runtime_error'
what():  NCCL Error 1: unhandled cuda error
I am very upset and confused.
Anyone can help me ?

Hi, @xianxuan-z, I am not very sure. But it seems it complains that your GPU 4 might be a different model and has less memory, e.g., your GPU 0-2 could be Titan RTX, while your GPU 4 is 1080Ti.
My random guess is these two different GPU models might require different versions of NVIDIA driver or cuda version,  which leads to the error.
One way might be you run CUDA_VISIBLE_DEVICES=0,1,2.
As this is not relevant to the repo, I'll close it

Thank you so much! I have solved this problem.
1.torch version changed from 1.7.0 to 1.6.0
2. CUDA_VISIBLE_DEVICES="0,1,2,3" is set
The reason I changed the version was because I saw someone saying that when using a higher version of torch, the same problem as mine occurred.
在2021-12-13 02:39:35，Yonglong @.***写道：
Hi, @xianxuan-z, I am not very sure. But it seems it complains that your GPU 4 might be a different model and has less memory, e.g., your GPU 0-2 could be Titan RTX, while your GPU 4 is 1080Ti.
My random guess is these two different GPU models might require different versions of NVIDIA driver or cuda version, which leads to the error.
One way might be you run CUDA_VISIBLE_DEVICES=0,1,2.
As this is not relevant to the repo, I'll close it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Add configurable decimals rounding of display value and upgrade coinmarketcap to 5.0.3 (#14437)

Description:
Upgraded coinmarketcap to 5.0.3 which supports the new public v2 API, with that comes a few changes to how the API handles input. It is no longer possible to query a ticker by the name of the currency (i.e. bitcoin). The API instead uses IDs (integers) for these tickers (i.e. bitcoin -> 1).
I also made the rounding of decimals configurable, because it made no sense to round to 2 decimals if display_currency was set to BTC, some values showed up as 0.00.
Related issue (if applicable): fixes #14437
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#5422
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: coinmarketcap
    currency_id: 1027
    display_currency: 'EUR'
    display_currency_decimals: 3

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox.

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

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

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hi @kotlarz,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Breaking change:
The coinmarketcap was updated to supports the new public v2 API. It is no longer possible to query a ticker by the name of the currency (e. g., bitcoin) instead IDs (integers) for these tickers (e. g., 1 for bitcoins).

Although this is not ideal(IMHO), at the very least we should provide a link to the list of ID->Currency rather than the docs which say something different.
https://api.coinmarketcap.com/v2/listings/