Change action of the main FAB after opening the menu

How can I change the action of the main FAB after opening the menu, kind of like how Inbox does it:

See my comment on issue #52. To set a label on the menu button you can use fab:menu_fab_label="Compose". If you want to also change the color, you can add that it the animation listener used in my comment on the issue:
scaleInX.addListener(new AnimatorListenerAdapter() {
	@Override
	public void onAnimationStart(Animator animation) {
		if (fabMenu.isOpened()) {
			fabMenu.getMenuIconView().setImageDrawable(iconClosed);
			fabMenu.setMenuButtonColorNormal(colorClosed);
		} else {
			fabMenu.getMenuIconView().setImageDrawable(iconOpened);
			fabMenu.setMenuButtonColorNormal(colorOpened);
		}
	}
});

Try this, it worked for me;
fabMenu.setOnMenuButtonClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {
        if (fabMenu.isOpened()) {
            // Menu is opened and menu button was clicked
        }

        fabMenu.toggle(true);
    }
});

@Croutonix Thanks, I'll try this

@Croutonix what about changing the icon with the current rotation animation?

Try this maybe:
fabMenu.getIconToggleAnimatorSet().addListener(new AnimatorListenerAdapter() {
	@Override
	public void onAnimationStart(Animator animation) {
		if (fabMenu.isOpened()) {
			fabMenu.getMenuIconView().setImageDrawable(iconClosed);
		} else {
			fabMenu.getMenuIconView().setImageDrawable(iconOpened);
		}
	}
});

@Croutonix You removed your example, was there something wrong with it?

Yes sorry, but there was a problem with, fabMenu.getIconToggleAnimatorSet() returns null and not the default animation. You could always change the icon in the click listener like this:
if (fabMenu.isOpened()) {
    // set icon for closed state
} else {
    // set icon for opened state
}
fabMenu.toggle(true);

The problem with this is that it isn't called if you manually close the menu with fabMenu.close(true). You could also try setting a custom animation that is the same of the default animation.
Here's the implementation of the default animation:
private static final int ANIMATION_DURATION = 300;
private static final int ROTATION_ANGLE = -45f;

private AnimatorSet openAnimatorSet;
private AnimatorSet closeAnimatorSet;

private void createDefaultIconAnimation() {
	openAnimatorSet = new AnimatorSet();
	closeAnimatorSet = new AnimatorSet();

	ObjectAnimator collapseAnimator = ObjectAnimator.ofFloat(mImageToggle, "rotation", -90f + ROTATION_ANGLE, 0f);
	ObjectAnimator expandAnimator = ObjectAnimator.ofFloat(mImageToggle, "rotation", 0f, -90f + ROTATION_ANGLE);

	openAnimatorSet.play(expandAnimator);
	closeAnimatorSet.play(collapseAnimator);

	openAnimatorSet.setInterpolator(mOpenInterpolator);
	closeAnimatorSet.setInterpolator(mCloseInterpolator);

	openAnimatorSet.setDuration(ANIMATION_DURATION);
	closeAnimatorSet.setDuration(ANIMATION_DURATION);
}

Then you have to call fabMenu.setIconToggleAnimatorSet(openAnimatorSet) when you want to open the menu and fabMenu.setIconToggleAnimatorSet(closeAnimatorSet) when you want to close it.
In theory that works.

Here's my working snippet of changing the icon during animation (had to slightly adjust @Croutonix example):
private static final int ANIMATION_DURATION = 300;
    private static final float ROTATION_ANGLE = -45f;
    private AnimatorSet mOpenAnimatorSet;
    private AnimatorSet mCloseAnimatorSet;

    private void setUpCustomFabMenuAnimation() {
        mOpenAnimatorSet = new AnimatorSet();
        mCloseAnimatorSet = new AnimatorSet();

        ObjectAnimator collapseAnimator = ObjectAnimator.ofFloat(mFloatingActionMenu.getMenuIconView(),
                                                                 "rotation",
                                                                 -90f + ROTATION_ANGLE, 0f);
        ObjectAnimator expandAnimator = ObjectAnimator.ofFloat(mFloatingActionMenu.getMenuIconView(),
                                                               "rotation",
                                                               0f, -90f + ROTATION_ANGLE);

        final Drawable plusDrawable = ContextCompat.getDrawable(MainActivity.this,
                                                                R.drawable.ic_add_black_24dp);
        expandAnimator.addListener(new AnimatorListenerAdapter() {
            @Override
            public void onAnimationEnd(Animator animation) {
                mFloatingActionMenu.getMenuIconView().setImageDrawable(plusDrawable);
                mFloatingActionMenu.setIconToggleAnimatorSet(mCloseAnimatorSet);
            }
        });

        final Drawable mapDrawable = ContextCompat.getDrawable(MainActivity.this,
                                                                R.drawable.ic_map_black_24dp_png);
        collapseAnimator.addListener(new AnimatorListenerAdapter() {
            @Override
            public void onAnimationEnd(Animator animation) {
                mFloatingActionMenu.getMenuIconView().setImageDrawable(mapDrawable);
                mFloatingActionMenu.setIconToggleAnimatorSet(mOpenAnimatorSet);
            }
        });

        mOpenAnimatorSet.play(expandAnimator);
        mCloseAnimatorSet.play(collapseAnimator);

        mOpenAnimatorSet.setDuration(ANIMATION_DURATION);
        mCloseAnimatorSet.setDuration(ANIMATION_DURATION);

        mFloatingActionMenu.setIconToggleAnimatorSet(mOpenAnimatorSet);
    }

I would like to make the same change in the library itself can you create a pull request so that i can make the same changes


start and end time

Assumed that as you suggests that media server can stream from a specific frame position as for here https://github.com/obastemur/mediaserver/issues/6
it's not clear to me how to handle this on the client side. I mean I have start and end times in seconds. Of course I have the mp3 file info from ffprobe so I should calcolate by my self the frames and cut the audio file before then passing it to the mediaserver?
In that case it's just a cut with ffmpeg but it would prefer to do this in streaming without cutting the file.
Since you are using the method getRange like
var range = getRange(req, total);

I assume I could send you a req.headers.range to support this, right? If so could you make an example of the range, it seems it's like a bytes range separated with a -, so it's to me like
setHeader('range:'+ start_bytes+'-'+end_bytes)

Thank you.

So I get these info with ffprobe like
ffprobe -i /root/audio/12380187.mp3 -show_frames -show_entries frame=pkt_pos -of default=noprint_wrappers=1:nokey=1 -hide_banner -loglevel panic -read_intervals 20%+#1
480279

where 20 is my start position in seconds that turns out to be 480279 in bytes.

[UPDATE]
I think this maybe due to a severe issue on Google's Chrome so I have asked to SF in Content-Range working in Safari but not in Chrome and in Streaming audio in Node.js with Content-Range.
My latest attempt was using the suggested Access-Control-Allow-Headers header set to Range
{ 'Content-Length': 5751405,
  'Content-Type': 'audio/mpeg',
  'Access-Control-Allow-Origin': '*',
  'Access-Control-Allow-Methods': 'POST, GET, OPTIONS',
  'Access-Control-Allow-Headers': 'Range',
  Expires: 0,
  Pragma: 'no-cache',
  'Cache-Control': 'no-cache, no-store, must-revalidate',
  'Accept-Ranges': 'bytes',
  'Content-Range': 'bytes<PHONE_NUMBER>37/5751405' }

as referred in https://github.com/mozilla/pdf.js/issues/3150 and in https://github.com/mozilla/pdf.js/issues/4530#issuecomment-188059771

Thanks for the update. Closing it for now then.


Update CompCert

Motivation for this change
This update allows to unpin the version of Coq (3.0.1 required exactly 8.6).
See commit messages for details.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests) (N/A)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!


MongoDB changefeed not fetching updated doc

In the mongodb changefeed, when the operation is an update, the query to return a record returns None. The pipelines code does not notice the error because None is silently discarded.
https://github.com/bigchaindb/bigchaindb/blob/0c8927dbbe80471f59241b556b24e31dc4396c3f/bigchaindb/backend/mongodb/changefeed.py#L88
The code is:
doc = self.connection.conn[dbname][table].find_one(
    {'_id': record['o2']},
    {'_id': False}
)

The record is:
{'ts': Timestamp(1487159111, 2), 't': 1, 'h':<PHONE_NUMBER>939816142, 'o2': {'_id': ObjectId('58a43f4791e1123be4bc91bb')}, 'o': {'$set': {'assignment_timestamp':<PHONE_NUMBER>.4288526}}, 'ns': 'bigchain_test.backlog', 'op': 'u', 'v': 2}

So instead of {'_id': record['o2']}, it should be {'_id': record['o2']['_id']}

It's also theoretically possible that the record could be removed from the table before the update can be processed. There is not really a fix for this other than to not rely on updates in the changefeed.


Make a Windows build

For easy drag and drop tests in external projects.

CPU only v0.2.2 released - https://github.com/getnamo/Llama-Unreal/releases/tag/v0.2.2


Pinned section container ignores bottom padding 8dp

on Gingerbread
when using custom bitmap background

Wont fix because:

Pinned section disappeared from Material Design specs.
Assuming low number of apps will be using non-solid background color.
Assuming low number of apps will be deployed to 2.3 devices.


Create a Signal group for organisers and volunteers

What
Create a Signal group for the organisers and volunteers at Design System Day 2023.
Why
Allows the group to message each other on the day with urgent questions or updates.
Who needs to review this
@ImranH-GDS
Done when

[ ] The group is created
[ ] It has been shared in the dsday-23planning channel
[ ] It has been added to any team docs, inc. for volunteers

New members need approval but the link to join is here: https://signal.group/#CjQKIEXhsTzXeUFSjTSr9jNeOGTo3i24tcd7PXSt4nz2nU7JEhAoKbIEJtZRxk6IqgmkxxMm


feat(cli): add phase option to node set command

Checklist:

[X] My organization is added to USERS.md.

This adds the missing 'phase' command to the node cli command, eg:
argo node set my-wf --node-field-selector displayName=approve --phase Succeeded
would set the phase for the suspend node with displayName=approve to Succeeded

Needs codegen and commit signing

Needs codegen and commit signing

Fix for this is out now in https://github.com/argoproj/argo/releases/tag/v2.12.4

Fix for this is out now in https://github.com/argoproj/argo/releases/tag/v2.12.4


[BUG]kubedl test connection is Error and request 404

What happened:
I install with helm, the test connect is error for this:

it's logs is:
kubectl -nkubedl-system logs -f kubedl-test-connection
Connecting to kubedl:80 (<IP_ADDRESS>:80)
wget: server returned error: HTTP/1.1 404 Not Found

I look the kubedl log, it not any error:
kubectl -nkubedl-system logs -f kubedl-695dd84cb-hmtpq
I0818 03:26:47.631764       1 request.go:655] Throttling request took 1.046385496s, request: GET:https://<IP_ADDRESS>:443/apis/dex.coreos.com/v1?timeout=32s
2021-08-18T03:26:50.288Z        INFO    controller-runtime.metrics      metrics server is starting to listen    {"addr": ":8080"}
2021-08-18T03:26:50.288Z        INFO    setup   setting up scheme
2021-08-18T03:26:50.288Z        INFO    setup   setting up gang schedulers
I0818 03:26:50.288908       1 registry.go:35] register gang scheduler kube-batch
I0818 03:26:50.288923       1 registry.go:35] register gang scheduler kube-coscheduler
I0818 03:26:50.288926       1 registry.go:35] register gang scheduler volcano
I0818 03:26:50.290387       1 workload_gate.go:102] workload CRD Inference found.
I0818 03:26:50.290498       1 controllers.go:42] workload Inference controller has started.
I0818 03:26:50.291922       1 workload_gate.go:102] workload CRD TFJob found.
I0818 03:26:50.292072       1 controllers.go:42] workload TFJob controller has started.
I0818 03:26:50.293529       1 workload_gate.go:102] workload CRD XDLJob found.
I0818 03:26:50.293651       1 controllers.go:42] workload XDLJob controller has started.
I0818 03:26:50.294874       1 workload_gate.go:102] workload CRD XGBoostJob found.
I0818 03:26:50.294961       1 controllers.go:42] workload XGBoostJob controller has started.
I0818 03:26:50.296326       1 workload_gate.go:102] workload CRD Cron found.
I0818 03:26:50.297701       1 workload_gate.go:102] workload CRD TFJob found.
I0818 03:26:50.298970       1 workload_gate.go:102] workload CRD PyTorchJob found.
I0818 03:26:50.300316       1 workload_gate.go:102] workload CRD MarsJob found.
I0818 03:26:50.301841       1 workload_gate.go:102] workload CRD XGBoostJob found.
I0818 03:26:50.303139       1 workload_gate.go:102] workload CRD XDLJob found.
I0818 03:26:50.304395       1 workload_gate.go:102] workload CRD MPIJob found.
I0818 03:26:50.304415       1 controllers.go:42] workload Cron controller has started.
I0818 03:26:50.305554       1 workload_gate.go:102] workload CRD ModelVersion found.
I0818 03:26:50.305601       1 controllers.go:42] workload ModelVersion controller has started.
I0818 03:26:50.306918       1 workload_gate.go:102] workload CRD MPIJob found.
I0818 03:26:50.307010       1 controllers.go:42] workload MPIJob controller has started.
I0818 03:26:50.308384       1 workload_gate.go:102] workload CRD ElasticDLJob found.
I0818 03:26:50.308491       1 controllers.go:42] workload ElasticDLJob controller has started.
I0818 03:26:50.309858       1 workload_gate.go:102] workload CRD MarsJob found.
I0818 03:26:50.309966       1 controllers.go:42] workload MarsJob controller has started.
I0818 03:26:50.311211       1 workload_gate.go:102] workload CRD PyTorchJob found.
I0818 03:26:50.311294       1 controllers.go:42] workload PyTorchJob controller has started.
2021-08-18T03:26:50.311Z        INFO    setup   setting up storage backends
I0818 03:26:50.311339       1 registry.go:35] register new object backend: mysql
I0818 03:26:50.311352       1 registry.go:40] register new event backend: aliyun-sls
2021-08-18T03:26:50.311Z        INFO    setup   starting manager
I0818 03:26:50.311404       1 monitor.go:29] setting up client for monitoring default registry on port: 8443
I0818 03:26:50.311549       1 leaderelection.go:243] attempting to acquire leader lease kubedl-system/kubedl-election...
2021-08-18T03:26:50.311Z        INFO    controller-runtime.manager      starting metrics server {"path": "/metrics"}

What you expected to happen:
the connect test can run well.
How to reproduce it:
use helm to install helm in the master branch.
Anything else we need to know?:
Environment:

KubeDL version: kubedl/kubedl:daily
Kubernetes version (use kubectl version): Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.8-r1-CCE21.3.1.B003-19.38.2", GitCommit:"e796d381b893b3adcf99c50873f28fd6ef7cd7b1", GitTreeState:"clean", BuildDate:"2021-03-23T12:40:52Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}
OS (e.g: cat /etc/os-release): windows
Kernel (e.g. uname -a): windows
Install tools: helm and kubectl

@shikanon hi shikanon, it' caused by the generated post-install hook under /helm/kubedl/template/tests, which intends for testing service reachable, however kubedl does not provide services like web-applications, so it always failed, and we'd remove post-install hook.

@SimonCqk We can use /metrics instead of /， that can testing the service reachable, because I find the kubedl's monitoring is turned on by default.

@SimonCqk We can use /metrics instead of /， that can testing the service reachable, because I find the kubedl's monitoring is turned on by default.

good point! then we'd default values in .service.port  to proxy traffic to kubedl metrics port.


Message::delete(float $delay)

    /**
     * Deletes the message from the channel.
     *
     * @return \React\Promise\Promise
     */
    public function delete(?float $delay=0.0){
        $deferred = new Deferred();

        $this->discord->getLoop()->addTimer($delay,
        function () use($deferred){
//IDK bruh
                $this->http->delete("channels/{$this->channel_id}/messages/{$this->id}")->then(
            \React\Partial\bind([$deferred, 'resolve']),
            \React\Partial\bind([$deferred, 'reject'])
        );
        }
        );
        return $deferred->promise();
    }

Trying to follow the Part-Repository architecture so parts shouldn't be able to delete themselves, sorry.


Issue with Renaming Excel Export in IE/Edge

I'm currently running in to an issue with exporting data to Excel using the Buttons plugin in Internet Explorer. It's being caused by the limited options in the Save as type dropdown. When I attempt to export it, the default file name is the title of the page I'm on plus .xlsx; if I change nothing and just hit Save, it works. However, if I attempt to rename the file I have to manually add the extension because the Save as type is All Files (.) and there are no other options. So if a user who doesn't realize they have to add the file extension does this, it saves it as a plain File instead of a worksheet.
Compare this to Chrome that works perfectly because the Save as type is Microsoft Excel Worksheet, so it doesn't need the file extension added. Is this something that can be fixed in the code or is it a limitation of the browser?

Apologies if the request in the github new issue template and the contributing document aren't clear enough. Could you post support requests in the DataTables forums please, so that there is a single searchable point for support requests.
When you post in the forum could you link to a page showing the issue, even if that is on the DataTables' site (so I know the configuration being used). Also, if you could post a link to your question in the forum here, so if anyone does happen to find this one they know where to go.
Thanks,
Allan

Sorry about that, I've asked the question here: https://datatables.net/forums/discussion/34820/issue-with-renaming-excel-export-in-ie-edge?new=1


Icons are not arranged correctly is “Adopt me” page

Describe the bug
Icons are not arranged correctly is “Adopt me” page
To Reproduce

Go to Website
Click on ‘Meet Pets’
Click on ‘Adopt me’
We can see that icons are not placed perfectly

Expected behavior
Icons should be aligned properly with the input tag
Desktop (please complete the following information):

OS: MacOS
Browser: Chrome

Screenshots
Present:

Expected:

go for it


Update CORS to use allowed headers from yml

When using additional custom CORS headers in Serverless, the local serverless webpack serve command does not allow those headers when developing locally. This fixes that.

Is this a duplicate of #74?

@dolfelt  It would be great if you could comment in #135 if you use the serve functionality or not, so that we can decide how to proceed there.

serve will be removed in the next v3.0.0 release in favor of serverless-offline (see #152 ). No efforts will be spent on the function anymore. So I'll close this issue.


[ISSUE-408] Support submit to ray in console

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Error with 22.2 Object Based files (Deezer/Amazon Music)

Hi, I'm using your decoder for converting 360RA music files leaked on internet, especially files who apparently comes from Tidal (7.1.4) and works well, but I recently got 22.2 files who comes from Deezer & Amazon Music and I followed your instructions but I still keep getting error like these : https://github.com/ittiam-systems/libmpegh/issues/7, https://github.com/ittiam-systems/libmpegh/issues/23. I did the modifications on Visual Studio, rebranded the MP4 container of the files with MP4Box.

Hi @SonMaxime, The workaround suggested in #1 works well only for streams which satisfy some conditions. A possibility is that the streams for which You are facing decode failure may have configurations that are not supported by our decoder.
Thanks!

Do you plan on adding Level 4 MPEG-H decoding to your decoder in the future ?

I was able to decode 360 RA (HQ) files downloaded by deemix. Also 22.2 target layout, some have as many as 24 objects, also LC@4 I believe. Which track/album are you looking at?

I was able to decode 360 RA (HQ) files downloaded by deemix. Also 22.2 target layout, some have as many as 24 objects, also LC@4 I believe. Which track/album are you looking at?

Koi Ha Ryusei by Minako Yoshida https://bayfiles.com/R0r2Y0W6y1/06_Yume_De_Aetara_mp4

We will not be supporting this issue as the source of the streams mentioned here doesn't sound appropriate

I was able to decode 360 RA (HQ) files downloaded by deemix. Also 22.2 target layout, some have as many as 24 objects, also LC@4 I believe. Which track/album are you looking at?

Koi Ha Ryusei by Minako Yoshida https://bayfiles.com/R0r2Y0W6y1/06_Yume_De_Aetara_mp4

I actually can reproduce this on the current build (same behavior). However, I tried using a previous build (git clone) and it worked. It seems like I cloned the repository on 2022/10/13.


AJ-1005: Create Dockerfile to include postgres client for prod WDS

We need to invoke pg_dump in WDS during cloning of workspaces -- this Dockerfile allows us to include the Postgres client -- it extends from app-sec blessed code here: https://github.com/broadinstitute/dsp-appsec-blessed-images/blob/main/jre/Dockerfile.17-alpine
Reminder:
All PRs merged into main will generate a PR in https://github.com/broadinstitute/cromwhelm to update the WDS image deployed to kubernetes.
After your merge, you must go to this repo and approve this generated PR and make sure it merges.
That merge will then generate a PR in https://github.com/DataBiosphere/leonardo.  This may or may not automerge; be sure to watch it to ensure it merges.

LGTM!

looks good, wonder if you want to pin the postgres-client version as you had in a previous commit, but I wouldn't hold the PR for that

I realized that our postgres images in k8s-deployed-via-Leo apps are pinned to latest, so I figured here I'd continue the client to be lock-step with it, thus not pinning.
I agree we should pin both though -- I'd message dsp-azure for now.


Introduce SystemSchema tables (#5989)

SYSTEM_TABLES provide visibility into the druid segments, servers and tasks.

SEGMENTS table provides details on served and published segments
SERVERS table provides details on data servers
SERVERSEGMETS table is the JOIN of SEGMENTS and SERVERS
TASKS table provides details on tasks from overlord

Sample sql queries to retrieve data from these tables:
SELECT * FROM sys.segments where datasource='wikipedia';
SELECT * FROM sys.servers;
SELECT * FROM sys.tasks where status='FAILED';

@surekhasaharan can you fix the merge conflicts?

Reviewed the core SystemSchema class, will do a pass to review the other files

@surekhasaharan can you merge with master to make it easier for UT to pass?

@surekhasaharan I think this is failing UT

@surekhasaharan I think this is failing UT

Yes, will check that.

@jihoonson any more comments?

@fjy reviewing now.

Thanks so much @jihoonson for the review. I've addressed your comments, let me know if you have any further comments.

Spoke with @gianm about if he has any further comments and he does not so I am moving forward with merging.


Client Allocation Request for: aoeu

Client Allocation Questions
Core Information

Name: aoeu
Website / Social Media: aoeu, (aoeu, aoeu)
Region: Africa
DataCap Requested: 32 GiB
Addresses to be Notarized: aoeu
Notary Requested: Simon

This user’s identity has been verified through filplus.storage


[WIP] overriden binary operations on objects derived of derived classes

This PR needs discussions. To fix the issue #436 ,I have tried to use ast.NodeTransformer to convert all BinOp nodes to respective Call nodes . For Example , a*4 BinOp will be converted to a.__mul__(4)
This is a temporary fix. I am not sure if this is the way we're to supposed to tackle the issue. @freakboy3742  @eliasdorneles @cflee

Closing this PR. @freakboy3742's reviewed and pointed out that this not the right way to fix this issue.


Make Target drawable methods nullable as per the Picasso Javadoc

Going to a repository page crashes the app and this fixes it.
Picasso hadn't added @nullable annotations to their Target methods, so Kotlin crashes the app as null is being passed to a method which requires non nullability. I've sent a PR to Picasso in attempt to fix this issue for future uses of Picasso#Target (square/picasso#1825).

I'm. Xf
On Mar 11, 2018 14:34, "Veyndan Stuart"<EMAIL_ADDRESS>wrote:
Going to a repository page crashes the app and this fixes it.
Picasso hadn't added @nullable annotations to their Target methods, so
Kotlin crashes the app as null is being passed to a method which requires
non nullability. I've sent a PR to Picasso in attempt to fix this issue for
future uses of Picasso#Target (square/picasso#1825
https://github.com/square/picasso/pull/1825).
You can view, comment on, or merge this pull request online at:
https://github.com/pockethub/PocketHub/pull/1185
Commit Summary

Make Target drawable methods nullable as per the Picasso Javadoc

File Changes

M app/src/main/java/com/github/pockethub/android/util/AvatarLoader.kt
https://github.com/pockethub/PocketHub/pull/1185/files#diff-0 (12)

Patch Links:

https://github.com/pockethub/PocketHub/pull/1185.patch
https://github.com/pockethub/PocketHub/pull/1185.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pockethub/PocketHub/pull/1185, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGUtqscnuu8SBJ1an9XwzPcstfhlA3nks5tdWAhgaJpZM4SlxzR
.


semver.VersionInfo allows negative numbers

Situation
If you pass negative numbers into semver.VersionInfo it doesn't raise any errors:
>>> import semver
>>> semver.VersionInfo(-2)
VersionInfo(major=-2, minor=0, patch=0, prerelease=None, build=None)

Tested with latest 2.11.0
Proposal
I guess, this shouldn't happen. We should raise a ValueError exception.
Would integrate that into a 2.12.0 release and into the upcoming 3.0.0-alpha0.
@tlaferriere Thomas, can you conform? Do you agree?

I agree. It is doesn't make sense to have negative version numbers, whichever way you put it.

Hi all,
according to https://semver.org/

A normal version number MUST take the form X.Y.Z where X, Y, and Z are non-negative integers, ...

So according spec, negative integers are not allowed.

Thanks to both of you for confirming. It's surprising that this bug could survive that long. :smile:


Could not setup PHP 7.0 on Ubuntu

Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
Ubuntu
PHP versions
7.0
To Reproduce
https://github.com/zephir-lang/php-zephir-parser/runs/5169829161?check_suite_focus=true

@Jeckerson Please try restarting the workflow, must be a network issue in fetching the build.


Malformed calls from JS : field Sizes are different

Hello,
I have a JSON array structured more or less like that :
[
    {
       label: "Can you upload a photo of a document?",
       id: 0
    },
    {
       label: "Can you upload a photo of a second ID document?",
       id: 1
    },
    ...
]

If I'm using a simple FlatList to render the item's array, such like that for example :
<FlatList
      data={signUpPreciseInfos}
      renderItem={({ item }) => this.renderScene(item)}
      keyExtractor={item => item.id}/>

It's working well, however, if I'm using KeyboardAwareFlatList :
 <KeyboardAwareFlatList
                  enableOnAndroid={true}
                  innerRef={item => { this.scroll = item }}
                  getItemLayout={(data, index) => ({ length: 90, index })}
                  data={signUpPreciseInfos}
                  contentContainerStyle={{ marginHorizontal: Metrics.horizontalPadding }}
                  ref={(item) => this.flatList = item}
                  renderItem={({ item }) => this.renderScene(item)}
                  keyExtractor={item => item.id}
              />

it's throwing this screen :

Can't figure it why it's doing that. BUT ⚠very important ⚠KeyboardAwareFlatList is actually WORKING if I set the length of the array to 10 items max. If I'm adding  more that 10 items, then the error occured...
Any ideas why it's doing that ?
Thanks for the help

^ same question, i'm having this problem too. insane that 2 years later it's still an issue


 SyntaxError: Cannot use import statement outside a module when running tests with Jest

When running recoil nexus inside jest tests, the test fails to run and the following message is displayed.
Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/en/ecmascript-modules for how to enable it.
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /Users/antonymitchell/aems-client/node_modules/recoil-nexus/build/RecoilNexus.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){import { useRecoilCallback } from 'recoil';
                                                                                ^^^^^^

    SyntaxError: Cannot use import statement outside a module

      1 | /* eslint-disable camelcase */
      2 | import axios, { AxiosResponse } from "axios"
    > 3 | import { getRecoil, setRecoil } from "recoil-nexus"
    ```

Hi, perhaps same as #12 ?

That seems to be the case, my app was created using create-react-app which uses a jestSetup.js, rather than jest.config.js. If it works there, I'll report back.
When I tried googling this, I didn't manage to findt#12. So I'd be happy to raise a PR adding a section discussing how to add this to the readme.md, if you'd like?

So I've had a look, and this seems to be the same issue as #12. However I'm using create-react-app, we put the line of code in a different place. The issue can be sorted by putting the following in setupTests.js in a create-react-app application.
const config = {
    transformIgnorePatterns: []
}

export default config

alternatively we can add this to the top level of the package.json
  "jest": {
"transformIgnorePatterns": []

  },


Update reports with most recent data for Cytomining Ecosystem

This PR updates the data and reports for the Cytomining Ecosystem by running the poethepoet workflow cytomining_ecosystem_fall_2023. This adds new data for all reports and removes IDR_Stream from the earlier landscape analysis report.
Small changes were also made to the targeted visualization notebook and report for new publication counts for pycytominer, and the landscape visualization notebook + report for invalid HTML title length.
Thank you for any thoughts and feedback you may have! Please don't hesitate to let me know if you have any questions.

Thank you @gwaybio !


add Step block on goal

What does your feature do?
add a "click / use a block" functionality in a quest step like the "starting block" function so that players can "look for / search" the blocks during the steps
because the function "to go to a radius" around a block is not suitable for research or excavation during exploration for example
How would people use it?
the players doing the quest

Thanks for the suggestion. It appears there was an error in the Wiki page description for this, so let me clarify:

In the Quests Editor, selection 5 - Set block start is for right-clicking a chosen block to start the quest, while 6 - Set region requires the player to stand in a particular area (as small as 1x1) if WorldGuard is installed.

Thanks for the suggestion. It appears there was an error in the Wiki page description for this, so let me clarify:

In the Quests Editor, selection 5 - Set block start is for right-clicking a chosen block to start the quest, while 6 - Set region requires the player to stand in a particular area (as small as 1x1) if WorldGuard is installed.


Added version in init.py

hey @rosaboyle, you can use the get_version method that is already exported instead of accessing __version__ directly.

Why not use the standard procedure everyone uses?
On Sun, Dec 15, 2024 at 12:43 PM Derich Pacheco @.***>
wrote:

Closed #126 https://github.com/resend/resend-python/pull/126.
—
Reply to this email directly, view it on GitHub
https://github.com/resend/resend-python/pull/126#event-15661614097, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZX2ZR73QIB4REHHWPHFQUT2FW5VNAVCNFSM6AAAAABR65DL6KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGY3DCNRRGQYDSNY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rosaboyle you actually have a point and I changed my mind 👍

👍
On Sun, Dec 15, 2024 at 1:39 PM Derich Pacheco @.***>
wrote:

Merged #126 https://github.com/resend/resend-python/pull/126 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/resend/resend-python/pull/126#event-15661750565, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZX2ZR37WTXZ7LECL4HTIFL2FXEH3AVCNFSM6AAAAABR65DL6KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGY3DCNZVGA2TMNI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Implement an homogenous reactor with scipy.ode.ivp and Jacobian Matrix

Ref.
https://cantera.org/examples/python/reactors/custom.py.html
#1

It is now implemented in the auto-ignition case, but it seems that it shows no good for small size of reaction mechanism :) Anyway, will look into the details in the future.

Evaluation Jacobian using AD would be very slow. So that we disable it for now.


"Double Dijkstra monads" seem to be no longer used

The online tutorial says that F* uses "double Dijkstra monads", tracking both the weakest precondition and the weakest liberal precondition.
However, looking at source code such as ulib/prims.fst, I suppose that F* does not use double Dijkstra monads any longer.
Shouldn't the document be fixed?

Thanks, yes indeed. This part of the tutorial is stale. Thanks for reporting

Thanks for pointing this out, I was in the middle of reading that section of the tutorial. It seems like weakest liberal preconditions are not used anymore.
I was looking at the code history and found some relevant commits (mainly for my reference):

Introduces WLPs in the tutorial sec. 9.3: e6c592525fbdb312e74fa13ba11a01bea4bee616
NewPrims without WLPs: 4673818bf00f871f891b878a860ea87d6a3bbcb5

PR #2123 tries to quickly remove any mentioning of WLPs in the tutorial, and unless anyone objects I'll merge that soon.


Update Parquet best practice

This was pretty old.  This shifts the language to think more about speed/storage/consistency tradeoffs rather than ease of parallel loading (which was true regardless).  We also drop the mention to Impala (somewhat dated).

I wonder if it's also worth linking out to the dedicated parquet page here

I think we link to that page at the end of this section

Thank you for resolving the numbering issue.  Assuming that we're ok with not talking about engines and that we're ok with me pulling numbers out of my brain, are we good here?


swiftformat: Fix build

Description of changes
On Macs with XCode 14, swiftformat builds no longer work because the value for 'no code sign identity' was changed from an empty string to a dash.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 249379 --checkout commit run on aarch64-darwin 1

This seems to fail on modern macOS due to trying to write to the (non-existent) home directory.
       > Build Preparation
       > Couldn't create workspace arena folder '/var/empty/Library/Developer/Xcode/DerivedData/SwiftFormat-hajpfumuugmdlbdnzikitboqvztg': You don’t have permission to save the file “SwiftFormat-hajpfumuugmdlbdnzikitboqvztg” in the folder “DerivedData”.
       >
       > ** BUILD FAILED **
       >
       >


CSwin Code for Segmentation with MMSegmentation

Hi,
Thank you for your work!
I wonder if you plan to release the mmsegmentation code you used for the downstream segmentation task, just like in Swin-Transformer repo.
Best,

code and models released, please create a new issue for any new problems.


feat: enable uniform timestamps on discord

closes #9
open questions:

should this support forum threads in the overview? (the cards on pages like this). it's possible, but I did not want to waste time on an unnecessary feature.
is the base Fixer class an improvement?

Thanks for the PR

Let's keep it out for now, worst case you can still navigate to the 1st forum message, and if there's interest we can do it.
Fixer looks very promising! and helpful for keeping the fixers clean, I like how it also addresses the tiktok fixer where there will be multiple targets.


Smoke tests

Description

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Codecov Report

Merging #1575 (73bfcf3) into master (b6cfd84) will decrease coverage by 86.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1575       +/-   ##
==========================================
- Coverage   89.73%   3.42%   -86.32%
==========================================
  Files         388     388
  Lines        8317   12684     +4367
  Branches     1218     223      -995
==========================================
- Hits         7463     434     -7029
- Misses        679   12249    +11570
+ Partials      175       1      -174

Impacted Files
Coverage Δ

app/models/role.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/team.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/alert.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/contact.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/country.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/product.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/accident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/incident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/location.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/alert_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 349 more

Continue to review full report at Codecov.

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


Change LETKF ctests to use vertical remapping

Description
This is a follow up of #317. Sorry for not reviewing this PR, but while the letkf test is technically OK, the members need to be mapped into the same vertical coordinate.
To Reproduce:
LETKF test.
Output
member 2 - member 1, layer thickness (layer 5):

:point_up: "no good ..."

Closing due to lack of activity.


bring IBC consensusStates into IBC key spec

in building the relayer, discovered that our state storage key for consensusStates was out of spec

Should we be using the state key pattern here?

Should we be using the state key pattern here?

Yeah, sounds good — that would be a good refactor to do separately from this issue!


JSX: Used import does not generate a dependency

Consider the following class:
export class MyComponent {
  render() {}
}

If you use it :
import {MyComponent} from "./valuebasedUsage";
var foo = <MyComponent/>

emit:
var foo = React.createElement(valuebasedUsage_1.MyComponent, null);

The code generated does not contain the emit for import {MyComponent} from "./valuebasedUsage";
/cc @RyanCavanaugh
Refs : https://github.com/TypeStrong/atom-typescript/issues/442

What commit are you on? Should have been fixed in #3675; I can't repro using current bits.

I can't repro using current bits.

Sorry. I should have tried latest :rose: fixed indeed.

Is this fixed in 1.6.2? I have a line that says:
import { Dock } from 'react-dock';

And the generated code is:
var react_dock_1 = require('react-dock');


feat(compile): add args attribute

This allows passing the index url and other parameters to uv via
a string list instead of using env variables. This means that the
we don't need to resort to direnv or .bazelrc to change the
indexes used by uv.

Landed configurability in #55


can't run the UI

After this command "python app.py" the following message show up.
D:\AI\OmniGen\app.py:229: SyntaxWarning: invalid escape sequence '\/'
  "<img><|image_1|><\/img> What item can be used to see the current time? Please highlight it in blue.",
D:\AI\OmniGen\app.py:271: SyntaxWarning: invalid escape sequence '\<'
  description = """
Traceback (most recent call last):
  File "D:\AI\OmniGen\app.py", line 9, in <module>
    from OmniGen import OmniGenPipeline
  File "D:\AI\OmniGen\OmniGen\__init__.py", line 1, in <module>
    from .model import OmniGen
  File "D:\AI\OmniGen\OmniGen\model.py", line 3, in <module>
    import torch
ModuleNotFoundError: No module named 'torch'

Please follow the instructions provided below (in the README.md file) and install all required libraries before running the demo. The error you get simply means that the required library torch is not installed or not installed properly. If you have follow the installation instructions before, please uninstall torch (via pip or conda, depending on how you did it) and reinstall it again to fix improper installation.
https://github.com/VectorSpaceLab/OmniGen?tab=readme-ov-file#5-quick-start

After follow the exact step in the link provided, after I enter "python app.py" this is showing
Traceback (most recent call last):
  File "D:\AI\OmniGen\app.py", line 9, in <module>
    from OmniGen import OmniGenPipeline
  File "D:\AI\OmniGen\OmniGen\__init__.py", line 1, in <module>
    from .model import OmniGen
  File "D:\AI\OmniGen\OmniGen\model.py", line 9, in <module>
    from diffusers.loaders import PeftAdapterMixin
ModuleNotFoundError: No module named 'diffusers'


Added populating next right pointers in each node.

Coverage remained the same at 100.0% when pulling 77d96baae6e0f5b3da60b9ccf64237f531d290fc on populating-next-right-pointers-in-each-node into 244a8dd69ce3f4b34eb7e7bb0d879c0ade648e79 on master.


cannot save anymore

Just like that, I stopped being able to export my figures and I get the following error in
Ghostscript error: perhaps /private/var/folders/zs/6cx0l9h149sfqrhss15r48300000gp/T/tpbe8b45fb_bc20_457b_a729_bdf1b91739cd.pdf is open by another application
or maybe you have another gs executable in your system's path
Ghostscript options: -q -dNOPAUSE -dBATCH -sDEVICE=pdfwrite -dPDFSETTINGS=/prepress -sOutputFile="/private/var/folders/zs/6cx0l9h149sfqrhss15r48300000gp/T/tpbe8b45fb_bc20_457b_a729_bdf1b91739cd.pdf" -dEPSCrop -sFONTPATH="/usr/share/fonts:/usr/local/share/fonts:/usr/share/fonts/X11:/usr/local/share/fonts/X11:/usr/share/fonts/truetype:/usr/local/share/fonts/truetype" -f "/private/var/folders/zs/6cx0l9h149sfqrhss15r48300000gp/T/tpf527a7a5_cb9c_47ef_b862_5bec9b6a5d22.eps"
export_fig error. Please ensure:
that you are using the latest version of export_fig
and that you have Ghostscript installed
and that you do not have multiple versions of export_fig installed by mistake
and that you did not made a mistake in the expected input arguments
If the problem persists, then please report a new issue.
Error using eps2pdf (line 173)
GPL Ghostscript 9.19: **** Could not open the file
/private/var/folders/zs/6cx0l9h149sfqrhss15r48300000gp/T/tpbe8b45fb_bc20_457b_a729_bdf1b91739cd.pdf .
**** Unable to open the initial device, quitting.
Error in export_fig (line 779)
eps2pdf(tmp_nam, pdf_nam_tmp, 1, options.append, options.colourspace==2, options.quality,
options.gs_options);
'export_fig' not found.
/Users/olawork/Documents/MATLAB/SuperRes/sim/Sanghy2/altmany-export_fig-4282d74/export_fig.m

It seems that you either do not have full read/write in your /private/var/folders/zs/6cx0l9h149sfqrhss15r48300000gp/T/ folder, and/or you have an invalid installation of export_fig (this would explain the last error, that export_fig was not found).
Also, the error message indicates that you're using a relatively old version of export_fig - try to install and use the latest version.


[Event Request] table 27 Item - OnValidate Trigger Type

Describe the request
Hello, can we get a new event to be able to skip this check.
 field(10; Type; Enum "Item Type")
        {
            Caption = 'Type';
            trigger OnValidate()
        //>>>>>>>>>>>>>>>>>>>>>>>>>
            var
                IsHandled: Boolean;
            begin
              IsHandled := false;
                OnBeforExistsItemLedgerEntry(Rec, xRec, CurrFieldNo, IsHandled);
                If not IsHandled then
                       if ExistsItemLedgerEntry() then
                          Error(CannotChangeFieldErr, FieldCaption(Type), TableCaption(), "No.", ItemLedgEntryTableCaptionTxt);
        //<<<<<<<<<<<<<<<<<<<<<<<<<
                TestNoWhseEntriesExist(FieldCaption(Type));
                CheckJournalsAndWorksheets(FieldNo(Type));
                CheckDocuments(FieldNo(Type));
                if IsNonInventoriableType() then
                    CheckUpdateFieldsForNonInventoriableItem();
            end;
        }

    //>>>>>>>>>>>>>>>>>>>>>>>>>
    [IntegrationEvent(false, false)]
    local procedure OnBeforExistsItemLedgerEntry(var Item: Record Item; xItem: Record Item; CallingFieldNo: Integer; var IsHandled: Boolean)
    begin
    end;
    //<<<<<<<<<<<<<<<<<<<<<<<<<

Additional context
Need this event to be able to skip this procedure when type is validated.Internal work item: AB#493704

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14518.


Signature error with exceptval

Just upgraded to 0.27.1, psyched to try out @cython.exceptval.  Tried it out on pandas._libs.tslib._Timestamp._compare_outside_nanorange:
Existing:
    cdef bint _compare_outside_nanorange(_Timestamp self, datetime other,
                                         int op) except -1:

New:
    @cython.exceptval(-1)
    @cython.locals(self=_Timestamp, other=datetime, op=int)
    @cython.returns(cython.bint)
    @cython.cfunc
    def _compare_outside_nanorange(self, other, op):

build_ext complained that _compare_outside_nanorange is not declared in the pxd file, so I added it:
cdef class _Timestamp(datetime):
    cdef bint _compare_outside_nanorange(_Timestamp self, datetime other, int op) except -1

build_ext now produces the error message Signature not compatible with previous declaration.  AFAICT the signatures should be compatible; am I missing something?

A couple of remarks:
Since you have a .pxd, you don't need the Decorators at all. Well, not in a .py file, at least. Not sure about .pyx.
There's no use in typing "self" since its type is obvious.
@cython.locals() does not change the signature. That's a known limitation, can't look up the ticket right now.

Since you have a .pxd, you don't need the Decorators at all. Well, not in a .py file, at least. Not sure about .pyx.

This is in a pyx.  The pxd file contains only the small subset of functions exported.  In fact, under the status quo, the _Timestamp class doesn't exist in the pxd at all.

There's no use in typing "self" since its type is obvious.

Makes sense, thanks.

@cython.locals() does not change the signature. That's a known limitation, can't look up the ticket right now.

I'm not sure what to make of this.  Elsewhere I've been using (in .pyx files)
@cython.locals(foo=bar)
@cython.cfunc
def somefunc(foo):
    [...]

interchangeably with
cdef somefunc(bar foo):
    [...]

In some of these cases I haven't bothered putting somefunc in the accompanying pxd file, in others it would just be cdef somefunc(bar foo).  Have I been using this incorrectly?

May I ask why you seem to prefer the rather verbose decorator syntax over the (IMHO) much more concise dedicated Cython syntax in .pyx files?
If I were to start using a more Python-compatible syntax in .pyx files, I'd certainly prefer the PEP-484 syntax over @locals() and @returns().

A big part of it at this point is just trying to understand what's going on.
Until very recently I was having no luck getting coverage working in non-trivial cython code, so the main motivation was getting pure-python compatibility in order to get coverage statistics.  (other tooling e.g. flake8 was/is part of the motivation, but a much lower priority).

I'd certainly prefer the PEP-484

I'm largely working on pandas and statsmodels, which still support py2.

"I'd certainly prefer the PEP-484 syntax"
I'm largely working on pandas and statsmodels, which still support py2.

Question is, do they have to support it uncompiled? If it's enough to serve Py2 with Cython compiled modules, you can happily use Py3 syntax in them (and compile them with language_level=3).


influxdb-writer doesn't use the correct host

Trying to follow this tutorial : https://medium.com/mainflux-iot-platform/how-to-connect-raspberry-pi-to-mainflux-fbef303e480c, I have noticed that, in addition to this issue https://github.com/Mainflux/mainflux/issues/345, the host declaration for influxdb-writer is not correct.
I have changed this line : https://github.com/mainflux/mainflux/blob/master/docker/addons/influxdb/docker-compose.yml#L35
--- MF_INFLUX_WRITER_DB_HOST: influxdb
+++ MF_INFLUX_WRITER_DB_HOST: mainflux-influxdb

Dunno if it is correct, but at least, it works now.

@ks156 Thank you for your effort. We can confirm this behavior and this issue will be resolved as soon as we resolve #345. It is correct, you should use mainflux-influxdb here.


Add hook sample for transformers v4.43

fix the issue https://github.com/openvinotoolkit/openvino.genai/issues/709

@eaidova https://github.com/openvinotoolkit/openvino.genai/pull/801/commits/33a34c219dcf9ae87fcce854ed8fe1999e36a475#diff-2f690f6027653d8401edfa436d44a4f6a379874c33f5d557f09d72ae6a1c2a3aR152-R155
Since this .clone() causes up to 33% 2nd token latency regression on SPR for most weekly models (llama-3-8b, llama-2-7b, qwen2-7b, chatglm3-6b etc.).
We have replaced  .clone() with reference as it is in the old version of _sample(), do you see any risk?

@peterchen-intel no, I do not see any risks for that, reason of addition clone is not applicable for us, but it makes uncomparable perf with original optimum and llm bench (as issue still will be visible for model without hook

@wgzintel @peterchen-intel do you have plan to fix sample code (with clone/without clone)? Looks like initial issue still reproducible
@eaidova  Will be fixed without changing transformers _sample() code except adding time profiling.

Documentation on OpenVINO https://github.com/openvinotoolkit/openvino/pull/26950


[rustls] identity: handle elliptic curve private keys

This commit improves the user experience of the users leveraging rustls.
When building the identity to use for requests, it's now possible to use also private keys using Elliptic Curve that are encoded with SEC1 standard.
This is the default encoding used by openssl when EC keys are created. Prior to this commit users had to convert those keys from the SEC1 encoding to the PKCS 8 one, which is something not trivial.
This feature has been recently added to rustls-pemfile crate. This commit update the rustls-pemfile crate to its latest stable release.
I also had to change a little the code used to extract the private keys from the PEM file. That's because rustls-pemfile doens't provide a high level API that extracts Elliptic Curve keys.

Thanks for the PR! I'm generally pro on this addition. I'm starting to worry about a bunch of this TLS code that doesn't really have tests, but that's not the fault of this PR. More just me worrying... Anyways, I think this just needs a rebase, since I already merged a different PR that updated the dependency version.

Thanks for the PR! I'm generally pro on this addition. I'm starting to worry about a bunch of this TLS code that doesn't really have tests, but that's not the fault of this PR. More just me worrying...

I get it... thanks for being positive about this PR

Anyways, I think this just needs a rebase, since I already merged a different PR that updated the dependency version.

Done


Metadata object is share across different clusters and hosts

Signed-off-by: tianqian.zyf<EMAIL_ADDRESS>Currently, our host metadata use map<string, google.protobuf.struct > type, because of the use of google.protobuf.struct, which contains a lot of type reflection information, which takes up more memory space. Even without any metadata information, an empty object would take up 144 bytes.
This PR does the following two optimizations:

Metadata objects are shared across clusters and hosts
Optimize the empty value for existing metadata

Description:
Risk Level: low
Testing: existing tests
Docs Changes: N/A
Release Notes: N/A
[Optional Fixes #Issue] #9532
[Optional Deprecated:]

The results of the actual test were excellent. The 100W host did not come with any metadata information, occupying about 1.3g, about 1.9g with metadata, and reduced to the only 960M after optimization

The core implementation is ObjectSharedPool, and the rest of the code is mostly interface changes.

/retest

@asraa  @zuercher ping you

@zuercher @asraa Is there any other issue for this pr to review?

sudo ./ci/run_envoy_docker.sh ./ci/do_ci.sh bazel.tsan //test/common/shared_pool/... --runs_per_test=100
Target //test/common/shared_pool:shared_pool_test up-to-date:
  bazel-bin/test/common/shared_pool/shared_pool_test
INFO: Elapsed time: 82.990s, Critical Path: 26.80s
INFO: 100 processes: 100 processwrapper-sandbox.
INFO: Build completed successfully, 101 total actions
//test/common/shared_pool:shared_pool_test                               PASSED in 2.6s
  Stats over 100 runs: max = 2.6s, min = 1.6s, avg = 2.4s, dev = 0.2s

Once all of the tests use the real dispatcher and real threads, can you make sure to test a TSAN build with runs_per_test=1000 to check for flakes? Thank you!

Thanks a lot, tsan did find several thread safety issues in the test code.

sudo ./ci/run_envoy_docker.sh ./ci/do_ci.sh bazel.tsan //test/common/shared_pool/... --runs_per_test=1000
Target //test/common/shared_pool:shared_pool_test up-to-date:
  bazel-bin/test/common/shared_pool/shared_pool_test
INFO: Elapsed time: 162.931s, Critical Path: 5.18s
INFO: 1000 processes: 1000 processwrapper-sandbox.
INFO: Build completed successfully, 1001 total actions
//test/common/shared_pool:shared_pool_test                               PASSED in 3.4s
  Stats over 1000 runs: max = 3.4s, min = 1.9s, avg = 2.4s, dev = 0.1s

@htuch Thank you for your valuable comments

@htuch  I am sorry for the late reply, Github don't seem to have informed me


Created a number pattern program in cpp

A number pattern program which takes rows as input and print the pattern in the pyramid form.

Follow all the rules in Readme and too small commit @rudraparmar76


⚠ ClamAV has degraded performance

In a60b255, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 251 ms

Resolved: ClamAV performance has improved in bbfc3d6 after 6 minutes.


Blog Component Designs Needed

No blog Components currently present.
A blog Component UI/UX design Needed which can further be used as to develop the blog component of bookshlf.
Designs can be made Using Figma or any other UI/UX Tools.
Both Small and Large Screen are needed.
Designs should contain these
- Recent blogs
- Search blogs
- Write new Blog

No blog Components currently present.
A blog Component UI/UX design Needed which can further be used as to develop the blog component of bookshlf.
Designs can be made Using Figma or any other UI/UX Tools.
Both Small and Large Screen are needed.
Designs should contain these

Recent blogs
Search blogs
Write new Blog

Hey @Mrhb787 I would like to work on this issue can you assign this to me?

@rockingrohit9639 I have assigned this to you

Thank you.

@rockingrohit9639 Any Updates ?

Hey! Can I work on this issue as well?


fix: Add empty map to execute_command_configuration to avoid plugin crash from interface conversion

Description

Add empty map to execute_command_configuration to avoid plugin crash from interface conversion

Motivation and Context

Closes #61

Breaking Changes

No

How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 4.0.1 :tada:


For example:
using Markdown: Markdown, MD

md = MD(Markdown.Code("julia", "gate_concurrence(U) = gate_concurrence(weyl_chamber_coordinates(U)...)"))

will print
gate_concurrence(U) = gate_concurrence(weyl_chamber_coordinates(U)...

Notice that there is a parenthesis missing at the end.
For more details, see the notebook at https://gist.github.com/goerz/bb2f1589d2aa523ffa8ccd4d0addcd86
It is not clear to me whether this is a bug in IJulia or a bug in Jupyter. Please advise if I should report this elsewhere.

If you're submitting a bug report, be sure to include as much relevant information as
possible, including:

The output of versioninfo()
How you installed Julia

Julia was installed via juliaup, running on macOS. Jupyter was installed through CondaForge (completely independent of Julia installation). Running JupyterLab 4.0.8.
Output of versioninfo() is
Julia Version 1.10.2
Commit bd47eca2c8a (2024-03-01 10:14 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: macOS (arm64-apple-darwin22.4.0)
  CPU: 10 × Apple M1 Pro
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-15.0.7 (ORCJIT, apple-m1)
Threads: 8 default, 0 interactive, 4 GC (on 8 virtual cores)
Environment:
  JULIA_CONDAPKG_BACKEND = Null
  JULIA_NUM_THREADS = auto

Also reported as https://github.com/jupyterlab/jupyterlab/issues/16112

For more details, see the notebook at https://gist.github.com/goerz/bb2f1589d2aa523ffa8ccd4d0addcd86

Huh, it seems like this issue does appear in the renderer used by Github, or NBViewer! Thus, I suppose this has to be an issue in the frontend, which most likely puts the problem with JupyterLab.
Here is a screenshot from JupyterLab running on my laptop, showing the problem:


[Communication] - SMS - Managed Identity Support for SMS Client

Draft for now: Blocked on running the Live tests for Managed Identity support for SMS. Waiting for service team to enable this functionality and should be available after Jan 15th.
SMS also has some additional complexity because it uses static resources in the live tests. For now, I have skipped the implementation of new live tests for managed identity until we have a solution going forward.

For live tests, sync up with Jorge.. my understanding is that we can still record and do the playback tests while keeping the live tests disabled.. I have asked him to schedule a follow-up to brainstorm the possible options for eventually enabling them

@ankitarorabit Just checked with Jorge. We cannot record tests for AAD right now because that Prague team is blocking us. We will be able to record and do playback tests after we are unblocked.

For live tests, sync up with Jorge.. my understanding is that we can still record and do the playback tests while keeping the live tests disabled.. I have asked him to schedule a follow-up to brainstorm the possible options for eventually enabling them

@ankitarorabit Just checked with Jorge. We cannot record tests for AAD right now because that Prague team is blocking us. We will be able to record and do playback tests after we are unblocked.

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests

/azp run net - communication - tests


Restore musllinux builds

Building musllinux wheels was blowing up because it couldn't find the go compiler - setup-linux.sh seems to be running successfully in that container, though.
Investigate why, and fix it.

This is apparently impossible at this time: https://github.com/golang/go/issues/13492
Building c-shared objects with cgo and musl doesn't currently work.


E   pyarrow.lib.ArrowNotImplementedError: Function 'not_equal' has no kernel matching input types (string, double)

pyarrow/error.pxi:91: ArrowNotImplementedError

looks like it's got something to do with pyarrow.
Reverting to ixmp4 version 0.6.0 fixed the test.
Just talked to @meksor and he said that you, @glatterf42, would be the best person to take a look.
FYI @danielhuppmann.

I'm not sure this is ixmp4's fault. My interpretation is that
/opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/site-packages/ixmp4/data/db/meta/repository.py:194: in bulk_upsert
    super().bulk_upsert(type_df)
/opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/site-packages/ixmp4/data/db/base.py:376: in bulk_upsert
    self.bulk_upsert_chunk(df)
/opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/site-packages/ixmp4/data/db/base.py:394: in bulk_upsert_chunk
    cond.append(df[col] != df[updated_col])

and
 E   pyarrow.lib.ArrowNotImplementedError: Function 'not_equal' has no kernel matching input types (string, double)

pyarrow/error.pxi:91: ArrowNotImplementedError

indicate that df[col] and df[updated_col] don't have compatible types. This could be on ixmp4 for trying to update the wrong column (which seems odd) or it could be on the pyam test setup for trying to pass an incompatible type.
I'll have to install pyam and run the tests myself to inspect test_df more closely, I think.

Thanks a lot for the detective work @glatterf42.

A dataframe like this converted to a dict like this.

Im running it locally with pandas 2.2.1 and the newest ixmp4 version...

Ok update, if I install pyarrow /alongside/ ixmp4, the tests in ixmp4 also fail... Seems pandas uses pyarrow if its available, breaking this test....

OK so pandas version 1.5.3 still works. Seems pandas version >2 will change its behaviour if pyarrow is installed. converting the columns to each other's types just yields another pyarrow error.
I would suggest reverting the update to pandas 2, seems a bunch of stuff broke...

We bumped pyam to depend on pandas >= 2.0 a while ago to take advantage of the fast speed and improved API, so pinning ixmp4<2 isn't really an option...

It's happening in df = self.merge_existing(df, existing_df) already, which makes me think this could be related to dask instead of pandas.
https://github.com/dask/dask/issues/10631 might be related.

fixed with https://github.com/iiasa/ixmp4/releases/tag/v0.7.1


[REQUEST]  BranchMenuView in staus-bar to select a branch

Description of issue:
In the status-bar of Atom the current branch is mentioned. With active github package I can click on this button and select another branch I would like to checkout. With git-plus only (without the core package github) it is not possible to checkout another branch in this way.
I would suggest to implement such a similar tooltip menu like the BranchMenuView in the core package github:

Until about July, clicking on the branch was the equivalent of the git-plus:checkout command but I removed it since the github package was doing the same thing. It does make sense to re-enable the feature though.

Thanks! I would like it very much, because I do not use github and therefore I deactivated the core package...

+ò-

Fixed in the 8.3.0


REI Oversights

Describe the Bug
For splashing (and possibly other fan processing types as well) the output percentage does not display in the tooltip, but the background is still checkerboard to indicate it is a percentage.
In addition, for splashing recipes, REI does not correctly support up to six recipes, like JEI does on Forge
Reproduction Steps

Create a splashing recipe via datapacks/kubejs/etc. with more than three outputs and a chance output
Observe the recipe via REI

Expected Result
The hover tooltip for outputs should have their percent chance of output shown and recipes with more than three outputs should rearrange them to be two columns next to each other
Screenshots and Videos
As seen in my testing:

Sequenced assembly correctly displaying percent chance output on hover:

Simi's statement and demonstration of expected behavior:

Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.4.1
Minecraft Version
1.18.2
Other Mods
No response
Additional Context
Great job on getting the port done! Been enjoying it with some Fabric mods I haven't used before and am having a blast!

Update: after installing (and then removing b/c crashes) some mods, the percentages show up again for some reason

Update 2: The percentages are only shown for some recipes
https://youtu.be/0xUQ5rlIjIA

Yep, if you click on the hyper link someone has said its not just Create that has this problem but also tinkers construct

see #1616


Implementation is technically unsound

I'm talking about lines 167 to 173 of lib.rs:
#[repr(C)]
#[derive(Copy, Clone)]
#[doc(hidden)]
pub struct TraitObject {
    pub data: *mut (),
    pub vtable: *mut (),
}

The macro expansion then goes on to use this declaration like this:
pub unsafe fn downcast_ref_unchecked<T: $trait_>(&self) -> &T {
    let trait_object: $crate::TraitObject = ::$core::mem::transmute(self);
    ::$core::mem::transmute(trait_object.data)
}

This code makes an assumption that the memory layout of &dyn Trait for any trait Trait is the layout of TraitObject, which isn't a real guarantee that the Rust compiler is required to abide. Using the same TraitObject struct from core::raw would require nightly, but break compilation when this assumption would be broken because of a change to how trait object references are stored, averting possible undefined behavior by stopping compilation of the code.
The only sound alternative to this right now is to depend on a nightly compiler, make use of the unstable Pointee trait and await its stabilization, thus removing the nightly requirement.
Keep in mind that this is a breaking change. Code which depends on the 0.2.x line and is expected to be compiled with the stable compiler would break if mopa starts requiring nightly on that release line. This fix must therefore be deployed as 0.3.0 or, preferably, 1.0.0-rc1 (since Pointee is not stable yet and its fate is not absolutely guaranteed). The previous 0.2.x and 0.1.x release lines may then be yanked.

The only sound alternative to this right now is to depend on a nightly compiler, make use of the unstable Pointee trait and await its stabilization, thus removing the nightly requirement.

Not really necessary, since one can get the data pointer out of a &dyn … ("thin the pointer") by casting raw pointers:
pub unsafe fn downcast_ref_unchecked<T : $trait_>(&self) -> &T {
    &*(self as *const _ as *const T)
}

And this has already been done in this repo: it was done 5 years ago! https://github.com/chris-morgan/mopa/commit/2bb823b4da60316868a8adb8be573b6b5dd6f508
But the crate seems to be unmaintained, and that commit never made it to crates.io

https://github.com/advisories/GHSA-2gxj-qrp2-53jv Just linking the relevant CVE here.

Any chance this can be fixed?
It seems that shred, used by specs depends on this

Thank you Daniel! I'll follow up with shred when I can to make sure all
shred users (including specs, amethyst etc.) get the fix.
On Sat, 9 July 2022, 12:25 am Daniel Henry-Mantilla, <
@.***> wrote:

Sadly a patch section only works for the downstream user, so it would
require that specs use it when testing, and that further downstream users
do the same, etc.
I've decided to go and publish https://crates.io/crates/mopa-maintained
to palliate that.
The fix is then to do:

mopa = "0.2.2"+ mopa-maintained = "0.2.3"

—
Reply to this email directly, view it on GitHub
https://github.com/chris-morgan/mopa/issues/13#issuecomment-1179049634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIRUHV5C37UUACAQS57O2TVTA25BANCNFSM45472VXQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just closing the loop:
@torkleyy removed shred/specs/amethyst dependency on this crate as a fix for this issue and the crate not being maintained.
This fixes my usage as well :)
Thanks to @Torkleyy

Not really necessary, since one can get the data pointer out of a &dyn … ("thin the pointer") by casting raw pointers:
pub unsafe fn downcast_ref_unchecked<T : $trait_>(&self) -> &T {
    &*(self as *const _ as *const T)
}

@danielhenrymantilla sorry to ping this response after 2 years :) I just felt curious why casting can solve the problem here?  Isn't it still unsound to cast a fat pointer?

@shinmao if using as then by design it is not unsound: a common misconception is that as is just a non-unsafe transmute for certain cases, but as can actually perform conversion operations (in the pointer to usize case, it can "expose the provenance", in the unsizing case, it can embed the necessary metadata so as to get a wide pointer, and when the input is a wide pointer and the output is a thin (data) pointer), the as cast then becomes a legal "deünsizing"/wide-pointer-truncating operation:
fn as_<T : ?Sized>(p: *const T) -> *const () {
    p as _
}

which is guaranteed to extract the data pointer off the *const (impl !Sized) wide pointer, thanks to language semantics. That is, even if, say, the data pointer were laid out after the vtable reference, then this as cast would be guaranteed to extract the data pointer at that offset.

(the operation which cannot be done in current stable Rust (1.72.0)) is the metadata-extracting one: ::core::ptr::metadata())

transmute (or transmute_copy() or ptr.cast().read()) is a very precise operation which takes a value (and a size), and reinterprets that span of bytes.
We could imagine having:
//! Since the layout of wide pointers is unspecified, neither
//! `option_a` nor `option_b` are guaranteed to be sound!

fn option_a(p: *const dyn Trait) -> *const () {
    let at_p = <*const _>::cast::<u8>(&p);
    const OFFSET_TO_DATA_PTR: usize = 0; // ???
    unsafe { at_p.add(OFFSET_TO_DATA_PTR).cast().read() }
}

fn option_b(p: *const dyn Trait) -> *const () {
    let at_p = <*const _>::cast::<u8>(&p);
    const OFFSET_TO_DATA_PTR: usize = mem::size_of::<&'static VTable>(); // ???
    unsafe { at_p.add(OFFSET_TO_DATA_PTR).cast().read() }
}

with the option_a also being expressible as:
fn option_a(p: *const dyn Trait) -> *const () {
    unsafe { transmute_copy(&p) }
}

And since 0x142_u32 as u8 == 0x42_u8 == transmute_copy::<_, u8>(&0x142_u32), I was just hinting at the ideä/notion that:

we could be tempted to see x as type as being kind of analogous to transmute_copy::<_, type>(&x) (papering over non-Copy cases),
which in turn is similar to <*const _>::cast::<u8>(&x).add(0).cast().read()
which in turn is assuming that the data pointer of a wide pointer would necessarily be at offset 0

And since this whole issue #13 is about the precise layout of wide pointers being unspecified, that would mean usage of as  would not be fine. That's how I have interpreted your question anyhow.
And my point was basically that this first 1. bullet is wrong: as is truly a special (overly-)multipurpose operation, which in the case of a wide pointer magically "knows" how to extract its data pointer in a sound and robust fashion, much like, on nightly, the dedicated ::core::ptr::metadata() function is capable of doing for extracting the non-data pointer (metadata) component of a wide pointer.


Enable sorting by column

It would be nice if you could sort a column by clicking on the column header.

Hi @dougwaldron, this feature has been included in the release 0.6.0.

Thanks! Just gave it a try and it's working great.


exceptions: add dev.lizardbyte.dev.Sunshine

Sunshine requires the ability to start applications.
See: https://github.com/flathub/dev.lizardbyte.app.Sunshine/issues/36#issuecomment-2203990470

Can you please say what applications you need to starts and why?

Sunshine is a game streaming host for Moonlight.
The user can add any game or application they want as a launchable command. The user then gets a UI in the Moonlight client with posters to each application/game they added.
The most common applications to start are probably Steam, Lutris and other game launches, but many users also add individual games.

Ok thanks, I'm assuming this is the docs for that https://docs.lizardbyte.dev/projects/sunshine/en/latest/about/setup.html#application-list

it should be up in a few hours; you can check with https://docs.flathub.org/docs/for-app-authors/linter#check-currently-published-exceptions

Sorry for the delay. Yes those are the docs for adding applications.
Thank you!


Add a possibility to work with topics where cleanup.policy is delete not only compact

Now for the purpose of data saving, there is a strict check if Kafka topic has cleanup.policy=compact, otherwise, Exception is thrown and the application fails.  I think it will be a great idea to add some flag, and if I agree with possible data loss I can work with a topic which has cleanup.policy=delete too. It will be good for educational purposes because you could not create topics with compact policy on free Kafka online services.

I agree, I have exactly a use case where I don't care for very old data. I want to track the login users of a system, and the user usually logout or the login expires (inactivity) after a few hours.

Added config kafkacache.topic.require.compact which you can set to false

Does the manually created topic in this case has to be single partition or can it have multiple partitions?

It can have multiple partitions.


Error appearing during export, possibly due to deleting a field group or section

Hello, there is a good chance that this is not a bug in this plugin specifically, but I'm seeing behavior that I don't understand and am not able to fix.
When running the export command in the terminal, I'm seeing this output:
Exporting fields
No mapping found for source section:3

The "No mapping found..." message seems to be from the 'fields' export. Also, in the terminal, Schematic reports that it is 'Exporting fields' twice: Once near the beginning and once at the very end, making me believe that it's trying to export fields twice, which seems strange to me:
Exporting plugins
Exporting sites
Exporting volumes
Exporting assetTransforms
Exporting fields
No mapping found for source section:3
Exporting generalSettings
Exporting sections
Exporting globalSets
Exporting categoryGroups
Exporting tagGroups
Exporting userGroups
Exporting userSettings
Exporting elementIndexSettings
Exporting fields
No mapping found for source section:3
Exporting userGroups
Exported schema to config/schema.yml

I started noticing this behavior after I:

Created a new Fields group
Moved fields from an old group into the new group
Deleted the old group
Deleted a Section that was using fields in the old group

I'm guessing that the issue has appeared due to my deleting the field group and/or the section. Would anybody be able to give me an idea why I'm seeing the "No mapping found..." message, why Schematic reports it's exporting fields twice, and what I could do to fix these issues? I'd be very appreciative for any hints!

Schematic needs to import fields twice because of sources that are imported later.
Because of the current implementation fields are also exported twice.
The error indicates a field has a source 'section:3' but that a section with id 3 can not be found.
'section:3' should still be in the database somewhere. At the top of my head in the fields table.


Cookie Corner favorites

You currently can only instabuy the items in your favorites, but sometimes you want to have multiple items of those. There are different options to fix this, not sure what would be the best.

Won't fix. The instabuy products disappear when you add one product, making this feature a bit useless. We considered some other options, but none seem very handy to implement due to the small screen size of the cookie corner pc.


add matchExactPath trigger to map subscript

It forces to get value for provided keypath otherwise returns nil
For example:
If json
{ error = "Some error text" }
and mapping for this json
errorText <- map["error.message"]
Actual result is "Some error text" which is wrong because requested keypath is different.
if matchExactPath == true result will be nil

Unless I'm missing something, what you describe above is what the nested value is used for in the subscript.
If you write map["error.message", nested: true], the key that will be match will be error.message not error -> message

I faced the problem when error message could be at "message" or "error.message" keypath.
Also, don't ask me why :), error message could be at "error" keypath, which in my case shouldn't be mapped, but it will be because of "error.message". Mapper checks if value is a dictionary or an array and if it's not, it returns value at "error" keypath (in my situation it is String), but i expect nil from "error.message".
See the following code:
    var possibleMessage: String = ""
    possibleMessage <- map["message"]

    var possibleMessage2: String? = ""
    possibleMessage2 <- map["error.message"]

    message = [possibleMessage, possibleMessage2 ?? ""]
        .flatten()
        .filter { $0.length > 0 }
        .joinWithSeparator("\n")

if json will be { error = "some_text" } then message will be equal "some_text", but as you can see in code above there is no "error" keypath defined in map. Thats why it would be good to have ability to force mapping to get value from full keypath but not from some part of it.
map["error.message", nested: true] do the same as map["error.message"].


[No ticket] Hide link to wiki if wiki is not enabled

Ticket: [No ticket]
Feature flag: n/a

Purpose

Hide the link to a Registration's Wiki if wiki is disabled

Summary of Changes

wrap logic around the leftnav button for wiki
add test

Screenshot(s)
Overview page links for a registration with wikiEnabled: false

Overview page links for a registration with wikiEnabled: true

Side Effects

None

QA Notes

The LeftNav Link to the Wiki should not be visible if the backing project has its wiki disabled

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 71.876%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4763

Relevant Lines:
6354

💛  - Coveralls


Rename address-book to address_book

Cannot have it named address-book since it is breaking the type generation on the frontend.

Codecov Report
Patch and project coverage have no change.

Comparison is base (350c55c) 88.76% compared to head (db564ba) 88.76%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   88.76%   88.76%
=======================================
  Files           6        6
  Lines         623      623
=======================================
  Hits          553      553
  Misses         70       70

Impacted Files
Coverage Δ

contracts/address_book/lib.rs
32.75% <ø> (ø)

contracts/address_book/tests.rs
100.00% <ø> (ø)

contracts/address_book/types.rs
0.00% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Bug] valid_to within hubspot__company_history needs to be adjusted

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
The current logic for the valid_to field within the hubspot__company_history model is as follows:
https://github.com/fivetran/dbt_hubspot/blob/2c8c15a2fb56b59a47d758ccaa0046b6c552c136/models/sales/history/hubspot__company_history.sql#L17
However, this model is intended to represent the historical view of the change of properties. Therefore, we will want to instead adjust the partition to include both company_id and field_name to appropriately capture the historical changes of the company properties.
Relevant error log or model output
N/A

Expected behavior
The model captures the proper valid_from and valid_to for each historical property change for the hubspot company.
dbt Project configurations
N/A
Package versions
packages:

package: fivetran/hubspot
version: [">=0.5.0", "<0.6.0"]

What database are you using dbt with?
postgres, redshift, snowflake, bigquery
dbt Version
Core:

installed: 1.1.0
latest:    1.1.0 - Up to date!

Plugins:

bigquery: 1.1.0 - Up to date!

Additional Context
N/A
Are you willing to open a PR to help address this issue?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hello @fivetran-joemarkiewicz ,
I just wanted to inform you that this issue was also found on the Deal & Contact history models.
Would it be possible to fix those too?
It would be great to have a timeline on this too, as we are currently unable to deploy our dbt logic on PROD and schedule it through Fivetran. We have to launch our code locally and manually until this is fixed. Meaning that this bug is currently blocking a PROD delivery.
Thank you for your help.

HI @ramiouanes thanks for highlighting that for us!
We will most likely be picking this issue up in our next sprint (starting next Wednesday 6/1 and ending 6/15). However, if you would like a more immediate fix, you can actually fork this repo and apply the changes highlighted to your fork. You can then install your forked repo instead of the official version of this package using the git package installation method.
If this works for you, we are always excited and encourage community made PRs! If not, we will be able to address this in the next release of the package which we hope to have live by the end of our next sprint.

Happy Tuesday @ramiouanes! I just cut a release for dbt_hubspot v0.5.3 and the updates outlined in this PR should be ready to go in the next couple of hours, please watch out for it on the dbt package hub! :)


Logout does not redirect to provider for logout

In the code it seems that the logout in IdentityManager only removes the token from local storage. It does not call the redirectForLogout function which would redirect to IdentityServer's endsession endpoint and effectively clear the cookies. This, in my mind, would be the prefered way to log out.
We can surely modify the code to our liking, I would just like to ask about the rationale behind the current solution.

Well, it's using OAuth (not OIDC) so there is no concept of single signout. At some point I'll formally add OIDC support, which can then do signout.

That's a damn good point. So it only has the access token, basically, to access the identity manager API, but no id token to actually identify the user. Thanks again for a quick response!


raftstore: forward the replica's safe timestamp

Signed-off-by: linning<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #9384
What is changed and how it works?
Forward the replica's safe_ts by CheckLeader RPC and raft heartbeat.
TODO: add test
Check List
Tests

Unit test
Integration test

Release note

No release note

/merge

The follower may not able to handle stale read requests while applying snapshot, I will leave a TODO for now, and handle it at the next PR. /cc @BusyJay

PTAL @BusyJay

PTAL @nolouch @JmPotato

/merge

/merge

/run-integration-copr-test

/run-integration-copr-test


jsonpickle doesn't work with sgdclassifier

Description
Error given
Steps/Code to Reproduce
import numpy as np
import jsonpickle
from sklearn.linear_model import SGDClassifier
clf = SGDClassifier(loss='log', penalty='l2', alpha=1e-4, n_iter=100, random_state=42, n_jobs=-1)
np.random.seed(13)
x = np.random.rand(10,5)
y = np.random.randint(low=0, high=2, size=10)
clf.fit(x, y)
clf2 = jsonpickle.decode(jsonpickle.encode(clf))

Expected Results
get the same clf2 as clf
Actual Results

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "python2.7/site-packages/jsonpickle/__init__.py", line 149, in decode
    return unpickler.decode(string, backend=backend, keys=keys)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 26, in decode
    return context.restore(backend.decode(string), reset=reset)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 127, in restore
    value = self._restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 283, in _restore_object
    return self._restore_object_instance(obj, cls)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 348, in _restore_object_instance
    return self._restore_object_instance_variables(obj, instance)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 393, in _restore_object_instance_variables
    instance = self._restore_state(obj, instance)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 398, in _restore_state
    state = self._restore(obj[tags.STATE])
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 451, in _restore_dict
    data[k] = self._restore(v)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 192, in _restore_reduce
    if f == tags.NEWOBJ or f.__name__ == '__newobj__':
AttributeError: 'dict' object has no attribute '__name__'

Versions
>>> import sys; print("Python", sys.version)
('Python', '2.7.8 (default, Aug  3 2017, 16:13:26) \n[GCC 5.4.0 20160609]')
>>> import numpy; print("NumPy", numpy.__version__)
('NumPy', '1.13.1')
>>> import scipy; print("SciPy", scipy.__version__)
('SciPy', '0.19.1')
>>> import sklearn; print("Scikit-Learn", sklearn.__version__)
('Scikit-Learn', '0.19.0')

You may need to load the numpy extension.
Please see http://jsonpickle.readthedocs.io/en/latest/extensions.html for how to do that.
Feel free to reopen this issue if you're able to help reproduce the issue in unittest form.  tests/numpy_test.py is where we keep the numpy tests.  There are certainly lots of libraries that can pose challenging.
A good place to start is to run make test and see if the current tests pass in your environment.
We use nosetests, so sudo apt-get install python-nose (or the python3 one) will be needed to run the test suite.  I use a bunch of virtualenvs so my usual workflow is often:
vx env27 make test &&
vx env35 make test &&
vx env36 make test

where the env dirs are virtualenvs for the various python versions that I've setup beforehand and vx[1] is a simple virtualenv command runner.  The quickest way to build and test your own env is to do:
virtualenv env
vx env pip install -r requirements-dev.txt
vx env make test`

Let me know if the numpy extensions help at all.  We should probably tag and relese what's in master since there are some improvements near the code in the backtrace, so it might be worth trying what's in master (it's hard to tell whether or not the backtrace is using an older release).
[1] https://github.com/davvid/vx

If I load the extensions, a different error appears:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "python2.7/site-packages/jsonpickle/__init__.py", line 149, in decode
    return unpickler.decode(string, backend=backend, keys=keys)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 26, in decode
    return context.restore(backend.decode(string), reset=reset)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 127, in restore
    value = self._restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 283, in _restore_object
    return self._restore_object_instance(obj, cls)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 348, in _restore_object_instance
    return self._restore_object_instance_variables(obj, instance)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 393, in _restore_object_instance_variables
    instance = self._restore_state(obj, instance)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 398, in _restore_state
    state = self._restore(obj[tags.STATE])
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 451, in _restore_dict
    data[k] = self._restore(v)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 275, in _restore_object
    instance = handler(self).restore(obj)
  File "python2.7/site-packages/jsonpickle/ext/numpy.py", line 258, in restore
    base = self.context.restore(base, reset=False)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 127, in restore
    value = self._restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 169, in _restore
    return restore(obj)
  File "python2.7/site-packages/jsonpickle/unpickler.py", line 275, in _restore_object
    instance = handler(self).restore(obj)
  File "python2.7/site-packages/jsonpickle/ext/numpy.py", line 255, in restore
    arr = super(NumpyNDArrayHandlerView, self).restore(data)
  File "python2.7/site-packages/jsonpickle/ext/numpy.py", line 161, in restore
    buffer = b64decode(values)
  File "python2.7/site-packages/jsonpickle/util.py", line 516, in b64decode
    return base64.b64decode(payload)
  File "python2.7/base64.py", line 73, in b64decode
    return binascii.a2b_base64(s)
TypeError: must be string or buffer, not float

About the release:
>>> print("Jsonpickle", jsonpickle.__version__)
('Jsonpickle', '0.9.5')

Thanks, I have a fix I'll push up shortly.  This looks like an edge case.. unfortunately I'm unable to reproduce it with just numpy, but I added scikit to the test suite nonetheless.

Thank you for such a quick fix

This seems to happen for 0d numpy arrays, here is a reproducer with only numpy:
import jsonpickle
import jsonpickle.ext.numpy as jsonpickle_numpy
jsonpickle_numpy.register_handlers()

import numpy as np
jsonpickle.decode(jsonpickle.encode(np.array(0.)))


rustc: package build troubles

Issue description
I try to build parity package with latest rust & cargo but have troubles in build stage:
building path(s) ‘/nix/store/7f6pm324bwnq69lwmq8m19cm43cbimgv-parity-1.8.3’
unpacking sources
unpacking source archive /nix/store/kr5andf56d7ishrzl2520c2bz5vf9jb8-source
source root is source
patching sources
configuring
building
Running cargo build --release
error: failed to load source for a dependency on `serde_derive`

Caused by:
  Unable to update registry https://github.com/rust-lang/crates.io-index

Caused by:
  failed to update replaced source `registry https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to decode `.cargo-checksum.json` of bn v0.4.4

Caused by:
  invalid type: unit value, expected a string at line 1 column 1899
builder for ‘/nix/store/b3579h9x9pc4akz7798hyb9339ra0617-parity-1.8.3.drv’ failed with exit code 101
error: build of ‘/nix/store/b3579h9x9pc4akz7798hyb9339ra0617-parity-1.8.3.drv’ failed

In my opinion vendorizing stage is broken cause my port to latest rust & cargo version works.
Steps to reproduce
$ git clone https://github.com/airalab/airalab-channels
$ cd airalab-channels
$ nix-build -A parity-beta

Technical details

system: "x86_64-linux"
host os: Linux 4.9.61, NixOS, 18.03pre120540.b8f7027360 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 1.11.15
channels(root): "nixos-18.03pre120540.b8f7027360"
channels(akru): "aira"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

I have the same issue.

Fixed by package source patch in https://github.com/NixOS/nixpkgs/pull/34441


Polyline accuracy problem

Is this a bug report?
i don't know
Have you read the Installation Instructions?
yes
Environment
react-native: 0.58.4
react: 16.8.3
react-native-maps: 0.23.0
Target: Android 27
Steps to Reproduce

create a simple project (react-native init projectName)
install the library (npm i react-native-maps)
link the library (react-native link react-native-maps)
install mapbox polyline library (npm i @mapbox/polyline) cause i'm using Google Directions API

Expected Behavior
Polyline drawn correctly on streets
Actual Behavior
Polyline drawing above houses

Reproducible Demo
https://github.com/brunojs02/react-native-maps-example

This issue is not related to react native maps. The reason is you are decoding the overview line, which is just the overview. You need to decode the steps instead.

When I decode the steps, the map gets frozen and takes about 8/10 seconds to center and show the route!!
Ps: with the link example above, i get 17704 latlng.


ModelicaMatIO.c not fully platform independent

Reported by peter.nilsson on 17 Sep 2014 11:47 UTC
The file Modelica 3.2.1\Resources\C-Sources\ModelicaMatIO.c has the following:
/* The size of `long', as computed by sizeof. */
#define SIZEOF_LONG 4
which is not always true. E.g. for gcc on 64-bit Windows and Linux, long is typically 64 bits (SIZEOF_LONG 8).
There might be more of the same kind in that file, havn't checked.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1569

Comment by beutlich on 17 Sep 2014 13:02 UTC
Currently these macros are only used if HAVE_HDF5 is defined which is not the default cause. As long as you do not (want to) compile ModelicaStandardTables with enabled support for reading MAT-Files of v7.3 (=HDF5) this issue can be ignored.
Nevertheless I try to find a general solution (probably based on INT_MAX, LONG_MAX etc.).

Modified by beutlich on 17 Sep 2014 13:03 UTC

Comment by beutlich on 18 Sep 2014 12:40 UTC
Resolved by e5babdaa4f5723b00a681207993f8bc46303a108 and aba3f66e0798f99d84b7af7c7b318a00ca1adba3.

Modified by beutlich on 18 Sep 2014 12:40 UTC

Modified by beutlich on 11 Dec 2014 22:36 UTC

Modified by beutlich on 5 Jan 2015 13:17 UTC

Changelog modified by beutlich on 5 Jan 2015 13:17 UTC
Avoid platform dependency of SIZEOF_LONG (and other) in sources of the Modelica Standard Tables


Offline Message from Maryann Martone: For some reason in NIF, this expands to...

This Github issue is synchronized with Zendesk:
Ticket ID: #23819
Group: Support
Requester<EMAIL_ADDRESS>Issue escalated by: Ko-Wei Lin
URL:
Original ticket description:

From: Maryann Martone<EMAIL_ADDRESS>Phone:
Location: La Jolla, United States
URL: https://scicrunch.org/scicrunch/interlex/view/ilx_0111154
Department:
For some reason in NIF, this expands to include the Ventral Tegmental Area.  They are in two different regions of the brain, so that can't be right and I don't see it as a synonym here.  So I'm not sure what is going on.
----
Zopim
https://www.zopim.com

Looks like it is hitting on the tegmental substring in the synonym listed below.
	UBERON:0006119
		abbreviations: []
		acronyms: []
		categories: ['anatomical entity']
		definitions: ['A group of cells ventral to the nucleus of brachium of inferior colliculus in the mouse (Paxinos-2001).']
		deprecated: False
		iri: http://purl.obolibrary.org/obo/UBERON_0006119
		labels: ['subbrachial nucleus']
		synonyms: ['subbrachial nucleus', 'tegmental area of tsai']


VariantSet metadata not correctly filled

Solves issue #677

Good start but there are a few things that need fixing

Thank you for the review. I think I addressed your comments.
If it looks OK to you, I'll squash commits.

yep, looks good to me

+1

+1; merging. Thanks @pcingola!