TouchableOpacity around a card?

"react-native": "0.41.2"
"native-base": "2.0.12"
"react": "15.4.2"

Hi, so I am just wondering the best approach to make the whole card itself clickable, to navigate you to other pages:
I just figured I could put a TouchableOpacity around the card itself:
<TouchableOpacity onPress={() => goToCategoryDetails(category.handle, category.id)}>
      <Card>
        <CardItem cardBody>
          <Image defaultSource={require('../../img/sandbox-loading.png')} style= {{ flexGrow: 1, resizeMethod: 'resize', height: 200 }} source={{uri: category.image.src}}>
            <View style={styles.titleBGMid}>
              {
                category.title.length > 30 ?
                  <Text numberOfLines={3} ellipsizeMode={'tail'}  style={styles.longTitle}>{category.title.toUpperCase()}</Text> :
                  <Text numberOfLines={3} ellipsizeMode={'tail'}  style={styles.shortTitle}>{category.title.toUpperCase()}</Text>
              }
            </View>
          </Image>
        </CardItem>
      </Card>
    </TouchableOpacity>

But that doesn't seem to have the response I was expecting. I also tried wrapping it around the native-base button, and it just seems to get no response. I put a console log inside of the onPress function, to see if it was at least hitting that function ... but the console log wasn't going off either.
Is there another way I'm suppose to do this? Any suggestions appreciated!
Thanks!

You can just add onPress to your CardItem

Try adding pointerEvents="none" to the card component.
<TouchableOpacity onPress={() => goToCategoryDetails(category.handle, category.id)}>
      <Card pointerEvents="none">
        <CardItem cardBody>
          <Image defaultSource={require('../../img/sandbox-loading.png')} style= {{ flexGrow: 1, resizeMethod: 'resize', height: 200 }} source={{uri: category.image.src}}>
            <View style={styles.titleBGMid}>
              {
                category.title.length > 30 ?
                  <Text numberOfLines={3} ellipsizeMode={'tail'}  style={styles.longTitle}>{category.title.toUpperCase()}</Text> :
                  <Text numberOfLines={3} ellipsizeMode={'tail'}  style={styles.shortTitle}>{category.title.toUpperCase()}</Text>
              }
            </View>
          </Image>
        </CardItem>
      </Card>
    </TouchableOpacity>

Ah yes, @sankhadeeproy007 ... adding on press to card item works out great! Thanks guys ... appreciate the help!

Happy to help!


fix: 🐛 ensure the dataset parameter is provided in endpoints

All the endpoints require the dataset parameter, at least, for the moment. The auth_check and the get_cache_entry_from_steps methods require dataset to be a string (None is not supported) and we have to check before that it's the case.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


How many GPUs are required for this work?

It depends on your batch_size, one GPU per video. You can use 8-16 GPUs.


Power BI: Colors for more than 5 categories

As a user a want to have available a suggestions of colors for when I have more than 5 categories so that I can be consistent and not choose my own.
As a user a want to have more colors added to the json file, so that when I have a chart with many categories, than brand colors are automatically used and not some random Power BI colors.  (currently only the 8 in the color picker are in the json)
Today we have suggested colors up to 5 categories. Should have examples here. Many charts can be down differently if there are so many categories/bars/lines to be displayed that need to have separate colors. Might be better with separating into smaller charts (small multiples/trellis) for example.

This is a duplicate of #2475 . Closing this issue.


Unable to install PowerToys

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Duplicate of #8761

Sorry - I am new here - and I dont understand your comments...#


Jail refactor

[x] Until jailedbit migrated, remove jailed validator from universe
[x] Creates a JailBit on every validator account
[x] Un-jailing can be done by validator after mining a threshold of blocks
[x] Creates a MinerList, separate from ValidatorList needed for resetting counters if lists diverge.
[x] FullnodeState is used for payment and uses MinerState for proof count, removing redundancy
[x] ValidatorUniverse VM can remove a validator
[x] ValidatorUniverse Validator can remove self from list
[x] ValidatorUniverse: a validator can add self to list, as long as has been mining above a threshold in epoch. https://github.com/OLSF/libra/issues/349
[x] Fix bug with fullnode able to increment count https://github.com/OLSF/libra/issues/337
[x] transaction for removing self from validator set
[x] transaction, rejoin for unjailing and adding to validator set
[x] Test for onboarding happy path

Will not do this in this update:

Reconfigure checks JailBit for eligibility in validator set

First need to upgrade to migrate state, so all accounts have JailedBit, before enforcing it in reconfigure.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: lpgeiger:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@sm86
I think these are all great ideas. And I agree that these need to be revisited now before merging. Also can you take a look at the tests to make sure they are passing as you expect?

@lpgeiger please confirm if we have CLI that allows users to add themselves to validator set, after mining threshold proofs?

@lpgeiger please confirm if we have CLI that allows users to add themselves to validator set, after mining threshold proofs?

This should be done automatically on reaching the mining threshold. But yes there are tx scripts for unjailing.


swap cx destroy metrics

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message: Fixes the reversed metrics count of cx_destroy_remote_with_active_rq and cx_destroy_local_with_active_rq for mongo_proxy
Additional Description:
Risk Level: Low
Testing: tests were already in place but the expectation were also reversed.
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

thanks @antoniovicente @cpakulski , I opened this PR just to remind myself to pick it up later, didnt expect it to be review immediately, but thanks for reviewing & confirming, I've updated the tests as well, CI seems not very stable so I rerun it twice, this can be reviewed I think, thanks!


WingCell.getLoft() error when using contour coordinates on cells spanning several segments

When using the get_loft method with larger models, an unusual behavior occurred: with a small test case (Beam_Composite_Export_Test_32_cell2.xml) with one segment, the get_loft method works for several cells. In a larger test case with several segments (IEA-10.0-198-RWT_CPACS_contour_coordinates.xml), only the cells on the first segment are created correctly. Is that a bug or can only one segment be used with the contour coordinate definition of the cells?
CPACS.zip
Originally posted by @hardt-fa-flb in https://github.com/DLR-SC/tigl/issues/815#issuecomment-924832487

This looks like a bug, but I am not 100% sure what is going on yet.
Principally, the contour coordinates support cells that span several segments, but the code behind it is quite complicated, because we have to track the topology of all faces across all segments that are potentially part of the cell, together with their parametric ranges in local u-v-coordinates and in global contour coordinates. In consequence, it is very likely that the code contains bugs.
Thanks for the test file. I will check it out.


Any chance for .NET 3.5 target?

For some unknown reason I still have some libraries that target .NET 3.5. I don't want to drop that target just yet. I want to use the Polyfill to simplify my life.
And .NET 3.5 SP1 is going to be supported until at least 2029, according to official Lifecycle policy.
If not - it's ok. I just took my chance.

let me take a look at how much of a pain it will be

ok. i looked into this. i think it is a significant amount of work. if u want to do the work, i will accept the PR and maintain it


Unable to serve cgi

Hi there,
I have an internal server with CGI stuff served by Apache on port 81 and I would like to serve it through the reverse proxy, it seems easy enough as the CGI stuff is working when I access the direct ip:port but as soon as I create the new proxy host, it always end up in 504 Gateway Time-out error (even though it somehow works as it redirect to HTTPS properly, then it goes around in circles and timeout.
Here is the vhost Apache configuration I need to proxy to :
<VirtualHost *:81>
    DocumentRoot "/var/www/html/cgi"

    ErrorLog "/var/log/apache2/cgi_error_log"
    CustomLog "/var/log/apache2/cgi_access_log" combined

        <Directory "/var/www/html/cgi/">
            Options +Indexes +FollowSymLinks -MultiViews +ExecCGI
            AllowOverride All
            Require all granted
        </Directory>

    AddHandler cgi-script .cgi

</VirtualHost>

Is there anything obvious I'm missing ?

Indeed there was something obvious : the reverse proxy is in another VLAN and it does not have access to port 81...


LocalForage 1.4.3 : Firefox 52_0_1 : Private Mode

On call LocalForage 1.4.3 : Firefox 52_0_1 : Only in Private Mode
LocalForage.getItem(KEY)
.then()
.catch()
in console InvalidStateError  from indexeddb.js (line 135)
function _getConnection(dbInfo, upgradeNeeded) {
return new Promise$1(function (resolve, reject) {
if (dbInfo.db) {
if (upgradeNeeded) {
_deferReadiness(dbInfo);
dbInfo.db.close();
} else {
return resolve(dbInfo.db);
}
}
var dbArgs = [dbInfo.name];
if (upgradeNeeded) {
dbArgs.push(dbInfo.version);
}
var openreq = idb.open.apply(idb, dbArgs); // on this line
It's not critical, all seems works as expected only a uncatched error message in a console.

Yes, this is a known issue. Unfortunately offline storage generally does not work well in private browsing mode on many browsers.

Can you try v1.5? I think that we more prevent the error to bubble and
reach the window.

--
Thodoris Greasidis

Yes, I can try v1.5 but not now. Thanks.


Remove experimental flag for F12 Bicep Registry

Flip default behavior to Bicep if source is available.

This was checked in.


test eager loading belongs to nested with same foreign key

This is belongs to #537

This PR's test must fail

SELECT * FROM awesome_people **WHERE author_id=?' contains "**WHERE author_id IN(?,?)


Understand how FreeBMD1 is searched

needed by #12

In addition to analytical evidence (have asked Barrie about this on Slack), could design a survey?

https://docs.google.com/forms/d/1EwLq-kL0ayd2tc0VHjsMU7kBwFZ46GsDKExx85tG60g/edit

Now live.

Let's add this to the FreeBMD homepage too.

@PatReynolds  we will need to create a mock-up for the change we would like on the homepage and the one line copy that will go alongside the link. Task has been added to TRAC (#656).

Currently on hold until it has been reviewed/revised following feedback.

New volunteer to potentially help out with this.

@richpomfret to find the files for Sobia to work on

Sobia is now unable to begin until June. @avielbassalian-freeukgen has begun analysis for field combinations / frequency of use. More can be extracted.

Added @avielbassalian-freeukgen as he has managed to extract some bits from the JSON file provided by Barrie.

INterviews as next steps here (this will become a parent /epic).

I will introduce you (Aviel) to Barrie re: new JSON file.

Do we need a new survey or use the previous survey to add onto the old FreeBMD1 site? @neslihanu and @avielbassalian-freeukgen to take a look.

This could be a good way of collecting info on the search results page and how best to show information.


Templates.json does not work like documented (optionally prefixed with the port)

Bug description
When specifing a port like "20001:8080/tcp" in template.json it will show 20001:8080 at the container side and leaves host empty.
Documentation says:
It can be optionally prefixed with the port that must be mapped on the host in the port: form.
Expected behavior
Container side should only containt 8080 while Host side should contain 20001
Steps to reproduce the issue:
Create a template with
"ports": [
"20001:8080/tcp"
],
in it
Technical details:

Portainer version:
portainer/portainer:latest

Docker version (managed by Portainer):
17.05.0-ce

Platform (windows/linux):
linux

Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
docker run -p 9000:9000 portainer/portainer

Browser:
Chrome

Additional context
While fixing this bug it should be maybe also possible to prefix the bound ip at the same place. For example we could write
"ports": [
"<IP_ADDRESS>:20001:8080/tcp"
],

Hi @egofelix
It was an issue with the documentation, the documentation was published before the release and this feature is only available in Portainer >= 1.19.0.


Water refreezing

This capability would currently be useful for the VIPER REXUS project, for simulating the process of freezing cylindrical ice samples. Right now they have to use a crude approximation to estimate how long the freezing process will take.


Text nodes leave litter across the screen in Chrome

As reported in https://github.com/phetsims/gravity-and-orbits/issues/57:
Here's a picture, but please see the source ticket for more information:

According to referenced issue, this doesn't happen anymore.


Fix version autodetection from git tags

Fixes #14 (provided a new release of this package to PyPI)
#10 dropped the setup_requires option by moving build requirements to pyproject.toml. Sounds fair enough, but #10 also dropped use_scm_version, while the versioning strategy hasn't changed (at least, there's no explicit version option passed to setup()). So it looks like it was an oversight, and this PR adds the option back.

Thanks @RonnyPfannschmidt :-)

When next release?

after changelog

should be out after the pipeline finishes

@RonnyPfannschmidt Can confirm the new 1.1.1 release got our CI pipelines succeeding again! Thanks a ton for jumping in so promptly. :-)

Sorry for my lack of culture. I was beetwen two meetings. Thanks for fast reaction and bugfix.


Advanced Data options is not available for "Recommendation" Scenario.

System Information (please complete the following information):

Windows 10 Enterprise, Version 20H2
ML.Net Model Builder (Preview): <IP_ADDRESS>5505
Microsoft Visual Studio Enterprise 2019: 16.9.3
.Net: 5.0.201

To Reproduce
Steps to reproduce the behavior:

Create new console app with .Net 5.0;
Add model builder by right click on the project;
Click "Recommendation" Scenario and select "Local CPU" environment;
Click "Browser..." to load the file;
Click "Advanced data options..."
See info: Currently, Model Builder only supports limited columns for Recommendation scenario. Does your machine learning model need more columns?

Expected behavior
According to “Advanced Data Test Pass”, can see the three columns: "user column", "item column" and "rating column" on advanced data options dialog.
Screenshots

Additional context
Recommendation: https://testpass.blob.core.windows.net/test-pass-data/recommendation-ratings.csv

Fixed by changed test pass: https://github.com/dotnet/machinelearning-tools/blob/main/docs/tests/AdvancedDataTestPass.md


[REQUEST] Distributed data parallel training

Currently, Betty only supports torch.nn.DataParallel. Compared to torch.nn.parallel.DistributedDataParallel, torch.nn.DataParallel is much slower even in the single-machine multi-gpu settings. Therefore, we need to replace torch.nn.DataParallel with torch.nn.parallel.DistributedDataParallel for better training speed and the multi-machine multi-gpu support.

can't agree more!

I understand the difficulty of implementing this feature, but in my opinion, this is the single most important feature that betty should have while similar repos like https://github.com/facebookresearch/higher doesn't. To make GML/MLO more impactful, it should be applied to large scale problems in the wild. Distributed training is a must have for real applications.
The ad-hoc option looks good to me, can't wait to try it 👍

Hello @gongbudaizhe,
I apologize for the late reply! I finally implemented the distributed training feature for the multi-node/multi-gpu setting.
To try this feature, you should install the nightly version by directly cloning the most recent commit as:
git clone https://github.com/leopard-ai/betty.git
pip install betty

In detail, you can enable distributed training by 1) setting distributed=True in EngineConfig as:
engine_config=EngineConfig(distributed=True, ...)

and 2) launching the training script with torch.distributed.launch on every node as standard PyTorch distributed training.
In the future, we plan to simplify the launching procedure as Microsoft's DeepSpeed or HuggingFace's Accelerate. To do this, we need to write a custom launcher code. If you are willing to contribute to this, that is very welcome!
I am sorry again for the delay, and let me know if you have any questions!
Best,
Sang


Omnichat Module docs

@mochaul Done.

Then, add a new page: http://docs.kata.ai/module/omnichat/introduction

This can only be put on /modules/module-list/omnichat/introduction, to follow the document structure.

Thank you @resir014 ! /modules/module-list/omnichat/introduction would be fine 👍

@mochaul It's been done. Please review again.

Nice, thank you @resir014 ! Can you add this also please:
add a new page:
http://docs.kata.ai/module/omnichat/introduction
that contains this markdown:
https://kata.quip.com/uihwAWATDmXA/Introduction-to-Kata-Omnichat
Thank you @resir014

It's here:
https://deploy-preview-93--kata-platform-docs.netlify.com/modules/module-list/omnichat/introduction/
This is still not merged to production, so it's not in the docs.kata.ai url.

Thank you so much @resir014 ! All good.


Can you please add the latest version of Ngrok extension for Visual Studio

The current version of Ngrok Visual Studio extension is 0.9.14.

When I start the extension from the tools, I get this pop-up:
, which sometimes fails.
When will the latest version be shipped?

It fails everytime now. Got the same issue, and it blocks me to proccess forward.

I was able to get this working by installing ngrok via Chocolatey: choco install ngrok


patch/consistent-naming-ingressClassName

Which problem is this PR solving?
The ingress-beta and ingress-grpc Helm templates use one naming convention for an Ingress Class Name: ingress.className, whereas the ingress template uses a different convention: ingressClassName.
The problem solved is to adopt a consistent naming convention. I've opted to the latter one, ingressClassName, as it's used in Kubernetes example docs.
Otherwise, users have unexpected inconsistent behavior when trying to specify the Ingress Class Name for the different ingresses. And, populating the grpcIngress's Ingress Class Name currently would have to happen within the ingress: block in the values, which is unintuitive.

Closes #N/A

Short description of the changes
Changes the ingress-beta.yaml and ingress-grpc.yaml ingress.className to ingressClassName to be consistent with ingress.yaml and Kubernetes convention.
How to verify that this has the expected result
Specify an ingressClassName in a values.yaml file and dry-run. No tests because it's a minimal change that seems to fit the "small and obvious" acceptance criteria laid out in the lifecycle-and-practices doc.

@zenon-was-here Can you also add className with an empty string value to both the ingress and grpcIngress sections in the values.yaml

@TylerHelmuth , understood, thanks for the feedback

@zenon-was-here thanks again for working on this, I am going to integrate your changes into an upcoming v3.0.0 version of the chart.


No way to detect current port number for remote.

"There might be a firewall."  :-)

I changed this bug.  It is possible to set the port in the project, but seemingly no way to retrieve that value from within BP, which I need for another bug... :-)

Bleh.  Clearly I need my eyes checked.  :-D

huh....wut?


ci: Fix for setting test_tmpdir in non-RBE

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

pushing through to fix CI caching


[cga] adding error checks around all kernels calls

@ahehn-nv wanted to close the loop on this one. Looks like that CUDA 10.0 genfe error shows up when using c++14 standard (which was being required for constexpr usage in the kernels). In this PR I've changed it from constexpr to const. Let me know if this is okay, or if it'll cause perf issues down the line (or if this change will limit future work).
Once we standardize to CUDA 10.1 we will be okay, but for now CI is limited to CUDA 10.0

It could have performance implications, depending on how smart the compiler is. I haven't checked...
In any case, I would not just replace them by const, because we will forget about it and never change it back that way. I would use a macro CGA_CONSTEXPR or something, which you define depending on the NVCC version. How does that sound to you?

If this can wait one day, I'd like to give it another thought.

@ahehn-nv yeah can definitely wait. i like the nvcc version based ifdef, but wasn't sure if that could be queried.

Yes, I guess the best solution is a macro definition depending on the compiler version. You can check the version via __CUDACC_VER_MAJOR__ + __CUDACC_VER_MINOR__ or __CUDACC_VER_BUILD__.

@ahehn-nv since the WAR for CUDA 10.0 was checked in, this works now

@ahehn-nv if all looks good, can you approve?


Updated two PowerShell sections

Two PowerShell scripts were deploying Windows-based clusters. The resource groups were called mylinux. I have changed to mycluster to match the rest of the document and to take confusion away between windows and Linux clusters.

@PixelRobots : Thanks for your contribution! The author, @aljo-microsoft, has been notified to review your proposed change.

@PixelRobots this doesn't make sense as a complete experience.
You updated PS, but didn't update CLI; the experience is identical and calls the same API's, but your change is proposing they be different?
I still see "myLinux"

@aljo-microsoft The cli versions setup a Linux cluster. The PowerShell versions setup a windows cluster. So thats the reason I only updated the PowerShell ones. Hope that makes sense.


Spike: apply recipe command

Adds capability to apply a custom OpenRewrite recipe found in a public GitHub Repo, like here
on a given application, like this.
In this first version, JitPack is used to create a jar from the given repo.
The jar os downloaded and loaded into a separate ClassLoader.
The code implements a known message (e.g. through an interface) and the code gets executed and creates the recipe instance. Spring-cli then takes the recipe and applies it to the given project.
Steps:

Clone git clone https://github.com/fabapp2/spring-petclinic-detached given
Build spring-cli
Call apply recipe https://github.com/fabapp2/spring-cli-command-playground given

This PR contains the changes from #154

Nice spike!  A couple questions

How can we run the existing OR recipes?

For example https://docs.openrewrite.org/recipes/java/testing/junit5/junit4to5migration which points to this recipe 'source' - https://github.com/openrewrite/rewrite-testing-frameworks/blob/main/src/main/resources/META-INF/rewrite/junit5.yml

What about running a recipe that is already packaged up as a jar and hosted in artifactory?  We have some nice classes in SCDF to pull jars from repos.

Closing this, will provide a new draft


Kernel Gateway endpoints with multiple notebooks

What is the configuration necessary to enable multiple different notebooks to all have endpoints when using Kernel Gateway?

Right now, it's not supported. https://github.com/jupyter/kernel_gateway/issues/225 is tracking the enhancement. Happy to take a pull request for it!


Button position is absolute and not relative to the content in the card

Merge conflicts not fixed as earlier

Merge conflicts not fixed as earlier


Please add a License

And a LICENSE file

Added MIT license as this was also used in the original MD4C source.

Thank you! I think this ticket can be closed.


Recommend

Recommend to set the default value of two disable cooldown options to false in order to prevent any user of this mod from accidentally banned from some servers.

Why would it be banable, item cool downs are shown in the slots already. It doesn't add any new info

Never mind.


AppDynamics Pro version compatibility?

Is there a map of what AppDynamicsREST version is compatible with what AppDynamics controller version? I ran sample.py from AppDynamicsREST 0.4.5 against an on-premise AppDynamics controller version <IP_ADDRESS> build 11, and got this error:
python sample.py -c http://appdhost:8090/ -u user -p *****
APP1 0 0 0 0
APP2 2 1 0 0.0
APP3 0 0 0 0
Traceback (most recent call last):
  File "sample.py", line 19, in <module>
	art = metric_data.by_leaf_name(c.AVERAGE_RESPONSE_TIME).first_value()
  File "build\bdist.win32\egg\appd\model\metric_data.py", line 57, in first_value
  File "build\bdist.win32\egg\appd\model\metric_data.py", line 45, in __getitem__
IndexError: list index out of range

On second thought this isn't a version problem; I guess the app after APP3 didn't have a .first_value(). I suppose sample.py should check for that before trying to access empty list.


Add ZT Systems to LF-Edge lanscape

Add logo and metadata for ZT Systems

The logo we have is not high enough res to recreate an svg. Reaching out to the company for another.


[master] Bump Mesos to nightly master 2a1c5d5

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/270a3dce490d5b334f9a0011ea416ffc42e187e4...2a1c5d518b43be21673b2cfdf72fc2e60658a826

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review


PD e2e tests are flakey

e.g. for "PD should schedule a pod w/ a RW PD, remove it, then schedule it on another host"
we intermittently see failures like:
"Error deleting PD".
Presumably it needs a retry.

@cjcullen actually spent a while on this test and the underlying controller, and it was running rock solid for quite a while. If's flaky, it's probably because something broke. :/

For what it's worth, we're no longer seeing this specific error on Jenkins.  Not sure exactly what fixed it though.

We're seeing this again intermittently in Jenkins, e.g:
http://kubekins.dls.corp.google.com/job/kubernetes-e2e-gce/4704/
http://kubekins.dls.corp.google.com/job/kubernetes-e2e-gce/4705/
PD should schedule a pod w/ a RW PD, remove it, then schedule it on another host
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/pd.go:113
Error creating PD
Expected error:
<*exec.ExitError | 0xc2081680f0>: {
ProcessState: {
pid: 13567,
status: 256,
rusage: {
Utime: {Sec: 0, Usec: 428000},
Stime: {Sec: 0, Usec: 80000},
Maxrss: 46240,
Ixrss: 0,
Idrss: 0,
Isrss: 0,
Minflt: 23222,
Majflt: 0,
Nswap: 0,
Inblock: 0,
Oublock: 40,
Msgsnd: 0,
Msgrcv: 0,
Nsignals: 0,
Nvcsw: 82,
Nivcsw: 30,
},
},
}
exit status 1
not to have occurred
===============
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/pd.go:176
Error creating PD
Expected error:
<*exec.ExitError | 0xc208294468>: {
ProcessState: {
pid: 30582,
status: 256,
rusage: {
Utime: {Sec: 0, Usec: 444000},
Stime: {Sec: 0, Usec: 60000},
Maxrss: 46172,
Ixrss: 0,
Idrss: 0,
Isrss: 0,
Minflt: 23206,
Majflt: 0,
Nswap: 0,
Inblock: 0,
Oublock: 40,
Msgsnd: 0,
Msgrcv: 0,
Nsignals: 0,
Nvcsw: 81,
Nivcsw: 30,
},
},
}
exit status 1
not to have occurred
====================
PD should schedule a pod w/ a RW PD, remove it, then schedule it on another host
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/pd.go:113
Error creating PD
Expected error:
<*exec.ExitError | 0xc2082dc3b8>: {
ProcessState: {
pid: 21005,
status: 256,
rusage: {
Utime: {Sec: 0, Usec: 472000},
Stime: {Sec: 0, Usec: 44000},
Maxrss: 46108,
Ixrss: 0,
Idrss: 0,
Isrss: 0,
Minflt: 23206,
Majflt: 0,
Nswap: 0,
Inblock: 0,
Oublock: 40,
Msgsnd: 0,
Msgrcv: 0,
Nsignals: 0,
Nvcsw: 85,
Nivcsw: 26,
},
},
}
exit status 1
not to have occurred
=================
PD should schedule a pod w/ a readonly PD on two hosts, then remove both.
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/pd.go:176
Error creating PD
Expected error:
<*exec.ExitError | 0xc208038040>: {
ProcessState: {
pid: 27748,
status: 256,
rusage: {
Utime: {Sec: 0, Usec: 460000},
Stime: {Sec: 0, Usec: 60000},
Maxrss: 46252,
Ixrss: 0,
Idrss: 0,
Isrss: 0,
Minflt: 23200,
Majflt: 0,
Nswap: 0,
Inblock: 0,
Oublock: 40,
Msgsnd: 0,
Msgrcv: 0,
Nsignals: 0,
Nvcsw: 85,
Nivcsw: 30,
},
},
}
exit status 1
not to have occurred

Closing as a dupe of #7972 where it is being actively debugged.


Make dart-reddit Dart2 compatible

i literally just did the exact same thing with messier git commits -_-

Thanks!

pushed v0.6.0


Support for Toggl's reports api

Apologies for letting this linger for 21 months. I was dealing with a heavy workload and the aftermath of a car crash in Nov 2013. I'll try to make time to review this in the near future. Do you have any other changes to contribute?
I'm hoping to package this up into a gem and replace test.rb and atest.rb with proper rspec tests.

Thanks @itaymendel for your work on the Reports API. I listed you as a contributor on my README.
When I added support I started from scratch and added tests as I went. I can't remember what it was but when I first looked at your PR I thought I saw a better design for something that would match the existing code. If I had known then that I wouldn't have the time till now to take care of it I would have merged your PR and moved on with life. I no longer use Toggl or write Ruby code and this project was put on the back burner.
My hope is that this library is nearly complete now and any incoming changes will be fairly minor. If you have any changes to recommend, I will be more responsive.
Cheers!


Build nightly-build on avocano-cart: FAILURE

Cloud Build encountered an error:

Project: avocano-cart
Trigger: nightly-build
Commit: 1bc0fa1 on branch main

View Logs
I'm an automation.  Googlers: go/avocano-admin-notifiers

        # Expect failure
>       expect(page.locator(".dialogWrapper")).to_have_text(re.compile("Oh no!"))
E       AssertionError: Locator expected to have text 're.compile('Oh no!')'
E       Actual value: None
E       Call log:
E       LocatorAssertions.to_have_text with timeout 5000ms
E       waiting for locator(".dialogWrapper")
E       waiting for locator(".dialogWrapper")

Nightly succeeded the next run (logs)


SQL-based ingestion cannot generate daily segments taking time zone into account.

Hi all,
The SQL-based ingest by MSQ is a great feature. I have tried it and in my environment it has roughly doubled the data ingestion speed.
However, it seems that all segment granularity and specific intervals, which are possible with Native Batch Ingestion, are not possible with MSQ.
Thus, it is not possible to generate daily segments that take time zones into account.
The following is an example of specifying with Navite Batch Ingestion.
{
  "spec": {
    "dataSchema": {
      "granularitySpec": {
        "segmentGranularity": "all",
        "intervals": [
          "2000-01-01T15:00/2000-01-02T15:00"
        ]
      }
    }
  }
}

Confirmed on 25.0.0-SNAPSHOT and 24.0.0。
It would be useful if MSQ could generate daily segments that take time zones into account.
Considering the overwrite, in my timezone, the only partition option in MSQ is HOUR.

It is great to hear that you are having positive results with the new MSQ based ingestion 😃
Based on the granularitySpec that you mentioned in the issue, does filtering the data and then ingesting it like the following query work?
INSERT INTO
...
WHERE WHERE __time >= TIMESTAMP '2000-01-01 15:00:00' AND __time < TIMESTAMP '2000-01-02 15:00:00'
PARTITIONED BY ALL

Alternatively, what if we supported something like PARTITIONED BY TIME_FLOOR('P1D', '-09:00'), or perhaps PARTITIONED BY TIME_FLOOR('P1D', 'America/Adak')?

@gianm Thank you for your response.
Your idea is great.
And it would be even more useful if we could apply the "sqlTimeZone" context.
Here is the first query and context I tried.
INSERT INTO inline_data
WITH ext AS (SELECT *
FROM TABLE(
  EXTERN(
    '{"type": "inline", "data":"{\"time\":\"2000-01-01T15:00:00\"}"}',
    '{"type": "json"}',
    '[{"name": "time", "type": "string"}]'
  )
))
SELECT TIME_PARSE("time") AS __time
FROM ext
PARTITIONED BY DAY

{
  "sqlTimeZone":"+09:00"
}

The above result was 2000-01-02T00:00:00.000Z/2000-01-03T00:00:00.000Z, but I expected 2000-01-01T15:00:00.000Z/2000-01-02T15:00:00.000Z or 2000-01-02T00:00:00.000+09:00/2000-01-03T00:00:00.000+09:00.

Any updates on this issue?
I face this problem too and want this use case to be real.
Many thanks


The system cannot find the file specified

I'm getting this error and I'm already CD into the Terraform project folder, where Terraform init has already been run successfully.

Tried the above in a brand new installation of Python in a fresh VM and it's the same result.
Any ideas how to fix it?

Sorry I have not encountered this before @chaoscreater but have you tried the Docker image? I haven't updated it in a while but last I remember it was working


[feature] Send diff within the notification

I would like to monitor changes, but want to notify people that should not have access to my install or are outside my network, but they need to see the diff. Obviously I don't want to open port to everyone, so sending the change inside the mail (that is what I use for notification) would be a nice option.
Thanks for considering this.

The diff is generated in JS, needs some other way to build it

+1 for this feature!
And many thanks for this good work!

And we're in!!!!!


Streamline the release process

Before going to prod, we need to make sure we can handle emergencies quickly and effectively. This means, sometimes, being able to solve the issue and perform a hotfix release within a few hours.
Currently, our release process is manual and relies on a few individuals' knowledge on how we work, personal publish keys (!) and has no standard environment to be performed. All of this means it's really cumbersome and error prone and the bus factor is high.
Thus, we have the following requirements for the release process:

It needs to be automatic, preferably a single click process
It needs to rely only on some kind of institutional account
It needs to be well documented
It needs to be reproducible
It needs to be really fast
It needs to allow for development releases for testing (say nightlies, betas, release candidates...)

A suggested approach is taking advantage of GitHub Actions for this. We would need:

One pipeline enforcing conventional commits
One pipeline keeping around one or more release PRs
One pipeline that takes care of publishing the crates to crates.io and creating appropriate tags, branches and GitHub releases

Pipeline 1. is a way to automate changelog updates. It reduces issues with conflicts on the CHANGELOG.md file blocking pipelines. This is key both for quick response and pipeline 2. to actually be at all possible.
Pipeline 2. makes sure the "single click approach" is achievable. It should run on merge to main and release/* branches, updating the changelog based on conventional commits and cargo semver-checks to detect and report breakage. The generated PRs should be per-tracked branch (hotfixes shouldn't carry all of the changes in main) and properly reviewed to make sure the changelog and versions are correct, tests don't break, etc.
Pipeline 3. centralizes the release authority. It can use some mechanism like the conventional commit message used by the PR generated by pipeline 2. or being ran on tag pushes to know whether it needs to run. With this workflow we can use a single GitHub secret to keep the publish key.
It should create tags of the form vx.y.z[.p][-d] following semantic version conventions with the extensions .p indicating a that it's a the p-th hotfix and -d that it's a development release of d nature, branches named release/vx.y.z for the release, where the tg vx.y.z[-p] always point to the tip of this release branch. From the tag, it needs to publish to crates.io and create an associated GitHub release with the release notes in the description and, optionally, pre-built binaries of cairo-vm-cli to allow use of cargo-binstall for quicker install.
We need some work planning the exact steps and to check whether the ordering is correct, but essentially these workflows should enable releasing quickly from specific branches in a clean and reproducible way.

The documentation about the release process is available in the RELEASE.md file


Add option to use no shizuku

Just asking but can you make an option to make the app not use shizuku to just install apps because it's kinda bad for me to enable shizuku everytime when I want to install an app. So maybe make a tab for settings and make a toggle to not use shizuku and remove any options to install that needs shizuku, I mean like if I don't wanna use shizuku then the options that need shizuku to work will be dimmed which means that it won't work but if I click on the option that needs shizuku, a popup will appear and say that it needs shizuku for it to work. So maybe a yes or a no? If you add it then this will be my main app installer ¯\_(ツ)_/¯

All of the options require Shizuku as implemented. The point of this app is to expose more advanced install flags without having to use ADB shell, not necessarily to be a general purpose APK installer; there are a lot of other options for that.

Well it's not a terrible idea, given that the system/package installer doesn't seem know what to do with bundles/splits archived in any of the popular file extensions they are distributed as, the best case is - it will open them as what they are - archives and they can't natively be installed (AFAIK), technically if Zacharee wanted IWO could be made to just and only do that - install bundles/splits and maybe setting an install location (internal/external) only on A.9 and below (as pointless as it might seem), other than that every other option can be disabled/hidden/greyed-out because they can't work without an authorizer, being Shizuku/Sui/Root. I use a file manager that can install bundles without any options or picking which split to install from a bundle archive, which is bad, but at least I have options like SAI or IWO if I have an authorizer (Shizuku/Root), plenty of session installers out there, but nothing that advanced as IWO (short of InstallWithOptions).

Well it's not a terrible idea, given that the system/package installer or rather most file managers, especially the build-in/system ones don't seem know what to do with bundles/splits archived in any of the popular file extensions they are distributed as, the best case is - they will open them as what they are - archives and they can't natively be installed (AFAIK), technically if Zacharee wanted IWO could be made to just and only do that - install bundles/splits and maybe setting an install location (internal/external) only on A.9 and below (as pointless as it might seem), other than that every other option can be disabled/hidden/greyed-out because they can't work without an authorizer, being Shizuku/Root (and maybe even Dhizuku - uses DeviceOwner - aka. elevated Device Admin with a command, which doesn't have to be executed after a reboot). I use a file manager that can install bundles without any options or picking which split to install from a bundle archive, which is bad, but at least I have options like SAI or IWO if I have an authorizer (Shizuku/Root), plenty of session installers out there, but nothing that advanced as IWO (short of InstallWithOptions).

I just like the UI and looks of it cuz I get bored of the old UI and when I keep seeing it everytime I install an app plus this app is more reliable for more tasks to not waste my time ya know what I mean ;)


Add recommended k8s labels

This PR adds recommended k8s labels on resources.
Implementation for proposal #2497
Config: added labels and updated selectors for all resources in /config
Doc: added /docs/developers/resources-labelling.md
NOTE: this will introduce a breaking change when trying to update deployments as label selectors have changed.

/cc @vdemeester

/test tekton-pipeline-unit-tests

/test tekton-pipeline-unit-tests

/hold
I feel like there are probably some superfluous labels

/hold cancel
I removed app.kubernetes.io/name and app.kubernetes.io/version when they had no value.
@vdemeester how about using app.kubernetes.io/version everywhere instead of pipeline.tekton.dev/release, triggers.tekton.dev/release, dashboard.tekton.dev/release, etc ?

/test tekton-pipeline-unit-tests

/hold cancel
I removed app.kubernetes.io/name and app.kubernetes.io/version when they had no value.
@vdemeester how about using app.kubernetes.io/version everywhere instead of pipeline.tekton.dev/release, triggers.tekton.dev/release, dashboard.tekton.dev/release, etc ?

yeah we could, but we will need to keep those (at least for a while) there as tools depend on them.

I know the dashboard depends on it, i am adding support for app.kubernetes.io/veersion there.
Any other tool ?

I know the dashboard depends on it, i am adding support for app.kubernetes.io/veersion there.
Any other tool ?

tkn
OpenShift Pipeline
most likely a bunch of other tools that we do not control :stuck_out_tongue_closed_eyes:

We should add the new one, mark the old one as "deprecated" (in release note) and decide on a time where it gets removed (if we want to remove it :stuck_out_tongue: )

/meow

@vdemeester do you want to include changes from https://github.com/tektoncd/pipeline/pull/2508 in this PR ?

@vdemeester do you want to include changes from #2508 in this PR ?

oh yeah we can :+1:

Done, i took the objectSelector approach, hope it's ok for you.

/kind feature


Add lazyloading pattern for images

https://sidewind.js.org/sources/#x-intersect + <noscript> to cover browsers without JS. This should go into a component.

I found out it's possible to implement this at the user side so it's not a generator concern.


Implement direct deployment in Argo with api from the experimant manager

Currently, a user needs to open the Argo WF engine manually.
Add a button to submit the generated WF via the Arrgo API

Comming soon


refactor(interactive): flex storage with mmap array

refactor flex storage with mmap array

Related issue number

Fixes

can we add an interactive doc to introduce how the mmap can help offloading graph data so that we can support larger graph.


Unable to jump to async step definitions

Have an async step definition, build the project and try to navigate from step to the step definition.
It shows an error
Unable to jump to the step definition. No source location detected.

(Extracted from #21 by @z41)

This has been fixed in v1.1.2. //cc @z41


Terminal log output generates unnecessary log error TypeError: expected string or bytes-like object

The new log terminal output code added in #112 is causing unnecessary log errors on anything but stdout messages.
The error is not able to serialize: expected string or bytes-like object with setup and disconnect messages like ['setup', {}] and ['disconnect', 1]
Traceback (most recent call last):
  File ".../handlers.py", line 62, in send_json_message
    if pattern.search(content[1]):
TypeError: expected string or bytes-like object

I assume we'd really want a check like:

def send_json_message(self, content):
        json_msg = json.dumps(content)
        pattern = re.compile(r'^(\w|\d)+')
        try:
            if isinstance(content[1], (str, bytes)) and pattern.search(content[1]):
                self.log_terminal_output(f'STDOUT: {content[1]}')
        except TypeError as e:
            self._logger.error(f'not able to serialize: {e}')
        self.write_message(json_msg)

This would avoid the type error for well-expected content values.
As a side note, it would be ideal to get LOG_TERMINAL_OUTPUT env variable in the constructor, then check it before building potentially big f-strings from STDOUT.  I haven't confirmed this really causes any significant performance issues, so only wanted to note this as a potential.

Both suggestions sound reasonable to me, mind making a PR?


add Most Recently Used (MRU) list of files

It would be good to have a list of the last opened files under the file menu.

I played around a little and came with this API.
Do you have any thougths about it?
namespace AnnoDesigner.Core.Models
{
    public class RecentFile
    {
        public RecentFile(string pathToUse, DateTime lastUsedToUse)
        {
            Path = pathToUse;
            LastUsed = lastUsedToUse;
        }

        /// <summary>
        /// The path to the file.
        /// </summary>
        public string Path { get; }

        /// <summary>
        /// The last time the file was loaded/used.
        /// </summary>
        public DateTime LastUsed { get; set; }
    }
}

namespace AnnoDesigner.Core.Models
{
    public interface IRecentFilesHelper
    {
        /*
         *  void Load();
         *  void Save();
         *  are called implicitly in ctor, AddFile and RemoveFile
         *
         *  ctor has a pahToSettingsFile, a IRecentFilesSerializer and a IFileSystem as parameter
         */

        /// <summary>
        /// Occurs when the <see cref="RecentFiles"/> property has been updated.
        /// </summary>
        event EventHandler<EventArgs> Updated;

        /// <summary>
        /// Gets the recently used files.
        /// </summary>
        /// <value>The files.</value>
        List<RecentFile> RecentFiles { get; }

        /// <summary>
        /// Gets or sets the maximum item count.
        /// <para />
        /// The default value is <c>10</c>.
        /// </summary>
        /// <value>The maximum item count.</value>
        int MaximumItemCount { get; set; }

        /// <summary>
        /// Adds the file to the top of the list of recently used files.
        /// </summary>
        /// <param name="fileToAdd">The file.</param>
        /// <exception cref="ArgumentNullException">The <paramref name="fileToAdd"/> is <c>null</c>.</exception>
        void AddFile(RecentFile fileToAdd);

        /// <summary>
        /// Removes the file from the list of recently used files.
        /// </summary>
        /// <param name="fileToRemove">The file.</param>
        /// <exception cref="ArgumentNullException">The <paramref name="fileToRemove"/> is <c>null</c>.</exception>
        void RemoveFile(RecentFile fileToRemove);
    }
}

namespace AnnoDesigner.Core.Models
{
    public interface IRecentFilesSerializer
    {
        /*
         * Implementations could serialize to XML or JSON or SQLite or ...
         * I would prefer JSON in the "AppData\Local\AnnoDesigner" directory. The app-settings are already there.
         */

        /// <summary>
        /// Serializes the list of recently used files to a file.
        /// </summary>
        /// <param name="path">The path to the file with the recently used files.</param>
        /// <param name="recentFiles">The list of recently used files.</param>
        void Serialize(string path, List<RecentFile> recentFiles);

        /// <summary>
        /// Loads a list of recently used files from a file.
        /// </summary>
        /// <param name="path">The path to the file with the recently used files.</param>
        /// <returns>The list of recently used files.</returns>
        List<RecentFile> Deserialize(string path);
    }
}

I probably would change a couple of things -

I think I would rename LastUsed to DateModified, so that it is synonymous with terminology used in other file systems.
I would rename IRecentFilesHelper to IRecentFilesManager (or something similar) - any class that implements this interface holds the state of the recent files list.
We can just serialize the list directly into user settings - then we don't need to worry about dealing with a path that resolves close to where the settings are stored, and we wouldn't need the IRecentFilesSerializer interface/implementation.

I think the name LastUsed is more appropriate because it reflects when the file was last used. When I read DateModified I immediately think it is the time when the file was last modified. That is not the case here.
Yes it is more a manager than a helper class.
If you want to serialize it to the user settings, which also was my first thought, then the implementation needs to be in the main app.
If we abstract the serialization away, the implementation could be in the .Core.dll and be used in other applications.
At least I do not know at the moment how to get a reference to the Settings of an application in a library.

Maybe LastAccessed?
I see what you mean about abstraction - seeing as the MRU could have a separate window, could we create a view model for it that takes a reference to the an IAppSettings object in its constructor?

LastAccessed sounds to technic. What about LastOpened?
It seems there is a better way to abstract the serialization.
The interface needs to reference to a filePath. It is the job of the implementation to figure it out where to Load and Save the list.
So there could be an implementation which uses the application/user settings and the API is still useable in other applications.
So my suggestion would be:

namespace AnnoDesigner.Core.Models
{
    public class RecentFile
    {
        public RecentFile(string pathToUse, DateTime lastOpenedToUse)
        {
            Path = pathToUse;
            LastOpened = lastOpenedToUse;
        }

        /// <summary>
        /// The path to the file.
        /// </summary>
        public string Path { get; }

        /// <summary>
        /// The last time the file was loaded/used.
        /// </summary>
        public DateTime LastOpened { get; set; }
    }
}

namespace AnnoDesigner.Core.Models
{
    public interface IRecentFilesHelper
    {
        /*
         *  void Load();
         *  void Save();
         *  are called implicitly in ctor, AddFile and RemoveFile
         *
         *  ctor has a IRecentFilesSerializer and a IFileSystem as parameter
         */

        /// <summary>
        /// Occurs when the <see cref="RecentFiles"/> property has been updated.
        /// </summary>
        event EventHandler<EventArgs> Updated;

        /// <summary>
        /// Gets the recently used files.
        /// </summary>
        /// <value>The files.</value>
        List<RecentFile> RecentFiles { get; }

        /// <summary>
        /// Gets or sets the maximum item count.
        /// <para />
        /// The default value is <c>10</c>.
        /// </summary>
        /// <value>The maximum item count.</value>
        int MaximumItemCount { get; set; }

        /// <summary>
        /// Adds the file to the top of the list of recently used files.
        /// </summary>
        /// <param name="fileToAdd">The file.</param>
        /// <exception cref="ArgumentNullException">The <paramref name="fileToAdd"/> is <c>null</c>.</exception>
        void AddFile(RecentFile fileToAdd);

        /// <summary>
        /// Removes the file from the list of recently used files.
        /// </summary>
        /// <param name="fileToRemove">The file.</param>
        /// <exception cref="ArgumentNullException">The <paramref name="fileToRemove"/> is <c>null</c>.</exception>
        void RemoveFile(RecentFile fileToRemove);
    }
}

namespace AnnoDesigner.Core.Models
{
    public interface IRecentFilesSerializer
    {
        /*
         * Implementations could serialize to app settings or XML or JSON or SQLite or ...
         */

        /// <summary>
        /// Saves the list of recently used files.
        /// </summary>
        /// <param name="recentFiles">The list of recently used files.</param>
        void Serialize(List<RecentFile> recentFiles);

        /// <summary>
        /// Loads a list of recently used files.
        /// </summary>
        /// <returns>The list of recently used files.</returns>
        List<RecentFile> Deserialize();
    }
}

Yeah -  see what you mean. The interface shouldn't care, but the implementation could.
Yep, that API looks good.

Cool. Good thing I already started to implement it 😁  (a few weeks ago).
So after the FandomParser got some love and the whole project got converted to the new project format, this would be my next work.


Deinit cause crash when cameraDevice = .front

Hi, seems like the self.cameraDevice = .back inside stopAndRemoveCaptureSession() in deinit is causing crash if the cameraDevice is .front, I have to manually calling stopAndRemoveCaptureSession() in my viewController to stop it before dismiss the viewController.
Please check this issue, thank you.

+1

I was able to work around this by setting cameraManager.shouldKeepViewAtOrientationChanges = true, since the issue is happening in the async closure at:
https://github.com/imaginary-cloud/CameraManager/blob/b0075ed1d8af926d65988dd44883c23c4257ff15/camera/CameraManager.swift#L1097
The specific runtime error is : Thread 1: EXC_BAD_ACCESS (code=1, address=0x20)

@TienVu-PW and @davidortinau,
we are currently looking at this issue. Thanks for the feedback 👍

+1

@TienVu-PW @davinic @lucasmpaim,
this issue should be fixed in v.4.2.3.
Please let me know if it worked.

@torrao Seems to be working fine now, thanks!


Resolve GEM unit test failures

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

Work Item: W-8248351


UniFi - Config Options to configure SSID filter

Breaking Change:

Description:
This PR needs to get multi select support available in frontend before it can be completed.
Related issue (if applicable): fixes #27243
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hey @Kane610, what should we do with this PR?

@balloob I was planning to start looking at backend support for @bramkragten multi select support. I have an issue or two to solve first

Could we close this PR in the meanwhile so it cleans up our PR board ?

Absolutely


fix: Update LICENSE to Apache 2.0

Closing PR because Noel doesn't know how Semantic PRs work apparently....


initial commit in Highcharts 5.x.x types

Thought I was on my repo. =)
It's working already but missing tests.
Help is welcome to create them, there are lots of interfaces :laughing:
I'll create a new PR when it is tested properly.


Can you give me a little help on how to setup this bot?

I've been trying to run it since yesterday but I can't succeed 😅
(Trying to guide myself using the AI, I'm still learning to code and for now I only know the basics of Python)
For now I have:

Cloned the repository with git clone
Changed directory with cd chatgpt-twitter-bot
Used npm install to install package.json dependencies
After installing the dependencies I used the command npm run build to compile
After compiling I used the command node build/index.js

Are these the correct steps?
After running node build/index.js it throws the error:
Error: Dynamic require of "util" is not supported
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:382
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:1535
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:462
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:4405
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:462
at file:///Users/bruno/chatgpt-twitter-bot/build/index.js:1:4967
at ModuleJob.run (node:internal/modules/esm/module_job:194:25)
Node.js v19.3.0
I believe I misconfigured the .env file, I'm having trouble understanding how to fill the ChatGPT credentials even though I read the entire NPM package readme

Running build isn't necessary; it's more of a test step.
I'm using tsx to execute the script directly.
So npm i -g tsx and tsx src/index.ts
Assuming your .env is setup correctly. Getting all the required .env files is a huge pain unfortunately, especially twitter since we have to use both v1 and v2 of the APIs...

Ahh, I get it!
I think I'm really missing something in the .env then
After running npm i -g tsx and tsx src/index.ts the terminal threw this error now:
(node:86611) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use node --trace-warnings ... to show where the warning was created)
refreshing twitter access token
node:events:491
throw er; // Unhandled 'error' event
^
Error: getaddrinfo ENOTFOUND undefined
at __node_internal_captureLargerStackTrace (node:internal/errors:491:5)
at _node_internal (node:internal/errors:713:10)
at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:107:26)
Emitted 'error' event on Keyv instance at:
at KeyvRedis. (/Users/bruno/chatgpt-twitter-bot/node_modules/keyv/src/index.js:60:46)
at KeyvRedis.emit (node:events:513:28)
at KeyvRedis.emit (node:domain:489:12)
at EventEmitter. (/Users/bruno/chatgpt-twitter-bot/node_modules/@keyv/redis/src/index.js:18:40)
at EventEmitter.emit (node:events:513:28)
at EventEmitter.emit (node:domain:489:12)
at EventEmitter.silentEmit (/Users/bruno/chatgpt-twitter-bot/node_modules/ioredis/built/Redis.js:460:30)
at Socket. (/Users/bruno/chatgpt-twitter-bot/node_modules/ioredis/built/redis/event_handler.js:189:14)
at Object.onceWrapper (node:events:628:26)
at Socket.emit (node:events:525:35) {
errno: -3008,
code: 'ENOTFOUND',
syscall: 'getaddrinfo',
hostname: 'undefined'
}
Node.js v19.3.0


Add GitHub Workflow for periodic imports

Solves #32
For now every 2 hours

haha, just worked on this too 😆
name: scheduled taks

on:
  schedule:
    # run every hour
    - cron: '0 * * * *'

jobs:
  staging:
    name: crawl events in staging
    runs-on: ubuntu-18.04
    steps:
      - name: STAGING  Run remote commands
        uses: appleboy/ssh-action@master
        with:
          host: "staging.muenster-jetzt.de"
          username: root
          key: ${{ secrets.SSH_DEPLOY_KEY }}
          script_stop: true
          script: |
            set -e
            cd muenster-jetzt/staging
            docker-compose exec -T api python -m mj crawl
  production:
    name: crawl events in production
    runs-on: ubuntu-18.04
    steps:
      - name: PRODUCTION  Run remote commands
        uses: appleboy/ssh-action@master
        with:
          host: "muenster-jetzt.de"
          username: root
          key: ${{ secrets.SSH_DEPLOY_KEY }}
          script_stop: true
          script: |
            set -e
            cd muenster-jetzt/production
            docker-compose exec -T api python -m mj crawl

so lets compare results:

every two hours vs every hour: i think every hour is still gentle for the apis called and would be more up to date in case of an event getting cancelled?
i've used two different jobs to run the imports in parallel and don't prevent a run in a case where e.g. staging is broken and crawling does not work
i did cd the docker-compose directories to ensure the .env file loading but I guess it's obsolete if we stick to use docker-compose exec which uses the already running container

Yep, yours is better. Didn’t thought of points two and three. Very valid points. I would say scrap my version and use yours


Embedding Taipei-Torrent

A couple functions like ListenForPeerConnections and (*TorrentSession).AcceptNewPeer take a btConn struct, but this struct is not exported, while these functions are public.  Change btConn to BtConn, even with all of its fields filtered would be fine?

+1
A nice implementation of a pure torrent engine is https://github.com/mafintosh/torrent-stream though it's JavaScript (Node). It can list files within torrents, then selectively download ranges of those files, using quite a simple API:
var torrentStream = require('torrent-stream');

var engine = torrentStream('magnet:my-magnet-link');

engine.on('ready', function() {
    engine.files.forEach(function(file) {
        console.log('filename:', file.name);
        var stream = file.createReadStream();
        // stream is readable stream to containing the file content
    });
});

It'd be awesome if we could use this type of API in our own Go apps.

Sorry for the spam. I just noticed that the referenced PR had been merged, which means this is possible now, I presume? Are there any examples of simple programmatic use?


Issue #124: enhance NER to load Phase3 override KB

Enhance NER to load multiple override files. Update to use Bioresources 1.1.20 containing MITRE PhaseIII override KB.

This is good, but I wonder if we can come up with a naming convention for the override files. Phase3-Override.tsv.gz seems very arbitrary... Maybe we can have NER-Grounding-Override-ddd.tsv.gz, where the ddd indicates the priority in loading (the smaller the number the higher the priority)?

Sure, that sounds fine but there are some other considerations with this file that we should discuss at our weekly meeting tomorrow.

I just re-merged master into this branch after starting the pull request, then pushed again. I am hoping that Travis will do the right thing and restart the PR tests?

Restart Travis testing.


chore: release v10

Placeholder for (minimal) changes we make as we release v10.
Changelog
Changed

Default feature flags, flipping everything but UI shell one.

Removed

src directory from carbon-components package.

Testing / Reviewing

Is there any feature flag that we shouldn't flip?

Closing for #2208 👍


Use lib instead of CMAKE_INSTALL_LIBDIR for pioasm install

The CMake GNUInstallDirs module prior to CMake version 3.23.1 used an architecture-specific CMAKE_INSTALL_LIBDIR on debian, which is not searched by find_package calls from the SDK
Fix this in the same way as picotool https://github.com/raspberrypi/picotool/issues/117 by hard-coding the lib directory

Fix this in the same way as picotool raspberrypi/picotool#117 by hard-coding the lib directory

For cross-reference porpoises :dolphin:  https://github.com/raspberrypi/picotool/commit/6ad9c2352cf90b4abf183143e3cbd9ec933b9ad1


Mago-MI refactoring and fixes

There's a "Merge branch master" commit. I guess you should rebase this.

Not sure how to do it.
on my master branch, git rebase upstream/master?

Not sure how to do it.
on my master branch, git rebase upstream/master?

Sounds about right. I'm not an expert on the git command line, though. I use TortoiseGit which guides you through the process interactively.

did rebase
i'm not an expert, too
just hope all is ok

Unfortunately, it's even worse now because your commits appear twice and the merge commits are also still there. You can see these in the list of commits.
Maybe you should start with a new branch based on rainers/master (I don't like those upstream/origin terminology as it is too ambiguous) and cherry pick your commits to this.

See new pull request.
Closing this one,


Create Jenkins Job to Deploy

First create a job to a) choose branch (i.e. "build with parameters), and b) build on myhpom-dev.edc.renci.org. After that, add the target machine to the build parameters so we can build on other machines.

Merged, and myhpom and myhpom-dev are now deployed using this work.


Adding Repeated Search Tests for Landing and Setup/Search Pages

This change is

:+1:  Great to see these tested!


Improve make PR (force build and help message)

Add make pr to make help and force build assets before launch (mostly to update ui)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@fde0562). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #10178   +/-   ##
=========================================
  Coverage          ?   43.41%
=========================================
  Files             ?      576
  Lines             ?    79642
  Branches          ?        0
=========================================
  Hits              ?    34578
  Misses            ?    40789
  Partials          ?     4275

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 fde0562...b7758f0. Read the comment docs.


NotFoundError: Node was not found

/js/application-cd7f4ede96.js in l/this.stop at line 18:22204
/js/application-cd7f4ede96.js in a/this.check/l at line 18:19773
/js/application-cd7f4ede96.js in a/this.check/d< at line 18:20347
/js/application-cd7f4ede96.js in n at line 28:18683

reproduced in Firefox and Firefox Mobile

Hello, I can not recreate the error in Firefox and the error lines do not match with the FuckAdBlock file.
Do you have a site to show me as an example?

No, i got this error from Sentry
I think the error in this method
https://github.com/sitexw/FuckAdBlock/blob/v4.x/fuckadblock.js#L323


revert paket push to WebClient

revert paket push to WebClient.
add PAKET_PUSH_HTTPCLIENT to test out the httpclient push,
once stable we remove all parts ifdefd with USE_WEB_CLIENT_FOR_UPLOAD.

Should fix https://github.com/fsprojects/Paket/issues/2501.
Once I'm happy with the HttpClient implementation the WebClient will go away forever hopefully :)
Will add the tests with the PR removing the WebClient, this PR only tries to restore the push feature.

Can you please release it right away?
Am 10.07.2017 18:55 schrieb "Matthias Dittrich"<EMAIL_ADDRESS>
Should fix #2501 https://github.com/fsprojects/Paket/issues/2501.
Once I'm happy with the HttpClient implementation the WebClient will go
away forever hopefully :)
Will add the tests with the PR removing the WebClient, this PR only tries
to restore the push feature.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/pull/2505#issuecomment-314167721,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNCfD7J7x8TT2cObL6LxjbWS297G7ks5sMld2gaJpZM4OTHlw
.


Document i18n string escaping

See -> https://github.com/Manvel/cmints/commit/86c63fa3218d2622ed8ac7215c85974e18a28122

This issue is irrelevant anymore, see -> https://github.com/Manvel/cmints/issues/109


TODO transferred from staged-recipes/libigt2

Coped from https://github.com/conda-forge/staged-recipes/pull/1658#issue-179137529. Checkboxes checked in staged-recipes are crossed out here.
TODO before merging:

License

[x] libgit2
[x]  pstdint.h (http://www.azillionmonkeys.com/qed/pstdint.h)  checked in staged-recipes

[x] Add optional dependencies, https://github.com/libgit2/libgit2#optional-dependencies

[x] pthreads (non-Windows), addressed in ( https://github.com/conda-forge/libgit2-feedstock/issues/3 )
[x] OpenSSL (non-Windows)  checked in staged-recipes
[x] LibSSH2, addressed in https://github.com/conda-forge/libgit2-feedstock/pull/2
[x] iconv (OSX)  checked in staged-recipes

[x] Add tests  checked in staged-recipes

Per https://github.com/conda-forge/staged-recipes/pull/1658#issuecomment-251263011:

[x] libiconv is available on all platforms. Seems they only care about OS X.  checked in staged-recipes
[x] Having openssl working would be good. Need to pin it though.  checked in staged-recipes
[x]     libssh2 would be a very nice add.
[x]     ... it looks like it requires more thought. Maybe table it and raise an issue on the feedstock. addressed ( https://github.com/conda-forge/libgit2-feedstock/issues/3 )
[x] Please add curl if you haven't already.  checked in staged-recipes

The items listed in the description have been raised as separate issues. If the license looks good, I am closing this issue.

License looks fine @shadowwalkersb . GPL2 with linking exception is pretty clear. As we distribute the license file in all packages too, people can look at that for more info if needed. Closing this sounds fine to me. 👍

Thanks, @jakirkham !


Ribbon Control

Feature request type
New component
Component name
No response
Is your feature request related to a problem?
No response
Describe the solution you'd like
I would like to have a  ribbon Control.
Have you seen this feature anywhere else?
Example from another repo:
https://blazorfluentui.azurewebsites.net/ribbonMenuPage
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[x] I agree to follow this project's Code of Conduct

By any chance, are you looking for "tabs"?
https://mudblazor.com/components/tabs


docs(*): update markdown files and fix typos - I257

Signed-off-by: Michael Zhou<EMAIL_ADDRESS>
Updated markdown files by correcting for incorrect markdown links, integrating existing links into new markdown, and ensuring consistency in markdown style. Also fixes typos, grammar, and punctuation errors in documentation. Some tests failed due to code vulnerabilities.
Changes

Modified markdown to be more appropriate with content of associated links
Fixed typos and miscellaneous errors in punctuation and grammar

Related Issues

Cicero Template Library Issue 257
Pull Request #280

These changes aren't affecting the travis build, merging.


64 bits compatibility

It says that its not compatible because of the 64 bits compatibility ?

This is just a python script, so there shouldn't be any compatibility issue until you have the correct version of python installed. But if you are still facing some issue, use open python sandboxes like pythoneverywhere.com

This is just a python script, so there shouldn't be any compatibility issue until you have the correct version of python installed. But if you are still facing some issue, use open python sandboxes like pythoneverywhere.com


shared: removed 'text()' from cursor.execute

:heart: Thank you for your contribution!
Description
Rolls back part of https://github.com/psaiz/invenio-db/commit/5fc0fc91446a361f46daabfab51d57e90c73f2ec.
It looks like the connection.execute and cursor.execute expect different arguments. The former is a text, the latter a string.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings (for relevant code).
[ ] I've followed the CSS/JS and React guidelines (for relevant code).
[ ] I've followed the web accessibility guidelines (for relevant code).
[ ] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Hi @utnapischtim,
Thanks for looking into this. I see this error in the pipelines of other modules that depend on invenio-db. It might be that invenio-db requires an old version of SQLAlchemy, and then this is still something that should be addressed in the invenio-db module

@psaiz
I think there is another problem in the invenio-previewer package. Invenio-db  works with the same installed SQLAlchemy version as invenio-previewer, for FlaskSQLAlchemy it is the same version too.
Further i don't think that invenio-db should handle that. Invenio-db is working as expected with the installed packages. Otherwise the error should also pop up in on the invenio-db tests. My problem with removing the compatibility with newer SQLAlchemy versions in invenio-db is that it would be a problem in the future so i think it is the better way to improve invenio-previewer first.
What i see too is that invenio-previewer is using sqlite, maybe that introduces a problem which is not tested in invenio-db.
Since the error is happening on doctest you could update those packages (e.g. Sphinx) first on invenio-previewer and see if that fixes the problem?

@utnapischtim it turned out that we were never testing sqlite here in invenio-db. In the PR below, we have added the sqlite tests in the CI matrix.
The change proposed by @psaiz is the one in the official documentation: https://docs.sqlalchemy.org/en/20/dialects/sqlite.html#foreign-key-support
Closing because superseded by: https://github.com/inveniosoftware/invenio-db/pull/171


How can I override the feature semantic for multi-dimensional features?

I am trying to run a RF regression based on my dataset. My dataframe looks like this [below is only the first 5 rows]:
              target                                          feat_1  \
county  year
County_1 2000   0.047879  [0, 2, 10, 9, 12, 10, 9, 20, 35, 51, 0, 0, 0, ...
        2001  -0.112184  [0, 1, 0, 2, 1, 2, 4, 9, 18, 34, 0, 1, 0, 1, 1...
        2002   0.060659  [0, 0, 0, 0, 3, 24, 33, 32, 42, 58, 0, 0, 0, 2...
        2003   0.098047  [0, 0, 1, 5, 13, 22, 40, 38, 29, 42, 0, 0, 0, ...
        2004  -0.053559  [0, 1, 0, 2, 6, 8, 14, 33, 34, 64, 0, 0, 1, 1,...

                                                      feat_2
county  year
County_1 2000  [1.8121698113207556, 0.938584905660378, -0.568...
        2001  [2.6941509433962274, 3.888301886792455, 2.8169...
        2002  [-3.4043396226415084, -3.458113207547169, -3.5...
        2003  [-1.9566037735849044, -2.3393396226415084, -2....
        2004  [-3.2046226415094323, -3.502075471698112, -2.9...

When running the code to do the regression using only feat_1, the code works perfectly fine.
FEATURES_IN = ['feat_1']
learner = ydf.RandomForestLearner(label='target', features=FEATURES_IN, task=ydf.Task.REGRESSION)
model = learner.train(df_train)

[it works well. I can use model.describe() later to read the model]
However, as I include feat_2 in the regression,
FEATURES_IN = ['feat_1', 'feat_2']
learner = ydf.RandomForestLearner(label='target', features=FEATURES_IN, task=ydf.Task.REGRESSION)
model = learner.train(df_train)

It raises the error:
ValueError: Cannot import column 'feat_2' with semantic=Semantic.CATEGORICAL_SET as it contains floating point values.
Note: If the column is a label, make sure the correct task is selected. For example, you cannot train a classification model (task=ydf.Task.CLASSIFICATION) with floating point labels.

In this case, I am not sure how to override the feature semantic for multi-dimensional features. I could not find it in your documentation. I tried to use
FEATURES_IN = [
    ydf.Feature("feat_1", ydf.Semantic.NUMERICAL),
    ydf.Feature("feat_2", ydf.Semantic.NUMERICAL),
]
learner = ydf.RandomForestLearner(label='target', features=FEATURES_IN, task=ydf.Task.REGRESSION)
model = learner.train(df_train)

but it seems not to work:
ValueError: Cannot convert NUMERICAL column 'feat_1' of type numpy's array of 'object' and with content=array([array([  0,   2,  10,   9,  12,  10,   9,  20,  35,  51,   0,   0,   0,
                0,   0,   3,   7,  13,  12,  36,   0,   0,   0,   4,   0,   6,
               10,  11,  36,  63,   0,   0,   1,   0,   3,   8,  27,  34,  60,
               93,   0,   0,   0,   0,   0,   3,   8,   9,  25,  18,   0,   0,
                0,   0,   0,   4,   2,  13,  11,  15,   0,   3,  30, 179, 159,
              102,  87,  85,  60,  68])                                       ,
       array([ 0,  1,  0,  2,  1,  2,  4,  9, 18, 34,  0,  1,  0,  1,  1,  5,  8,
              30, 44, 67,  0,  0,  0,  1,  0,  2, 13, 26, 33, 63,  0,  0,  0,  0,
               0,  0,  3, 13, 21, 27,  0,  0,  0,  0,  1,  1,  4,  6, 11, 25,  0,
               0,  1,  2,  4,  1,  9, 20, 40, 41,  1,  1,  2,  6,  4, 10, 19, 25,
              18, 47])
[The error message prints the whole array, which is way too long. I omit the contents in the middle here.]
      array([  136,  2545, 11700, 18486, 15007, 10840,  7356,  5265,  5448,
               5890,    84,   140,   119,   156,   260,   646,  1778,  2549,
               2890,  2992,     0,     1,     3,     8,    17,    20,    72,
                 91,   151,   179,     0,     0,     1,     4,     5,    12,
                 16,    18,    68,    98,     2,     1,     0,     0,     7,
                 16,    15,    21,    46,    94,     0,     0,     2,    11,
                 12,    23,    16,    34,    67,   117,     4,    10,    25,
                 34,    31,    30,    43,    66,    97,   180])             ],
      dtype=object) to np.float32 values.
Note: If the column is a label, make sure the training task is compatible. For example, you cannot train a regression model (task=ydf.Task.REGRESSION) on a string column.

In this case, I am not sure how to deal with it.

In the current implementation, YDF does not support unpacking pandas dataframes to multi-dimensional features, but I'll take this as a feature request to add this functionality. In the first case you're describing, YDF actually creates categorical set features for each column, which is probably not what you want.
If you want to unpack multi-dimensional features, feed the dataset e.g. as a dictionary of Numpy arrays, as shown in this tutorial
Improving the feature handling is a goal for the next version.

Thank you for your kind reply! I will try reformatting the feature as Numpy arrays for the model. Besides, given this, I think that at the present stage, it might be helpful if you would consider adding some notes and explanations in that tutorial so that other people can also know the function of your library better. Thank you again for your help!


fix: update projected in main

What type of PR is this?
/kind bug

What does this PR do?

In order to accommodate the painless migration, I compared the master subpod projected the same when I updated it
Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

/assign @duanmengkk


Add longer timeout on Windows

For https://github.com/microsoft/go/issues/700

I think our builders simply run more slowly than upstream's. Start out with increasing the timeout.
https://github.com/golang/go/blob/833367e98af838a2511ee7e4e19dc8f1da7b8ed7/src/cmd/dist/test.go#L553 includes t.timeout(300), which should enable the multiplier to work in this case. (Some dist tests don't have this and would need something like it to be patched in to work.)

Confirmed, the new timeout let this job succeed:
##### GOMAXPROCS=2 runtime -cpu=1,2,4 -quick
ok  	runtime	359.114s


[1.9.x] Removing unnecessary locators when detecting intraprocess only usecase [7041]

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows


Publishing aar to mavenCentral

Please, is it possible to publish your aar to mavenCentral instead of build an aar manualy ?
Thanks a lot !

Yes, this is going to happen soonish. Please see the discussion in #636 .

Update: there is now a SNAPSHOT version available. To access it, you need to touch at least 3 files in your app:

settings.gradle.kts

dependencyResolutionManagement {
    repositories {
        maven(url = "https://s01.oss.sonatype.org/content/repositories/snapshots")
    }
}

build.gradle.kts:

dependencies {
    implementation("io.github.zxing-cpp:android:2.1.0-SNAPSHOT")
}

your-app.kts

import zxingcpp.ZXingCpp

Before I publish the final release, I'd like to hear from at least 2 of @markusfisch @benjohnde @adrienbusin @okarmazin that this works for them. Please note that the class-path/namespace recently changed from com.zxingcpp to zxingcpp.

Works for me 👍

Works for me too 👌

Brilliant! Nice :)

Hello @axxel did you push 2.2.0 version to mavenCentral ? I don't find it, but in your readme It's written that 2.2.0 is available into MavenCentral ?
Thanks 🙂

Sorry for the confusion but the 2.2 release is not out, yet.
See the recent discussion about remaining inconsistencies. (Github mobile app does not help me in finding the #-number... :-/ )

Ok ! @axxel so when do you plan to release a new version add push it to mavenCentral ? thanks 🙂

Done: https://central.sonatype.com/artifact/io.github.zxing-cpp/android


Scanner state freezes - unable to scan new QR codes

Severity - High
Case 1
Steps to reproduce:
Hold camera up to QR code
Move camera away from QR code
Navigate back to home - Go back to scanner
Result: Scanner still has value from previous QR code, Submit button is inactive and scanner won’t read any new codes
Case 2
Steps to reproduce:
Hold camera up to QR code
Hit Submit - Get “Received” message
Move camera away from QR code
Navigate back to home - Go back to scanner
Result: Scanner still says “Received”, button is disabled, scanner not reading new QR codes
I have seen this hard crash the whole app but have not yet reproduced consistently. I'm on iOS v13.3 on an iPhoneX

video 1
IMG_9134.MP4.zip

video 2
IMG_9135.MP4.zip

merged into experimental-v2 branch: 956956e4c89bf87d7d44e2a4d8e99dc703511cda

Please verify on TestFlight Build 51, thanks @woznaut @g9singh

This fix is also included in Build 53 and 54 (798 and 799 for Android)


cmpctmalloc: Tweak bucket sizes to optimize power-of-2 allocs.

With this change allocations of > 128 bytes that are powers of two will be allocated at the exact size requested, instead of being subjected to worst-case round-up of 1/8th (12.5%).
Fix and test heap_trim better.  The tests now do trims with tricky free sizes at the start and end of OS allocation areas (just around 1 page).
Add an assert for the most obvious double frees in debug mode.

merged


Clash of licenses

This project is Open Source and use an open Apache license. But some/many files are sublicensed (?) with a strong non-open copyright by Scholz and Volkmer.
E.g. dbbahnhoflive-ios/public/OpenSourceAPI/MBNewsRequestManager.m
//
//  MBNewsRequestManager.m
//  MeinBahnhof
//
//  Created by Heiko on 19.11.19.
//  Copyright © 2019 ScholzVolkmer. All rights reserved.
//
Please change ALL files to respect the Apache license. An Open Source project should be open.
Thomas

One good way to do that would be to follow the conventions outlined by the REUSE project: https://reuse.software/faq/#step-2. That would mean adding information in the headers which looks like this:
// SPDX-FileCopyrightText: [year] [copyright holder] <[email address]>
//
// SPDX-License-Identifier: Apache-2.0

Hi @tursics, hi @cornelius ,
thank you for bringing this topic to the surface. We will take care of it and work on a more transparent and clear way to communicate the license handling in this project.

replaced license information in the source files


[PrintAsObjC] Handle the importer's compatibility typealiases.

These are TypeAliasDecls whose Clang nodes are not TypedefNameDecls. This worked all right for classes, but dropped the tag keyword (e.g. struct) for tag decls with names of their own, and didn't print any name at all for C types that used the typedef-for-anonymous-tag pattern.
rdar://problem/32514335

@swift-ci Please smoke test


Enable SimpleCov code coverage in CI

Closes: https://github.com/belgamo/comarev/issues/15
What?
Enable simplecov coverage gem in CI.
Adding minimum coverage setting to 90%.
Why?
This way we can keep the code coverage next to 100%.
How to test?
Just looking if the GH actions creates an artefact (coverage.zip) file, with the coverage generated by simplecov.

Looks nice !


Upgrade Netty to 4.1.77.Final and netty-tcnative to 2.0.52.Final

Motivation

release notes https://netty.io/news/2022/05/06/2-1-77-Final.html
some highlights:

improves Alpine / musl compatibility

could help issues such as #14534 #11415 #11224 #10798

improves shading compatibility
fixes a bug related to the native epoll transport and epoll_pwait2

Modifications
Upgrade Netty to 4.1.77.Final and netty-tcnative to 2.0.52.Final

Move release/2.7.5 label to #16896


add template for creating Jenkins server and projects

use https://github.com/stelligent/dromedary as a guide/spec

https://blogs.aws.amazon.com/application-management/post/TxMJROUIFQZ4HS/Setting-Up-the-Jenkins-Plugin-for-AWS-CodeDeploy - another good reference for CF, CP, and CD


pip-compile fails on unavailable irrelevant constraint

pip-compile tries to find a match for all constraints, event those that are not a requirement and come from a constraint file.
As a result if a constraints file defines a constraint on a package that is unavailable (be it a typo, a private lib on a currently unavailable repository, or a deprecated version no longer available) pip-compile won't work, even for requirements.in that will not need that package.
Steps to replicate
requirements.in:
-c constraints.txt

constraints.txt:
invalid-dependency~=0.0

Expected result
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile requirements.in
#

Actual result
Could not find a version that matches invalid-dependency~=0.0 (from -c constraints.txt (line 1))
No versions found

This should be fixed by #1175

Hello @clslgrnc,
Thanks for the issue an the fix! Looks like pip doesn't resolve constraints too if they are not presented in requirements.txt.

@clslgrnc Thanks for this report - I think I've run into this today, maybe you would consider adding the following as a test case? Or could you confirm that your fix in #1175 will solve this issue?
The reason I'm asking is because the msgpack constraint is on a "real" package, but the version of < 0 is unfulfillable, which is slightly different to the example in your report above.
The second reason I'm asking about this is because it might open the door to another way to solve https://github.com/jazzband/pip-tools/issues/333 < I'm currently trying to install firebase-admin keeping msgpack-python as a dependency and not updating to the renamed msgpack.
Example files
requirements.in:
-c constraint.txt
pip-tools

constraint.txt:
msgpack < 0

Current behaviour
With pip-tools==5.2.1 on Python 3.7, pip-compile -v fails:
                          ROUND 1
Current constraints:
  msgpack<0 (from -c constraint.txt (line 1))
  pip-tools (from -r requirements.in (line 2))

Finding the best candidates:
Could not find a version that matches msgpack<0 (from -c constraint.txt (line 1))
Tried: 0.5.0, 0.5.1, 0.5.2, 0.5.3, 0.5.4, 0.5.5, 0.5.6, 0.6.0, 0.6.0, 0.6.1, 0.6.1, 0.6.2, 0.6.2, 1.0.0, 1.0.0
Skipped pre-versions: 1.0.0rc1, 1.0.0rc1
There are incompatible versions in the resolved dependencies:

:point_up: That's the end of the output - there are no resolved dependencies listed.
Expected behaviour
pip-tools doesn't need msgpack, so we don't care that there isn't a version < 0. Compile should pass with the usual output.

@jamescooke have you tried #1175?

@atugushev No I've been lazy and just typed :disappointed:

It works with #1175: parsing the constraint does not fail and it does not try to resolve msgpack<0 since it is not refered to by pip-tools.
I am not sure how it could help solve #333 though.

@atugushev I've given #1175 a go and it works well for the example above :raised_hands: - thanks @clslgrnc for your work on this :+1:
Now using #1175, I can create an impossible constraint against an existing package with < 0. This prevents that package ever being included in the compiled requirements - and the fix in #1175 means that pip-compile will pass when the banned package is not included in the requirements.
In this case, constraining with msgpack < 0 (as per the example above) means that no compiled requirements can ever include msgpack. The newly named msgpack causes a conflict with the originally named package msgpack-python so until it can be resolved in the project I'm working on, it needs to be prevented from being installed. I think this is a similar issue to what's described here https://github.com/jazzband/pip-tools/issues/333#issuecomment-245785086 with enum and enum34.
Example files
constraints.txt:
msgpack < 0

requirements.in:
-c constraints.txt
firebase-admin<4

pip-compile gives an error:
Could not find a version that matches msgpack<0,>=0.5.2 (from -c constraints.txt (line 1))
Tried: 0.5.0, 0.5.1, 0.5.2, 0.5.3, 0.5.4, 0.5.5, 0.5.6, 0.6.0, 0.6.0, 0.6.1, 0.6.1, 0.6.2, 0.6.2, 1.0.0, 1.0.0
Skipped pre-versions: 1.0.0rc1, 1.0.0rc1
There are incompatible versions in the resolved dependencies:
  msgpack<0 (from -c constraints.txt (line 1))
  msgpack>=0.5.2 (from cachecontrol==0.12.6->firebase-admin==3.2.1->-r requirements.in (line 2))

This is great because it's what I'm looking for - msgpack is banned.
At this stage, if the resolver had increasing search functionality, then it might find the solution - but I'm not complaining :blush: ...
We can get compilation to pass by pinning one of the sub-dependencies:
-c constraints.txt
firebase-admin<4
cachecontrol<=0.12.4

Now (with #1175) this compilation passes and we get a set of requirements for firebase-admin which are guaranteed not to include msgpack. :raised_hands:

@jamescooke
Cool! Thanks for the detailed feedback! 👍

Interesting use case @jamescooke thanks for the explanation.


Add LPNT token

为确保 Token Profile 的顺利通过，务必逐一确保以下检查项并补充项目资料：
.json 及 .png 文件检查项

[x] logo 尺寸：120*120 px
[x] logo 背景：透明
[x] 代币有至少一笔成功的转账记录
[x] 此项对于 ERC-20 项目方：合约地址有大小写（做 checksum）
[x] .json 和 .png 文件名一致（包括大小写）

项目资料：
官方公告：
团队背景：
项目基本情况：
媒体报道：
收录的交易所：
转账所需 Gas Limit（对 ERC-20 项目方）：（默认为 60000）

Please check the following to ensure a successful verification of Token Profile
.json and .png document checklists

[x] logo dimensions: 120*120 px
[x] logo background: Transparent
[x] At least one transaction on the token contract
[x] For ERC-20 project teams: Upper and lower capitalized alphabets in token address (For checksum purposes)
[x] Same document name for both .json and .png (Including capitalization)

Project background:
Official announcement:
Project team background:
Project basic information:
Media publications:
Tradeable exchanges:
Recommended Gas Limit (For ERC-20 project teams) for transaction: (Default is 60,000)

Official Announcement: https://twitter.com/officialLPNT/status/1423329141311492097?s=19
Project basic information:
LPNT (LUXURIOUS PRO NETWORK) Group is an undisputed global leader in the domain of luxurious transportation and forex trading.
We are a group of companies handling a variety of business streams since marking inception in 2010.
Many multibillionaire investors are already rubbing shoulder with us on the investment front. Many prominent fund managers associated with us specialize in dealing with trading in forex.
Coming to the point, LPN TOKEN is the venture of LUXURIOUS PRO NETWORK Group. This Ethereum Blockchain based cryptocurrency for safe, speedy, and decentralized transactions can change the way you transact.
Recommended Gas Limit (For ERC-20 project teams) for transaction: 60,000

Need delete the json file except LPNT in the branch


Propagate concrete types of arguments to apply

Final Part of https://github.com/apple/swift/pull/13991/ that propagates concrete types of arguments to apply. This handles non-self arguments as well.

@slavapestov @atrick This PR is part of https://github.com/apple/swift/pull/13991/ and just focuses on concrete type propagation for all arguments of an Apply.

Reminder: check for inout

@atrick find sometime to review this?

Thanks @atrick . I pushed a new commit.

@swift-ci Please test

@swift-ci Please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - 9452e0f271ec44dfed0f0b31e6cd81b1ae742d1c

Build failed
Swift Test OS X Platform
Git Sha - 9452e0f271ec44dfed0f0b31e6cd81b1ae742d1c

@rajbarik Unfortunately this broke the asan bot. I reverted it.

@eeckstein Anything I can do to help?

That's (most likely) caused by a bug in your code. I recommend to reproduce the issue locally with that asan configuration.

@eeckstein @rajbarik normally the log file from the ASAN bot has enough information to give you a clue where to look in the code.

I see. I never did this before. Could you either point me to the log of
ASAN bot or steps to reproduce it at my end?
On Tue, Sep 11, 2018 at 5:34 PM Andrew Trick<EMAIL_ADDRESS>wrote:

@eeckstein https://github.com/eeckstein @rajbarik
https://github.com/rajbarik normally the log file from the ASAN bot has
enough information to give you a clue where to look in the code.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apple/swift/pull/17370#issuecomment-420471588, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFBwaVCTmSsqLQ_TuqItP8IKC1kNdglRks5uaFaUgaJpZM4Uv6d2
.

@atrick I think I narrowed down the problem. I was storing stack allocated CEI structs in a loop using a map and then was using the map outside the loop later:
for (unsigned ArgIdx = 0; ArgIdx < Apply.getNumArguments(); ArgIdx++) {
 991     auto ArgASTType = Apply.getArgument(ArgIdx)->getType().getASTType();
 992     if (!ArgASTType->hasArchetype())
 993       continue;
 994     const ConcreteExistentialInfo CEI(Apply.getArgumentOperands()[ArgIdx]);
 995     if (!CEI.isValid())
 996       continue;
 997
 998     CEIs.insert(
 999         std::pair<unsigned, const ConcreteExistentialInfo *>(ArgIdx, &CEI));
1000
1001     if (CEI.ConcreteType->isOpenedExistential()) {
1002       // Temporarily record this opened existential def in this local
1003       // BuilderContext before rewriting the witness method.
1004       OpenedArchetypesTracker.addOpenedArchetypeDef(
1005           cast<ArchetypeType>(CEI.ConcreteType), CEI.ConcreteTypeDef);
1006     }

I can think of two ways of fixing this:

make the type of map to be of <unsigned, const ConcreteExistentialInfo> in which case when we store, we copy [this requires that your code

ConcreteExistentialInfo(ConcreteExistentialInfo &) = delete;

be removed and CEIs are copied around. Is there any particular reason you disabled the copy constructor?
2) A ConcreteExistentialInfo be allocated/deallocated on the heap. Lots of boiler plate code be embedded for new/delete. I would prefer the first one though.
Any recommendations?

Deleting the copy constructor just signals the original intention that's it's a collection of local variables that we don't want to accidentally pass-by-copy or store beyond its caller's scope. In general, you can migrate something like this to a collection of elements using unique_ptr. However, in this case there's nothing inside ConcreteExistentialInfo except pointer-like things, so the default copy constructor is cheap and does the right thing (you can just remove the deleted copy ctor). Copying them into a hash map of dynamically allocated storage one by one is fine.

@atrick @slavapestov  could you review https://github.com/apple/swift/pull/19287 now?


Fix EEPROM Read & Write

Currently the EEPROM read and write do not properly wait after commanding to read or writing data to the register. This is necessary if you plan to read or write more then a single byte at a time. I've added a wait for EEPROM with timeout that now allows sequential byes to be successfully read or written.

@melopero
Any plans to merge this?
I was going crazy because my EEPROM write/read did not work as expected.
This change does fix the problem.

Thank you


Add discussion of noexcept

At the discussion in the LWEG group in Kona, Hawaii 2023 it was suggested that this type should follow the Lako rule.  However these value types have been build to model std::unique_ptr and allocator aware types such as std::vector

Fixed by #151