screenshot in different language is not working

trying to take screenshot of https://www.hindi100.com/

how do I fix this?, running my handler in aws lambda.
Any help will be great
const CDP = require('chrome-remote-interface');

function timeout(seconds){
  let p = new Promise((resolve) => {
    setTimeout(resolve, seconds*1000);
  });
  return p;
}

function screenShot(url, x, y, w, h, format) {
  const scalef = 4;
  format = format || 'png';
  let p = new Promise((resolve, reject) => {
    try {
      CDP(async function(client) {
        try {
          const {DOM, Emulation, Network, Page, Runtime} = client;
          await Page.enable();
          await DOM.enable();
          await Network.enable();

          const deviceMetrics = {
            width: 1920,
            height: 1080,
            deviceScaleFactor: 1,
            mobile: false,
            fitWindow: false
          };

          await Emulation.setDeviceMetricsOverride(deviceMetrics);
          await Page.navigate({url});
          await Page.loadEventFired();
          await timeout(1);

          const { data } = await Page.captureScreenshot({
            format,
            fromSurface: true,
            quality: 1,
            clip: {
              width: +w,
              height: +h,
              x: (+x)*scalef,
              y: +y,
              scale: 1
            }
          });
          resolve(data);
          client.close();
        } catch(err) {
          client.close();
          reject(err);
        }
      });
    } catch(err) {
      reject(err);
    }
  });

  return p;
}

export async function screenshot(event, context, callback) {
  let body = new Buffer(event.body, 'base64').toString();
  console.log(body);
  let { url, x, y, w, h, format } = JSON.parse(body);
  let data = '';
  try {
    data = await screenShot(url, x, y, w, h, format);
  } catch(err) {
    console.log('main error', err);
  }
  return callback(null, {
    statusCode: 200,
    body: data,
    isBase64Encoded: true,
    headers: {
      'Content-Type': 'image/png'
    }
  });
}

@typedefstructer I guess you need to put fonts with supported language into ~/.fonts/, like it's done in:
https://github.com/pahud/lambda-layer-headless-chromium/blob/b74492960cf826ba270980642915405983c7878b/build.sh#L9

thanks, will try to do this..

trying to take screenshot of https://www.hindi100.com/

While I'm producing PDFs, I've tried adding text from that site to my documents:

Then I've added (from google search) first available font mangal.ttf into ~/.fonts/ it worked:

nice!, will close this issue, but any reason why all these fonts are not added by default.?


Add search APIs

This PR adds new methods for searching for enheter and underenheter by any
parameter supported by the API:

Client.search_enhet()
Client.search_underenhet()

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (029dfd4) 92.95% compared to head (61f67d5) 95.88%.

Files
Patch %
Lines

src/brreg/enhetsregisteret/_client.py
94.44%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   92.95%   95.88%   +2.92%
==========================================
  Files           5        8       +3
  Lines         142      267     +125
  Branches        8       14       +6
==========================================
+ Hits          132      256     +124
- Misses          7        8       +1
  Partials        3        3

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


General questions about Example #2

Example #2 deals with the use of object-based datatable rows in memory. I have a couple of questions about this example that I was unable to answer myself:

Does adding, editing and deleting rows from the dataSet variable also update this variable?
When adding and editing the console.log shows the new object. Can this be disabled?
Similar to the situation above, the console.log writes a message, e.g., Missing AJAX configuration for INSERT. Does this mean we have to provide a function for INSERT, DELETE and UPDATE to update the dataSet variable?

Some background information on how I would like to use the editor. I got a dataset in a variable that I use to plot using a JS plotting library. I want to use this dataset into datatables in combination with the alt-editor. Since the dataset is in memory, I need the editor to update the contents of the variable dataset on all actions.
I figure that I can use the AJAX configuration also for the redrawing of the chart after an INSERT, DELETE or UPDATE action. But before that I need to know if the variable is updated by the plugin or I have to write code to do this myself.

Hello @schonhose . I'll try to answer everything.

* Does adding, editing and deleting rows from the `dataSet` variable also update this variable?

Not sure of this, I think not. That variable is used only for initialization, it is not linked to the Datatable, AFAIK. Don't rely on this.

When adding and editing the console.log shows the new object. Can this be disabled?

No it can't. It's some debug code that I forgot somewhere in the code. You can safely find and remove every "console.log" line in the script.

Similar to the situation above, the console.log writes a message, e.g., Missing AJAX configuration for INSERT. Does this mean we have to provide a function for INSERT, DELETE and UPDATE to update the dataSet variable?

Not exactly. You have to provide these functions (callbacks) if you want to update you backed - whatever it is. Usually you want call some a server, via AJAX (example n. 3). In your case, you can use these callbacks to update dataSet variable..

Thanks @luca-vercelli!
One more question: do you have any suggestion on how to find the object in the dataSet variable that needs to be updated?

Uhm, I never had to do that...
Maybe do you have some (readonly) id column in your dataSet, and you can search for it?

@schonhose  please let us know if you solved that, and if this issue can be closed.

This can be closed as far as I am concerned.


feat/event-loop

feat/event-loop

https://github.com/Road-of-CODEr/interview/pull/19 https://github.com/Road-of-CODEr/interview/pull/18


Add FERRETDB_USERNAME stub

Description

Closes #4245.
Readiness checklist

[ ] I added/updated unit tests (and they pass).
[ ] I added/updated integration/compatibility tests (and they pass).
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

I think FERRETDB_SETUP_USERNAME is already handled by FerretDB via the usual Kong CLI

@AlekSi What's the goal of this issue then? It's possible to set setup username in docker already.

We should check that it works

It works, I've already checked that while working on #4246. Should I write unit test for that?

No, I don't think there is a need for it


Add link from review app to Frontend Sass docs

Partly addresses #1892.
The front page of the review app contains a link ('Sass documentation') to http://govuk-frontend-review.herokuapp.com/docs/.
Seeing the same Sass content in 2 places might be confusing for users.
This PR replaces the current link with one to the Sass content on our Frontend docs site.

I'm not totally sure of the value of this. Whilst de-duplicating documentation makes sense—especially as both are generated from the same source—the two are potentially distinct enough in function that they make sense being separated.
The SassDoc docs are a little easier to use by virtue of having navigation and search that is specific to Sass code, as well as showing extra information that isn't currently in the Frontend Docs version. Conversely the Docs have usage examples, which SassDoc doesn't.
Given the change doesn't stop SassDocs being created (it just removes the link to it) there isn't any significant reduction in 'effort' happening. If we wanted to consistently direct people to the Frontend Docs over SassDocs, we should probably ensure the two have feature parity before switching SassDoc off entirely.

@36degrees Are you still planning to take a look at this, or should we put this back into the Backlog?


ActiveJob integration example

Can anyone provide example of integration activejob in reflex flow? Something as simple as possible. e.g. re-render template in job?

Hi @biow0lf,
I hope that you got this sorted out. I just posted what I hope is a definitive example of this pattern on an earlier issue.
Polymorphic comments controller example
If this issue has been resolved, would you mind closing it?

You'd typically want to use CableReady for use-cases not directly related to a reactive UI. You can learn more about CableReady here. https://cableready.stimulusreflex.com/setup

@biow0lf and anyone looking in the future: https://blog.taskbill.io/2020/06/28/cableready-is-magic/


React & Redux group project - Space Travelers' Hub

Redux: Fetch data and update the Redux store
Render UI: lists
Redux: Write actions and reducers for booking rockets/dragons and joining missions
Redux: Write actions and reducers for canceling rockets/dragons and leaving missions
Render UI: conditional components rendering
Render UI: My Profile section

I am very happy that your project has been approved and I wish you the best of luck with the rest of your projects!


Update last.fm in sites.json

last.fm now provides a simple "Close account" button in its profile settings page.

We had changed it to impossible on #12, because you can reactivate it at any time, can you confirm this is not the case anymore?

Hm, well that's not very useful... Their site says nothing about it until you actually disable your account. From what I could see, they don't have any limit on how long they retain that data... I'll change my PR to just update the notes to reflect that if you're ok with that.

Ok!

There we go, this should be good to go after CI checks.


Unable to install airflow-provider-kafka package on Azure Cloud Airflow VM instance

Hello,
We have Airflow version(2.5.1) running in one of the Azure VMs, unable to install the airflow-provider-kafka package. I tried to install it on this python version  3.7.5.   Seeing Failed building wheel for confluent-kafka

Anybody please suggest on this.
@AlexandrChikur

Looks like librdkafka isn't installed on that machine. Check with the confluent installation guides for how to complete this.


Can you clarify what role the ColBERT server plays?

It's not obvious to me from the documentation -- is a ColBERT server required to use DSPy? I think the answer is yes, but looking for a definitive answer. No matter the answer I think it can be made more clear how ColBERT fits into the bigger picture system design.
And more generally speaking I think the documentation can be improved to describe the bigger picture concepts more clearly. I definitely can buy the major premises of the library but I'm having trouble understanding the lifecycle of a query and index and how that depends on an LLM, vector database, and ColBERT. I would like to see a simpler description of what is happening behind the scenes, e.g. a made up explanation would be something like

during training, question-answer pairs are sampled from the dataset (how?) and prompts are updated (how?) automatically to maximize some cost function (how?). during inference, a question is sent to LLM for embedding and then compared against a vector db (which vector DB, is ColBERT involved?) and a ranking algorithm is applied to do x,y,z...)

I'd recommend creating a higher level sequence (system) diagram. Happy to help make this happen, I just need to figure out the answers myself first :)

Hey @jhyearsley !
Great question. This does seem confusing right now.
ColBERT is just a retriever. (It happens to be one that we built and one that we like, but that's all.)
This server hosts a search index over Wikipedia. If you need to search over Wikipedia, that's pretty handy. If you don't need to search over Wikipedia, you don't need it at all.

The simplest explanation I wrote of DSPy was on HackerNews.
It said this:
You give DSPy (1) your free-form code with declarative calls to LMs, (2) a few inputs [labels optional], and (3) some validation metric [e.g., sanity checks].
It simulates your code on the inputs. When there's an LM call, it will make one or more simple zero-shot calls that respect your declarative signature. Think of this like a more general form of "function calling" if you will. It's just trying out things to see what passes your validation logic, but it's a highly-constrained search process.
The constraints enforced by the signature (per LM call) and the validation metric allow the compiler [with some metaprogramming tricks] to gather "good" and "bad" examples of execution for every step in which your code calls an LM. Even if you have no labels for it, because you're just exploring different pipelines. (Who has time to label each step?)
For now, we throw away the bad examples. The good examples become potential demonstrations. The compiler can now do an optimization process to find the best combination of these automatically bootstrapped demonstrations in the prompts. Maybe the best on average, maybe (in principle) the predicted best for a specific input. There's no magic here, it's just optimizing your metric.
The same bootstrapping logic lends itself (with more internal metaprogramming tricks, which you don't need to worry about) to finetuning models for your LM calls, instead of prompting.
In practice, this works really well because even tiny LMs can do powerful things when they see a few well-selected examples.

Does this sound useful?

@okhat yes, this high level description is super helpful! thanks for quick response, much appreciated. The part that is still a bit hazy to me is how DSPy is interacting with the LLM and the Vector DB during the bootstrapping phase. I understand my chunked data would be stored in a Vector DB, and that an LLM can be used to augment retrieval from that Vector DB but I'm having a hard time visualizing how and when DSPy is integrating with the LLM and Vector DB (e.g during bootstrapping is DSPy reading / writing data to the Vector DB?)
Another point of clarification on ColBERT as a "retriever" -- my impression from skimming through the ColBERTv2 paper is that there is some additional secret sauce re: single-vector representations vs multi-vector representations. Does that mean if I want the sauce I need to either use ColBERT or reimplement the methods with my own Vector DB? Or is the sauce built into DSPy no matter what?
I saw another issue which mentioned adding Pinecone and I'd like to add MongoDB Atlas Vector Search as an integration to the library (since this is the Vector DB I will be using while evaluating the tool).

@jhyearsley : you can look at this example to see how to make this work with ColBERT and with our own data: https://github.com/stanfordnlp/dspy/blob/main/pyserini.ipynb

@okhat yes, this high level description is super helpful! thanks for quick response, much appreciated. The part that is still a bit hazy to me is how DSPy is interacting with the LLM and the Vector DB during the bootstrapping phase. I understand my chunked data would be stored in a Vector DB, and that an LLM can be used to augment retrieval from that Vector DB but I'm having a hard time visualizing how and when DSPy is integrating with the LLM and Vector DB (e.g during bootstrapping is DSPy reading / writing data to the Vector DB?)
Another point of clarification on ColBERT as a "retriever" -- my impression from skimming through the ColBERTv2 paper is that there is some additional secret sauce re: single-vector representations vs multi-vector representations. Does that mean if I want the sauce I need to either use ColBERT or reimplement the methods with my own Vector DB? Or is the sauce built into DSPy no matter what?
I saw another issue which mentioned adding Pinecone and I'd like to add MongoDB Atlas Vector Search as an integration to the library (since this is the Vector DB I will be using while evaluating the tool).

I think many DSP.Retriever integrations are getting added, I saw one for PineCone as well for which there was a merge request.

during bootstrapping is DSPy reading / writing data to the Vector DB

We never write to a vector DB internally. You can do that to add documents to search, if you want.
We will only search things in the vector DB / retrieval index. But a lot of read/writes happens to the cache on disk and to some internal representations in the compiler (in memory).


python does not create a config file

all parts installed, firing up the python script, it welcomes me and wants me to hit Enter in order to start creating the config-file - but then nothing happens. it idles and I am not asked for my zotero credentials or any info which I can see the script should ask for. Any ideas?

This is strange, since that's just a simple input() function waiting for confirmation (that's also why it's important to hit Enter and not just any key). Could you specify the python version and operating system you are using?

sure: I am on a Win10 computer with Git Bash, running rmapi by ddvk https://github.com/ddvk/rmapi and python 3.12. I did clone your repository, pull the requirements, cd'ed to the installation, fired up python and got the "welcome message" (Couldn't find....), hit "Enter"....and nothing

I have tried to reproduce the issue on a Windows 10 VM which I configured with Git Bash (both MinTTY terminal and default Windows CMD) and latest Python 3.12 package for Windows (3.12.7), but I wasn't able to. As you can see from the screenshot attached, the config questions come up as expected. Please try to check if there are any updates available for Windows, Git on Windows and Python, maybe it comes down to a bug that has been fixed upstream.

Thx. Setting it up manually helped - at least for one step...now I don't know how to troubleshoot the following error when trying to push (after all settings set correctly, rmapi working, etc.)

Any help appreciated

The error itself was caused by a logic issue, where the variable could not be defined in certain cases. I just pushed a commit that should fix it. But this bug would only appear if the script fails to actually get the file from Zotero, so it probably won't help you get the file on the tablet. Precisly, the step failing is the download from Zotero, Please double check the library ID, API key and whether you are using WebDAV or Zotero's own cloud storage for syncing.

Thx. This now worked after me also correcting my config file (had forgotten to set ' around True...)


Failing blockchain tests now emit a state diff

Intended to address #1562
- Instead of failing with "these block hashes do not match", blockchain
  tests now fail with "this part of the state tree does not match".

- The tests were inefficient, they first checked that the state hash
  matched and then checked that the states matched. It's unlikely the
  states will ever not match!

- Some validations needed to be deferred until after state-tree
  comparison, or else they would fail the test early with an unhelpful
  message.

- minor: tools.fixtures.helpers.chain_vm_configuration didn't support
  Constantinople

What was wrong?
Failures used to look like this:
E       eth_utils.exceptions.ValidationError: Mismatch between block and mined block on 3 fields:
E        - header.state_root  :
E           (actual)  : b"R\xafB',,\xd3\xcb\x8b#W\x98)wL\xc4fLm\x90m\x08\xbb\xed\xc0\x03\x8e\x7f\xfcO:X"
E           (expected): b"pv\xa0\x01\xd5\xc2\xab]\x94\xdbj\xcac\x17\x84\x1fW2QZ\xf7\x1f>:\x00\xb3\x8f\xe2'Sy\xfc"
E        - header.receipt_root:
E           (actual)  : b'\n\x106\x18\xe1\xe1)0\xf1%\xef2\x94\xd9\x0fQ\xed\x12\x1d+\x19\xae\xd7\t\xcf\x92\x8c4]\xd9zy'
E           (expected): b'\x0e\xf4\xbaL\x0be\xce\xcau\xb1\xb6~i\xf5\xaf\xb1\xabd\xe7_~\x96!`m\xe5\x17M\xce\x9eD\x05'
E        - header.gas_used    :
E           (actual)  : 128058
E           (expected): 113058

And now they look like this:
E           AssertionError: State DB did not match expected state on 3 values:
E           0x1000000000000000000000000000000000000000(storage[1]) | Actual: 859287 | Expected: 849245
E            - 0x2adc25665018aa1fe0e6bc666dac8fc2697ff9ba(balance) | Actual:<PHONE_NUMBER>000160716 | Expected:<PHONE_NUMBER>000170758 | Delta: 10042
E            - 0xa94f5374fce5edbc8e2a8697c15331677e6ebf0b(balance) | Actual:<PHONE_NUMBER>99739284 | Expected:<PHONE_NUMBER>99729242 | Delta: -10042

How was it fixed?
I'm running with all the hard work @veox did, this is an attempt to put it into a more-committable form.
Cute Animal Picture

These test failures happened because Slave 'gwXX' crashed while running, when I run each of these failing tests locally they work, which makes me thing it might be something transient, maybe a timeout? I don't have permissions to rerun the build though.

It may mean the tests need to be split into smaller chunks.

Even if it's not the cause of the crash, it would be best to split the tests.
Note how we had split the state tests into 5 parts, in order to get some balance on the length of the test jobs. With this PR, all 5 of those, and the blockchain tests, are combined into a single job.
See how they were split before here:
https://github.com/ethereum/py-evm/blob/f6162987c03ae6f6c52724061b2ad80710faf694/tox.ini#L38-L45
(which may or may not help)

it looks like some of the SLOWEST_TESTS are so slow they can't even be run in circleci?

Yeah, or maybe too memory-intensive? Haven't had a chance to dig in.

Don't know what your development process looks like, I think what I'll do is turn the BlockchainTests/GeneralStateTests off again, and enable them in a different PR so this one can stay focused?

Yeah, that sounds right. The more focused the better. Probably just rebase away the state test changes here, and move those to another PR.

I think the next step here is to copy over some of the tools state tests have:

Yup, all three LGTM.

Comments on #1562 would be appreciated! I think it would make the test a lot easier to debug if state tests were dropped in favor of their BlockchainTest equivalents.

Which open questions on 1562 are you looking for feedback on? On my quick scan, I didn't see any glaring unsolved problems.

it looks like some of the SLOWEST_TESTS are so slow they can't even be run in circleci?

Yeah, or maybe too memory-intensive? Haven't had a chance to dig in.

Anecdotal: a local test run (still on-going) is currently consuming 3 GiB.

Accidental close :cry:

Which open questions on 1562 are you looking for feedback on? On my quick scan, I didn't see any glaring unsolved problems.

The one I'm concerned about is whether it's a good idea to drop the state tests and instead use the one embedded within the blockchain tests

The tests are all passing but:

GitHub rate limited us while adding a commit status of success: You have triggered an abuse detection mechanism. Please wait a few minutes before you try again.

While you're at it, could you also reverse the calculation on this line?.. (I've mentioned this in passing elsewhere.)
https://github.com/ethereum/py-evm/pull/1573/files#diff-38aa0bc8e408266959b43002a08f0cd4R81
For example, as things stand, if a fixture-expected value is 1000, and the actual produced by py-evm is 500, the printed message will show a delta of 500 - that is, a net positive, whereas what truly happened is py-evm having a value that's too low. This is confusing: say, a miner's account balance showing "Delta: 500" makes one think that the miner received 500 wei more than expected (which is not true).
If not, I can open a separate PR.

 Please wait a few minutes before you try again.

Yeah, that issue is a recent-ish, known, and annoying.

Blame Microsoft.

While you're at it, could you also reverse the calculation on this line?.. (I've mentioned this in passing elsewhere.)
https://github.com/ethereum/py-evm/pull/1573/files#diff-38aa0bc8e408266959b43002a08f0cd4R81

I was juuust about to merge this, and it would be nice to rebase #1577 on this. Maybe @lithp can sneak the change into #1577

While you're at it, could you also reverse the calculation on this line?
Maybe @lithp can sneak the change into #1577

I didn't want to sneak an unrelated change into #1577 but made a tiny PR for it: https://github.com/ethereum/py-evm/pull/1580


Fetch seems not to be using a proxy

Environment
rkt Version: 1.5.1
appc Version: 0.7.4
Go Version: go1.6.1
Go OS/Arch: linux/amd64
Features: -TPM
--
Linux 4.3.0-trunk-amd64 x86_64
--
PRETTY_NAME="Debian GNU/Linux stretch/sid"
NAME="Debian GNU/Linux"
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
--
systemd 228
+PAM +AUDIT +SELINUX +IMA +APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ -LZ4 +SECCOMP +BLKID +ELFUTILS +KMOD -IDN

What did you do?
# echo $HTTP_PROXY
10.254.xxx.xxx:80
# rkt --insecure-options=all --debug fetch quay.io/appcontainers/mysql:debian

What did you expect to see?
To download the image using the proxy
What did you see instead?
rkt attempted to connect directly to the end host.
image: searching for app image quay.io/appcontainers/mysql
image: remote fetching from URL "https://quay.io/c1/aci/quay.io/appcontainers/mysql/debian/aci/linux/amd64/"
image: fetching image from https://quay.io/c1/aci/quay.io/appcontainers/mysql/debian/aci/linux/amd64/
fetch:
  └─error downloading ACI
    └─Get https://quay.io/c1/aci/quay.io/appcontainers/mysql/debian/aci/linux/amd64/: dial tcp <IP_ADDRESS>:443: getsockopt: connection timed out

More info
Without setting a proxy, rkt stops here:
# rkt fetch quay.io/appcontainers/mysql:debian
image: searching for app image quay.io/appcontainers/mysql
fetch: Get http://quay.io?ac-discovery=1: dial tcp <IP_ADDRESS>:80: i/o timeout

When using HTTP_PROXY the discovery succeeds but image retrieval fails, as shown above.
Also tried HTTPS_PROXY, http_proxy and https_proxy, with a various combinations of URLs, HOST:PORT and Hostnames/IPs. None worked.

Without --insecure-options=all it seems to work

From the logs, HTTP_PROXY is used by the discovery but not for the download itself.
$ git grep ProxyFromEnvironment
Godeps/_workspace/src/github.com/appc/spec/discovery/http.go:           Proxy: http.ProxyFromEnvironment,

It seems that the --insecure-options=all breaks the proxy feature there:
https://github.com/appc/spec/blob/master/discovery/http.go#L69
	ClientInsecureTLS = &http.Client{
		Transport: &tInsecureTLS,
	}

The http.Transport is overwritten without the proxy.

https://github.com/appc/spec/issues/609

There was already similar problem and fix for this was trivial.
Fixed in https://github.com/coreos/rkt/pull/2354

This is still blocked on appc PR, de-milestoned until that goes through.


bash run/train_voc_first.sh

When I run this step bash run/train_voc_first.sh, it appears as shown in the figure. It has been in this state for a day. I don't know how to solve it. Can you help me？Thank you.

It appears not very clear to me where could come from the error, maybe you can do a monitoring of the GPU running state with nvidia-smi to check if the training has already started? If not, maybe it's better to go step by step to spot the issue

It appears not very clear to me where could come from the error, maybe you can do a monitoring of the GPU running state with nvidia-smi to check if the training has already started? If not, maybe it's better to go step by step to spot the issue

I use nvidia-smi to check the running status of GPU. I find that the GPU is not working, but the terminal always displays the status of the above picture, and it does not report any errors. Where should I check for errors? Thank you.

Maybe the first thing to check is wether the code has gone from line-321 to the training loops in line-399.
For a step-by-step debugging, you could use the ipdb libraries. Hope that will help you.

hello, have you fixed it yet? I met the same problem with you, but my nvidia-smi shows that it's working

it remains like this

@YoungXIAO13 @chh6936


fix: missing type definition

What:
Fixes MDXContentProps type missing from types.d.ts

Why:
This issue prevented a basic Next.js setup with mdx-bundler from building. Fixes #75.

How:
Add MDXContentProps to types.d.ts and import it in ./client

Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged

Unfortunately there's an issue with the types. Do you mind looking into that?

@All-contributors please add @vladmoroz for code


Arjun

I added my code for the stats visualization for some basic stats- questions per student and answers per TA.

Codecov Report

Merging #24 into master will decrease coverage by 0.52%.
The diff coverage is 30.43%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   48.97%   48.45%   -0.53%
==========================================
  Files          63       63
  Lines         784      807      +23
==========================================
+ Hits          384      391       +7
- Misses        400      416      +16

Impacted Files
Coverage Δ

api/urls.py
100% <ø> (ø)
:arrow_up:

frontend/urls.py
100% <ø> (ø)
:arrow_up:

stats/urls.py
100% <ø> (ø)
:arrow_up:

stats/views.py
42.22% <30.43%> (-12.33%)
:arrow_down:

Continue to review full report at Codecov.

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


Some bug in Polymer 0.13.0

 Issue by RWander
Originally opened as dart-lang/sdk#20725

Could you please confirm that fix of the bug https://github.com/Polymer/ShadowDOM/commit/864ca98 was included in Polymer.dart 0.13.0? After Pub Update to Polymer.dart I was able to reproduce this bug!

 Comment by jakemac53

Set owner to @jakemac53.
Added Accepted label.

 Comment by jakemac53

This is included in the 0.6.0 version of the web_components package (which polymer has a dependency on). However, the lower bound of the constraint is still at 0.5.0+1. Can you try doing a pub upgrade to get the latest web_components version and see if you are still experiencing the issue?

Added Waiting label.

 Comment by anders-sandholm

Added Area-Pkg, Pkg-Polymer, Triaged labels.

 Comment by RWander

please see also the opened issue https://code.google.com/p/dart/issues/detail?id=20141#c4
I was hoping that this error is corrected in Polymer 0.13.0.

 Comment by RWander

I have updated to web_component 0.6.0. I can still reproduce this issue in the latest FireFox.. This is blocker issue for my project..

 Comment by jakemac53

Added Duplicate label.
Marked as being merged into dart-lang/polymer-dart#245.

 Comment by RWander

Attachment:
firefox_issue.7z (169.24 KB)


[HDFS-461] Revert #990: Include FRAMEWORK_NAME when rendering xml endpoints

Reverts #990 with some commentary on what's going on.
Scheduler is reproducing what the tasks do when rendering the xml connection files locally.

LGTM
Please include the [JIRA-###] in your PR title.


fixed eslint no var, added correct chat sdk version to meeting sample.

chat sdk, fixed URL rewrite issue affecting deploy.
What
Removed not needed eslint fields, updated chatSDK information in package.json, fixed a URL rewrite function to return a string to the push state
Why

fixes minor issues with the meeting sample.

https://skype.visualstudio.com/SPOOL/_workitems/edit/2691295
https://skype.visualstudio.com/SPOOL/_workitems/edit/2691317
How Tested
Ran locally to ensure new bugs weren't introduced.

How is this PR fixing the bug https://skype.visualstudio.com/SPOOL/_workitems/edit/2691317 ?


Abnormally slow performance of VSCode Intellisense

After any change to the source code in a file with mtcute imports Intellisense operations take up to 5-10 sec to perform.
To Reproduce
Try to invoke .call method for TelegramClient object, e.g.:
const tg = new TelegramClient({...});
tg.call({...})

After typing "tg." it takes 5-10 sec to show method list.
Expected behavior
"Go to Definition" and "Autocomplete" operations perform in less than 1 sec.
Logs

Info 492  [17:26:49.904] request:
{
"seq": 99,
"type": "request",
"command": "quickinfo",
"arguments": {
"file": "/Users/.../mtcute/test/index.ts",
"line": 42,
"offset": 27
}
}
Perf 493  [17:26:49.916] 99::quickinfo: elapsed time (in milliseconds) 12.0900

Environment

OS: MacOS 14.4.1
Runtime: Node.js 20
mtcute version: 0.10.1

Additional context
The codesandbox repo below can be opened locally with VSCode.
CodeSandbox

i feel like this is an issue with vscode/tsserver in itself, which doesn't handle large discriminated unions too well.
if i remove the tg.call() call things seem to become better, so it's probably because of that. or the use of the MustEqual<T> helper in the .call().
either way, it's unlikely there's much i can do to improve things performance-wise
i'll keep this issue open for now, things might improve with the upcoming esm codegen rewrite

ok in fact it is related to MustEqual


Update Readme with how to get news api key

It is not documented in the install guide you need a local News API key.
Please update the documentation to include how to obtain this and set in the app.js file.

I submitted a pull request for this issue 😃

Please assign it to me


Mermaid not rendering diagrams as expected

Describe the bug
Mermaid is not rendering diagrams, instead its being rendered as codeline.
To Reproduce

Default Quartz instalation v4.2.3
Plugins ordered like so:

transformers: [
      Plugin.FrontMatter(),
      Plugin.CreatedModifiedDate({
        priority: ["frontmatter", "filesystem"],
      }),
      Plugin.SyntaxHighlighting({
        theme: {
          light: "github-light",
          dark: "github-dark",
        },
        keepBackground: false,
      }),
      Plugin.ObsidianFlavoredMarkdown({ enableInHtmlEmbed: false }),
      Plugin.GitHubFlavoredMarkdown(),
      Plugin.TableOfContents(),
      Plugin.CrawlLinks({ markdownLinkResolution: "shortest" }),
      Plugin.Description(),
      Plugin.HardLineBreaks(),
      Plugin.Latex({ renderEngine: "katex" }),

Markdown (timeline.md) page like so:

```mermaid
timeline
	2002 : LinkedIn
	2004 : Facebook : Google
	2005 : Youtube
	2006 : Twitter
```.

(the dot is there for github to render the code properly)
4. Results:

Expected behavior
Expected to see timeline diagram similar to obisdian:

Screenshots and Source

Github Pages: https://igorgsouza.github.io/guide-to-tellus/História/Timeline
Github Repository: https://github.com/igorgsouza/guide-to-tellus

Desktop (please complete the following information):

Quartz Version: v4.2.3
node Version: v20.14.0
npm version: 10.7.0
OS: Windows
Browser: Brave

Additional context
The issues also happens locally and on Chrome

this renders fine for me, you probably just need to use spaces instead of tabs


Add constant Hbc::Container::Air::INSTALLER_PATHNAME

Because DRY.

Will this make it easier for someone installing scratch to understand or follow?

It won't make any difference in the output they see; the only people who will ever see it are those who make changes to the cask. The point is to remove unnecessary code duplication.

we shouldn't be using interpolation, gsub with regexes, or other language features in Casks either

As much as I love regular expressions, one of these days, I'll sit down and learn Ruby, for the sole purpose of fixing the version string and turning it into a more robust object. Some of those substitutions are horrifically unreadable :grinning:

Some of those substitutions are horrifically unreadable :grinning:

They really are. Wrapping the version string in a class is also on my to-do list. We should probably open an issue for it.

Just noticed https://github.com/caskroom/homebrew-cask/issues/12827 is exactly that

@Amorymeltzer do you have any issues with this being merged?

Well, I think it's a bad idea for usability.  It'd be the first time ruby code is used for something other than a switch, completely obscuring the natural path, which is not, IMO, a great road to go down.
That being said, this is a weird, special case where we already have a built-in workaround, and it's not likely to make a huge difference to anyone's life, so I don't really mind.  I wouldn't say no to a single comment offering some modicum of explanation, but it's not a big deal.

I certainly understand @Amorymeltzer’s reservations, but also think in this case it doesn’t really matter. There’s at least one other cask that will benefit from this, and it actually made me think if instead of this solution, having something like uninstall :adobe-air-app => '/Applications/Scratch 2.app' wouldn’t be both clearer and more useful, but to be honest I don’t think we should care in this case, and merge this as is. Adobe AIR will just get less and less popular over time, and apps made in it will start to die out, so it isn’t worth the time.

Seems like we’re all fine with this, so I’ll go ahead and merge it.


Cooperation task game order and game/trial counts

When we just played it, we had 3 or 4 positive outcome games in a row before getting a negative outcome game.
I think it's probably better to just have it alternate from positive game to negative game to positive game, etc.
Also, can the counts on the top show the number they're out of? So the game would say 1/20, 2/20, etc. and the trial number would be 1/16, 2/16, etc.
Also, can we change "Trial Number" to "Choice Number"?

Alternated the games 34af8be3adc353759030c3bd19f1b8a4c7c458e9
Show total games & choice 01d557654448b1584d2e1c75d052f88ef758d274
"Trial Number" -> "Choice Number" 01d557654448b1584d2e1c75d052f88ef758d274


👋

Hi,
I've been working on some scraping ideas for your sources and wanna talk
If you do, my discord username is sexyredman

Hi, please do not use Github Issue to talk about personal stuffs. If you need to reach me out there is my email in my personal page
https://giammirove.github.io/


feat: support spaces in columns via replacement sign

This could perhaps be an alternative to supporting single quotes for column names with spaces. It follows the same idea as Sharepoint's OData Conventions (https://learn.microsoft.com/en-us/sharepoint/dev/business-apps/power-automate/guidance/working-with-get-items-and-get-files#order-by-query) by expecting _x0020_ instead of spaces. So column [Spaced Column] should be sent like Spaced_x0020_Column.
MS themselves leaning on this sounds like there is no generic solution the OData protocol itself promotes.
EDIT: btw I do realize I didn't yet add docs or extended the example. I first wanted to check back whether this approach is worth pursuing.
Potential workaround for #14

ping @maciejpieprzyk


Configure security sync in vdc helm chart

When deploying our vDC chart, security sync needs to be pointed at ranger endpoint etc

The VDC environment will need significant re-work to adapt to the many changes made in Egeria to better support
metadata integration. Additionally the helm charts used for the lab & a simple base config have evolved to better support
different types of services, exposing of ports, persistent storage etc.
As such specific incremental changes to the current - now old - charts do not really add value.
As such closing for now


[iOS][TextInput] I got a TextInput  fixed at bottom , when I type text , how to make TextInput  not  covered by the keyboard .

http://stackoverflow.com/questions/29313244/how-to-auto-slide-the-window-out-from-behind-keyboard-when-textinput-has-focus/30773343#30773343

Awesome!  thanks, buddy !

No worries, @brentvatne @vjeux this can be closed


”TÂRNĂVENI” is listed in the drop-down menu

Description:
"TÂRNĂVENI" 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 "TÂRNĂVENI" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiul TÂRNĂVENI, MUREȘ".


[release-1.6] Prevent HPA scaling from being overridden on V1 Telemetry when re-installing the control plane

Only set V1 Telemetry spec.replicas if autoscaleEnabled: false and. replicaCount is set
Makes the Deployment consistent with that of policy and istiod and avoids istioctl force scaling in/out the Deployment during an install, which the HorizontalPodAutoscaler then has to correct.
Fixes #28916 (although needs cherrypick to release-1.7 and not sure on process for this)
[ ] Configuration Infrastructure
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @chadlwilson. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

cc @douglas-reid @mandarjog is this OK to merge?

@chadlwilson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.6
891d326918ada81140cfadfc3326647c60b14520
link
/test lint_istio_release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

seems reasonable to me.

Hello @jacob-delgado - do I need to fix the lint failures on unrelated shell scripts? Got the same errors locally; I inferred that the rules had changed on latest tools but not been tidied on this branch.
Seems automator has done some changes on master for at least one of the failed lints.

Possibly should be cancelled in preference for a cherry-pick of #28952 into release-1.6., which I just noticed.

@googlebot I consent

Closing in lieu of https://github.com/istio/istio/pull/29051


Feature/#17 single metadata per donor

Instead of separate biospecimen.json and analysis.json files, create one metadata.json file per donor that has all of the donor metadata.  The upload portion of generate_metadata.py needs to be updated to be aware of this change.

@chkw your readme needs to be updated, the following is incorrect:
python generate_metadata.py -v \
--biospecimenSchema biospecimen_flattened.json \
--analysisSchema analysis_flattened.json \
sample_tsv/sample.tsv

@chkw our wires are crossed here... you interpreted this as "one metadata.json per donor" and I was talking about "one metadata.json per upload".  Let's discuss in person today.  See #17

Including the workflows in a workflow array breaks merge_generated_metadata.py and esquery.py.  You need to update these (after addressing the problem above) before I can merge the branch.

Does this do the file upload and metadata.json uploads in two seperate calls with Beni's tool!?!
INFO:root:First, upload workflow outputs to get file_uuids.
INFO:root:done uploading files
INFO:root:upload took 0:03:54.068985 s for 48 files.
INFO:root:Now, upload metadata json files.

It can't be done this way since the metadata.json and files uploaded need to be done at the same time so everything is logically grouped in the same upload bundle.

Can you walk me through the uploads done and how they are done?    You won't actual know this since you need to do the file upload and metadata.json upload at the same time.  What we could do is include the databundle ID in the workflow metadata.json.
Let's talk about this tomorrow.

Another comment, I still see a separate biospecimen.json:
088bda20-30d2-5944-a5fd-fee9a223f1a7_biospecimen.json

I thought you would merge this?
I think you and I should just go over the metadata files and review this pull request together tomorrow.  It will be faster than leaving comments here.  I think you're close to having this right but I want to make sure we agree on the final format.

It seems that ucsc-upload.sh works in 2 steps (with one java call per step).... The first step is generating a manifest file which is used in the second step for the actual file uploading. The manifest file contains the 'object ids' for each file in the upload. Since we're planning on reimplementing the functionality of ucsc-upload.sh, maybe we could get the file_uuids from there and add the "file_uuid" to metadata.json between manifest generation and file upload.
One problem that I was running into was files that have identical file names seem to get clobbered/ignored by ucsc-upload.sh, even if they are found in different paths. This means that only one "metadata.json" can be uploaded at a time. Actually, I don't know what really happens if you upload a "metadata.json" in a subsequent upload. The upload completes, but does it replace the old one, or do they coexist together? What happens to the assigned object ids?

So the new per-upload UUID directory looks good, the generate_metadata.py script looks good.  But the downstream script will need some tweaks
(env) boconnor@Brians-Mac-Pro:~/Development/gitroot/BD2KGenomics/dcc-storage-schemas$ python merge_generated_metadata.py `for i in output_metadata/*; do echo -n "$i "; done`
FILE! output_metadata/1329abd5-a293-5a20-a32e-a31b7f37799d/gene_expression.rsem
Traceback (most recent call last):
  File "merge_generated_metadata.py", line 190, in <module>
    donor_uuid_arr = find_values('donor_uuid', json_data)
  File "merge_generated_metadata.py", line 171, in find_values
    json.loads(json_repr, object_hook=_decode_dict)  # return value ignored
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 351, in loads
    return cls(encoding=encoding, **kw).decode(s)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded


Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of Svg which is inside StrictMode.

Fix:
https://stackoverflow.com/a/63603903/5676613
I just don't want this amazing library to die because react won't support findDomNode.
The workaround for users who want to use react-native-svg is just to not use strict mode in your index.js.
If you do, just ignore the warning.

I believe that particular issue is not relevant. However, if you encounter any further problems with react-native-svg, please feel free to open a new issue.
Thank you.


adds support for allDay option

Hi!
Here it is a PR for the feature asked in issue #2
Thanks for the plugin :)

Merged manually, thanks!


checkov: 2.0.594 -> 2.0.595

Motivation for this change
Update to latest upstream release 2.0.595
Change log: https://github.com/bridgecrewio/checkov/releases/tag/2.0.595
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146973 run on x86_64-linux 1

1 package built:

checkov

@ofborg eval


Ensure float size is correct when encoding to pg values

Description
See https://github.com/PRQL/prql/pull/3669#issuecomment-1762545507
It looks like we're passing Float8 in the postgres row descriptor, but the column values are actually f32.
I don't have a reproducible sql example yet...

TL;DR: We don't support binary formatting as of now.
Querying using the rust client (query method) assumes the data we get is in binary format, but currently, we only support text format.
let rows = client.query("select 11111111", &[]).await?;
let val: i64 = rows[0].get(0);

The above code works since the string 11111111 has 8 bytes (equivalent to int8). Any other string length would fail.
This would work if the test uses simple_query and parse the string as f64 but that would require implementing a different test client in prql.

Thanks for looking at this! Maybe this issue sfackler/rust-postgres#882?

Thanks for looking at this! Maybe this issue https://github.com/sfackler/rust-postgres/issues/882?

Yeah, this issue is related.


python_by_example_issue1

Thanks, @jstac .
Issue: the original code does not consist of importing functionality, which is mentioned in the context.
Improvements:

Add an importing part;
Add some comments dividing the code into 4 parts and
Modify the following context to match the code.

Thanks for drawing my attention to this @shlff --- you are right that changes are necessary.  I'm very glad you pointed that out.  But I will close this PR and make my own changes.


Zoom by Scroll gesture

Hello,
sorry this is more a question than a bug report:
I'm evaluating OSD for a project similar to https://madmeg.org/delizie/ (which is currently using d3-zoom and a lot of custom code). OSD could be a replacement, with two caveats:

a) the URL tracking part (I've found https://github.com/openseadragon/bookmark-url which I could adapt to a less verbose format)
b) the "springing" methods used to change panning and zoom level: they make me ‘sea-sick’, iow I feel I'm not in control of the movement.

Re: panning, I've managed to have it work about the same by using a quite high value for springStiffness: 2 (or more makes it almost linear—ideally I'd want a totally linear change).
Re: zooming however, I haven't been able to find how to do the same. The case where I'm failing is on the faux wheel event generated by the scroll gesture on a Macbook's touchpad (image below).
In that case, OSD seems to be following its own logic of "zooming through several levels", where I'd want to fit it tightly to the scroll gesture as it does on the d3-zoom version. I think I've tried all the different OSD parameters and couldn't find the right one.
Have I missed something? Is this something that could be changed with a plugin? Should I instead use d3-zoom to pilot OSD?
Thank you

Wow, that's an amazing image! Is it your work?
Anyway, regarding the zooming, I think there are two factors potentially at work here. One is the springs, and the other is the fact that scroll wheel handling on the browser is unfortunately very idiosyncratic.
On the spring front, changing the stiffness is good. You might also try setting the animationTime down to something very small. Looks like zero is a valid value; perhaps that will fix it right there. If not, and if you're willing to build your own custom OpenSeadragon, you could try adding , false at the end of this line:
https://github.com/openseadragon/openseadragon/blob/4ab8cc748188f6c4dc5ad7d23e91945dd0194173/src/viewer.js#L3159
That will disable the springs entirely for zooming with the scroll wheel (and by "scroll wheel" I mean anything that generates the browser's wheel event, which includes the scroll gesture on the Magic Trackpad). Of course if setting animationTime to zero does the trick, you don't need to bother with this.
On the wheel handling front, I would first disable the springs somehow as above, but if that's not enough, then it's likely a difference in how OpenSeadragon tries to normalize the wheel events versus how D3 does it. If so, you can still use OpenSeadragon, but handle the wheel events yourself, and inform OpenSeadragon when it needs to zoom as appropriate. If you end up going that route, I can give you pointers if needed.

Thanks for pointing out this location! I inserted D3's code like so and the result is now "correct" (at least according to one person's opinion).
     thisScrollTime = $.now();
     deltaScrollTime = thisScrollTime - this._lastScrollTime;
+
+    function defaultWheelDelta(event) {
+          return -event.deltaY * (event.deltaMode === 1 ? 0.05 : event.deltaMode ? 1 : 0.002) * (event.ctrlKey ? 10 : 1);
+    }
+
     if (deltaScrollTime > this.minScrollDeltaTime) {
         this._lastScrollTime = thisScrollTime;

@@ -3153,7 +3158,7 @@ function onCanvasScroll( event ) {
         if ( !event.preventDefaultAction && this.viewport ) {
             gestureSettings = this.gestureSettingsByDeviceType( event.pointerType );
             if ( gestureSettings.scrollToZoom ) {
-                factor = Math.pow( this.zoomPerScroll, event.scroll );
+                factor = Math.pow( this.zoomPerScroll, 5 * defaultWheelDelta(event.originalEvent) );
                 this.viewport.zoomBy(
                     factor,

(I had to multiply d3's "wheel delta" by 5 because the base for D3 is 2, and for OSD it is 1.2).
animationTime = 0 doesn't solve this issue (but it does solve the other issue as well as setting stiffness to 2).
See https://observablehq.com/d/bff8fb5a70c8f3f2 for my tests (and the current settings I'm using). (note: Observable has a "fullscreen" menu item which I encourage you to try.)
One last (but minor) problem is that when I pan quickly by clicking and dragging on the trackpad, then release the click, the image suddenly "jumps" to some final destination a bit farther than where I stop the movement.


Code that show cyrrent date and time of

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Code that show cyrrent date and time of
Closes #XXXXX

@micromarket Thank you for submitting a pull request (PR). I have closed the PR because the changes you have made is either stylistically inconsistent with other files in the repository or do not improve anything.

Note for collaborators: this PR was not reviewed in order because it was not labelled by the bot.


Pin haystack to latest version that supports django<1.11

Fix #4005

By the way, hystack doesn't have its changelog updated, but the 2.8.0 tag contains this commit https://github.com/django-haystack/django-haystack/commit/514023e01136e9393112252fec7eb3b101932d0b#diff-2eeaed663bd0d25b7e608891384b7298L15

I just saw this doesn't fix completely #4005, with the latest pip version (10), the dependencies are checked in a very strict way, I'm going to do another PR for the other incompatible package since it's more open to discussion.

Ok, my other PR fails to use pip 10 on travis, but we only need to fix recommonmark, so that can be discussed here :)
recommonmark and commonmark were upgraded a long time ago 64ee6f6 cb219e8
BUT recommonmark requires commonmark>=0.7.3
https://github.com/rtfd/recommonmark/blob/3238b029b6e19b9cbceb9b85d81304f86b12c012/setup.py#L18

Ok, that wasn't goo, I will revert the change, we only need to update our docs (doing this on other PR).

Seems an issue with recommonmark, but seems to be solved https://github.com/rtfd/recommonmark/pull/63, but it's very weird that the version that is on pypi requires an incompatible version :/

Believe we should be deleting haystack from our dependencies, and code base. It isn't used in production, and just has a few references in our old v1 API.

@ericholscher I'm going to see if I can remove it completely

Blocked on #4039. We won't probably need this PR if we completely remove haystack.


fixed #202

Introduced some new parameters.
Motivation:
I just wanted to fix #202 :)
Modifications:
Created new parameters for the file and line newPromise method inside ChannelOutboundInvoker.
Result:
File and line numbers are now passed to the eventLoop object, all unit tests have been passed.

@tib can you please amend your commit to follow our commit template for the message ? Basically what you added here as description in the issue :) Also please squash. Thanks!

@swift-nio-bot test this please

@normanmaurer I made the changes, now it should be fine.

@tib awesome... will merge once CI finish

@swift-nio-bot test this please

Thanks again!


feat: add selected tab id in query params

@MihaelaCretu11 Can you please add a refs to taskman ticket?


Why reinvent the wheel instead of using KeyRoutedEventArgs.KeyStatus.RepeatCount ?

https://github.com/microsoft/microsoft-ui-xaml/blob/c99f508d96b395c49c9730360294e39e0e14e8dd/dev/NavigationView/NavigationView.cpp#L2178

Correction: CorePhysicalKeyStatus.WasKeyDown should be used instead of CorePhysicalKeyStatus.RepeatCount

That sounds like a good idea. We can avoid the member variable as well. @chingucoding as FYI. @eugenegff would you like to contribute a fix ?

@eugenegff Would you like to contribute a fix, or would you be fine with me creating a PR for this?

@eugenegff Would you like to contribute a fix, or would you be fine with me creating a PR for this?

I'll be fine :)


Win11 dark mode, and switching RN Gallery to light mode causes some rendering issues

Shared steps
###Repro:

On win 21H2, with dark mode as default
Launch RN Gallery app
Click hamburger
CLick settings
Select light theme

Issue 1
Repro

Click hambuger

Result
White on white menu, even no scrollbar

Issue 2
Repro

Go to settings

Result
On the settings page, notice the theme dropdown has no borders indicating it is a dropdown...

Repro (cont)

Hover over the dropdown

Result
Notice that now the text 'light' is white on white:

Repro (cont)

Click on the theme dropdown

Result
Notice that the dropdown still renders using dark theme:

I had thought that the in-app toggle for light/dark mode was supposed to only toggle the control samples, not the whole app. But it's certainly changing much of the app, and inconsistently. As is, using this feature makes the app unusable.

Filing a PR soon to fix first issue of blank menu.
Issue two is an RNW limitation. Unfortunately, we are unable to toggle native theme at runtime (attempting to do so crashes the app given the way RNW theme has been set up). So changing the toggle to light theme does not change the native elements to light theme hence why the border appears missing because the app background has switched to light theme but the picker has stayed in dark theme.
Originally theme toggle was just code snippets but we started the work to toggle the entire app, so that we could support opening the app in light theme and dark theme. But given the issue 2 limitation we can't support the mismatch. I'll have a PR out soon to disable the toggle for now, so people don't walk themselves into the weird state.


Install dependencies and report errors during connection test (#903)

This PR fixes the problems reported in #903.

Thank you for your contribution!
Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You have made the needed changes to the docs
[X] You have written a description of what is the purpose of this pull request above

@gjsjohnmurray everything lgtm, but before merging, check my previous message, I believe we can just get rid of that weird piece of code. let me know your thoughts


feat(dev): add nix-based dev-env

Describe your changes
Trying to make it easier to spin up a fullnode on localhost. The team has been focused on writing tooling, like pindexer, and the manual steps to munge CometBFT configs and setup a localhost postgres database are not awesome. In addition to pd/cometbft spin-up, the dev-env includes:

metrics
postgres
pindexer

Not yet including grpcui, because the latest published version doesn't support the new server reflection API. Will tack that on in a dedicated follow-up.
The dev-env is strongly opinionated and uses nix [0] to manage cross-platform, polyglot dependencies. There's other great work in our ecosystem, e.g. [1], and this early push tries to be consistent with that.
Updates the relevant first-run docs. Much more work should be done on the developer-focused content in the guide overall, including an overhaul of discussing RPC endpoints and web-relevant resources, but those changes can land in the future. This PR aims to establish a basis for a sane, supported dev-env, providing a pleasant first-run experience for newcomers (and existing devs too).
[0] https://nixos.org
[1] https://github.com/informalsystems/cosmos.nix
Here's an example of what the just dev command looks like:

Issue ticket number and link
No issue, but this PR builds on the excellent foundation by @cratelyn in #3574.
Checklist before requesting a review

[x] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

No changes to application code: focuses on docs and scripts for dev/ci.

Testing and review
In order to review these changes, one should check out this feature branch, build the dev docs in docs/guide/, and view them locally, then follow the instructions at http://<IP_ADDRESS>:3000/dev/dev-env.html. Pay attention to what's confusing, what's unpleasant, and write those thoughts down as part of the review.
Then proceed to http://<IP_ADDRESS>:3000/dev/devnet-quickstart.html and run just dev as recommended. Does anything break? Does it work as advertised? Is anything unclear? Try running ctrl+c to stop the setup, and then just dev to resume it. Does it come back up OK? Is that a sane behavior?

Guide was easy to follow, but I got this error when trying to run nix develop on macOS:
nix develop
error (ignored): error: end of string reached
error:
       … while calling the 'derivationStrict' builtin
         at <nix/derivation-internal.nix>:9:12:
            8|
            9|   strict = derivationStrict drvAttrs;
             |            ^
           10|

       … while evaluating derivation 'nix-shell'
         whose name attribute is located at /nix/store/iqxdbcjlg9wx8gdc8bdhy9nsd5imcbjp-source/pkgs/stdenv/generic/make-derivation.nix:334:7

       … while evaluating attribute '__impureHostDeps' of derivation 'nix-shell'
         at /nix/store/iqxdbcjlg9wx8gdc8bdhy9nsd5imcbjp-source/pkgs/stdenv/generic/make-derivation.nix:449:7:
          448|       __propagatedSandboxProfile = unique (computedPropagatedSandboxProfile ++ [ propagatedSandboxProfile ]);
          449|       __impureHostDeps = computedImpureHostDeps ++ computedPropagatedImpureHostDeps ++ __propagatedImpureHostDeps ++ __impureHostDeps ++ stdenv.__extraImpureHostDeps ++ [
             |       ^
          450|         "/dev/zero"

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: invalid regular expression '.*.(no_lfs|param||bin)$'

Cool, looks good to me. I hit a few macOS-related stumbling blocks but pushed up fixes. I was able to get everything running in the end.

I hit a few macOS-related stumbling blocks but pushed up fixes.

Nice, thanks for taking a look! Some of the macos-specific entries broke the dev env on Linux for me, so let me tinker a bit more...

Nice, thanks both for testing!


Build settings-template.zip on publish of package and docs

settings-template.zip is currently checked into the repo, but doesn't need to be. We should build it before publishing the package and also publishing docs so that it can be downloaded

Fixed in Harmonium 6.0.0


Relative imports

Hi !
Juste made some imports relative, useful when the lib is installed manually.
I made the changes online and didn't test, so merge carefully !
Cheers !
Olivier

Coverage remained the same at 83.982% when pulling a9408472f79098755301cc72a228d72869e2b598 on olivierdalang:relative_imports into 3633d43fa3e97bc9f92ea83a749925bbad215393 on reclosedev:master.

I'm closing this as I don't believe there is any advantage to relative imports. Feel free to resubmit this if you have more details on how this helps with local development. Thanks!

It helps in many different scenarios, when you don't install through pip, such as having the library included as a submodule in an existing project. With absolute imports, you need to add the path to the folder to the python path, otherwise import requests_cache doesn't work.

Ah, thanks for clarifying. Git submodules aren't very common in python projects (since you can pip install a git repo), but that makes sense. In that case I'll go ahead and merge this.
There are some failing unit tests, although those are unrelated to your changes, and probably the result of some dependency breaking within the ~1.5 years since this project was last built (!!). I'll look into that soon.


Document AnimationRootNode

The bullet list was mostly taken from the linked tutorial, but heavily adjusted for [AnimationNodeBlendTree].

@fire: has this been superseded by https://github.com/godotengine/godot/pull/76549, or should I refactor my PR to incorporate that PR's content?

I think a second paragraph can be added to the description in a form of "Examples of built-in root nodes include [AnimationNodeBlendTree] (allows blending nodes between each other using various modes), ..."

@YuriSizov: Done.


add MsgDelegate ica callback

What is the purpose of the change
Add DelegateCallback to handle delegation state changes.
Brief Changelog

Add DelegateCallback proto message (stores serialized / deserialized callback args)
Update SubmitTx... functions to take in a callback id, args
Remove delegate callback logic from ibc_handlers.go
Add DelegateCallback function (+ argument serialization / deserialization functions)
Call SubmitTxsStrideEpoch with "delegate" callback id and marshalled args

Testing and Verifying
integration tests
integration_tests.bats
 ✓ [INTEGRATION-BASIC] address names are correct
 ✓ [INTEGRATION-BASIC] ibc transfer updates all balances
 ✓ [INTEGRATION-BASIC] host zone successfully registered
 ✓ [INTEGRATION-BASIC] liquid stake mints stATOM
 ✓ [INTEGRATION-BASIC] tokens were transferred to GAIA after liquid staking
 ✓ [INTEGRATION-BASIC] tokens on GAIA were staked
 ✓ [INTEGRATION-BASIC] redemption works
 ✓ [INTEGRATION-BASIC] claimed tokens are properly distributed
 ✓ [INTEGRATION-BASIC] rewards are being reinvested (delegated balance increasing)
 ✓ [INTEGRATION-BASIC] exchange rate is updating
 ✓ [NOT-IMPLEMENTED] reinvestment and revenue amounts are correct

11 tests, 0 failures

run through staking flow, observe record deletion

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not documented

@sampocs this is ready for another review, I restructured this so that callbacks take in an Acknowledgement_Result

last commit looks good, go ahead and 🚢  when you're ready!


Not getting the response from the APi

Not getting the response from the APi Please Advice

@irfannoor135 Please provide some more details, e.g. Request Url and Payload

yes here is sample code not even getting error or anything please advise
<EMAIL_ADDRESS>  // The API version. Currently must be 1.0
  'Version' => '1.0',
  // The API method to call.
  'Action' => 'GetProducts',
  // The format of the result.
  'Format' => 'XML',
  // The current time formatted as ISO8601
  'Timestamp' => $now->format(DateTime::ISO8601)
);

// Sort parameters by name.
ksort($parameters);

// URL encode the parameters.
$encoded = array();
foreach($parameters as $name=>$value){
	$encoded[]=rawurlencode($name).'='.rawurlencode($value);
}

// Concatenate the sorted and URL encoded parameters into a string.
$concatenated = implode('&', $encoded);

// The API key for the user as generated in the Seller Center GUI.
// Must be an API key associated with the UserID parameter.
$api_key = '9ffe39feb6efcdafc2ac3c045c7ec4fa3cf26924';

// Compute signature and add it to the parameters.
$parameters['Signature'] = rawurlencode(hash_hmac('sha256', $concatenated, $api_key, false));

// Replace with the URL of your API host.
$url = "https://api.sellercenter.net";

// Build Query String
$queryString = http_build_query($parameters, '', '&', PHP_QUERY_RFC3986);
echo $url."/?".$queryString;

// Open cURL connection
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, $url."/?".$queryString);

// Save response to the variable $data
curl_setopt($ch, CURLOPT_FOLLOWLOCATION,1);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, TRUE);

$data = curl_exec($ch);
print_r($data);

// Close Curl connection
curl_close($ch);
?>

making a call like
'https://api.sellercenter.net/?Action=GetProducts&Format=XML&Timestamp=2016-12-07T06%3A33%3A49%2B0000&UserID=xxx%40xxxxxxxx.com&Version=1.0&Signature=f08d4979a39ab9596216e217479ca81255a0538fc0ff0f6f413b3b4946d5c49a'

Please have a look at the following lines

// Replace with the URL of your API host.
$url = "https://api.sellercenter.net";

You are using a dummy URL, this URL does not really exists. Please replace with the correct one for your Seller Center installation. You get the API URL for example on the Page [SellerCenterUrl]//api-explorer
Additionally the code snippets do not refer to our provided SDK since the SDK does the sorting and signing of the params via the provided library. Please have a look at the code examples on https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/tree/master/samples

Thanks,
how to creat product sample ?
2016-12-07 15:21 GMT+07:00 Ulf Kirsten<EMAIL_ADDRESS>
Please have a look at the following lines
// Replace with the URL of your API host.
$url = "https://api.sellercenter.net";
You are using a dummy URL, this URL does not really exists. Please replace
with the correct one for your Seller Center installation. You get the API
URL for example on the Page [SellerCenterUrl]//api-explorer
Additionally the code snippets do not refer to our provided SDK since the
SDK does the sorting and signing of the params via the provided library.
Please have a look at the code examples on https://github.com/rocket-
internet-berlin/SellerCenterSDK-PHP/tree/master/samples
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/issues/17#issuecomment-265384901,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Nx1Fcx11AhTq0v7NdeJJ24pdJKL-Gks5rFmx2gaJpZM4LEKx5
.

--
Thân ái
Nguyễn Phú Thụy
Phone: 0985.910.350
Email<EMAIL_ADDRESS>Website: Himi.vn

see

https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/master/samples/endpoint/product/productUpdate.php
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/master/samples/endpoint/product/ProductCreate.php

please also refer to the readme on https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP which should answer the most questions

i know but my host is not support Composer
2016-12-07 16:57 GMT+07:00 Ulf Kirsten<EMAIL_ADDRESS>
see

https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/
master/samples/endpoint/product/productUpdate.php
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/master/samples/endpoint/product/productUpdate.php
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/
master/samples/endpoint/product/ProductCreate.php
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/blob/master/samples/endpoint/product/ProductCreate.php

please also refer to the readme on https://github.com/rocket-
internet-berlin/SellerCenterSDK-PHP which should answer the most questions
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rocket-internet-berlin/SellerCenterSDK-PHP/issues/17#issuecomment-265404708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-NxwK2aJKqRahTlA4AeYTvNFJd9Jcgks5rFoMQgaJpZM4LEKx5
.

--
Thân ái
Nguyễn Phú Thụy
Phone: 0985.910.350
Email<EMAIL_ADDRESS>Website: Himi.vn

Composer can be installed on any system, please refer to https://getcomposer.org/doc/00-intro.md
Eventually you need a running composer to install/use our SDK


fix(search): project type facet serialization

Fixed #11

It seems that I cannot sign the CLA.

Hey, sorry for the above issue. We are currently undergoing some changes in our CLA signing process. Please send a request to<EMAIL_ADDRESS>and and add your name to contributors list.


Icons on "Attend" link to the wrong pages

The following icons link to the wrong pages:

Fixed once the cache expires.


[Planned for Q4] Improve the usability of the documentation

Context
It's been a few years since we released the Kedro documentation and we have yet to make any major updates to the look and feel beyond some basic skinning for brand changes and changes to the order of pages. We did some user research recently and, while there are other high priorities that were reported, such as search and content updates, we did receive some feedback about the design of the docs. I'll share that separately to maintain the privacy of the interviewees.
I think it's also clear that our docs don't compete with other similar projects in presentation style, and they don't stack up well against our brilliant website and blog design. We receive several thousand visits to our docs landing page in any 30 day period so this matters. Having a competitive style and improved usability is key to ongoing success of the project.
Detailed description of some of the problems
As mentioned, I'll share quotes separately but it is clear from looking at the documentation home page that we could improve it.

Issues include: white space on the right hand side of the homepage but a wide span of text on all other pages except search, home page is just a list of links that mirrors the table of contents, limited in-page navigation.
There is a separate discussion to be had on the organisation of the content into more persona-led "journeys" since we don't segment the readership into different paths depending on their needs. However, I want this ticket to focus on finding great examples of documentation in the same arena as ours, and building a set of proposed designs. We can separately decide on content organisation if/as we agree on the most usable layout for Kedro users as the project moves forwards.
Requirements
I don't want to influence @stephkaiser in terms of design but I think these are great examples of documentation that we could emulate.

DVC
scikit-learn (For the homepage design. One of our survey participants specifically called out these docs as something we should aspire to become, but I feel the actual content pages have quite a few of the problems that we do)
sktime was also suggested to us by one of our interviewees

Some themes we could consider:

awesome sphinx

Sphinx book

Also from kedro-org/kedro#2394 (from @astrojuanlu)

Furo

PyData Sphinx Theme

Lutra

Possible approaches

One possible approach to this is to pick one of the sphinx themes above as the basis of our work and repurpose it.
An alternative is to rebuild docs entirely into the website if we find a look and feel in docs that isn't Sphinx based and don't believe we can adequately reproduce it.

Next steps
I need to discuss further with @stephkaiser and the broader team to evaluate available time from Design and beyond.

The RTD ad covers the "Next" button 😱

The RTD ad covers the "Next" button 😱

There are two different paths here to solve this issue:

Opt-out from Ads: https://docs.readthedocs.io/en/stable/advertising/ethical-advertising.html#opting-out
Use an explicit placement for the ad in your theme: https://docs.readthedocs.io/en/stable/advertising/ad-customization.html#controlling-the-placement-of-an-ad

Let me know if them works for your use case.

Thanks for the pointers @humitos. I don't think we're opting out for now, and to find a better placement we need to do some design work anyway. We might try to solve this more urgently though and then do a full redesign at a later stage.


make table to be responsive to mobile view

Clear and concise description of the problem
when a page is opened on mobile, half of the table is visible and half is not,

Suggested solution
in module NTable and NDataTable make it happen
Alternative
give a prop name mobile, let it figure out the screen resolution and display the table based in it
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

We might not specifically adapt to the mobile end.


Allow TS non-null assertion in object literal context

{
  isOpen,
  inputValue!,
}
---
{
  isOpen,
  inputValue: inputValue!,
}

Already done. Or did you have an instance where it doesn't work?

I didn't even realize! Was just programming in TS today and felt like Civet should have this... little did I know it already did!


GB should not send the OTR Query Messages unless the user sends a message

At the moment, if I click on a buddy's name, it sends the OTR Query Message ("?OTRv2?").
Occasionally, I touch the wrong person's name and then get a response from them saying "what was that message about?"
It would be good if Gibberbot did not send anything to the contact until I entered an actual message and sent it (then it could send both at the same time).

You can just this in Gibberbot->Menu->Settings. It is entered as "automatic" by default.


Incorrect link to "upload attachments"

In Step 2 > second bullet of "Further customize your form" https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/customize-list-form#further-customize-your-form

"upload attach,ents" incorrectly links to -- https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/controls/properties-text
this should be -- https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/controls/control-attachments

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

ID: 0eacf34e-9ae6-5c64-5fec-c250fc9986d1
Version Independent ID: 7d248abd-225e-c914-e871-f8d2424ed8c0
Content: Customize a SharePoint list form - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/customize-list-form.md
Service: powerapps
GitHub Login: @AFTOwen
Microsoft Alias: anneta

Thanks, @embraces0ul. Replaced the link; closing issue.


[Bug] Fix constraints enforcement rollover test

resolves #1009
Problem
This test started failing. We skipped it to release and need to address the failure. We suspect there was a change with delta tables in Databricks that we're not properly accounting for.
Solution

unskip the test
TBD

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

The test appears to be working locally now. I ran it a few times and it was successful. The changes that I made might still be worth including as quality of life improvements. But the main change is simply unskipping the test.

The original issue showed up in main, which is now 1.8.latest. I'll confirm this is passing on previous branches as well.


Fix Nixinova submodule issues

Description

Nixinova submodule was manually added without using script, which results in it getting deleted from .gitmodules and grammers.yml files when other languages are added or deleted
Also added a second positional argument in script/add-grammer, to override the repository name. In this case repo name is tmLanguage and the intended name is Nixinova-tmLanguage.
Exact command used (after deinit the exiting submodule):
$ script/add-grammar https://github.com/Nixinova/tmLanguage Nixinova-tmLanguage

Fixes https://github.com/github/linguist/issues/5291
Checklist:

[x] I am updating a grammar submodule

A more descriptive title for this PR would be Add optional output folder argument to script/add-grammar.

Also added a second positional argument in script/add-grammer, to override the repository name. In this case repo name is tmLanguage and the intended name is Nixinova-tmLanguage.

Erh, why not rename your repository to nixinova-linguist-grammars instead? Or nixinova-grammars, nixinova-textmate, etc… Moreover, this would break consistency between a repository's name and its submodule's name.

why not rename your repository to nixinova-linguist-grammars instead?

That's what I suggested to him in the original issue, twice.

A more descriptive title for this PR would be

Not really, as I mentioned in the issue and in the description, the main problem with current master is that you did not use script/add-grammar and now it is causing problems for others.

why not rename your repository to nixinova-linguist-grammars instead?

That's what I suggested to him in the original issue, twice.

A more descriptive title for this PR would be

Not really, as I mentioned in the issue and in the description, the main problem with current master branch is that when someone tries to add a new grammar, it deletes your grammar because it was not added properly using script/add-grammar script.
When we do try to add your grammar using script/add-grammar script, the linguist compiler gives 10+ fatal errors which were fixed by https://github.com/Nixinova/tmLanguage/pull/4. This PR is potentially updating the submodule to fix the repo state.

this would break consistency between a repository's name and its submodule's name.

Well, there aren't any other repos in this called tmLanguage so just leaving this as-is would work.

Thanks for this @am11. As with @Alhadis, I'm not keen on allowing peeps to break the consistency between a repository's name and its submodule name as used by Linguist as it adds unnecessary complexity when trying to track problems down.

Well, there aren't any other repos in this called tmLanguage so just leaving this as-is would work.

Indeed, however I'd much rather have you rename to a less-generic name which has less risk of clashing or causing confusion.

@lildude, agreed; that was my first preference as well. I will update the PR when the repo is renamed.

I had changed the repo name again to Nova.tmLanguage while you were doing this.

or

I changed my mind (at least) one hour later and renamed again

otherwise timestamps and passing CI make no sense

It was changed literally 10sec later. See above, at how https://github.com/Nixinova/Nova.tmLanguage/issues/5 is located above your commit.

Ah must be GH redirects. I committed 51 minutes after your comment https://github.com/github/linguist/pull/5292#issuecomment-811322602, then six CI legs ran.

It was changed literally 10sec later.  how Nixinova/Nova.tmLanguage#5 is located above your commit.

Not sure why you deleted your old comment as that would've been what happened.

It was not relevant to your 10sec comment, which I read after the page refresh. The difference was one hour not 10 sec.

@am11 would you mind updating this PR for the final new repo name?

I think you need to pull master into your branch as you appear to be missing the NimLime grammar change just merged.


Deep Key support

I can't figure out why deep key doesn't work.
Let's say I have { button: { save: "Save" } } in my locale file en.json, I would like to access it by doing t('button.save').
I tried solutions like adding keySeparator to the config file but it doesn't work. In fact, adding nested object make the whole file fail the translation.
I'm sure I'm not the only one facing this issue but it looks like a missing feature. Am I missing something ?

This is an i18next issue, as next-i18next doesn't handle any low-level API details like that. Good luck!

I don't believe this is due to i18next, I feel there is an issue somewhere in next-i18next that is causing this.
When a deep key is added the entire JSON is effectively discarded client side.
I've been through the next-i18next code/config and I can't work out whats causing the issue.
The same setup when using react-i18next seems to work based on their examples.
@isaachinman do you have a working example of next-i18next with deep keys?
I have forked the next-i18next repo and added a deep key example:
https://github.com/tomevans18/next-i18next/tree/master/examples/deep key example

@tomevans18 What unexpected behaviour are you experiencing?

The translations are picked up client side when there is a deep key within the JSON.
So when translating between languages nothing happens.
It seems like having any nested values breaks the setup.

https://www.loom.com/share/b9c6676591f44f09865083b6f5388d2b

Not sure how that could possibly be related to next-i18next, as it seems the component is failing to rerender, and the useTranslation hook comes from react-i18next directly.

@tomevans18 Can you see what happens if you play around with a key set to locale, similar to here?

Not sure how that could possibly be related to next-i18next, as it seems the component is failing to rerender, and the useTranslation hook comes from react-i18next directly.

This is whats confusing me as react-i18next on its own works with this setup (there are lots of examples doing this), which is why I'm looking at next-i18next as the possible cause but I can't see what would cause this.

@tomevans18 Can you see what happens if you play around with a key set to locale, similar to here?

I'm not sure I follow what you mean here

@tomevans18 Unfortunately I lost about 10 minutes on this one, until realising that your JSON is invalid. Perhaps you might want to request the upstream i18next packages emit a warning when encountering invalid JSON.

Damn, sorry. I can't believe it was down to a trailing common. Same issue in my main repo, at least its Friday!


cri: selinuxrelabel=false for /dev/shm w/ host ipc

This is a followup to #4699 that addresses an oversight that could cause
the CRI to relabel the host /dev/shm, which should be a no-op in most
cases. Additionally, fixes unit tests to make correct assertions for
/dev/shm relabeling.
Discovered while applying the changes for #4699 to containerd/cri 1.4:
https://github.com/containerd/cri/pull/1605
Signed-off-by: Jacob Blain Christen<EMAIL_ADDRESS>

@crosbymichael I am not sure if this is necessary but it seemed like a reasonable guard to have in place (don't attempt to relabel host /dev/shm).

Is this one of the flaky timeouts? https://github.com/containerd/containerd/pull/4705/checks?check_run_id=1375887385

I have verified that this fixes an actual problem in our 1.3 and 1.4 forks we maintain for k3s

cherry-pick (https://github.com/containerd/cri/pull/1605) was merged, but needs a re-vendor


Add cloud support

https
S3
Google Cloud

Verified that both https and s3 sources are addressed by way of http key/value store driver support in #37. Google Cloud may
also be included or may require gcs driver support. https://google.github.io/tensorstore/kvstore/gcs/index.html
It would be good for someone with an active Google Cloud access to test that support.


Fix JSON parser to allow control characters in JSON string input

JSON string input can contain control characters, e.g., new line character \n. It can be
correctly handled by Presto, but not allowed by simdjson lib. If it exists in input,  simdjson
will return UNESCAPED_CHARS error, then Velox will output null. Simdjson only allows literal
\n (represented by \\n). See its code link.
Here is a test result with presto.
SELECT json_extract('{"c1":"ab\ncd"}', '$.c1');
  _col0
----------
 "ab\ncd"
(1 row)

Just created one discussion thread in simdjson community: https://github.com/simdjson/simdjson/issues/2287.

Hi @PHILO-HE , I am working on canonicalization of jsons (https://github.com/facebookincubator/velox/pull/11284); Ideally we aim to handle escaping during canonicalization (i.e from json_parse or when say cast(x to json)).
Looking into the example you raised, I am wondering if this is another way we should handle canonicalization (cc: @gggrace14 )

@kgpai, if simdjson accepts the proposal for JSON canonicalization or some other required functionalities like this, I think it would be better.


Invalid timestamp

Hello,
GELF Format Specification Version 1.1 (11/2013) expect a number as timestamp http://docs.graylog.org/en/2.2/pages/gelf.html but the library send a String:
https://github.com/t0xa/gelfj/blob/master/src/main/java/org/graylog2/GelfMessage.java#L227
This is a problem for any input respecting GELF format specification as all logs emit by the library will be drop because of invalid timestamp.
Thanks.

This came up in a bug report for Graylog 2.3.0-rc.2: https://github.com/Graylog2/graylog2-server/issues/4027
The "timestamp" field should definitely be a floating point number and not a string.

+1
any ETA on this ?  It is hurting my app and we are dropping a lots of messages.
Thanks

@bonzi316 Graylog doesn't drop GELF messages in which the timestamp field is a string.
See the referenced Graylog2/graylog2-server#4027 for details.


How to crop the image and get the dataUrl?

I understand how to set Croppr up, but how do I actually get the cropped image after the user is done cropping the image?

Croppr does not actually crop the image for you. It's function is to simply retrieve the cropping coordinates so that other services (such as your backend) can do the actual cropping. It was made on the understanding that it's safer to perform server-side image manipulation rather than trusting the user's input entirely.
However, if you do insist on performing client-side cropping (or a mix of both), you are free to use your own implementation for the actual image cropping part. You can refer to these links for further information:

https://yellowpencil.com/blog/cropping-images-with-javascript/ (Refer to step 4)
http://stackoverflow.com/questions/17402948/crop-and-upload-image-on-client-side-without-server-side-code-involve

Ah, I see. Wonderful. Thank you @jamesssooi :)


feat: update conditions

📝 Description
Adds aria-invalid to the _valid and _invalid condition
Also adds _top, _right, _bottom, _left conditions. Those are quite handy when using ark-ui, so I figure I'd include them. Might remove them if mantainers judge it unecessary.
Also updated the docs which was outdated
💣 Is this a breaking change (Yes/No):
No

I discussed this with the team, and we decided not to add this PR for now.
Please update your panda config for now. We'll consider it in the future.


Added Cohere and fixed issue #93

[X] My submission is formatted according to the guidelines in the contributing guide
[X ] My addition is ordered alphabetically
[X ] My submission has a useful description
[ X] The description does not have more than 100 characters
[X ] The description does not end with punctuation
[ X] Each table column is padded with one space on either side
[ X] I have searched the repository for any relevant issues or pull requests
[X ] Any category I am creating has the minimum requirement of 3 items
[X ] All changes have been squashed into a single commit

Hi @Shaurya50211, thank you for the contribution.
Just a couple of things before we merge:

Does this service offer an API where developers could retrieve data? I just found their SDK after a quick look. If so, could you link to it instead?
Please keep the title of the project as it is.

Let me know if you have any questions.

Sure, I will change that

Did you mean https://docs.cohere.ai/ link?

Did you mean https://docs.cohere.ai/ link?

Sure, that works. Thank you!

Wait, what is wrong with the title and description?

Wait, what is wrong with the title and description?

For the moment I'd prefer to keep the title and description of the project as they are, as they're also used on the website.
In general, contributions to the resources are always welcome, but the project's information should remain as is.
Btw, I'm referring to this change here.

Thanks for clarifying

Done!

I also fixed issue #93

Thank you very much!

Thank you very much!

My Pleasure!


Open Source : 6.31 Explain everything related to D3.js

Description
Description
Open Source : 6.30 Explain everything related to D3.js
Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@Adi7171  Go ahead 🎉🎉


Unimplemented function in regulariser plugins are opaque

There are a number of unimplemented functions in regularisers.py. The following in non-exhaustive.
These have comments:
https://github.com/vais-ral/CCPi-Framework/blob/1803cbd445c408588fecbf705fb8b4df486029fc/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L164-L166
https://github.com/vais-ral/CCPi-Framework/blob/1803cbd445c408588fecbf705fb8b4df486029fc/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L261-L263
This one prints a warning:
https://github.com/vais-ral/CCPi-Framework/blob/1803cbd445c408588fecbf705fb8b4df486029fc/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L184-L188
This one is unmarked:
https://github.com/vais-ral/CCPi-Framework/blob/1803cbd445c408588fecbf705fb8b4df486029fc/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L109-L110
They all return 0.
It is understandable for them to be unimplemented, but it would be nice for it to be a little more transparent to the user. Would it make sense to (a) give them all a warning message, and (b) have them return NaN so that the code can run on, but that the results will clearly be invalid when the user attempts to plot them?

Thanks @ashgillman, I'm generally inclined to raise an error rather than returning nan or similar. Here, instead we return 0 which is even wronger.

An error makes the most sense if calling these is dangerous (i.e., if it turns out that these may be causing some errors I am seeing in other discussions).
If its the case that, for example, the optimisers will run fine without say __call__ but that the

Somewhat related, I don't think FGP_dTV.__call__ is correct, this seems to be TV not dTV.
https://github.com/vais-ral/CCPi-Framework/blob/1803cbd445c408588fecbf705fb8b4df486029fc/Wrappers/Python/cil/plugins/ccpi_regularisation/functions/regularisers.py#L229-L232

I believe that the regularisers are meant to be used in our optimisation framework only with their proximal method, which in itself is a minimisation problem. That is the reason why they are functions where only proximal is developed.
For TV and dTV that expression of __call__ might be correct but it's better to ask a mathematician, @jakobsj @epapoutsellis .

Any regulariser exposed by the plugin should have a correct call method
provided by the plugin, so that for example objective function value is
reported correctly in optimisation algorithms. If that is not the case, it
must be fixed.
On Wed, 10 Feb 2021, 15:19 Edoardo Pasca<EMAIL_ADDRESS>wrote:

I believe that the regularisers are meant to be used in our optimisation
framework only with their proximal method, which in itself is a
minimisation problem. That is the reason why they are functions where only
proximal is developed.
For TV and dTV that expression of call might be correct but it's
better to ask a mathematician, @jakobsj https://github.com/jakobsj
@epapoutsellis https://github.com/epapoutsellis .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vais-ral/CCPi-Framework/issues/755#issuecomment-776736417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMDSCF7YEL6OYKNOU4UIK3S6KIWXANCNFSM4XIVCUUQ
.

From @epapoutsellis
https://github.com/vais-ral/CCPi-Framework/issues/758#issue-805671962

All of the regularisers that are not TV have a wrong implementations of the __call__ method.

To start with I suggest we remove the definition (override) in the derived class. If no implementation is given the base class method will be called, which already raises NotImplementedError.
Then we can develop the appropriate evaluation of the regulariser Function, but at least the user will not be tricked by getting a wrong value.
Currently, dTV implementation will be implemented in https://github.com/vais-ral/CCPi-Regularisation-Toolkit/issues/159

If we use raise NotImplementedError then none of the algorithms will run because objective at the iteration=0 must be computed. I suggest to return 0. plus a warning message as a quick fix.

You could also throw an error, and then catch it at a known place where it is called but the value isn't necessary

I believe at this point a value needs to be returned for the algorithms to run as mentioned by @epapoutsellis. For a quick-fix I think that returning nan as suggested by @ashgillman is safer than 0, and is the best option at this point, since it clearly indicates that something is not right, but does allow the algorithm to run. Probably with a warning, but a warning may easily be overlooked which is why returning 0 is dangerous.

Agreement is:

return numpy.nan
issue a warnings.warn
Change the message to be more intelligible to users.

Fixed in current version