Fix illegal instruction on RPi 4

The absl library tries to make use of the crypto libraries on arm64 platforms. However, it turns out that the RPi 4 chip does not have the crypto extensions. Hence, we need to patch absl to refrain from using the hardware crypto stuff and fallback to slower software implementations. This is only required for a random back-off timer at the moment, so shouldn't matter too much.
Note that the RPi 5 has the crypto extensions but our build is the same for both.
Fixes https://github.com/mavlink/MAVSDK-Python/issues/654.
Fixes https://github.com/mavlink/MAVSDK-Python/issues/651.

Isn't it the same as https://github.com/abseil/abseil-cpp/issues/662? Weird that they seem to have fixed it :thinking:.
In any case, if there really is a need for a patch, I would do it only for the relevant platform. But I wonder if that could not be solved somehow with ABSL_RANDOM_HWAES_ARM64_FLAGS :thinking:

waiting for this fix 🤞

@JonasVautherin it looks related. I'm not sure. What I found is that hardware crypto is enabled for aarch64, and that might make sense for most platforms, but the Broadcom BCM2711 chip is lacking the crypto extensions.

Of course updating absl was a mistake and it breaks the build.

Seems to work, thanks a lot!

Thank you, we faced the exact same issue in AxoSyslog running in an alpine based container on an RPi4. This PR was a good confirmation for us. :)
P.S.: OpenSSL does runtime checks in init time to determine which CPU features are available on the runner host for certain operations, it would be nice to have something similar in the upstream libabseil, too.


switch to the latest nightly commpiler

on aarch64, hermit-sys uses the target aarch64-unknown-none-hermitkernel to build libhermit

bors r+


Lasse gave up - time to change...

Lasse, the orignal developer of the TBCEditor, gave up this project. Thanks a lot to him for all he done!
Now it's time to change.
If you are interested to support this project, you are invited to do this. Please feel free to offer you assistance here...

Lasse deleted this project. Nothing foretold such sad decision.  May be Lasse will renew the BoneCode?

Some days before, I received this note from Lasse:

There is about 40 days time to finish this. After that I am going to travel around the world (3rd time) and after that I will finish my 3D engine which may still take a year or two...

Yesterday, I wrote a E-Mail to him ... the mail address I contacted him some days before is not longer valid.
Further informations I don't have.
But whatever the reasons are, why he don't want to support this project further more ... he deleted this project and his repository completely. Maybe this is the answer, if he is plannig to go ahead somewhen later on ... or not.

EditBone and TBCEditor are great, best and very usefull projects.
This projects have to proceed, but additional information is necessary.
It isn't necessary to hurry. Perhaps, Lasse will change the mind.
Lasse phone number: +358 45 120 9795
https://ytunnus.fi/yritykset/8311671/bonecode/amp/

Please remove that phone number. Should now everybody call him and beg to come back?!

I think there is someone adready bought this project (may be TXS)! Let wait and see.

I asked GitHub, if they restore the repository - but they wont.
I don't have the latest update of the sources - so I can't fill the repository. Can somebody provide the latest source?

@bonecode: hope this help
https://bitbucket.org/egrange/tbceditorxe

@thachngopl:

only Delphi XE is the only version supported

I don't think it's a good idea to use a backport. I think we should look forward...

@thachngopl:

only Delphi XE is the only version supported

Hm, why do you mark this version as "XE only"?
I think it would nice to have a version, which works for XE and higher together.

Wow, this is a pretty low move to delete it like this without any warnings and any care that other people might have been affected. What a lousy move and pretty upsetting. Having said that, let's move forward and forget about Lasse.
I'd like to help where I can. The last sync I did was 5 days ago on Feb 5th, 2017. Looks like the code on bitbucket is different from what I have. I either have older or newer code. I would need to do a diff.
I'm willing to contribute where I can since I was planning on replacing my existing customized SynEdit editor with TBCEditor. My open source project is at https://github.com/ThomasJaeger/VisualMASM
I'm not even sure if we should keep the name since there are many things that could be simplified with TBCEditor.  This should be a lot simpler.
Anyways, what do you guys think?

Changing the name means changing the URL of this repository. Changing the URL means search engines will push back this URL in the results, since lesser links inside the net are locating to the URL (some links are linking the old URL, some linking the new URL).

@NilsHoyer Honestly, I would not worry about the name or URL since it was very short-lived since it was killed off by the author. If we are in for the long haul, it would be our contributions that would established a name people can start to count on (and not deleted for sure).

@ThomasJaeger The most important thing for every project is: How to get public? This name and this URL are more public than a new name / new URL. What would be the advantage of a new name / new URL to accept to be found lesser?

I'm sorry, but I think we have to find a solution to provide this project ... without GitHub.
Why? GitHub is very comfortable.

I have all BoneCode source code backups at 05.02.2017 23:59,
but we need last backups of EditBone and TBCEditor at 10.02.2017.
IMHO, Lasse is great developer and everything is explained by it.
Now, I suggest to recreate the project as the original.
Step by step, later...

I'm sorry, but I think we have to find a solution to provide this project ... without GitHub.
Why? GitHub is very comfortable.

I meant: GitHub will not provide the sources to me. I like GitHub and I think this project is located here well.

@NilsHoyer The thing is, I'm doing my VisualMASM open source project simply because I love the idea of it and to make a mark in the world for others to enjoy. I'm a supper, super busy guy but I do try to find time to work on something I do not get paid anything for. I love the see this vision come true and see the completed Visual Assembler IDE. I think when you put the passion into the project, you will eventually find others who also like it, use it, provide feedback, contribute, etc. I had to modify SynEdit in my case to make it work the way I wanted it. But, when I found TBCEditor just a couple of weeks ago, I thought it had some nice features that did not exist in SynEdit, yet. That's when I was playing around with it and had a plan to replace my modified SynEdit pieces with TBCEditor. Now, I'm not sure what to do since Lasse killed it off like this in a rotten way.
The reason I'm saying all this is because being associated to TBCEditor is a bad thing now after what Lasse did. That is a low character move and you can't trust somebody like that. Why would you want to continue with TBCEditor as a name?
Whatever the reasons were for Lasse to delete it, I don't get it. There would have been no harm to just make somebody else the owner. He didn't have to delete it. However, he did put in a lot of work and time and for that we are all thankful.
You can always provide a referral link to a new project on Github with a new name that could use TBCEditor as a base and provide thanks for Lasse as the original author and ideas but a new name would signal a fresh start with new leadership.
I'm not sure where I will go from here, I might just fork the entire TBCEditor and supporting components and create a new editor with my additions and architecture changes. Or, I just might continue with my modifications of SynEdit. I don't know, yet. My VisualMASM needs a good editor before I can finish the visual drag & drop features. I would be willing to put the work into a new editor but I just don't want to be associated with Lasse after what he did. It doesn't matter if he was the greatest developer in the world, you just don't do what he did.

TBCEditor-master (05.02.2017 10h59m).zip
Components-master (22.01.2017 06h05m).zip
Common-master (02.02.2017 19h31m).zip
EditBone-master (05.02.2017 12h58m).zip
OraBone-master (16.05.2015 08h47m).zip

I have master backup from this Thursday (09,02,2017) here http://ntscorp.ru/tmp/TBCEditor-master.zip

http://bonecode.com

Note! Service provider starts a transition to a new server 16.2.2017. Domain remains the same for a while, EditBone is going to have an own domain, code signature, and installer.

@NilsHoyer
In original source code: 0D0A at line end
In current - 0A
It will complicate merge.

@delphioracle
Can we handle problems in separated issues?
... I will take a look to this issue from here


[FLINK-13154][docs] Fix broken links of web docs

What is the purpose of the change
Fix Flink's broken link before release-1.9
Brief change log
Fix Flink's broken link before release-1.9
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot attention @bowenli86 for I have removed one line in sqlClient.md.


[FEATURE REQ] Allow not spawning new threads on each ServiceBusProcessorClient instance start.

Is your feature request related to a problem? Please describe.
When ServiceBusProcessorClient instance is started (start() method) it creates a new thread (Executors.newSingleThreadScheduledExecutor()). While having significant number of queues (assume hundreds) it means significant number of threads spawned. In some scenarios it's sufficient to use a shared threadpool rather than having a thread per client.
Describe the solution you'd like
Extend ServiceBusProcessorClientBuilder so it allows to set executor (threadpool) that can be reused by different instances of ServiceBusProcessorClient throughout the application.
Describe alternatives you've considered
Use/allow using static threadpool for ServiceBusProcessorClient.
Additional context
Use case: significant number of queues that are not heavily used, thus they don't need a dedicated thread.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Description Added
[X] Expected solution specified

Will do Josh!

@LukaszGebel-TomTom Can you please help me understand your scenario some more? Are you looking to share a threadpool among many (hundreds) Processor clients and use the shared threadpool to actually do the processing of messages?
Or do you just want to replace the call to create a new executor inside ProcessorClient.start() with a shared pool?
If it is the instance that you have mentioned,

When ServiceBusProcessorClient instance is started (start() method) it creates a new thread (Executors.newSingleThreadScheduledExecutor()).

Here the thread is used to monitor the state of the connection every 10 seconds and restart the processor if needed, and is not actually used to process any of the messages. We are looking in replacing this executor with reactor call.

Hi @ki1729, replacing this executor with reactor call is ok as well. Thank you.

Thank you for the update @LukaszGebel-TomTom , will update the issue when we have the change ready to merge

Thank you @ki1729.


[Bug] Unintentional Horizontal Scroll occurs in Diff Editor on clicking near the right end of the modified editor instance

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:1.63.2
OS Version: Mac OS - Big Sur 11.6

Steps to Reproduce:

Select any file in the Diff Editor Mode from the source control tab
Click on the right end of the modified editor instance
The content is getting selected automatically and editor is scrolling unintentionally

Video explaining the issue
This video is recorded in VS Code Desktop Application with all the extensions disabled as mentioned in the comment
https://user-images.githubusercontent.com/29809906/148763270-2b0657fd-c61a-4ebd-aadb-49a336360315.mov
Additional Info
This issue occurs only when there is a click happening very close to the right end of the editor, But I'm not sure about the exact distance from the right side
This issue is reproducible in monaco editor playground also

This reproduces in any editor (not just in a diff editor) and it also reproduces when clicking near the bottom of the viewport.

@alexdima Thanks for fixing the issue


acorn_loose generates bad AST for incomplete TemplateElement

Consider the following snippet:
`

When the entire contents of the file are only a back-tick, the loose parser recovers (good), but it creates a node in the quasis list that has no type. Looks like you just need a finish call in lp.parseTemplate, like:
this.finishNode(curElt, "TemplateElement");

on line 328 in expression.js

@mrennie Thanks for the report! Please submit a PR as it's a better medium when you already know what to fix :)


Change Au test openssl lib version condition to use version traffic_server was built with.

@dragon512 @shinrich @zwoop please review.

@zwoop A concern I have about this change is whether objdump (a GNU binutil I think) is present and works the same on FreeBSD and OSX.  (Au test works on OSX doesn't it?)

@randall I changed it to use the nm utility instead of objdump, can you try it again please?

I forgot to comment, I tried this on my mac, and it didn't show anything? Is that expected?
munin (11:58) 284/0 $ nm /opt/ats/bin/traffic_server | grep -F 'OPENSSL_' | tr ' \t@' '\n' | grep '^OPENSSL_[0-9]' | sort -u | tail -1
munin (11:58) 284/0 $

The more I think about this, I think a much better approach would be to augment traffic_layout. Something like
$ traffic_layout info --versions
source: /home/leif/apache/trafficserver
build: 8.0.4-78
openssl: 1.0.1a
brotli: 0.1.2
...

I've started hacking something up:
munin (17:44) 426/0 $ ./src/traffic_layout/.libs/traffic_layout info --versions
openssl: 0x1010102f

See proposal in https://github.com/apache/trafficserver/pull/5239

I have to agree with @zwoop - traffic_layout should provide this information. That will be more reliable across multiple platforms.

This is replaced by https://github.com/apache/trafficserver/pull/5239 .


remove "no longer maintained" note

Fixes #803, relevant to #792

👍  Also not in code, but "Twitter for Python! ** No longer maintained **" at the top of the homepage needs updating.

@Aaron1011 Please can you also update "Twitter for Python! ** No longer maintained **" at the top of the homepage? (And maybe add a link, e.g. http://docs.tweepy.org.)Thanks!

@hugovk: Unfortunately, I don't have permissions to do that at the moment - @joshthecoder needs to give me admin access.


New ui

if you are working on new UI perhaps you can take a look at https://github.com/neomjs/neo as the framework.

Looks nice but we're going in another direction.


Fix comments as per Josh's feedback

Some feedback from @gotjosh that was missed before https://github.com/prometheus/alertmanager/pull/3278 was closed.

Not a problem, thanks for the quick approve! ❤

Test failed for nflogs, forced pushed to restart tests.


kill applications using synced folders before unsyncing

There's a problem involving sync'd folder integrity if applications are using the sync'd files when the folder is unsync'd. This was solved in psd by killing any running applications (browsers) that are using this space.
I think one solution for ASD might be to have an array of "applications_syncd" in the config file. ASD could kill these apps when ASD is stopped (unsync) the same way that PSD does with browsers. Perhaps even like PSD, ASD could refuse to start if any of those applications are running when ASD is started (sync).
I've been trying to think of a good solution that doesn't involve killing processes, but I'm having difficulty.
Explanation of problem:

ASD starts, moves folder to /dev/shm/ and replaces folder with a symlink to /dev/shm/folder
Application opens, following symlink and opening file handles to /dev/shm/folder/files{1,2,3...}
ASD stops, deleting the symlink and moving the folder from /dev/shm back to disk. Application still holds some file handles to the now non-existant /dev/shm/folder/files{1,2,3...}
Application continues to run, opening new files, now on the disk at /real/path/to/folder/files{4,5,6...}
When application closes it flushes and closes all of its file handles, some to /dev/shm and some to disk. The files on disk are now in an inconsistent state and might not be what the program expects the next time it starts.

Ideally ASD starts at boot and ends at shutdown, meaning it will start before the application(s) and end after the application(s) ends. But if ASD is ever updated, ASD will stop so its package can be removed, then start again after the new package is installed, making the issue extremely likely during package upgrades to ASD.
At the very least the documentation should warn users to stop applications and services that are using ASD before updating its package or otherwise stopping ASD.

Uhh? asd be better aware of open files, as regular startups and shutdowns (e.g. manual, during updates, uninstall) can cause data loss/corruption just as well.

Yes, I have this noted in the man page:
BUGS
       Discover a bug? Please open an issue on the project page linked below.

              o  Currently, asd cannot handle open files on a sync target so if a hung process has something open there,  it
                 can be messy.

Sorry I didn't notice the man page update.
Well, a fix should only require a call to lsof $PATH before proceding or aborting.
A simple check would elevate asd's reliability so much for system files etc.


apm still not working.

I have tried several times to download Atom from here as well from the website and I can't get apm to work, I go to the atom folder and I never have a bin folder.

Found the issue, I had a typo in the path and the bin folder was in my D cd for some reason..


CORE-20185 Token Claim idempotency API changes

Adjust the TokenClaim API to force users to provide a deduplication ID which by the external event framework. This Id ensures non deterministic flows will reuse a deterministic ID provided by the client. This Id is used by the Token Selection worker to dedupe Token Claims.
API change:
https://github.com/corda/corda-api/pull/1572
Branched from the Persist API changes:
https://github.com/corda/corda-runtime-os/pull/5920

@LWogan, is there a follow-up PR where the deduplication ID is used? It seems that this PR passes the ID around but it is not used.

build e2e

build e2e


Getting "Something went wrong" error pop when add a custom bot to a group chat

When trying to add a custom bot to a group chat, if this chat hasn‘t install any other app before. Getting “Something went wrong” error pop, and the browser console showed "failed to get installedApp"。
If this chat has installed one other app from app store or installed by uploaded custom. Then when I upload my custom bot and add to this chat, there is no error
I tested "bot-sso" example created by TeamsToolkit, Getting the same error also.

@woojayjon - Thanks for reporting your issue.
We will check this at our end and will get back to you.

@woojayjon - We are able to repro that issue, we have raised bug for the same and concerned team is looking into it. We will inform you once we get any update.

@woojayjon - The issue is fixed now. Microsoft Teams Version <IP_ADDRESS>80 (64-bit)
We are able to add the custom bot to a group chat. We have tested "bot-sso" example created by Teams Toolkit. Please verify.


PHP notice Undefined index: rel in Flysystem/Adapter/FedoraAdapter.php

In system based on isle-dc I see numerous PHP notices from Islandora Notice: Undefined index: rel in Drupal\islandora\Flysystem\Adapter\FedoraAdapter->getMetadataFromHeaders() (line 155 of modules/contrib/islandora/src/Flysystem/Adapter/FedoraAdapter.php).
https://github.com/Islandora/islandora/blob/8.x-1.x/src/Flysystem/Adapter/FedoraAdapter.php#L155 assumes rel attribute to exist.

Seeing this over at Born-Digital now that we're on the Fcrepo 6.0.0.  Should be a very simple fix.


Timesheet - 28/04/2021- Manish Ram

Task 1: Timesheets

Reviewed timesheets
Reviewed Docs and left comment

Proof of Work

Commented "Approved" for approved timesheets or left other comments to ask for additional Infos to the timesheet issues dated 27/04/2021
https://github.com/thenewboston-developers/Contributor-Payments/issues

Time Spent
1 hour

Task 2: Audit Discuss

Audit related minor discussion

Proof of Work

Can be verified by Audit Team Lead

Time Spent
0.5 hours

Task 3 : Payments
Audit contributor payments for 21/04/2021 [After Payment]

Checked sent payment amounts were equal to worked hours x hourly rate.
Checked payments made to the correct account.
Checked payments were not missed.
Labeled Paid timesheets and closed them.
https://docs.google.com/spreadsheets/d/1Eh8cwsmBxcBBcnoL85Vids3y0NsdtGI576Bs1xeH99I/edit#gid=0

Time Spent
0.5 hours

Total Time Spent
2 hours
Account Number
0f92e975101eb96cea80dce430d74efd3172fa229377818d467ded85dcf2ab6c

Approved


Dedup activity heartbeat timeout timer task creation (#3032)

Coverage increased (+0.05%) to 67.844% when pulling 481f4deeef5a47d660f31a5ca8603152b4a0e2eb on yycptt:0.11.x-hotfix into d5cd7b0b165c604cf23746df8826d8d08ae62dc0 on uber:0.11.x.


setting AWS_PROFILE in a build step causes s3-secrets failure

If I set the AWS_PROFILE environment variable on a build step the s3-secrets plugin's aws-cli commands fail if that profile hasn't already been setup in ~/.aws/credentials.
The failure is occurring here.
https://github.com/buildkite-plugins/s3-secrets-buildkite-plugin/blob/master/lib/shared.bash#L8
I was easily able to reproduce this by logging into a buildkite instance as the buildkite-agent user and running the following command.
$ AWS_PROFILE="some-profile" aws s3api head-object --region=us-east-1 --bucket secrets --key private_ssh_key
Traceback (most recent call last):
  File "/usr/bin/aws", line 27, in <module>
    sys.exit(main())
  File "/usr/bin/aws", line 23, in main
    return awscli.clidriver.main()
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 58, in main
    rc = driver.main()
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 193, in main
    command_table = self._get_command_table()
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 102, in _get_command_table
    self._command_table = self._build_command_table()
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 122, in _build_command_table
    command_object=self)
  File "/usr/lib/python2.7/dist-packages/botocore/session.py", line 719, in emit
    return self._events.emit(event_name, **kwargs)
  File "/usr/lib/python2.7/dist-packages/botocore/hooks.py", line 227, in emit
    return self._emit(event_name, kwargs)
  File "/usr/lib/python2.7/dist-packages/botocore/hooks.py", line 210, in _emit
    response = handler(**kwargs)
  File "/usr/lib/python2.7/dist-packages/awscli/customizations/preview.py", line 69, in mark_as_preview
    service_name=original_command.service_model.service_name,
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 317, in service_model
    return self._get_service_model()
  File "/usr/lib/python2.7/dist-packages/awscli/clidriver.py", line 334, in _get_service_model
    api_version = self.session.get_config_variable('api_versions').get(
  File "/usr/lib/python2.7/dist-packages/botocore/session.py", line 279, in get_config_variable
    elif self._found_in_config_file(methods, var_config):
  File "/usr/lib/python2.7/dist-packages/botocore/session.py", line 306, in _found_in_config_file
    return var_config[0] in self.get_scoped_config()
  File "/usr/lib/python2.7/dist-packages/botocore/session.py", line 383, in get_scoped_config
    raise ProfileNotFound(profile=profile_name)
botocore.exceptions.ProfileNotFound: The config profile () could not be found

Here is the successful run when AWS_PROFILE isn't set.
$ aws s3api head-object --region=us-east-1 --bucket secrets --key private_ssh_key
{
    "AcceptRanges": "bytes",
    "ContentType": "text/plain",
    "LastModified": "Wed, 14 Feb 2018 23:12:20 GMT",
    "ContentLength": 3243,
    "VersionId": "8zNleT1gVCq10Wxrcgd3DSccPo7BO2d2",
    "ETag": "\"1828798ae87a3dbc4dbd6ef7ff5f9d28\"",
    "Metadata": {}
}

I'm able to workaround this but is there anything that can be done within this plugin to make it easier for users?

Hmmm, that's a tricky one! Presumably you are setting AWS_PROFILE in an environment hook or similar? It being set in a build step shouldn't impact the plugin.

No, I'm really just setting it in the build step like this.
  - label: simple demo
    command: echo "hello world"
    env:
      AWS_PROFILE: bogus-profile

You can see that the first which uploads the pipeline yaml successfully downloads the s3 secrets.

You can also see here that its environment doesn't include the AWS_PROFILE. This is all normal behavior.

But here's the simple demo step. It fails to find any secrets in the s3 bucket.

And you can see that AWS_PROFILE is included in this step's environment because that's how I defined the step.

As I described in my original post the failure is occurring here because the aws s3api head-object command is being run with AWS_PROFILE set to a profile that hasn't yet been configured in ~/.aws/credentials.
https://github.com/buildkite-plugins/s3-secrets-buildkite-plugin/blob/master/lib/shared.bash#L8
I'm not sure of the best solution but perhaps it's possible for the s3-secrets plugin to unset AWS_PROFILE? But we would want to make sure that AWS_PROFILE is still correctly set after the s3-secrets plugin runs so the rest of the step works as expected.

Sorry for the slow response, spent some time thinking over what the correct API boundary was for these. I think you're correct and that we shouldn't be loading in credentials or config from ENV outside of what is configured for the plugin.

I’ve just tested this with the latest build of the secrets hook and a job environment AWS_PROFILE doesn’t interfere with the s3 secrets fetch any more :tada:
This was likely fixed with v2 of this plug-in and v5 of the Elastic CI Stack that uses it:

Let us know if you’re still running into this with an updated stack and we can take another look 🙇♂


Add support for alpha channels to Image and HSV Elements

Right now, the RGB Element supports an optional Alpha channel, but other Raster types do not.  It would be very handy to be able to control the alpha (opacity) from a data source, and it seems straightforward to add this functionality.
@philippjfr suggests that we should first unify the GridImage and Image classes (with RGB and HSV subclasses), which are currently separate for historical reasons (GridImage being added to support the new multidimensional gridded data interfaces).  There is no semantic reason why these should be separate objects, and the extra work required to add support for alpha channels to both versions can be avoided if the two are merged into a single class with different data backends.

The main problem is that if we implement it for Image now, then we have to go through the additional effort of making that code compatible with GridImage later. I could technically already support it for GridImage though, would look like this:
hv.GridImage({'x': x_coords, 'y': y_coords, 'z': val_array, 'alpha': alpha_array}, vdims=['z', 'alpha'])

The other question I have is whether you can imagine any alternative interpretation for an Image with two channels?

Well, an HSV+A image would have four channels if you count alpha, so there are definitely cases needing alpha that have more than two channels.  But no, I don't know of any two-channel images that make sense as anything but array-to-colormap + array-for-alpha-channel.
Bokeh has image_rgba, but I don't know whether the non-_rgba version supports alpha.


JDBC Impala SSL

Description
I'm triying to conect to impala with jdbc driver and SSL.
I added the .pem to my key store with:

keytool –import –alias my_alias     –file  \A\ImpalaCert\Ca_bundle.pem

and I tried with several string connections:
jdbc:impala://impala-host:21050/default;ssl=1;SSLTrustStore=C:.keystore;SSLTrustStorePwd=xxxxxx
jdbc:impala://impala-host:21050/default;ssl=1;SSLKeyStore=C:.keystore;SSLKeyStorePwd=xxxxxxxxx

I copied \jdk-17.0.1\lib\security\cacerts  to replace dbeaver cacerts file

but nothing worked.
DBeaver Version
Comunity Edition <IP_ADDRESS>304091457
Operating System
Windows 10
Database and driver
Impala with JDBC driver
Steps to reproduce
Configure JDBC strings as mentioned in description
Additional context
Herer bellow a log
[Simba]ImpalaJDBCDriver Error setting/closing session: javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target.
javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
unable to find valid certification path to requested target
unable to find valid certification path to requested target

Hello @lvillal
Could you please change this setting, restart DBeaver, and try to connect again?

Hi,
I don't have these options:
Here a screenshot:
[cid:de09cd1c-e67b-46ab-a500-339963d2af40]
[cid:c2a21462-ece1-4c07-ad79-b7168a6cca6b]

De: Anastasiya @.>
Enviado: jueves, 20 de abril de 2023 05:34
Para: dbeaver/dbeaver @.>
Cc: Leonardo Villalva @.>; Mention @.>
Asunto: Re: [dbeaver/dbeaver] JDBC Impala SSL (Issue #19750)
Hello @lvillalhttps://github.com/lvillal
Could you please change this setting, restart DBeaver, and try to connect again?
[2023-04-20 11_33_05-Preferences]https://user-images.githubusercontent.com/45152336/233308576-e03c03b4-12a2-41e1-9719-7dec48312a84.png
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/19750#issuecomment-1515938985, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAUH3D7LJBNDUTYMLTILJCTXCDYIXANCNFSM6AAAAAAXEO522Q.
You are receiving this because you were mentioned.Message ID: @.***>

Please provide your image via this GitHub page. It doesn't work via e-mail.

Here are my screenshoots
My connection config

Am I correctly understand, that you unchecked Use Windows trust store setting, restart DBeaver and it didn't help?

Let me know if the issue is still actual.
It's been a while since no update here, so I'm closing it.

Hi Friends,
I am trying to connect to impala through DBeaver24.0.3.
Getting the same error "unable to find valid certification path to requested target".
Is it possible to face this issue if the certificate is not valid?
Any suggestions please.

Hi Friends,
I am trying to connect to impala through DBeaver24.0.3. Getting the same error "unable to find valid certification path to requested target". I tried above steps, but no luck. Is it possible to face this issue if the certificate is not valid?
Any suggestions please.

Hi Friends,
It turned out to be an issue of an invalid certificate.
The issue is resolved by providing a valid cert irrespective of the toggling "Use Windows trust store" option.


Align "Upgrade from Gogs" document with more recent versions of Gogs

Gitea version (or commit ref): N/A
Git version: N/A
Operating system: N/A
Database (use [x]): N/A

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io: N/A

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
https://docs.gitea.io/en-us/upgrade-from-gogs/ is not up to date with more recent versions of Gogs. For example it mentions gogs dump but it was replaced with gogs backup in v0.10.8.
I'm not even sure it's still possible to migrate from current version of Gogs...

I just did a manual migration from gogs by copying over these tables from the gogs database into the gitea one (I omitted tables without data, there may be others that need to be copied):
action
attachment
comment
follow
issue_label
issue
issue_user
label
milestone
public_key
pull_request
repository
star
team

The migration seemed to work without totally breaking the app, but I get 404s now when I try to browse to a repo, or issue, I'll see if it's possible to fix that.

Yea same issue here

@Jackysi Luckily since I don't have a ton of repos I was able to fix mine by following this to recreate the missing repo_units table.
For type I set that to 1 and set config to {}, then just added the repo_id and created_unix from the repositories table for each repo. So when you're done, the table should look like this:
+----+---------+------+-------+--------+--------------+
| id | repo_id | type | index | config | created_unix |
+----+---------+------+-------+--------+--------------+
| 69 |      16 |    1 |     1 | {}     |<PHONE_NUMBER> |
| 70 |      16 |    2 |     2 | {}     |<PHONE_NUMBER> |
| 71 |      16 |    3 |     3 | {}     |<PHONE_NUMBER> |
| 72 |      16 |    4 |     4 | {}     |<PHONE_NUMBER> |
| 73 |      16 |    5 |     5 | {}     |<PHONE_NUMBER> |
| 74 |      16 |    6 |     6 | {}     |<PHONE_NUMBER> |
| 75 |      16 |    7 |     7 | {}     |<PHONE_NUMBER> |
+----+---------+------+-------+--------+--------------+

The above example has different things for type, not sure what all those values mean, but the important thing is that it 's not empty.
Depending on what features were used in Gogs there more manual building of tables would probably be needed (Gitea has some tables like repo_units that don't exist in Gogs), but this at least allows one to bring over all their repos, PRs and issues.

Before I migrated to gitea, I used gogs backup (on Ubuntu, run this in the path /opt/gogs) to dump the database, unzipped it, changed the Version.json from 17 to 15 and used again gogs to restore the database. This way, I had no database problems when using gitea 1.4 rc1. All my repos can be browsed including issues and I can pull and push.

I spent more time trying to fix the database (repo_units) etc than I would resetting my gogs setup. So I've started from scratch (except the config). Got it running pretty fast ^^.
I love that Gitea has API, but its lacking some cool features like github has. Specially downloading release by API key / token (sadly my Go knowledge is limited to add this) and few API calls like getting Tags (I fixed that by editing code and changing "tags: false" to true on the Releases route controller.
Anyhow. Great project. Love it. Love performance and all about it =) Might contribute when my Go knowledge gets far enough!

We managed to migrate from gogs 0.11.34 to gitea 1.3.3
Actually it was easier then expected :)
There was only one problem with the MariaDB when gogs/gitea complained about a

max key length is 767 byte

We installed this patch before we started with the migration and after that, everything was flawless.

To backup all your data on gogs you can follow this HowTo. Or just backup the data from your gogs install folder manually and dump the DB with mysqldump -u <user> -p<pswd> -v <gogs_db_name> > gogs_db_backup.sql.

Create you new install folder for gitea (e.g. /opt/gitea/) and copy the folder custom and data from the gogs install folder into the new gitea folder

mv $gogs_path/custom/conf/app.ini $gitea_path/custom/conf/gogs_app.ini

wget https://raw.githubusercontent.com/go-gitea/gitea/v$gitea_version/conf/app.ini -P $gitea_path/custom/conf

wget https://dl.gitea.io/gitea/1.0.1/gitea-1.0.1-linux-amd64 -P $gitea_path

$gitea_path/gitea-1.0.1-linux-amd64 web

Open the gitea website in your browser of choice and finish the setup ... look at your previous configuration at $gitea_path/coustom/conf/gogs_app.ini

Stop gitea

wget https://dl.gitea.io/gitea/1.2.0/gitea-1.2.0-linux-amd64 -P $gitea_path

$gitea_path/gitea-1.2.0-linux-amd64 web and let the migration take place (see gitea logs for details)

Stop gitea again and to the same for version 1.3.3

=> done :+1: maybe this helps others how want to do the migration too...

https://github.com/go-gitea/gitea/pull/3516 are also trying to resolve the max key length is 767 byte problem.

Update documentation. See: https://github.com/go-gitea/gitea/pull/3559

Should be resolved by #3559

@Schroedingers-Cat I am facing the same issue "Not a git repository ..."
Have you managed to solve it ? I have landed on gitea via migration from gogs. Its my first time using gitea.
Thanks,

@amit-handa
That error was just written in the log files. The actual interactions with the repositories from the webinterface were all possible. As nobody from the dev team (I also asked about this on the discord server) seemed to have an idea what was going on there, I just ignored it because everything was working fine.


Add support for Postgres array types

When using tap-postgres on a table with the string array type varchar[], the Singer schema message produced uses a sdc_recursive_string_array. target-redshift fails in this scenario with the error:
CRITICAL ('`schema` is an invalid JSON Schema instance: {...} is recursive')

In the future it would be great to add support for postgres array types.

Hey @teej, yeahhhhhhh...that type...on the one hand, I know we have very little interest to ever support infinitely recursive JSON Schema for obvious reasons 😉. On the other, this is a standard type that Singer can spit out. I'm wondering if the wisest way to handle this would be to not only detect when a type of that particular name comes up, but also to detect the actual schema itself. ie, not only compare the name, but also the type definition with something hardcoded in the code base.
Out of curiosity, what resultant type in Redshift would you expect this to be and how would you want it to be serialized in? Asking since the data itself would come in potentially an array, and then also creating TEXT columns (their equivalent) in Redshift is hella expensive...

@teej we're debating next features to tackle. Any feedback/input on the above?

@teej and @AlexanderMann - This came up in a search for similar error messages. Did either of you find a resolution or workaround found on this? We've run into something similar on another target and wondering if the better path is just to switch to another postgres tap variant rather than trying to support this type.

@aaronsteers I ended up abandoning singer and just wrote custom ETL.

@teej - Very sorry to hear that - and thank you for sharing your experience. One of the challenges with Singer (as you've seen first hand) is that it is very difficult for each target to handle all the edge cases and possible inputs from every other tap. In this case, I think the core issue may have been in the tap but regardless, I totally understand the frustration.
If you do have appetite to try Singer again in the future, or if you would like to build your own tap or target on an existing stack, you may want to consider the new Singer SDK. We recently launched the SDK as a way for all Singer taps and targets to have the same core capabilities, even at edge and niche cases - such as is represented here in this issue.
As more taps and targets migrate to the SDK, we hope and expect this can be a win for the stability and maintainability of the whole ecosystem.


missing https:// in the github repository link on first page?

It's not linking correctly in the PDF - I think there's a missing https://?

Yes, you are right. Thank you for catching this!

This can be updated in the final published version when the first issue is released.   Along with any other typos found!

fixed by #190


Improvement Request - Always tile a moving windows on mouse release based on the tiling configuration of that monitor

Symptom
When picking a window with the mouse and drag it by a tiny bit, the window goes in floating mode. A system to prevent this would be greatly appriciated. Moreover, when dragging a window on another screen , it always goes on floating mode. It would be nice to auto-tile based on that screens configuration. Only auto-float windows on mouse release once the window is released on a floating layout
How to Reproduce

Launch '....'
Press key '....'
See error

Expected behavior
A concise description of what you expected to happen.
Environment

Distro: [Fedora 40]
KWin version: [6.1.4]
Krohnkite version: [0.9.7]
List of KWin scripts in use: [krohnkite]

Notes
Anything that you want to tell developers about.

This is my biggest feature request. I use KDE because I want the ability to use my mouse to navigate and rearrange windows, but I want the windows to tile by default. It used to work pretty flawlessly in X11 but I know there's been a ton of work to get any of these Kwin scripts working in Wayland.
Here's my env:

Distro: [6.6.51-1-MANJARO]
KDE Plasma: [6.1.5]
Krohnkite version: [0.9.7]
List of KWin scripts in use: [krohnkite]

Hey @andrei93r I ended up figuring this out, and will make a PR, but this turns out to be a relatively easy fix I believe.
The issue is that even though the option "keepTilingOnDrag" which corresponds to the option "Always preserve the tiling status of a window upon dragging" isn't being set right. The fix you can do is run the command:
kwriteconfig6 --file kwinrc --group Script-krohnkite --key keepTilingOnDrag true
This can also be done by editing the kwinrc file manually:

Open the file: ~/.config/kwinrc
Add a field keepTilingOnDrag=true under the section [Script-krohnkite].

You may have to log out and back in for the changes to be reflected.

@Davey-Hughes
Hi,
I just tested this. It works with the provided instructions.
Thank you.


Add JVM flag UnlockDiagnosticVMOptions to default JVM options

Was recently working with a customer that was doing some performance testing. Customer was experiencing about a 10x performance decrease vs earlier testing runs. It appears that the JVM flag -XX:+UnlockDiagnosticVMOptions was lost between the customer test runs. Based on helping customer debug this and reading about it this JVM flag activates many performance related things to include intrinsics and inlining. To see this you can compare the output of:
1. `java -XX:+PrintFlagsFinal -version | grep Intrinsic` and `java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -version | grep Intrinsic`, and
2. `java -XX:+PrintFlagsFinal -version | grep Native`and `java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -version | grep Native`

Based on [..,] reading about it[,]  this JVM flag activates many performance related things to include intrinsics and inlining.

@dlmarion, would you mind dropping some references as links here? Curious to learn more about this.

I closed my browser tabs, but I will try to drop some knowledge here :-)
https://chriswhocodes.com/ - this site shows different options for different JVMs. I assume it's mostly correct.
https://alidg.me/blog/2020/12/10/hotspot-intrinsics
https://www.infoworld.com/article/2073676/hotspot-jvm-options-displayed---xx--printflagsinitial-and--xx--printflagsfinal.html
java -XX:+PrintFlagsInitial -version - prints flags and default values
java -XX:+PrintFlagsFinal -version - prints flags and values after JVM startup
java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -version - includes diagnostic options
So, now go back to https://chriswhocodes.com/ and select the JVM that you are using. Let's use https://chriswhocodes.com/hotspot_options_openjdk11.html as an example. Now, change the availability drop down from Show All to diagnostic. I believe that UnlockDiagnosticVMOptions unlocks all of these. Note the pages at bottom right. Included in this list are inlining of various native calls and UseSha256Instrinsics, UseSha512Intrinsics, UseAESIntrinsics, etc.

Likewise, I think that -XX:+UnlockExperimentalVMOptions unlocks all of the properties listed in the experimental availability dropdown item. A lot of Shenandoah options in there

My thought, in adding this to default Accumulo configuration, is that even though the option is called UnlockDiagnosticVMOptions, it's really unlocking some performance related things too.

It seems like a poorly designed side effect of "unlock diagnostic vm options" that this would automatically activate some performance related options.  That being said, whether we add this in to the default set of options should be based on how reliable those options are.  If we can count on this performance increase regardless of the updates in JVMs that come down the pike, then I am ok with this.

I think you would end up having to add -XX:+UnlockDiagnosticVMOptions in any case. For example, let's say that you want to use the GHASH Intrinsic (added in https://bugs.openjdk.org/browse/JDK-8073108).
It's not enabled by default:
~$ java -XX:+PrintFlagsFinal -version | grep -i GHash
openjdk version "11.0.15" 2022-04-19 LTS
OpenJDK Runtime Environment 18.9 (build 11.0.15+9-LTS)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.15+9-LTS, mixed mode, sharing)

If you enable just that option, you get the message saying you have to add -XX:+UnlockDiagnosticVMOptions:
~$ java -XX:+PrintFlagsFinal -XX:+UseGHASHIntrinsics -version | grep -i GHash
Error: VM option 'UseGHASHIntrinsics' is diagnostic and must be enabled via -XX:+UnlockDiagnosticVMOptions.
Error: The unlock option must precede 'UseGHASHIntrinsics'.
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

When added:
~$ java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -XX:+UseGHASHIntrinsics -version | grep -i GHash
     bool UseGHASHIntrinsics                       = true                                   {diagnostic} {command line}
openjdk version "11.0.15" 2022-04-19 LTS
OpenJDK Runtime Environment 18.9 (build 11.0.15+9-LTS)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.15+9-LTS, mixed mode, sharing)

But also enabled if you don't add -XX:+UseGHASHIntrinsics (and it's supported on your platform, etc.):
~$ java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -version | grep -i GHash
     bool UseGHASHIntrinsics                       = true                                   {diagnostic} {default}
openjdk version "11.0.15" 2022-04-19 LTS
OpenJDK Runtime Environment 18.9 (build 11.0.15+9-LTS)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.15+9-LTS, mixed mode, sharing)

So, if you add -XX:+UnlockDiagnosticVMOptions you enable all of the "diagnostic" options. If any of them is unreliable, then you will likely have to disable it using -XX:-<option>.

If any of these are marked experimental, then I think we should not be enabling them by default.  Would providing an "experimental" config, along side (or enabled from) the standard configuration be a better path?  We could point to these things as you are likely to want, but you should test in your environment, rather than just making them our default - which seems to add weight that we know that they are appropriate?

If any of these are marked experimental

I don't believe these are experimental. There is another option for experimental properties, you can see them with:
java -XX:+PrintFlagsFinal -XX:+UnlockExperimentalVMOptions -version | grep experimental

Well then I have to ask whether there are other options that the UnlockDiagnosticVMOptions does not automatically enable that we may want to use?

I think you would have to find the properties that you may want to use first....

This issue is where they made all of the instrinsics diagnostic. This issue notes that they had to update all of the intrinsic tests to add UnlockDiagnosticVMOptions to the JVM options as some of the intrinsics moved from product (and enabled by default) to diagnostic. Follow-on issue to update the documentation for instrinsics to note that UnlockDiagnosticVMOptions is now required.

I cannot find any documentation that says that setting this flag automatically enables some diagnostic options. Everything I can find seems to suggest that you need to set this flag in order to manipulate these diagnostic options (to diverge from their default, which may already be on). So, unless we are using a diagnostic option by default (if we are, which one?), there's no point in us setting this by default on its own, because a user can always do that when they want to use one. The same goes for experimental flags.

Search for intrinsic in https://docs.oracle.com/en/java/javase/11/tools/java.html#GUID-3B1CE181-CD30-4178-9602-230B800D4FAE

So, I agree that the documentation says Flags that control intrinsics now require the option -XX:+UnlockDiagnosticVMOptions. However, as shown in my comment above, supplying only the -XX:+UnlockDiagnosticVMOptions property does enable some intrinsics. I think the comments on this PR provide enough documentation on this issue. I'm fine if we close this without merging.

So, I agree that the documentation says Flags that control intrinsics now require the option -XX:+UnlockDiagnosticVMOptions. However, as shown in my comment above, supplying only the -XX:+UnlockDiagnosticVMOptions property does enable some intrinsics. I think the comments on this PR provide enough documentation on this issue. I'm fine if we close this without merging.

That's not how I interpret the output you're seeing. I think all you've shown is that supplying only that option when printing the list of options allows the diagnostic options to show up in the printout. When you don't supply that option, none of the diagnostic options are included in the printout. When you do supply that option, it shows the current values of diagnostic options that you can control. I don't think you're actually enabling any options by setting the unlock flag... you're just revealing them. For the example you gave of UseGHASHIntrinsics, I think the default is true. So it's going to show up as enabled, when you include diagnostic options in the printout. But, it doesn't mean it's turned off when the flag is hidden... it just means you can't see it... because all diagnostic flags are suppressed from the printout unless you add the unlock flag.

Consider this:
diff -u <(java -XX:+PrintFlagsFinal -version) <(java -XX:+PrintFlagsFinal -XX:+UnlockDiagnosticVMOptions -version)

This appears to show a mix of options, defaulting to on and off. The extra ones with the unlock flag seems to just include the diagnostic options in the output.
A related discussion can be seen at https://github.com/trinodb/trino/issues/12253 and https://github.com/trinodb/trino/pull/12251. There, it is clearly suggested that the unlock flag is no longer needed, since the underlying diagnostic option defaults to the value they desire for performance. That would only make sense if the unlock flag is merely revealing the default values for these flags, and not actually altering them. It really wouldn't make sense that unlocking a set of options would alter their values.
A similar user experience exists on Android phones for developer options, as well as Chrome browser flags for experimental options. It is a common thing to hide flags like these in a locked area. But, unlocking that area to change the flags doesn't actually change them from their defaults. It merely grants you access. It really seems like that's the same design here. Unless we can actually demonstrate that the unlock flag is enabling features, by actually measuring it, and not merely interpreting the printout, I'm inclined to think that is the same design here... and this flag isn't needed at all.

For the example you gave of UseGHASHIntrinsics, I think the default is true.

The default is false. I believe it becomes true when the JVM is running on hardware that supports it or maybe when the source is compiled for a specific OS.

That's not how I interpret the output you're seeing. I think all you've shown is that supplying only that option when printing the list of options allows the diagnostic options to show up in the printout. When you don't supply that option, none of the diagnostic options are included in the printout. When you do supply that option, it shows the current values of diagnostic options that you can control.

That could be, I had not considered that.

Unless we can actually demonstrate that the unlock flag is enabling features, by actually measuring it, and not merely interpreting the printout, I'm inclined to think that is the same design here... and this flag isn't needed at all.

I'm happy to concede that I still don't know how this works. You have supplied a sufficient counterpoint that I would have to go back and re-read everything on this topic. My initial comment indicates that merely providing this flag, the customer had a 10x increase in performance. Maybe there is more information that I was not told. Again, I'm fine with closing without merging.

For the example you gave of UseGHASHIntrinsics, I think the default is true.

The default is false. I believe it becomes true when the JVM is running on hardware that supports it or maybe when the source is compiled for a specific OS.

I have been trying to dig through the hotspot code to see if I can understand how the defaults are set. The globals file you link to and the values shown there don't match the default values output to my screen. So, I don't trust that those match the actual defaults for any particular installation.
The description of the unlock flag says that it just "Enables normal processing of flags". It doesn't mention anything about turning anything on or off... just how it processes other flags. I've been trying to confirm this in the OpenJDK source code, but I am having a hard time following that code. The closest I've come is finding:

the check that ensures the diagnostic flag is set to prevent diagnostic flags from being processed if it isn't
the check that ensures the flag is unlocked to determine whether or not to print it

I can't find any code related to altering flags when the unlock field is set... only command-line processing checks and print filtering.

The globals file you link to and the values shown there don't match the default values output to my screen. So, I don't trust that those match the actual defaults for any particular installation.

My understanding is that during JVM startup an intrinsic may be enabled due to the machine architecture or presence of certain capabilities on the host. So, the value may default to false in the source code, but may be reported as true in PrintFlagsFinal.

My understanding is that during JVM startup an intrinsic may be enabled due to the machine architecture or presence of certain capabilities on the host. So, the value may default to false in the source code, but may be reported as true in PrintFlagsFinal.

Yeah, that was my understanding as well. The only "default" value we care about for this conversation (and the one I meant before we started linking to source :smiley_cat: ) is "no flags specified" default, not the "hard-coded in source" default.

Closing this, I think we have documented enough information here for now. We can re-open when we get more information.


[FIX] Fix Common principles Key/value files section level

Fix Common principles Key/value files section level.

I assume it makes sense to group all considered file formats under the same section/it was an oversight to put Key/value files in an upper level.

Thanks @jhlegarreta


umask is too strict in ~/.pub-cache/hosted/pub.dartlang.org/* directories

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

Create a package with any dependencies from pub.dartlang.org
run pub get
See that the permission mask of ~/.pub-cache/hosted/pub.dartlang.org/<dependency> is 700 rather than 755.

What is the expected output? What do you see instead?
I expect the permissions of those directories to be the same as the parent, 755. Instead the permissions are 700.
What version of the product are you using? On what operating system?
Ubuntu 13.10
Pub 1.0.0+10.r30798
Please provide any additional information below.
In particular this causes issues when trying to serve a Dart application via mod_userdir in Apache during development. Apache follows the package symlinks until it reaches the package directory, then cannot read from the directory to serve the files.

This issue has been moved to dart-lang/pub#812.


Improve nullability and testing

Codecov Report

Merging #7 into master will increase coverage by 20.35%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master       #7       +/-   ##
===========================================
+ Coverage   72.67%   93.03%   +20.35%
===========================================
  Files          10       10
  Lines       24779    24779
  Branches    19847    19847
===========================================
+ Hits        18009    23054     +5045
- Misses          2        4        +2
+ Partials     6768     1721     -5047

Impacted Files
Coverage Δ

src/SortingNetworks/Assert.cs
40.00% <ø> (-60.00%)
:arrow_down:

src/SortingNetworks/SNBestKnown.Comparison.cs
87.19% <ø> (+12.80%)
:arrow_up:

src/SortingNetworks/SNBoseNelson.Comparison.cs
93.68% <ø> (+21.61%)
:arrow_up:

src/SortingNetworks/SNBestKnown.Comparable.cs
87.99% <0.00%> (+14.07%)
:arrow_up:

...c/SortingNetworks/SNBestKnown.ComparisonPointer.cs
88.92% <0.00%> (+15.01%)
:arrow_up:

.../SortingNetworks/SNBoseNelson.ComparisonPointer.cs
93.45% <0.00%> (+21.64%)
:arrow_up:

src/SortingNetworks/SNBoseNelson.Comparable.cs
93.62% <0.00%> (+21.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Releasing the AlphaFold package on pypi

Thank you for sharing the AlphaFold codebase!
I wanted to suggest releasing the package to pypi to make it easier to install especially in production environments, if possible, including the run_alphafold script.
Please let me know if you'll consider this suggestion.

Hi, it should be available as the alphafold package - https://pypi.org/project/alphafold/. Can you test it?

@tfgg Oh, thank you!
It wasn't mentioned in the README or the release page, so I assumed it wasn't hosted on pypi without checking for myself.

Thanks, we'll add to the README.

@tfgg This may be worth opening a separate issue, but I'd like to bring your attention to this issue I encountered while trying to run the run_alphafold.py script with the alphafold package installed (instead of cloning the repo).
# python run_alphafold.py run_alphafold.py
Traceback (most recent call last):
  File "run_alphafold.py", line 31, in <module>
    from alphafold.data import pipeline
  File "/opt/conda/envs/[CONDA_ENV_NAME]/lib/python3.8/site-packages/alphafold/data/pipeline.py", line 26, in <module>
    from alphafold.data import templates
  File "/opt/conda/envs/[CONDA_ENV_NAME]/lib/python3.8/site-packages/alphafold/data/templates.py", line 31, in <module>
    from alphafold.data.tools import kalign
ModuleNotFoundError: No module named 'alphafold.data.tools

And indeed, here are the contents of alphafold/data/ in the site packages:
# ls /opt/conda/envs/deep_chain/lib/python3.8/site-packages/alphafold/data/
__init__.py  __pycache__  mmcif_parsing.py  parsers.py  pipeline.py  templates.py

They're missing the tools directory which does exist in the source: https://github.com/deepmind/alphafold/tree/main/alphafold/data

This is due to a missing __init__.py in alphafold/data/tools. We will push a fix for that in the next bugfix release.

This is due to a missing __init__.py in alphafold/data/tools. We will push a fix for that in the next bugfix release.

TY Augistin!


Write PyTests for Stock

Write tests for all classes/methods/functions in the file stock.py

All the tests are failling with pytest

@tandrade96 what is the working directory path you are running the tests from?

/marketools <= running from here
/marketools
/analysys
/stqscraper
/tests

It works for me, perhaps it is some issue with operating system. Are you running it on Windows? I personally use Kubuntu.

Yeah I'm running Windows
El mié, 2 dic 2020 a las 23:27, Albert Ratajczak<EMAIL_ADDRESS>escribió:

[image: image]
https://user-images.githubusercontent.com/39762396/100892756-f55ce780-34ba-11eb-8c74-48d6a546c94c.png
It works for me, perhaps it is some issue with operating system. Are you
running it on Windows? I personally use Kubuntu.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlbertRtk/marketools/issues/3#issuecomment-737301994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADF45ZIJ4MUIWTVWZMQ6TXLSSZMEPANCNFSM4UGQDTMA
.

@kushkansara02 did you have similar issue? What is operating system you use?

I tested test_wallet.py, passed all tests.

How and from where did you call it?
I'm working now but in a couple hours I can keep trying
El jue, 3 dic 2020 a las 3:19, kushkansara02<EMAIL_ADDRESS>escribió:

[image: Screen Shot 2020-12-02 at 2 18 26 PM]
https://user-images.githubusercontent.com/57120148/100920529-41069580-34a9-11eb-95d5-694d8df90405.png
I tested test_wallet.py, passed all tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlbertRtk/marketools/issues/3#issuecomment-737441011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADF45ZIO6T7H6RLYYAHOZFLSS2HK7ANCNFSM4UGQDTMA
.

I use PyCharm. I set the working directory to the root directory (marketools/), and set up a test for test_wallet.py. When I executed that test, everything worked fine.

Solved it, thanks guys!


feat(users): update create user form design

Closes #221

:tada: This PR is included in version 1.11.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Couple changes to the toolbar button

One of the first things that people will do after they try the sample is to create a new project. The tool bar space should be reserved for things that people will do more often.

[ ] New Project should be on the tool bar - brand new users won't look in "more" for this
[ ] I think Open Project should be on the tool bar too ... kind of goes with New
[ ] Embed should be moved to the "More" menu.

Looking at "Embed" ... I see that also covers "Publish". Maybe it should be on toolbar but it is just confusing that is where you publish something

Needs to be discussed.

I don't mind either way.

We have moved the Open Project on the toolbar and kept the new Project inside the more menu. Closing this.


Is Zuul being maintained and enhanced so that it makes sense to build an API routing solution around it?

We are using Zuul as our API Gateway but we are wondering how long Zuul will be a good foundation for an API Gateway.  Zuul hasn't had a release for 2 years.  The commit history appears to be mostly dependency upgrades by dependabot and minor enhancements and fixes.  As API usage changes with new approaches, just wondering if it is likely this project will change to meet new challenges and opportunities.

Yes, Zuul is still supported and used widely in production.

Thanks for confirming @artgon.  We've been very happy with Zuul and impressed with how well it scales.


Send Gift:Wrong message.

The messages here are still wrong. The 30 day sentence is the instructions in the previous screen. And the message from the user is the second message, not what is here.

@ASN-BitHyve I'll make the necessary changes once I've apt message from your end.

Gift Card > Gift sats

"You have received a gift from (name of gift giver). You can choose to add the sats to any of your accounts, or forward the gift to any of your friends and family (at no extra fees!)"

@Parsh

Verified this issue on staging app v2.0.5(382)


Version bump to 2.183.0

Auto-generated by fastlane 🤖
Changes since release '2.182.0':

[action][danger] add fail-if-no-pr flag to Danger action (#18657) via Jorge Revuelta
[docs] update AppStoreConnect.md (#18651) via Adam St. Onge
[Ruby 3.0] replace obsolete URI.escape with alternatives (#18646) via Satoshi Namai
[scan] fix regular expression to find failed tests suite-name (#18661) via Manish Rathi
[action][git_pull] clean-up the verify-blocks for Boolean validation (#18662) via Manish Rathi
[docs] spaceship README - update paths + url to apple auth services (#18667) via Mike Hardy
[fastlane_core] fix parsing of passwords with trailing spaces due to wrong whitespace handling in Fastlane::Shell::password method (#18684) via Artem Bliednov
[Regression][fastlane_core][deliver] fix deliver is unable to automatically select the latest build and submit it for review (#18682) via Lukasz Grabowski

[Ruby 3.0] Migrate google-api-client gem and drop Ruby 2.4 (#18656) via Satoshi Namai
The bot got confused😄 @joshdholtz This is missing ^
I'm not sure if this will be used changelog stuff

@ainame Thank you! I confused the bot when I merged that PR into non-master branch lol 🤦♂

No worries! Looking good now.


carpet ''gui'' it's not working with deceptive Dinos

it says it dont detect anygame while i am in one, the ''set bones'' it's not working tho, i tryied to use it but it just gives me a error

@VillainsRule4000 can u fix it? or tell gliz to fix it

@therealgliz  @lmfao-jude

@VillainsRule4000 can u fix it? or tell gliz to fix it

Why ping me LOL?
I'll look at it.

LOL

I ping you cuz you only one that check the issues, other contributors dont check

@VillainsRule4000 Did u fix the hack?

(async () => {
    let n = document.createElement('iframe');
    document.body.append(n);
    window.alert = n.contentWindow.alert.bind(window);
    window.prompt = n.contentWindow.prompt.bind(window);
    window.confirm = n.contentWindow.confirm.bind(window);
    n.remove();
            try {
                function reactHandler() {
                    return Object.values(document.querySelector('#app > div > div'))[1].children[1]._owner;
                };

                let fossils = Number(parseFloat(prompt('How many fossils do you want?')));
                reactHandler().stateNode.setState({ fossils: fossils });

                alert('Fossils added!');
            } catch (e) {
                if (confirm('Game error occured please report to discord server as soon as possible')) {
                    window.open('https://discord.gg/Nj9Zs5VtFp');
                };
            };
})();

This should work ^^^

skill issue L


[eBPF] Fix MySQL packet loss problem

Phenomenon and reproduction steps
Root cause and solution
Impactions
Test method
Affected branch(es)

main

Checklist

[ ] Dependencies update required
[ ] Common bug (similar problem in other repo)

commit-msg 完善下


Understanding how data is passed for action selection and training

Hi. I am having trouble understanding how the batch data is passed to the controller during action selection and training. From the paper, my understanding is that, if a batch is composed of 32 episodes with, let's say, 100 transitions, the data is fed to the controller one step at a time, as in:

Transition 1 of all 32 episodes
Transition 2 of all 32 episodes
...
Transition 100 of all 32 episodes

This is the loop I'm talking about. Is my understanding correct? Is this done due to the presence of the recurrent layer in the agent?
https://github.com/oxwhirl/pymarl/blob/e633fdf50c08a27a8ed43ffc69c7156bc44b5024/src/learners/q_learner.py#L47-L52
Then, during action selection, I don't understand if the data is passed to the controller in this same way. I'd imagine it would be passed as:

Transition 1 of the current episode
Transition 2 of the current episode
...
Current last transition of the current episode

Then the actions would be selected from the output of this last time step. I couldn't find where this is done in the code.
Thanks in advance.

Another thing that is causing me some confusion is this line:
https://github.com/oxwhirl/pymarl/blob/e633fdf50c08a27a8ed43ffc69c7156bc44b5024/src/learners/q_learner.py#L50
Is forward operating over the transitions from batch at time step t, or is it operating over a slice of transitions from time step 0 up until t? So something like:

Transitions [1] of all 32 episodes
Transitions [1, 2] of all 32 episodes
...
Transitions [1, ..., 100] of all 32 episodes

Again, from the paper, it looks like it's the first option, but I am having a bit of trouble understanding how the Batch and ReplayBuffer objects work and I am having trouble debugging the code, so these clarifications would help me a lot...
Thanks again and sorry for the confusion.


cargo make in workspace does not work when members are not explicitly listed

Problem Description
cargo make panics when workspace members are not explicitly listed.
Code Example
I have the following crate structure:

workspace_test is a root workspace crate
member1 and member2 are members

workspace_test
├── Cargo.lock
├── Cargo.toml
├── member
│  ├── member1
│  └── member2
└── src
   └── lib.rs

In workspace_test I have the following Cargo.toml:
[package]
name = "workspace_test"
version = "0.1.0"
authors = ["Azriel Hoh<EMAIL_ADDRESS>
[dependencies]
member1 = { path = "./member/member1" }
member2 = { path = "./member/member2" }

[workspace]
# note: no members explicitly listed

This is a valid set up based on Cargo's workspace section docs:

All path dependencies residing in the workspace directory become members.

Error Stack/Info (if any)
When I run cargo make I get:
[cargo-make] info - cargo-make 0.3.58
[cargo-make] info - Using Build File: Makefile.toml
[cargo-make] info - Task: default
[cargo-make] info - External file not found, skipping.
[cargo-make] info - Setting Up Env.
[cargo-make] info - Setting Up Env.
[cargo-make] info - Running Task: init
[cargo-make] info - Setting Up Env.
[cargo-make] info - Running Task: workspace
[cargo-make] info - Setting Up Env.
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', /Users/travis/build/rust-lang/rust/src/liballoc/vec.rs:1555:10
stack backtrace:
   0:        0x1060c0af3 - std::sys::imp::backtrace::tracing::imp::unwind_backtrace::hea646f639f5724a9
   1:        0x1060c441a - std::panicking::default_hook::{{closure}}::h85c76fcfe66fd3c4
   2:        0x1060c40d2 - std::panicking::default_hook::h3eec3c203ceba436
   3:        0x1060c6692 - std::panicking::rust_panic_with_hook::h70dc815ddbea04a3
   4:        0x1060c6574 - std::panicking::begin_panic::h964e463242cdda22
   5:        0x1060c6442 - std::panicking::begin_panic_fmt::h5836acd93f32a20b
   6:        0x1060c63aa - rust_begin_unwind
   7:        0x1060ef373 - core::panicking::panic_fmt::hf5ab42af92a0e0c7
   8:        0x1060ef2e8 - core::panicking::panic_bounds_check::h0d8311933e2799f3
   9:        0x105ff9d6b - cargo_make::command::run_script::hb0271043a70e11b9
  10:        0x105ffd6a7 - cargo_make::runner::run_task::h1e124d98163bfa20
  11:        0x105fffca5 - cargo_make::runner::run_flow::h4fa2ea28900b8575
  12:        0x106002c4a - cargo_make::cli::run_for_args::hb238ff9e91649f98
  13:        0x10600480f - cargo_make::main::heaf0a96bae02441f
  14:        0x1060c776c - __rust_maybe_catch_panic
  15:        0x1060c6ae8 - std::rt::lang_start::h97e5b1c9e293b92d

No line numbers, because it's probably release mode, and also I'm on OS X which rustc doesn't give line numbers anyway. However we can at least see the panic happens in command.rs#run_script.
Btw, if I add a glob like so:
[workspace]
members = ["member/*"]

which is also a valid configuration:

The path dependency can also use globs to match multiple paths.

Running cargo make from the workspace root will run the default task on workspace_test and member, but not member2
I'd've started looking into why, but then I had to type this twice because I accidentally clicked on a link and lost the first post ._.. Btw awesome crate!

Right now I need those workspace members defined in the cargo.toml, but you are right that it is not a must based on the cargo spec.
So I'll have to add support for that.
Thanks for reporting it :+1:

By the way, for a quick workaround so you could continue working you could just define them.
I'll try to add support for it as soon as possible but it might not be trivial.

Just published a new version that adds members if defined as dependencies that start with './'.
Hope that helps.
Hopefully I'll also be able to add glob member support later this week.

Just added glob support, so that was the last thing that was missing for full workspace support.
Please check and tell me if it all works well for you or if anything else is missing.


Remove Reststate implementations

I'm the maintainer of the Reststate libraries, and they will not be maintained going forward. This removes them from the implementations page so readers will be directed to maintained libraries instead.

@CodingItWrong thanks for the update!


chore(main): release 1.5.0

:robot: I have created a release beep boop
1.5.0 (2023-02-17)
Features

更新文章搜索、详情、评论、用户资料查看、昵称修改 (18959ab)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mason369/Heimatoutiao/releases/tag/v1.5.0 :sunflower:


Configuable NCCL timeout

Feature request
Is there a way to make the NCCL timeout configurable as we often get timeout problems with the starcoder model?
[E ProcessGroupNCCL.cpp:828] [Rank 1] Watchdog caught collective operation timeout: WorkNCCL(SeqNum=1288224, OpType=ALLREDUCE, Timeout(ms)=60000) ran for 66470 milliseconds before timing out. [E ProcessGroupNCCL.cpp:455] Some NCCL operations have failed or timed out. Due to the asynchronous nature of CUDA kernels, subsequent GPU operations might run on corrupted/incomplete data.
https://github.com/huggingface/text-generation-inference/blob/5a1512c0253e759fb07142029127292d639ab117/server/text_generation_server/utils/dist.py#L53
Motivation
That is to fix the timeout problem with NCCL
Your contribution
It is quite easy to make it configurable with an env variable e.g., (NCCL_TIMEOUT). If that is ok, I can create a PR.

Increasing the timeout will only make you crash later, it will not fix the issue.
When do you have this issue?

Thanks @OlivierDehaene , it is quite random, and no specific input leading to it. And the docker container is crashed after that.

Usually NCCL timeouts because one of the shard OOMs and the other shard end up waiting indefinetly for the OOMed shard. Can you check if this is what is happening in your case?
The solution then is to decrease max-batch-total-tokens.

I faced this issue today as well. After digging around for sometime I found that setting environment variable NCCL_P2P_DISABLE=1 fixes the issue.  Try and see if this works for you.

Thank you so much, let me try it

I'm having the same issue, and I can't quite figure it out.
docker run --gpus '"device=0,1"' --shm-size 1g -e HUGGING_FACE_HUB_TOKEN=$HUGGING_FACE_HUB_TOKEN -p 8000:8000 -v /mnt/machinelearning/:/data ghcr.io/huggingface/text-generation-inference:1.0.3 --model-id meta-llama/Llama-2-7b-chat-hf --sharded true
I'm a bit out of my depth here, but here's what I found so far:
Both (or all GPUs) get 100% utilization, get a bit increased memory utilization, but then never more, so it doesn't seems like it's an OOM issue

It happens for our H100 PCIe, unfortunately I have nothing else to compare to. From what I can tell, P2P should be working fine and throughput is high, so disabling it neither solves the issue nor does it seem sensible for us. I've tried setting various specific NCCL_P2P_LEVELS, with no success.
Model size seems to have no impact, as you can see this happens even on models that should easily fit a single GPU.

What type of server is it? How are the H100s inter-connected?


Subprojects / Multi-project builds

One of the things that Gradle and Maven both do well that few others do is multi-project builds. Tasks like "compile", "test", and "pack" are defined per project rather than once at the root (although you can define them once in a way that applies to all subprojects). In large solutions, the ability to run tasks for particular projects rather than the entire solution is critical to efficient developer productivity. As an example in Gradle, when I am developing in a particular subproject, I can run the tests for just that project by executing gradle myproject:test.
I think this is actually one of the most challenging problems for a .NET build system to solve, because of our reliance on MSBuild. In Gradle, dependencies between projects are expressed in the build script, allowing Gradle to ensure that all dependent projects are built. For example, if project B depends on project A, and I execute gradle B:compile, Gradle knows that A:compile must run first and that the artifacts from A must be on the compilation classpath for B. This ensures that any dependencies of A:compile are also run, and that B has what it needs to compile, all from a single dependency declaration. But in .NET, dependencies between projects are expressed as project references in the csproj file. We can't realistically break that convention, as the majority of developers rely on the csproj for their IDE integration (Visual Studio / SharpDevelop / Xamarin Studio). The new project system (project.json) may improve that situation somewhat, but we should assume that the MSBuild format will remain dominant for some period of time. Which means that dependencies are declared in a separate file. This creates two problems:

Creating a Gradle-like dependency graph in Bau will require either forcing dependencies to be duplicated in both the csproj and the Bau project file, or Bau will have to read the MSBuild project file and interpret the references (or some other solution to this that I haven't thought of; CoreFX does some interesting things with Nuget dependencies and references from within MSBuild that are worth investigating).
Using MSBuild to build a project will automatically build referenced projects as well outside of Bau, meaning that there is no way from within Bau to ensure that dependencies of referenced build tasks will be executed. (There is a MSBuild property called BuildProjectReferences which may help with this).

Even with these potential issues in the way, I think the advantage of clean multi-project build support are too great to ignore.

Interesting.
A naive question: if I have A.csproj which declares B.csproj as a dependency, does MSBuild not know to build B.csproj if I tell it to build A.csproj?

It certainly does. However, when MSBuild builds B.csproj, Bau doesn't know that it's doing that. So we can't build any kind of workflow that involves building B in Bau (such as generating files before the build).
The trick of course is that even if we do build such a workflow, neither Visual Studio nor any other .NET IDE will be aware of it.
I am working on building a basic plugin on top of the currently existing API to demonstrate the concept and determine more specifically what the gaps are.

So if I understand you correctly, you want to explore an integration between msbuild targets and bau tasks, like interweaving them? I know a few people to ask, I'll see if they have any thoughts on that. Also another thing to consider is that msbuild may end up as a legacy tool in a few years so there may be some real benefit to this kind of thinking.

So if I understand you correctly, you want to explore an integration between msbuild targets and bau tasks, like interweaving them?

Not exactly. Let me try to illustrate with an example. Imagine the Bau build file looked, in part, something like this:
bau.Subproject("core", "src/Bau")
    .MSBuild("build").Do(msb =>
    {
        msb.Solution = "Bau.csproj";
        ...
    })
    .Exec("pack").DependsOn("build").Do(exec =>
    {
        exec.Run(nugetCommand)
            .With(
                "pack", "Bau.nuspec",
                ...
            );
    });

I could then run build core:pack, and it would run build and pack for just the core project. Now I add a similar definition for Bau.Exec:
bau.Subproject("exec", "src/Bau.Exec")
    .MSBuild("build").Do(msb =>
    {
        msb.Solution = "Bau.Exec.csproj";
        ...
    })
    .Exec("pack").DependsOn("build").Do(exec =>
    {
        exec.Run(nugetCommand)
            .With(
                "pack", "Bau.nuspec",
                ...
            );
    });

Now I can run build exec:pack, and it will build and pack the exec project. This level of granularity doesn't matter much for a small solution like Bau, where we can quickly build the entire solution; but it is a big win for large multi-project solutions, where invoking tasks specifically related to the part of the code that I am currently working on is much more efficient.
However, the downside to this is that building the exec project with MSBuild will also cause the core project to be compiled, but in a way that Bau has no visibility to. This is unintuitive to me as the user, since I may have defined other workflow in my Bau build script related to "core:build" (for example I might make it depend on another task that generates source files before the build). I would expect that any time core is built, the dependencies that I have defined will run first. But in this case they wouldn't. I can work around this by telling MSBuild not to build dependent projects, and defining those dependencies in my Bau build script instead:
bau.Subproject("exec", "src/Bau.Exec")
    .MSBuild("build").DependsOn("core:build").Do(msb =>
    {
        msb.Solution = "Bau.Exec.csproj";
        msb.Properties = new { BuildProjectReferences = false };
        ...
    })

Now, when I run "build exec:build", Bau will run "core:build" - and any of its dependencies - first, and then run "exec:build", but without attempting to re-compile core. This is exactly what we want, except that we are now required to specify dependencies between projects at the task level, which is redundant with the project references that already exist in the csproj file. If we add a new reference to the csproj file in Visual Studio, but we don't update the Bau build script, then our compile will fail at best, or worse will use an outdated build of the new referenced project, probably without us realizing it, leading to bugs and strange behavior that will be difficult to track down. We may able to mitigate this by inspecting the csproj file and determining build dependencies automatically; but I haven't attempted to prove that out yet.

To reduce redundancy I wonder if a simpler approach would be to integrate Bau into each msbuild project using a tool. Something as simple as bau.Task("core.csproj:before") and bau install core.csproj may take us pretty far and allow for integration with tools that revolve around MSBuild, like visual studio and well... MSBuild itself.

@aarondandy While that might make it easier to define "before" steps in MSBuild, I worry that the integration would be unintuitive when invoked from Bau. Using our previous example, if I invoke exec:build, and it invokes MSBuild to build the exec project, which builds the core project, which invokes the "before" task in Bau, will this second instance of Bau have any of the context of the original instance of Bau (such as what tasks have already been invoked)? If not I think it would be difficult for me as a user to reason about my build. I also think this awareness would be difficult to achieve, unless we move the execution of MSBuild in-process, which may or may not be feasible.
I still think our best bet is to not rely on MSBuild's automatic dependency building (by setting BuildProjectReferences=false), but instead read the dependency information from the csproj file and use to create the task dependency graph in Bau.

As I have thought more about this and done some spiking using Bau's build, subprojects don't necessarily have a lot of value on their own. After all, granular tasks such as in my example above can be done using the current API and a naming convention; you don't need subprojects for that. Subprojects don't really start to show their value until you start adding in other features that make creating dependency graphs easier. I am going to log some of those potential features separately and reference this issue.

Still thinking about this...one of the big benefits of subprojects is that they create a convenient scope for applying plugins, such as #220. The plugin can use the configuration of the project (most importantly its location) to configure itself automatically.
It might help if I make the discussion a little less abstract. If we combine this issue with #219, #220, #221, and a few other things, I can imagine the Bau build file someday looking like this:
// parameters
var versionSuffix = Environment.GetEnvironmentVariable("VERSION_SUFFIX");
var nugetVerbosity = Environment.GetEnvironmentVariable("NUGET_VERBOSITY") ?? "quiet";

// solution specific variables
var version = File.ReadAllText("src/CommonAssemblyInfo.cs").Split(new[] { "AssemblyInformationalVersion(\"" }, 2, StringSplitOptions.None).ElementAt(1).Split(new[] { '"' }).First();

// solution agnostic tasks
var bau = Require<Bau>();

bau
.ForAll<MSBuild>(msb => {
	msb.MSBuildVersion = "net45";
	msb.Verbosity = bau.GetProperty<Verbosity>("msBuildFileVerbosity");
})
.ForAll<NuGetTask>(n => {
	n.Verbosity = nugetVerbosity;
})
.ForAll<Pack>(n => {
	n.Version = version + versionSuffix;
	n.Property("Configuration", "Release");
})
.Project("bau").At("src/Bau").CSProj().NuGet()
.Project("exec").At("src/Bau.Exec").CSProj().NuGet()
.Project("xunit".At("src/Bau.Xunit").CSProj().NuGet()
.Project("test").At("src/test").With(p =>
	p
	.ForAll<Xunit>(x => {
		x.Html().Xml();
	})
	.Project("unit").At("Bau.Test.Unit").CSProj().Xunit()
	.Project("component").At("Bau.Test.Component").CSProj().Xunit()
	.Project("acceptance").At("Bau.Test.Acceptance").CSProj().Xunit()
)
.NuGetRestore("src/Bau.sln")
;

bau.Run();

There are ways it could get even more succinct, but I hope that gives you the idea of where I imagine Bau going.


Support to give docker pull command to scan docker image

Currently it only support upload tar file.
The user should give the docker image name and tag while creating project and the tool should pull the image from backend if docker is already installed in the machine.

@sameer1046 Good idea, PR welcome :)

@sameer1046 this is now part of the latest release https://github.com/nexB/scancode.io/releases/tag/v21.5.12 if you can try it.
Basically you can input a URL in this style: docker://debian:buster-slim like below. Just take anything you would docker pull xxx and use  docker://xxx instead.

NICE!!!

Feature Implemented


ansible isn't installed onto the vagrant machines and playbook didn't run...

When I do a vagrant up all 3 machines get started, but when I ssh into them ansible isn't installed and the playbook doesn't appear to have run...can you help me figure out what I am doing wrong?

Hi, @adespain Ansible only install in master1, could you check master1 has been installed yet?


(GH-15) Added ViewServiceStack Unit Tests

This PR is for #15
I added the following:

View Stack Service unit test
TopModal and TopPage to display what is currently on the top of the stack

Great work @RLittlesII
Thanks so much for doing it!
Now we need to add those tests to the cake script.


fix: prevent postinstall script from running when installed as dependency using pnpm

The postinstall script runs patch-package even when the package is installed as a dependency, causing installation failures for users who don't have patch-package installed in their projects. This issue is particularly evident when using pnpm, which handles module isolation more strictly.
Problem:
The postinstall script attempts to run patch-package, which isn't available in user projects, causing installation to fail.  The workaround is for the user to add patch-package as a top-level dev-dependency in their own project.
Solution:
This fix updates patches/patch.js to check if the script is running inside a node_modules directory. If it is, the script assumes it's being installed as a dependency and skips running patch-package.
Compatibility:
Tested with npm, pnpm, and yarn. The fix works across all package managers.
Testing:

Verified that postinstall runs during local development.
Confirmed that installation succeeds without errors in a test project.

Hey @badeball  - I read your proposal in cucumber's repo.  Would you still consider completing this PR to unblock pnpm users?

Would you still consider completing this PR to unblock pnpm users?

This would at the very least have to test the mentioned package managers. Preferably using a differents jobs to test latest version of each, using one of the examples.

I'm also not a fan of the heuristics implemented here. You're removing the possibility of working inside a node_modules directory. While I don't do this myself, I see no reason for why this library should not allow it.

I tried using pnpm to install dependencies of one of the examples (esbuild-cjs) and it runs seemingly fine and without errors. What issue have you observed?


Imports with "assert" keyword are not supported anymore

Sorry, if this is not the correct repo to report the issue. I can move it to a different repo :)
I am using the @iconify-json/tabler package which uses JSON imports with the assert identifier.
The assert identifier has been removed by Node.js in favor of the new ECMA proposal of Import attributes.
It will be nice if we can use the newer syntax :)

Deno also deprecated the assert identifier starting from v1.46 released weeks ago. Hope this project can keep up with the changes :)

Sorry, forgot about this issue. Was away when it was initially posted, saw it on phone, thought that I should test it when I get back... and forgot about it. Thanks for bumping it.

Done.
Currently all packages are being updated. Should take an hour or so to update everything.
Packages with new syntax have a new minor version 1.2.x, so those devs who use Node 16 can continue using 1.1.x with old syntax. I don't think this change is big enough to warrant a new major version.

All packages have been updated

Thanks @cyberalien ❤


YCSB benchmark: update OrientDB implementation

Update YCSB benchmark with last OrientDB release and the usage of autosharded index instead of the dictionary.
https://github.com/brianfrankcooper/YCSB/tree/master/orientdb

fixed long time ago


Initial working implementation of TactilityManager

An (at some point in the last week) confirmed working implementation of amplitude modulation in response to pressure being applied by the user on the grabbable object (sphere). This represents the first - and in principle testable - version of the TactilityManager and the stimulator command interface stack.

I would still like to see a few of the issues resolved before merging into the main branch, which should be considered stable.


[Enhancement] Please consider keeping bendpoints fixed when only one element is being moved

The RelativeBendpoint class causes weighted shifts of bendpoints of a relationship between two elements if only one of them is being dragged. Hence the shifted bendpoints need to be repositioned manually very often. By using a modified derivative of the RelativeBendpoint class that will keep bendpoints fixed in their absolute positions and will update the relative positions to the reference points when only one element is being moved this problem will be omited. The modified class could also contain a reference to the respective IDiagramModelBendpoint and update the startX/Y and endX/Y values automatically as well. I can provide a PoC patch if you are interested in this enhancement.

No a simple change in fact. It is easy to have stick bendpoints (if I remember well there's an option for that, at least we have a branch somewhere with it), but we have to clearly define and agree with the needed behavior (which might change depending on whether or not we use orthogonal anchors).
A good start would be to share here the target behavior, discuss it and agree on it. Once done, code will be a simple task (and I'll contribute with pleasure as I also would like to enhance bendpoints).

I tried to do this once and failed. Problems occurred when one or more connected figures were nested inside of others, As you can see from the commit in the branch below, any changes affect the eCore model, GEF commands, edit policies, and edit parts. Also, if a working solution is found it would need to convert old models to the new format.
For reference:
https://github.com/archimatetool/archi/tree/absolute-bendpoints

My 2 cts on this:

Bendpoints must be relative when the two ends of the relation are being moved
Bendpoints mustbe fixed otherwise, but

some bendpoints (usually the one nearer to the object being moved) should be moved (translated) the same way as the object being moved with some slight adjustments (yet to be defined and agreed) which depend on the anchor algorithm (orthogonal or not).

I should be able to specify this with enough details, but it will take some time (I'm a bit overbooked atm).

Archimatetool enhancement.zip
Hi everybody,
I tried to specified the target behavoir for initial positionning and when only one component is being encreased horizontaly. This is included in the archimate file joined
Let me know if this is helpful
Best regards


Typo in vic_init() prevents snow_band Pfactor from being read

The fix is trivial: change line 843 from
soil_con[i].BandElev[j] = (double) dvar[i];
to
soil_con[i].Pfactor[j] = (double) dvar[i];

closed via #434


Add OpenSim to Tools category

Project URL

Category

Description

Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[ ] Has unit tests, integration tests or UI tests
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 1 link issue, a project collaborator will take care of these, thanks :)

Link issue by awesome_bot

Line
Status
Link

1275
301
https://github.com/hyperoslo/Malibu redirects tohttps://github.com/vadymmarkov/Malibu

  Generated by :no_entry_sign: Danger

Oops, it has no tests! Removing PR.

Hey @alexhunsley, thanks for contributing! Feel free to reopen when you add some tests 👍


app.setHost is not a function

I get this error when trying to start the server

I also get this error when trying to run the server.js script. This sample is kind of outdated also...

Using app.swagger.api.host = undefined; instead of the app.setHost worked for me

@PierrickI3 thanks, worked like a charm.

@PierrickI3 thanks, worked like a charm.


Jack up Tests for Logic Parser package

Codecov Report

Merging #204 (00d9933) into master (986d963) will increase coverage by 1.17%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #204      +/-   ##
============================================
+ Coverage     77.39%   78.56%   +1.17%
- Complexity      809      823      +14
============================================
  Files            99       99
  Lines          2181     2179       -2
  Branches        310      309       -1
============================================
+ Hits           1688     1712      +24
+ Misses          382      365      -17
+ Partials        111      102       -9

Impacted Files
Coverage Δ

.../address/logic/parser/AddToOrderCommandParser.java
100.00% <100.00%> (+30.00%)
:arrow_up:

...eedu/address/logic/parser/RemoveCommandParser.java
100.00% <100.00%> (+30.00%)
:arrow_up:

...eedu/address/logic/parser/DeleteCommandParser.java
80.00% <0.00%> (+6.66%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
97.91% <0.00%> (+8.33%)
:arrow_up:

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <0.00%> (+9.52%)
:arrow_up:

.../seedu/address/logic/parser/HelpCommandParser.java
100.00% <0.00%> (+10.52%)
:arrow_up:

...rc/main/java/seedu/address/logic/LogicManager.java
100.00% <0.00%> (+22.72%)
:arrow_up:

Continue to review full report at Codecov.

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


Simplified formatting of the notebooks to ensure correct rendering of documentation

Fancy HTML features completely messed up Sphinx documentation rendering, so I've decided to change all notebooks to markdown-only format.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #19 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #19   +/-   ##
=======================================
  Coverage     81.6%   81.6%
=======================================
  Files           59      59
  Lines         1392    1392
  Branches       138     138
=======================================
  Hits          1136    1136
  Misses         226     226
  Partials        30      30

Continue to review full report at Codecov.

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