Unable to source ROS source files (creates temporary files in /tmp/)

When I'm trying to use the ROS framework for robotics, I'm not able to source the overlay files. The command source /opt/ros/indigo/setup.zsh fails with the following error
/opt/ros/indigo/setup.sh:52: file exists: /tmp/setup.sh.CkuqltdAon
Failed to run '"/opt/ros/indigo/_setup_util.py" ': return code 1

The curious fact is, the command when added to .zshenv works fine, but throws the above error when it is run from the terminal or when added to .zshrc (The error output shows up at the beginning of a new terminal when added to .zshrc)
This is the code snippet which fails (lines 41-52 from /opt/ros/indigo/setup.sh which is called by /opt/ros/indigo/setup.zsh) :
# remember type of shell if not already set
if [ -z "$CATKIN_SHELL" ]; then
  CATKIN_SHELL=sh
fi

# invoke Python script to generate necessary exports of environment variables
_SETUP_TMP=`mktemp /tmp/setup.sh.XXXXXXXXXX`
if [ $? -ne 0 -o ! -f "$_SETUP_TMP" ]; then
  echo "Could not create temporary file: $_SETUP_TMP"
  return 1
fi
CATKIN_SHELL=$CATKIN_SHELL "$_SETUP_UTIL" $@ > $_SETUP_TMP

This snippet fails at the last line.
The file mentioned in the error inside /tmp/ is only a temporary file made with a random string name appended to it and the script being sourced runs fine when used in .zshenv which leads me to believe it is a bug. Ideally I should be able to run it inside the terminal as is possible when not using a framework.

Line 22 of modules/direcotory/init.zsh is your problem.
I have unset that option because I have got burned overwriting the wrong files with no backups in the past. You may delete that line, add setopt clobber in zshrc bellow Customize to your needs... or temporarily setopt clobber then unsetopt clobber around the sourcing of your file.

Thank you for the quick resolution - the issue has changed now. Your suggestion took care of the previous error, but this is the error it throws now:
setup.sh:88: parse error near `\n'

I've checked the file in question and this appears to be the last line in the file, which is blank. The file is valid since it works fine when sourced from .zshenv, so this must not be an error due to a missing brace or such.

That code may be BASH-only and will not work on Zsh.

setup.sh is a file that is internally sourced by setup.zsh (which is the file I'm trying to source - please refer to issue details on top for more details ) after setting a few variables in place for zsh compatibility, so it should work in zsh.
Again, it works perfectly fine when setup.zsh is sourced from within .zshenv (as mentioned up top) so I hope there is some way to get around this. I can gist the files if so required.

You shouldn't be sourcing it in zshenv but zprofile.
Syntax errors are not a Prezto-induced problem since it would not work with or without ROS.
Without seeing those ROS files, I have no way of figuring out what the problem is.

Please find the files setup.zsh, setup.sh and _setup_util.py here.

These files are autogenerated by ROS,  not hand-written / edited.
There isn't a syntax error in them since sourcing the same files work fine when done from zprofile or zshenv ( or when not using Prezto).
All three files gisted at the above link are inside $_CATKIN_SETUP_DIR.

I sincerely thank you for your patience, I hope you take your time and get to the bottom of this at your convenience.

After I setopt clobber, it sources and executes fine on my end. I can't help more.

For anyone stumbling by this issue in future, the problem was caused by this custom alias I had set
alias done="exit"

forgetting that done was a valid shell keyword which the alias may overwrite. Serves me right for only checking if the term was an executable/program before setting the alias. Needless to say, renaming the alias solved the parse error.
@sorin-ionescu :  Only once I got over the first error as per your suggestion, was this possible to debug on my own. Thank you for the quick response.


chore(main): release 3.11.0

:robot: I have created a release beep boop
3.11.0 (2024-12-17)
Features

Create automerge.yml (8cec193)
Update automerge.yml (3ddfd2e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v3.11.0
:sunflower:


Uses deprecated Fixnum constant

The Fixnum constant is deprecated in Ruby 2.4, to be replaced with Integer. This causes a warning to be printed:
/home/mike/.gem/ruby/2.4.0/gems/formulaic-0.3.0/lib/formulaic/form.rb:11: warning: constant ::Fixnum is deprecated

Let's find a way to silence that warning.

We can use Integer instead of Fixnum and all will work as desired.
~% ruby -v
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
~% irb
irb(main):001:0> 1.is_a?(Integer)
=> true
irb(main):002:0> 1.class
=> Fixnum

~% ruby -v
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-linux]
~% irb
irb(main):001:0> 1.is_a?(Integer)
=> true
irb(main):002:0> 1.class
=> Integer

Verified all the way back to 2.0.0

I saw this yesterday. Thought it was Rails' fault.


No worky as advertised

I paid thousands to use this, but the garment didn't even fit. I tried to plug it in and I fell hard. None of this works right.

Honestly, I don't even know why we're friends.


Support for write through cache.

Could you please provide some details/functionalities for the same. I would like to pick this up.


update description on VNet requirement

update description on VNet requirement to indicate the requirement of custom domain.

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


Tagging a 4.2.1 release?

@Naios hi!
We consume the library through conan, and were wondering whether you could tag a 4.2.1 release to include the two minor fixes accumulated since 4.2.0 (the fix to the unsafe_unlocker compilation failure + the clang coroutines detection fix).
I think once a tag is made I could open a PR to include it in https://github.com/conan-io/conan-center-index/blob/master/recipes/continuable/all/conandata.yml .
Thanks for considering!

Hey,
sure, I have created a 4.2.1 tag that includes the latest commits:
https://github.com/Naios/continuable/releases/tag/4.2.1

Thanks! 🙏


improvements in native calls with optional arguments

Consider the HttpRequest.open has this API:
open(method, uri, {bool async, String user, String password});

Both Dartium and dart2js have special logic to deal with these optional arguments. They both call
open(method, uri); if no arguments are specified, and open(method, uri, async, user, method); if they are specified. However, Dartium checks whether all values are null, whereas dart2js checks for the presence of the arguments at the call site.

These should be made consistent.

@alan-knight @rakudrama - what are your thoughts on this?

Another concern when moving from optional named parameters to optional positional parameters is
send(m, u, user:user) // dart
send(m, u, async?, user) // how to pass through default?
This seems to be a general problem with dart's optional params, wish we had a USE_DEFAULT symbol

Having a symbol for default would be very useful in allowing skipping of positional arguments and having apis that go from named optional to positional optional
class Base {
String a;
String b;
void set([String a: 'a_default', String b: 'b_default']) {
this.a = a;
this.b = b;
}
class Derived extends Base {
@override
void set([String a, String b]);
void funnySet({String a, String b}) => set(a, b)
}
new Derived()..set(); // should have a=  'a_default' and b =  'b_default'
new Derived()..funnySet(b:'foo') // should have a=  'a_default' and b =  'foo'
trying to read an unset param should be a compile time error.

what is the direction of translation here?
are we starting with dart code and dart2js creates supposedly equivalent js code?
if that's the case, then
"...whereas dart2js checks for the presence of the arguments at the call site"
seems like a bug.
on the other hand, if we are going from js to dart, then dart2js is a misnomer ("dart 2 js").
if we have 2 hand-coded implementations that some person claims are
semantically equivalent, then again, we seem to have a bug.
i'm not sure why we'd want the js semantics since they don't even make sense in dart:
callee's can't directly detect whether they've been passed params, although there is a kinda-sorta workaround that works as long as the caller is in a different library than the callee:
const _UNDEFINED = const Object();
void f({x: _UNDEFINED}) {...}


Login with Opera

Cant login to contriboard with Opera!

Didnt get this error with normal usage. Was able to log in with Opera on Windows and Linux. Maybe this issue shows up when the API is experiencing a high load.
Next time it would be good to include the network log from the browser (CTRL+SHIFT+J).

This works, yeah

Tested and working
Contriboard client: v1.2.66
API: v0.4.26
IO: v0.2.7
IMG: v0.1.9
Opera:
Version:	30.0.1835.88 - Opera is up to date
Update stream:	Stable
System:	Ubuntu 14.04.2 LTS (x86_64; Unity)


Backport: Upload sha256 sums of cargo artifacts

 Warning

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.14.0. Please double check that you specified the right target!

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

This is a backport of https://github.com/rust-lang/cargo/pull/3372 to ensure we upload these artifacts on beta.
@bors: r+

:pushpin: Commit e3c0e07 has been approved by alexcrichton

:hourglass: Testing commit e3c0e07 with merge b50cb72...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing b50cb7291a05694d0dfe85cc310c1d934543156c to rust-1.14.0...


PreferJavaTimeOverload: don't flag methods on Reactor's Flux type

For example, one can either a buffer up to a certain number of values or for a certain amount of time; the latter operation is not always a suitable alternative to the former. Similar arguments apply to the cache, replay, skip, take, and window operations.
I'm not sure whether Reactor is used inside Google, but I hope that this library is deemed sufficiently popular to meet the threshold for inclusion.

Force-pushed and updated the commit message.

Thanks - I've got a CL out to add this internally.

Tnx for the quick response! Just for my curiosity: the tests were left out to avoid accruing too large a set of odd test dependencies over time?

to avoid accruing too large a set of odd test dependencies

Indeed :-)


Cloud Logging Error serialization

Environment details

OS: OSX 10.12.1
Node.js version: v7.0.0
npm version: 3.10.8
google-cloud-node version: 0.44.2

Steps to reproduce

require google-cloud
Use something like log.error.

const entry = log.entry('gce_instance', new Error('oh no!'));
log.error(entry);

NOTE: For what it's worth, I created the above snippet on-the-fly, so it may contain mistakes or not run as-is -- my existing code is intertwined with project-specific configuration and business logic, so it's not feasible to copy/paste directly. The above snippet sufficiently illustrates my perspective though! 😄
Description
Per a bit of trial-and-error, it appears as if there's not a default serializer of some kind for Javascript Error objects when using Cloud Logging through log#write or any of the helper functions (eg. error, alert, critical).
While a new logging entry is indeed created within the GCP Logging administrative view, it doesn't include anything beyond the default metadata for each entry written. In other words, I know that an error occurred, but the extent to which I'm able to troubleshoot it from there is pretty close to nil. In a sense, it effectively skips over the "payload" of the log entry (the Error itself) and creates said entry with the body omitted.
Specifically, the scenario in which this is pertinent is with catching and logging errors. Typically, my callback function or promise error handler would invoke logger.error(err) -- in fact, that's what led me to discover this existing handling in the first place! For many of my production logging entries, I was seeing that errors were being thrown, but the log entry itself wasn't very useful apart from that 😄
TL;DR / Proposal
I think slightly more intuitive behavior would be to have some sort of reasonable default in place as a serialization mechanism for the Error type/class, like using Error.message / toString().
Currently, I've switched my code to doing something like the following in the interim: (err && err.stack).

@dsimmons that's a pretty sane idea. Right now, I believe you'll have the expected behavior if you just nest the Error object inside of an object, i.e.:
const entry = log.entry('gce_instance', {
  error: new Error('oh no!')
});

// Create entry:
Entry {
    metadata:
     { timestamp: 2016-12-12T16:23:44.685Z,
       payload: 'jsonPayload',
       logName: 'projects/nth-circlet-705/logs/sys_log',
       resource: [Object],
       protoPayload: null,
       textPayload: '',
       jsonPayload: [Object],
       severity: 'ERROR',
       insertId: 'yz0oqlf9i4i8v',
       httpRequest: null,
       labels: {},
       operation: null },
    data: { err: 'Error: Uh oh!' } }

We could convert an error object using the two standard properties, name and message:
var error = new Error('Uh oh!')

var entry = log.entry(error)
// same as
var entry = log.entry({
  name: error.name,
  message: error.message
})

What do you think?

Doh, I hadn't thought to try that!
I think that's a very sane proposal and right around what I had in mind, especially as a "lowest common denominator" / "out of the box" serialization mechanism.
Like you said, the above two properties are standard, and for that reason, they make a lot of sense. It may be worth considering/investigating the possibility of doing some sort of "enhanced" behavior that captures the full stack trace -- if I'm not mistaken, error.message just captures the first line / the string that was used to create the Error object. However, I'm not sure there's a fully standard way of capturing/accessing the stack trace across the board, so for that reason, I'm less confident as to whether it makes sense to be handled by the library versus left up to the consumer.
But, to wrap back around, the above would just be an added nicety. I think in any situation, what you proposed would be very helpful and a fantastic place to start -- even just knowing what the error.message is without the stack trace is helpful in diagnosing issues (especially versus the alternative of knowing that an Error was thrown somewhere and nothing else).

This issue may no longer be relevant due to its age. Feel free to re-open.

I am no longer on the team


Different outcome between command line and stand-alone Linux version

Hello,
I am getting different results when I extract a PDF table using the stand-alone version and the java command line.
For my tests, the stand-alone version performed better than the command line one.
Is this a known issue?
Do you need an example file?

You will need to provide the file and commandline/configuration. If you can't upload the file publicly, say so.

I kind of have the same issue on MacOSX. I run tabula-java with an area defined and tabula-java seems not to be handling well the coordinates (I had to use the -g param). On the other side when I select an area with tabula it works well. In addition the -g usually find the table but sometimes has an issue with the order of rows ( putting the first ones at the bottom ). This issue doesn't happen if I have selected an area with tabula.

Here is the file : Test.pdf
Here is the command :
java -jar ../TabulaJava/tabula-java/target/tabula-1.0.0-jar-with-dependencies.jar -p 32 -a 275,0,1263,892 -o result.csv Test.pdf

Oddly enough if I put the whole document and select -p 32 I don't have exactly the same result that if I print the page I am interested in, as pdf ( from a pdf ) and process the one page with -p 1.

The area you provided does not look right.
Afaik the unit is 1/72 inch.
You can verify your area with the integrated debug util (see note below Usage Examples):
$ java -cp ../TabulaJava/tabula-java/target/tabula-1.0.0-jar-with-dependencies.jar \
technology.tabula.debug.Debug -p 1 --region --area 432,0,1263,892 Test.pdf

This produces the following image Test-1.jpg:

You may also want to use "stream" mode, as the pdf does not seem to have any spreadsheet rulings.
After some GIMPing this is my resulting area & result csv:
$ java -jar ~/build/tabula-java/target/tabula-1.0.0-jar-with-dependencies.jar \
--area 302.967,30.024,762.048,564.048 Test.pdf --stream
"",,Number of Shares,Percent of
Name and Address,,Beneficially Owned,Class
Capital Research Global Investors (1),,,
333 South Hope Street,,,
"Los Angeles, CA 90071",,"51,791,241",12.02%
"The Vanguard Group, Inc. (2)",,,
100 Vanguard Blvd,,,
"Malvern, PA 19355",,"26,239,799",6.09%
"BlackRock, Inc. (3)",,,
55 East 52nd Street,,,
"New York, NY 10055",,"25,402,449",5.89%
The Growth Fund of America (4),,,
6455 Irvine Center Drive,,,
"Irvine, CA 92618",,"23,402,761",5.43%
Reed Hastings (5),,"11,823,357",2.71%
Jay C. Hoag (6),,,
528 Ramona Street,,,
"Palo Alto, CA 94301",,"6,428,870",1.49%
Neil Hunt (7),,"1,368,281",*
Ted Sarandos (8),,"679,088",*
Greg Peters (9),,"241,016",*
David Wells (10),,"192,756",*
A. George (Skip) Battle (11),,"165,090",*
Richard N. Barton (12),,"117,676",*
Leslie Kilgore (13),,"66,468",*
Timothy M. Haley (14),,,
c/o Redpoint Ventures,,,
3000 Sand Hill Road,,,
"Building 2, Suite 290",,,
"Menlo Park, CA 94025",,"61,785",*
Ann Mather (15),,"36,718",*
Bradford L. Smith (16),,"13,965",*
Anne M. Sweeney (17),,"13,965",*
All directors and executive officers as a group (16 persons),(18),"21,561,565",4.90%

@criztovyl Thanks a lot. Trying your suggestion now.


Clarify first part of install instructions in README

We had some initial confusion with where to place the stylecop directory after cloning this repo. With these changes in the README I hope that it should be more explicitly clear for people in the future.
Thanks for creating this great utility!

Awesome! Thanks for your contribution!


[SPARK-15160][SQL] support data source table in InMemoryCatalog

What changes were proposed in this pull request?
This PR adds a new rule to convert SimpleCatalogRelation to data source table if its table property contains data source information.
How was this patch tested?
new test in SQLQuerySuite

Test build #57895 has started for PR 12935 at commit 53e7d27.

Test build #57895 has finished for PR 12935 at commit 53e7d27.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57895/
Test PASSed.

Looks good. The only thing is that the write path seems to be already handled even before your patch. Is that true? I tried removing the InsertIntoTable check in FindDataSourceTable and things still work. I don't think that code path is used anywhere.

Oh, It was used when we call df.write.insertInto, but then I found there is a bug and it doesn't work under hive either. So I change to use df.write.saveAsTable and that code path is not used now.

I'll update it after https://github.com/apache/spark/pull/12949

Test build #58338 has started for PR 12935 at commit 98cea9a.

Test build #58338 has finished for PR 12935 at commit 98cea9a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58338/
Test PASSed.

https://github.com/apache/spark/pull/12935/files#r62910625  is my comment. Other parts look good.

Test build #58434 has started for PR 12935 at commit 1018537.

Test build #58436 has started for PR 12935 at commit ef6399a.

Test build #58434 has finished for PR 12935 at commit 1018537.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58434/
Test FAILed.

Test build #58436 has finished for PR 12935 at commit ef6399a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58436/
Test FAILed.

test this please

Test build #58443 has started for PR 12935 at commit ef6399a.

LGTM pending jenkins

Test build #58443 has finished for PR 12935 at commit ef6399a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58443/
Test PASSed.

Merged build finished. Test PASSed.

Thanks. Merging to master and branch 2.0.


permission?

I saw these error messages on my terminal :
remote: Permission to amjltc295/PythonHomework.git denied to DennyKuo0809.
fatal: unable to access 'https://github.com/amjltc295/PythonHomework.git/': The requested URL returned error: 403
@amjltc295 Excuse me. I don't know what happened , and how can I fix it?

You should fork this repo, and push your commit to your forked repo.

If you want the send a PR, press 'New pull request' in your own repo.
If you want to push a new commit, make sure that you are pushing into your own repo instead of this.

@DennyKuo0809  You were pushing commits to this repository, which is prohibited.
I guess you did something like
git push origin master
Please make sure your origin is your forked repository, not this one.
You could check that by git remote get-url origin
To set another remote that is your forked url, use
git remote add forked <your forked URL>
If forked already exists, try git remote set-url forked <your forked URL>
Once the URL is set, try
git push forked master to push commits to your forked repo


Can we add new feature so that we can forward particular message.

selected msg should have a option to forward that msg to other users.

You can use message-actions prop with message-action-handler to add a custom option.
You can add custom options to selected messages with message-selection-action and message-selection-action-handler


multiples users

Hi, your great software work only with principal user
in my island have 4 users playing
best regards

Why was this closed? Was it fixed?

I'd be interested in that, too. I also play mainly with the secondary character


nixos/fontconfig: add fonts.fontconfig.hinting.interpreterVersion

Description of changes

I want an easier and documented way to enable Infinality subpixel rendering (yes, this is still supported by upstream), because at least in my monitor (~110PPI) it results in, in my opinion, better font rendering. So this PR adds support for changing it by setting fonts.fontconfig.hinting.interpreterVersion.
To avoid breaking anyone setups, this accepts a null value that is also the default. This is also good to avoid having a hardcoded value in case upstream updates the interpreter version again.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test fontconfig-default-fonts

CC @K900.


No API docs for Geodesic Module

🐞 Problem
https://docs.einsteinpy.org/en/latest/api/geodesic.html
This is mostly empty for some reason. @JeS24 you might have an idea why this was not pushed?
🎯 Goal

💡 Possible solutions

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request.
Add this in the comments:

Addresses #<put issue number here> if you are partially fixing the issue.
Fixes #<put issue number here> if you are completely fixing the issue.

Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

I would like to work on this

@shreyasbapat Weird. Let me take a look.

Fixed in #548.


allowed tauchen to take non-integer n_std positional argument

A long time ago I asked whether tauchen should limit n_std to being an Integer:
https://discourse.quantecon.org/t/integer-n-std-in-tauchen/541
I don't think there should be such restriction. This PR relaxes n_std to be a subtype of Real.

Thanks @amrods , this seems reasonable to me.  I'll leave it to the maintainers to have their say.

aside, it seems that the julia side of the project has been dormant. is that the case?

It has for a while, @amrods , but there's been a heap of work behind the scenes to bring it up to date.
@mmcky and @jlperla are running the show but several RAs are also hard at work.  The updated lecture site should go live in a week or so.

Yes @amrods  Lots of progress.  The reason things seem dormant  is because of the move towards the new build system based on https://jupyterbook.org/intro.html  as well as a large number of changes in the notebooks to move to VS Code from Atom.
See the source in https://github.com/QuantEcon/lecture-julia.myst

@amrods  For this particular change:  Up to you guys as I am not a maintainerit, but the best approach is to just drop all of those type declarations entirely.  tauchen is not using multiple-dispatch, so it is always better to leave function arguments without a type.
You just never know what types someone will push through it.  Case in point: if there were no type declarations in the first place, then this PR wouldn't even have been necessary as everything would have just worked.

Agreed, best not to use type declarations whenever possible. I was just following the style already present.


Fixed name of OS X target

Fixes #5

I see 2 commits, not clean use of git.
I will not merged this one
(Use rebase instead of pull to avoid merge commit, or/and use branch if you use your repo)
I add you as colaborator so you can checkout this repository or add a new remote origin if not done automatically
Then commit the fix here
In commit message add fix #5 to autoclose the issue

I usually do a rebase (or force FF merge), but I used a GUI tool without thinking about it. My bad. It's down to the single commit now, with the expanded commit message setup to close this issue.

I have done it fa9f392cead20a5649eec323c1e98aa125344f0c without seeing that you fix your pr


Support for additional node flags like --loader

Hey 😀
I am wondering if there is any option to provide additional node flags like --loader to this plugin? In my case, I usually use something like node --loader esmock [...] to start my tests. Any hint is highly appreciated!
Cheers 🚀
Robert

Kinda related: https://github.com/connor4312/nodejs-testing/issues/4

I think this could be solved by adding an additional setting and then passing that along to
https://github.com/connor4312/nodejs-testing/blob/17a5620b44e4a250a5d24f49c7e429581cc4d17e/src/runner.ts#L190
The options could be named nodejsParams and would be similar to the existing nodejs-testing.include.
probably supporting the launch variable like ${workspaceFolder} would make sense so you could then set it like so
nodejsParams Array:

--import
${workspaceFolder}/my-file.js
--loader
esmock

and then we could just pass it along to the spawn
such a feature would solve this and #8
@connor4312 shall I prepare a Pull Request for this feature?

Sure, I'd be happy to take a PR, just haven't had a lot of time to work on this recently.

nice - I will start working on it 👍

Published in 1.1.0


CIS Scans: Clicking on a scan in progress shows no data

Install the CIS Benchmark tool.
Go to CIS Benchmark > Scans and Create a scan.
Click on the scan name - it doesn't show that the scan is in progress, or the profile - compare this with what you see when the scan has completed.
Sometimes I would see that if you go back to the scans page and wait, the page does not update - this might be for the very first scan you do.

Can't reproduce the issue:
https://zube.io/files/rancher/41302387947b7d7a05e35d810ce63103-4171.mov
@gaktive should we close this issue?

Let's close since this is super old.


Error: Could not find module remark-preset-lint-recommended

I get an error Error: Could not find module remark-preset-lint-recommended while trying to use linter-markdown.
I'm using Atom 1.23.0 for debian. I installed the following package locally, in the same directory as the markdown file I want to lint:

"remark-preset-lint-consistent": "^2.0.1",
"remark-preset-lint-markdown-style-guide": "^2.1.1",
"remark-preset-lint-recommended": "^3.0.1"

I get a basic package.json file
$ more package.json
{
  "name": "dummy-package",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "repository": {
    "type": "git",
    "url"<EMAIL_ADDRESS>  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "remark-preset-lint-consistent": "^2.0.1",
    "remark-preset-lint-markdown-style-guide": "^2.1.1",
    "remark-preset-lint-recommended": "^3.0.1"
  }
}

It is not clear for me how to install and use linter-markdown for Atom.

@oupala If you use your own configuration, for example if you’d rather not use the defaults, you can use locally installed modules with a remark config file. If you plan on doing this, could you show me the config file as well?
Otherwise, if you’d like to stick with the defaults, you don’t need to add any dependencies.

For a default configuration, I don't need any dependencies. But I still have to install remark and remark-cli globally. Am I right?
If I want my own configuration, I have 2 solutions:

either I install all dependencies globally, alongside remark and remark-cli
either I want a local and specific configuration, I have to add the dependencies in the | package.json` file

Am I still right?
If yes, I think that it needs to be more clearly explained in the README file.

For a default configuration, I don't need any dependencies. But I still have to install remark and remark-cli globally. Am I right?

If you’re not going to configure linter-markdown, you don’t need to install anything other that linter-markdown in atom.

If I want my own configuration, I have 2 solutions:

Yes, but your first solution doesn’t work currently.

If yes, I think that it needs to be more clearly explained in the README file.

Feel free to create a PR for the docs!

PR proposed, see #167

I'm rather confused. If want to use the default configuration that comes "out of the box" with the Atom linter-markdown package, all is well, but if I want to do any configuration whatsoever, I have to install other stuff?
This seems bizarre. I would have thought that I could have just tossed some settings in a ~/.remarkrc file to disable some of the default stuff.
As a novice user, I'm more than satisfied with the default set of plugins. I just want to tweak the settings they're using. Is this not possible without installing other things?

As  a novice user, I'm more than satisfied with the default set of plugins.  I just want to tweak the settings they're using. Is this not possible  without installing other things?

That's it. Due to some bugs and limitations, you cannot tweak configuration without installing some modules.

OK, thanks for the confirmation. I'm going to log an issue for that, because this dramatically reduces the utility/convenience of the linter. I don't have npm or nvm installed, so I would have to install one of those, then remark and remark-cli, then whatever packages I'm using, just to get back to the basic functionality that comes with the package...

@oupala Have you tried uninstalling and re-installing?

Nope, I did not tried this.
But what would you reinstall? Which npm modules?

Oh sorry, I meant to uninstall the linter-remark plugin, and add it again. Maybe that helps.

Something must have changed somewhere because it does now work again while I have not modified anything...


Add signatures to followers file

Must review and merge #580 first

Coverage decreased (-0.3%) to 26.231% when pulling fd6b04b313f82607b302dc20d7c88c55fb349d9a on followsig into 98735ac9e0f4f3073fdb718e4d7bc623118c2bdf on master.

Coverage decreased (-0.04%) to 26.327% when pulling 9290c23018c8bc590d0b2555bb4e76ce6ff4af56 on followsig into 865a4240308e98146515bc8fefc6dda2cf390fe4 on master.


Added delegation option to multisig aids

Test with delegated AIDs won't work until the latest Verifier and API changes won't be merged


internal/graphicsdriver/directx: directx: IDXGISwapChain::Present failed: HANDLE(2289696769)

Ebitengine Version
v2.6.0-alpha.2
Operating System

[X] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
go1.19.4
What steps will reproduce the problem?
Not sure. This error was reported in a real world. Probably this error happens when switching fullscreen, but we are not sure.
What is the expected result?
No crash
What happens instead?
An application ended with this error: directx: IDXGISwapChain::Present failed: HANDLE(2289696769)
Anything else you feel useful to add?
No response

/CC @wasedaigo

Now this issue is pretty rare (w/ DirectX 11). Let's clear the milestone.


Git Commit Message Wraps lines at 50 characters instead of 72

Type: Bug

Use Git for editing Git commit messages
Use it to edit a message (the name in the box in the bottom right corner is "Git Commit Message"
Note the wrap lines: one at 50 characters for the subject, another at 78 characters for the body
Place the cursor in some long body text
Use option-q (or alt-q?) to wrap the long text
See that it wraps at the 50 character line instead of 72

FWIW it wrapped at 72 characters until a few weeks ago.
VS Code version: Code 1.87.0 (Universal) (019f4d1419fbc8219a181fab7892ebccf7ee29a2, 2024-02-27T23:42:56.944Z)
OS version: Darwin arm64 23.3.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Max (14 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
2, 2, 2

Memory (System)
36.00GB (0.13GB free)

Process Argv
--crash-reporter-id 0f3c5b21-cecd-40d4-b17d-e913247e5700

Screen Reader
no

VM
0%

Extensions (33)

Extension
Author (truncated)
Version

vscode-openapi
42C
4.21.1

terraform
4op
0.2.5

vscode-sql-formatter
adp
1.4.4

vscode-multimarkdown
and
0.1.1

vscode-abnf
arn
0.2.0

markdown-mermaid
bie
1.21.0

mermaid-markdown-syntax-highlighting
bpr
1.6.0

language-hugo-vscode
bud
1.3.1

perl-toolbox
d97
1.4.0

yash
dao
0.2.9

xml
Dot
2.5.1

EditorConfig
Edi
0.16.4

varnish-vcl
flo
0.1.3

go
gol
0.40.3

hcl
has
0.3.2

crontab-syntax-highlight
hog
0.0.2

protobuf
kan
1.1.6

vscode-just
kok
2.1.0

markdown-shortcuts
mdi
0.12.0

vscode-docker
ms-
1.28.0

cpptools
ms-
1.18.5

live-server
ms-
0.4.13

java
red
1.27.0

vscode-yaml
red
1.14.0

rust-analyzer
rus
0.3.1831

rewrap
stk
1.16.3

code-spell-checker
str
3.0.1

even-better-toml
tam
0.19.2

shellcheck
tim
0.35.0

vscode-pets
ton
1.25.1

opa
tsa
0.12.2

sort-lines
Tyr
1.11.0

material-theme
zhu
3.16.2

(5 theme extensions excluded)

Same here, after updating to VSCode 1.86.1 this behavior broke. It no longer seems to respect the settings

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

Here you go. Note how when I try to wrap the body text, it wraps at the subject line length instead of the body line length.

That is not a command that is part of VS Code.
Could it be contributed by the rewrap extension?

Oh! I completely forgot that wasn't a core feature of VScode Perhaps a previous version of VScode only had the one wrap column at 72, and now also having one at 50 causes an issue because it's the first one that rewrap sees?
@stkb what do you think? Is there some way the rewrap extension can know which wrap column is relevant to which bit of text?

Closing the issue as it's related to an extension.

re-filed at stkb/Rewrap#395, thanks!

figured out a fix. The default ruler config for git-commit is:
  // Configure settings to be overridden for the git-commit language.
  "[git-commit]": {
    "editor.rulers": [50, 72],
    "editor.wordWrap": "off",
    "workbench.editor.restoreViewState": false
  },

The Rewrwap extension uses the first item in the editor.rulers array. The fix is to simply reverse those values. So my config is now:
  "[git-commit]": {
    "editor.rulers": [72, 50],
  }

And now rewrap (and presumably any other tool that relies on the first ruler being the primary) wraps at 72 characters.
Would it make sense for VSCode to change the default here?

Potentially, though that might break other extensions that rely on the order...

If the second column number was only recently added, ISTM like it might be the case that any existing extensions only read the first one previously. Moving it to the second place might be unexpected, as in this case.


Update version to 1.0.95-0 on staging

Update version to 1.0.95-0

🚀 Deployed to staging by @OSBotify in version: 1.0.95-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @yuwenmemon in version: 1.0.95-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Error migrating from Anchor 0.29.0 to 0.30.1 - Unable to pass &AccountInfo as an argument to a validation function in an Instruction

I’m migrating my code from Anchor 0.29.0, where everything was working correctly.
I need to pass an account as &UncheckedAccount or &AccountInfo to a static method which extracts a value from raw data that will be used as part of the PDA seed.
Here is the code that is now broken:
#[account(
    mut,
    seeds = [
         UserMessageNonce::SEED_PREFIX,
         sender.key().as_ref()
    ],
    bump
)]
pub user_message_nonce: AccountInfo<'info>,

#[account(
    mut,
    seeds = [
         SEED_PREFIX_SENT,
         &UserMessageNonce::value(&user_message_nonce).to_le_bytes(), // BREAKS HERE
         sender.key().as_ref()
    ],
    bump,
)]
pub other_account: UncheckedAccount<'info>,

When I compile this, I encounter the following error:
error[E0425]: cannot find value `user_message_nonce` in this scope
   --> programs/my_program/src/instructions/generic_send_message.rs:134:39
    |
134 |             &UserMessageNonce::value(&user_message_nonce).to_le_bytes(),
    |                                       ^^^^^^^^^^^^^^^^^^ not found in this scope

At the moment, the only workaround I’ve found is to avoid providing the seed in the Instruction and instead validate the account address in the code. However, this solution is neither optimal nor clear.

@gwalen I think you shouldnt check constraint in account rather than check inside logic of program.

Thanks for creating this issue, but this is a duplicate of https://github.com/coral-xyz/anchor/issues/2903, so I'm closing this to not divert attention from the existing issue. However, feel free to reopen if you think this is a different issue.
I think the parser needs to be smarter about external functions and omit resolution data when necessary. At the very least, we should show a better error message that explains why the error happens. In the meantime, please consider the workaround mentioned in https://github.com/coral-xyz/anchor/issues/2903#issuecomment-2053637446.


Disable jumping to "tomail:" when refusing to "Share" on a mobile device:: share_plus_web

Use case
when refusing to "Share" on a mobile device then jumped to mail.
but i don't want it

Proposal
plus_plugins/packages/share_plus_web/lib/share_plus_web.dart
  /// Share text
  Future<void> share(
    String text, {
    String subject,
    Rect sharePositionOrigin,
  }) async {
    try {
      await _navigator.share({'title': subject, 'text': text});
    } catch (e) {
     // i want to new condition
      if(e.toString()!="AbortError: Share canceled"){   //<<---- this line
        final uri = Uri.encodeFull('mailto:?subject=$subject&body=$text');
        return launch(uri);
     }
}

This seems to be a bug. We should not open the email composer if the user cancels the sharing action.

Ah I see what you mean now, this line here: https://github.com/fluttercommunity/plus_plugins/blob/main/packages/share_plus_web/lib/share_plus_web.dart#L33 is always going back to mailto also when the user cancels.
PRs with the fix welcome :)
Is there a better way than to compare the toString() of the error?


'installed: false' doesn't respect global TLS settings and namespace

I noticed that installed: false doesn't respect global TLS parameters and a namespace.
I've got the following in my logs:
exec: helm list ^cert-manager$ --tiller-namespace gitlab-managed-apps --tls --tls-key tiller.key --tls-cert tiller.crt --tls-ca-cert tiller-ca.crt --kube-context=gke_XXXXX
NAME        	REVISION	UPDATED                 	STATUS  	CHART              	APP VERSION	NAMESPACE
cert-manager	1       	Fri May 17 10:32:29 2019	DEPLOYED	cert-manager-v0.6.7	v0.6.2     	cert-manager

Deleting cert-manager
exec: helm delete cert-manager --purge --kube-context=gke_XXXXX
Error: could not find tiller

It's strange because I used the same version before without TLS and it worked.
v.0.54.2

It doesn't use release level TLS settings either.

@andrewnazarov Thanks for reporting! I think I've finally spotted the cause and fixed it in #683. Would you mind trying it?


Update the game over message

Update the game over message so people know how to play again!

this is testing


Error if api.netatmo.net is not responding

Hi!
node-red stops running if api.netatmo.net is not responding:
26 Jan 12:40:09 - Error: Authenticate error: No response
at netatmo.handleRequestError (/home/pi/.node-red/node_modules/netatmo/netatmo.js:46:15)
at netatmo. (/home/pi/.node-red/node_modules/netatmo/netatmo.js:115:19)
at self.callback (/home/pi/.node-red/node_modules/request/request.js:186:22)
at emitOne (events.js:96:13)
at Request.emit (events.js:188:7)
at Request.onRequestError (/home/pi/.node-red/node_modules/request/request.js:845:8)
at emitOne (events.js:96:13)
at ClientRequest.emit (events.js:188:7)
at TLSSocket.socketErrorListener (_http_client.js:310:9)
at emitOne (events.js:96:13)
at TLSSocket.emit (events.js:188:7)
at emitErrorNT (net.js:1278:8)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
is it a problem of node-red-contrib-netatmo, or the netatmo module?
Cheers,
Rainer

I haven't had this error and we are using the contrib-netatmo nodes 24/7 without errors for months.  It might be a bug in your code if you are sending bad credentials.  I'd double check the strings you are entering in the config nodes and your code that forwards them to netatmo module API.  I'll close this issue as it doesn't seem to be in my module.


all: remove repetitive word

Going to close, it's not worth the time of the CLA or our time reviewing for an unimportant typo. Good luck with all your other typo PRs.
Sorry if this seems harsh, but their are hours of OSS mantainers time consumed across all those typo PRs.


add max message length option

I don't think you expose settings directly to the user, so I used a global

Closed because I found https://github.com/nvim-lualine/lualine.nvim/wiki/Component-snippets#truncating-components-in-smaller-window


How I can create a server?

java -jar FoxLoader.jar --server honestly, I'm not sure how to tell you.
What you told me is of real concern, I tried to be as clear as possible.
You can go here https://discord.gg/38Vfes6NpR if you want faster responses.

I would highly recommend putting common use-cases directly in the README, I see no mention of how to launch this as a server within it. Most people use READMEs as a single (or first, at the very least) point of contact for documentation, as much as we'd all love for people to try --help options first :P.

@SpikeHD thank for the feedback, I updated the readme!
Should be good enough!


Outline addon broken link

That links 404's https://chrome.google.com/webstore/detail/outline-read-without-clut/daoolpmoieinofbnddaofhkhmbagfmnj

Fixed


zk-solc compilation does not work in hre.

When i run npx hardhat compile, I do not think it successfully compiles using the zksolc but the regular solc.
my artifacts folder is named 'artifacts' and not 'artifacts-zk'. How do I set it up so it uses the zksolc?

This shouldn't be the case if you imported the hardhat-zksync-solc plugin in your hardhat.config and configured it correctly (https://era.zksync.io/docs/api/hardhat/hardhat-zksync-solc.html). Can you paste here the hardhat.config that you are using?

You need to choose whether you want to use CommonJS module system (default for js files) or ES modules.
For CommonJS, just remove the "type": "module" from package.json and use require("hardhat-zksync-solc"); for importing and module.exports for exporting.
For ES modules, use "type": "module" in package.json (or .mjs file extension) and use import "hardhat-zksync-solc" and export default.
Here is the article explaining the differences of these 2 module systems https://www.knowledgehut.com/blog/web-development/commonjs-vs-es-modules

Deprecated zksolc compiler version in hardhat config file (1.3.6). Please use versions 1.3.13 to 1.3.14.
showing this problem when i deploy contact on zksync era mainnet


[DS-1443] Export Community/Collection structure with structure-builder

Somehow Travis didn't catch a conflicting patch, so fix it.

Merged without review to un-break master.  Sorry about the mess.


Add new DeviceReset Sysman API for supporting FLR based reset

Current DeviceReset API (zesDeviceReset) talk about device reset which in linux we are doing warm reset and user might be interested to do device reset for the following
FLR based reset
Warm reset
Cold Reset
need new API to support DeviceReset specifying the flag which reset user would like to have

@saik-intel suggested adding a new API to provide support:
ze_result_t zesDeviceResetExt(zes_device_handle_t hDevice, ze_bool_t force, ze_device_reset_type_t reset_type);

PR #134

Closed with PR #134


[MAS2.4.7] Focus indicator is not visible on "Upload File" and "Send" buttons.

Actual Result:
Keyboard focus indicator is not visible on "Upload card" and "Send" buttons.
Note:
Same issue repro at -

Url -> Type "accessibility" in text box -> Press Enter -> Navigate to "New message" button on the page by using Tab key - Focus indicator is not visible on “New Messages” button. Please refer the attachment "MAS2.4.7_New message button."

Expected Result:
Keyboard focus indicator should be visible on "Upload card" and "Send" buttons.
Repro Steps:

Open URL https://microsoft.github.io/BotFramework-WebChat/full-bundle in Edge browser.
Notice 'Upload File' and 'Send' button icons present in bottom of screen.
Navigate to Upload File' and 'Send' buttons by using Tab key.
Observe that focus indicator is not visible on "Upload File" and "Send" buttons.

User Impact:
If focus indicator is not visible on any control then the user who is visually impaired will not able to understand the landing of focus.
MAS Reference:
MAS 2.4.7 - Focus Visible (05A)
Test Environment:
OS: Windows 10
OS Build: 17134.345
OS Version: 1803
Browser: Edge
Attachment:

MAS2.4.7_Focus indicator _Upload file and send button.zip
MAS2.4.7_New message button.zip

As part of internal bug review, closing this bug.

#HCL;#Accessibility;#HCL_BotFramework_WebChat;A11yMAS;WCAG2.4.7


Sprint A - Scheduling Interviews with Existing Users

Create a list of existing users, craft an invitation for a UX Interview and invite them to an interview.


Not combining ALL active routes?

Im trying to add a query to main component in order to have "current user" data available in all routes, even in those, where I normally don't have query (static pages). However, these queries doesn't get combined.
Routes example
<Router
	history={browserHistory}
	environment={Relay.Store}
	render={applyRouterMiddleware(ReactRouterRelay)}
>
    // This should always be an active route but this
    // query doesn't get combined with sub-route queries
	<Route
		name="root"
		path="/"
		component={App}
		queries={ViewerQueries}
	>
		<Route
			name="post-list"
            path="/posts"
			component={PostList}
			queries={ViewerQueries}
		/>
		<Route
			name="single-post"
			path="/post/:nodeId"
			component={SinglePost}
			queries={NodeQueries}
		/>
	</Route>
</Router>

Relay container on main component (wraps whole app)
This query's supposed to be there for every single page in order to get "current user" data in every page.

react-router-relay will automatically generate a combined Relay query config with all queries and parameters from the active React Router routes

I was expecting the following two queries to be combined but they don't. Relay makes 2 separate queries instead.
I thought about using getFragment() but the problem is that this component wraps everything. I have no idea how to get unknown, nameless fragment and I couldn't find anything proving that it's possible.
export default Relay.createContainer(App, {
	fragments: {
		viewer: () => Relay.QL`
			fragment on Viewer {
				me {
					id
					name
                    email
				}
                # This is most likely not possible (a'la props.children behaviour)
                ${UnknownChild.getFragment('unknownChildFragment')}
			}`
	}
})

Relay container on post list
export default Relay.createContainer(PostList, {
    initialVariables: {
        count: 10
    },
    fragments: {
        viewer: () => Relay.QL
            `fragment on Viewer {
                posts(first: $count) {
                    edges {
                        node {
                            id
                            title
                            content
                        }
                    }
                }
            }`
    }
})

My question is:
Is it supposed to happen? (not combining queries)
Only alternative I see is to add Relay containers / queries to every single route, e.g static routes gets queries only with "current user" data? In other words: StaticComp usually doesn't have Relay container but now:
export default Relay.createContainer(StaticComp, {
	fragments: {
		viewer: () => Relay.QL`
			fragment on Viewer {
				me {
					id
					name
                    email
				}
			}`
	}
})

They're combined into a single query config for execution purposes. Each route however only sees its own queries – otherwise what would the route component Relay container even get if you had two queries with the same name? If you need additional data in your route components, you need to compose in the relevant query for the route.

I have posted a similar question on Stackoverflow: http://stackoverflow.com/questions/43684386/relay-how-to-merge-instead-of-override-queries-in-nested-routes.
How would one achieve to get the App component to query always for the logged in user? Could you give some code example on this?


0.3.0

Incorporate changes by @jeffmay
Circe to 0.6.1 (and cats to 0.8.1)
New discipline tests for symmetry (which currently fail)

@jeffmay can you take a look at the changes I made here?
I went ahead and published 0.3.0 so I can start baking it against my projects which use it (and now must have cats 0.8.x). I've asked bintray to let me publish snapshots; as soon as they allow it we can move to 0.3.1-SNAPSHOT and go from there.

Seems like a lot of commits for a small amount of changes. Maybe you want to squash before merging?

@jeffmay I'll squash and merge. Personally I don't mind the separate subproject, as mentioned above - it shouldn't impact the usage at all. But we can restructure it a bit for the next minor version.


install.sh won't run due to missing parameters

Hi,
In the very last part of your install.sh where you run the python program. The parameters you are passing in to your deidentify.py script aren't complete.
I.e missing log-id and one other.
Can you supply information on where those come from?

Yes, the arguments that the pipeline accepts are:
--input-pubsub -- to pass the subscription id to listen to
--log-id -- the LOG_ID part of the log name.
--interval -- the interval in seconds that the pipeline uses to aggregated logs

I've the script updated to handle the renaming of the argument that were done in #4

Thank you


Disable sharing on low level paths

This PR adds a config option in the storage provider to disable sharing permissions for folder up to a defined path level.

@dependabot rebase


Moves client JS to apos/apps

Every thing in its place.

the git thing actually isn't relevant. It seems like npm 7 has something against these github dependencies or something in CI. Before release we'll want to switch that to use the published version.


System.ComponentModel.Win32Exception when using ef tools

From @jhoward321 on Thursday, June 13, 2019 3:36:44 AM
Steps to reproduce
Run any dotnet ef command inside a working repo. I'm using the 2.2.107 sdk from the https://copr.fedorainfracloud.org/coprs/g/dotnet-sig/dotnet/ repo. Unfortunately there is not a newer sdk available.
Expected  behavior
Show existing migrations
Actual behavior
$ dotnet ef migrations list
System.ComponentModel.Win32Exception (2): No such file or directory
   at Interop.Sys.ForkAndExecProcess(String filename, String[] argv, String[] envp, String cwd, Boolean redirectStdin, Boolean redirectStdout, Boolean redirectStderr, Boolean setUser, UInt32 userId, UInt32 groupId, Int32& lpChildPid, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean shouldThrow)
   at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start()
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at Microsoft.EntityFrameworkCore.Tools.Exe.Run(String executable, IReadOnlyList`1 args, String workingDirectory, Boolean interceptOutput)
   at Microsoft.EntityFrameworkCore.Tools.Project.FromFile(String file, String buildExtensionsDir, String framework, String configuration, String runtime)
   at Microsoft.EntityFrameworkCore.Tools.RootCommand.Execute()
   at Microsoft.DotNet.Cli.CommandLine.CommandLineApplication.Execute(String[] args)
   at Microsoft.EntityFrameworkCore.Tools.Program.Main(String[] args)
No such file or directory

Environment data
$ dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   2.2.107
 Commit:    2212cac826

Runtime Environment:
 OS Name:     fedora
 OS Version:  30
 OS Platform: Linux
 RID:         fedora.30-x64
 Base Path:   /usr/lib64/dotnet/sdk/2.2.107/

Host (useful for support):
  Version: 2.2.5
  Commit:  0a3c9209c0

.NET Core SDKs installed:
  2.2.107 [/usr/lib64/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.NETCore.App 2.2.5 [/usr/lib64/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Copied from original issue: dotnet/cli#11542

From @RheaAyase on Thursday, June 13, 2019 2:13:55 PM
2.1.507 and 2.2.107 are the latest up to date.
Looking into this with @aslicerh @omajid

From @omajid on Thursday, June 13, 2019 2:47:16 PM
The output of dotnet ef -v migrations list  would be nice to have.

@@jhoward321 In addition to the --verbose output, could you share your csproj?

So I've tried this with a number of different projects. I've been trying to use it on a larger project of mine, and I created a new dummy project... always the same issue. Both are dotnet core 2.2 projects.
Here's the verbose output:
$ dotnet ef migrations add InitialCreate --verbose
Using project '/media/BigDrive2/Coding/ef_error/ef_error.csproj'.
Using startup project '/media/BigDrive2/Coding/ef_error/ef_error.csproj'.
Writing '/media/BigDrive2/Coding/ef_error/obj/ef_error.csproj.EntityFrameworkCore.targets'...
dotnet msbuild /target:GetEFProjectMetadata /property:EFProjectMetadataFile=/tmp/tmpSpTyF8.tmp /verbosity:quiet /nologo /media/BigDrive2/Coding/ef_error/ef_error.csproj
System.ComponentModel.Win32Exception (2): No such file or directory
   at Interop.Sys.ForkAndExecProcess(String filename, String[] argv, String[] envp, String cwd, Boolean redirectStdin, Boolean redirectStdout, Boolean redirectStderr, Boolean setUser, UInt32 userId, UInt32 groupId, Int32& lpChildPid, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean shouldThrow)
   at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start()
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at Microsoft.EntityFrameworkCore.Tools.Exe.Run(String executable, IReadOnlyList`1 args, String workingDirectory, Boolean interceptOutput)
   at Microsoft.EntityFrameworkCore.Tools.Project.FromFile(String file, String buildExtensionsDir, String framework, String configuration, String runtime)
   at Microsoft.EntityFrameworkCore.Tools.RootCommand.Execute()
   at Microsoft.DotNet.Cli.CommandLine.CommandLineApplication.Execute(String[] args)
   at Microsoft.EntityFrameworkCore.Tools.Program.Main(String[] args)
No such file or directory

Here's the dummy project's cs prof file. It doesnt seem related to the project because its happened with all of my projects. I've tried both sqlite and postgres backends.
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.2</TargetFramework>
    <AspNetCoreHostingModel>InProcess</AspNetCoreHostingModel>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.App" />
    <PackageReference Include="Microsoft.AspNetCore.Razor.Design" Version="2.2.0" PrivateAssets="All" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.2.4" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="2.2.4" />
  </ItemGroup>

</Project>

For what its worth I managed to get the tools working with the newest sdk binaries. Would love to get new versions packaged in the repos.

As I said... Our 2.1.507 & 2.2.107 are the latest, there is nothing better (short of compiling the master branch yourself.) Microsoft likes to cause confusion with version numbers, that's all there is to it.
Either way, if there is an issue, it will not be solved by itself with a newer version. That would mean that someone fixed it and released the fix in the newer version - there is no newer version, nobody fixed anything, and nobody can release a fix that doesn't exist.
If we want to actually resolve this, the solution is most certainly not in using Microsoft's blobs which were build in completely different build pipeline than any other open source packages.

@jhoward321 It looks like dotnet isn't on the path. How are you installing the .NET Core SDK?

dotnet is on the path otherwise it wouldn't run - that is dotnet's exception :P
Or rather, that is EF's exception, and for some reason it can't find some (hardcoded or env.var'ed) filepath.
We checked DOTNET_ROOT and ASPNETCORE_ENVIRONMENT which are both set.

Looks like the error comes when we try to execute this:

dotnet msbuild /target:GetEFProjectMetadata /property:EFProjectMetadataFile=/tmp/tmpSpTyF8.tmp /verbosity:quiet /nologo /media/BigDrive2/Coding/ef_error/ef_error.csproj

Can you try running that with \verbosity:normal?

 That command works no problem. I can build projects fine, its only EF tools that don't work.
$ dotnet msbuild /target:GetEFProjectMetadata /property:EFProjectMetadataFile=/tmp/tmpSpTyF8.tmp /verbosity:normal /nologo /media/BigDrive2/Coding/ef_error/ef_error.csproj
Build started 6/15/19 11:18:57 AM.

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:00.24

To keep this moving, dotnet is most certainly on $PATH, the initial issue is with EF throwing a dotnet exception on Linux system where nothing else other than dotnet itself could possibly do that :P (Therefor, dotnet is there, it is found, and it is running.)
@jhoward321 could we also have a dummy CS file to go with the projectfile if you have one? Or how can we create this on our end? Can you reproduce it with a dotnet new <something>? What would the something be, and then what do we need to add? (What nuget packages, what are the c# code changes, etc...)

An strace might be nice to have too:
strace -e trace=%process -fo strace.log dotnet ef -v migrations list

Can you upload the generated strace.log file here?

dotnet-ef.dll is running fine. It throws when we try to Process.Start() this:
dotnet msbuild /target:GetEFProjectMetadata /property:EFProjectMetadataFile=/tmp/tmpSpTyF8.tmp /verbosity:quiet /nologo /media/BigDrive2/Coding/ef_error/ef_error.csproj

My first hypothesis was that it can't find dotnet because it's not on the PATH. We need to debug into Process.Start() to better understand what's going on.

@omajid here's the strace
25063 execve("/usr/bin/dotnet", ["dotnet", "ef", "-v", "migrations", "list"], 0x7ffdec676250 /* 76 vars */) = 0
25063 arch_prctl(0x3001 /* ARCH_??? */, 0x7ffd77f3f820) = -1 EINVAL (Invalid argument)
25063 arch_prctl(ARCH_SET_FS, 0x7f805761f740) = 0
25063 clone(child_stack=0x7f8056bd2fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f8056bd39d0, tls=0x7f8056bd3700, child_tidptr=0x7f8056bd39d0) = 25064
25063 clone(child_stack=0x7f80563d1fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f80563d29d0, tls=0x7f80563d2700, child_tidptr=0x7f80563d29d0) = 25065
25063 clone(child_stack=0x7f8055bd0fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f8055bd19d0, tls=0x7f8055bd1700, child_tidptr=0x7f8055bd19d0) = 25066
25063 clone(child_stack=0x7f80553cffb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f80553d09d0, tls=0x7f80553d0700, child_tidptr=0x7f80553d09d0) = 25067
25063 clone(child_stack=0x7f8054bcefb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f8054bcf9d0, tls=0x7f8054bcf700, child_tidptr=0x7f8054bcf9d0) = 25068
25063 clone(child_stack=0x7f804fefdfb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804fefe9d0, tls=0x7f804fefe700, child_tidptr=0x7f804fefe9d0) = 25069
25063 clone(child_stack=0x7f804d542fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804d5439d0, tls=0x7f804d543700, child_tidptr=0x7f804d5439d0) = 25070
25063 clone(child_stack=0x7f804d501fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804d5029d0, tls=0x7f804d502700, child_tidptr=0x7f804d5029d0) = 25071
25071 exit(0)                           = ?
25071 +++ exited with 0 +++
25063 clone(child_stack=0x7f804d501fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804d5029d0, tls=0x7f804d502700, child_tidptr=0x7f804d5029d0) = 25072
25063 clone(child_stack=NULL, flags=CLONE_CHILD_CLEARTID|CLONE_CHILD_SETTID|SIGCHLD, child_tidptr=0x7f805761fa10) = 25073
25073 execve("/usr/bin/uname", ["uname", "-m"], 0x16e6530 /* 77 vars */) = 0
25073 arch_prctl(0x3001 /* ARCH_??? */, 0x7fffe664fa30) = -1 EINVAL (Invalid argument)
25073 arch_prctl(ARCH_SET_FS, 0x7fd8e1f60580) = 0
25073 exit_group(0)                     = ?
25073 +++ exited with 0 +++
25063 --- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=25073, si_uid=1000, si_status=0, si_utime=0, si_stime=0} ---
25072 waitid(P_ALL, 0, {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=25073, si_uid=1000, si_status=0, si_utime=140189029637312, si_stime=140189192188676}, WNOHANG|WEXITED|WNOWAIT, NULL) = 0
25072 wait4(25073, [{WIFEXITED(s) && WEXITSTATUS(s) == 0}], WNOHANG, NULL) = 25073
25072 waitid(P_ALL, 0, 0x7f804d501bb0, WNOHANG|WEXITED|WNOWAIT, NULL) = -1 ECHILD (No child processes)
25063 clone(child_stack=0x7f804c934fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804c9359d0, tls=0x7f804c935700, child_tidptr=0x7f804c9359d0) = 25075
25063 clone(child_stack=0x7f804c133fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804c1349d0, tls=0x7f804c134700, child_tidptr=0x7f804c1349d0) = 25076
25075 clone(child_stack=0x7f8047ffefb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f8047fff9d0, tls=0x7f8047fff700, child_tidptr=0x7f8047fff9d0) = 25077
25075 clone(child_stack=0x7f80477fdfb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f80477fe9d0, tls=0x7f80477fe700, child_tidptr=0x7f80477fe9d0) = 25078
25077 clone(child_stack=0x7f7fdd185fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f7fdd1869d0, tls=0x7f7fdd186700, child_tidptr=0x7f7fdd1869d0) = 25079
25078 clone(child_stack=0x7f7fdc984fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f7fdc9859d0, tls=0x7f7fdc985700, child_tidptr=0x7f7fdc9859d0) = 25080
25075 clone(child_stack=0x7f7fb7ffcfb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f7fb7ffd9d0, tls=0x7f7fb7ffd700, child_tidptr=0x7f7fb7ffd9d0) = 25081
25063 clone(child_stack=NULL, flags=CLONE_CHILD_CLEARTID|CLONE_CHILD_SETTID|SIGCHLD, child_tidptr=0x7f805761fa10) = 25082
25082 execve("/home/jake/.dotnet/tools/dotnet-ef", ["/home/jake/.dotnet/tools/dotnet-"..., "-v", "migrations", "list"], 0x16e6530 /* 77 vars */) = 0
25082 arch_prctl(0x3001 /* ARCH_??? */, 0x7fff079cf150) = -1 EINVAL (Invalid argument)
25063 clone(child_stack=0x7f804c0f2fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f804c0f39d0, tls=0x7f804c0f3700, child_tidptr=0x7f804c0f39d0) = 25083
25082 arch_prctl(ARCH_SET_FS, 0x7fb17f844740) = 0
25082 clone(child_stack=0x7fb17edf7fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17edf89d0, tls=0x7fb17edf8700, child_tidptr=0x7fb17edf89d0) = 25084
25082 clone(child_stack=0x7fb17e5f6fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17e5f79d0, tls=0x7fb17e5f7700, child_tidptr=0x7fb17e5f79d0) = 25085
25082 clone(child_stack=0x7fb17ddf5fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17ddf69d0, tls=0x7fb17ddf6700, child_tidptr=0x7fb17ddf69d0) = 25086
25082 clone(child_stack=0x7fb17d5f4fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17d5f59d0, tls=0x7fb17d5f5700, child_tidptr=0x7fb17d5f59d0) = 25087
25082 clone(child_stack=0x7fb17cdf3fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17cdf49d0, tls=0x7fb17cdf4700, child_tidptr=0x7fb17cdf49d0) = 25088
25082 clone(child_stack=0x7fb16fffefb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb16ffff9d0, tls=0x7fb16ffff700, child_tidptr=0x7fb16ffff9d0) = 25089
25082 clone(child_stack=0x7fb17c0f8fb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb17c0f99d0, tls=0x7fb17c0f9700, child_tidptr=0x7fb17c0f99d0) = 25090
25082 clone(child_stack=0x7fb1053aafb0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7fb1053ab9d0, tls=0x7fb1053ab700, child_tidptr=0x7fb1053ab9d0) = 25091
25082 exit_group(1 <unfinished ...>
25090 +++ exited with 1 +++
25091 +++ exited with 1 +++
25089 +++ exited with 1 +++
25088 +++ exited with 1 +++
25087 +++ exited with 1 +++
25086 +++ exited with 1 +++
25082 <... exit_group resumed>)         = ?
25085 +++ exited with 1 +++
25084 +++ exited with 1 +++
25082 +++ exited with 1 +++
25063 --- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=25082, si_uid=1000, si_status=1, si_utime=10, si_stime=4} ---
25072 waitid(P_ALL, 0, {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=25082, si_uid=1000, si_status=1, si_utime=0, si_stime=140187203053448}, WNOHANG|WEXITED|WNOWAIT, NULL) = 0
25072 wait4(25082, [{WIFEXITED(s) && WEXITSTATUS(s) == 1}], WNOHANG, NULL) = 25082
25072 waitid(P_ALL, 0, 0x7f804d501bb0, WNOHANG|WEXITED|WNOWAIT, NULL) = -1 ECHILD (No child processes)
25063 exit_group(1)                     = ?
25083 +++ exited with 1 +++
25081 +++ exited with 1 +++
25079 +++ exited with 1 +++
25078 +++ exited with 1 +++
25077 +++ exited with 1 +++
25076 +++ exited with 1 +++
25075 +++ exited with 1 +++
25070 +++ exited with 1 +++
25069 +++ exited with 1 +++
25068 +++ exited with 1 +++
25067 +++ exited with 1 +++
25066 +++ exited with 1 +++
25065 +++ exited with 1 +++
25064 +++ exited with 1 +++
25080 +++ exited with 1 +++
25072 +++ exited with 1 +++
25063 +++ exited with 1 +++

@RheaAyase I've tried this with multiple projects but I'll upload the dummy project I've created for this issue. Here's the dummy repo. I've tried sqlite and postgres with the same results.

@jhoward321 I also hit the same issue. For me it turned out to be a bad symlink i had created for dotnet (/usr/local/bin/dotnet). Doing an unlink on this fixed the issue for me.


Add HMAC to USB communication for devices > 5.0.0

We authenticate messages to the BitBox using an HMAC, beginning with v5.0.0. This needs to be implemented in this demo. Currently, only devices <5.0.0 are supported.

Addressed in https://github.com/digitalbitbox/ElectronDemo/pull/5


Add more smoke test and framework how to add more tests

Add smoke tests for gRPC and different servlet containers (similar to https://github.com/open-telemetry/opentelemetry-java-instrumentation/tree/master/smoke-tests/matrix).
The images for these should be in control of this project, build as part of the smoke-test job. It should be easy to add smoke tests for new frameworks.

We can certainly adopt OTEL smoke tests, my requirements are:

test capturing of payloads
test capturing of headers
the test images should be built and published to HT docker registry - e.g. ghcr or dockerhub

@pavolloffay  here the  2 steps I propose to integrate and modify OTEL smoke test with HT java agent:

Import current test from OTEL to HT repo and make sure all test are passing with hypertrace-agent-all.jar  . This will  require changes in smoke test as well as gradle files. This will include changes for publishing images to HT docker registry.
Enhance test app to send POST requests with required headers & add validation by enhancing test.


"Failed processing block" in Wallet Sweeper

Describe the bug
Using Wallet Sweeper for one particular key, it seems to get stuck in "Processing" with the following log:
Checking if input is a private key
Found 1 pending containing total 0.00443 XNO
Started generating PoW...
Failed processing block: Old block

while the Processing... just seems to keep on spinning forever.
To Reproduce

Go to Advanced Tools -> Wallet Sweeper
Enter a recovery seed, Start Index: 0, End Index: 10
Click START SWEEPING
See error in log

Expected behavior
"Finished processing all accounts" & accounts successfully sweeped/swooped/swiped/sweepened 😟
Web:

OS: macOS
Safari version 17.4.1

does selecting a different server in the app settings help?
feel like this might be an issue with server/node being out of sync, trying to process a block that's already confirmed on the network, I might be wrong though


runtime: improve arm64 memmove

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.9.2 linux/arm64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
What did you do?
Identified via inspection.
What did you expect to see?
The implementation can be improved by unrolling loops and using sets of ldp and stp instructions.
What did you see instead?

The intent of this bug report is to convey the fact that my team and I are developing an improved version of the memmove routine for arm64.  This is based on results from https://blog.cloudflare.com/arm-takes-wing/

The tree opens for performance work in early February. Please have code review(s) w/ benchstat performance numbers in the commits ready by then. The tree will close for performance work in early May.

(per https://golang.org/wiki/Go-Release-Cycle)

Will do.  Thanks, Brad.


cluster list and remove updates to be composable

list clusters ids dcos cluster list --quiet or dcos cluster list -q
remove a list of ids
composable dcos cluster remove $(dcos cluster list --quiet)
better documentation

The CI integration tests are NOT setup like the developer test instructions... there is no DCOS_DIR setup so these tests fail.   Analyzing the jenkins file and testing... it does not appear that the cluster directory structure (under the DCOS_URL) is ever tested in the CI env tests.    It appears that the dcos.snakeoil.mesosphere.com is associated with the DCOS_URL in the /etc/hosts file and that all tests are configured to use the tests/data/dcos.toml

@klueska all good here with integration tests

https://github.com/kensipe/dcos-cli/blob/2a6419c78fcd1314d9feae2bb1e7d96f598da53a/cli/dcoscli/data/help/cluster.txt#L26
Could we make it clear here that the remove command now accepts multiple clusters? eg. Remove configured cluster(s) from the CLI.

rebased and updated the help text as requested.

rebased... I will address nits later today

rebased

rebased and addressed @elezar feedback

run linux integration tests

@bamarni ready to roll!

@kensipe : as we already have #1015 in the pipeline, I'd rather leave this one on hold. It's less user-friendly and we don't have any other use-case for composition including the cluster list right now.
This feature can be great when one is also able to filter using criterias (similarly to docker ps used with docker rm or docker stop). So we might revisit it in the future.

@bamarni @klueska I think this is a mistake... I know there were 2 ways presented to potentially remove all clusters... however this way is the "best" way IMO and adds more... it adds the ability to make things composable.   I would want this even with #1015 added to the code base.

@kensipe : could you please elaborate on why it is better? I'm not closing the door forever here, just saying that I don't see any use-case for this at the moment (as we are going to add the --all flag to dcos cluster remove).


Discrepancies on cancancan access

When setting the app/models/motor/ability.rb to:
module Motor
  class Ability
    include CanCan::Ability

    def initialize(user, _request)
      can :read, :all
    end
  end
end

The user can see everything, even though I have hide some fields on the admin config

But if I user:
module Motor
  class Ability
    include CanCan::Ability

    def initialize(user, _request)
      can :manage, :all
    end
  end
end

It will respect the admin config and only show the fields I set to show.

Closing for now, it looks like after 1 week of restarting the server made it work.


allow select from a union select

give the capability to parse query from a union select

thanks for this - got me unstuck


Style unselected items

I have to style Unselected Items which on design looks as follows.
i.e Selected items are white color text where as unselected are slight gray.

But looks like there is property to style selected items only !
        settings.style.buttonBarBackgroundColor = .black
        settings.style.buttonBarItemTitleColor = .white
        settings.style.buttonBarItemBackgroundColor = .clear
        settings.style.selectedBarBackgroundColor = UIColor(red:0, green:0.66, blue:0.88, alpha:1)
        settings.style.selectedBarHeight = 2.0
        settings.style.buttonBarItemLeftRightMargin = 10.0
        settings.style.buttonBarItemsShouldFillAvailiableWidth = true

Thank you @bishalg for this.


Data cleaning

Added data distribution checks for imputation as well as US map for ratio of zip codes kept after merging.

Removing as these changes are being integrated in the consolidation branch


Handle rename of "MLIRGPUOps" to "MLIRGPUDialect".

LLVM renamed the library "MLIRGPUOps" to "MLIRGPUDialect".  Let the build find whichever one exists.
Please review carefully -- I don't actually know how to CMake.

..or perhaps you don't want to take this PR, and you'll just change everything when you upgrade to LLVM 18?

..or perhaps you don't want to take this PR, and you'll just change everything when you upgrade to LLVM 18?

That was the plan as far as I know. Trying to support multiple LLVM versions at the same time would be hard.

Yeah, I got this to work, but then the rest of the build...yeesh.  :)

This is probably something that should go into the llvm-head branch if it's
not already there, and from there gets merged back to main once we upgrade.
On Sat, Sep 16, 2023, 22:29 Justin Lebar @.***> wrote:

Yeah, I got this to work, but then the rest of the build...yeesh. :)
—
Reply to this email directly, view it on GitHub
https://github.com/openai/triton/pull/2315#issuecomment-1722309742, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJUBD7H73XUVSFFYW2SSITX2YD27ANCNFSM6AAAAAA43C5DQA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Navbar fixes

This PR builds on @vanduynslagerp's work in #21719 as a basis for many of our navbar related issues. :sweat_smile: Basically there's a bug in a Sass mixin, some unnecessary position: absolute, a missing .container-fluid, and some inconsistent sizing of navbar sub-components. That's all getting fixed here.

Correct navbar collapsing breakpoints resulting from the broken Sass mixin that was generating our .navbar-toggleable classes. Basically this wasn't generating mobile-first classes, so we were acting on the wrong media query values. Fixes #21691, fixes #21706, and closes #21719.

Redo the .navbar-toggler alignment by dropping the modifiers entirely. Instead of using absolute position, we run with the DOM order and make use of justify-content: space-between, forcing navbar brand and toggler to be on opposite ends of the navbar. This drops .navbar-toggler-left and .navbar-toggler-right Fixes #21659, fixes #21735, and closes #21729.

Resolve issues with .container-fluid not being fully supported within .navbar. Fixes #21530 and closes #21901.

Normalizes the sizing of navbar brand, text, and link elements by pulling in the changes from #21721 (which fixes #21705). This one might still need further love, or our inputs will; the text input and buttons are 38px vs the nav's 40px. Any changes to that will come in another PR.

Still to do:

[x] Update navbar doc examples to drop toggler modifiers and reorder brand/toggler
[x] Drop navbar toggler variants in docs

Awesome ! The justify-content: space-between for navbar-toggler is really clever !

Unsure where these test failures came from for our button JS... presently looking into that before tackling the navbar docs update.

I think this is due to a JQuery bug in 3.2.0 (updated in this PR #22218)
The bug is fixed in JQuery 3.2.1:  jquery/jquery#3579
Upgrading to JQuery 3.2.1 should fix the test.

Oof, fixed that but still have the Qunit failures with Sauce. The build from merging #21788 (https://travis-ci.org/twbs/bootstrap/jobs/212589569) is the earliest I see of the failure.

@mdo, it should be fixed here #22234.  So let me know if there is other Saucelabs errors and I'll fix them ASAP.

The last thing, outside these failures, is possibly renaming this class. The class name .navbar-toggleable- mostly comes from legacy Bootstrap versions where we weren't exactly thinking mobile-first on everything we did. Often times the navbar of a site or app is designed for desktop first still, so folks think "okay when does this need to collapse." On the flip side, if we start with mobile, it's when does this expand.
I could go either way, but I'm kinda inclined to leave it as-is for few changes from our last alpha to this first beta. That said, once we go beta, we're locked in. Thoughts?

@mdo, I think .navbar > .container-fluid need to be always width: 100%.
Please take a look here: http://codepen.io/zalog/pen/RpMoqY

@zalog Hmm, yeah, I see that now, too. Added a fluid container example to my growing js bin examples at https://jsbin.com/zixoxah/edit?output. I feel like that width: 100% should probably just go on .container-fluid and not be limited to the .navbar. Thoughts?

Yep, I was thinking on that too, I can help with a PR.
Love this changes, great work!

I could go either way, but I'm kinda inclined to leave it as-is for few changes from our last alpha to this first beta. That said, once we go beta, we're locked in. Thoughts?

I would personally favor a rename for consistency reasons. Thinking in a "responsive" way is really not that easy, especially for "beginners", so having every classes working the same way (i.e. "When viewport is higher than breakpoint apply this") make things a lot simpler. Plus I saw several questions recently around that recently.
Looking at the code it seems that currently the .navbar-toggleable- is the only class in that situation.

Working on a new class name, .navbar-expand, in https://jsbin.com/rikevuj/1/edit?output. In this setup, the classes work like so:

No modifier, just .navbar, is never expanded
.navbar-expand is always expanded, from 0 to infinity (and beyond!)
.navbar-expand-sm expands at sm and up
.navbar-expand-md expands at md and up
.navbar-expand-lg expands at lg and up
.navbar-expand-xl expands at xl and up

Put another way, the navbar is always collapsed until you tell it otherwise. The first several navbars in that example link above show navbars in that order. I think this should work for folks who have been scratching their heads at this component being in reverse of our general mobile-first approach everywhere else.
Would love feedback before I push up relevant changes for that.

Perfect !

@mdo on your jsbin the very first navber the dropdown menu causes navbar height to expand with it. https://jsbin.com/rikevuj/1/edit?output


Add comparison operator overloads for extended_symbol type

comparison operators (==, !=, <, >) are missing for the extended_symbol type in eos/libraries/chain/include/eosio/chain/symbol.hpp

Handled by #8540


No GPU/MPS device found. LLM training requires an accelerator

Hello, I am using a Windows 11 with NVIDIA GeForce RTX 3080 Laptop GPU, but still, no GPU can be found.
I receive this error message when running autotrain llm:
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\HTW\anaconda3\Scripts\autotrain.exe_main.py", line 7, in
File "C:\Users\HTW\anaconda3\Lib\site-packages\autotrain\cli\autotrain.py", line 47, in main
command = args.func(args)
^^^^^^^^^^^^^^^
File "C:\Users\HTW\anaconda3\Lib\site-packages\autotrain\cli\run_llm.py", line 14, in run_llm_command_factory
return RunAutoTrainLLMCommand(args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\HTW\anaconda3\Lib\site-packages\autotrain\cli\run_llm.py", line 473, in init
raise ValueError("No GPU/MPS device found. LLM training requires an accelerator")
ValueError: No GPU/MPS device found. LLM training requires an accelerator
How can i fix this problem?
Thanks

The error is straightforward. You need a GPU or MPS device to train LLM models.

Thank you for your answer, but I do have a GPU: NVIDIA GeForce RTX 3080 Laptop GPU.
Does the fine-tuning with autotrain not work with my GPU?

same
raise ValueError("No GPU/MPS device found. LLM training requires an accelerator") ValueError: No GPU/MPS device found. LLM training requires an accelerator

should be fixed now


Nesting separate vertical ListViews in a horizontal PageView in a vertical ListView

The current theory is that the outer ListView will handle all vertical scrolling. When it tries to set pixels on its ScrollPosition, the ScrollPosition will forward the delta to the "current" descendant ListView.

@HansMuller is looking at this currently.

There's two relevant use cases. One is a fully-delegated model, where you have two (or more) scrollables that are in lockstep. For this one, we want some activity-less ScrollPositions that defer everything up to a ScrollableState-less ScrollPosition, which pushes notifications back down so they come out of each scrollable.
The second use case is this bug. Tabs below an AppBar. I think for this case we want two fully operational ScrollPositions. They track their pixels and additionally a second value which is the last pixels value that they were set to. Each of these ScrollPositions has a "before" controller and an "after" controller. When the setPixels method is called to set the pixels to a value that is less than the minimum or greater than the maximum, if there's a controller in that direction, then the difference between the limit and the attempted value is sent to the controller in that direction, and we just clamp (but do not overflow). The second pixels value is used to determine this delta when we are already at the limit.

I tried the model described in the last paragraph and while I think it's a promising approach for parallel scroll views, it doesn't work for the tabs-below-app-bar use case.
After further thought, I think it's actually better to implement that as another delegating model, where you have activity-less ScrollPositions that defer incoming messages up to a ScrollPosition-like object that manages the Activities. Somehow they would send up their dimensions too, so that pixels could make sense at the top level, But they would have their own independent pixels positions. The leader ScrollPosition would only have two slots, "above" and "below". The "below" scrollable would be favoured for downwards movement (negative scroll deltas) and the "above" for upwards (positive) movement. Only once the "below" scrollable is its top would negative deltas go to the "above" scrollable, and vice versa.

That approach didn't work either because there's no good way to determine what it means to have a position for the activity manager in that world.
I have, however, learnt about a bunch of edge cases that are very interesting.

Suppose the outer list is scrolled to the top, and the inner list is scrolled to its bottom. Now, do an iOS jump to top.
Suppose the outer list is scrolled to the top, and the inner list is scrolled to its bottom. Now do a jumpTo() to a position that is about half way through the outer list's range of values. The outer list has to scroll down and the inner list has to scroll up at a different velocity.
If we allow multiple inner lists at once (e.g. because you are mid-way through paging a tab view, and scrolling the appbar is to scroll all views at once) then flinging when they have different lengths needs to do an iOS bounce at a different time on the two different inner lists.

This all argues for maybe having the inner and outer ScrollPositions both have their own ScrollActivity, still managed by the single ScrollActivityManager. For Driven and Ballistic activities, we can just drive them with their own dedicated activities (though this means activities need to be able to setPixels on something that isn't their manager). For Drag activities, we could have some logic to make dragging in one direction drag one and the other direction drag the other, unless they hit the limit, then go and drag the other one...

PR in https://github.com/flutter/flutter/pull/9070

Progress is being made. :)  ~half has landed.


Support for doc comments on same line.

This issue was originally filed by<EMAIL_ADDRESS>
Hi, Would be nice to be able to document code like this:
class MyClass {
    int     id;           /// Doc comment for id
    String  name;         /// Doc comment for name
    String  description;  /// Doc comment for description
    String  otherValue;   /// Doc comment for otherValue
}
Instead of:
class MyClass {
    /// Doc comment for id
    int     id;

    /// Doc comment for name
    String  name;

    /// Doc comment for description
    String  description;

    /// Doc comment for otherValue
    String  otherValue;
}

I don't think the spec should be in the business of defining doc comments.
How about we remove that from the spec and then kick this over to dartdoc?

This isn't something we've had continued demand for. Doc comments which are short enough to fit on the same are unlikely to add new information over the type and name so I'd think most of them can be omitted anyway.
Is there value in keeping this open?

Closing. If there is renewed interest in this proposal please add a comment and we can reopen.


⚠ Bofrost.at has degraded performance

In 12c4409, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1966 ms

Resolved: Bofrost.at performance has improved in 8c0bbaa after 19 minutes.


Gradle

Hi, first i want to thank you for this amazing work, i think many java developers would like this string interpolation feature available out of the box.
i have a java modular project with gradle, when i added to build.gradle => compile group: 'com.antkorwin', name: 'better-strings', version: '0.4' it did not work.
then i made a blank project with gradle but did not work either. afterwards i made a maven project and then it worked seamlessly.
dois it have to do with gradle or am i missing somenthing? thanks in advance for your time.

Hi, @raulest50
Thanks for the kind words!
I will check this build with gradle.
The previous version(0.3) doesn't work for you too?

i tried both .3 and .4 from https://mvnrepository.com/artifact/com.antkorwin/better-strings with modular and non modular gradle project.

I have made a simple gradle project:
https://github.com/antkorwin/better-strings-demo
when I run gradle clean test from command line - all works fine,
but if I use Intelij IDEA then I have a problem with annotation processors disabling.
I found a few topics about this problem at stack overflow:
https://stackoverflow.com/questions/42441844/annotation-processor-in-intellij-and-gradle
turn on enable annotation processing and select the gradle test runner in the IDEA settings works fine for me.
Do you use IntelijIDEA? or something else?

thanks so much, it works now,  im using netbeans, and created again a blank non modular gradle java project, then in the build.gradle i add in dependencies :
compile group: 'com.antkorwin', name: 'better-strings', version: '0.4'
annotationProcessor 'com.antkorwin:better-strings:0.4'
just adding annotationProcessor as you did in your example make it work fine.
perhaps this could be added to the readme in the "3. Getting started".
by the way, im posting the main.java and the gradle in case it may be of help for others. thanks again for the fast replay @antkorwin
Main.java:
package bstrings_test;
public class Main {
public static void main(String[] args) {

    var a = 3;
    var b = 4;
    System.out.println("${a} + ${b} = ${a+b}");

}

}
build.gradle:
apply plugin: 'java'
apply plugin: 'jacoco'
apply plugin: 'application'
mainClassName = 'bstrings_test.Main'
repositories {
jcenter()
}
dependencies {
testImplementation     'junit:junit:4.13'
// https://mvnrepository.com/artifact/com.antkorwin/better-strings
compile group: 'com.antkorwin', name: 'better-strings', version: '0.4'
annotationProcessor 'com.antkorwin:better-strings:0.4'

}

thanks for your feedback, I have updated documentation about the gradle build configuration and IntellijIdea settings.


WSL update not working on 26020-26040

Windows Version
26040.1000
WSL Version
2.0.0
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu-22.04
Other Software
No response
Repro Steps
running WSL --update or WSL --update --pre-release
Expected Behavior
update is succesfull, new wsl version is disaplyed when running wsl --version
Actual Behavior
seems goes through the updates steps, no errors
wsl --update
Checking for updates.
Updating Windows Subsystem for Linux to version: 2.0.14.
wsl --update --pre-release
Checking for updates.
Updating Windows Subsystem for Linux to version: 2.1.1.
wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.57
MSRDC version: 1.2.4485
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25880.1000-230602-1350.main
Windows version: 10.0.26040.1000
Diagnostic Logs
repro recording with feedback hub: https://aka.ms/AAotrdo

same issue
❯ sudo wsl --update --pre-release
Checking for updates.
Updating Windows Subsystem for Linux to version: 2.1.1.

❯ wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.57
MSRDC version: 1.2.4485
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25880.1000-230602-1350.main
Windows version: 10.0.22631.3085

❯ wsl --update --pre-release
Checking for updates.
Updating Windows Subsystem for Linux to version: 2.1.1.
Update failed (exit code: 1603).
Log file: C:\Users\windo\AppData\Local\Temp\wsl-install-logs.txt
Error code: Wsl/UpdatePackage/E_ABORT

This is unfortunately a known issue when upgrading from 2.0.0. The easiest way to fix this is to manually download and install the latest MSI package

That does indeed solve the issue Thanks!