Skip calling Windows console api with piped stdin

Fixes #50
I implemented the changes discussed in the issue. I also wrote a test that fails on Windows in the current master branch and passes after the changes, but it feels a bit hacky as it just runs one of the examples...

Looks like the 1.33.0 version of rustc used by appveyor doesn't understand the temporary borrow in the test. Adding braces around it like so
{
    let stdin = out.stdin.as_mut().unwrap();
    stdin.write_all("secret".as_bytes()).unwrap();
}

or upgrading the appveyor version to 1.44.0 (or whenever this was fixed) should fix it.

@Heliozoa Feel free to upgrade Rust version to 1.44 in appveyor & travis configs, that'll make things easier. If people want to upgrade, they'll just have to use 1.44 too.

:+1 on my end, and nice going on the test.

Awesome, thank you both! I'll go ahead and merge. Then I'll publish to crates.io. Because users need to potentially upgrade their version of Rust, it is a "breaking" change so it'll be a major version increase.

Just saw that there is another PR I need to merge before publishing to avoid multiple major version increases. I'll get around to doing this the coming weekend.
Until then, I encourage you to use the git way of defining the dependency:
[dependencies]
rpassword = { git = "https://github.com/conradkleinespel/rpassword", rev = "189c388" }


Apex callback refactoring

Fixes #4199
Proposed Changes

libhpx no longer contains any APEX symbols.
Any libraries or plugins or executables that do depend on APEX will still link with it
APEX now only has one dependency (or two, if HPXMP is used) - libhpx

Any background context you want to provide?
This eliminates circular dependencies between APEX and HPX, and is the first step to making APEX a module in HPX.

Thanks @khuck! Let me try to summarize your changes to see if I've understood them correctly:

HPX now exposes an external_timer class which allows registering callbacks for various events. HPX doesn't call anything from APEX directly.
APEX still depends on HPX the usual way. However, it is built as a shared library which initializes the callbacks through a global object in the APEX shared library. Dependent projects need to link to the APEX library, but this is handled through target_link_libraries(hpx INTERFACE apex) so that it's transparent to users.

Thanks @khuck! Let me try to summarize your changes to see if I've understood them correctly:

HPX now exposes an external_timer class which allows registering callbacks for various events. HPX doesn't call anything from APEX directly.
APEX still depends on HPX the usual way. However, it is built as a shared library which initializes the callbacks through a global object in the APEX shared library. Dependent projects need to link to the APEX library, but this is handled through target_link_libraries(hpx INTERFACE apex) so that it's transparent to users.

Yes, that sums it up nicely.  I'm addressing your review comments and will update the branch in a few...

@msimberg ok, I addressed your comments and fixed two inspect errors that slipped through.  let me know if I need to address anything else...  the failing tests on (not required) APEX build are lated to apex policies not exiting at shutdown, those have been failing for a while now and aren't related to this PR.  I will have to fix them eventually, of course.

I have no idea what Codacy is complaining about - those union members are used in the cpp file

@khuck I marked the Codacy 'errors' as resolved an re-triggered the analysis. All should be fine now.

Excellent, thanks!


ci: use intermediate env var in run

SEC-36
Using github variables in a GHA run step introduces a run shell injection vulnerability.
Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[ ] Assign members of your team to review the pull request.
[ ] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.
[ ] Wait until the pull request has been reviewed and approved by a teammate
[ ] After pull request is approved, and you determine it's ready add the label "Ready to merge" to the pull request and it will automatically be merged.

I fixed version name by moving it to run.


isFunctionLike cases contains duplicates

for example:
FunctionExpression is checked for on line 4547 and also 4559

thanks @Connoropolous for reporting this. should be fixed in master now.

No problem. Speedy fix!


Iox time series functions

Please let me know if I have missed any functions.  I know gapfill is not yet implemented.

Please ping me again when @sanderson 's comments have been addressed. I don't want to end up reviewing content that is rewritten

I asked Andrew about some of the temporal functions I left out and while they do work, I am not sure what the use case would be for some, with the exception of extract.  But I can certainly include them.

I asked Andrew about some of the temporal functions I left out and while they do work, I am not sure what the use case would be for some, with the exception of extract. But I can certainly include them.

I agree extract  has a usecase.

@alamb I am a little confused on the to_timestamp function usage
This works:
SELECT to_timestamp(time)
FROM cpu
LIMIT 1
However, the format in Datafusion's docs state this should work and I am getting errors:
basic syntax:  to_timestamp(timestamp, format)
SELECT TO_TIMESTAMP('2023-01-01 08:50:10', 'YYYY-MM-DD')
FROM h2o_feet
SELECT TO_TIMESTAMP('2020-01-31T09:26:56.123Z', 'YYYY-MM-DD')
FROM h2o_feet
Am I on the wrong track here?

Hi @lwandzura

However, the format in Datafusion's docs state this should work and I am getting errors:
basic syntax: to_timestamp(timestamp, format)

The docs I can find don't refer to the second argument (though they are also not very helpful in general):
https://arrow.apache.org/datafusion/user-guide/expressions.html#temporal-expressions
I don't think the to_timestamp function supports a second argument currently in DataFusion -- can you copy the exact error you are seeing?

When I run this I see
❯ SELECT TO_TIMESTAMP('2020-01-31T09:26:56.123Z', 'YYYY-MM-DD');
Plan("Coercion from [Utf8, Utf8] to the signature Uniform(1, [Int64, Timestamp(Nanosecond, None), Timestamp(Microsecond, None), Timestamp(Millisecond, None), Timestamp(Second, None), Utf8]) failed.")
❯

Which says to me that the function requires a single argument of one of the type in that list (which is a pretty bad message)


Docker Compose v2.19.0 needs to be patched to v2.19.1 to fix "container not connected to network" errors

Description
I'm writing this on behalf of my customer who filed an issue where they were seeing this error:
Error response from daemon: container 7377e46299f8ca5ba254a9491d390cf9599f86cfd299d505a120271017850627 is not connected to the network kind

Through their investigative work, they found we pushed a new base image last week, pushing docker to 2.19.0. Docker released a bug fix for this specific error in v2.19.1:

https://docs.docker.com/compose/release-notes/#2191

Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[X] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230625.1
Is it regression?
yes
Expected behavior
The container should connect to network during compose up job.
Actual behavior
Error that occurs during workflow run:
Error response from daemon: container 7377e46299f8ca5ba254a9491d390cf9599f86cfd299d505a120271017850627 is not connected to the network kind

Repro steps

Set up Docker Composer services which connect to a Kind docker network
Make sure the runner-image is up to date and using Docker Compose v2.19
Run an action which attempt to establish connection to the network and you may see this error (the bugfix notes this is sporadic)

Duplicate https://github.com/actions/runner-images/issues/7857

Hello @cgaudette,
I will close this issue as a duplicate of https://github.com/actions/runner-images/issues/7857.
Feel free to reach out in case of concerns


Double bucket creation

When starting serverless, I noticed that serverless-s3-local try to create bucket two times.
Serverless: starting handler
Serverless: S3 local started ( port:5645, family: IPv4, address: <IP_ADDRESS> )
Serverless: creating bucket: s3-storage-dev
error: Error creating bucket. Bucket "s3-storage-dev" already exists
info: PUT /s3-storage-dev 409 12ms -
offline: Starting Offline: dev/eu-west-1.
offline: Offline [http for lambda] listening on http://localhost:3002

serverless.yml
[...]
plugins:
  - serverless-offline
  - serverless-s3-local
[...]
custom:
  serverless-offline:
    httpPort: 3006
  s3: # Local S3
    host: localhost
    port: 5645
  additionalStacks:
    s3: ${file(.cloudformation/s3.yml)}
[...]

.cloudformation/s3.yml
Resources:
  StorageBucket:
    Type: AWS::S3::Bucket
    DeletionPolicy: ${self:custom.deletionPolicy.${self:provider.stage}, self:custom.deletionPolicy.other}
    Properties:
      BucketName: s3-storage-${self:provider.stage}
      AccessControl: Private

My guess is when configuring S3rver in index.js l.221 with configureBuckets, which describes all buckets configuration, also it creates them.
Then creating them again in the run function triggers the error index.js l.232.

Hi @HuEric
Thanks for your contribution! I agree with what you said. I fixed this issue by referencing your comment. Please try the latest version (v0.6.3) via npm.


learning 'jvrc_walk' and  'jvrc_step'

I tried to learn using 'jvrc_walk' and  'jvrc_step'.
The result of the leaning of 'jvrc_walk' is good, but  'jvrc_step' is poor.
I think good learning needs other conditions.
Could you please advise me of conditions you recommend?

Hi, can you give more details about what do you mean by "poor"?
jvrc_step is supposed to work out of the box (unless I made some inadvertent changes)

I attached 2 video files, both files are learned by jvrc_step.
The last poses of the humanoids of the both files are falling down.
Please take a look at both videos.
https://user-images.githubusercontent.com/23301778/231667558-020a1dbf-6f2b-4b7f-957e-828ac906ba59.mp4
https://user-images.githubusercontent.com/23301778/231667584-fb6b22d7-c8f5-4b5a-8f6e-ca29790a36a7.mp4

I can't play any of your videos: "Video can't be played because the file is corrupt"
Did you try the pre-trained weights provided here? What happens in that case?

The location you suggest is Uploads are disabled.
More over, this input box can't support extension .pt file type.

I meant to say that: did you try playing the pre-trained weights provided in this repository? The pretrained weights are provided here and the instructions to play are here.

The pretrained weights provided by you are certainly correct.
Then, what is the difference between your pretrained weights and my ones?

Then, what is the difference between your pretrained weights and my ones?

Last time I checked training jvrc_step should work totally fine. I would like to retrain again with the latest commit and verify if everything works as expected, but that might take a few days. Sorry about that.

Thank you for your reply. I attached 2 video files, both files are learned by jvrc_step. The last poses of the humanoids of the both files are falling down. Please take a look at both videos.
https://user-images.githubusercontent.com/23301778/231667558-020a1dbf-6f2b-4b7f-957e-828ac906ba59.mp4 https://user-images.githubusercontent.com/23301778/231667584-fb6b22d7-c8f5-4b5a-8f6e-ca29790a36a7.mp4

Hi sir,
I am wondering the ckpt of this video is trained by yourself or the authors'? I train the step version myself, but the humanoid can not climb the stairs.

对不起，我犯了一个错误。您预先训练的体重效果很好。
您提供的预训练权重当然是正确的。那么，你们的预训练权重和我的权重有什么区别呢？

请问下,您犯了一个什么错误,我也是到最后一步就摔倒了,请问下您是怎么解决的呢?


[No QA] Update webpack path for apple app association

Details
This fixes the path for the apple-app-association file in webpack, which is causing GH actions failures on desktop
Fixed Issues
Related to https://github.com/Expensify/App/pull/6385
Tests
Run npm run desktop-build locally and confirm no errors happen.
QA
None
Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Will you also fix the error,
/Users/runner/work/App/App/src/libs/actions/User.js
Error:   284:10  error  'subscribeToExpensifyCardUpdates' is defined but never used  no-unused-vars

in this PR?

🚀 Cherry-picked to staging by @Jag96 in version: 1.1.16-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.16-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by @Jag96 in version: 1.1.16-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.17-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Edited "Cause" section

Edited the "Cause" section to include common exception codes.

@EliotSeattle - This PR is ready to go to master / live.

@tedhudek - This PR can go to master/live when you have a moment.

@LuigiBruno - Adding these status codes seems reasonable.

Thank you @DOMARS.


Failed to click element

Hi,
I create a spec with this line :
select2("A first name A name", from: 'Artiste', search: true)

And I have this javascript :
$(this).select2(
  placeholder: "Artiste"
  minimumInputLength: 1
  ajax:
    url: '/artists.json'
    data: (term) ->
      q: term
    results: (data) ->
      results: data

It works on production but I have this error when I run my tests :
 Failure/Error: select2("A first name A name", from: 'Artiste', search: true)
 Capybara::Webkit::ClickFailed:
   Failed to click element /html/body/div[@id='page']/div[@id='main-content']/div/div/form[@id='new_painting']/div[2]/div[1]/div/div[@id='s2id_painting_artist_id']/a at position 781,

The spec fail at this line : https://github.com/goodwill/capybara-select2/blob/master/gem/lib/capybara-select2.rb#L27
Is there something to do?
Thanks!

I think the ```from`` field should be the content of the label you added to the original input, not the placeholder.


Import data from saprate json file

Hi
I want to import data from seprate json file.please help me how can i do..

Hi!
You could use jQuery's getJSON function for loading the data from a separate JSON file.

Hi demarchisd
I want the whole chart to appear in a smaller div therefore I need to decrease the width of "sg_time_header_item" for each hour to almost half of it.  Please suggest.
Regards,
Vaibhav.

Hi,
Please try setting the hourWidth option to a smaller value like 40 (value represented in px).
E.g.:
var options = {
    data: [...yourdata],
    style: {
       hourWidth: 40
    }
}

$("#your-div").stackedGantt(options)

Hi Demarchisd
Thanks for the brillent reply. This has solved my problem.
Further, I intend to make changes in tooltip:
Move  sg_tooltip_title downwords and move  sg_tooltip_description upwords.
Can you please help me with this.
Regards,
Vaibhav Aggarwal

Hi vaibhav12aggarwal!
I'm glad that solved your problem.
About the tooltip customization: unfortunately there's no way to do that yet. Could you open another issue requesting the development of this item, please? Then I'll develop it as soon as possible.
Though, if you're in a hurry, you could fork the project and customize the tooltip yourself (check the appendTooltipContent function in src/jquery.stacked-gantt.js file).
Let me know if you need anything else.


[Feature Request] make a daemon (service) for GNU/Linux

That would be great if there is a daemon to control and auto start/stop cow in GNU/Linux with simple command like:
# systemctl start/stop/reload/restart/status cow.service

Already done that for arch, since config is in user home folder, you must use the --user flag for systemctl or use the @yourloginuser hack.
Both are available here : https://github.com/fzerorubigd/archlinux-packages/tree/master/cow-proxy
cp<EMAIL_ADDRESS><EMAIL_ADDRESS>cp cow_user.service /usr/lib/systemd/user/cow.service

Running cow system instance for user account:
To reload system systemd modules, execute
systemctl daemon-reload

To start cow, execute:
systemctl start cow@user

To autostart cow on system start, execute
systemctl enable cow@user

Running cow user instance (will used current user account):
To reload user systemd modules, execute
systemctl --user daemon-reload

To start cow manually, execute:
systemctl --user start cow

To autostart cow on user login, execute
systemctl --user enable cow

@danialbehzadi Could you please check them for debian too? if there is any thing about the new jessie,  happy to help and get things done :)

@fzerorubigd thanks for help, I'm also an Arch user before switching to OS X :)
I've provided an init script. Maybe it's not the way to go on Linux system with systemd now.

@cyfdecyf Debian jessie use systemd by default, so the systemd version is needed. I make a PR when @danialbehzadi confirm it on debian jessie (I have no jessie sandbox available right now.)

Tested on debian jessie. After systemctl start cow@root I get:
<EMAIL_ADDRESS>- COW HTTP proxy service for root
   Loaded: loaded<EMAIL_ADDRESS>disabled)
   Active: active (running) since Mon 2015-05-18 02:35:19 IRDT; 1s ago
 Main PID: 1639 (cow)
   CGroup<EMAIL_ADDRESS>           └─1639 /usr/bin/cow

May 18 02:35:19 osmc cow[1639]: 2015/05/18 02:35:19 COW 0.9.5 listen http <IP_ADDRESS>:7070, PAC url http://<hostip>:7070/pac

Seems OK :+1:
But I think it would be better to be unified, with configuration files in /etc/cow/ directory and systemd service controller cow.service

@danialbehzadi I'll change cow to load config from /etc/cow if there's no ~/.cow and release a new version, maybe on this weekend.

@cyfdecyf there is two kind of files, one is config/blacklist/withelist and
the other is stat.
its the good idea to keep the stat inside the home folder (which the user
has access to). so must create the .cow folder anyway, and its a good idea
to create this folder automatically if its not there.
On Tue, May 19, 2015 at 7:29 PM, Chen Yufei<EMAIL_ADDRESS>wrote:

@danialbehzadi https://github.com/danialbehzadi I'll change cow to load
config from /etc/cow if there's no ~/.cow and release a new version,
maybe on this weekend.
—
Reply to this email directly or view it on GitHub
https://github.com/cyfdecyf/cow/issues/318#issuecomment-103537734.

--
http://fzero.rubi.gd

@fzerorubigd  I have to admit the config loading code is a mess.
I would like to also put stat file inside the same directory of rc file. If the user running cow is nobody, there maybe no $HOME directory anyway. Of course, user will have to make sure /etc/cow writable by cow process if we do this.

@cyfdecyf in this case, I can suggest another folder (like somewhere in
/var ) for stat folder, the /etc is not a good place to store this kind of
things.
ps -and also its a good idea to have an option in rc file to change stat
file position. I can do this part, if you are agree with the idea
On Sun, May 24, 2015 at 7:25 PM, Chen Yufei<EMAIL_ADDRESS>wrote:

@fzerorubigd https://github.com/fzerorubigd I have to admit the config
loading code is a mess.
I would like to also put stat file inside the same directory of rc file.
If the user running cow is nobody, there maybe no $HOME directory anyway.
Of course, user will have to make sure /etc/cow writable by cow process
if we do this.
—
Reply to this email directly or view it on GitHub
https://github.com/cyfdecyf/cow/issues/318#issuecomment-105022166.

--
http://fzero.rubi.gd

Waiting for #328 to fix this one.


Improve Cassandra SchemaVersionUpdaterWithDatastaxDriver.

So that we could reliably use it in ECS env.
Ping @Sarastro72, thanks.

Or ping @x1a0. Thanks.

Thanks. Merging...


Rich list showing empty

Not sure if this worked before, just noticed it, if it's supposed to work with XYM then the filtered ID is wrong ^^
Also note that you can customize the currency (not necessarily XYM), as a suggestion for an extra feature if not in the roadmap

@Vektrat Good catch, thanks mate :)

Also note that you can customize the currency (not necessarily XYM), as a suggestion for an extra feature if not in the roadmap

Yeah, it can build Advance Search to fully utilize filter enpoint.


Update request: Atlas  0.14.1→ 0.15.0

Package name: atlas
Latest released version: 0.15.0

Current version on the unstable channel: 0.14.1
Current version on the stable/release channel: Not packaged

[x] Checked the nixpkgs pull requests

Notify maintainers
@marsam

Note for maintainers: Please tag this issue in your PR.

I've made some progress on the updating the package with the new dependencies. Currently, I've encountered an issue where it can't find postgres.h. I'm sure it's something simple I'm missing, and I don't have the time right now. So, I thought I would share what I got so far. (:
{ lib, buildGoModule, fetchFromGitHub, installShellFiles, libpg_query, xxHash, postgresql }:

buildGoModule rec {
  pname = "atlas";
  version = "0.15.0";

  src = fetchFromGitHub {
    owner = "ariga";
    repo = "atlas";
    rev = "v${version}";
    hash = "sha256-qEui+Y7auNFJwLSUT90jJH7IPgNW06phbJel9y3C1bk=";
  };

  modRoot = "cmd/atlas";

  vendorHash = "sha256-6FABkSPXewOP8TmSoEyMcIDe2Coyf/aJBIqTI/hLoVw=";

  nativeBuildInputs = [ installShellFiles ];
  buildInputs = [ libpg_query xxHash postgresql ];

  env.GOWORK = "off";

  ldflags = [ "-s" "-w" "-X ariga.io/atlas/cmd/atlas/internal/cmdapi.version=v${version}" ];

  subPackages = [ "." ];

  postInstall = ''
    installShellCompletion --cmd atlas \
      --bash <($out/bin/atlas completion bash) \
      --fish <($out/bin/atlas completion fish) \
      --zsh <($out/bin/atlas completion zsh)
  '';

  meta = with lib; {
    description = "A modern tool for managing database schemas";
    homepage = "https://atlasgo.io/";
    changelog = "https://github.com/ariga/atlas/releases/tag/v${version}";
    license = licenses.asl20;
    maintainers = [ maintainers.marsam ];
    mainProgram = "atlas";
  };
}

Yup. I'm stuck here as well. I think the issue has to do with this change in atlas:
cmd/internal/sqlparse/pgparse: move to pgquery (https://github.com/ariga/atlas/pull/2143)
I was able to find this from discourse:
https://discourse.nixos.org/t/trouble-packaging-sqlc/18261/9
Trying the solution mentioned in the discourse leads to a different error and the runVend flag does not work either.

For those who need cloud support, look at the derivation in this comment: https://github.com/NixOS/nixpkgs/pull/267409#issuecomment-1814353365


[Feature Request] Attach to process associated with file

Looking at the code I see
command! UltestAttach call ultest#output#attach(ultest#handler#get_nearest_test(line("."), expand("%:."), v:false))
This gives me the impression that if no nearest test is found then no process will be attached to. What would be nice if given the following file
#! usr/bin/env python
import pdb

if __name__ == "__main__":
   pdb.set_trace()
   print("hello")

If we ran the :UltestAttach command, it would first check for nearest test and try to attach. If no test is found as a fallback it will try to find the (newest?) process which contains the file name and tries to attach to that.
Would something like this be feasible?

To expand on the above point, my use case is as follows. I am using python2 and the test passes control off to a different process during execution, this means that the client is only attached to the first thread and then the test executes without hitting any of the other breakpoints. Ideally I would like to be able to call a pdb.set_trace() or debugpy.breakpoint() when the part of the code I am interested in is being executed so it will be attached to the right thread. I don't think :Ultest will run this file as it wont see it as a testcase, instead it will be ran by direct execution from the commandline.
I also had difficulty replicating the gif from the README. It is a small file and looks basic but I was unable to attach and Ultest marked the test as passing, see asciicinema.

OK before I address the initial request, that asciinema definitely looks like something is wrong. Could you tell me more about your vim-test configuration and also attach a log of ultest. I replicated that file and have the expected behaviour myself. My only config is
let test#python#runner = "pytest"

Ah I only had the DAP python configuration done. Showing pytest makes it fail alright. It immediately shows a fail sign instead of a running sign and I am unable to attach to it still, but the floating window is showing some information and I see the test is ran using python2.7 so maybe if I get it to run with py3 it will resolve that issue. rec

Yep that's a local issue you can see that pytest is being called correctly.
17:50:43 | DEBUG | Thread-1 | processes.py:run:50 | Starting test process test_a.py with command ['pytest', 'test_a.py'], cwd = None, env = None

If you want to use python 2 you'll need to replace breakpoint() with pdb.set_trace()
On your initial request, this should be totally doable. However it's not vim-ultest you should be looking at, but rather vim-test. You can define custom runners to implement whatever logic you like for running tests as long as they implement the correct interface. See the docs here https://github.com/vim-test/vim-test/#extending. I'd also say to check out the pytest runner as it'll be a good place to start from. https://github.com/vim-test/vim-test/blob/master/autoload/test/python/pytest.vim.
Once you've defined the runner, it should just work with vim-ultest!

Cheers making own runner doesn't look too bad, will look into it and see if I can get it attaching, thanks!
On the :UltestAttach, interestingly using python3 didn't work either, it seems to open an integrated terminal in the floating window instead, see vid
log
14:31:07 | INFO | MainThread | logging.py:create_logger:98 | Logger created
14:31:07 | DEBUG | MainThread | __init__.py:__init__:43 | Handler created
14:31:07 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:58 | Starting job with group update_positions
14:31:07 | INFO | Thread-1 | tracker.py:_async_update:55 | Updating positions in test_a.py
14:31:07 | DEBUG | Thread-1 | file.py:parse_file_structure:25 | Converted pattern {'test': ['\\v^\\s*%(async )?def (test_\\w+)'], 'namespace': ['\\v^\\s*class (\\w+)']} to {'test': [re.compile('^\\s*(?:async )?def (test_\\w+)')], 'namespace': [re.compile('^\\s*class (\\w+)')]}
14:31:07 | DEBUG | Thread-1 | tracker.py:_async_update:82 | New test test_a.py found in test_a.py
14:31:07 | DEBUG | Thread-1 | tracker.py:_async_update:82 | New test test_b2181454078850371664 found in test_a.py
14:31:07 | DEBUG | Thread-1 | tracker.py:_remove_old_positions:125 | No tests removed
14:31:07 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:78 | Finished job with group update_positions
14:31:17 | INFO | MainThread | logging.py:create_logger:98 | Logger created
14:31:17 | DEBUG | MainThread | __init__.py:__init__:43 | Handler created
14:31:17 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:58 | Starting job with group update_positions
14:31:17 | INFO | Thread-1 | tracker.py:_async_update:55 | Updating positions in test_a.py
14:31:17 | DEBUG | Thread-1 | file.py:parse_file_structure:25 | Converted pattern {'test': ['\\v^\\s*%(async )?def (test_\\w+)'], 'namespace': ['\\v^\\s*class (\\w+)']} to {'test': [re.compile('^\\s*(?:async )?def (test_\\w+)')], 'namespace': [re.compile('^\\s*class (\\w+)')]}
14:31:17 | DEBUG | Thread-1 | tracker.py:_async_update:82 | New test test_a.py found in test_a.py
14:31:17 | DEBUG | Thread-1 | tracker.py:_async_update:82 | New test test_b-9088935148699828569 found in test_a.py
14:31:17 | DEBUG | Thread-1 | tracker.py:_remove_old_positions:125 | No tests removed
14:31:17 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:78 | Finished job with group update_positions
14:31:20 | INFO | MainThread | __init__.py:run_nearest:125 | Running nearest test in test_a.py at line 0
14:31:20 | DEBUG | MainThread | __init__.py:_register_started:281 | Registering test_a.py as started
14:31:20 | DEBUG | MainThread | __init__.py:_register_started:281 | Registering test_b-9088935148699828569 as started
14:31:20 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:58 | Starting job with group test_a.py
14:31:20 | DEBUG | Thread-1 | processes.py:run:50 | Starting test process test_a.py with command ['pytest', 'test_a.py'], cwd = None, env = None
14:31:25 | INFO | MainThread | __init__.py:get_attach_script:206 | Creating script to attach to process test_b-9088935148699828569
14:31:25 | INFO | MainThread | __init__.py:get_attach_script:206 | Creating script to attach to process test_a.py
14:31:32 | DEBUG | Thread-1 | processes.py:run:76 | Process test_a.py complete with exit code: 1
14:31:32 | INFO | Thread-1 | output.py:parse_failed:66 | Found failed test in output test_b in namespaces None of runner python#pytest
14:31:32 | DEBUG | Thread-1 | __init__.py:_register_result:292 | Registering test_a.py as exited with result {"id": "test_a.py", "file": "test_a.py", "code": 1, "output": "/var/folders/x4/wv98pc6d38n0ykn769h63jjw0000gn/T/ultest76wjx23i/test_a_py/test_a_py_out"}
14:31:32 | DEBUG | Thread-1 | __init__.py:_register_result:292 | Registering test_b-9088935148699828569 as exited with result {"id": "test_b-9088935148699828569", "file": "test_a.py", "code": 1, "output": "/var/folders/x4/wv98pc6d38n0ykn769h63jjw0000gn/T/ultest76wjx23i/test_a_py/test_a_py_out"}
14:31:32 | DEBUG | Thread-1 | __init__.py:_handle_coroutine:78 | Finished job with group test_a.py


[7.x] Filtering null's in hasMorph()

Closes #32458

If this bug is present on 6.x, please send this PR there instead of to 7.x.

This will get merged forward from 6.x.


Niketas Doukas

I'd like to make an in-game character for Niketas Doukas, Theodora's brother and Irene's father.  He'd be a military commander with the traits reckless, politically connected, and brilliant strategist.

Possible picture here: https://cdn.discordapp.com/attachments/529488044185747456/532629509753798676/unknown.png

Some options for the portrait

For injecting the character:

Perhaps an event when the civil war starts.  He would most likely go with Theodora/Irene's side (the democratic faction) due to what he experienced in later Vicky, although he has doubts about the cause.


Reorganize the guide menu

I find the navigation menu in the Yii2 guide to be very confusing, as it is often not quite clear, where to find information on a specific topic. This was much better organized with Yii 1.
An example: I want to read the section on "Assets", so where is it? Let's see, it sounds like a "Special Topic" ... nope. Ah, maybe "Key Concepts"... nope. Hmm. Could it be in "Widgets", as assets are often widget related? Again nothing there. Ok, last try "Handling Requests" - nothing either. By that point I open up all the main topics and finally find it under "Application Structure".
Aha, so "Assets" are part of "Application Structure". Then again "Logging" is not (it's in "Handling Requests"). Looks very arbitrary to me.
I'd suggest: Use less and more clear main topics, closer to what we had with Yii 1.

Any idea on how to reorder current topics?

I can only repeat myself: I'd use fewer main topics. The current ones are too vague to really be helpful when you look for something specific. Even one single long alphabetic list of all topics would be more useful to me than what there is now. As I'm no writer, I can't really come up with better suggestions - but there for sure are some experts on technical documentation that can come up with a clearer structure.

The organization is definitely not arbitrary. Asset is a static thing,
belonging to the app structure. Logging out s not so. I don't think we
should use fewer topics. Each of the current one already has a lot of
content. Alphabetical order is not acceptable either because it's not
friendly to new users. We may consider expanding the TOC by default so that
it's easier to do search.
On Monday, February 23, 2015, Michael Härtl<EMAIL_ADDRESS>wrote:

I can only repeat myself: I'd use fewer main topics. The current ones are
too vague to really be helpful when you look for something specific. Even
one single long alphabetic list of all topics would be more useful to me
than what there is now. As I'm no writer, I can't really come up with
better suggestions - but there for sure are some experts on technical
documentation that can come up with a clearer structure.
—
Reply to this email directly or view it on GitHub
https://github.com/yiisoft/yii2/issues/7395#issuecomment-75527763.

The search Works super Fine, so Menu isn't that important ... Assets will be found in a sec

We will definitively improve the search on the new website.

I think there's a point in what @mikehaertl said since as a user I'm not sure where assets should be.

I think there's a point in what @mikehaertl said since as a user I'm not sure where assets should be.

I don't think the point holds. The guide is organized like a book not a dictionary. If you are an experienced Yii user, you should use search to quickly locate the topic you are interested in.

Absoluttely agree with @mikehaertl. Can't say what exactly wrong with current Guide TOC, but something wrong. Yii 1 guide does not have such troubles.

Yii 1 guide does not have such troubles.

Yii 1 guide doesn't even have "assets" as a separate chapter.

Could it be in "Widgets", as assets are often widget related? Again nothing there.

If you already see "Widgets", you should see "Assets" is next to this entry in the TOC. So it seems to me the problem is exaggerated.
Anyway, the guide is organized in a way for new users to read from the beginning to end. As an experienced user, if you are looking for the explanation of a particular concept or a particular topic, you should really use search.

Specific to the assets example ... I could just type assets in the search box and it showed up as the first item to read.
I think the guide for yii 2 seems to be a work in process (few sections still in TBD and may get merged or divided). So as a user one probably has to manage this change... as they are not used to finding what where as in Yii 1.x. I however agree building the right TOC may help - as some topics in Yii 2 are new.

Let me add some more words to the current TOC design.

The TOC design is not arbitrary. It has taken quite some time to design and been reviewed by several people.
Even if some topics are marked as TBD, their location in TOC is not arbitrary.
Application Structure: this reflects the code organization in your application. Assets, like controllers and models, are static objects and organized in a separate folder in your app.
Handling Request: this explains the typical life cycles of request handling and the common objects you may touch along the life cycle.
Key Concepts: these serve as the foundation of Yii. They are not particularly related with Web applications or MVC framework.
The rest of the chapters: they are organized by individual topics. When a topic is very big, it is organized as a separate chapter; otherwise, they go into "Special Topics".

If you feel necessary to adjust the location of particular sections, please point them out. Otherwise, just claiming the TOC has something wrong is not constructive.

In fact I am saying it the other way around. IMO... the Yii 2 guide is more easier to search for specific functionality than Yii 1.x an example for assets above automatically leads you directly to the topic.
The point I was trying to make is about the change management needed from the user mindset when browsing a guide in Yii2 after coming over from Yii 1. I am ok personally with this change and can adapt to the structure since this is also logical.
On the TBD sections my point was, since some topics are TBD - the user perception maybe that information is not completely available - leading to again saying they cannot find the info they find. So once we have the entire guide filled up - and people read it over and over and can access it with a navigation flow tuned in their mind - it should work out.

Thanks, @kartik-v. :)


Code Style

Need

As a developer
I want a linter
to be lazy about white space but still have pretty code

Approach

[ ] configure linter to run on every push

This is complete!


Start rrdcached via tests

Currently test suite looks for a specific existing socket where rrdcached is running.
Better: Start rrdcached from within tests, stop afterwards.
Reference: http://godoc.org/github.com/lestrrat/go-tcptest

Fixed via c3bca3e8dcbdd146e22849b535d808e4d2c46959.


[WIP][IGNORE] Testing if everything is green with namespaced broker

Fixes #

Proposed Changes

Same as https://github.com/knative-sandbox/eventing-kafka-broker/pull/2478 but only testing the namespaced broker

Release Note

Docs

/retest

/retest

/close


Macaulay2

Project description
From Wikipedia: Macaulay2 is a free computer algebra system created by Daniel Grayson (from the University of Illinois at Urbana–Champaign) and Michael Stillman (from Cornell University) for computation in commutative algebra and algebraic geometry.
Metadata

homepage URL: https://faculty.math.illinois.edu/Macaulay2/
source URL: https://github.com/Macaulay2/M2
license: gpl2+ (source) or gpl3 (binary distributions)
platforms: FreeBSD, Linux, MacOSX (according to their website)

I got most of the build to work and described some problems in the above-linked issue. Here's my repository with the build script:
https://github.com/pdg137/macaulay2-build
Unfortunately it fails during the documentation part of the build process due to some dlopen issue that I don't know how to debug at all. I'd be happy to hear any suggestions.

I have this building a M2 binary now in a reasonable way, including several required sub-packages that are not in nixpkgs.  The dlopen command still fails but I disabled documentation so that the build can complete. I think the next step is to figure out some basic tests, but it is already worth trying out if you need Macaulay2 on Nixos.


[BUG] npx quartz sync delete pushed notes

Describe the bug

I use npx quartz sync to update myself repo, like change css , add index.md, and so on.
I use quartz-syncer plugin in obsidian to upload notes to content dir
3. this plugin just run git push (notes need publish) to my remote repo.
I find  npx quartz sync won't pull latest content
for example
the lastest repo content dir

after npx quartz sync , my local repo content dir , seems git pull  don't work well ?  but log don't have error

some dir is missing in local repo
Now , if I update some file (like change css), then exec npx quartz sync , the local repo will override remote repo !!!

Desktop (please complete the following information):

Quartz Version: [e.g. v4.2.2]
node Version: [e.g. v18.16]
npm version: [e.g. v10.1.0]
OS: [e.g. macos]
Browser [e.g. chrome, safari]

issue

You should remove the .gitkeep

Quartz relies on npx quartz sync to resolve conflicts so if you do it yourself, you can't guarantee npx quartz sync will work correctly. Closing as working as intended as you should either completely use npx quartz sync or not at all


[BUG] - live_response_api - Free JobWorker on timeout error

I am seeing this behaviour on: (please complete the following information):

OS: Windows
Carbon Black Cloud Products: Endpoint Standard
Python Version: 3.9.7

Describe the bug
When a live response session cannot be created  due to TimeoutError / 404:
Error encountered by JobWorker[7369023]: Timed out when requesting /appservices/v6/orgs/_org_id_/liveresponse/sessions/12345:7369023 from API with HTTP status code 404: Could not establish session with device 7369023

The JobWorker (Future) is not stopped and does not return. Keeping JobWorkers tied up and not allowing additional Live Response Jobs to begin work.
Steps to Reproduce
Steps to reproduce the behavior (Provide a log message if relevant):
Using the structure from examples/jobrunner.py
# collect 'future' objects for all jobs
    for device in online_devices:
        f = cb.live_response.submit_job(jobobject.run, device)
        futures[f] = device.id

    # iterate over all the futures
    for f in as_completed(futures.keys(), timeout=100):
        if f.exception() is None:
            print("Device {0} had result:".format(futures[f]))
            print(f.result())
            completed_devices.append(futures[f])
        else:
            print("Device {0} had error:".format(futures[f]))
            print(f.exception())

    still_to_do = set([s.id for s in online_devices]) - set(completed_devices)
    print("The following devices were attempted but not completed or errored out:")
    for device in still_to_do:
        print("  {0}".format(device))

If session create times out, the future is not marked as Done
Expected behavior
If a session cannot be established with a device, return the exception and free up the JobWorker
MaxWorkers is capped at 10. If one times out, it should get freed up.

Updated logs -- the JobWorker does exit and get deleted, but the Future object is not marked as completed, and so it never returns in the as_completed() loop.
2022-03-31 12:28:32,783 - cbc_sdk.live_response_api - DEBUG - url: /appservices/v6/orgs/orgid/liveresponse/sessions/123456:8696711 -> status: PENDING
2022-03-31 12:28:34,007 - cbc_sdk.live_response_api - DEBUG - url: /appservices/v6/orgs/orgid/liveresponse/sessions/123456:8696711 -> status: PENDING
2022-03-31 12:28:35,217 - cbc_sdk.live_response_api - DEBUG - url: /appservices/v6/orgs/orgid/liveresponse/sessions/123456:8696711 -> status: PENDING
2022-03-31 12:28:36,450 - cbc_sdk.live_response_api - DEBUG - url: /appservices/v6/orgs/orgid/liveresponse/sessions/123456:8696711 -> status: PENDING
2022-03-31 12:28:37,663 - cbc_sdk.live_response_api - DEBUG - Got item: <cbc_sdk.live_response_api.WorkerStatus object at 0x000001D0068761C0>
2022-03-31 12:28:37,664 - cbc_sdk.live_response_api - ERROR - Error encountered by JobWorker[8696711]: Timed out when requesting /appservices/v6/orgs/orgid/liveresponse/sessions/123456:8696711 from API with HTTP status code 404: Could not establish session with device 8696711
2022-03-31 12:28:37,665 - cbc_sdk.live_response_api - DEBUG - Entering scheduler
2022-03-31 12:28:37,666 - cbc_sdk.live_response_api - DEBUG - There are idle workers for device ids set()
2022-03-31 12:28:37,666 - cbc_sdk.live_response_api - DEBUG - 0 jobs ready to execute in existing execution slots
2022-03-31 12:28:37,666 - cbc_sdk.live_response_api - DEBUG - Waiting for item on Scheduler Queue
2022-03-31 12:28:37,667 - cbc_sdk.live_response_api - DEBUG - Got item: <cbc_sdk.live_response_api.WorkerStatus object at 0x000001D0068761F0>
2022-03-31 12:28:37,667 - cbc_sdk.live_response_api - DEBUG - JobWorker[8696711] has exited, waiting...
2022-03-31 12:28:37,667 - cbc_sdk.live_response_api - DEBUG - JobWorker[8696711] deleted
2022-03-31 12:28:37,667 - cbc_sdk.live_response_api - DEBUG - Entering scheduler
2022-03-31 12:28:37,668 - cbc_sdk.live_response_api - DEBUG - There are idle workers for device ids set()
2022-03-31 12:28:37,668 - cbc_sdk.live_response_api - DEBUG - 0 jobs ready to execute in existing execution slots
2022-03-31 12:28:37,668 - cbc_sdk.live_response_api - DEBUG - Waiting for item on Scheduler Queue

Fix in PR https://github.com/carbonblack/carbon-black-cloud-sdk-python/pull/324

Fix provided; issue closed.


Transactions fail to load in Postmix wallet

The Postmix section in one of my wallets will not load, showing message, "Error loading transactions: Failed to subscribe to path: m/0/...".
The Deposit, Premix, and Badbank sections of the same wallet load their respective balances/histories successfully.

This error occurs because the Postmix wallet is larger (more used addresses) than the other wallets. As such, it's not a Sparrow issue, but a server issue. The server is having trouble dealing with the size of this wallet.
Are you are using Electrs pre v0.9.0? If so, upgrading to the latest release will probably solve this.


A suggestion

You should leverage gh-pages to show off your element in a demo link. It gives insight onto your element and would increase adoption.
Just a suggestion ;)
Cheers.

I'll give it a try as soon as I can. I have recently started with GitHub (uploading code, I mean) and there is plenty to learn.
Thank you very much

No worries, best of luck!


cmake related issue on getting/referring header files path of module IotShadow-cpp

I am encountering errors while trying to build a new executable using cmake with the file "aws-iot-device-sdk-cpp-v2/samples/shadow/shadow_sync/main.cpp".
As prerequisite, I’ve downloaded aws-crt-cpp & aws-iot-device-sdk-cpp-v2 from git, built the libraries using CMake & make (gcc 7.3) in a different folder (e.g. ~/temp/bin)
I’ve copied the CMakeLists.txt (from shadow_sync) and modified to build my project.
It links with IotShadow-cpp, so I’ve used the below line to load the package.
find_package(IotShadow-cpp REQUIRED)
To find the package I’d to add the below lines
list(APPEND CMAKE_MODULE_PATH "${CMAKE_INSTALL_PREFIX}/lib/IotShadow-cpp/cmake")
list(APPEND CMAKE_MODULE_PATH "${CMAKE_INSTALL_PREFIX}/lib/cmake")
I had to rename the file to lower case (iotshadow-cpp-config.cmake) and I figured this out after trying several things!
Now, the package is loaded but include directories are not, causing compilation to fail.
Please advise if I’m missing anything or there is any issue elsewhere. I’m attaching the below files for your review.
Attachments:
New CMakeLists.txt
main.cpp
Original CMakeLists.txt

Rather than trying to mess with paths in a CMakeLists.txt, have you tried doing an install of the libraries you're depending on?  If you're not comfortable installing to the system, you can also install to a local directory under you're control.
The general steps look like:

Build the libaries with CMAKE_INSTALL_PREFIX set to where you want them to go (or leave blank for a system level install, but I wouldn't recommend that for an in-preview sdk yet)
make install the libraries
Build your app with CMAKE_PREFIX_PATH set to the install directory

If you include imported lib all include files are automatically added.


Fix typo in CreateCommand's CLI help text

Specific issue was an incorrect plural combined with if instead of it in the help for the --force flag.
Fixes #1394.
Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[ ] I added new tests to check the change I am making.
[x] All existing and new tests are passing.
[x] Any breaking changes are documented below.

Breaking changes
No breaking changes

I have read the CLA Document and I hereby sign the CLA


Update smtp_provider.md

Removed links to ANCIENT blog article.

Hello @Zaftula,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Feat(eos_designs): Support for Static Flood Lists, HER

Change Summary

Support for Static Flood Lists, HER
Component(s) name
arista.avd.eos_designs
Proposed changes

overlay_routing_protocol: < EBGP | IBGP | none | her | default for l3ls-evpn -> EBGP >

# When using head-end-replication, configure flood-lists per VNI.  | Optional
# By default HER will be configured with a common flood-list containing all VTEPs. This behavior can be changed
# to per-VNI flood-lists by setting `overlay_her_flood_list_per_vni: true`. This will make `eos_designs` consider
# configured VLANs per VTEP, and only include the relevant VTEPs to each VNI's flood-list.
overlay_her_flood_list_per_vni: < true | false | default -> false >

# When using Head-End Replication, set the scope of flood-lists to Fabric or DC | Optional
# By default all VTEPs in the Fabric (part of the inventory group referenced by "fabric_name") are added
# to the flood-lists. This can be changed to all VTEPs in the DC (part of the inventory group referenced
# by "dc_name")
# This is useful if Border Leaf switches are dividing the VXLAN overlay into separate domains.
overlay_her_flood_list_scope: < "fabric" | "dc" | default -> "fabric" >

How to test

Added molecule coverage. No other changes.
Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

@gmuloc @xaviramon please review this commit I added after your approvals. This is to avoid the constant churn in the flood-list caused by other PRs. https://github.com/aristanetworks/ansible-avd/pull/1982/commits/43afc4240d9963a361aaee906a4026ec1bc46719


fix: add babel-loader options types

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Fixes #5843
See all babel-loader options needed to be added here : https://github.com/babel/babel-loader#options

On the long run, wouldn't it make sense to directly contributed types for the loader? :thinking:

Would be great, however, this is not a bable option but babel-loader, which I couldn't find any type definition for its configuration.

Agreed, I'll take the time to propose a PR on babel-loader to add types.


ci: build app on pull request

Will this mean now when we get iOS builds working to main that the version will automatically get pushed up to TestFlight making field testing easier?


Remove the limit on the number of download threads.

a max number of threads is necessary, but 5 is too little. 64 or 128 is appropriate i think

I intentionally limited it to 32. because some users may increase it too much and cause the system to halt! there should be a maximum limit.


Enum does not place underscore before digit

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hi,
we are using multiple GraphQL implementations on our services and experienced an issue when switching to HotChocolate with Enum values.
public enum Strategy     {         Active1,         Active2,         Active3,     }
We do get this values now as "ACTIVE1"... from HotChocolate, while other implementations do give "ACTIVE_1"... So placing an underscore before the digit.
As per spec https://spec.graphql.org/October2021/#sec-Names I would read it as the underscore is to be used.
Steps to reproduce
[InlineData("Active3", "ACTIVE_3")]
        public void GetEnumName(string runtimeName, string expectedSchemaName)
Reproducable via the
HotChocolate.Types.Descriptors DefaultNamingConventionsTests
Added the above InlineData for testing
Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
11.2.2

Spec wise this is not an issue

A Name must not be followed by a NameContinue. In other words, a Name token is always the longest possible valid sequence. The source characters a1 cannot be interpreted as two tokens since a is followed by the NameContinue 1.

Where NameContinue can be Letter Digit or _. So it is a valid GraphQL Name.
It's more a matter of how you want to split it :)
You can override the naming convetions
    public class CustomNamingConventions : DefaultNamingConventions {
        public override NameString GetEnumValueName(object value)
        {
            return base.GetEnumValueName(value);
        }
    }

services.AddGraphQLServer.AddConvention<INamingConventions, CustomNamingConventions>();

@PascalSenn exactly. Also, you are referring to the wrong spec section if you are referring to enum names.
https://spec.graphql.org/October2021/#sec-Enum-Value

Enum values are represented as unquoted names (ex. MOBILE_WEB). It is recommended that Enum values be “all caps”. Enum values are only used in contexts where the precise enumeration type is known. Therefore it’s not necessary to supply an enumeration type name in the literal.

Apologies for referencing the wrong spec section. Will go with the custom naming convention then.


Review Kanban Board Setup

Link to Github Project:
https://github.com/users/firepicaso/projects/5/views/1?pane=issue&itemId=45164028
We are a group of three members

Hi @firepicaso, @mumbereh and @albertkantwi,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

I get 404 error when I visit your page. Kindly rectify that.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi Team,
Good job so far, indeed you have spent time on this project!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights

Fair distribution of tasks ✔
Good use of Kanban Board ✔

Required Changes ♻
Check the comments under the review.

[ ] While you've done a great job, you probably forgot to add a description to the cards individually, I will encourage you to do this so as to be clear to the person working on the project while at it.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thanks for the review @Olamarx.
We have added descriptions to all the tasks.

Hi <STUDENT_MENTION>,

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
✅ You used Gitflow correctly :+1:
✅ No linter error :+1:
✅ Good commit messages :+1:
✅ Good descriptive PR :+1:
✅ Well done :clap: you did all changes given by the previous reviewer :+1:
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


SonarCloud JS bug reported in Underlay.js

https://sonarcloud.io/project/issues?id=com.adobe.dx%3Areactor&resolved=false&types=BUG

Fixed with #33


OpenSSL::SSL::SSLError: could not stablish connection: SSL error.

Hi,
I'm using portus and Registry/Distribution on a Docker-Swarm (currently just 1 node) and behind a Docker Flow Proxy (HAProxy). The Registry runs at port 443 with a certificated signed with my own CA. Portus runs at port 443 and subRoot '/portus'.
When I try to setup the connection of the Registry and Portus it fails with "OpenSSL::SSL::SSLError: could not stablish connection: SSL error. You can skip this check by clicking on the "Skip remote checks" checkbox.".
Response of /portus/api/v1/registries/validate?name=Local+Registry&hostname=docker-registry01.local&external_hostname=&use_ssl=true&force=false&only%5B%5D=hostname:
{"messages":{"hostname":["OpenSSL::SSL::SSLError: could not stablish connection: SSL error."]},"valid":false}

Docker-Compose File
version: '3'

services:
  redis:
    image: redis:alpine
    command: redis-server /etc/redis/config.conf
    networks:
      - default
    volumes:
      - ./config/redis/config.conf:/etc/redis/config.conf

  registry:
    image: registry:2
    depends_on:
      - redis
    networks:
      - default
      - proxy
    ports:
      - 5000
    environment:
      - REGISTRY_AUTH_TOKEN_REALM=https://docker-registry01.local/portus/v2/token
      - REGISTRY_AUTH_TOKEN_SERVICE=docker-registry01.local
      - REGISTRY_AUTH_TOKEN_ISSUER=docker-registry01.local
      - REGISTRY_AUTH_TOKEN_ROOTCERTBUNDLE=/etc/ssl/certs/docker-registry01.local.crt
      - REGISTRY_HTTP_SECRET=****
      - REGISTRY_REDIS_ADDR=redis:6379
      - REGISTRY_STORAGE_DELETE_ENABLED=true
      - REGISTRY_STORAGE_CACHE_BLOBDESCRIPTOR=redis
    volumes:
      - ./data/registry:/var/lib/registry
      - /etc/ssl/certs:/etc/ssl/certs:ro
      - ./config/registry/config.yml:/etc/docker/registry/config.yml
    deploy:
      replicas: 1
      labels:
        - com.df.notify=true
        - com.df.port=5000
        - com.df.serviceDomain=docker-registry01.local

  mariadb:
    image: library/mariadb:10.0.23
    networks:
      - default
    ports:
      - 3306
    environment:
      MYSQL_ROOT_PASSWORD: ****
    volumes:
      - ./data/mysql:/var/lib/mysql

  portus:
    image: opensuse/portus:2.3
    depends_on:
      - mariadb
    networks:
      - default
      - proxy
    ports:
      - 3000
    environment:
      RAILS_SERVE_STATIC_FILES: 'true'
      RAILS_RELATIVE_URL_ROOT: /portus
      PORTUS_DB_HOST: mariadb
      PORTUS_DB_USERNAME: ****
      PORTUS_DB_PASSWORD: ****
      PORTUS_DB_DATABASE: ****
      PORTUS_DELETE_ENABLED: 'true'
      PORTUS_MACHINE_FQDN_VALUE: docker-registry01.local
      PORTUS_CHECK_SSL_USAGE_ENABLED: 'false'
      PORTUS_SECRET_KEY_BASE: ****
      PORTUS_KEY_PATH: /certificates/docker-registry01.local.key
      PORTUS_PASSWORD: ****
      PORTUS_SIGNUP_ENABLED: 'false'
    volumes:
      - ./config/portus:/etc/portus
      - ./secrets:/certificates:ro
    deploy:
      replicas: 1
      labels:
        - com.df.notify=true
        - com.df.port=3000
        - com.df.servicePath=/portus
        - com.df.serviceDomain=docker-registry01.local

#  portus-background:
#    image: opensuse/portus:latest
#    entrypoint: bundle exec rails runner /svr/Portus/bin/background.rb
#    networks:
#      - default
#    depends_on:
#      - portus
#      - mariadb
#    environment:
#      - PORTUS_DB_HOST=mariadb
#      - PORTUS_DB_USERNAME=****
#      - PORTUS_DB_PASSWORD=****
#      - PORTUS_DB_DATABASE=****

networks:
  default:
    external: false
  proxy:
    external: true

Portus Logs (NOTE PORTUS_LOG_LEVEL: debug does not provide more relevant information):
Started GET "/portus/admin/registries/1/edit" for <IP_ADDRESS> at 2018-02-06 17:01:12 +0000
Processing by Admin::RegistriesController#edit as HTML
  Parameters: {"id"=>"1"}
  Rendered admin/registries/components/_form.html.slim (2.7ms)
  Rendered admin/registries/edit.html.slim within layouts/application (3.3ms)
  Rendered shared/_header.html.slim (0.9ms)
  Rendered shared/_aside.html.slim (0.7ms)
  Rendered shared/_search.html.slim (0.2ms)
  Rendered shared/_notification.html.slim (0.1ms)
  Rendered shared/_notifications.html.slim (0.2ms)
Completed 200 OK in 12ms (Views: 7.8ms | ActiveRecord: 0.9ms)
Started GET "/portus/api/v1/registries/validate?name=Local+Registry&hostname=doc&external_hostname=&use_ssl=false&force=false&only%5B%5D=hostname" for <IP_ADDRESS> at 2018-02-06 17:01:17 +0000
SocketError: connection refused.
Started GET "/portus/api/v1/registries/validate?name=Local+Registry&hostname=do&external_hostname=&use_ssl=false&force=false&only%5B%5D=hostname" for <IP_ADDRESS> at 2018-02-06 17:01:18 +0000
SocketError: connection refused.
Started GET "/portus/api/v1/registries/validate?name=Local+Registry&hostname=docke&external_hostname=&use_ssl=false&force=false&only%5B%5D=hostname" for <IP_ADDRESS> at 2018-02-06 17:01:20 +0000
SocketError: connection refused.
Started GET "/portus/api/v1/registries/validate?name=Local+Registry&hostname=docker-registry01.local&external_hostname=&use_ssl=false&force=false&only%5B%5D=hostname" for <IP_ADDRESS> at 2018-02-06 17:01:23 +0000
Started GET "/portus/api/v1/registries/validate?name=Local+Registry&hostname=docker-registry01.local&external_hostname=&use_ssl=true&force=false&only%5B%5D=hostname" for <IP_ADDRESS> at 2018-02-06 17:01:25 +0000
OpenSSL::SSL::SSLError: could not stablish connection: SSL error.

So in you case the DFP is doing TLS and also Portus. So what happens here is that you have
REGISTRY_AUTH_TOKEN_ROOTCERTBUNDLE=/etc/ssl/certs/docker-registry01.local.crt
PORTUS_KEY_PATH: /certificates/docker-registry01.local.key

however when the registry is talking to portus it see another cert. the one for DFP.
If you want to do TLS on DFP and on Portus you have to do the communication between Portus <-> Registry not over public domain but service names. This is possible, you then have to enter in the registry admin section registry:5000 and in  External Registry Name  the public domain.
However I would recommend you do the  TLS termination only of DFP and leave the rest unsecure

Thanks for the fast response. I will try it tomorrow.

sorry I see it now in the config, i was telling you BS, you are not doing TLS on Portus.
It seems like you want to set the registry in the admin section.
However from your DFP config I can't see that you are doing  TLS on the registry, Try it without TLS first and see if that works then enable tls in DFP and Portus admin section.

When I set the hostname to registry:5000 it works. And if I connect from outside to https://docker-registry01.local with my browser it says my certificate is valid.

you have the same domain for registry and ui. docker-registry01.local its a bit tricky but it works.
I even have it setup without the /portus path. See. https://github.com/vfarcic/docker-flow-proxy/issues/409

I'll check both options thanks

I have similar issue :(
I checked out from git on fresh unRAID install. I have certificates set up and all the containers spin up.
I can login to the registry with docker login from my other server which is a different physical machine from unRAID server running registry and portus.
In Portus I can not make it connect with SSL to my registry. I tried different ways to address the registry container and nothing works. From portus container I can ping registry and it returns correct container IP. But registry:5000 still returns SSL errors and that it can not connect to registry.
Second issue I have is that I can not force new settings even with the Skip remote checks checkbox wich is also anoying.

@dlednik there must be something that you are missing in regards to SSL, which I'm sure is specific to your setup so I don't know if I'll be able to help much. From the Portus side make sure to use the proper PORTUS_CHECK_SSL_USAGE_ENABLED value to fit your case, and make sure that from the Portus container you can establish an SSL connection to the registry. Another common pitfall is a firewall blocking your requests and stuff like that.
I'm going to close this issue now, since I think this is not to blame on Portus itself. That being said, feel free to leave more comments.

I have set the external url to docker-registry01.local and it works fine now. But I dont get why my ssl setup was not working. @mssola Would it be possible to get a more accurate error message?

I solved it by installing harbor from vmware :)
It turns out even for me (not a beginner) it's just too much hassle to set it up and make it work. Might be just an issue with the unRaid setup and I don't want to trade that for something else as it serves as a NAS primarily. And running registry on top seemed like a good combination.

I have set the external url to docker-registry01.local and it works fine now. But I dont get why my ssl setup was not working. @mssola Would it be possible to get a more accurate error message?

Yes, that's something we can definitely improve. I just created #1683 to track this. Thanks for the feedback!


Select2 loading and jquery doesn't work

Hello kartik,
i have some problem with your great widgets, there are select2 can't work like i use it in my template ADMINLTE..

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


fix(state): index spending transaction IDs for each address

Motivation
When I compare zebrad and zcashd RPC responses for getaddresstxids, using this command:
zebra-utils/zcash-rpc-diff 28232 'getaddresstxids { "addresses": ["t3dvVE3SQEi7kqNzwrfNePxZ1d4hUyztBA1"], "start": 1, "end": 1661134 }'

All the spending transaction IDs are missing.
Depends-On: #4385
Specifications
It's not explicitly listed in the docs, but zcashd includes spending transaction IDs:
https://zcash.github.io/rpc/getaddresstxids.html
Solution

Correctly populate spending address transaction ID indexes
Increment the database format to rebuild address tx ID indexes

Related changes:

Split writing transaction indexes into transparent and shielded
Split writing transparent indexes into created and spent
Update non-finalized docs to prevent similar bugs

@gustavovalverde @dconnolly this PR might cause DevOps bugs or slow builds, because it increments the state version.
Review
@conradoplg and I wrote this code.
I've marked this PR as a high priority, because it changes the state version.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

It's really tricky to test this code, because transparent outputs can't be spent in the first 100 blocks, due to the coinbase maturity consensus rule. I'm testing it locally by doing a full sync, then re-running the manual comparison with zcashd.

I tested the comparison locally, and I got:

Response diff between zcashd and zebrad:
RPC responses were identical

CI is not rebuilding the cached state in this PR, even though the state version has changed.
So these tests failed:

fully-synced-rpc
lightwalletd-transactions

This was meant to be fixed as part of #4153, but that's a big ticket. Maybe I can do a smaller fix in this PR.
Or we could approve this PR and merge it to main. Then the cached state will get rebuilt, and the tests should work.

Took me a while to grok this (I always get confused by inputs/outputs) but I get it now, thanks!
I was going to replicate the test but it's taking a while to reindex zcashd and sync zebra, so I'll just approve it for now. I'll leave up to you if you'd like to merge this now, and how to handle the state issue. (I don't mind just merging it)

I have a bunch of release build fixes in PR #4369.
After they merge, I'll try to do a fix for the state rebuilds on top of those changes.
(We need to fix the state rebuild at some point, and we have another state version increment planned soon.)

CI is not rebuilding the cached state in this PR, even though the state version has changed.
So these tests failed:

fully-synced-rpc
lightwalletd-transactions

This should be fixed by PR #4385, which this PR is now based on.

https://github.com/ZcashFoundation/zebra/pull/4385 is working as expected here 🎉

The full sync failed, but gracefully, from a CI perspective, as it happened what it was meant to happen (skipping dependent jobs).
As this was approved from a Rust/Zebra perspective, and the full sync issue is not related to this PR, I'm re-approving.


LAPP-41 Add ServiceColletionExtension

The goal is to make easier the configuration on NetCore. This pattern is used in many libraries, for instance: Automapper

Codecov Report

Merging #130 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #130   +/-   ##
=======================================
  Coverage   83.73%   83.73%
=======================================
  Files          25       25
  Lines         756      756
  Branches       78       78
=======================================
  Hits          633      633
  Misses        122      122
  Partials        1        1

Impacted Files
Coverage Δ

Analytics/Client.cs
95.80% <ø> (ø)

Continue to review full report at Codecov.

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


Add support for romana agent store and glog

Store section in config for romana agent, support for glog in romana agent startup scripts.

Ok


Add open-laravel-herd-site extension

Description
This extension allows you to quickly open your Laravel Herd sites directly from Raycast in Visual Studio Code, Finder or your default web browser.
Screencast
https://github.com/raycast/extensions/assets/7975568/3d3b441c-6c20-4911-9d10-8a10ee2c7ab5
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi!
Thank you for your contribution! My name is Milena and I will be reviewing it today.
I was wondering what is the value your extension provides other than grouping QuickLinks? Also, on your README, there are no instructions on how to get it working. Do you mind fixing that?

Hi Milena!
The extension only works with Laravel Herd. It's an app for PHP development. Among other features, you can park it in a directory and it automatically makes all directories available with a .test domain.
If your folder looks like this:
📁 my-project-1
📁 my-project-2
📁 my-project-3
You will be able to access my-project-1.test in your browser.
This extension accesses these directories and lets you open them in Visual Studio Code, Finder or the browser. There is no setup required, but you'll have to have Laravel Herd installed and working of course.
Let me know if you have any other questions.


10302 - ACH US Footer Table - Add New Field

I'm needing to add custom field to standard 10302 ACH US Footer table for data exchange mapping. We've created a new table/page for client to enter data and have successfully added this field to 10300 – ACH US Header but also need it in the footer. Our developer cant subscribe to the event to add code. Does anyone know if this is possible?

Please provide more information - which version you have, are you doing extension, which event you are trying to subscriber or what should be added to solve your issue?

Closing, as no information about event have been provided


Report outstanding scope->permanent_status at destruction time

The code was silentely eating outstanding statuses with this comment:
// In most cases the status will have been consumed by the scope owner.

But as I'm debugging why ukernel statuses aren't reported in e2e matmul tests, after having fixed Issue #11843, I still don't see error statuses. Debugging, I found that I was landing exactly in this case (actually this had been hinted by @benvanik on Discord the last time I had brought up the topic).
I have not succeeded finding the more proper place, closer to the root cause, where to catch and report the scope->permanent_status. In iree_vm_invoke, the main thread goes through the vmvx module call for the ukernel, then it goes into a hal module call to fence.await where it waits for worker threads to finish work. Somewhere around this might be the place to see if the scope has an error status?
Anyway, the present PR might be good to have anyway: it seems generally worth echoing un-consumed error statuses to stderr?
Question - do I need to do something more than just iree_status_fprint to mark the status as having been consumed?

Filed #11862.

You can use a trace message (IREE_TRACE_MESSAGE_DYNAMIC) or something similar but no printing.

That wouldn't really help normal users who haven't scratched their heads for long enough to be at the point where they have trace messages enabled :-)

It'd be better to actually propagate the error.

+1

Agreed, but we really should not be ever producing code from the compiler that can fail - that you're using error messages here is only for your eyes, not users :)

Agreed, but we really should not be ever producing code from the compiler that can fail

And indeed, ukernel errors are essentially like assertions: we only enable in asserts-enabled builds, and they can only occur as a result of an internal bug (possibly straddling the compiler/runtime divide).
I was trying to let ukernels still return statuses and have that propagated to the e2e tests. But maybe I should stop trying to do that and instead just make ukernel error literally assertions, i.e. abort?
Oh - now i remember - that was actually my first intuition but since ukernels can't call the standard library I thought it was not so trivial to have them abort (and I don't want to resort to UB in a make-believe local implementation of abort). WDYT @benvanik  ?

One way is to have the ukernel caller, i.e. the vmvx module entry point function (eg iree_vmvx_mmt4d) abort on error ?

Thought a bit more - it wasn't actually correct when I said above that these errors are like assertions so we could just abort. There are various error cases that are not so clearly attributable to a programming error and shouldn't crash users on.

OK, i guess you're right. I was thinking maybe iree_uk_status_unsupported_huge_or_negative_dimension but... the user would have to have a dimension >= 2^31 to get that.
OK, so, assertion it is? In the vmvx/module entry points?

Note: one way to get ukernel errors would be to run a new vmvx bytecode module against an old IREE runtime. Do we need ukernel versioning? And what do we do on version mismatch?


publish<EMAIL_ADDRESS>

Requested by: @kamilogorek
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[ ] crates
[ ] pypi
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

#retract - due to incorrect craft config, see https://github.com/getsentry/symbolic/pull/693

#retract

#retract y u no work

i am the bot


feat(JiraIcon): Adding it

Overview

Adds the JiraIcon

Just wanted to use this icon, so I figured I should add it to Gamut instead of just my personal repo so I can benefit from all the other icon-y stuff.
PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: INT-72
[X] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@6.0.3-alpha.8c3afc.0@codecademy/gamut-kit@0.6.35-alpha.8c3afc.0@codecademy/gamut-labs@28.4.4-alpha.8c3afc.0@codecademy/gamut@42.1.3-alpha.8c3afc.0@codecademy/monaco@0.5.26-alpha.8c3afc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61bbaab7b8bb410a53db3aa1--gamut-preview.netlify.appDeploy Logs

this also should be added to the packages/gamut-icons/src/icons/regular/index.tsx file, then you're good to 🛳

@dreamwasp Is there a particular way to add that? It looks like that index is autogenerated/gitignored in my IDE

📬Published Alpha Packages:@codecademy/gamut-icons@6.0.3-alpha.e97dc7.0@codecademy/gamut-kit@0.6.38-alpha.e97dc7.0@codecademy/gamut-labs@28.6.1-alpha.e97dc7.0@codecademy/gamut@42.2.2-alpha.e97dc7.0@codecademy/monaco@0.5.28-alpha.e97dc7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61d31c4087cc532491ec2b73--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://61d47dfb7678e24862ca92a0--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://61d5a931fe1fba571d0629e3--gamut-preview.netlify.appDeploy Logs


Remove 'default-features = false' from ndarray and approx

Fixes issue where using linfa by itself fails to build. Cherry-picked from 0.5.1 branch.

Codecov Report

Merging #202 (5b386f4) into master (52fe418) will increase coverage by 2.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
+ Coverage   56.05%   58.41%   +2.35%
==========================================
  Files          92       93       +1
  Lines        8482     8144     -338
==========================================
+ Hits         4755     4757       +2
+ Misses       3727     3387     -340

Impacted Files
Coverage Δ

.../linfa-preprocessing/src/countgrams/hyperparams.rs
42.37% <0.00%> (-23.42%)
:arrow_down:

...ms/linfa-preprocessing/src/tf_idf_vectorization.rs
53.77% <0.00%> (-16.60%)
:arrow_down:

src/composing/multi_target_model.rs
63.04% <0.00%> (-6.01%)
:arrow_down:

algorithms/linfa-linear/src/ols.rs
82.89% <0.00%> (-5.84%)
:arrow_down:

src/metrics_regression.rs
63.54% <0.00%> (-5.78%)
:arrow_down:

src/composing/multi_class_model.rs
53.84% <0.00%> (-5.73%)
:arrow_down:

algorithms/linfa-linear/src/float.rs
27.77% <0.00%> (-5.56%)
:arrow_down:

...gorithms/linfa-preprocessing/src/countgrams/mod.rs
76.00% <0.00%> (-4.70%)
:arrow_down:

algorithms/linfa-pls/src/pls_svd.rs
45.45% <0.00%> (-4.55%)
:arrow_down:

algorithms/linfa-reduction/src/pca.rs
67.77% <0.00%> (-3.16%)
:arrow_down:

... and 37 more

Continue to review full report at Codecov.

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


rsync__exclude not excluding anything at all

I can sync directories without any issue but if I want to exclude certain file-type(s) it doesn't work at all. I wanted all the  .json and .sh files to be excluded, so I have this:
config.vm.synced_folder "./scripts", "/var/tmp/"+nodes_cfg['aws_env']+"/scripts",
                         type: "rsync", rsync__exclude: [ "*.json", "*.sh"]

which is syncing scripts alright but with everything in it, including the file-types in the rsync__exclude list. Did I do anything wrong here or anything am I still missing? Best!

Vagrant 1.7.2
vagrant-aws 0.6.0
I am experiencing the equivalent of this issue mitchellh/vagrant-aws#304 and mitchellh/vagrant#4248: rsync does not respect excludes. The suggestion in this comment, that the method implemented in mitchellh/vagrant-aws#156:
config.vm.synced_folder '.', '/vagrant', :rsync_excludes => ['chef/tmp', 'tmp']

resolves the issue is also not working for me.

This issue should be closed

@rtyler I'm not sure this issue should be closed - the last comment says "the method implemented in ... is also not working for me", though some of what I've replaced with an ellipsis makes it harder to parse that meaning.
I'm also not able to successfully exclude files from the rsync, using any of the documented configuration.

Currently, I'm working around this with the following:
vagrant ssh-config > ssh.config # One time, for the following command
rsync -ravz -e "ssh -F ssh.config" --exclude .vagrant --exclude .git --exclude-from .gitignore --delete ./ default:/vagrant/

An upside is that when the delta is small the bare rsync command is many times faster than the rsync command wrapped in Vagrant initialization and abstraction.

Actually - the implementation does work. The problem that I was running into, and perhaps the problem of others, is that (as documented) changes to the Vagrantfile require a vagrant reload to take effect, and I was expecting those changes to be immediately applied to a vagrant provision or a vagrant rsync command, and they were not.


[NEWS] Open spring-data-requery

I make spring-data-requery in last two month.
Please check out

Thanks for the info, might be better to post this in an article/blog post, github issues is for issues with the library, thanks!


[BEAM-25] Allow stateful DoFn. Implement for DirectRunner

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

@tgroh Please take another look. I put together a test that the cleanup works and confirmed that the test does break when the code breaks. It is a nightmarishly verbose test TBH.

Need to do some more substantive work. Closing for now.


Lots of ignored functions, due to std::shared_ptr

Brief Description
I tried to use CppSharp for a wrapper for some Orbbec SDKs and am struggling with some of the functions missing. Mostly the functions are missing because of returning std::shared_ptr<>, which seems to be not supported.
Is there anything I can do to continue to work?
OS: Windows / OS X / Linux (include version and/or distro)
Win 11
Used headers
libobsensor.zip
Used settings
        driver.Options.Verbose = true;
        driver.ParserOptions.Verbose = true;
        driver.ParserOptions.AddArguments("-fcxx-exceptions");

        var options = driver.Options;
        options.DryRun = false;
        options.GenerationOutputMode = GenerationOutputMode.FilePerUnit;
        options.OutputDir = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "out");
        options.GeneratorKind = GeneratorKind.CSharp;

        options.CheckSymbols = true;

Target: MSVC/GCC/Clang
I used the current nuget package with VS2022, running with  x64_x86 Cross Tools Command Prompt for VS 2019
(MSVC 14.29.30133)
Other settings
Stack trace or incompilable generated code
cppsharp.log

thats what I did, thanks.

options.CheckSymbols = true;

hi @frankenherm  how did you fix this please? i'm encountering the same.

I wrote a wrapper class layer, which manages the std::shared_ptr of each object and just expose a regular pointer instead.


Fix #8819.

Adds _import_node() that, when used in conjunction with _import_scene, recurses through the scene tree and exports all available nodes.
Important note: This is not tested with nested sprites, although it should work. More eyes on this are gladly welcome.

Could you squash the two commits together? See http://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#mastering-the-pr-workflow-the-rebase
Also if you could make the corresponding PR for the master branch, that would be nice. I prefer not to merge new features in the stable branch unless they are implemented in the development branch too.

Closed in favor of #8838, which is based on master.

Well I wouldn't mind merging that one too :)


CLI Install Fail on Ubuntu ARM VM in Parallels (macOS - M1): ELF Not Found

The CLI install is failing.  As per the install instructions on the remnux site (https://docs.remnux.org/install-distro/install-from-scratch), I'm getting an ELF Not Found.  This Ubuntu ARM VM is on Parallels (the ARM is required for the macOS M1).
Not sure if this is failing due to compatibility with Ubuntu ARM or for some other reason.  Input in greatly appreciated.

Sorry, REMnx is not compatible with ARM and won't work on M1 or M2 processors.

Kind of what I thought.  Appreciate the quick response.  Cheers.


Building duplicate interface declaration

Hi,
I have a mostly Obj-C App but I'm starting a migration to Swift so I've made all my cocoapods build as frameworks instead of static libraries. But the FBSnapshotTestCase has a problem when you try to build it as a framework.
I keep getting the error:
Duplicate interface definition for class 'FBSnapshotTestCase'
I've set up with a testing target and set up so the project builds exclusively for that testing target.
target 'Screenshot Tests', :exclusive => true do
pod 'FBSnapshotTestCase'
end
I've tried also installing only /Core but that doesn't work either.
I'm not having any problems with other pods, so I'm curious if something is configured oddly on this side of things.
I've set up a sample project here: https://github.com/HaloZero/FBSnapshotBuildError

Hi,
I am not sure what exactly is going wrong here, can you try to change the import in Screenshot_Tests.m from:
#import 'FBSnapshotTestCase.h'

to:
#import <FBSnapshotTestCase/FBSnapshotTestCase.h>

Seems to work that way, thanks!

Great! I am closing the issue.


limit length of response status line for _exc_with_message in forklift/legacy

addresses cases like #10501 where the Response Status line will be much larger than is reasonable.
in theory this is crashing the request at some part of our HTTP stack, I believe between our gunicorn servers and the TLS terminator right above it.
Fixes #10501.

addresses cases like #10501 where the Response Status line will be much larger than is reasonable.
in theory this is crashing the request at some part of our HTTP stack, I believe between our gunicorn servers and the TLS terminator right above it.
Fixes #10501.

gh pr checkout 10508


Update AbsorbCommandTest.java

While attempting to prove my hypothesis concerning failed tests in https://github.com/kolmafia/kolmafia/pull/1889 I ran 1889 so that Gradle only used one fork.  I got some surprising test failures.  I then ran the main branch tests with only one fork and got a failure in AbsorbCommandTest (which also occurred in 1889).  The test absorbs 15 items and expects the client that managed it to have 16 responses - 15 for the absorptions and one for a call to get the charpane.  The test failed because there were 17 responses.  The debugger identified the additional response as a fetch of the image https://d2uyhvukfffg5a.cloudfront.net/otherimages/gladiatar.gif  This is reasonable given that the charpane HTML used in the test referenced the image.
I don't know whether FakeHttpClientBuilder can leak or not.  But if it can then one explanation is that another test loaded the image.  Using one fork changed the order tests were run in and caused this failure.  The other test doesn't care how many responses there were to the request client.
This fix makes it so the test doesn't care whether there is a response for the image, or not.  Another fix would be to delete the check for the response count.  I could argue that checking that does not effect the robustness of the non-test code.  Verifying that FakeHttpClientBuilder cannot leak via image caching or fixing that is another possibility.

RabbitHoleManagerTest > Chess > canAutomateChessPuzzleFromRelayBrowser() FAILED
org.opentest4j.AssertionFailedError at RabbitHoleManagerTest.java:146
Let's see if it is reproducible.

Not immediately.  I have seen RabiitHoleManager fail elsewhere, intermittently.


CARLA setup error and Running CARLA server without display for evaluation

Hi,
I am facing some doubts while trying to evaluate the model, so please guide.
I trained the transfuser model by running
CUDA_VISIBLE_DEVICES=0,1,2,3,4,5,6 OMP_NUM_THREADS=48 OPENBLAS_NUM_THREADS=1 torchrun --nnodes=1 --nproc_per_node=7 --max_restarts=0 --rdzv_id=1234576890 --rdzv_backend=c10d train.py --logdir log --root_dir data --parallel_training 1 --epochs 32 --batch_size 48 --id transfuser
and saved the 32 model_n.pth and optimizer_n.pth
So now I'm trying to evaluate the model as in README.md,

When I setup and install the CARLA <IP_ADDRESS>, I face the following error ERROR 404: Not Found

I'm trying to evaluate the model using the HPC remote server, so in this case how to run the CARLA server without display?
Can I run the server for Without Display - Without Docker condition as,
SDL_VIDEODRIVER=offscreen SDL_HINT_CUDA_DEVICE=0 ./CarlaUE4.sh --world-port=2000 -opengl
and then evaluate the agent using,
./leaderboard/scripts/local_evaluation.sh <carla root> <working directory of this repo (*/transfuser/)>
Please guide on how to evaluate the model and save the output files instead of stream display.

Thanks in advance.
Regards,

The download link for carla changed. You can use this script to download it (will update later).

If the carla server and the evaluation script are on the same machine then yes, otherwise you need to set the ip and port etc.

I updated the links in the download scripts.


Error: server dropped connection, error=error code PROTOCOL_ERROR

Whenever, I request at of http2 smuggl on a any of domain showing this type of error - Error: server dropped connection, error=error code PROTOCOL_ERROR
How we get what is vulnerable or not

Same here

Hello,
The reason for this error is that the domain probably does not support HTTP/2 at all. Also, make sure you provide https link (not http one).


Add % for hexadecimal character

I think this should be 0x as a prefix in both cases, actually...

That makes sense. I have changed the prefix.

The test failure is unrelated: see #71


[WIP] Add test and Docs for simple motor

Joao, do you want me to already review it?

for sure !

I've already noticed some imports that are irrelevant for the code, that i will fix it in the next commit, but besides that would be glad to know if there's anything to improve.

Will do!