[tinyfiledialogs] Update version and download path to v3.6.3

This PR will increase the version of tinyfiledialogs to 3.6.3, as well as provide a frozen_version link from the author itself, guaranteeing the possibility to switch versions without much hassle

What does your PR fix? Fixes #12136

Which triplets are supported/not supported? Have you updated the CI baseline? edited

Does your PR follow the maintainer guide? I tried my best, so i hope it does :P

LGTM, thanks for the update!


Help: Casts not happening before API response

Regarding the use of https://laravel.com/docs/master/eloquent-mutators#attribute-casting with example:
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class User extends Model
{
    /**
     * The attributes that should be casted to native types.
     *
     * @var array
     */
    protected $casts = [
        'options' => 'array',
    ];
}

When I use the native PHP accessor
$user = \App\User::find(1);
$user->options['thing'] = 'stuff';
$user->save();

The JSON is automagically converted to an array, the key is updated, and then converted back to JSON when saved. This works as expected.
With Dingo in the mix I am responding to an API request:
// SomeController.php
    public function listUsers()
    {
        $users = User::all();
        // At this point each $user->options appears to still be a STRING of JSON data, it is not de-serialized until I try to access it as a $user->options property.
        $response = [
                    'success'  => true,
                    'message'  => '',
                    'users' => $users,
                    ];
        return response()->json($response);
        // My hope is that the Dingo response would notice that there are casts and somehow de-serialize the data before building a JSON response for the user. Is this an incorrect use?
    }

Dingo is not responding with the user array like in the accessor, but it is instead responding with the string of JSON. I can manually decode this JSON string within the Dingo JSON response manually, but that is rather odd to have to do.
Is there a better way to respond to the request that will properly de-serialize the JSON $user->options property before sending the response back to the user? Or is this not currently supported?

i try , the same like you .


Typo fix preperndOnceListener -> prependOnceListener

Just a small typo fix. Also, thanks for making this lib!

Thank you.


destination file is not rebuilded after change

I have this less task, watchLess works fine in console there is event that file changed but there is no change in destination files. Did I miss something?
lessFiles = ["./frontend-src/frontend-src/frontend.less","./frontend-src/admin-src/admin.less"]

gulp.task 'watch-less', ['sprite'],->
  gulp.src(lessFiles)
  .pipe(watchLess(lessFiles))
  .pipe(sourcemaps.init())
  .pipe(less().on('error', handleErrors))
  .pipe(sourcemaps.write())
  .pipe(gulp.dest('./web/assets/css'))
  .pipe(livereload())
  return

Yup, same here. I copied the code example verbatim, only changing the paths.

same issue here

Anyone experiencing my issue can temporarily use my fork if the need to.

@Craga89 Are you going to maintain this repository?

thanks @treshugart, hopefully the main repo is updated soon

No probs. There hasn't been any movement on my PRs yet :(


Use pyro for continuous distribution like speech recognition

Wonder if I can use pyro for cont. distribution like speech. Planning to use it for low resource speech recognition where phonemes are the states. Something like a DNN-HMM model. Any starting point ?

Hi, would you mind posting this question on the Pyro forum instead so that it's easier for others with similar questions to discover?  We prefer to use issues for bugs and Pyro development discussion.
You might find this forum thread about switching deep Markov models relevant to your question.


Change sender name

Hello,
I've been looking for a way to change the sender name but I didn't find in your index. The script works fine but the sender of the massage is always the first name of the email like "info". Is there any way to change the sender name to anything else?
Thanks

Hey @yassersharaf !
Tank you for using Mailjet ! :airplane:
If you begin with this wrapper, please be aware it won't be actively maintained anymore. We decided to focus on a lighter and cleaner version here .
With this new Wrapper, sending an email is easy

<?php
require 'vendor/autoload.php';
use \Mailjet\Resources;

$mj = new \Mailjet\Client(getenv('MJ_APIKEY_PUBLIC'), getenv('MJ_APIKEY_PRIVATE'));

$email = [
    'FromEmail' =><EMAIL_ADDRESS>    'FromName' => "Mailjet Pilot",
    'Subject' => "Your email flight plan!",
    'Text-part' => "Dear passenger, welcome to Mailjet! May the delivery force be with you!",
    'Html-part' => "
Dear passenger, welcome to Mailjet!

May the delivery force be with you!",
    'Recipients' => [[ 'Email' =><EMAIL_ADDRESS>]]
];

$response = $mj->post(Resources::$Email, ['body' => $email]);

$response->success() && var_dump($response->getData());
?>

Please let me know if you suceeded ! :)

Thanks a lot for replying

Does it work on your side ? :)

include("mailjet/php-mailjet-v3.class.php");
$mj = new Mailjet( $apiKey, $secretKey );
$to = '{"Email":"'.$user.'"}';
$params = array(
"method" => "POST",
"FromEmail" =><EMAIL_ADDRESS>"FromName" => "Sender name",
"Subject" => "Subject",
"Html-Part" => "",
"Text-Part" => "text",
"Recipients" => json_decode('['.$to.']',true)
);
$result = $mj->send($params);
if ($mj->_response_code == 200){
echo "success - email sent";
}
else {
echo "error - " . $mj->_response_code;
}

Remember to close the issue unless you have any other question :)


Bug 1920664: if no status is updated by the router, mark the route not admitted

Changed the routeSelector to
$ oc get ingresscontroller -n openshift-ingress-operator  -oyaml | grep " routeSelector" -C 3
    uid: d287087c-99d7-47b3-9c81-c2c3d903c4ca
  spec:
    replicas: 2
    routeSelector:
      matchLabels:
        app: directvolumemigration-rsync-transfer-foo
  status:

This made the route created by dvm unusable, dvm failed silently
$ oc get routes dvm -oyaml
apiVersion: route.openshift.io/v1
kind: Route
metadata:
 annotations:
   openshift.io/host.generated: "true"
 creationTimestamp: "2021-01-26T18:34:26Z"
 labels:
   app: directvolumemigration-rsync-transfer
 name: dvm
 namespace: permissions
 resourceVersion: "474876"
 selfLink: /apis/route.openshift.io/v1/namespaces/permissions/routes/dvm
 uid: 5e60db06-ac47-44f5-8779-3c7a161602ec
spec:
 host: dvm-permissions.apps.cluster-alpatel-aux-tools-451.alpatel-aux-tools-451.mg.dog8code.com
 port:
   targetPort: 2222
 tls:
   termination: passthrough
 to:
   kind: Service
   name: directvolumemigration-rsync-transfer-svc
   weight: 100
 wildcardPolicy: None
status: {}

with this fix I see the warning as expected
  conditions:
  - category: Advisory
    lastTransitionTime: "2021-01-26T18:34:26Z"
    message: 'Step: 11/23'
    reason: EnsureRsyncRouteAdmitted
    status: "True"
    type: Running
  - category: Required
    lastTransitionTime: "2021-01-26T18:32:36Z"
    message: Direct migration is ready
    status: "True"
    type: Ready
  - category: Warn
    lastTransitionTime: "2021-01-26T18:37:31Z"
    message: 'Some or all rsync transfer routes have failed to be admitted within
      3 mins on destination cluster. Errors: []'
    reason: NotReady
    status: "True"
    type: RsyncRouteNotAdmitted
  itinerary: VolumeMigration

Changed the routeSelector to
$ oc get ingresscontroller -n openshift-ingress-operator  -oyaml | grep " routeSelector" -C 3
    uid: d287087c-99d7-47b3-9c81-c2c3d903c4ca
  spec:
    replicas: 2
    routeSelector:
      matchLabels:
        app: directvolumemigration-rsync-transfer-foo
  status:

This made the route created by dvm unusable, dvm failed silently
$ oc get routes dvm -oyaml
apiVersion: route.openshift.io/v1
kind: Route
metadata:
 annotations:
   openshift.io/host.generated: "true"
 creationTimestamp: "2021-01-26T18:34:26Z"
 labels:
   app: directvolumemigration-rsync-transfer
 name: dvm
 namespace: permissions
 resourceVersion: "474876"
 selfLink: /apis/route.openshift.io/v1/namespaces/permissions/routes/dvm
 uid: 5e60db06-ac47-44f5-8779-3c7a161602ec
spec:
 host: dvm-permissions.apps.cluster-alpatel-aux-tools-451.alpatel-aux-tools-451.mg.dog8code.com
 port:
   targetPort: 2222
 tls:
   termination: passthrough
 to:
   kind: Service
   name: directvolumemigration-rsync-transfer-svc
   weight: 100
 wildcardPolicy: None
status: {}

with this fix I see the warning as expected
  conditions:
  - category: Advisory
    lastTransitionTime: "2021-01-26T18:34:26Z"
    message: 'Step: 11/23'
    reason: EnsureRsyncRouteAdmitted
    status: "True"
    type: Running
  - category: Required
    lastTransitionTime: "2021-01-26T18:32:36Z"
    message: Direct migration is ready
    status: "True"
    type: Ready
  - category: Warn
    lastTransitionTime: "2021-01-26T18:37:31Z"
    message: 'Some or all rsync transfer routes have failed to be admitted within
      3 mins on destination cluster. Errors: []'
    reason: NotReady
    status: "True"
    type: RsyncRouteNotAdmitted
  itinerary: VolumeMigration

/lgtm

/lgtm


Example of handleSort()

Does have somebody example of processing sorting? I can't get over it. I know it is not any issue, but I don't know how to right process it in database.

you get three parameters at handleSort - $item_id, $prev_id, $next_id
for example if you have column priority or order (number in database), you can set priority for $item_id = $prev_id_priority + 1 and on all items > $next_id set priority = priority + 1


Dot spacing rule should not consider KDoc references

Something like:
/**
 * ...
 * before the next [KtFile].
 *
 * ...
 */

triggers the rule.
It formats it to:
/**
 * ...
 * before the next [KtFile].*
 * ...
 */

Fixed in https://github.com/shyiko/ktlint/commit/9d60ec282655bdd673ae6066de5c9e8662d4595d (not yet released).

Oh thats good to know, thanks!


ADFS/Deployment: resurrect 2 internal links

Description:
As described in issue ticket #2863 (Broken Links), the first 2 links in this article use an outdated format of assetId:///e340cf8f-acf3-4cba-8135-a9353b85e714 which was supposed to point to the article itself and 2 sections inside this article, by means of anchor links (#BKMK_1 & #BKMK_2).
Thanks to Sean Harper (@harperse) for creating the ticket to point out this issue in the first place.
Thanks also to @fate0521 for reminding us.
Changes proposed:

Replace the outdated assetId:/// links with filename + anchor (as used in the anchor link BKMK_3 below).
Remove whitespace (blanks) at the end of all lines.

issue ticket closure or reference:
Closes #2863
Additional notes:
This issue might have been solved quicker if someone had pointed out the (now) obvious case that these 2 links simply are pointers internal to this page.

To hide whitespace changes (my removal of all the whitespace at the end of most lines),
open the /files page with: https://github.com/MicrosoftDocs/windowsserverdocs/pull/3684/files?utf8=✓&diff=unified&w=1
(or simply add ?w=1 to the end of the regular "Files changed" page link).
You can also use the GUI alternative:

#sign-off

CC: @billmath / @eross-msft

#sign-off

@billmath : From what I can see on the public page (latest commit as of today: 1ef166b on Sep 27, 2019) and the official page (last updated 05/31/2017), there is no sign of the required changes yet. May I ask you to look into what might be delaying the update you spoke of? The top links on the official page do not react to being clicked on, whereas the GitHub page () does not seem to have any working links displayed in the following text:

After you install the Active Directory Federation Services (AD FS) role service on your computer, you are ready to configure this computer to become a federation server. You can do one of the following:

Configure the first federation server in a new federation server farm

Add a federation server to an existing federation server farm

After you install the Active Directory Federation Services \(AD FS\) role service on your computer, you are ready to configure this computer to become a federation server. You can do one of the following:

-   [Configure the first federation server in a new federation server farm](assetId:///e340cf8f-acf3-4cba-8135-a9353b85e714#BKMK_1)

-   [Add a federation server to an existing federation server farm](assetId:///e340cf8f-acf3-4cba-8135-a9353b85e714#BKMK_2)

I can only guess that the update you mentioned has been put on hold, but it would be useful to know if there is anything I (or someone else) could do to resolve the issue ticket #2863 (Broken Links).


Push Notifications do not work in iOS

The problem is that when I try to send a notification from the FCM console, the notification is delivered only on Android, while nothing happens on iOS.
I followed this guide: https://medium.com/@felipepucinelli/how-to-add-push-notifications-in-your-cordova-application-using-firebase-69fac067e821
The Push Notification service is active on my Apple Developer account.
I generated the .p8 file and then uploaded it to the Cloud Messaging settings.
My versions:
Cordova 8.0.0
android 7.0.0
ios 4.5.4
could I have directions to solve the problem?
thank you

@pineapple-it-solutions did you call window.FirebasePlugin.grantPermission() to allow the app to receive notifications on IOS?

my application is an Angular app imported on Cordova, the index.js file is not present in my project.
My root folder is 'src' and outDir is 'www' in angular-cli.json file.
I tried to create index.js file and add the following code but the problem persists and I'm not asked for any permission on iOS:
`window.FirebasePlugin.grantPermission();
FCMPlugin.onNotification(function(data){
if(data.wasTapped){
//Notification was received on device tray and tapped by the user.
console.log( JSON.stringify(data) );
}else{
//Notification was received in foreground. Maybe the user needs to be notified.
console.log( JSON.stringify(data) );
}
});
FCMPlugin.subscribeToTopic(topicSub);`
where do I have to add the code exactly?
I attach the structure of my project.

Closing due to inactivity.  Please reopen if this is still an issue with v2 of the plugin.


Dump CX_SY_RANGE_OUT_OF_BOUNDS in  ZCL_ABAPGIT_SYNTAX_HIGHLIGHTER

Steps to reproduce:

clone demo
click on view diff near file /src/z_test.wapa.fragment.xml

Line contains
text="{i18n>Menu}"
it seems that the > is the root cause.

It seems that this regex for xml tags isn't sufficient.
https://github.com/larshp/abapGit/blob/26c6f4b4dc9db279c905146bb781cc9f9be89740/src/syntax/zcl_abapgit_syntax_xml.clas.abap#L22

had a quick look but I get an empty sintax check page.
Perhaps I need to set up a default code inspector variant or something?

It has nothing to do with syntax check page or code inspector. Just open the diff of z_test_wapa.fragment.xml.

Dumps. I will pick this up if you're not working on it already

Cool, I'm not working on it.

It's not as simple as changing a regex
First of all, our regexes are without submatches, so if I try to escape away
(?:"(?:(?:[^"]|(?:\.))*)")?([<>])
would actually work, and return the first submatch as < or >
I will extend the engine to support submatches, which will remove the dump
If we also want to properly highlight the tags we would need to remember we left one opened the line above.
Will look into it and add that too, otherwise we'll end up with something like this:

ok, that was a much smaller issue, works now:

did have to use submatches


Add HttpStatusListener to http client

An HTTP status listener hooks into Jetty's response listening so that users of this client can listen for the beginning of the response and examine the status before anything else from the request is processed. This type of listener has many uses cases one of which is to short circuit a response that has a status code that indicates the server is busy, no longer available, etc. It also gives users of this client an opportunity to throw an exception so that Jetty removes the connection from its connection pool. There are cases where Jetty's connection pool can get poisoned with bad connections.

@electrum all comments applied. Please merge when you can.


[3.0] Tile entities that are immediately powered after being pushed lose their metadata

only seems to happen with vanilla dispeners

Can confirm it still happens in Snapshot 2.1 and it does not work with Hoppers, only dispensers.

fixed for next release


Disable request diagnostics modal on install

Try out Leather build 07827c2 — Extension build, Test report, Storybook, Chromatic

This PR removes the "Help is improve" dialog from display upon installation of the extension. This change is intended to reduce friction during onboarding.
We may add the ability to disable pseudonymous analytics later inside of the app's general settings and outside of the onboarding flow.
Before

After

I did some refactoring here and got the tests to pass. We should get another review before squashing commits then approving and merging.


chore(main): release nvim 1.15.5

:robot: I have created a release beep boop
1.15.5 (2024-07-26)
Miscellaneous

nvim: add resession plugin (e8a9f3e)
nvim: adjust python settings (114d073)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

nvim-v1.15.5
:sunflower:


Update TypeHelper to allow implementations to require richer context

Replaces approach taken in 8b9cdb94410dce2f27c7d759ea138ed4dc7906f4
Much easier for implementations to run with other generators

CC @eredo

Really like this!


Live stats

Added panel with live stats.
Added "automatic rotation" button to rotate the model.
Added "automatic collection" in live stats, to run the stats collection and automatic rotation for 15 seconds.

/azp run


Figure out if rendering's changedTiles should be a tile to true map or an array

Benchmarking?
As an array there can be multiple entries for the same tile

pairs is slower


3.0.1 -> 3.0.2 broken browserHistory.listen

Version
3.0.2
Steps to reproduce

Configure webpack 2 (with tree-shaking). Not sure does it matter.
browserHistory.listen(someCallbackFunc)
Somewhere execute browserHistory.push('/1')

Expected Behavior

Browser URL will change andbrowserHistory.listen will fire callback

Actual Behavior

Browser URL will change, but browserHistory.listen won't fire callback

Check to make sure you have the right history version installed. The only change from 3.0.1 to 3.0.2 was adding the module entry to our package.json. No other changes were made.

I've found source of problem!
In my Root component I'am importing browserHistory like this:
// Root.jsx
import { browserHistory } from 'react-router/lib/browserHistory';

<Route history={ browserHistory }>...
``

but in child component I'am importing browserHistory another way:
```js
// SomeComponent
import { browserHistory } from 'react-router';

<button onClick={() => browserHistory.push('/some-route') }>...

and that was the problem — this is totally different browserHistory. On from ./node_modules/react-router/lib/browserHistory' and other from ./node_modules/react-router/es/browserHistory.js. So they had different sets of listeners.
I've changed every import of browserHistory to import { browserHistory } from 'react-router';


Move resource download to pytest code

Move the download process of the Patroni resource (and any other resource) to the pytest code and use the Charmhub API to get it.

@marceloneppel AFAIK all bits are coming from snap nowadays. Is it still topical?
CC: @dragomirp do you recall bits outside of our control?

@marceloneppel AFAIK all bits are coming from snap nowadays. Is it still topical? CC: @dragomirp do you recall bits outside of our control?

Patroni used to be a resource, but it's now in the snap. I don't think this is still topical.


Concurrent exhaustMap?

Something on the roadmap or is it possible to emulate it using existing operators?

Congratulations on the issue number 1337!
... can you give us an example of the expected behavior?  is it really just .map().exhaust()?

@blesh I think @ronag is noticing the lack of a concurrent value in exhaust and exhaustMap (like we have in mergeMap). We should probably add it to switch and switchMap while we're at it.

We should probably add it to switch and switchMap while we're at it.

Agree, it sounds useful and reasonable.

Closing due to lack of interest.


Added (44) Top DSA Interview Question.

Description
Added 44 Top DSA Interview Question.
Solved Issue
614
Type of change
New feature (non-breaking change which adds functionality)
Checklist:
[ yes ] My code follows the style guidelines of this project
[ yes ] I have performed a self-review of my code
[ yes ] I have commented my code, particularly in hard-to-understand areas
[ yes ] I have added tests that prove my fix is effective or that my feature works
[ yes ] New and existing unit tests pass locally with my changes
[ yes ] Any dependent changes have been merged and published in downstream modules
proof of changes

@ajay-dhangar I update PR.. please Review it..

@ajay-dhangar i added index.css.. its for styling dsa-interview.. please review it...


confirm SOTC format works for translation process

Sent draft SVG files from designer to your inbox, let me know if this works for our translation needs

I don't have any SVG files. Can you send them again?

Ignore that, they just came in.

Do you have 10 minutes to talk through this and a few other SOTC related items today?

Sure. my calendar is up to date

Matt Lee
Creative Commons
Austin, TX, USA
3pm-11pm UTC
On Wed, Nov 11, 2015 at 11:12 AM, rebeccalendl<EMAIL_ADDRESS>wrote:

Do you have 10 minutes to talk through this and a few other SOTC related
items today?
—
Reply to this email directly or view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/156#issuecomment-155849569
.

Thanks! Just sent an invite but feel free to adjust if needed. I'm open today except for 11-12 pst.

Just sent you updated from the designer. Let me know what your needs are if any for smooth translation and push to web. Thanks!

@mattl if you agree will you add Happy Birthday to the language strings when you do the translation outreach? We can use them in the 12/16 email, easiest to grab them when we grab the SOTC translations.

Update: we'll have final condensed version by EOB today, then we'll need to input into Google form so I can send on to affiliates for translation.

Cool, we'll do that first thing on Monday

Matt Lee
Creative Commons
Austin, TX, USA
3pm-11pm UTC
On Wed, Nov 25, 2015 at 4:00 PM, Jane<EMAIL_ADDRESS>wrote:

Update: we'll have final condensed version by EOB today, then we'll need
to input into Google form so I can send on to affiliates for translation.
—
Reply to this email directly or view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/156#issuecomment-159741141
.

Our affiliate translation deadline is today

@mattl if you want to create the basic google form, i can copy and paste in the final if you're off.

When will I get the copy? I'm done for Thanksgiving in 2 hours, and putting
it into a form takes a bit of time.

Matt Lee
Creative Commons
Austin, TX, USA
3pm-11pm UTC
On Wed, Nov 25, 2015 at 4:03 PM, rebeccalendl<EMAIL_ADDRESS>wrote:

Our affiliate translation deadline is today
—
Reply to this email directly or view it on GitHub
https://github.com/creativecommons/creativecommons.org/issues/156#issuecomment-159741720
.

ok

great, please just send to me with edit capabilities when ready and i'll do the pasting and send on to affiliates by EOD. thanks!

Shared the form with you. Basically you'll add a bunch of "Text" inputs on each page, and then split it up over the various pages of the report. The form text you enter will be the thing you want translated, and the response will be the translated text.


Long sentences

Hi,
the model seems to be working fairly well (tested after just 100K steps on a 100 speaker 24KHz dataset, it starts sounding reasonably well, but I guess it needs more epochs to achieve higher quality).
I just tested it on some random sentences, and I noticed the GPU ran out of memory for long sentences. What would be the best approach to synthesize long sentences? The baseline would be to split the mel spectrogram in parts and synthesize them separately, but I am not sure if this is the only way to go.
Thank you for your help!
PD: I'll report some results after 1M steps.

Splitting and synthesizing will work, though you may experience boundary effects near the split. You may want to split near silence.
The seamless way to do this is to modify the predict code to send in audio and spectrogram in chunks where each chunk is overlapped by the receptive field of the WaveNet decoder.
Out of curiosity, how long are your utterances?


Fix for early abort when next_token is not null

Results from the SSM API are returned on a best-effort basis. An API call may return 0 items but still yield a next_token key, meaning that the work done API-side yielded no results yet but future calls (searching further through the parameter space) may yield more results.
Previously an empty response.parameters would abort parameter fetching, even if next_token was not nil. This patch fixes that and adds an rspec example for it like the existing ones.

I think the CI failure here is unrelated to my changes


Handle some typecast compiler warnings

Can you resolve the conflicts? probably needs a rebase etc.

My mistake, you already merged these changes in https://github.com/mnunberg/jsonsl/pull/23


fix missing incref on flags

Fixes: https://github.com/numba/numba/issues/4775

Not sure if tests make any sense here, the corruption must be provoked differently on different OSs. I am not certain that a test would be reliable...

Given it's leaking right now, adding the thing that's segfaulting (at least on some platforms) as a test would provide evidence that this fixes it?

Sure, better than nothing.

doh!

The last commit was faulty and had some debugging output left from a different branch. I amended the commit and force pushed, I think in this case it was  O.K. since no code review comments were left on the PR yet.

Oh good; then the last commit did do the trick, good to know that old style classes will not work with @jitclass.


Strict plutus-cbor

Introducing more strictness to plutus-cbor
Could shave off about 30% on computation time on the expensive 100 asset case, but still not linear in input data! That means, that benchmarking in Haskell is still vastly different than evaluating & estimating execution costs in plutus.
Before:

After:

I just realized that this is breaking the normal plutus compilation of our validators using these functions :sweat:

I guess we can close that one?


Update erlang-23 package

This PR updates the version of erlang-23 to 23.2.4

Closing in favour of PR with newer version.


[OSE-166] Scaffolding for /v1/presentations/submissions

This adds the routing for Create, Get, List, and Review endpoints for submissions. None of the business logic was added unless it was boilerplate.

Codecov Report

Merging #171 (4820940) into main (7e576ca) will increase coverage by 1.93%.
The diff coverage is 61.29%.

@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
+ Coverage   26.16%   28.09%   +1.93%
==========================================
  Files          20       21       +1
  Lines        1594     1687      +93
==========================================
+ Hits          417      474      +57
- Misses       1106     1135      +29
- Partials       71       78       +7

Impacted Files
Coverage Δ

config/config.go
8.08% <0.00%> (-0.72%)
:arrow_down:

pkg/server/router/submission.go
66.21% <66.21%> (ø)

pkg/server/server.go
79.45% <72.72%> (-0.55%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Syntax error on valid command when reading .CAQ

I get a syntax error when I load (or run) a .CAQ file that was created on either the OG Aquarius via USB BASIC or via any legacy emulator (VAQ or Aqualite). It seems to be when I have a SET FAST ON statement in the code. The issue is resolved if I re-type the line back in on the AQ+ itself, and save the file. The code runs perfectly after that. A couple of screen shots showing the issue...

The SET FAST ON statement is plusBASIC specific. So if you type that in other BASIC versions it will be encoded as regular text instead of the correct plusBASIC tokens.

Hi. I know that set fast on is a plusbasic command. My code has clauses to only run those commands if an Aq+ is detected (and I am rubbing in a +). Those lines never run on the original Aquarius. If I retype the line, exactly the same... On the AQ+ it runs perfectly.

I think you don't understand the point I'm trying to make. The binary CAQ format stores BASIC commands as tokens (single byte values), in the same way the program is stored in memory. So a statement such as PRINT is stored as a single byte instead of 5 characters. If you type a statement that is not recognized it will be stored in full (not tokenized) as separate bytes. So if you enter the 'SET FAST ON' on the legacy BASIC it will be stored as separate bytes instead of 3 tokens. When you try to run that code on an Aq+ it will see that as a syntax error. When you retype the line it is tokenized by plusBASIC as 3 tokens and then it can recognize the statement.

Ahh... Thank you for that explanation. That does make my life a little more difficult when writing stuff for both legacy and plus though. But I get why it doesn't work now. Thank you.

I did also notice that the token is different for the Locate command. It seems to be translated to 'down'

Although it makes my development process a wee bit more involved... I understand why this happens, and I am happy to close the ticket.


I hope to add Chinese character reading and Chinese translation to GameYob.

I hope to add GameYob's support and translation of Chinese characters, because GameYob will display garbled characters when reading Chinese characters……

With the way you're wording your post, are you saying you're going to add Chinese support yourself?

If necessary, I can provide Chinese translation for it.
好耶( ᐛ )！！！
@.***
---Original---
From: @.>
Date: Sun, Apr 28, 2024 11:42 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [DS-Homebrew/GameYob] I hope to add Chinese character reading andChinese translation to GameYob. (Issue #17)
With the way you're wording your post, are you saying you're going to add Chinese support yourself?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[BEAM-10959] Store a fixed amount of known process bundle instructions to prevent failures due to concurrency within a runner.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi @udim

CC: @chamikaramj @ananvay
This cleans up the "instruction not found" failures Dataflow runner v2 saw (and likely other portable runners as well)

Thanks Luke! Should we check this against one of our benchmarks? (alternatively, we can keep an eye out for changes after it gets imported).

Nice! Thanks.

It should be very easy to verify with x-lang KafkaIO. I see this warning few times in every run.

yeah, though the other concern I had was any potential performance regressions...

@lukecwik is there documentation for the control stream mentioned in the bug?

@lukecwik is there documentation for the control stream mentioned in the bug?

What detail are you looking for?

how the control stream is used to process a bundle: https://s.apache.org/beam-fn-api-processing-a-bundle
how the control stream is used to report progress: https://s.apache.org/beam-fn-api-progress-reporting
gRPC streams are ordered: https://grpc.io/docs/what-is-grpc/core-concepts/

Thanks, that explains it. I wanted to know what was in the control stream (ProcessBundle, progress, and split requests are all received in the same stream).

This this bug is unrelated to SDF, right? Would this be erroneously reporting successful bundle processing, or is it a minor issue since the pipeline is being shut down?

@udim It causes a race for users/tests that finalization might be in progress when the pipeline is shutting down.


Build fails - ./configure not generated?

When trying to install rashell via opam, on my computer the compilation fails at ./configure --prefix /home/armael/4.02.3.
It looks like that indeed, the ./configure script is not shipped in the archive. Is a call to autoconf missing in the build instructions, in the opam file?

Would #14 be the correct way to fix that?

IMO the right way to handle the configure script is to not have it in the repository (it can be generated) and to have it in the distribution tarballs (it has to be canonical).
Which version did you try to build? In the latest release (v0.2.1) the configure script is in the archive:
% tar tzf rashell-0.2.1.tar.gz | grep configure
rashell-0.2.1/configure
rashell-0.2.1/configure.ac

If you pin the repository, you will need to generate the configure file manually, this is documented in the README file (on master).

Ah, my bad. I only tried installing from opam (which only has v0.2.0); and then tried to opam pin using directly the http url of the repository, which I understand is not supposed to work.

No problem! :) And thank you for preparing a patch, even if it does not turn out to be used! :)
I think there is no action to be taken here, if you also think so, you are welcome to close this issue! :)

Alright. Will the v0.2.1 be released in opam?

It will, we however have more and more problems with the CI suite in opam-repository, so that my PRs are a bit lagging.

Ok, no problem.


Issues with Signing in to ArcGIS Portal

Hello,
Our organisations ArcGIS Portal is internally facing and within a DMZ and all users are added based on IWA - I have been experiencing issues trying to sign in to the Portal using the ArcGIS Online Assistant.
When I put in the URL of the Portal and my username and password on the Portal, the "Logging In..." continues for a long period of time and does not log in. When looking at the Inspect element and running the Log On button, the Console gives the following error back:
Unable to get property 'generateToken' of undefined or null reference.
I was wondering is this maybe because our Portal is not public facing - or is it possibly another issue?

@MScott-GIS can you post a .HAR from Fiddler or console Network Tab log of the sequence of requests that your browser sends to the Portal before failing?

@MScott-GIS have you tried this workflow since we made some updates to the login process on the hosted ago-assistant app? I would like to know if you're still having these problemsor we could close the issue?

@slibby Thanks for your response - unfortunately we are not allowed to use Fiddler in our organisation - otherwise I would send those to you. However, the changes you have made to the PKI/IWA Login have made a difference to a degree.
I can still not sign in to that original portal that we have been facing (10.3.1), but we have anew portal site that is 10.5.1 and it works perfectly with that. So I feel that it may be an isolated issue with our old portal site - unfortunately there were other issues with that site before, so I wouldn't say that there is anything wrong with your web tool!
Thank you for investigating! :)

Thanks for the follow up @MScott-GIS and @slibby. Closing this one out as not reproducible.


Kubectl graph fails to resolve DNS names behind a VPN

When a VPN is being used then DNS lookups will fail:
$ GODEBUG=netdns=2 kubectl graph pods
go package net: built with netgo build tag; using Go's DNS resolver
go package net: hostLookupOrder(kubernetes.vpn.example.com) = files,dns
Unable to connect to the server: dial tcp: lookup kubernetes.vpn.example.com on X.X.X.X:53: no such host

For comparison, kubectl installed via homebrew is using the regular C resolution process:
$ GODEBUG=netdns=2 kubectl get pods
go package net: using cgo DNS resolver
go package net: hostLookupOrder(kubernetes.vpn.example.com) = cgo
NAME                             READY   STATUS    RESTARTS   AGE
example-application-77b7568d4b-csbnc   1/1     Running   0          6d5h
example-application-77b7568d4b-tl9v6   1/1     Running   0          6d5h
example-application-77b7568d4b-vsbtq   1/1     Running   0          6d5h

A solution already implemented in kubectl: https://github.com/kubernetes/release/issues/469

In my opinion, the only valid solution is to build the binary for macOS with CGO_ENABLED=1.


Emission materials should export as Unlit.

Fixes https://github.com/KhronosGroup/glTF-Blender-IO/issues/554.

^Ah, noticed I've broken a test.  Let me fix that and perhaps add another test case for this example.

Still not entirely sure this is the right way to go, see thread in https://twitter.com/jimmygunawanapp/status/1199455335066390528. But given that Blender seems to discourage use of the Background node this way, we need something, and an isolated Emissive node seems like the closest reasonable thing.

Shouldn't you also set
    "roughnessFactor": 0.9,
    "metallicFactor": 0.0

for fallback purposes?

Arguably, but it complicates an otherwise simple implementation (__gather_roughness_factor needs to check whether it's an emissive material), and the fallback seems unnecessary for me in what is arguably the simplest glTF extension. And if there were a fallback to PBR for the Emission node, a pure emissive material might make more sense? I guess I'd just rather not change that at all in this PR...

Has your thinking evolved since #211? I'm curious what changed.

Yes, what complicated that idea was @julienduroure's observation (https://github.com/KhronosGroup/glTF-Blender-IO/issues/220#issuecomment-453760813) that Blender's documentation says Background nodes "should only be used for the world surface output." If you use the Add --> Shader menu in Blender >2.80's node-based shader window, Background isn't even listed as an option, although you can still select it in the Material sidebar.
Since an Emission node is created by the importer, I'm inclined to go ahead and support in on the export side too. Not the most intuitive, but at least roundtrips will work consistently.


Fix URLs to Ben Furfie blog articles

He has changed the blog route from /blog to /articles

Good spot @lklion – naughty me for forgetting to update this.


Update MD file on new branch 2017-21-10_03-11-16

:warning: Validation status: warnings

File
Status
Preview URL
Details

split/ref.yml
:warning:Warning

Details

split/updateTestRef.md
:white_check_mark:Succeeded

split/ref.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref2.yml.
Line 1: [Warning] Invalid file link:(~/content.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added the ability to do the SSL Cert bypass via constructer

Added the ability to do the SSL Cert bypass via constructer
Fixed spelling mistakes and adjusted things to match the C# naming conventions
Updated to use object initializers.

Will need additional testing before merging.

Tested and working, using this change in production inside an AWS Lambda .NET 6 (ARM64)

Hi!
Thanks for this PR. Sorry, I didn't have much time for this project recently, but I'm trying to make time for it at least once in a month. Bypassing SSL check was already possible with overriding internal HttpClient:
var handler = new HttpClientHandler
{
    SslProtocols = SslProtocols.Tls12,
    CheckCertificateRevocationList = true,
    ServerCertificateCustomValidationCallback = (message, cert, chain, errors) => true // i.e. bypass cert validation
};
var httpClient = new HttpClient(handler);

var authInfo = new AuthInfo("user", "password", "xyz", "region");
var urlInfo = new UrlInfo() { Host = "hostname" };
var snowflakeClient = new SnowflakeClient(authInfo, null, urlInfo);
snowflakeClient.SetHttpClient(httpClient); // override internal http client

However dedicated configuration option might be more convenient - I'll think about this. As for this PR - it does have too many changes at once. I like most code style fixes, but in some rare cases I think they do hurt readability (and sometimes IDE suggests just bad changes). So I have reviewed all solution for IDE fix suggestions and applied most of them in my own commit.


fix: pass notifiable object to shouldInterrupt

Pass object to method
Cover with tests + improve existing tests

Closes #30

Codecov Report

Merging #31 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #31      +/-   ##
============================================
+ Coverage     98.16%   98.18%   +0.01%
- Complexity       76       78       +2
============================================
  Files             9        9
  Lines           218      220       +2
============================================
+ Hits            214      216       +2
  Misses            4        4

Impacted Files
Coverage Δ
Complexity Δ

src/Models/ScheduledNotification.php
100% <100%> (ø)
22 <6> (+2)
:arrow_up:

Continue to review full report at Codecov.

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

@empresarrollo Mind giving this a quick check for me? :-)

Tagged in 1.0.1 :)
https://github.com/thomasjohnkane/snooze/releases/tag/1.0.1
Thanks everyone!

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.18%. Comparing base (f1072ce) to head (c272e8a).
Report is 34 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #31      +/-   ##
============================================
+ Coverage     98.16%   98.18%   +0.01%
- Complexity       76       78       +2
============================================
  Files             9        9
  Lines           218      220       +2
============================================
+ Hits            214      216       +2
  Misses            4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement basic API doc skeleton

Sphinx was producing an error that it couldn't find api docs that don't exist. This PR implements basic API docs and the necessary skeleton to render properly. Thanks to Pey-Lian for the __all__ heads up!

Thanks for the review, Brett!


Desired behavior when casting Tensor to cuda?

Right now, if I cast a ByteTensor to cuda, as in
a = torch.ByteTensor()
a:cuda()

it will have type CudaTensor rather than CudaByteTensor. Because some modules (like maskedFill) require a CudaByteTensor, it's rather awkward getting an object of type CudaByteTensor.

Is this the desired behavior? (I'm not sure if the reason the conversion is done this way is because not all operations have been implemented for different Tensor types, or maybe another reason)
Is there a way to easily get around this i.e. construct a Tensor which will be a ByteTensor on CPU, and a CudaByteTensor on GPU?

this is unfortunate, and cant be changed because of backward compatibility.
This probably needs a new function that retains the type, like maybe :cuda2() or something, but i dont know of a decent name / path for this..

Yes, that's unfortunate, and dates back from the time cutorch didn't have other tensor typed than float.
When creating a new module (like CMaxTable), it seems then that you might need to overwrite the type function for the module, and add a special treatment for cuda tensors, as done for example in nn.Max  or in here https://github.com/torch/nn/blob/master/Normalize.lua#L26

Thanks for both those tips - that's very helpful. I think I'll go with constructing a CudaByteTensor similar to how the Normalize layer implements it. It seems like this doesn't completely fix the problem, since then gt expects the two inputs to both be CudaByteTensors if the second argument isn't a constant, but I think I can get around this by using two masks and copying between the two (suboptimal, but at least doesn't allocate memory)

I came across this issue recently, and was wondering whether now that there several different types of Cuda tensors, it would be worth having TYPETensor:gpu() and corresponding CudaTYPETensor:cpu() functions which convert to the appropriate Cuda type and back? I would be happy to submit a PR if you think it'd be worthwhile?

i think that's worth adding, and its great that it wont crush the backward-compatibility. The naming seems okay too.


Add note for starting a native app for the first time

The long time running known issue (UICORE-211) that it takes a while to start native apps for the first time will be removed in 8.10 as the situation has improved a lot. It can still be a little slower than usual, so this adds a note to the getting started guide to set expectations for first time users.

Ok @jobvs no problem. Could you give me an estimate of how long these "first starts" can take vs regular ones? I'd like to be able to right "Your first time starting a native app may take up to three minutes" or something like that. Specificity can help users manage expectations more.

Sounds good. It's usually 30-40 seconds.


1.1/Fix CMS CQL ExpressionBuilder issues - part 1

The first batch of small repairs to the ExpressionBuilder so we can process the CMS measures.
Also includes a reorganization of the input CQL+ELM files now we want to have both our original and CMS measure CQL included in the repo.

build.txt file must be removed

The term Package is quite overloaded, since we already have NPM Package and FHIR Package. And we have the CRMI $package operation. I suggested Project or Solution, but this is more editor terminology.  LibraryCollection?

The term Package is quite overloaded, since we already have NPM Package and FHIR Package. And we have the CRMI $package operation. I suggested Project or Solution, but this is more editor terminology. LibraryCollection?

Agreed to settle on 'LibrarySet'


[FEATURE REQUEST] Quick budget option to fix overfunding in credit card category

Is your feature request related to a problem? Please describe.
I have multiple currencies in my budget, so there's always a forex adjustment factor. After reconciling, that results in underfunding  or overfunding in a credit card category.
Describe the solution you'd like
I would like an option to address overfunding similar to what's already there for fixing underfunding, via the quick budget buttons.
Describe alternatives you've considered
Currently the only way I've found to reclaim the overbudgeted money is to manually do the math between the cc balance and the category balance amount, and then manually reduce the assigned amount.
Additional context
N/A

If I'm understanding you correct, this is exactly what the Paid in Full Credit Card Assist budget feature does. It adds a rectify difference button to the budget inspector. (Although the feature is slightly bugged right now; it doesn't show up sometimes unless you refresh the page.)

From what I see as I'm looking into this suggestion, I think you're then referring to the "rectify difference" button? That seems to only appear if the credit card category is underfunded, so similar to the "underfunded" button?  Testing that out has giving me a bit of a workaround - I can randomly change the assigned amount to then make one of the buttons show up and use it to get to the right amount, but I think my original request stands.  Or at least make the rectify button show every time the category balance <> the card balance and it wouldn't have to take up any additional real estate!

The rectify difference button will show up if your credit card category is overfunded as well. The feature is bugged right now so it's not showing up when it should, but I submitted a fix for it so that should be resolved soon.

OK, I'm happy to check again if that gets fixed, but right now I do see it,
just only when underfunded!
On Thu, Aug 25, 2022 at 5:51 AM Adam @.***> wrote:

The rectify difference button will show up if your credit card category is
overfunded as well. The feature is bugged right now so it's not showing up
when it should, but I submitted a fix for it so that should be resolved
soon.
—
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/2923#issuecomment-1226740633,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5USAYKR33YBWUIBG7ECU3V23UUHANCNFSM57G6VWFQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Cannot build project with Parchment in pods

Hello, I've just installed xCode 13 and now cannot build app, errors in Parchment.
My stack is:
xCode 13.0 (13A233)
CocoaPods 1.11.2
Parchment (3.0.1)
Tried pod repo update, pod deintegrate and install again, cleaning derived data - nothing helps.

Hi @andrii-lisovskiy! This is due to a bug in Xcode 13: https://github.com/rechsteiner/Parchment/issues/594#issuecomment-925155470. You can target the xcode-13 branch for now. If this isn't fixed in the official release, I will look into either dropping iOS <11 or splitting the SwiftUI code into a separate package.

Thanks for quick response, waiting for fix.
BTW, looks like with SPM everything works smoothly.

Yeah, installing and building works fine, but you get the same issue when archiving the app. Did the xcode-13  branch work for you?

Tried
pod 'Parchment', :git => 'https://github.com/rechsteiner/Parchment.git', :branch => 'xcode-13'
And still I have errors, cannot build.

Hmm, okay. This might be a different issue then. Need to look more into it

Yes, I confirm that the fix from xcode-13 doesn't work :(. I hope you will find the solution soon. Thanks in advance.

Feel free to fork it and test out some stuff ourselves though. I’m a bit busy these days, but happy to merge any fixes

The only workaround that I found is to use version 2.2.0 of Parchment.

I also encountered the same problem as @andrii-lisovskiy, is there any solution?

I have same problem too. Hope you fix that soon.
Thank you !

One way to workaround this issue is to raise the deployment target of this pod to 11.0:
Add this to your Podfile:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      # https://github.com/CocoaPods/CocoaPods/issues/9884
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '11.0'
    end
  end

In my case, this problem occurred when Xcode beta version(13) and Xcode release version(13) were installed together.
It may be because xcode-select with beta version.
So you can solve the problem by deleting the Xcode beta version or by run xcode-select with the xcode version you are currently using.

One way to workaround this issue is to raise the deployment target of this pod to 11.0:
Add this to your Podfile:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      # https://github.com/CocoaPods/CocoaPods/issues/9884
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '11.0'
    end
  end
end

Hello, this really worked. Thank you.
I have a question for you :)
I made the deployment version 12.0. What is the difference between making 11.0 12.0 or 13.0?
Deployment target of one of my projects is 12.0 and the other is 13.0

One way to workaround this issue is to raise the deployment target of this pod to 11.0:
Add this to your Podfile:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      # https://github.com/CocoaPods/CocoaPods/issues/9884
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '11.0'
    end
  end
end

It's better to set deployment target for Parchment only, like here.

actually i was getting errors on most pods but after doing that all the errors are gone. :)


E:/week39/Azure API Delegation Portal/node_modules/ng2-select/components/select/select.d.ts(2,10): error TS2305: Module '"E:/week39/Azure API Delegation Portal/node_modules/@angular/platform-browser/index"' has no exported member 'DomSanitizer'.

Error in ng2-select When I get ng2-select It shows this error bit it working perfectly right with previous version

I'm also having this issue. I just installed ng2-select, imported SelectModule into my index.ts and attempted to create a <ng-select></ng-select> element and am getting this error.


Klee’s Algorithm (Length Of Union Of Segments of a line)

Problem
The normal method for calculating the length of the union of line segments is inefficient and time-consuming. This poses a significant challenge for frequent users of this calculation.
Solution
I propose implementing Klee's algorithm, known for its superior efficiency in calculating the length of the union of line segments. By adopting this algorithm, we can optimize the computation process and enhance overall performance.
Alternative
Alternative approaches are available but often involve complex and cumbersome code, resulting in decreased performance. Klee's algorithm offers an optimal solution with reduced runtime, making it the preferred choice for streamlining this calculation.
Additional context
Accurate calculation of the length of the union of line segments is crucial in computer graphics, computational geometry, and data analysis. Implementing Klee's algorithm will improve efficiency, enabling faster processing of large datasets and facilitating real-time analysis. This enhancement will positively impact applications in various industries that rely on precise and efficient computations.

Assigned!
@hlw-aryan : C, C++, Java & Python


Problems with importing Shapely after import of another module that dlopens GEOS

Originally reported in https://github.com/Toblerity/Fiona/issues/276. This feels like a problem with ctypes. TODO:

[x] research on ctypes and dlopen and why you can reliably import shapely after fiona on Linux, but not on some OS X systems.
[x] experiment with cffi instead of ctypes.

Data point: the bug (empty geometry collection from unary union) does not show up when Fiona and Shapely access the very same dylib. It does show up when each of them accesses differently delocated copies of the same dylib.

Closed via #327.


Add "parallel" iter::empty() and iter::once()

These are simplistic producers, mirroring their std namesakes.  They
are essentially just optimized equivalents to iterating None and
Some(x), where empty() is a ZST, once() doesn't need the enum tag
for an Option, and neither have the associated enum branches.

I need to add docs before we merge this.

bors r+


Feature/stability fixes

Stability and lifetime fixes. Less prone to crashes. Also added manufacturer data of 0xFF (test id): needed in watch-bridge 5.2

There's a lot to parse here, so I probably missed quite a bit, but overall looks like an improvement. Sleeping is always a bit sus, but if it makes things work more reliably, then why not 🤷


Posting the chunked base64 encoded image returns error from script.google.com

Hi,
Thanks for publishing your code. It gives us details of how to handle larger images when uploading to google drive.
When I try to send a big file that gets chunked, google returns Internal Server Error.
On going through the .gs code - what I understand is that the received string is decoded as base64.
Is it needed to concat the chunks before decoding the full conatenated chunks as base64? Could that be the reason for the internal server error I receive. The code from gsampallo works fine and is able to upload smaller files without any issue.
It will be of huge help, if you can let me know if this is correct - and whether the gs code should be modified to concat the chunks received.
Thanks and regards,
-Arijit

Without any futher information it is hard to tell. I would start by capturing small pictures before proceeding to bigger.

Thank you for your response. Will give it a try with a smaller picture and see if I am able to get any file saved on to google drive for the same.


Source field for Datasets

In addition to the 'host' field, I suggest the 'source' field of the same type Host that points at the canonical source of the data. This is needed for 99% of datasets in the EE catalog, as we are mostly a mirror.

In #179 is is suggested to just link them using a link with rel type derived_from until there is a more concrete standard to describe provenance. I would imaging that this standard would also include something to link to the source. Other than that, I think most parts of my first comment in #225 also apply here...


[help wanted] RotateTransformation transparent (white) fields become black after saving file

Description
I have the method for saving file after transformations
        public Task<Stream> GetImageStreamAsync()
        {
            ImageService.Instance.InvalidateMemoryCache();

            var task = ImageService.Instance.LoadStream((cancelToken) => FileInfo.GetImageStreamAsync());

            var taskWithTransforms = task
                    .WithCache(FFImageLoading.Cache.CacheType.Disk)
                    .Transform(GetTransformations());

            var isJpeg = Path.GetExtension(FileName).ToLower() == ".jpg";

            if (isJpeg)
                return taskWithTransforms.AsJPGStreamAsync(100);
            else
                return taskWithTransforms.AsPNGStreamAsync();
        }

After saving "white fields" are becoming black. Why does it happen?
Steps to Reproduce

Rotate an image to get white fields
Save with rotation via my method

Expected Behavior
White or Transparent colors OR property to set the color
Actual Behavior
Black color instead of White or Transparent.
Basic Information

Version with issue: Lastest
Platform: Android and UWP (iOS did not test)

Could you attach some files, so I could test it?

Yes, sure
Here they are

I need source images (before transformation) rather than output images.

The original images

Hi Daniel, do you any updates on it?

Could you try newest prerelease nuget?

I have tested it with version <IP_ADDRESS>2-pre. Unfortunately, it does not work on UWP and Droid (I didn't test on ios) :(
Should I set a specific parameter for it?

Could you make a repo then?

Yep, here it is https://github.com/luberda-molinet/FFImageLoading/compare/master...JTOne123:master
Please, try it out on Android or UWP
Also, I've checked iOS everything works well for current stable build and master branch

ohhh, good idea, thank you)))


Wallet portfolio

Please describe the problem that your request should resolve.

Describe the solution you'd like

Add total value in the wallet and total tokens as blocks
Add table with assets list
resolve 2 minor UX improvements from previous implementation https://github.com/aeternity/aescan/pull/549#issue-1961137672

Is it already possible to achieve the same outcome in any other way? If so, how?

Additional context

This will need a new endpoint.
Requested here https://github.com/aeternity/ae_mdw/issues/1624
Also some refinement of the design would be nice, as we didn't make solid conclusion of what should be displayed there.


Add Django 2.2rc1 to test matrix

mainly b/c this is going to be released soon and I want to check compatibility before we integrate with your library.

Oops, duplicate of #238


Add join/leave actions with buttons and status column

In this PR, I did the following changes:

Implemented missions actions: join/leave.

Your PR is APPROVED!!!
Comments

The Actions are well written and functions as they should
No Linter error
The App ran as expected
Folder structure is clean and well structured

Required Change ♻

[ ] None for now

Optional suggestions

[ ] None for now

Every comment with the [OPTIONAL] prefix won't stop the approval of the PR. However, I strongly recommend you to take them into account as they can make your code better and addressing them will really improve your application
((I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application))
Cheers and happy coding! 👍 👍 👍
Feel free to leave any question or comment in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-review. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise


Update for Kubernetes 1.26.4

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


not working

Challenge http://freecodecamp.com/challenges/waypoint-link-to-external-pages-with-anchor-elements has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
cod isn't working although I've wrote it right.

You have a space at the beginning of your a tag. Remove that space and rerun and it works perfectly. e.g. <a href instead of < a href
For future reference - this is not a bug and should not have been added to the Github issue queue as it could easily be resolved by seeking out help from fellow devs beforehand.
See https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Help-I've-Found-a-Bug for all the things you should try before submitting an issue on Github. Thanks!


Add a stand-alone "timing_load_creds" program (was PR 15088/16540)

Copy of #16540 by @richsalz, plus new fixups.

No, to early :(

FYI, @DDvO, I can write to the Akamai repo...

Since @beldmit and @paulidale approved this, updating labels.

Update the various comma-delimited compound statements.

Done.
So asking @beldmit and @paulidale to re-approve.

If this had been my code, it would have looked differently.
I also fixed a mem leak on contents.
Hopefully this little tool code is now in sufficient state.

Unfortunately I have to ask once more for re-approval.

Still OK

This pull request is ready to merge

Merged - thanks @tmshort for your comments and @beldmit and @paulidale for comments and approvals.


Simplify rh.here logic from what it was before.

Before, we were importing rh.here upon initialization of import runhouse as rh. So, we had to do some specific work with list_actors in order to check
if the ClusterServlet was ready before calling Ray code. This is because we
could cause a circular dependency since Runhouse was imported within the
EnvServlet and ClusterServlet.
However, now since

rh.here will only be called after these servlets are set
up, and
We attempt to connect to an existing Ray cluster instead of
creating a new one when we initialize the object store

We can simply initialize the object store and see if itw as able to set up
a cluster servlet to connect to.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #368

PR #382   👈

This stack of pull requests is managed by Graphite.


: Status -> Overview -> Active DHCP Leases: Table sort using header only lasts until next refresh (not working in 23.05)

Steps to reproduce:
This was previously reported in #6640

Go to router home page or Status -> Overview.
Scroll down to table of Active DHCP Leases.
Click on any of the table headings to sort table.
Wait a few seconds for page to refresh, table sort reverts to default.

Actual behavior:
Table sort works on click but reverts to default sort on next refresh (after a few seconds).
Expected behavior:
Table sort should be retained at least until page is fully reloaded.
Additional Information:
Table sort works correctly, being retained, on the Network -> DHCP & DNS -> Static leases page
cgi-bin/luci/admin/network/dhcp in the Active DHCP Leases table.
Happens on multiple Target and Arch
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='23.05.0'
DISTRIB_REVISION='r23497-6637af95aa'
DISTRIB_TARGET='mvebu/cortexa9'
DISTRIB_ARCH='arm_cortex-a9_vfpv3-d16'
DISTRIB_DESCRIPTION='OpenWrt 23.05.0 r23497-6637af95aa'
DISTRIB_TAINTS=''

Duplicate of #6640

The version you're running predates the fix.


problem with getWindow().setExitTransition

hi.
method getWindow().setExitTransition just get android.transition.Explode as a parameter.
but i want to use this kind of transition in android api below 21.
how can i do that?

sorry, activity and fragment transitions not backported for now. it's not so simple.
but you can imitate exitTransition by yourself. instead of calling finish() you can call beginDelayedTransition with explode transition as argument and remove all views from root view. and add listener to explode transition with  finish() in onTransitionEnd callback

duplicate of https://github.com/andkulikov/Transitions-Everywhere/issues/11


Added tests for definitions of cswap, cu1, cu3, cx

Summary
Partially fixes #2964
Added in tests for cswap, cu1, cu3 and cx under test_gate_definitions
Details and comments
Done as part of #2964
Future PRs will aim at the other standard gates

Thank you for approving it!


Inconsistent Outputs from tf.nn.conv2d_transpose on GPU

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.15.0
Custom code
Yes
OS platform and distribution
Ubuntu 22.04.3 LTS
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
cuda : 12.2 / cudnn 8.9.04
GPU model and memory
Tesla V100S-PCIE-32GB
Current behavior?
I've encountered an issue where tf.nn.conv2d_transpose produces inconsistent outputs on a GPU device across different runs, despite using the same input and operator. This inconsistency is seen both on xla compiled mode and eager mode.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np

class Model1(tf.keras.Model):
    def __init__(self):
        super().__init__()

    @tf.function(jit_compile=True)
    def __call__(self, inp):
        testout = tf.nn.conv2d_transpose(inp, inp, output_shape=[26, 57, 32, 1], strides=1, padding="VALID")
        return testout

inputs = [
tf.random.uniform(shape=[26, 32, 1, 9], dtype=tf.float16),
]
model1 = Model1()
device = "gpu"
with tf.device(device):
    tf.config.run_functions_eagerly(True)
    out1 = model1(*inputs)
    out2 = model1(*inputs)
    print(f'=========eager_output(version:{tf.__version__})================')
    print(tf.sysconfig.get_build_info()["cuda_version"])
    print(tf.sysconfig.get_build_info()["cudnn_version"])
    try :
        for i in range(min(len(out1),len(out2))):
            np.testing.assert_allclose(out1[i].numpy(), out2[i].numpy(), rtol=0.001, atol=0.001, err_msg=f'at checking {i}th')
        print("XLA_eager does not trigger assertion")
    except AssertionError as e:
        print("XLA_eager triggers assertion")
        print(e)
    tf.config.run_functions_eagerly(False)
    out1 = model1(*inputs)
    out2 = model1(*inputs)
    print(f'=========compiled_output(version:{tf.__version__})================')
    try :
        for i in range(min(len(out1),len(out2))):
            np.testing.assert_allclose(out1[i].numpy(), out2[i].numpy(), rtol=0.001, atol=0.001, err_msg=f'at checking {i}th')
        print("XLA_complie does not trigger assertion")
    except AssertionError as e:
        print("XLA_complie triggers assertion")
        print(e)

Relevant log output
=========eager_output(version:2.15.0)================
12.2
8
XLA_eager triggers assertion

Not equal to tolerance rtol=0.001, atol=0.001
at checking 0th
Mismatched elements: 178 / 1824 (9.76%)
Max absolute difference: 0.125
Max relative difference: 0.002499
 x: array([[[3.156],
        [2.463],
        [1.905],...
 y: array([[[3.156],
        [2.463],
        [1.905],...
2023-11-26 16:06:46.480802: I external/local_xla/xla/service/service.cc:168] XLA service 0x557fce97e690 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2023-11-26 16:06:46.480861: I external/local_xla/xla/service/service.cc:176]   StreamExecutor device (0): Tesla V100S-PCIE-32GB, Compute Capability 7.0
WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
I0000 00:00:1701014806.982871   61794 device_compiler.h:186] Compiled cluster using XLA!  This line is logged at most once for the lifetime of the process.
=========compiled_output(version:2.15.0)================
XLA_complie triggers assertion

Not equal to tolerance rtol=0.001, atol=0.001
at checking 0th
Mismatched elements: 133 / 1824 (7.29%)
Max absolute difference: 0.125
Max relative difference: 0.002611
 x: array([[[3.156],
        [2.463],
        [1.905],...
 y: array([[[3.156],
        [2.463],
        [1.905],...

@GwiHwan-Go, I was able to replicate the reported behaviour with TF v2.13, TF v2.14 and tf-nightly.. Currently we are investigating the issue & will deep dive into the issue and provide the root-cause for the same. Thank you!

Thank you for your response! Please don't hesitate to request any additional information if needed.

I think the issue is specific to GPU, I was able to reproduce the issue on colab with GPU runtime.
But, when the inputs are changed to float64 precision, the results are as expected.
Here is the attached gist for reference.
The reason could be due to optimized way of calculating numbers on Nvidia which is different compared to others, there will be small amount of precision errors.
Thank you!


how to tackle cycle-dep problem?

Hi gurus, was going to try use your master-piece to build ASN.1 compiler but after defining grammar in PEGTL it turned out to have a lot of cycle dependencies (loops) shown with pegtl::analyze:
problem: cycle without progress detected at rule class asn1::Type
problem: cycle without progress detected at rule class asn1::BuiltinValue
problem: cycle without progress detected at rule class asn1::BuiltinValue
problem: cycle without progress detected at rule class asn1::ConstrainedType

Can you please give a hint how to resolve it? I mean how should I change grammar to get rid of such loops?

First step, let's look at the cycles in the grammar. The first one around Type is easy to spot, given the rules
struct Type : peg::sor< BuiltinType, ReferencedType, ConstrainedType > {};
struct ConstrainedType : peg::sor< peg::seq< Type, sp, Constraint >, TypeWithConstraint > {};

we can see how Type can immediately (without consuming antyhing prior) call ConstrainedType which in turn can immediately call Type again.
The cycles around BuiltinValue are not so easy to spot, I'll see whether I can easily extend the grammar analysis to print the rules along the cycle, that should make it easier.
Regarding a solution for this cycle one needs to analyse the context of these grammar rules and see what one can come up with, do you know how to tackle this? I'll take a look at the first cycle above and see whether I can come up with something when I have a bit more time.
Further, considering the grammar and these issues I'm getting the idea that the grammar was actually a CFG and not a PEG, which also means that there might be other problems, i.e. other places where simply using the CFG as PEG does not create equivalent matching behaviour, have you in general looked into this for your ASN.1 grammar?

There might be more than one cycle containingBuiltinValue. The first one that I found starts with BuiltinValue which is an sor which can call InstanceOfValue which is Value which in turn is an sor with BuiltinValue. This is unfortunately not very PEG-esque...

Yes, this grammar isn't PEG-based since ASN.1 is rather old (not even written in ABNF but in its own style of CFG-like grammar).
But is there a way to change such grammar to be PEG? I mean any trick(s) how to unwind/break those loops?
With Bison/Yacc it is possible but I really like PEGTL so much more than clumsy Flex/Bison syntax...

The good news is that yes, it is possible, however the bad news is that it can be difficult. As an example, have you looked at the comments at the beginning of src/example/pegtl/lua53.hpp?

I do know you have Lua grammar but didn't look into it yet. Now you've got me thrilled. :)

Thanks, just read it... now I understand "the bad news". :(
bummer indeed as I'm not so grammar-friendly to come up with your trick applied there but at least I have an example to give it a try. Thanks a lot!
P.S.: still I wouldn't mind to have pegtl::analyze improved a bit showing cycles in depth. :)

I just pushed a change that lets analyze() output more information when called with an argument greater than 0, however it's not pretty because the analysis doesn't analyse the actual structure of the grammar, it analyses an equivalent form that is reduced to as few basic combinators as possible in order to simplify the algorithm. Therefore some of the rules that will be listed as part of a cycle will be rules that don't actually occur in the original grammar in that form. Still, it's better than nothing, and it is what helped me find the second cycle mentioned above...

Good luck with the CFG-to-PEG transformation, let us know if you get stuck and we'll see what we can do. You're at a good point if you now understand the "bad news"...

Final note: The grammar analysis does a good job of telling you that a problem exists, however the printed details on what and where exactly the problem are not always reliable, that's a long-term work-in-progress.

Thank you very much @ColinH! I'll try new analyze and to come up with any idea to break the loops.

let us know if you get stuck and we'll see what we can do
I really appretiate your generocity but will use it only as a last resort of course. :)


Improve parser errors around is

Category
Other
Describe the feature you'd like to request
There is some room to provide more specific parse errors for malformed is expressions.
To make this issue concrete: is currently parsers expecting a Name for the entity type. If the following text doesn't parse as a Name the user sees a cryptic "expected token" error message (see #176). We could improve these errors message by instead expecting an Expr, and then raising an specific error in the CST to AST conversion if the Expr isn't a Name.
Other parse error improvements improvements are covered by #176, #391, #407
Describe alternatives you've considered
.
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

To be even more concrete, here are some is parse errors (as of #396) that could be improved:
permit(principal in Group::"friends" is User, action, resource);
current error: "expected a entity uid or template slot"

permit(principal is User::"alice", action, resource);
current error: "unexpected token `"alice"`"

permit(principal is User in 1, action, resource);
current error: "expected a entity uid or template slot, found a `literal` statement"

permit(principal, action, resource is 1);
current error: "unexpected token `1`"

permit(principal is User in ?resource, action, resource);
current error: "expected a entity uid or template slot"

permit(principal, action, resource) when { principal is 1 };
current error: "unexpected token `1`"


currency mak

I would like to create a NSStringMask for money format in Swift:
       let myMask: NSStringMask = NSStringMask.init(pattern: "^[+-]?[0-9]{1,3}(?:,?[0-9]{3})*(?:\\.[0-9]{2})?$")
but I will get error in runtime: EXC_BAD_INSTRACTION
What am I doing wrong?

@BesatZardosht what are you trying to achieve? The first thing you must have in mind is that NSStringMask only works if you have at least one group parenthesis, and you have none. By adding ?: right after opening parenthesis you're explicitly telling the regex interpreter not to use the parenthesis group as a result group. Also, it appears your regex is missing escape back-slashes before that last dollar sign $.
If you're trying to add dots and commas to a currency input, I don't think NSStringMask will work because it reads from left to right and it would need to read from right to left in order to do it:
Imagine you have the number 123456. If you read from left to right, you'd read 3 characters and add a comma, which would yield 123,456$.
In order to properly mask 123456 you need to read it from right to left:
1 - Read two characters and place a dot: 1234.56
2 - Read 3 more characters and place a comma: 1,234.56
As I said, NSStringMask doesn't read from right to left so I don't think there's a pattern that would do what I think you're asking. If you want to improve NSStringMask in order for it to be achievable, you're more than welcome, just submit a pull request and I'll review it.

Btw, a good tool to test your Regexes is http://regexr.com/

Closing due to inactivity


Unexpected data in form of key-value pair in menu

I'm submitting a ...

[x] bug report
[ ] feature request

Current behaviour:
A string came in the menu upon clicking the menu drawer.

                    { _id: 59dfe35df3xxxxxxxxxxxxx,
  Hasaccess: true,
  HasAccess: false,
  Password: 'xxxxxx',
  username: 'rowhit' }

Expected behaviour:
It should show the name of admin who is currently logged in.
Steps to reproduce:

[ ] Changing the respective file with correct path

Hey, I tried everything is working fine try pulling new changes your fork is 30 commits behind.

yes @chaitanyagiri, I forgot to pull. Thanks for your quick response. Can you please review my PR #57


Add email to ruleset for notification

Rulesets should have a notification area with email and maybe some other options (error only, etc.)

Email field has been added.


Contact details - required or for ease of access by aid provider

Letter references and contact details. If the client has received correspondence from the company, any letter reference should be noted, together with any contact details.
https://askcpag.org.uk/?id=-216944#:~:text=Letter references and contact details. If the client has received correspondence from the company%2C any letter reference should be noted%2C together with any contact details.

agent details as well - is it ease of access or requirement to be able to contact


config pretrain_indomain_base_resnet50_mlm_itm.json  is missed

we couldn't find the config pretrain_indomain_base_resnet50_mlm_itm.json mentioned in the Pretraining scripts
. Could you update it ? Thanks!

It is this config pretrain_image_text_base_resnet50_mlm_itm.json, we forgot to sync the file names when clean the code, sorry for the confusion.


Move test dependency to the dev section

I believe to not be necessary to have flutter test in the dependency section and it is currently preventing this package to be used in the latest version of flutter since the SDK depends on test 1.3

also experiencing this issue, currently unable to build my app.

@jcraven-worksight Change the dependency in pubspec.yaml to:
  graphql_flutter:
    git:
      url: git@github.com:fabiocarneiro/graphql-flutter.git
      ref: patch-1

thank you @fabiocarneiro

:tada: This PR is included in version 1.0.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Should add some more e2e tests that attempts to fill the custom promp for playground

Is your enhancement related to a problem? Please describe
Should create additional playground e2e tests that try to create a custom prompt.
Describe the solution you'd like
create a new test.
Describe alternatives you've considered
No response
Additional context
No response

related to https://github.com/containers/podman-desktop-extension-ai-lab/issues/2101


Add Validation Applicative Functor

See

learning Scalaz — Validation
Scalaz Validation
A Monad in Practicality: First-Class Failures

Note: As John stated, a list of errors can also be collected with the sequence function (which will be rolled out in the broad with #767) by turning a List<Either> to Either<List>.

@enelson The Validation type should be located in javaslang.control. I would use Try as reference impl (some function args will be re-visited soon with #865).
We should keep an eye on the functional interfaces Try.Checked*. We use them also in Future and maybe in Validation. There was a recent discussion in #865 if it would make sense to move them outside of Try. But I'm not convinced yet.

@danieldietrich Thanks for the info. I'll do some initial work in another branch and see what you think about it. I want to make sure its something that feels right for this project based on your ideas and goals.

:+1:

We need to take care that Validation is no monad, it is an applicative functor.
See also http://robotlolita.me/2013/12/08/a-monad-in-practicality-first-class-failures.html

Yep, exactly what I was going for. If it was a Monad, it would short circuit after the first error, and we want an aggregation of all errors.

Great, I will clear the way (see #947)

Thanks @danieldietrich.

@danieldietrich I would assume that you'd like to keep your current pattern of putting classes with multiple permutations (ie. Function0, Function1, etc) in src-gen, instead of embedding each one as static classes in one Java file? I'm working on a ValidationBuilder class that will have up to ValidationBuilder8 (coinciding with Function8).

@enelson Yes, we generate redundant code. However, it better to first write a stable 'prototype' which later is moved to the generator (because writing generator code is not really fun) :-)
I would like to take a look at the builders. I ask myself if we really need them. I have state transitions in mind when I think of Validation. I'm very excited to see your approach!

@danieldietrich I'll send over a pull request of what I have so far and see what you think about the approach. I'm VERY frustrated with the lack of support for higher kinded types in Java, so I want to see what ideas you have for a more generic approach.

Thx

Here is an interesting discussion at a PR of Eric: https://github.com/javaslang/javaslang/pull/986/files#r48573515
A possible implementation of Applicative could make use the Kind type:
// trait Apply[F[_]] extends Functor[F]
public interface Applicative<F extends Kind<F, ?>, A> extends Functor<A> {

    // def ap[A,B](fa: => F[A])(f: => F[A => B]): F[B]
    <B> Applicative<F, B> ap(Kind<F, ? extends Function<? super A, ? extends B>> f);

    @Override
    <B> Applicative<F, B> map(Function<? super A, ? extends B> f);
}

// type constructor: TYPE<T> is split in TYPE<?> and T
interface Kind<TYPE extends Kind<TYPE, ?>, T> {
}

interface Validation<E, A> extends Kind<Validation<E, ?>, A>, Applicative<Validation<E, ?>, A> {

    // def ap[EE >: E, B](x: => Validation[EE, A => B])(implicit E: Semigroup[EE]): Validation[EE, B]
    <B> Validation<E, B> ap(Validation<E, ? extends Function<? super A, ? extends B>> f);

    @Override
    <B> Validation<E, B> map(Function<? super A, ? extends B> f);

    // def ap[A, B](fa: => Validation[L, A])(f: => Validation[L, A => B]) = fa ap f
    @SuppressWarnings("unchecked")
    @Override
    default <B> Validation<E, B> ap(Kind<Validation<E, ?>, ? extends Function<? super A, ? extends B>> f) {
        return ap((Validation<E, ? extends Function<? super A, ? extends B>>) (Object) f);
    }
}

@danieldietrich I'm still here. :) Just getting back into the work groove, so my progress on this issue has slowed for a bit. I'll commit some updated stuff soon for review. Hope you had a great New Year.

@enelson Good to hear :) I thought of you the last days. Thx, the New Year was huge, hope yours too. Looking forwards to the review!

I think we finished adding Validation. Implementing Value is another topic.


WSL support

When 1password and 1password CLI are installed in Windows, but Git is used inside of WSL, it is necessary to specify "op.exe" as the command instead of just "op". I have currently made this change to the source manually for my install, but it would be nice if this condition was automatically detected.

Hi @Jckf,
i see the problem, however i think it would be best to not do things specific for WSL or try to find a (probably) faulty detection. :)
I think adding a ENV-controlled path to the desired binary should to the trick, since you are right: it's a bit naive to asume that it's just op everywhere.
Would that work for your issue?
BR

@ethrgeist That sounds like a better approach, and should work fine for what I'm doing.


Move Sieve to before Minesweeper

Fixes #112
If a student is progressing through the exercises in order, this seems
like a reasonable place to put this exercise. They should be familiar
enough with Rust's iterator functions that they can arrive at the
solution more idiomatically.

I say 👍


Use Spin telemetry crate

Ensures compatibility with containerd-shim-spin which already inits a tracing subscriber
Bumps to latest Spin dependencies

@radu-matei can you give this a review


Syedhussainsaber/Issue #159 is Updated

This project is a Modern NFT Marketplace, where anyone can list, sell, buy and update the NFTs. This completly decentralised and secured platform.

I have completed the backend part and today is last date to submit the project, so I have contributed this, Later I will also complete the frontend part also.

Hi! @Vikash-8090-Yadav  can u merge my project into the repository web3.0-World. I have completed the project


Featureport

Portfolio finish mobile-first code base

[image: image.png]
please check figma dimensions it is  width :1440px for desktop version
On Fri, Jan 20, 2023 at 4:00 PM Francis Wayungi @.***>
wrote:

@.**** requested changes on this pull request.
Hi @codeWithHyder https://github.com/codeWithHyder ,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't addressed yet.
[image: image]
https://user-images.githubusercontent.com/99656687/213673676-d3f08618-5cde-4ab1-8554-1b2a5b39638e.png
Highlights

Great work using the right GitHub Flow 😌
Linter checks are passing 💯
Project Documentation (PR, README) 👍🏾

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#discussion_r1082331145
:

Please remove unused commented code from your code base. This makes
your code base dirty and can be a source of errors later on. You can use
git to move between different versions of your project.
Please remove all occurrences of commented code from your code base

In index.html
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#discussion_r1082334808
:

Multi-Post Stories

               A daily selection of privately personalized reads; no accounts or sign-ups required. This has

               been the industry's

               standard dummy text ever since the 1500s, when an unknown printer took a standard dummy text.

HTMLCSSJavascript

See Projects

See Project

Please fix this issue too. the problem is still persisting. 🧰

In index.html
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#discussion_r1082328770
:

About me

Hey there! I can help you build a product, feature or website. Look through some of my work and

                   experience! If you like

                   what you see and have a project you need coded, don’t hesitate to contact me.

Get My Resume

As suggested above, please fix the position of this background
image. You can start by fixing the overflowing content then the back image
will be a piece of cake.
[OPTIONAL] You can also put the correct image. The one you are
currently placing as the background is not the correct one. Take a look at
it below

correct background image for the about me section
[image: bg-1]
https://user-images.githubusercontent.com/99656687/213671352-9ceefb02-9bca-47b6-9c06-7a221ff32c25.png
In style.css
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#discussion_r1082318453
:

+* {

margin: 0;

padding: 0;

box-sizing: border-box;

font-family: 'Poppins', sans-serif;

+}

+/* ------------------navbar section------------------- */
+.nav-container {

display: flex;

width: 1440px;

height: 72px;

background-color: #3e3c3c;

align-items: center;

+}

Please fix this issue as requested by the previous reviewer. Your
nav-contianer class has a width of width: 1440px. This is making your
page to overflow. Please set the width as a percentage of the device screen
with eg: width: 80% 🍏

—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#pullrequestreview-1263266646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDCZVOTVNY7IRGCYNUTWTJSNXANCNFSM6AAAAAAT74M4YA
.
You are receiving this because you were mentioned.Message ID:
<codeWithHyder/portfolio-mobile--finish-first/pull/2/review/1263266646@
github.com>

--
Very regards
Ghulam Hyder
email #1   * @.**** @.***>

Hello Benjamin
I tried every possible code to remove this right white space but i could
not, either help me or let me go this time, and will fix it surely  in
coming days .....
On Fri, Jan 20, 2023 at 8:05 PM Osama Ashraf @.***>
wrote:

@.**** requested changes on this pull request.
Hi @codeWithHyder https://github.com/codeWithHyder ☺,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are
still some which aren't been addressed yet.
[image: you-are-near]
https://camo.githubusercontent.com/24e9f6bcfc9c66d6f90ce4fc937ec10b7ba92a648dadb764fb2c37246d36fbce/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f53535231374b36795a5776474359726655682f67697068792e676966
Highlights ✔

Descriptive READme file in a professional way.
No linter errors. 💯
Good usage in correcting GitHub flow

Required Changes ♻
Check the comments under the review.
Optional suggestions

Kindly try to fix the margin and padding of the section and elements
to make your design attractive for the visitors😊

Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#discussion_r1082613808
:

   John Doe

Portfolio

About

As stated 😃 by the previous reviewer kindly try to fix the overflow
of the body because I see that there is a scroll in the horizontal, so
kindly fix that to make your design attractive for the visitors😊

Check below to see the difference 😊⤵
Figma design Your design
[image: image]
https://user-images.githubusercontent.com/83360636/213722874-e99bc650-c937-44da-b9ff-e3b3abfc321b.png [image:
image]
https://user-images.githubusercontent.com/83360636/213722965-56625ce2-f628-4add-bb42-b43ed26a7bda.png
—
Reply to this email directly, view it on GitHub
https://github.com/codeWithHyder/portfolio-mobile--finish-first/pull/2#pullrequestreview-1263698562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE2WDHEKUL6RQ477UATKZTWTKPB5ANCNFSM6AAAAAAT74M4YA
.
You are receiving this because you were mentioned.Message ID:
<codeWithHyder/portfolio-mobile--finish-first/pull/2/review/1263698562@
github.com>

--
Very regards
Ghulam Hyder
email #1   * @.**** @.***>


[WIP][TwigComponent] add component attribute system/helper (alternative)

Q
A

Bug fix?
no

New feature?
yes

Tickets
n/a

License
MIT

This is an alternative to #220. This version makes attributes a primary feature (doesn't require a trait to use). In your components, an attributes object is always available. Properties passed to the component twig function that cannot be mounted to your component object are added to this object. This pattern is well established in blade/vue.
I haven't given a huge amount of thought yet to LiveComponent's. I'm thinking the attributes passed to the twig function will be passed back and forth via http with a special proprietary attribute.
This will bring some BC breaks:

Properties passed to the component twig function that do not exist on your component class will no longer throw an exception. Instead they are added to the attributes object. Since attributes are required to be string/scalar I could throw an exception if the passed extra values are not.
Using ComponentFactory::create() directly (unlikely scenario imo). This method now returns a ComponentContext object instead of the component object.
Assuming we release this feature to both TwigComponent and LiveComponent at the same time, if someone upgrades one without the other, there will be problems.

These seem fairly minor to me (we are still experimental) but any ideas to mitigate the above would be appreciated!
TODO

[ ] Tests
[ ] LiveComponent support
[ ] Documentation
[ ] Finalize ComponentAttributes methods

Usage
{# templates/components/my_component.html.twig #}

  My Component!

{# render component #}

{{ component('my_component', { class: 'foo', style: 'color:red' }) }}

{# renders as: #}

  My Component!

Defaults
Set default attributes that can be fully overridden by passed attributes
{# templates/components/my_component.html.twig #}

  My Component!

{# render component #}
{{ component('my_component', { style: 'color:red' }) }}
{{ component('my_component', { class: 'foo', style: 'color:red' }) }}

{# renders as: #}

  My Component!

  My Component!

Merging Defaults
Set defaults but allow them to be appended to by passing these values to the component function:
{# templates/components/my_component.html.twig #}

  My Component!

{# render component #}
{{ component('my_component', { class: 'foo', style: 'color:red' }) }}

{# renders as: #}

  My Component!

Only
{# templates/components/my_component.html.twig #}

  My Component!

{# render component #}
{{ component('my_component', { class: 'foo', style: 'color:red' }) }}

{# renders as: #}

  My Component!

Without
{# templates/components/my_component.html.twig #}

  My Component!

{# render component #}
{{ component('my_component', { class: 'foo', style: 'color:red' }) }}

{# renders as: #}

  My Component!

About BC (which we don't need to guarantee at this point, but we should be thoughtful about if breaking):

Properties passed to the component twig function that do not exist on your component class will no longer throw an exception. Instead they are added to the attributes object. Since attributes are required to be string/scalar I could throw an exception if the passed extra values are not.

Looks minor to me. Adding a check and an exception seems good, but we should probably allow for Stringable objects? Or should we be safer and stricter and only allow scalars (that might be better).

Using ComponentFactory::create() directly (unlikely scenario imo). This method now returns a ComponentContext object instead of the component object.

If we stick with this ComponentContext idea, this is indeed a BC break, but needed.

Assuming we release this feature to both TwigComponent and LiveComponent at the same time, if someone upgrades one without the other, there will be problems.

We should be able to manage this with the required version of twig-components from live-components, right?

We should be able to manage this with the required version of twig-components from live-components, right?

If we release this feature in say, 2.1 of both packages, we can ensure live requires a compatibly version of twig but not the other way, right? The BC break will be if someone is using 2.1 of twig and 2.0 of live (which is valid)... I think.

Closing in favor of #220. After discussion with @weaverryan, we decided this method adds too much complexity and BC breaks/problems.