VncRecordingContainer creates an unnecessary delay during video export.

Hey @nachg, can you please add a bit more context to the PR?

Hey @nachg, can you please add a bit more context to the PR?
I put context here:
https://github.com/testcontainers/testcontainers-java/issues/5268

Can you please elaborate on how copying the file before reencoding fixes the issue at hand?

Can you please elaborate on how copying the file before reencoding fixes the issue at hand?

VNC recorder working with the output file as with a stream. If we will start the encoding process on that file, it length will be increased during the process. That's why I create a copy - to "stop" extanding the file length by the VNC. Another possible solution is to stop the VNC process and run encoding after it. But I was not able to find the way how it can be done.


User network failure when data-root has been changed

I'm changing docker data-root(I need to keep volume data in different location) by doing the following steps:
Open up /lib/systemd/system/docker.service
Replace ExecStart=/usr/bin/dockerd -H fd:// $DOCKER_OPTS
with ExecStart=/usr/bin/dockerd --data-root /data/ -H fd:// $DOCKER_OPTS
Restart:
systemctl daemon-reload  systemctl restart docker
Now I'm creating user network docker network create --subnet=<IP_ADDRESS>/16 shop-network and launching container docker run -it --net shop-network --ip <IP_ADDRESS> shopliquibase.
I'm trying to ping external resources(internet) as well as another container started on shop-network and in both cases getting no response.
I've tried to do the same thing on default configuration (without data-root being set) and it works. I've also tried with set data-root on default bridge network and it also works.
Could you give any hint or reference on how could I investigate the problem.

Hey @mvb131988 I'm not sure this issue is related to docker app, I will close it, you should try moby/moby, if you think however that it's related to docker app feel free to reopen it.


Fix the problem with writing a file using streams and not using RAM

Summary
This PR fixes the problem with writing a file using streams and not using RAM. It solves the problem of writing large excel files.
Test plan
import * as fs from 'fs';
import { stream } from 'exceljs';

const output_file_name = "/test.xlsx";

const writeStream = fs.createWriteStream(output_file_name, { flags: 'w' });
const wb = new stream.xlsx.WorkbookWriter({ stream: writeStream });
const worksheet = wb.addWorksheet("test");

const headers = Array.from({length: 256}, (_, i) => i + 1).map((i) => 'test' + i);

for (let i = 0; i < 100000; i++) {
  const row = headers.map((header) => header + '|' + i);
  await worksheet.addRow(row).commit(); // This raw will be immediately written to disk and will not clog RAM.
}

await worksheet.commit(); // This is not necessary because await wb.commit() is used, but you can also write to disk not raw by raw, but worksheet by worksheet.
await wb.commit();

Related to source code (for typings update)
https://github.com/zlooun/exceljs
https://www.npmjs.com/package/@zlooun/exceljs

Thank you for the contribution; I believe it enhances the handling of large file writing 🎉. Please bear with me, as this change breaks backward-compatibility and will need to wait for the next major release (5.0).

Hello just hoping to bump some attention here. This PR would be very helpful for my work. I'd offer to help get it in but honestly I learned everything I know about streams and buffers trying to understand why await workbook.commit() was hanging forever. In my case, it's because my S3 upload releases the stream, which I expected to do after committing the workbook. So effectively that await will be forever if I put it before the S3 upload terminates.
(Or then again, I could be off base here - like I said, new to streams).


Set idate after ajax request

How do i set idate attribute after an ajax request (and after the template gets loaded)?
If I have something like
MyService
.getSomeData()
.then(function(response)) {
$scope.datepickerdate = response.date;
}
in my controller and
<ionic-datepicker idate="datepickerdate" ...
in my template, this datepickerdate is never set, because the idate attribute is "loaded" together withe the controller. In other words, I needed to set this attribute with a date that comes from my backend app.

Hi @chrisbenseler, In some cases, specifically in api calls, the view might not update with the returned value. If this is the case, may be you can use $apply.
You can see lot of posts, which might help you.

hi @rajeshwarpatlolla
I tried this
MyService
.getSomeData()
.then(function(response)) {
setTimeout(function () {
$scope.$apply(function() {
$scope.datepickerdate = response.date;
});
}, 2000);
}
and the problem still happens (also, I tried to use $timeout instead of native setTimeout, no success too).
Debugging ionic-datepicker.js, I noticed some things:
scope has a ipDate attribute with the date from now, instead of those I set with idate attribute.
ipDate: Tue Aug 04 2015 07:51:25 GMT-0300 (BRT)
but proto prints out "Invalid Date"
Even if I set a date in the $scope and pass it to idate, without any ajax call, like:
$scope. datepickerdate = new Date(2000, 10, 1);
the console says:
ipDate: Wed Nov 01 2000 00:00:00 GMT-0200 (BRST)
proto: Invalid Date
I think there are 2 different problems: one (the first I reported) is with the binding of data after the render of the element (it seems the ionin-datepicker sets the value of ipDate when it's created, instead of when user clicks to open it) and the other is with parsing dates.

hi @rajeshwarpatlolla
I tried this
MyService
.getSomeData()
.then(function(response)) {
setTimeout(function () {
$scope.$apply(function() {
$scope.datepickerdate = response.date;
});
}, 2000);
}
and the problem still happens (also, I tried to use $timeout instead of native setTimeout, no success too).
Debugging ionic-datepicker.js, I noticed that inside element.on("click", function () { }) that scope.ipDate is null.
In line 193, there is:
refreshDateList(angular.copy(scope.ipDate));
console.log(scope.ipDate); prints "Tue Aug 04 2015 08:05:50 GMT-0300 (BRT)" instead $scope.datepicker's new value. It seems that there's something happening with the binding of the value after the ionic-datepicker is rendered.
I'll take a look further, if there's something I'm implementing wrong. Thanks for the help.

Please try with the new version v0.7.0

destroy the datepicker inside the success & call again with beforeShowDay function outside the ajax function:
(Note: dynamicCalender fn is called when select dropdown in other select input)
var holidays = '{!! $holidays !!}';
$( "#appmnt_date" ).datepicker({
beforeShowDay: nationalDays,
minDate: "+1D",
dateFormat:"dd-mm-yy",
changeMonth: true,
changeYear: true,
maxDate: "+1D +3M",
});
var booked_dates = '';
function dynamicCalender(){
const service = jQuery("#service option:selected").val();
jQuery.ajax({
method : 'GET',
url: {{ url('filled-dates/${service}') }},
async: false,
cache: false,
success : function(response){
booked_dates = response.booked_dates
console.log(booked_dates)
     }
  })
  $("#appmnt_date").datepicker("destroy");

}
$('#appmnt_date').datepicker({
beforeShowDay: nationalDays,
});
function nationalDays(date) {
var string = $.datepicker.formatDate('yy-mm-dd', date);
  if((date.getDay()==0 || date.getDay()==5 || date.getDay()==6) || holidays.indexOf(string)!=-1){
     if(date.getDay()==0 || date.getDay()==5 || date.getDay()==6){
        return  [!(date.getDay()==0||date.getDay()==5||date.getDay()==6), 'weekend-date']
     }
     return [holidays.indexOf(string) == -1, 'holidays']
  }
  else if(booked_dates.indexOf(string) != -1){
        return [booked_dates.indexOf(string) == -1, 'booked-dates'];
  }
  return [true, 'no-slots'];

}


BUYC-691 Now able to use the username parameter to create a coupon

Overview
The coupon parameter can now be used when creating coupons in-game.
Will need to be added to https://help.buycraft.net/article/141-coupon-commands.

should there not also be a change around https://github.com/BuycraftPlugin/BuycraftX/blob/master/common/src/main/java/net/buycraft/plugin/client/ProductionApiClient.java#L242 ?


More engaging entrance page

http://recordit.co/rr3c3dbHyM

https://www.kirupa.com/html5/ken_burns_effect_css.htm

yes!


Release fixes

Codemod needs a hashbang to specify interpreter when it is run via npx
We need to resolve from src/index.ts rather than .js
Convert Section to PickerSection and ComboBoxSection - this still doesn't handle dynamic collections. For that we would need to wrap the function child in <Collection>
Small fix for cursor in NumberField stepper buttons
Fix build of illustrations so that it includes our wrapper (actually processed by our Icon transformer).

I fixed the lint errors but it looks like the lint script does not exit with a non-zero code anymore which means CI does not fail when the linter has errors. We might need to bring back concurrently @snowystinger.

I'm not exactly sure how to get the Illustrations built though

It's part of make build


Added IntelliJ Clion Files

Reasons for making this change:
IntelliJ IDE are now getting popularity a lot. CLion is one of their IDE for C and C++. Their project structure contains .idea folder similar to the ones in Android Projects which should be ignored while committing the code.
Links to documentation supporting these rule changes:
https://www.jetbrains.com/help/clion
https://intellij-support.jetbrains.com/hc/en-us/articles/206544839-How-to-manage-projects-under-Version-Control-Systems

Thanks, but see Global\JetBrains.gitignore


samples(pubsublite): add admin samples for lite

Adds admin (create, get, list, update, delete) for Pub/Sub Lite.

cc @tmdiep for review as well

Thanks Alex! Meta comment: should we organize the samples into subpackages, e.g. admin, publish, subscribe (similar to the pubsub samples)?

Thanks Alex.
Btw the tests failed because the Pub/Sub Lite service hasn't been enabled for the project used for the tests.


feat(tls): AWS Libcrypto Support

Motivation
I want to use aws-lc-rs with tonic.
Solution
adds a feature flag tls-aws-lc in parallel to tls, removed tls feature flag dependency from tls-*-roots features

@jenr24-architect could you try to force push again to see if we can get CI to kick off again?

@jenr24-architect could you try to force push again to see if we can get CI to kick off again?

Does it have to be a force push?? It doesn't look like my last push triggered it

With this change, users can now enable both ring and aws-lc-rs, and when they enable both at the same time, users will need to specify which of these they want to use as the default provider. This setting was already an exposed dependency of rustls as an interface for setting the default provider for rustls, but this was not an issue until now as only ring was supported. I think it would be needed to be able to re-expose these in tonic so that users can set which crypto provider to use when both are enabled in order to allow us to use rustls as an internal dependency. It also seems necessary to state that setting the default provider using rustls directly is not intended to be stable for tonic.

With this change, users can enable both ring and aws-lc-rs, and when they enable both at the same time, users will need to specify which of these they want to use as the default provider. This setting was already an exposed dependency of rustls as an interface for setting the default provider for rustls, but this was not an issue until now as only ring was supported. I think it would be needed to be able to re-expose these in tonic so that users can set which crypto provider to use when both are enabled in order to allow us to use rustls as an internal dependency. It also seems necessary to state that setting the default provider using rustls directly is not intended to be stable for tonic.

With this change, users can enable both ring and aws-lc-rs, and when they enable both at the same time, users will need to specify which of these they want to use as the default provider. This setting was already an exposed dependency of rustls as an interface for setting the default provider for rustls, but this was not an issue until now as only ring was supported. I think it would be needed to wrap these configuration APIs in tonic so that users can set which crypto provider to use when both are enabled in order to allow us to use rustls as an internal dependency. It also seems necessary to state that setting the default provider using rustls directly is not intended to be stable for tonic.

With this change, users can enable both ring and aws-lc-rs, and when they enable both at the same time, users will need to specify which of these they want to use as the default provider. This setting was already an exposed dependency of rustls as an interface for setting the default provider for rustls, but this was not an issue until now as only ring was supported. I think it would be needed to wrap these configuration APIs in tonic so that users can set which crypto provider to use when both are enabled in order to allow us to use rustls as an internal dependency. It also seems necessary to state that setting the default provider using rustls directly is not intended to be stable for tonic.

I had thought about this -- I initially had written some code to add ClientTlsConfig::provider(provider: rustls::CryptoProvider) and ServerTlsConfig::provider(provider: rustls::CryptoProvider) so that I could write both a connect_handles_tls_ring and connect_handles_tls_aws_lc integration test, but the scope of that change started to get out of hand.
I would be willing to add that feature, in either this or a separate PR if we're interested in adding that functionality :)

@jenr24-architect looks like there are some more CI failures

@LucioFranco The CI failure does not seem to be caused by this pull request's change. I opened a pull request to address it #2021.

@LucioFranco The CI failure does not seem to be caused by this pull request's change. I opened #2021 to address it.

merged master after #2021 merged, this PR is ready for CI to re-run and to enter the merge queue


HOOBS terminal issues

So I'm trying to get my Roomba synced to the app.  The problem is, when I try to go to the terminal in the Hoobs app, it keeps asking for my username and password.  When I type it in it asks again and I'm stuck in a continuous loop.  When I try to paste cd /home/hoobs/.hoobs/node_modules/dorita980 && npm install && node ./bin/getpassword.js  into my Macbook's terminal, it tells me no file or directory found.  Please help?

Hi Nicolle,
At first i would like to mention, that HOOBS is not the official Homebridge. Its a fork. I would suggest to switch to the official homebride. You find everything on https://homebridge.io/
if you try to Access the Terminal you need a login to the PIs OS, not to Homebridge. The HOOBS default is: username: hoobs; password: hoobsadmin.
An if you open a Terminal on your Mac you type command to that Mac, not the Pi. There is no Roomba Plugin on your Mac. You need ro conntecr to your PI first by typing this command to the Mac Terminal:
ssh<EMAIL_ADDRESS>User and Pass are as above.


TypeCodersTest is never executed

TypecodersTest relies on utils.check_compiled('apache_beam.coders') to skip if cython is not installed. But apache_beam.coders is never compiled (apache_beam.coders.coder_impl is) so this test is never executed.
https://github.com/apache/beam/blob/62904f469f4290546802e03189e5a8faa9af5b7d/sdks/python/apache_beam/coders/typecoders_test.py#L58
Imported from Jira BEAM-14411. Original Jira may contain additional context.
Reported by: bhulette.

Unable to assign user @theneuralbit. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.


feat: Support start/stop slave's replication and create temporary table in database

Description
About database:
Add methods createTemporaryTable/stopReplicating/startReplicating to Database\BaseConnection class.
Add method selectIntoTemporaryTable to Database\BaseBuilder class.
Reset some states in Database\BaseConnection class's __destruct method such as transDepth.
Reason for temporary table: When we use temporary table, we may just want to get a random table name that was not exist, this process should be a packaged method for DB connection instance.  To satisfy the scene that name of temporary table should be defined explicitly, this method also provide a pass in param $name which's default value is null.
Reason for replicating: When we use "Read/ Write Splitting" database, some "write statement" may be required not to be re executed in slaves such as grant powers to a DB user only exists in master.
Reason for reset: Someone like me may run this framework in "Resident Process" such as Swoole, which is necessary to reset it's states before put a connection back to the pool.
About controller:
Pass $this->request, $this->response, Services::logger() to new $this->controller() instead of call $class->initController() in CodeIgniter's createController method.
Call $this->initController in Controller's __construct method (For backward compatibility).
Reason:  __construct method is the official initiator,use it may be more intuitive.
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[X] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

I'm sorry that I can't submit a feature request to the forum because I can't register it successfully. The screen  displays Please solve the reCAPTCHA to verify that you're not a robot.. The console of my chrome browser displays:
Refused to execute script from 'https://forum.codeigniter.com/jscripts/theme-effects.js' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled. @member.php:1
Failed to load resource: net::ERR_CONNECTION_TIMED_OUT. @www.google.com/recaptcha/api.js?onload=captchaOnloadCallback&render=explicit:1

I just checked the registration process and had no problem. The blocked script has nothing to do with Recaptcha - it does not affect the registration process. Please make sure you're not using any adblockers.

One PR should cover only one problem. All changes should be documented and have tests. All breaking changes should be as minimal as possible, see: https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md

initController() is not going anywhere - this is part of the design choice.
"replication" in this shape and form will not be accepted. This is a rather complex topic that should be working automatically. It will require a lot of changes.
temporary tables - I'm not sure about the presented concept, but it should handle all database drivers correctly, which is not the case now.

Please, save yourself some time and post on the forum first.


Added invalid value field error with details

Changes

Added new error type (Invalid Value Error with Details)
Needed for https://github.com/knative/serving/pull/11883

/kind enhancement

/lgtm
/approve


Add HID appearance

Added some common appearance for HID, fix the wrong GAMEPAD appearance code

Thanks for the suggestion! I've added the prefix and updated the example code

Thanks for the PR! I think rather than adding more of these values to trouble, we should start to use the values from bt-hci https://github.com/embassy-rs/bt-hci/blob/main/src/uuid/appearance/categories.rs

Thanks for the PR! I think rather than adding more of these values to trouble, we should start to use the values from bt-hci https://github.com/embassy-rs/bt-hci/blob/main/src/uuid/appearance/categories.rs

wow I didn't know that! Since we already have those values from bt-hci, I'm closing this PR


AGP8.0以上直接适配，无需修改原来写的代码

点此前往！！

how to test or check if the arouter-register is still working after implement your solution?

how to test or check if the arouter-register is still working after implement your solution?

When you delete classpath "com.alibaba:arouter-register:?" and still see the following log: （当你删除 classpath "com.alibaba:arouter-register:?" 还能看到如下日志的时候）
Load router map by arouter-auto-register plugin.

但是我试了直接去掉 arouter的插件依赖，项目运行也没问题，各种跳转也不受影响，不知道为啥

但是我试了直接去掉 arouter的插件依赖，项目运行也没问题，各种跳转也不受影响，不知道为啥

额……你还没懂Arouter的原理。不谈原理直接简单粗暴一点解释就是，使用插件就是把相关的路径信息类找出来并写入安装包内，不使用插件就是在运行时动态的寻找这些类。使用插件的好处就是运行时不用寻找这些类了。ARouter用不用插件都可以跑起来，就看你如何选择了

log

i don't get this console log, but already implemented the class ARouterMatch in my project, should I call it anywhere on my project?

First, if you want to see Arouter's log, make sure you have turned on the switch for Arouter's related logs. Secondly, make sure you have read my adaptation guide in detail and tested it. This guide is a complete adaptation, and no other steps are required.

First, if you want to see Arouter's log, make sure you have turned on the switch for Arouter's related logs. Secondly, make sure you have read my adaptation guide in detail and tested it. This guide is a complete adaptation, and no other steps are required.

First, if you want to see Arouter's log, make sure you have turned on the switch for Arouter's related logs. Secondly, make sure you have read my adaptation guide in detail and tested it. This guide is a complete adaptation, and no other steps are required. In addition, I hope you can at least give a star after benefiting from it, thank you!

I've already replicate your solution, and seems like the code is not call when I run my app

log

i don't get this console log, but already implemented the class ARouterMatch in my project, should I call it anywhere on my project?

First, if you want to see Arouter's log, make sure you have turned on the switch for Arouter's related logs. Secondly, make sure you have read my adaptation guide in detail and tested it. This guide is a complete adaptation, and no other steps are required. In addition, I hope you can at least give a star after benefiting from it, thank you!

I've already replicate your solution, and seems like the code is not call when I run my app

I have described in detail the various possible problems and their principles in the guide, and I published the guide after testing it. If you still cannot succeed, please check whether you have configured the AndroidAOP library correctly according to the access steps. It is not enough to just copy the classes in the guide.


Disable git LFS

Using a mono repo with a 15MB file, we're exceeding 70GB of bandwith per month. Most of these clones have no need for the LFS files. It'd be great if we had an option in git clone to ignore the LFS files and not download them from GitHub. This is similar to the skip submodules feature.

Workaround:
#!/usr/bin/env bash
set -e

echo "" > `which git-lfs`
echo "Git LFS disabled"

Thanks for the feature request @bootstraponline !
Do you have a suggestion, how it should be disabled in this step? E.g. a git flag or similar?

Maybe lfs.fetchexclue = "*" or the -X flag.
https://github.com/git-lfs/git-lfs/issues/227

-X only works with selected git lfs commands, not regular git. It's not supported on checkout.

creating a .lfsconfig in the source dir makes the working directory dirty.

cat << 'EOF' >> ./.lfsconfig
[lfs]
	fetchexclude = *
EOF

Zeroing out the lfs binary seems to be the best solution.
echo "" > which git-lfs``

What do you think about a PR that adds a checkbox to the git clone step that removes the lfs binary when enabled?

Disabling / removing the binary seems to be a bit too dramatic solution to be built into the step, steps should not affect the system as much as possible, as users might run them on their own Mac/Linux using the CLI.

If we go with the .lfsconfig file approach, then I think that'll cause conflicts if someone has that file already checked into the repo.
I'm not sure there's an elegant non-destructive solution. I agree if people are running workflows locally, deleting the binary is not a great solution.

Maybe git lfs uninstall after git clone would work.

Sounds better, but I'm still not sure if this should be part of this, the Git Clone step. Git LFS is not installed or configured by this step, so far there's no functionality in this step related to LFS.

I tested and the filtering stage where LFS downloads happen is skipped.
git lfs uninstall ;\
git "init" ;\
git "remote" "add" "origin" "git@github.com:example/repo.git";\
git "fetch";\
git "checkout" "master"
git "merge" "origin/master"
git "submodule" "update" "--init" "--recursive"

Checking out files: 100% (6239/6239), done.
Branch 'master' set up to track remote branch 'master' from 'origin' by rebasing.

without uninstall:

Checking out files: 100% (6239/6239), done.
Filtering content: 100% (2/2), 23.62 MiB | 5.07 MiB/s, done.
Branch 'master' set up to track remote branch 'master' from 'origin' by rebasing.

What do you think about a PR for that? ^

Git LFS is not installed or configured by this step, so far there's no functionality in this step related to LFS.

Git clone is how LFS is being run though. With the current step, there's no way to clone without LFS activating.

Adding a Script before the Git Clone step with git lfs uninstall would be my current recommendation.

I'll bring this up for a discussion with the team, but for me it seems strange to have an LFS uninstall option/functionality in this step. Still a bit too destructive for local runs..

It sounds like a LFS remove step that runs only on bitrise (and no-ops for local runs) is the best solution. Thanks for your feedback!

In any case I created an internal task for this, we'll do some research to see if there's any other option to disable LFS, either just temporarily while the step runs, or just for the current project instead of for the whole machine - preferably both at the same time, only for the repo and only for the duration of the step :)

It seems like you could check the output of lfs uninstall, if it did anything then run lfs install at the end of the step.

Indeed, something like that might work - we'll see if that's the best option ;)

FYI git lfs uninstall doesn't seem to work after all:
https://www.bitrise.io/build/f485c04ca85bcc7e
probably I'll do something like:
LFS=`which git-lfs`
mv $LFS $LFS.bak
echo "" > $LFS
mv $LFS.bak $LFS

Ping me if you find a better way. 😄

Ohh my.. I definitely will 😂

One of the LFS core devs shared this:

git config --global lfs.fetchexclude '*'`

That should disable LFS fetching. https://github.com/git-lfs/git-lfs/issues/2919#issuecomment-374393246

Awesome, thanks for the info @bootstraponline !
Did you verify that it works if you run it as a Script step before the Git Clone one?

Did you verify that it works if you run it as a Script step before the Git Clone one?

yes

I made a step and it's working: https://github.com/instructure/steps-disable-git-lfs

Looks great, thanks for sharing @bootstraponline ! ;)


TypeError: Cannot convert undefined or null to object

To test this, only first line in input.csv was kept. The following message/errors come.
I use node v12.11.1; lighthouse:v5.6.0.

Run 1 of 3: URL 1 of 1
Caught error for https://www.acehardware.com:
TypeError: Cannot read property 'message' of undefined
Start run 2
Run 2 of 3: URL 1 of 1
Caught error for https://www.acehardware.com:
TypeError: Cannot read property 'message' of undefined
Start run 3
Run 3 of 3: URL 1 of 1
Caught error for https://www.acehardware.com:
TypeError: Cannot read property 'message' of undefined
(node:15028) UnhandledPromiseRejectionWarning: TypeError: Cannot convert undefined or null to object
at Function.values ()
at getOutput (E:\MyDocuments\VisualStudio\WebDevelop\DevTools\Lighthouse\multihouse-master\src\index.js:240:33)
at E:\MyDocuments\VisualStudio\WebDevelop\DevTools\Lighthouse\multihouse-master\src\index.js:213:37
at
(node:15028) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:15028) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Thanks for filing the issue! I'm traveling at the moment, but I'll take a look when I get back to the office.

Thanks again for the heads up.
This  commit should fix the problem you describe.
The problem AFAICT was that I was using an older version of Lighthouse, for which results.runtimeError was defined, even if results.runtimeError.message wasn't.
If you can, please try pulling the update and confirm that the fix works for you.

No comments, so I'm assuming this is fixed.
Feel free to reopen if necessary.
Thanks again!

That's great — thanks for letting me know.
On Wed, 30 Oct 2019, 06:41 Ruby<EMAIL_ADDRESS>wrote:

Thanks. It works now. Sorry for later to test it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/samdutton/multihouse/issues/1?email_source=notifications&email_token=AABSDKWRXLNYYP2AJIZKCOLQRET75A5CNFSM4JEAF6A2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECTCM4I#issuecomment-547759729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABSDKSAKD6JHNJT6D37FIDQRET75ANCNFSM4JEAF6AQ
.


how to bind app.service property to the view?

Hi, I'm trying to use app.service to get/set global loading state.
in app.html I print the loadState, {{loadState}} which supposed to be bound to appState.get().loadState

  loading : {{loadState}}

    <router-outlet></router-outlet>

app.ts
export class App{

  constructor(public appState: AppState) {
    this.appState.set('loadState', false);  //initial value.
  }
  get loadState() {
    return this.appState.get().loadState;
  }
}

however I'm getting this error: EXCEPTION: Expression 'loading: {{loadState}}    in App@0:23' has changed after it was checked. Previous value: 'false'. Current value: 'true' in [loading: {{loadState}}    in App@0:23]
the error is clear, but I can't figure out how to fix it? am I using app.service the wrong way?

I'm going to change the angular2-hmr api and remove the getter so it's not confusing

sorry this may sound super rule but I'm closing long standing issues and refactoring the codebase. If you run into this problem again feel free to open another issue


make Sentry work for us

[ ] why do we see a lot more crashes?
[ ] how do we get a quick overview if we have problems or if everything looks ok

@kkuepper
We should probably separate the different platforms into separate Sentry projects.
I think it makes sense in our case, as we have different versions of the app written in different languages etc.
So instead of every error going into bcc-media-platform, we have separate projects like this:

bcc-media-flutter
bcc-media-tvos
bcc-media-web

This will help a lot with tracking and filtering down issues.
Any arguments against it?

Crashes vs Errors
We don't seem to see a lot more crashes in Sentry; we simply see a lot more events overall.
Sentry picks up all exceptions and errors, while Crashlytics is more directed towards application crashes and what caused them.
We can configure the level of events to track in Sentry: https://docs.sentry.io/platforms/flutter/configuration/options/#diagnostic-level
Setting this to a higher level might help reduce some noise.
Browser errors
Based on the issues logged in Sentry, browser errors makes up 23 of the 40 unresolved issues in the bcc-media-platform project at the time I am writing this.
That is almost 60% of all issues reported; more than tvOS and Flutter combined.

projects:

bccm-mobile
bccm-tvos
bccm-web
live-mobile
live-tvos
live-web
live-androidtv
kids-mobile
kids-web
bmm-mobile
bmm-web
chromecast

bccm
live
kids

@kkuepper The Android TV app is just the Flutter app, which means we probably cannot send TV events to the androidtv Sentry project. At least not easily, since Sentry is initialized at startup while checks for Android TV is done at runtime

@kkuepper The Live AndroidTV app is just Flutter, so has same errors as the Android app. So I think it will not be necessary with a separate project in Sentry for that.

Everything is now connected to Sentry, except the BMM apps 🎊


add update_display_from_props methods to Node and Edge

This avoids cloning props unnecessarily.
Related to #172

Thanks! This coul be reasonable, but while we do no have stable api, I would avoid merging this.


Seach hotkey

Fixes #1054 (At least partially)
Adds a new keybinding to toggle Search, and gives focus to the input when pane is opened. Unfortunately it's not very practical to focus the input outside the view as it would require storing the input ref in the state or a global. So instead the hotkey just toggles the pane. We can revisit this once I've changed the focus mechanism so it's controlled bottom-up.

/azp run


I got an error and i dont know what to do

I used the save editor to modify some stuff like my perks and my character level, and i got this error when i tried loading my save...
TypeError: Error #1009: Il est impossible d'accéder à la propriété ou à la méthode d'une référence d'objet nul.
at global/classes.Engine.Map::InShipInterior()[E:\Build\Flash\TITS3\classes\Engine\Map\InShipInterior.as:25]
at classes::TiTS/showLocationName()[E:\Build\Flash\TITS3\includes\game.as:109]
at classes.DataManager::DataManager/loadBaseData()[E:\Build\Flash\TITS3\classes\DataManager\DataManager.as:1875]
at classes.DataManager::DataManager/loadGameData()[E:\Build\Flash\TITS3\classes\DataManager\DataManager.as:1701]
at classes::TiTS/buttonClick()[E:\Build\Flash\TITS3\classes\TiTS.as:813]

Is your game the latest version.

Is your game the latest version and what format do you use air or swf

Attach your save and I might be able to figure out what is wrong.
Most likely you added one of the debug items though.

The debug shield generator works on the latest (as of about a week ago), but the tachyon laser gives me an error.

It was the tachyon laser, thanks for your help!


Redis ACL (user/pass) support?

Is it currently possible to utilize a redis username AND password when authenticating to Redis?
The documentation mentions the ability to pass a password, but quite a few years back REdis becan supporting true ACLs, meaning you can now authenticate with a username AND a password.
I'd like to know if i pass like: "OAuth2username" or "username" or "OAuth2redisusername" - will it be able to authenticate with a username and password?

yes, see: https://github.com/OpenIDC/mod_auth_openidc/blob/v<IP_ADDRESS>/auth_openidc.conf#L686-L689

As a follow-up question (and apoloiges if this is super basic) - how does one provide any of the redis arguments:
Looking at: https://github.com/OpenIDC/mod_oauth2/blob/aa178d4bf115207a28517616919b0abe7a796388/oauth2.conf#L27-L57
would i do something like:
OAuth2Cache redis host myhost port 1234 password mypassword
Or:
OAuth2Cache redis
OAuth2Cache host myhost
OAuth2Cache port 1234
OAuth2Cache password mypassword
Or:
OAuth2Cache redis host myhost
OAuth2Cache redis port 1234
OAuth2Cache redis password mypassword
I've tried googling but the internet doesnt seem to have examples either. I see the doc defines the syntax as: #OAuth2Cache  []
But that seems a bit ambiguous (maybe just to me?)

it is a query-encoded format, as documented - with samples - in:
https://github.com/OpenIDC/mod_oauth2/blob/aa178d4bf115207a28517616919b0abe7a796388/oauth2.conf#L68-L73
I notice that since the order of the statements in the sample changed (OAuth2Cache needs to be specified before others), this is less than obvious now, I'll improve that
so OAuth2Cache host=myhost&port=1234&password=mypassword
there are also a few examples in the README.md which is probably what most people go with


Gradle not resolving after compiling rainbow hat

Error:Execution failed for task ':app:preDebugAndroidTestBuild'.

Conflict with dependency 'com.android.support:support-annotations' in project ':app'. Resolved versions for app (24.2.0) and test app (25.4.0) differ. See https://d.android.com/r/tools/test-apk-dependency-conflicts.html for details.

Can you post the line in your gradle file you're using to import the
rainbow hat?
On Sun, Dec 17, 2017, 3:10 AM soorajshankar<EMAIL_ADDRESS>wrote:

Error:Execution failed for task ':app:preDebugAndroidTestBuild'.
Conflict with dependency 'com.android.support:support-annotations' in
project ':app'. Resolved versions for app (24.2.0) and test app (25.4.0)
differ. See
https://d.android.com/r/tools/test-apk-dependency-conflicts.html for
details.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/androidthings/contrib-drivers/issues/85, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADI58_LG4rCxBB0LEdrNqRRXtmUA-iehks5tBPa2gaJpZM4REns5
.

I fixed that by removing expresso from dependency .
But seems some version mismatching is there.
BTW this was the way I imported rainbowhat

compile 'com.google.android.things.contrib:driver-rainbowhat:+'

also tried with older version 0.7 and all

Min SDK version 26
target SDK version 26 (tried with 24 as min also then things was not compatible)

Looks like Rainbow Hat uses 24.2.0 of Android Support Annotations, but only for testing. I don't know why it would conflict with your version.

I think Android Support Annotations are defined as Implementation but not Test Implementation
I doubt that could be a problem if I use another support library with different version

Yes you're right it isn't defined as testImplementation, but I do wonder why this library is using it. It probably should be updated or perhaps removed.
But implementation shouldn't pass the dependencies through AFAIK.


Revert "Revert "Merge branch 'modprobe-fix' of git://github.com/abbradar/nixpkgs""

Revert a revert of a merge that shouldn't have been in master but was intentionally in staging.
Next time I'll do this right after the revert instead of so far down the line...
This reverts commit 9adad8612b082bcbae30c81678a04b79a44079a4.

Looks good to me.  Apparently we now have the commits of PR #17738 twice (at least in the staging branch), that's why they do not need to be reverted in this PR...

Yep, the proper way would have been 1. revert bad merge into master 2. immediately merge master into staging 3. immediately revert revert in staging.

Yeah, well, git doesn't offer a "true unmerge". At least not yet...


Fix error in narrative for CodeableConcept

There was an error in the html-template for the narrative of a CodeableConcept. It displayed 'true' instead of the value.

Coverage decreased (-0.009%) to 73.008% when pulling fee70b1a7a704af83366c68dc0dbe999262ba91a on RuthAlk:fix_codeableconcept_narrative into 9fd9d196785c7e8bed5c572728341079fa959d17 on jamesagnew:master.

Thanks for the PR! Merged.


Add ALUMET

Insert URLs to the project here: https://github.com/alumet-dev/alumet

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All new projects listed will be posted on our Mastodon channel.

@TheElectronWill @titouanj @AngeCyp


Occurs on single post entry pages (example: http://demo.ghost.io/welcome/)
mobile phones with small screens (example iPhone 5 SE)
burger menu for mobile break point would be nice

Technical details

Casper Version: 2.1.2
Ghost Version: current version of Ghost(Pro)
Browser Version: 11.0
OS Version: iOS 11.0

Hey @zeitrechnen 👋
I'm closing this as it's a duplicate of #344
tl;dr: the theme nav is scrollable


Alt text, titles, etc

Need to update....right now they are lame

Update all app-specific alt text for figures
Check alt-text for top level 3-panel figure


refactor: disable tests that depend on GTF release

Issue #, if available:
Description of changes:
Why is this change necessary:
Comment out tests in the template that depends on network steps as the steps implementations depend on a new release of GTF.
How was this change tested:
Any additional information or context required to review the change:
Checklist:

[ ] Updated the README if applicable
[ ] Updated or added new unit tests
[ ] Updated or added new integration tests
[ ] Updated or added new end-to-end tests
[ ] If your code makes a remote network call, it was tested with a proxy
[ ] You confirm that the change is backwards compatible

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Just do a git revert on the offending change.


Feature: "Condensed mode" in structured list

May be a legacy feature, closing unless we see otherwise.


Disabling highlights and use underline

If possible, how could I change the highlight behavior to not highlight errors nor warnings? I prefer to have them underlined to avoid visual noise.
The following is an image of my Nvim setup. The Ale plugin is enabled with highlights disabled/hidden i.e:
let g:ale_disable_lsp = 1
let g:ale_sign_column_always = 1
let g:ale_sign_error = ''
let g:ale_sign_warning = ''
let g:ale_set_highlights = 0

highlight clear ALEErrorSign
highlight clear ALEWarningSign

The error is due to a multi-line string, yet to be commented. I'd like for everything to remain the same except for colored highlights to become colored underline.

Apologies in advance if this is the wrong forum for such a question.

Are you using diagnostic-nvim? If you are, this setting controls virtual text https://github.com/nvim-lua/diagnostic-nvim#enabledisable-virtual-text . It looks like diagnostic-nvim also has a setting to use underlines. It should be on by default. https://github.com/nvim-lua/diagnostic-nvim#enabledisable-underline .
If you're using ale on it's own I don't see a way to disable virtual text. (That might be a good issue over there.) Potentially you could make this time delay really large though as a workaround.

I am using diagnostic-nvim. The below image was taken using the following settings:
" Diagnostic nvim configurations
let g:diagnostic_enable_virtual_text = 1
let g:diagnostic_virtual_text_prefix = ''
let g:diagnostic_trimmed_virtual_text = '20'
let g:diagnostic_sign_priority = 20
let g:diagnostic_enable_underline = 0
let g:diagnostic_insert_delay = 1 " Dont show diagnostics while in insert mode

Note the ALEDisable command was ran prior to creating some_fn. I was in insert mode while testing with some_fn the g:diagnostic_insert_delay = 1 prevented the virtual text from displaying but not the highlighting.

Found the culprit. nvim-treesitter plugin was highlighting text while I was typing. Adding highlight link TSError Normal fixed the mentioned issue.


feat(util): Add WeakDictionary

copilot:all

Current dependencies on/for this PR:

main

PR #4109   👈

This comment was auto-generated by Graphite.


CODE RUB: NullCompletionException Class Update v2.10.1

Updated NullCompletionException class by removing constructors in which the error message was hard coded in. Updated all relevant service files to include message.

Closed Pull Request. Resolved in PR #557.


🎁 prepare release 0.6.0

Note: I've bumped from 0.5.1 to 0.6.0 due to upgrading wasmtime to a new major version (10.0.0)
Cutting a new release, this currently contains steps 1 through to 5 of Releasing Viceroy.

Below are the steps needed to do a Viceroy release:

Make sure the Viceroy version has been bumped up to the current release
version. You might need to bump the minor version (e.g. 0.2.0 to 0.3.0) if
there are any semver breaking changes. Review the changes since the last
release just to be sure.
Update the Cargo.lock files by running make generate-lockfile.
Update CHANGELOG.md so that it contains all of the updates since the
previous version as its own commit.
Create a local branch in the form release-x.y.z where x, y, and z are
the major, minor, and patch versions of Viceroy and have the tip of the
branch contain the Changelog commit.
Run make ci locally to make sure that everything will pass before pushing
the branch and opening up a PR.
After you get approval, run git tag vx.y.z HEAD && git push origin --tags.
Pushing this tag will kick off a build for all of the release artifacts.
After CI completes, we should publish each crate in the workspace to the
crates.io registry. Note that we must do this in order of dependencies. So,
(cd lib && cargo publish)
(cd cli && cargo publish)
Now, we should return to our release PR.
Update the version fields in lib/Cargo.toml and cli/Cargo.toml to the
next patch version (so z + 1).
Update the dependency on viceroy-lib in cli/Cargo.toml to the next
patch version.
Update all the lockfiles by running make generate-lockfile.
Restore the ## Unreleased header at the top of CHANGELOG.md.
Get another approval and merge when CI passes.

This looks fine, but the fact that the Ubuntu CI job is failing is worrying. Any ideas? (If this is a known flake in our CI, I can approve, but I'm hesitant to hit approve with a CI job failing.)

@acfoltzer the release has happened, this is now ready for final reviews before merging


dead temporary pets do not despawn

Temporary pets captured through necromancy do not despawn when killed.
This means any new pet resurrected with necromancy stays hostile on resurrection.

despawn coroutine was not properly nulled when stopped


how where can i get adtag

hi
how and where can i get adtag ?

I used @MTProxybot but i got this error
Incorrect secret value. It must contain 32 hex characters like
00112233445566778899aabbccddeeff.
because my secret code was 60 characters, so i use 32 characters of zero in the bot and i got this tag
Success!
Your proxy has been successfully registered. You can now pass this proxy tag to the software you are using: 30e5d73419aa335e0116060c665b8b70.
Here is a link to your proxy server: https://t.me/proxy?server=17...1*5&port=443&secret=00000000000000000000000000000000
but my sponsor channel is not shown
this is my config:
PORT = 443
name -> secret (32 hex chars)
USERS = {
"tg":  "00000000000000000000000000000001",
# "tg2": "0123456789abcdef0123456789abcdef",
}
MODES = {
# Classic mode, easy to detect
"classic": False,
# Makes the proxy harder to detect
# Can be incompatible with very old clients
"secure": False,

# Makes the proxy even more hard to detect
# Can be incompatible with old clients
"tls": True

}
The domain for TLS mode, bad clients are proxied there
Use random existing domain, proxy checks it on start
TLS_DOMAIN = "www.google.com"
Tag for advertising, obtainable from @MTProxybot
AD_TAG = "30e5d73419aa335e0116060c665b8b70"


Eliminate ContinueWith from Renderer.cs. Fixes #6385

This keeps the event handler disposal logic on the renderer's synchronization context, which should eliminate the hard-to-repro IndexOutOfRangeException reported in #6385.
As we've noted before, the logic in Renderer.cs is pretty intricate (methods rely on knowing details about the differing sync/async and error handling expectations of other methods, which you can only know about by reading the code extensively, hence all the comments). Hopefully we will improve on this in our planned refactoring.

@javiercn You're right. This was a poor implementation - thanks for pointing it out. I've improved this in #9635.


database for socio-economic data

make a sqlite database with roughly the following schema

one table with person characteristics that are fixed over time, ie gender, birth year, municipality of birth, .... create table person_background as [...]
 create unique index idx_pb_person on person_background(rinpersoonid)

feel free to use a better/shorter column name for rinpersoonid

one table with annual data, ie income. unique index on (rinperson, year)
a table with marriages. I think it should have rinpersoon1, rinpersoon2, and background info on the marriage, ie marriage date/year/place. index on (rinperson1, rinperson2).

use following convention for index names: idx_a_b, where a is an abbrevation of the table name on which the index is created, and b indicates which columns constitute the index

@dakota0064 , https://github.com/odissei-lifecourse/life-sequencing-dutch/commit/f75afd657ea6e0edefae2768131d591592d3103e creates a separate db for each table, is that correct? I thought we could have all tables in the same single sqlite database? or is there a reason to keep them in separate files?

In my past work I kept tables as separate files because the data was structured in a way that was convenient. But there's no benefit to doing that here, so I will combine them all together.

Going to close this as we now have the background info in one database.


mysql/json: support comparison

Hi,
This PR init comparison for type json.   It's based on (https://github.com/pingcap/tikv/pull/1874) ,we could start to review  once PR-1874 has been merged:

[x] https://github.com/pingcap/tikv/pull/1874

Meanwhile, the related PR in tidb is  (https://github.com/pingcap/tidb/pull/3307) and related source file is compare.go
@BusyJay @hicqu @andelf  PTAL

@hicqu @andelf @hhkbp2 PTAL

LGTM

@hicqu @hhkbp2 @lishihai9017 PTAL,THX

Rest LGTM

LGTM

LGTM.


Provide APIs to set active (hovered) and tooltip elements.

Chart.setActiveElements will set the hovered items.
Chart.tooltip.setActiveElements will set the tooltip items.

Resolves #2850

Nice one! I'll review tomorrow.


ROADMAP

I have been mostly prodding at this GitHub repository to see what Blurtooth can do - https://github.com/oesmith/gatt-xml/blob/master/
Some notable services that Enviro could integrate:

org.bluetooth.service.current_time and org.bluetooth.service.reference_time_update - since it's outside the scope of Enviro BLE to use WiFi, perhaps we can set the time from a host via Bluetooth so sensors can periodically sleep
org.bluetooth.service.automation_io - generic digital/analog IO access (trigger a remote plant watering event on Grow for example)
org.bluetooth.service.battery_service - battery monitoring, I think this is self-explanatory
org.bluetooth.service.alert_notification - is this a sensible way to broadcast watering events, sensor low/high triggers?

Some general questions:

How tolerant can a Bluetooth client be to devices sleeping/waking on a schedule to save battery?
Would we need some way to notify the server device that we're satisfied with our readings, and that it can go back to sleep?
Are there any bugs in Enviro (https://github.com/pimoroni/enviro/issues) that need fixed here?

Are there any features we should incorporate? Bearing in mind the KISS principle here.

BLE Beacon?
Can pico W powered ble be used to create beacons for rough positioning? I'm thinking from a robot point of view, the enviros I use are typically in fixed positions from each other, so if I could use the time of flight of a Bluetooth signal I should be able to work out how far a robot has travelled. Or I might be talking out of my kevsrobots branded hat.

Something about indoor positioning was definitely in the XML nightmare I was browsing in lieu of braving the specs - https://github.com/oesmith/gatt-xml/blob/master/org.bluetooth.service.indoor_positioning.xml. I did wonder if it would be useful for anything. I don't think it costs much in terms of RAM to throw up another few services/characteristics. Especially if they don't require ongoing updates.
I guess devices that use these for positioning rely on the signal strength and some kind of location tagging- looks like latitude and longitude which might be a bit of an ask for household positioning. Though if all your boards have a unique ID (the Pi's flash serial number) and you know roughly where they are (fixed locations) then there might already be enough information to do triangulation from just the "device information" characteristic without having to load the devices down with even more services.
Definitely some research required here! But the linked XML file gives some insight into what kind of data a location beacon would share. Stuff like floor number makes sense to me, Lat/Lon seem somewhat optimistic. I guess that's why there's a confidence value 😆


Basin area discrepancy between NLDI and NHD

I found two USGS stations where the area of the NLDI's delineated basin geometry is way off compared to the total drainage area provided by NHD. The IDs are:

USGS-01446000: NLDI area is 307 sqkm whereas in NHD it's 95 sqkm
USGS-11295230: NLDI area is 502 sqkm whereas in NHD it's 76 sqkm

USGS-01446000 has a stated drainage area of 36.7 sqmi in NWIS which matches NHD.

This looks like it's got the wrong major river.
USGS-11295230 has a stated drainage area of 29.3 sqmi in NWIS which matches NHD.

This looks OK to me... not entirely clear what would be wrong in this case.
@cheginit -- where are you deriving the NHD identifiers that link gages to the network from?

@dblodgett-usgs I got the associated NHD IDs of the station from the Gage layer of the NHDPlus (MR). From this layer, I extracted the NHD areas and IDs of all CONUS stations and put them in a gist.

When we add up drainage area, we ignore diversions but when we delineate a basin polygon we include all network connections (diversions included).

That's an interesting distinction between the basin polygon and drainage area.

Not sure there's really anything to do here. Reopen if you want to follow up.


Shortcodes

Just wondering how to enable Shortcode parsing in Content Blocks using Shortcodeable module

Assuming you have both modules installed: If you have a block with an HTMLText db field and WYSIWYG editor form field, a shortcode button will appear on the WYSIWYG editor toolbar.
If you'd like Block objects to be Shortcodable (embed blocks inside HTMLEditorField contents), you'll need to add the Shortcodable interface to your Block subclasses ie. class MyBlock extends Block implements Shortcodable See readme at https://github.com/sheadawson/silverstripe-shortcodable for details on that.


Basic ValidJSON Question

Using the TypeScript library. How do I pass a schema to ValidJSON? I tried passing a schema or metadata.schema property in my data. Also trie ValidJSON({schema}) and nothing in Cursor fix with TypeScript is working.

Example code:
Eval("Extract Claims", {
  data: () => {
    return [
      {
        input: claim,
        expected: 1,
      },
    ];
  },
  task: async (input) => {
    return extractClaims(input);
  },
  scores: [ValidJSON],
});

Think I got it with scores: [ValidJSON.partial({ schema: schema })],

Yep partial should solve it! Were you able to sort it out?

Thanks. Technically no, any schema I pass seems to be ignored. Still testing but I'm not seeing it be used.

Yup, I'm good. Thank you!


Add a JS script (backp.js) that automatically backs up /etc/hosts

@vindu939 no, there is no need of interval scheduled backups, as of now we just need a file that would backup the hosts file to a provided location whenever we run the script. Afterwards we'll integrate the backup.js with blocker.js such that whenever blocker.js is called, automatically backup will be done.
Also, for now just keep the location for backup in a variable (location can be of your choice), we can change it later on


Feature Request: STDERR redirected through websockets

It seems this program swallows stderr output,
./websocketd --loglevel=debug --port 8080 --devconsole -- perl -E'$|++; say for 1,2,3,4; warn $_ for 7,7,7; die 42;'

Yet when I check out ws://localhost:8080 I don't see the messages for 7,7,7 or 42 anywhere...
open
ws://x1c7:8080/
onopen

onmessage
1
onmessage
2
onmessage
3
onmessage
4
onclose
[Clean: false, Code: 1006, Reason: none]
send »

Is there a way to tell websocketd to wrap the stream in JSON and tell us what stream it is, or something of the like?

It seems this was brought up here https://github.com/joewalnes/websocketd/issues/315

What you're looking for goes to log (also it's not printing 7, since warn does not uses $_ implicitly) :

How would you expect things to behave here?

We discussed that before besides #315. Normally you can just send your stderr to stdout and read all of the lines in Javascript.
Prefixing them or changing how they sent in the socket would go into territory of messing with program's output and it's not something websocketd should be concerned with.

Yes, it does go into the log which is better than nothing, but that's not really great if you're building a terminal interface to a shell program. Your users are not going to want stderr silently dropped or logged remotely. They're going to want it made available to the end user on web sockets.
Just consider an repl/eval bot/ or anything else.
I'm not arguing against the log. I'm just saying it would be nice to not have to ad-hoc ship them back, and to have access to this stream right away.

Perhaps it reminds me #141; there could be middleware that mixes stderr/stdout

@EvanCarroll : might not be exactly what you're looking for but I ended up working on some sort of generic wrapper which outputs 3 json-based events (stdout, stderr & exit). See https://github.com/joewalnes/websocketd/issues/402


sublime3: make sublime text work with pkexec and gksudo

When sublime doesn't have rights to write a file, it asks for root rights by calling either
/usr/bin/gksudo /bin/cp TMP_FILE FILE
or
/usr/bin/pkexec /bin/cp TMP_FILE FILE
whichever it finds first.
Paths /usr/bin/pkexec, /usr/bin/gksudo and /bin/cp are all hardcoded in sublime_text binary. /bin/cp can be simply replaced by cp in binary itself (and padded with necessary amount of zero bytes), but to deal with /usr/bin/pkexec and /usr/bin/gksudo we have to use LD_PRELOAD trick to intercept glibc calls.
Tested it locally with system="i686-linux" and system="x86_64-linux".
cc @wmertens.

LGTM, but why not rewrite the paths in binary like cp?
Note that I use Atom now, so if you would like to set yourself as
maintainer that would be great…
On Wed, Oct 21, 2015, 18:43 Demin Dmitriy<EMAIL_ADDRESS>wrote:

When sublime doesn't have rights to write a file, it asks for root rights
by calling
/usr/bin/pkexec /bin/cp TMP_FILE FILE.
(it also attempts to call gksudo, but we don't have gksudo in nixpkgs)
Paths /usr/bin/pkexec and /bin/cp are hardcoded in sublime_text binary.
/bin/cp can be simply replaced by cp in binary itself (and padded with
necessary amount of zero bytes), but to deal with /usr/bin/pkexec we have
to use LD_PRELOAD trick to intercept glibc calls.
Tested it locally with system="i686-linux" and system="x86_64-linux".
cc @wmertens https://github.com/wmertens.
You can view, comment on, or merge this pull request online at:
https://github.com/NixOS/nixpkgs/pull/10517
Commit Summary

sublime3: fixed pkexec paths

File Changes

M pkgs/applications/editors/sublime3/default.nix
https://github.com/NixOS/nixpkgs/pull/10517/files#diff-0 (10)
M pkgs/build-support/libredirect/libredirect.c
https://github.com/NixOS/nixpkgs/pull/10517/files#diff-1 (21)

Patch Links:

https://github.com/NixOS/nixpkgs/pull/10517.patch
https://github.com/NixOS/nixpkgs/pull/10517.diff

—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/10517.

--
Wout.
(typed on mobile, excuse terseness)

The reason why we can't just rewrite strings in binary is that if you change the length of the string then you'll essentially shift all bytes after that point. I believe most, if not all executable formats including ELF are very sensitive to this kind of change and it will corrupt file (most likely you'll get SIGSEGV).
But usually strings in binary are C-strings, so we can shorten them by replacing unnecessary bytes with \0. So I changed /bin/cp to cp and leave pkexec/gksudo to figure out actual file path by searching in  PATH.
We can't do the same with gksudo/pkexec because sublime_text tries to make stat syscal before calling them and stat requires full path. But we can't rewrite /usr/bin/gksudo to full path (/nix/store/.../bin/gksudo) because it's too long.
I know, it's all dirty tricks, but there is not so much that you can do with compiled binary.

Alright, I added myself as a maintainer.

Indeed, that all makes sense. One more request: could you convert this PR into two commits, one for the libredirect change and one for using it with sublime? Then I'll merge. We like pretty commits in nixpkgs :)

Done.

:beers:


v2.4.0-MOD_1.3 does not work with OnePlus6T/LineageOS-microG, while v2.3.1-MOD_3.0 worked

Describe the bug
v2.3.1-MOD_3.0 works on an OnePlus 6T with LineageOS 20, next versions do not work
To reproduce
Steps to reproduce the behavior:

OnePlus 6T
LineageOS 20-20230704-microG-fajita
Magisk 26.1 with zygisk enabled + MagiskHide Props Config 6.1.2-v137
device fingerprint set to OnePlus6T:11
if I install  safetynet-fix v2.3.1-MOD_3.0 all checks pass (with BASIC auth, even if I didn't force it via props)
If I install latest version I get "Basic Integrity" and "CTS Profile does not match" errors (it tries HARDWARE_BACKED, even if I force BASIC auth via props)
if I reinstall  v2.3.1-MOD_3.0 all checks pass again

Expected behavior
Also latest version works
Screenshots of SafetyNet & Play Integrity API tests result
Testing SafetyNet:

Screenshot_YASNAC_failure_with_v2.4.0-MOD_1 3
Screenshot_YASNAC_success_with_v2.3.1-MOD_3 0

Testing Play Integrity API: I don't have Google Play store installed
Device info
Device model: OnePlus 6T
Android version: 13
ROM name/version: LineageOS 20-20230704-microG-fajita
Logs
issue.log
Additional context
n/a
Checklist

[x] I confirm that the SELinux status is Enforcing on my device.
[x] All information is present
[x] Screenshots are attached
[x] Logs are attached

Use version 1.3-microG


Update README.md

Minor text updates.

Doesn't need updating from our side. This is Realms code base, PRs should be made directly to them by forking here https://github.com/solana-labs/governance-ui


Last update 0d72a89...cce5503


Update to latest version in pip

It seems that the latest version via pip is 1.5.0
please bump to latest version

+1 This would be really nice

+1 (and maybe add a field version to the package)

+1, would love this.

For anyone else coming across this issue: For now, the best way to install the latest version, which includes some essential updates, appears to be directly from the git repo.
This worked for me:
pip install -e git+git://github.com/yoavaviram/python-amazon-simple-product-api.git#egg=amazon-simple-product-api

This will install the lxml and bottlenose dependencies, but will not install the required dateutil package.  So you also need:
pip install python-dateuitil

Done. Appologies for the delay, was extra busy recently.


Taxon Name autocomlete - prioritize single cap with species

Like:
"A bus"
"C eu"

When I do search on sandstorm for
'N fenestratus' and 'N. fenestratus' I get different results


adding .local options and broadcast

To supersede #45 and close #47

I think this fixes the spirit of #45 in documentation.


CORS and Custom Headers

Hello,
First of all thank you so much for this great framework. I really appreciate the fact that the library is divided in multiple modules and that we can handle what the server does. This is way better than Microsoft WebAPI for sure.
Now concerning my problem. I am using Nancy to implement a REST API. This API should allow cross domain requests.
Everything is working fine with normal webrequests. But if I add a custom header to my query, Nancy totally ignores the request and I never receive any event informing that a request has been received by Nancy.
Please find more information below concerning my code.
Here is my NancyModule :
public AuthModule() : base("/auth/")
{
    Before += ctx =>
    {
        return null;
    };

    Post["/login"] = x =>
    {
          return new Response { StatusCode = HttpStatusCode.OK };
    };

The before event is never raised if a custom header is added to the WebRequest.
In my bootstrapper I override the RequestStartup to initialize the CORS headers as followed:
pipelines.AfterRequest.AddItemToEndOfPipeline(ctx =>
{
        ctx.Response.Headers.Add("Access-Control-Allow-Origin", "http://localhost:8080");
        ctx.Response.Headers.Add("Access-Control-Allow-Methods", "POST,GET,DELETE,PUT,OPTIONS");
        ctx.Response.Headers.Add("Access-Control-Allow-Credentials", "true");
        ctx.Response.Headers.Add("Access-Control-Allow-Headers", "Accept,Origin,Content-type,MY_HEADER");
        ctx.Response.Headers.Add("Access-Control-Expose-Headers", "Accept,Origin,Content-type,MY_HEADER");
});

As you can see in the 2 last lines, I add a custom header to be available in cross domain requests.
I then use the stateless authenticate as followed:
StatelessAuthentication.Enable(pipelines, configuration);

Ajax WebRequest :
$.ajax({
                beforeSend : function(xhr) {
                  xhr.setRequestHeader('MY_HEADER', 'my value');
                },

                method: httpMethod,
                url: "http://localhost:8081/auth/login",
                data: data,
                dataType: 'json',
                accept: {
                    json: 'application/json'
                },
                async: true,

                success: function (result) {
                    if (successCallback != null) {
                        successCallback(result);
                    }
                },
                error: function (result) {
                    if (failureCallback != null) {
                        failureCallback(result);
                    }
                },

                crossDomain : true,

                xhrFields: {
                    withCredentials: true
                }
            });

If the line xhr.setRequestHeader('MY_HEADER', 'my value'); is removed the request is received by Nancy. If a header is added the request is never received. Or at least I never get any event raised.
When a header is added this result in the following : "XMLHttpRequest cannot load http://localhost:8081/auth/login. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access."
Finally here are the headers displayed in Chrome:
// Request Headers
Accept:*/*
Accept-Encoding:gzip, deflate, sdch
Accept-Language:fr,en-US;q=0.8,en;q=0.6
Access-Control-Request-Headers:accept, content-type, my_header
Access-Control-Request-Method:POST
Cache-Control:no-cache
Connection:keep-alive
DNT:1
Host:localhost:8081
Origin:http://localhost:8080
Pragma:no-cache
Referer:http://localhost:8080/
User-Agent:Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.81 Safari/537.36

// Response Headers
Allow:OPTIONS, TRACE, GET, HEAD, POST
Content-Length:0
Date:Thu, 28 May 2015 19:55:45 GMT
Public:OPTIONS, TRACE, GET, HEAD, POST
Server:Microsoft-IIS/10.0
X-Powered-By:ASP.NET
X-SourceFiles:=?UTF-8?B?QzpcVXNlcnNcVGhvbWFzXERldmVsb3BtZW50XEhlcmVBbmRUaGVyZVxIZXJlQW5kVGhlcmUgLSAwLjFcSGVyZUFuZFRoZXJlLldlYkFwaVxhdXRoXGxvZ2lu?=

You can see that no Access-Control-Allow-Origin is present in the response headers.
I greatly appreciate your help.
Linvi

Interesting! Lots of informations!

Hi phillip,
Any idea what could be going wrong?
I don't know Nancy well enough to know how to retrieve ALL the WebRequests received by the server.
Thanks for looking into it.
Linvi

I'm still at work so I haven't looked into it yet. Just happy you gave a lot of information. I'll take a closer look this weekend to see if I can reproduce and figure out whats going on.

This stack overflow answer looks like it might be what you need.

Hi,
Thanks for your answer.
I already looked at this post (and tried it). The problem here is that the WebRequest is never handled by Nancy. This code is never invoked and yet Nancy returns a status code of 200 without the additional headers that this solution or mine try to add.
Cheers,
Linvi

Hi here,
Any luck with identifying what could be going wrong with this issue?
Cheers,
Linvi

My flatmate has my old laptop and I can't buy my new one because apple has no stock. :( so I'm waiting for him to get home so I can steal the laptop for an hour.
Sent by Outlook for Android
From: linvi
Sent: Monday 1 June 5:30 pm
Subject: Re: [Nancy] CORS and Custom Headers (#1947)
To: NancyFx/Nancy
Cc: Phillip Haydon
Hi here,
Any luck with identifying what could be going wrong with this issue?
Cheers,
Linvi
—
Reply to this email directly or view it on GitHub.

Sorry @linvi, I just don't have a laptop at the moment with OSX to test on :(
I am stealing it for the weekend tho because I'll be traveling to Thailand so I'll try reproduce then.

Hi,
No worries, but just to be clarify, I am not reproducing this issue on OSX, I have this issue on Windows 8.1.
Cheers

Ahhh I've confused this with another bug I'm trying to reproduce with Xamarin on OSX.
SORRY!
I'll try and look at this tonight then.

Any luck?

I haven't been able to reproduce it yet. Still trying. I only got to spend an hour on it last night.

Hi, Thanks for taking the time to look into it. Do you want me to create a repro project?

It you could that would be great. Sorry I am in Thailand.
Sent by Outlook for Android
On Mon, Jun 8, 2015 at 3:01 AM -0700, "linvi"<EMAIL_ADDRESS>wrote:
Hi, Thanks for taking the time to look into it. Do you want me to create a repro project?
—
Reply to this email directly or view it on GitHub.

I'm still awating for this!

I guess it's because you didn't handle preflight sent by the browser.

I was able to pass through CORS using the code below in the Configure method:
`
    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public virtual void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
    {
        loggerFactory.AddConsole(Configuration.GetSection("Logging"));
        loggerFactory.AddDebug();

        // Shows UseCors with CorsPolicyBuilder.
        app.UseCors(builder =>
           builder.WithOrigins(new[] { "http://localhost:4200" }).AllowAnyHeader()
                    .AllowAnyMethod()
                    .AllowCredentials()
           );

        app.UseOwin(a => a.UseNancy(b => b.Bootstrapper = new NancyBootstrap(this.Container)));

    }

`


Error when running flash_attn_func

I'm trying to run inference with flash attention and I'm getting this error.
from flash_attn import flash_attn_func
import torch

def main():
    batch_size = 8
    seqlen_q = 1
    seqlen_k = 239
    nheads = 9
    d = 32
    device = "cuda"
    causal = True
    window_size = (-1, -1)
    dtype = torch.float16
    torch.random.manual_seed(0)
    q = torch.randn(batch_size, seqlen_q, nheads, d, device=device, dtype=dtype)
    k = torch.randn(batch_size, seqlen_k, nheads, d, device=device, dtype=dtype)
    v = torch.randn(batch_size, seqlen_k, nheads, d, device=device, dtype=dtype)
    out = flash_attn_func(q, k, v, 0.0, causal=causal, window_size=window_size)

    # print("Final Output: ", out)

main()

Error:

@tridao, pytest is failing for test_flash_attn_causal under test_flash_attn.py

Looks like an installation error. Please make sure to uninstall the old version completely before installing the new version of flash-attn.


[DOC] User routes

Details

Added user routes documentation as mentioned in issue #469.
Changed tag name from Auth to User.

Screenshots

Closing issues
#469

@Anmolbansal1 can you please check this ?


Show menu bar on macOS

When menu bar is disabled on macOS it's not possible to copy, paste etc. I suppose it was disabled because it looked ugly on other OS. On macOS it doesn't look bad because menu bar is included in top bar - see screenshot:

Just brilliant, thank you for that ☺


Add org-cite to :tools/biblio

Sorry; because of the massive recent merges, and my shaky git skills, I had to redo #5212 to get the git correct.
Summary
This adds support for native org-mode org-cite citations. Specifically it:

configures the native org-cite processors
adds the oc-bibtex-actions processor for the vertico module, with "follow", "activate", and "insert" capabilities
~adds and configures the new org-ref OC rewrite: https://github.com/jkitchin/org-ref-cite~ (removed for now because of unclear future)
installs citeproc-el, which is required for the oc-csl export processor, and will be needed also for forthcoming preview functionality

Should bump org along with this, as this depends on a recent change in org (namely that now loads oc-basic, while it didn't initially; so I removed that use-package! statement).
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=745f0913fc153322f5abb11aba377415106bd97a
Issues and Questions
What to do with #2888?

Drop org-ref from there (can revisit in the future).
Merge this.
That can rebase on this and just focus on org-roam-bibtex.

On org-ref specifically:
It's status has evolved quite a bit over the last few months.
Its author initially published a development repo for an org-ref-cite package, which would have fit beautifully here, and which I initially added.
But since then he basically dropped that and in effect reimplemented aspects of org-cite in an incompatible syntax in org-ref v3.
In the process he removed some of the problematic dependencies and such, and it is possible to use org-cite (for citations) with org-ref (for its other functionality).
It's also possible to use org-ref for all of this, and ignore org-cite.
IMO this was a bad move from the broader org POV, and puts developers, users, and projects like Doom in an awkward situation. An explanation is included in the README, the first part I agree with, the second which I think is just personal opinion that many will not agree with.
I personally don't think we should support it and focus instead on native org citations. But others may have reasonable differences of opinion.
Keybindings?
This is tricky.
We need to bind org-cite-insert and org-open-at-point, as these are the two entry points for the org-cite module.
But how, and where?
For now, I have added a binding for org-cite-insert.
But org-open-at-point is also more general, so probably should be in more than one place.
A biblio submenu would also be helpful given the above, and that bibtex-actions (for vertico) has multiple entry points, and a keymap to access them (which is also used by embark).

FYI @iyefrat @tecosaur I closed the other PR and opened this one to get a clean history with the latest develop commits.

While this is ready for input on the "issues/questions", I'm reverting to draft as, after talking to its author, I'm going to include at least one additional third-party processor: org-ref-cite.

So psyched for this, thanks for how much work you have put into it.

With https://github.com/hlissner/doom-emacs/pull/5290/commits/ffdad5920571ab444b3fc07766fc2d1c9bc3403c I've reverted addition of org-ref-cite because of its unclear future.
The linter is complaining, so I will likely to need to rebase all this.

It's just my suggestion, but I think it's reasonable
Maybe store all of the keybindings in SPC-m-R space? it would stand for 'reference' - it's not ideal, but it's the best I can think of since SPC-m-c, SPC-n-r and SPC-m-r are taken
Another option is SPC-m-z as it has one less keystroke, but it has no mnemonic (z like Zotero maybe?)

@pgronkievitz what is SPC-m?
I personally use the prefix n b (like note -> bibliobraphy) which is by default bibtex-actions-open-entry, so repurposing that might be an idea as well
(map! :leader
      (:prefix "n b"
       (:prefix "a"
        "a" #'bibtex-actions-add-pdf-attachment
        "l" #'bibtex-actions-add-pdf-to-library)
       ("c" #'org-cite-insert)
       ("d" #'bibtex-actions-dwim)
       (:prefix "i"
        "b" #'bibtex-actions-insert-bibtex
        "c" #'bibtex-actions-insert-citation
        "k" #'bibtex-actions-insert-key
        "p" #'bibtex-actions-insert-preset
        "r" #'bibtex-actions-insert-reference)
       ("I" #'org-cite-insert)
       (:prefix "o"
        "o" #'bibtex-actions-open
        "e" #'bibtex-actions-open-entry
        "l" #'bibtex-actions-open-link
        "n" #'bibtex-actions-open-notes
        "p" #'bibtex-actions-open-pdf)
       ("r" #'bibtex-actions-refresh)
       ("R" #'bibtex-actions-read)))

@pgronkievitz what is SPC-m?

yeah, I probably should use a bit better wording. What I meant is to use "localleader" instead of "notes" prefix, because "notes" are available globally and those commands are useful exclusively (or at least mostly) inside org file, so in my opinion it doesn't make any sense to have access to e.g. #'bibtex-actions-insert-bibtex in dashboard.

What I meant is to use "localleader" instead of "notes" prefix, because "notes" are available globally and those commands are useful exclusively (or at least mostly) inside org file, so in my opinion it doesn't make any sense to have access to e.g. #'bibtex-actions-insert-bibtex in dashboard.

It's actually more complicated than this though.

some commands only make sense in org; namely org-cite-insert
some commands (arguably) only make sense in latex or markdown (the bibtex-actions insert commands)
but it can be useful to browse outside these modes, which is why the original binding for ivy-bibtex and helm-bibtex is what it is (global)

I've been playing around with this PR, since I'm very interested in moving away the fairly bloated org-ref package. While I've been able to insert citations and open PDFs with org-cite just fine, one issue I've been having is getting bibtex-actions to play nicely with the existing notes in my roam folder. I set bibtex-actions-notes-path to my roam folder, where I have an existing set of notes for citations of the form bibtexkey.org. When I try to access the citation notes through Embark Act, I get the error find-file-noselect: Wrong type argument: stringp, nil. Any advice on fixing this?

When I try to access the citation notes through Embark Act, I get the error find-file-noselect: Wrong type argument: stringp, nil. Any advice on fixing this?

It's likely a bug. Can you report it on the bibtex-actions tracker?
While you do, can you report if bibtex-actions-open-notes works correctly? My suspicion is not; that i'ts not an embark issue.

Sigh ... something went wrong with git.

Could you please make another PR to fix the name change of your package if you have time? it breaks doom-cli since there is no bibtex-actions any more.

I think we recent changes, can probably remove that entirely.

Even better!

Except, as I think about it, does it make any practical difference?
The command only makes sense in an org buffer. And if one opens an org
buffer, it should load the binding.
Right?

@bdarcus I'm no longer sure which of my two messages your comments are referring to.

@bdarcus I'm no longer sure which of my two messages your comments are referring to.

Let me move it to the thread it belongs, now that i"m a computer.

We need to bind org-cite-insert and org-open-at-point, as these are the two entry points for the org-cite module.
But how, and where?

How about adding a citation clause to +org/dwim-at-point (which is already on RET in normal mode and org already binds C-c C-o to org-open-at-point)?
       (while (and context (memq type '(verbatim code bold italic underline strike-through subscript superscript)))
         (setq context (org-element-property :parent context)
               type (org-element-type context)))
       (pcase type
+        (`citation
+         (org-cite-follow context arg))
+
         (`headline
          (cond ((memq (bound-and-true-p org-goto-map)
                       (current-active-maps))
                 (org-goto-ret))

As for inserting a citation, considering org-cite-insert is now built into org, I think we can add a localleader key to the :lang org module for it, perhaps under the i prefix with the other insert commands.
That said, I'm to redesign our org module after the CLI rewrite. It's much overdue, so I'm fine with deciding on a temporary keybind for now.

We need to bind org-cite-insert and org-open-at-point, as these are the two entry points for the org-cite module.
But how, and where?

How about adding a citation clause to +org/dwim-at-point (which is already on RET in normal mode and org already binds C-c C-o to org-open-at-point)?

That seems sensible at first glance (I wasn't aware of that function), though you'd also want it available for citation-reference.
With citar, I have left-click on the mouse set to run org-open-at-point, and right-click to run embark-act. The org-cite-activate-processor attaches a map with those and other bindings to the citations.

As for inserting a citation, considering oc is now built into org, I think we can add a localleader key to the :lang org module for org-cite-insert, perhaps under the i prefix with the other insert commands.

Makes sense.

That said, I'm to redesign our org module after the CLI rewrite. It's much overdue, so I'm fine with deciding on a temporary keybind for now.

I have no strong opinion, but lean towards deferring to the rewrite. I think it's fine as is for now.

Oh sorry @hlissner - I just bumped the module again to incorporate a minor typo.

Sorry for the incredibly long turnaround time, but I'm happy this is finally merged. Thanks again for all your help!


feat: fallback implementation for the providers

Currently, we don't have a fallback in-case the provider fails or rate limits us.
We either need:

a fallback
a warning notice on our docs that this provider may be unreliable

Resolved by #624


[DOCS] - Design Brief: Beta Release

Working release. See commit details.

Screw it, i'll add more design references later


embedding layer

Hi,
What is the easiest way to extract the embedding layer so we can get a  768-dim or x-dim for every molecule ?

Hello @rohanvarm. The model is a pure decoder-only transformer, trained without any representation learning objective, so I will not recommend using the embeddings for any molecular prediction task.
Nevertheless, using molfeat, it's easy to have a new MolecularTransformer to extract the representation.  I have pushed a detailed notebook here as an example: https://github.com/datamol-io/safe/blob/529003341ea75a50c172a9592307b1cc2f46b595/docs/tutorials/extracting-representation-molfeat.ipynb
I would recommend using one of the many pretrained models in  molfeat instead !
Best.

Hi, I see that does make sense. What about some sort of a vector DB consisting of say all of zinc, could you not use it then as the discriminator? I'm just trying to think of potential use-cases beyond molecular design

If your goal is something like similarity search or some form of fragment-based virtual screening, this might actually be quite promising.
I suspect, finetuning with some predictive tasks could also enhance the representation quality or lead to fair predictive models with the added benefit of providing fragment-based explainability.  Those are still speculations that I would personally like to investigate when time allows.
Should you experiment with any of these ideas or try something different, I would be grateful if you could share your findings with me.


Fire button not removing URL

This issue has popped up again. When using the fire button, the "flame" effect trips, but the url is not removed and can be accessed using the Go button (bottom right).
DDG v<IP_ADDRESS>
iOS 17.6.1 (21G93)
Originally posted by @mattlikespie in https://github.com/duckduckgo/iOS/issues/2435#issuecomment-2292626582

Hi @mattlikespie - Can you check this on a more recent version please?  Please re-open if you're still seeing it.  Thanks!


Suggestion to upgrade using pnp cli tools don't support 1.11

In the "tip" box that has the text If your application is built using an older version of the SharePoint Framework, you can easily upgrade it to SharePoint Framework v1.11 or newer using Office 365 CLI., the linked page for the pnp tool doesn't yet support 1.11.
Text on the linked page reads:
The spfx project upgrade command helps you upgrade your SharePoint Framework project to the specified version. If no version is specified, the command will upgrade to the latest version of the SharePoint Framework it supports (v1.10.0).

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

ID: 06d649b5-f4bc-ffd9-191b-ab7e758d9a54
Version Independent ID: c02df482-fcbf-18be-cc15-485ec1d3379b
Content: Prepare your SharePoint Framework application for publishing to the marketplace
Content Source: docs/spfx/publish-to-marketplace-checklist.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Thanks... this will be true when SPFx v1.11 ships & the corresponding O365 CLI update ships for the v1.11 release.
@waldekmastykarz FYI


Installation problems - cannot add token/edit settings

Since you were able to close this issue same day I assume you figured out the problem.  I too am unable edit the token and I'm wondering what your solution was.  I'm on OSX El Capitan
Thanks

n/m  - I was able to copy only the token setting into my User Settings and it worked.  Couldn't edit the default settings.
`{
// Your GitHub API token
// see: https://github.com/condemil/Gist#generating-access-token
"token": "#######################################"
}`


add QGIS2Leaf plugin

Can you add it to a proper category?

The PR has been rebased. I created a new section under "3rd party integration" named "GIS integration". What do you think?

@paulovieira lets not create categories just for a single plugin. Moreover the 3rd party integration category is pretty short.

Somehow I feel it belongs to the "Overlay formats" (AKA vector formats) section.

Yes, the "Overlay data formats" is a better place for this plugin. The PR has been updated.
Great work with plugin organization @IvanSanchez!


docs howto run in production has an incomplete sentence

At

Do not increase the number of Hypercorn workers per instance. Prometheus Python client metric export, as implemented, will not work in multiprocessing scenarios. The supported method of scaling is via increasing

the rest of the sentence is missing,

If there's a better place to point to for instructions on launching a production instance, please let me know.

If there's a better place to point to for instructions on launching a production instance, please let me know.

That section is intended to be the reference for that.  Will update it in today’s release.
There is also configuration that uses AWS EKS/K8s/possibly Terraform (?), it is located in another repository (https://github.com/ietf-ribose/bibxml-infrastructure/). It is maintained by a colleague, but I did not do a close audit. For my part as BibXML core developer I try to guarantee the approaches covered in this repository only (i.e., via Docker/Docker Compose, no EKS/K8s).

That sentence is supposed to say:

The supported method of scaling is via increasing the number of running containers.

In other words, for the “web” service, there should be one Hypercorn worker within a container, but there can be any number of containers.
That said, it is important that Prometheus instance is aware of all containers, if you use Prometheus. Every running container must be scraped for metrics to ensure metrics are accurate. (Otherwise, if e.g. an RFC request is load-balanced to web container B, but Prometheus only gets metrics from web container A, then Prometheus will not learn about that request.)
Meanwhile, for the “celery” service, neither workers nor containers should be scaled.

(https://github.com/ietf-ribose/bibxml-service/commit/9e523cc916bfe3f48f5e95db55c4e88553621cdd)

A lot of what is in that last comment should actually be in the documentation. Yes, we will use prometheus.
There are links at the top of the document that do not resolve.
Overall, as currently written, this page feels more as notes to someone who already understands the deployment rather than instructions to an operator unfamiliar with the project for how to deploy. Please adjust it (or build another page) focusing on that audience.

Overall, as currently written, this page feels more as notes to someone who already understands the deployment rather than instructions to an operator unfamiliar with the project for how to deploy.

Originally, it was not given what the exact preferred way of operating the service will be, the requirement I was going off of is that it must be delivered as a set of Docker containers. Thus, the goal of the documentation was to keep it open as to how these containers can be run but provide advice useful for any environment (not tying to any particular provider or deployment method).
As a side note, the container reference part of documentation may be helpful. It breaks down the individual sub-services that comprise BibXML service. Each service is a Docker container specified in the Compose file, and these containers can be run in whichever preferable way of running containers (as long as the containers can talk to each other, which Docker Compose ensures they can).
I’ll to add a section that clarifies this and describes in a bit more detail one possible way of deploying the service.

A lot of what is in that last comment should actually be in the documentation. Yes, we will use prometheus.

That part is already in documentation in a highlighted admonition block, I sort of paraphrased it in my comment.

I have updated production documentation as follows:

https://dev.bibxml.org/static/docs/howto/run-in-production.html is now a proper how-to describing a minimal production setup guide, as it should’ve been.
https://dev.bibxml.org/static/docs/topics/production-setup.html covers various aspects of production setup.


DO NOT MERGE: nightly releases with fabric8/josdk nightlies

This PR is meant to produce nightly builds using the latest nightly builds of the fabric8 client and JOSDK, produced version is 999.F8-SNAPSHOT.

@metacosm I think this is now handled in the next-fabric8-version branch right?

Yes, this is a PR that's open to track that branch.


Formatting errors in table of contents for examples.

Title: Formatting errors in table of contents for examples.
Moderator: @martinjuckes
Requirement Summary: (1) Numbering of index items is not consistent with target examples (e.g index entry 7.8. Climatological seasons links to Example 7.9. Climatological seasons) , (2) examples 7.1 and 7.15 have no index entry, (3) Section 7.5 Geometries (including example 7.15) is included in the source document but omitted from the web page and pdf document.
Technical Proposal Summary: (1) Edit toc-extra.adoc to correct numbering of index items, (2) Correct an apparent error in defining the anchor for example 7.15 in ch07.adoc.
Benefits: Corrected index and inclusion of missing sub-section.
Status Quo: Index is incomplete and inacuurate.
Detailed Proposal:

The numbering of the entries for examples 7.2 to 7.14 need to be corrected in toc-extra.adoc (currently set to 7.1 to 7.13);
New entries need to be added to toc-extra.adoc for examples 7.1, 7.15 and 7.16 and for table 7.1;
In ch07.adoc correct [["timeseries-with-geometry"]] to [[timeseries-with-geometry]].
The web page is more than a year behind the latest version in github .. missing Geometries section : is this intended?

To avoid potential conflict (unlikely) with formatting changes proposed in https://github.com/cf-convention/cf-conventions/pull/161, these changes are probably best tackled after that request is accepted.

Dear @martinjuckes
Thanks for raising this issue, the best part of four years ago, that identifies a number of errors. I suspect no-one responded at the time because of the possible interaction, which you highlighted, with https://github.com/cf-convention/cf-conventions/pull/161. If you have a few minutes available, it would be helpful if you could say whether any of these errors still remain now. Perhaps they were corrected by another issue at some point.
Best wishes and thanks
Jonathan


Update fonts, icons and file types from seti-ui version 1.6.1

Hi,
like your say in comment here: PR#44. I create PR with updated fonts, icons and file types from newest version of seti-ui.
Cheers,

Awesome @wadik12, thank you!

I looked on on opened issues about wrong size of icons. Maybe it's fault of changes with horiz-adv-x in seti.svg? I cannot check this, because I'm using OSX here and Ubuntu in work and in both looks and works fine.


Compiling arm run time libraries based on newlib selects inferior memcpy-stub.c instead of memcpy-armv7a.S

Hi,
I am using the 18.1.3 release of LLVM and LLVM-embedded-toolchain-for-Arm. Additionally I use newlib-4.3.0.
I am building the following library variants for a baremetal arm 32bit cortex-a9 target on ubuntu.
For reference I have attached the git patch with the changes I need to setup above.
LLVM-embedded-toolchain-for-Arm.patch
add_library_variants_for_cpu(
armv7a
SUFFIX hard_neon
COMPILE_FLAGS "-mfloat-abi=hard -march=armv7a -mfpu=neon"
MULTILIB_FLAGS "--target=armv7-none-unknown-eabihf -mfpu=neon"
QEMU_MACHINE "none"
QEMU_CPU "cortex-a8"
QEMU_PARAMS "-m 1G"
BOOT_FLASH_ADDRESS 0x00000000
BOOT_FLASH_SIZE 0x1000
FLASH_ADDRESS 0x20000000
FLASH_SIZE 0x1000000
RAM_ADDRESS 0x21000000
RAM_SIZE 0x1000000
STACK_SIZE 4K
)
add_library_variants_for_cpu(
armv7a
SUFFIX thumb_hard_neon
COMPILE_FLAGS "-mfloat-abi=hard -march=armv7a -mfpu=neon"
MULTILIB_FLAGS "--target=thumbv7-none-unknown-eabihf -mfpu=neon"
QEMU_MACHINE "none"
QEMU_CPU "cortex-a8"
QEMU_PARAMS "-m 1G"
BOOT_FLASH_ADDRESS 0x00000000
BOOT_FLASH_SIZE 0x1000
FLASH_ADDRESS 0x20000000
FLASH_SIZE 0x1000000
RAM_ADDRESS 0x21000000
RAM_SIZE 0x1000000
STACK_SIZE 4K
)
My problem is that the run-time libraries seems to select an inferior memcpy in newlib called:
memcpy-stub.c.
I Basically determined it to be inferior during tests of memcpy.
When testing our current setup with gcc and newlib based run-time libraries memcpy is almost 6x faster compared to an identical test with clang and newlib based run-time libraries.
This is looks to be because our gcc build run-time libraries select #include "memcpy-armv7a.S" instead of the memcpy-stub.c selected in our clang build run-time libraries.
I have identified why memcpy-stub.S is selected as follows:
memcpy-stub.S is selected by
repos\newlib\newlib\libc\machine\arm\memcpy.S
Basically because
repos\newlib\newlib\include\arm-acle-compat.h is called with __ARM_ARCH defined.
When arm-acle-compat.h is called with __ARM_ARCH defined then
repos\newlib\newlib\include\arm-acle-compat.h does not define __ARM_FEATURE_UNALIGNED.
When  __ARM_FEATURE_UNALIGNED is not defined
repos\newlib\newlib\libc\machine\arm\memcpy.S does not
#include "memcpy-armv7a.S"
but instead includes
memcpy-stub.c
I have not been able to determine for sure who defines __ARM_ARCH which is key to all of it.
Therefore I am hoping someone here in the forum knows if the above behavior is on purpose or how I can setup my run-time libraries to use memcpy-armv7a.S

The __ARM_ARCH macro is defined by the compiler in https://github.com/llvm/llvm-project/blob/main/clang/lib/Basic/Targets/ARM.cpp#L740
The __ARM_FEATURE_UNALIGNED is defined by the compiler in https://github.com/llvm/llvm-project/blob/main/clang/lib/Basic/Targets/ARM.cpp#L787 but only when -munaligned-access is selected.
So it looks like you may need to add -munaligned-access to the compilation flags for the library. I'm not sure at the moment whether adding it to the multilib flags will work as I'm not sure unaligned access can be used as a parameter to select multilib on right now.
For a general toolchain, if we only have one library variant for unaligned access, I think we'd want to compile without unaligned access for maximum compatibility (many systems disable unaligned access).

I have tried setting the -munaligned-access, it betters things since we are not running the implementation in memcpy-stub.c anymore then.
However now libc.a contains an

libc_a-aeabi_memcpy-armv7a.o, that DOES NOT use the arm vectorization instructions, and
libc_a-memcpy.o, that DOES use the arm vectorization instructions.

When compiling our source code with the above run-time library (libc) using clang, clang somehow maps our memcpy calls to __aeabi_memcpy (libc_a-aeabi_memcpy-armv7a.o). This is an issue because __aeabi_memcpy DOES NOT use the arm vectorization instructions and hence it is slower than it could be if it exploited the  arm vectorization instructions.
I have raised the issue in the llvm forum see https://discourse.llvm.org/t/setting-mcpu-cortex-a9-mfpu-neon-for-arm-target-does-not-make-clang-pick-memcpy-optimized-for-the-co-processor/79336/5


Adapt deprecated ActionList (semantics & focus)

What are you trying to accomplish?
Some accessibility issues in the deprecated Primer React ActionList component have been discovered during the Primer React SelectPanel component accessibility engineering review, and since the SelectPanel uses the component in the body.
This PR is to fix the focus and semantics for the deprecated Primer React ActionList component.

https://github.com/github/primer/issues/1045

What approach did you choose and why?
Use useFocusZone hook in deprecated Primer React ActionList component
Screenshots
N/A
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Hi @bolonio, are there any changes that we should carry to the new ActionList?

Hi @bolonio, just curious, are there any changes here that we should carry to the new ActionList?

Hi @siddharthkp, I totally forgot to close this PR. The changes to the deprecated/ActionList will be done here: https://github.com/primer/react/pull/2138
The changes to the NEW ActionList are here: https://github.com/primer/react/pull/2126
You can follow the progress in those two PRs. Closing this one.


fix(): delete docs 2.0.0-rc.test

Delete files rc.test;
Change version api files to rc.0;
Change menu of rc.test to rc.0.

Thanks @ramonornela! You rock :metal: