Added default nullable filter.

Hi,
I am using this nice Xml extension on a project i am working on.
I added a default nullable filter (perhaps this can be done prettier?) to the code.
Do you mind if i contribute to this project? Great job on using c# 6 :+1:

Thank you for helping this library. Your idea is nice. But for example in my case I prefer to return a 'false' string. Filters that this libary should include could be lowerCase, uperCase, dateTimeShort, dateTimeLong, etc... I mean things that we all could need.
Since that is a preference for your case, you should add it only to your Compiler class, as explained here https://github.com/beto-rodriguez/SuperXml#filters.
here is an example that can help you:
Compiler.Filters["nullable"] = x =>
            {
                if(x.ToString() == false.ToString())
                      return "";
                 return x.ToString();
            });

Hope it helps! thank you again!

Hi,
But in my situation the return value of False (on every nullable field) is
undesired.
Could i make that a config option of the Compiler? So people can toggle the
behavior ?
Kind Regards,
Sander van Tilburg
2015-09-14 18:14 GMT+02:00 Alberto Rodriguez<EMAIL_ADDRESS>
Thank you for helping this library. Your idea is nice. But for example in
my case I prefer to return a 'false' string. Filters that this libary
should include could be lowerCase, uperCase, dateTimeShort, dateTimeLong,
etc... I mean things that we all could need.
Since that is a preference for your case, you should add it only to your
Compiler class, as explained here
https://github.com/beto-rodriguez/SuperXml#filters.
here is an example that can help you:
Compiler.Filters["nullable"] = x =>
{
if(x.ToString() == false.ToString())
return "";
return x.ToString();
});
Hope it helps! thank you again!
—
Reply to this email directly or view it on GitHub
https://github.com/beto-rodriguez/SuperXml/pull/1#issuecomment-140128777
.

I like that. What causes it is this ugly plain false.
https://github.com/beto-rodriguez/SuperXml/blob/master/SuperXML/Compiler.cs#L573
I think it would be a good idea to create a string property OnNullReturn. and instead of returning that ugly false, return the value of that property.
I can add it later, or it would be awesome if you add it.

Allright, i will add that tomorrow, and remove the filter. Thanks.

Can you take a look at the last checkin? The option is added.

I will push your suggestions on next release. thanks

How Can I get this change? I am trying to use OnNullOrNotFound property but not getting. @beto-rodriguez have you pushed this changes in the latest release?
Btw, superXML is very much helpful for generating a complex XML.

Unfortunately he never merged this change into main.
2017-07-28 8:23 GMT+02:00 niteenb4u<EMAIL_ADDRESS>
How Can I get this change? I am trying to use OnNullOrNotFound property
but not getting. @beto-rodriguez https://github.com/beto-rodriguez have
you pushed this changes in the latest release?
Btw, superXML is very much helpful for generating a complex XML.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/beto-rodriguez/SuperXml/pull/1#issuecomment-318571062,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC4tvlVTBZdQqwXHTERf53tEW9WIIDU1ks5sSX5kgaJpZM4F865h
.

thanks for information @stilbu , can you please share the changes with me? I am trying same like you to give empty value in XML if attributes is passed to NULL or EMPTY.


Ecommerce: Users with @ symbol in their usernames cannot make purchases

Steps to Reproduce

Using a user with an @ symbol in their username, make a purchase (e.g. DEDP exam authorization)

Expected Behavior

The order will be fulfilled

Actual Behavior

The order will fail

Details

The order passes the username to Cybersource.  See this document for details

Cybersource labels it as: customer_account_id
It can be found in the receipt object as: req_consumer_id

Nathan was suggesting it an approach where we hash the username before sending it to Cybersource.

Consult him before implementing

This document contains some background about this problem

@jkachel it's fine not to store it, just make a note where it's computed that it shouldn't be changed

This tested OK - closing.


Instance segmentation does not return instances

I trained on custom dataset with INSTANCE_ON = True and only visuallize the highest score predictions["instances"] but the result is always a group of instance.
how can I solve this?
example: my dataset have 2 classes and this is the result https://imgur.com/2Rfytwo

I have met the same issue too. the log shows: detected 100 instances in 20.43s.

I have met the same issue too. the log shows: detected 100 instances in 20.43s.

@bowenc0221 could you please take a look?

I guess i didn't match config file and model parameter.

@snoopybingo @sangtrx yeah, I'm also getting the same error. Detected 100 instances.
And also while evaluating it always gives me this error:
DefaultCPUAllocator: can't allocate memory: you tried to allocate<PHONE_NUMBER> bytes. Error code 12 (Cannot allocate memory).
Please help me out guys.!


Add generalised spiral array function

Function to generate spiral arrays for any (odd) dimension, initial direction (up, down, left, right) and direction (clockwise/anticlockwise).
I haven't replaced any of the code that uses the hard coded spirals yet, but I figure this can come later?
I'm not sure if this belongs in the _util or preprocess modules? I thought _util seemed more appropriate seeing as it's not used directly for preprocessing images.

Closing in favour of #79


spatial/r2: implement Rotate

Fixes gonum/gonum#1513.
Please take a look.

Gio is using a more general type: Affine2D that not only handles rotations but also scaling, translations, etc...

https://godoc.org/gioui.org/f32#Affine2D
https://git.sr.ht/~eliasnaur/gio/tree/master/f32/affine.go#L12
https://git.sr.ht/~eliasnaur/gio/tree/master/f32/affine.go#L51 (Affine2D.Rotate(origin Point, angle float32) Affine2D)
https://git.sr.ht/~eliasnaur/gio/tree/master/f32/affine.go#L122

having used for implementing dashes and strokes for Gio and vg-gio, it's well thought out.
but I haven't had the use for such a complete affine transformation in my day-to-day "science-y" work.

Yeah, fair enough.

do you know what this "Travis CI - Branch" item is?
it doesn't seem to ever go green (nor red).
the r3-vec-rotate PR had that field yellow for the whole Fri->Mon interval...

It looks like the action that travis uses to run travis-ci.com is not aware of the situation when it doesn't have access to the branch. I can make that one not required, but you should be able to merge anyway with one extra click.

yeah, that's what I did for r3-vec-rotate :)


App gets stuck on white screen after installing my app from the Playstore.

Steps to Reproduce

... Download App from Playstore.
... Install app on physical device

Logs

Logs

[√] Flutter (Channel stable, 2.0.2, on Microsoft Windows [Version 10.0.18363.1440], locale en-ZA)
• Flutter version 2.0.2 at C:\src\flutter
• Framework revision 8962f6dc68 (6 days ago), 2021-03-11 13:22:20 -0800
• Engine revision 5d8bf811b3
• Dart version 2.12.1
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)

@liquidedgesolutions
There's not much actionable information you shared in this issue. We need to see verbose logs to check if it shows any exceptions.
Does the issue occur in non-prod environment locally ? On what devices does this issue occur ?
Can you try flutter build apk --verbose and then install it and see if issue occurs ? If so, provide the log to better address this issue along with a reproducible code sample.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Add helper for extracing a repo id from a purl

Not sure how this will fit in to the higher level rule, but let's take a bottom-up approach and figure it out as we go.
Ref: https://issues.redhat.com/browse/EC-847

Leaving as draft since I might want to change it later.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (ef32a68) to head (57c5e28).

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #1126   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          115       117    +2
  Lines         5791      5813   +22
=========================================
+ Hits          5791      5813   +22

Files with missing lines
Coverage Δ

policy/lib/purl.rego
100.00% <100.00%> (ø)

policy/lib/purl_test.rego
100.00% <100.00%> (ø)

I cleaned it up based on the suggestions, specifically:

Don't return null if there's no repoid
Less pointless commentary
Explanation for the purl.parse test
Explanation for the non-standard rule name

New PR is coming soon. It will incorporate a version of this probably.


feat(trustpilot): add the Trustpilot component: Logo and Stars

Summary 🍿
Add Trustpilot component to render Trustpilot logo and stars ⭐ .
Usage 🧱
import { Trustpilot } from '@zopauk/react-components';

<Trustpilot />;

Preview 📸
See working preview here.

:tada: This PR is included in version 4.30.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(mis): job table supports mysql

Description
MIS supports fetching jobs from mysql, eliminating the need to deploy a separate mariadb only to store  export-jobs jobs.
API Changes
Add a new property fetchJobs.db.type in mis.yaml to specify the type of job table. Acceptable values are mysql and mariadb.

Codecov Report
Base: 65.06% // Head: 65.06% // No change to project coverage :thumbsup:

Coverage data is based on head (2063e76) compared to base (3d1e169).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   65.06%   65.06%
=======================================
  Files          94       94
  Lines        2502     2502
  Branches      310      310
=======================================
  Hits         1628     1628
  Misses        836      836
  Partials       38       38

Impacted Files
Coverage Δ

apps/mis-server/src/tasks/fetch.ts
91.89% <ø> (ø)

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

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


Appointment accepts 24:00 as time

Appointment accepts invalid days of the month. E.g. 31 Feb

Command Result box does not update with error when entering an invalid appointment.
E.g. appoint 2 ap/ 32/01/2018 23:00 1


Part2 DateTime functions

Changelog category (leave one):

New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Kusto DateTimePart2 functions.
date_diff doesn't work with nanosecond, millisecond and microsecond, Since CH doesn't support .
but it works with Unixtime stamps and make_datetime.

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

Thanks @kashwy , I'll update the release notes.


Sync config.json with JS track

palindrome-products is not my favourite exercise, but I wouldn't delete it :stuck_out_tongue_winking_eye:
Has it been deleted by mistake?

@rchavarria it looks like the ES track and the JS track are more divergent than I thought.  There are several exercises from the JS track that have been skipped over, and palindrome-products should have been included far later than it was.  I'll make an audit of what's missing and put the exercises in as separate PRs.

Check out the "todo" key:

http://x.exercism.io/tracks/javascript
http://x.exercism.io/tracks/ecmascript

I could write a quick script that would get the list of problems that are implemented in javascript but not ecmascript, and the reverse.

Does the todo key tell the defined exercises (in x-common I think) that are not yet implemented in the track?
That could be a better source of new exercises to add to the track.

There are a large number not yet implemented in the JS track too.

Yeah, that's exactly what it says: exercises that are in x-common that are not in the problems list. It doesn't take the foregone and deprecated ones though.

@kytrinyx you offered a while back to compile the exercise list not yet implemented in the ES track-- can I impose on you to do that?  Time to add some new things!

This is that list http://x.exercism.io/v3/tracks/ecmascript ... but I think we actually talked about figuring out what's in JavaScript that's not in ECMAScript, right?

Currently everything that is implemented in JavaScript is also implemented in ECMAScript.

I was thinking about what to add next (to both tracks.)  I see that the
'todo' node in the JSON has unimplemented items.
Is anything on the 'to-do' list fair game?  Or is there some sort of
precedence on which to add first?
BTW I'm thinking at some point it might make sense to merge the JS and ES
tracks.  Seems like browser support is too uneven to do that yet.
On Fri, Jan 15, 2016 at 10:48 AM, Katrina Owen<EMAIL_ADDRESS>wrote:

Currently everything that is implemented in JavaScript is also implemented
in ECMAScript.
—
Reply to this email directly or view it on GitHub
https://github.com/exercism/xecmascript/pull/130#issuecomment-171996676.

--
Matt Morgan
Director of Technology
Dulcet Software LLC
(734) 926-8142

Anything there is fair game.
You can use http://x.exercism.io/problems/:slug (e.g. http://x.exercism.io/problems/clock) to see which tracks have implemented it, so you can go crib ideas from there.
Also, some of the tracks have a json file with inputs/outputs used in test suites over in the x-common directory (:slug.json).

I added a ticket for a feature that would have been useful here: https://github.com/exercism/x-api/issues/94


Rework the test hierarchy and make the TCK more consistent with HV testing

Note that it also includes a fix for BVTCK-125.
Once integrated, the HV CI will fail until all related fixes in HV are in. Don't worry about it, it's all under control!
https://hibernate.atlassian.net/browse/BVTCK-121

Don't worry about it, it's all under control!

:)

@gunnarmorling I answered your comments. I don't see anything to fix on my side.


Local Shadow information

Scrape all Shadow information and save as {csv? json?}. Makes it a lot easier to deal with in terms of program output and not having to rely on the wiki's wildly inconsistent formatting.

9547fb5d1cb9f33dd19eb749ccf78e7487588b0c adds JSON output. Don't want to use local info until I do some manual verification

json output for Shadow weakness/resistances from #10 looks good


[tests] Improve windows DHCP coverage

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

/kind enhancement
What happened:
We test linux pretty well with our internal DHCP server but we don't do the same for windows. This led to a regression: https://github.com/kubevirt/kubevirt/issues/6462.
What you expected to happen:
Let's add a few more windows tests around networking.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/remove-lifecycle stale

@dteplits / @AlonaKaplan can we close this issue, now that https://github.com/kubevirt/kubevirt/pull/6833 was merged ?

/close
#6833 added the requested tests .


Db/add taggable resource interface

Draft for now:
We expected similar behaviors and traits from these TaggableResources.
Putting these in a TaggableResourceInterface interface simplifies:

Getting the specific type of the TaggableResource
Getting the resource ID that the tag belongs to
Getting the specific tag ID from the list of tags belonging to the TaggableResource

@davidbloss is adding support for User and InfrastructureResource also in scope for this PR?

@davidbloss is adding support for User and InfrastructureResource also in scope for this PR?

Not for this PR. Those resources need Tag support introduced in the first place with tests.


Consider adding issue metrics action

https://github.com/marketplace/actions/issue-metrics

redundant with http://<IP_ADDRESS>:32001/connections


[UWP] UWP has several unit test failures

Platform

UWP

Details
The following UWP tests are failing in main:

[x]  FallbackTests (Fixed by #4627)

ColumnFallbackAsync (crashes test host)

[x] ObjectModelTest (Fixed by #4626)

ChoiceSetInput
DateInput
TextInput
ToggleInput

[x] UnitTests

TestAllCards (crashes test host)

[x] v1.2/Elements/AdaptiveCard.BackgroundImage.DataUri.json. Fixed by #4652
[x] v1.2/Elements/Column.Fallback.json. Same issue as FallbackTests failure. Fixed by #4627.
[x] Out of date round-trip json files causing test failures Fixed by #4645

All fixes are in and confirmed that UWP tests pass now in main.


awscli: 1.16.215 -> 1.16.263

Motivation for this change
To obtain a more recent version of the AWS CLI (updates from 9 Aug 2019 to 18 Oct 2019).
Things done

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

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @muflax

it's already on staging https://github.com/NixOS/nixpkgs/commit/b5623ef56f34fd4402b1695cee0eec80004ed4c7 albeit, 261 instead of 263. but boto causes 1000's of rebuilds, so closing for now.
Thanks for taking the time to create the PR though, hopefully staging gets merged soon

@jonringer Did I open against the wrong branch?

not really, it's just that the change got grouped with a lot of other large changes, and now that PR will take a while to make it's way into master :(


Fix typo in optmanager.py

Description
assigment -> assignment

Checklist

[ ] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

Thanks!


Fix off-by-one errors and integer interpretation errors.

With this pull request, I would like to correct some problems in marquee.js:

Changing / Fading colors (color 1 / rgb) were not taken over into the background feature
The integer in properties "16" + "3" + 1 were interpreted as string (1631) - added parseInt to ensure it is handled as a number
The edges of the marquee were set twice, each edge had it's own way to start over. This leads to each edge having a potentially inconsistent marquee pattern (see screenshot).
The selected marquee color is not applied but added (merged) with the background color. This significantly reduces the reachable color space for the marquee. It could be made an option (@sandinak, what's your opinion on this?), but if the user needs merged colors on the marquee, he could select the marquee merged color right away.

This is how it was looking before this pull request:

This is how it looks after this pull request:

Coverage: 28.084%. Remained the same when pulling caad1808b6e35f3c41aaf08f5dbd238cd76a804b on hjtappe:rgbScript-marquee into 25e992223a75e81c2da666400d492e21b0b8f6e6 on mcallegari:master.

Yes, if the amount of colors does not devide evenly, then there is a point in which you see the seaming point between start and end of the pattern.
One way around this would be to not provide the distance between the marque bulbs but the number of bulbs to be distributed along the edges. That could be handled much more evenly by the script through Math.round().
What we still could do is:

Take the wanted distance between marquee bulbs:
Calculate the amount of points to be spread along the edges: Math.floor(length / (distance + 1))
Calculate the mediumDistance between the points: length / amount
Use the mediumDistance to detect if a pixel should be highlighted (instead of the raw marqueeCount): p % mediumDistance < 1

Something like that, but I was not sure what the intention was and how normally marquee in the wild would handle that (probably by letting the user choose only distances which devide the length to a full number).
I can take care, but I am most probably on radio silence for the next two weeks.

I  was able to find the brief time for distributing the required spacers into the marquee so the seaming spot is no more as obvious. @sandinak, please briefly check if this is what you were looking for and if I might have missed a matrix size for which it might not work.

Thanks for all your work on this @hjtappe. This looks like a really nice matrix effect.

Thanks for all your work on this @hjtappe. This looks like a really nice matrix effect.

Kodos to @sandinak for introducing it. :-)

Thanks @sandinak and @hjtappe for the efforts
Please just let me know when this is ready to be merged

After the background fix today, I think it is ready to go. @sandinak can you please check again?

While I think let's increase the range limits. We often had feedback that arbitrary small limits cause unnecessary problems on users with larger Matrices.

While I think let's increase the range limits. We often had feedback that arbitrary small limits cause unnecessary problems on users with larger Matrices.

Well, keep in mind that for those knowing what JavaScript is, it will be super easy to edit the script and increase the limits if needed. I wouldn't worry too much about magic numbers right now.
If you're both happy, can I merge this?

Yes, Thank you.

👍

Thanks for all the hard work on this!


Can't set c files to open with vscode as default option on windows 11

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

VS Code Version:  1.81.1
OS Version: Windows 11 22621.2215

Steps to Reproduce:

Right click on a file with a .c extension
Click on open with,  then select visual studio code
The option to always use vscode and not just once does not appear, even though it does for other file types like java

I did check the windows settings for default apps, went to the "by file extension" menu and .c was not there as an option either.
I attached 3 images. One showing that you can't set vscode to be used always for c files (it wants to use notepad instead).
The second image shows the same method works for a java file. The third image shows that in the windows settings there's no .c option to set default apps by file extension.

I have no idea why that is on your machine, seems to work fine for me:

Please open a support ticket with the Windows team. Thanks!


Getting the hang up connection between nexrender-server and nexrender-worker if internet connection fails during communication 2 systems

Hi ,
I have running the nexrender server another system and worker another window server systems, Due to internet disconnection , I have getting the message :
{ FetchError: request to http://<IP_ADDRESS>8:6006/api/v1/jobs/pickup failed, reason: read ECONNRESET
at ClientRequest. (C:\Users\Administrator\AppData\Roaming\npm\node_modules@nexrender\worker\node_modules\node-fetch\lib\index.js:1455:11)
at ClientRequest.emit (events.js:198:13)
at Socket.socketErrorListener (_http_client.js:392:9)
at Socket.emit (events.js:198:13)
at emitErrorNT (internal/streams/destroy.js:91:8)
at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
at process._tickCallback (internal/process/next_tick.js:63:19)
message:
'request to http://<IP_ADDRESS>8:6006/api/v1/jobs/pickup failed, reason: read ECONNRESET',
type: 'system',
errno: 'ECONNRESET',
code: 'ECONNRESET' }
The message is still going on if the internet connection back on the worker computer and unable work or render any job until , we restart the window system or restart the nexrender worker again ... due to these process  I am facing a lot of hectic issues during the render process so please suggest the best way for ignore the re-start the nexrender worker manually process

I'm assuming that is something that happens within the internals of nodejs itself. So that would probably mean it requires the whole executable to be restarted.
That seems like a serious problem, and a surface-level search that I just did seems to indicate that it indeed not only related to nexrender. Unfortunately, I don't think I have an idea for solving this issue at that time. Maybe you have some suggestions?

Hi,
Sorry if i'm slightly off-topic but two things catched my attention here.

In the logs you provided: <IP_ADDRESS>8 is not a valid ipv4 adress. This might be related to your problem.
Your nexrender server at <IP_ADDRESS>  on port 6001 is publicly accessible. You may want to shield it ;)


mergeGeometries vs mergeBufferGeometries with Three 0.161.0

When i want to use the path tracer component I always got this error:
import * as BufferGeometryUtils from 'three/addons/utils/BufferGeometryUtils.js';

./node_modules/three-gpu-pathtracer/src/utils/GeometryPreparationUtils.js
Attempted import error: 'mergeBufferGeometries' is not exported from 'three/examples/jsm/utils/BufferGeometryUtils.js' (imported as 'mergeBufferGeometries').

I figured out that three/addons/utils/BufferGeometryUtils.js has no mergeBufferGeometries export just a mergeGeometries.
I used R3F 8.15.16 and Nextjs 14.1,0
Are those two exports identical? How can I eliminate the error?

Seems like an upsteeam issue. Please follow up with three-gpu-pathtracer


Adding user supplied CA for custom OIDC providers

Adding the ability to use your own CA cert for custom OIDC providers
From my understanding of the ticket, This PR can do the following

User can supply their own CA cert which will be placed within /certs and used to validate the auth of a custom OIDC provider by checking against the signed ID token and Access token issued by the provider.

I placed the cert within /certs as I heard /trusted-ca is used for TLS certs and this isn't TLS but just identity auth from what I understand
lmk if you prefer different var names or if you want the cert location to be /trusted-ca same with Trillian CA. I will create a test in user_provided just to check if the cert is being consumed correctly and placed within /certs. Not sure if there is a manual option to configure to check CA against an OIDC or it's configured to try automatically, but will also set if possible.

@SequeI Changes seem reasonable to me but you should create some sort of steps or example of configuring using Keycloak or equivalent.

I think this looks really good, I just let one minor comment inline; I also wanted to ask you whether you actually tested that this works. Thanks!

@bkabrda Tested, correctly get's consumed and formatted into a .pem file at /certs within the fulcio-server container alongside the other ingress certs. SSL ENV already instructs the golang crypto/tls package to look at this custom cert dir.


fix: property name updated to latest XRIT API TRNG-1397

Description

Fixes:

An issue where selecting an interactable object while been on the latest XRIT will throw multiple errors in the console.

An issue where setting a training scene throws a warning.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

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


Delete unnecessary spaces

When copying from the web page, there is an unnecessary space after the code.

@changeworld @Mikejo5000 - We're in the process of pulling these commits to the private repo, so we can verify the content on staging. We'll post a new PR # shortly and then close this PR.

@changeworld : Thanks for your contribution! The author, @mikejo5000, has been notified to review your proposed change.

@changeworld @Mikejo5000 - The commits in this PR were moved to the private repo (PR #3851) for validation and staging. We'll close this PR now.


intel_adsp: ace: do not use external kconfigs in code

use SOC_INTEL_ACE15_MTPM instead of CONFIG_ACE_VERSION_1_5.
CONFIG_ACE_VERSION_1_5 leaked from SOF.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

Should that be CONFIG_SOC_INTEL_ACE15_MTPM instead? (with CONFIG_ prefix)


Add tatar language

Task:
Add tatar language
Steps:

Prepare text by running make transprepare
Insert translation
Compile text by make compile

To work with translations, GNU gettext must be installed, otherwise error will occur


Add ostruct impl to stdlib

Imported from Ruby 2.6.3, rubocoped, and modified to work with mruby.
OpenStruct required implementing other parts of core:

String#[] with Regexp and Regexp + capture arguments

This PR is in support of GH-85.


[ISSUE #3434] Consumers start slowly

fix #3434 and #2738

Coverage increased (+0.1%) to 54.498% when pulling e2868b17ef40f7b78edcd07ba49c9a0e65db78f6 on Cczzzz:master into 1b420cedfd74cf47b788abb4599f98d2a5217faf on apache:master.

Codecov Report

Merging #3759 (e2868b1) into master (1b420ce) will increase coverage by 0.06%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master    #3759      +/-   ##
============================================
+ Coverage     48.15%   48.21%   +0.06%
- Complexity     4593     4605      +12
============================================
  Files           555      556       +1
  Lines         36755    36778      +23
  Branches       4841     4844       +3
============================================
+ Hits          17698    17733      +35
+ Misses        16826    16811      -15
- Partials       2231     2234       +3

Impacted Files
Coverage Δ

...rocketmq/client/impl/factory/MQClientInstance.java
51.47% <75.00%> (+1.84%)
:arrow_up:

...ocketmq/client/impl/consumer/HeartbeatService.java
81.81% <81.81%> (ø)

...in/java/org/apache/rocketmq/test/util/MQAdmin.java
38.88% <0.00%> (-5.56%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
58.11% <0.00%> (-1.71%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
78.21% <0.00%> (-1.49%)
:arrow_down:

...main/java/org/apache/rocketmq/store/CommitLog.java
73.83% <0.00%> (-0.12%)
:arrow_down:

...mq/client/impl/producer/DefaultMQProducerImpl.java
45.47% <0.00%> (+0.24%)
:arrow_up:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.92% <0.00%> (+0.31%)
:arrow_up:

...e/rocketmq/client/impl/consumer/RebalanceImpl.java
40.23% <0.00%> (+0.39%)
:arrow_up:

...he/rocketmq/client/impl/consumer/ProcessQueue.java
61.39% <0.00%> (+0.46%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

A cached queue of length 1 to ensure that the last task will be executed

It seems that there are some unnecessary commits in this pr.

It seems that there are some unnecessary commits in this pr.

It seems that there are some unnecessary commits in this pr.

It seems that there are some unnecessary commits in this pr.

that is because i  checkout -b from master

why it closed @RongtongJin


Update logback-classic to 1.2.7

Updates ch.qos.logback:logback-classic from 1.2.6 to 1.2.7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback", artifactId = "logback-classic" } ]

labels: test-library-update, early-semver-patch, semver-spec-patch

http://logback.qos.ch/news.html


Support basic and digest authentication

I noticed that asp .net core does not support basic or digest authentication, and at least for Basic it seems to be by design.
However note that there are some scenarios where there is a need for it, for example because the client software used to access the app does not support any other method. Also these are standardized.
So what should we do in case we need to use basic or digest authentication? Say I am creating a git repository server that requires authentication? Is the solution to use third party addons or to write our own auth handlers?

Either 3rd party ones, or write your own. It's not something we want to provide or support.

well just not quite understood why to make that path more difficult by not supporting it out of the box. Especially digest auth. I understand the motivation behind not recommending it, but not quite behind just outright refusing to support it even with a big warning. If someone needs it he will definitely provide it one way or another anyway.

We have limited resources, and securing those auth types would simply take too much time better served on other features.

Does it require so much efford? basic is insecure always, use ssl. :) over basic, if you really need it. not sure how much efford you need to secure the normally insecure Basic.

There's time in code, testing, security reviewing and documentation. Things like certificate and kerberos authentication take higher priority. Given there are community implementations this isn't something we feel is necessary for us to implement and support.

well, still these auth methods are also way simpler than both of them. Somehow it feels weird that the most basic as in, standardized and widely supported auth methods are the only ones not supported here. although it is mostly esthetic argument or something like this... or something about having to look for third party libraries for such stuff. But okay, understood.

It's not the simplicity it's the support cost with the potential insecurity. I'm afraid third party packages are your only option, heck I even wrote one myself.


Added reset content option to service's update method

@teslovych thx for the PR, i've release v1.3.1 which includes your changes.


🍏  test/image-tests: temporarily skip azure_rhui image testing

This manifest is intended only for internal use and is currently failing
in nightly pipelines. This will be moved to a different test script in
the future, see COMPOSER-1397.
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

This either needs admin merge from @ondrejbudai or rebase after https://github.com/osbuild/osbuild-composer/pull/2401 is merged


fix(*): update browser worker bindings

#830  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Brooooooklyn and the rest of your teammates on  Graphite


I sent Celo gota Coinbase to Coinlist and the transaction has been pending for over 2 hours

Describe your issue here.
Environment

Elixir Version: 1.9.0

Erlang Version: 22

BlockScout Version: d09fdbfd87d54ba1a80de57ea48c5c7e6f4f61f8

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0.3 Safari/605.1.15

Steps to reproduce
Tell us how to reproduce this issue. If possible, push up a branch to your fork with a regression test we can run to reproduce locally.
Expected Behaviour
Tell us what should happen.
Actual Behaviour
Tell us what happens instead.

@aaash91 did the transaction complete or is this still an issue?

@aaash91 I am closing this issue due to inactivity. Please reopen and let us know if it is still an issue.


Flash screen: Add bos picture

When starting up the app, add the bos logo for 3 seconds before showing home screen.

Added


ApplyConfiguration panic

Out of index panic occurs when the gateway channel in loraserver is set to be greater than 8.

if multiSFCounter >= channelCount {
					return gc, errors.New("gateway: exceeded maximum number of multi-sf channels")
				}

The error:
errors.New("gateway: exceeded maximum number of multi-sf channels")
is intentional as the sx1301 / sx1308 on your gateway does not support > 8 multi-spreadingfactor channels.

I see, but if the number of channels set on loraserver is >8, gateway-bridge will panic instead of returning an error.

should be changed to >= in source code

Ah, good catch! Would you like to make a pull-request to change this?

Closing as the pull-request fixing this has been merged.


Report bug in cache image

Prerequisites

[ ] I have written a descriptive issue title
[ ] I have verified that I am running the latest version of ImageProcessor
[ ] I have verified if the problem exist in both DEBUG and RELEASE mode
[ ] I have searched open and closed issues to ensure it has not already been reported

Description
 Stack trace:    at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
   at ImageProcessor.Web.Caching.DiskCache.<AddImageToCacheAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at ImageProcessor.Web.HttpModules.ImageProcessingModule.<ProcessImageAsync>d__34.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Web.TaskAsyncHelper.EndTask(IAsyncResult ar)
   at System.Web.HttpApplication.AsyncEventExecutionStep.OnAsyncEventCompletion(IAsyncResult ar)

Steps to Reproduce

System Configuration

ImageProcessor version: 2.5.4
ImageProcessor.Web version  4.8.5
Other ImageProcessor packages and versions:
Environment (Operating system, version and so on): win 2012
.NET Framework version: 4.6.1
Additional information:

Request URL: https://www.iranketab.ir:443/Images/Slides/76b5746fb04c4c1caf77574d0d4df862.jpg?width=660&height=300&mode=crop
Request path: /Images/Slides/76b5746fb04c4c1caf77574d0d4df862.jpg

Hi @MajidSafari
What was the actual error message? The image you've linked to is working for me and changing the querystring parameters works also.

Hi @JimBobSquarePants
Some times happen  this
: The process cannot access the file 'D:\RootWebPages\iranketab.ir\app_data\cache\1\7\c\0\e\f\17c0ef7b573f98e96b708945d944c53aff45e3af.jpg' because it is being used by another process.

    ImageProcessingModule.ValidatingRequest += (senderr, args) =>
        {
            if (!string.IsNullOrWhiteSpace(args.QueryString))
            {

                var queryCollection = HttpUtility.ParseQueryString(args.QueryString);

                queryCollection.Add("bgcolor", "fff");
                queryCollection.Add("contrast", "25");
                args.QueryString = queryCollection.ToString();

            }
        };

i just add this ,

What has this got to do with the raised issue? Please don't waste my time.


RFC: Allow a query to pull multiple time series

For a hypothetical query like this one:
SELECT
  'Experiment: ' + experiment_type as label,
  sample_unix_timestamp as timestamp,
  sample_value as value
FROM
  myExperimentsTable
WHERE
  sample_unix_timestamp BETWEEN $__from/1000 AND $__to/1000
  AND experiment_type in ('foo', 'bar', 'baz')

The query itself selects which timeseries are to appear, it prepares the data into 3 columns with predefined names.
Expectation:
The grafana-sqlite-datasource plugin should produce a graph that shows all 3 timeseries (my experiment_type) and the label for these timeseries should be the value I specify as the label column, they are time-indexed by the timestamp column and valued by the value column.
Exact column naming is up to the maintainers but ability to select multiple timeseries in one query is the essence of this RFC.

Sounds interesting and I'll take a look either way. What would really help me out though, would be an example query (preferably with a CTE as data source) that I could pop in to Postgres to see how they handle it.
I assume the Postgres data source (as an example of a native SQL data source) already implements the desired behavior?

I have an sqlite db file that contains my computer's loadavg 1, 5, 15 every
1 minute for several months. It contains a lot more, so I'll look into
extracting just those and simplifying the schema to the bare minimum.
As for PostgreSQL, I have no idea and I'm not interested. I'm comparing
your plugin to the Prometheus data source.
On Wed, Jun 22, 2022, 08:13 Sergej Herbert @.***> wrote:

Sounds interesting and I'll take a look either way. What would really help
me out though, would be an example query (preferably with a CTE as data
source) that I could pop in to Postgres to see how they handle it.
I assume the Postgres data source (as an example of a native SQL data
source) already implements the desired behavior?
—
Reply to this email directly, view it on GitHub
https://github.com/fr-ser/grafana-sqlite-datasource/issues/88#issuecomment-1163238913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIKO6EAWIUUDMGROEVHYR3VQMUQLANCNFSM5ZQR4WFA
.
You are receiving this because you authored the thread.Message ID:
@.***>

This file is the Sqlite3 DB that contains one table with 3 datasets: loadavgs.zip
This file is an example Grafana dashboard that queries and displays the data in the loadavgs.zip file: Sqlite Load Averages-1656180006210.zip
Image 1:

In the image above, the 3 time-series are added as individual Grafana query. It requires exact knowledge of how to query each individual time-series. This is the least ideal solution, but it is supported this way today. Note, that I'm not sure how to control the label showing loadavg1 loadavg1 instead of just loadavg1.
Image 2:

In the image above, the 3 time-series are added via a single SQL query in a single Grafana query. To achieve the 3 time-series in one, it is required by this plugin to have the 3 time-series as separate columns. This means a dataset that has all in one table needs to be transposed from an inner query, or other ways would require two joins. This also requires exact knowledge of the dataset contents and is tedious to use if we have something like 100 time-series.
The ideal should look and feel like Image 2, but the query should ideally look like this:
SELECT
  unix_timestamp,
  format("loadavg %d", avg_span_minutes) as label,
  value
from load_averages_t
where unix_timestamp between $__from/1000 and $__to/1000

This allows users to specify labels precisely using SQLite.
Or imagine a query such as this one:
SELECT
  sale_timestamp,
  region as label,
  sum(sale_amount) as value
from sales_t
where unix_timestamp between $__from/1000 and $__to/1000
group by sale_timestamp, region

looks like it would involve modifying pkg/plugin/variables.go to handle the case of all other variables in the query and replace them with the appropriately interpolated values (likely using reflect to discriminate between array/string values).  It may be that you can link against the sqlite go bindings and use their escaper directly.

replace(replace(replace('$var','{','("'),'}','")'),',','","')

Using that as a CTE looks to work for the multivariate case.  Not pretty, but may be the answer

Example of CTE that works with vars to have multi-series charts:
WITH RECURSIVE cte(org, part, rest, pos) AS (
  VALUES(replace('$process','}',''), '',replace('$process','{','')|| ',', 0)
  UNION ALL
  SELECT org,
         replace(SUBSTR(org, pos+1, INSTR(rest, ',') ),',',''),
         SUBSTR(rest, INSTR(rest, ',')+1),
         pos + INSTR(rest, ',')
  FROM cte
  WHERE INSTR(rest, ',') > 0
) SELECT
time,
name as displayName,
(rss / 1024) as rss
FROM procsummary
WHERE
name IN (select part from cte)
AND
time >= $__from / 1000 and time < $__to / 1000

Queries a table of processes by name with their rss in kbytes (reduced to mb), with the $process var being the distinct process names seen in the table.
Good enough for my purposes, but an IN style interpolation would certainly save a lot of effort.

This could be supported with variable expansions. E.G.
SELECT
name as label,
time,
value
FROM some_table
WHERE
time between $__from/1000 and $__to/1000
AND
name IN $names

where you would have the dashboard var $names populated by:
SELECT distinct(name) FROM some_table;

I am not sure how this addresses the initial use case of this issue. How is this related to showing the results as multiple labelled time series?

@corto25, after playing around a bit with your requirements it seems to me that they are already working (with the time series option instead of a table option in case you haven't tried that yet). The query below generates a result for me that splits the result into multiple series with one query
with data(time, temperature, city) AS (VALUES
(724125600, 10, 'London'), (724161600, 11, 'London'),
(724161600, 20, 'New York'), (724197600, 21, 'New York'),
(724125600, 25, 'Washington'), (724197600, 26, 'Washington')
) SELECT * from data
order by time

I could also use your sample data to generate such a view
SELECT
  unix_timestamp,
  format("loadavg %d", avg_span_minutes) as label,
  value
from load_averages_t
where unix_timestamp between<PHONE_NUMBER> and<PHONE_NUMBER>
order by unix_timestamp asc

Could you clarify for me if this solves your use case already or am I missing something?

Anyhow, having to write a recursive CTE in a WITH clause in order to filter by var easily is less than ideal.  I'll file a separate issue on that.

The main goal is to avoid having to specify which time series to include.
So as new data comes through, new time series will appear automatically.
RFC is to add support for the select timestamp, label, value ... query
style and produce individual lines for each distinct labels.
On Mon, Jun 27, 2022, 09:58 George S. Baugh @.***>
wrote:

This could be supported with variable expansions. E.G.
SELECT
name as label,
time,
value
FROM some_table
WHERE
time between $__from/1000 and $__to/1000
AND
name IN $names
where you would have the dashboard var $names populated by:
SELECT distinct(name) FROM some_table;
Variable expansions can be single values, multivariate or "all", which
will require different interpolations.
currently the plugin interploates single as the bare value, and multi/all
as the values in braces {} separated by commas.
as such, this is currently broken for multi/all variable selection.
Not sure what has to be changed to fix the interpolation here, assuming
this is just default behavior for plugins.
—
Reply to this email directly, view it on GitHub
https://github.com/fr-ser/grafana-sqlite-datasource/issues/88#issuecomment-1167499150,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIKO6ARXY6GWZREDL77F2DVRHMTVANCNFSM5ZQR4WFA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@corto25, after playing around a bit with your requirements it seems to me that they are already working (with the time series option instead of a table option in case you haven't tried that yet). [...]

@fr-ser I had not noticed that option. It looks like my RFC is already supported. The only comment I would have is to avoid putting the value column name as prefix for univariate timeseries (ie instead of "value loadavg5", just putting "loadavg5" would be fine).
Also, I've done a quick experiment with multivariate + multi-series together and the labelling quickly become awkward.
This here:
SELECT
  format("loadavg %d", avg_span_minutes) as label,
  unix_timestamp,
  value,
  value/2 as half

Intuitively I would expect 6 curves on the panel: "value loadavg1", "half loadavg1", and so on for loadavg5 and loadavg15. But the result is this:
half half loadavg1
half half loadavg15
half half loadavg5
value value loadavg1
value value loadavg15
value value loadavg5

I'm not sure if there's a way to control this; I haven't found one without overriding them individually.
Also, this may be a matter of preference, but I would rather have them display as "loadavg1 value" and "loadavg1 half" as this will sort things together in a more relevant way (though this is where I suppose people could disagree).
I'll be closing this ticket, thank you.

RESOLUTION: Feature is already supported. The dropdown menu that shows "Table" by default just below the Query Name (showing A, B, C, etc), when changed to "Time series" achieves the requirements of this RFC.


PIC-1795: remove OFFENCE table and OffenceEntity

@rschallapalli Something's gone wrong with the CI build... can you try force pushing to see if that kicks it into gear?


Is there a way use native input type=date field on mobile devices?

Is there a way to fall back to native input type="date" on devices that support them, or at least mobile devices?. Perhaps the default input field could be type="date" anyway?

I think we can use mobile-detect with Modernizr for v-if: use datepicker or native input
var md = new MobileDetect(navigator.userAgent),
    grade = md.mobileGrade();
Modernizr.addTest({
    mobile: !!md.mobile(),
    phone: !!md.phone(),
    tablet: !!md.tablet(),
    mobilegradea: grade === 'A'
});

I think this is best handled upstream in your own application


Use Icon for Edit Button

Replace "Edit" button for cells (lists, items) with an edit icon, like a pencil.
Add to Legal page

Commit: https://github.com/johneastman/todo/commit/15ad642b93b426ebe59c5e2b64659c7401a8ba06


Can not modify the UI

Component your question relates to

[ ] Frontend library
[ ] Frontend UI library

Background
I am attempting to modify the initial UI that appears when running Pixel Streaming from my local host. However, when trying to make any changes, such as replacing "Click to start" with "Click to Play," I am unable to locate the referenced file. The documentation is not very clear in this regard.
Question

Where can I find this file, and how can I modify it?
Do I need to perform any preliminary installations?

The "Click to start" text is set in the ui-library in src/Overlay/ConnectOverlay.ts
Most of the UI is created in this library and after changing the source you will need to rebuild the UI by either supplying the --build parameter to the platform script or by invoking webpack manually in some way.


base vs mix

According to https://github.com/tangrams/tangram-docs/blob/gh-pages/pages/styles.md a style can "inherit" or "descend" from one of the built-in styles, in which case you use base, or from another named style, in which case you use mix.
Is there any other difference between base and mix? Do I recall that mix can take a list of styles?
cc @bcamper

They are very distinct functions. (In a previous, pre-mix incarnation, these two functions were more closely bound into one feature, but that was not flexible enough.)
base defines the expected input geometry type, and the vertex and fragment shader used for rendering (which custom blocks are then injected into). base can only be one of polygons, lines, points, or text.
mix copies properties from one or more custom styles. You can't mix the base types like polygons, instead mix acts more like inheritance. But the best way to think about it is as something that copies properties from another style. For example, to create a new style called styleB that "inherits from" an existing style called styleA, and then adds custom shader blocks to styleB, you would do:
styleB:
   mix: styleA
   shaders:
      blocks:
         color: ...

You can change any property of a mix'ed style, including its base. For example, if you have a polygon-based style that has custom shader blocks that you want to apply to lines instead, you can create a line-based version like this:
fancy-lines:
   mix: fancy-polygons
   base: lines # change the base to lines

Or you could mix in an existing style, but disable lighting:
fancy-but-no-lighting:
   base: fancy
   lighting: false

If multiple styles are mix'ed, the properties are copied from them in order, with the later styles taking precedence over earlier ones, e.g. styleC properties will overwrite those in styleB (which will overwrite them in styleA):
styleD:
   mix: [styleA, styleB, styleC]

The mix parameter is also useful for mixing multiple effects together, e.g. to apply both the windows and halftone effects simultaneously:
weird:
   mix: windows, halftone

Excellent, thanks for the details – I'll update the docs.

fixed in https://github.com/tangrams/tangram-docs/commit/633a85212f9a0cf62677a091ebd8e572e6dfa257


Order by name when tied for openness

It seems that when LLMs are tied for openness score it's fairly random in which order they're appearing in the table. As an automated commit like this one shows, there are various changes to the table beyond just the single LLM I was adding.
Can we rank by openness, then within that by LLM name in consolidate_csv.py?

fix0red


Added TargetLib

Description
This library was created for implementing faster and reliable network ranges, generating random ips, check for double targetips, randomize targetlist.
Fixes #35
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I've edited the README.md and link to my code.
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@avinashkranjan Can you please add the DWOC and Level tags for this PR

Yes Yes, I forgot..Doing it now

Thanks a lot !!


CGFloat & NSInteger causing migration issues on device change & restore iCloud backup

We are seeing issues with properties defined as CGFloat or NSInteger in our realm.
If a user is on a 32 bit device (4S 5 or 5C) and then upgrades to a 64 bit device (5S, 6/6+ , 6S/6S+) and restores their phone from backup, that back up binary still has the realm for the 32 bit device and doesn't match the 64 bit device causing a migration failure.
Any reason why Realm allows CGFloat or NSInteger ? These types that are defined by arch type should not be allowed as properties.

There's no way for us to detect them. Code using CGFloat will compile to the exact same thing as code using whichever type it's an alias for on the current platform, since C typedefs don't actually create new types.

What we are seeing on our end is a user who is now on a 6S (previously on a 5 or 4S) crashing on launch with a migration error " Old type 'float', new type 'double' ".

and the properties responsible are of type CGFloat

I'll add a note to our documentation clarifying that using platform-dependent types will lead to having platform-dependent database files.
I'm sorry that was not clear, @adomanico.

OK thanks, I definitely overlooked this. Just curious: Why even allow them? Would it be simpler to just force the use of the primitive float, double , etc ?

Why even allow them? Would it be simpler to just force the use of the primitive float, double , etc ?

Not possible, as @tgoyne stated above:

There's no way for us to detect them. Code using CGFloat will compile to the exact same thing as code using whichever type it's an alias for on the current platform, since C typedefs don't actually create new types.

Ah ok so it all looks the same on your end. Ok thanks


SubscriberInfoIndex does not support sub-modules

I' have done an experiment, the result is that,  generated SubscriberInfoIndex does not include Subscriber in sub-modules.

You probably have to add the EventBus annotation processor to each submodule, then just add the index class for each one in your main project. -ut

No response. Closing. -ut


Add MLCS2k2 model

This could use some testing and docs, but I think we can merge it now.
I'd also like to avoid the transpose in read_3dgriddata_fits and possibly combine the function with read_griddata_fits, but again, we can look at that after merging.
cc @saurabhwjha

@kbarbary I think leaving the transpose in the might be okay. The problem seems to be that the FITS standard favors a column-major format. Since that FITS file might be used elsewhere, I feel like it should follow the FITS convention, and environments which use the opposite convention should be responsible for doing the transpose (e.g. Python). If I rewrote the file the other way around, someone's IDL code would then need to do the transpose... But I guess I could be convinced to do it.
https://astropy.readthedocs.org/en/v0.1/io/fits/appendix/faq.html#what-convention-does-pyfits-use-for-indexing-such-as-of-image-coordinates
Speaking of that file, I realized I used the wrong set of MLCS vectors to create it. I've updated it based on a better set (which is also the "default" MLCS2k2.v007 model in SNANA). If you have cached the mlcs2k2.modelflux.fits file, you should delete it and have it re-downloaded. Maybe I should rename it to something more informative about which set of vectors it is? And we can have multiple versions in the url json file?
Right now we haven't included the model errors in any way. I should do this in the same way it's done for SALT in sncosmo, but that will take some time, so it probably does make sense to merge now and have the model included.
Finally, I updated the demo notebook to work with the current way it is set up:
https://github.com/saurabhwjha/sncosmo-mlcs2k2-demo/blob/master/sncosmo MLCS2k2 demo.ipynb

I agree that the file should follow the FITS convention and that Python should adapt as needed. For example, this is what is done with images: In FITS, the "x" axis is NAXIS1 and the "y" axis is NAXIS2. In Python, we follow the FITS convention of the "x" axis being the axis contiguous in memory, so we simply index images like data[y, x], even though it looks a bit weird.
For this file, regardless of what is done in Python, it seems like wavelength should be the axis contigous in memory (NAXIS1), phase should be next (NAXIS2) and delta should be the slowest index (NAXIS3). I don't have a really good reason for this, except that one most often wants to access a 1-d spectrum slice of the array, which would then be a contiguous block of memory.
I'm aware of a FITS convention that "x" is NAXIS1 in images. Is there a similar convention for arrays like this that mix various dimensions (wavelength, time, other)?

The Travis failure is due to RegularGridInterpolator not being available in scipy < v0.14. We'd have to bump the minimum scipy version to 0.14. It was released May 2014, so that's pretty recent - unlikely to be available in many system package managers. An alternative would be to make the MLCS2k2 model optional functionality depending on the scipy version available.
Yet another alternative that I thought of recently would be to instantiate a RectBivariateSpline object for each value of delta and linearly interpolate between two delta values ourselves. This has the advantage of higher-order spline interpolation in phase and wavelength. Would probably be slower, but no slower than the SALT2 model.

I agree with this, and didn’t realize that NAXIS1 is contiguous in memory. I thought it was the other way around. Yes, we definitely want the wavelength axis contiguous.
Okay, so I’ll update the code that creates the FITS file and reverse it, and update the read3dgrid routine.

On Oct 12, 2015, at 2:24 pm, Kyle Barbary<EMAIL_ADDRESS>wrote:
I agree that the file should follow the FITS convention and that Python should adapt as needed. For example, this is what is done with images: In FITS, the "x" axis is NAXIS1 and the "y" axis is NAXIS2. In Python, we follow the FITS convention of the "x" axis being the axis contiguous in memory, so we simply index images like data[y, x], even though it looks a bit weird.
For this file, regardless of what is done in Python, it seems like wavelength should be the axis contigous in memory (NAXIS1), phase should be next (NAXIS2) and delta should be the slowest index (NAXIS3). I don't have a really good reason for this, except that one most often wants to access a 1-d spectrum slice of the array, which would then be a contiguous block of memory.
I'm aware of a FITS convention that "x" is NAXIS1 in images. Is there a similar convention for arrays like this that mix various dimensions (wavelength, time, other)?
—
Reply to this email directly or view it on GitHub.

sounds good!

Any progress on this, @saurabhwjha?

I've made a new FITS file (you will need to delete .astropy/cache/sncosmo/models/mlcs2k2/mlcs2k2.modelflux.fits and have it download the new version). This flips the order of the axes, and also is now 32 bit floating point instead of 64 bit (which was overkill) so that makes it half the size.
I've updated the read_3dgriddata_fits function to deal with this; no transpose anymore.
These are all committed on my fork: https://github.com/saurabhwjha/sncosmo/commit/5ad0e4e0979f423e7132226ec0f7e3a2355332a1

Great. One remaining item: do you want to put some sort of version number in the file name? That will make any future bug fixes easier for users: They won't have to manually delete anything.
For the SALT2 model for example, v1.0 was the original, v1.1 was a bug fix.

Okay, I've created a v1.0 MLCS2k2 model. I've added some HISTORY keywords in the FITS header to specify which MLCS model this is. In MLCS2k2 language, this SED corresponds to "MLCS2k2 v0.07 rv19-early-smix vectors".
The download link for v1.0 is:
http://www.physics.rutgers.edu/~saurabh/mlcs2k2.modelflux.v1.0.fits
The old file (without the v1.0) has been removed, so please update the urls.json file.

I changed the urls.json file and made a PR on your branch that should fix failing tests.

Travis failures are some unrelated problem with the conda path. Merging this.
Thanks Saurabh!


Clickhouse working slow

Q.1 > If I will do 10000 insert query in a single batch so that will consider as a single query or multiple queries?
Q.2 > In my current application I am facing an issue that insertion and SELECT at same time and in that I found merging working in the backend, So I am facing slowness from retrieving data from clickhouse and utilization goes up to 90-95 %. So for this, any solution or any option is there to handle this?

@KrunalT you need to put all N rows into single INSERT INTO statement, not N separate INSERT INTO statements with one row each. Aren't you doing the latter?

For insert, use a single transaction then run many insert, then commit.

@4ertus2
What about this ??
In my current application I am facing an issue that insertion and SELECT at same time and in that I found merging working in the backend, So I am facing slowness from retrieving data from clickhouse and utilization goes up to 90-95 %. So for this, any solution or any option is there to handle this?

You need to batch inserts. ClickHouse expects one INSERT statement per second or less. Each of them should contain lots rows of data. If you're inserting your data correctly and face with full hardware utilisation you probably need a better hardware.


Unexpected Object Type when parsing files

Why does the comment parser return a list instead of a TerraformObject?
https://github.com/wayfair-incubator/pyterraformer/blob/bd8b31405ab5949fea24ea76c59793c368485eab/pyterraformer/serializer/human_resources/engine.py#L327
When parsing through resources of a given file, we are expecting there to be a _type which is present on the Comment class; but that is found on the second element in the array. I feel like the purpose of this was to be able to identify a comment by it's row (likely for modification); but I think we should instead check for _type and then check metadata for start_pos (or something like that). This would keep us consistent with modifying actual ResourceObjects (where we check _type and then check tf_id).
That way, we could reliably return a list of TerraformObjects and ensure that we're properly handling any downstream checks / assignments. Thoughts?
For example, here is a list from an existing resource.tf file I have (each object is displaying by printing the type() and then the value)
['comment-0', comment(text="//")]
<class 'list'>
['comment-3', comment(text="//   Any of the Terraform modules can be utilized here as well as Terraform")]
<class 'list'>
['comment-79', comment(text="//   `resource` and `data` statements.")]
<class 'pyterraformer.core.resources.resource_object.ResourceObject'>
google_service_account(account_id="var.dataproc_service_account_name", display_name="var.dataproc_service_account_name", project="var.gcp_project_us")
<class 'pyterraformer.core.modules.ModuleObject'>
module(source=""https://artifactory.service.bo1.csnzoo.com/artifactory/terraform/modules/tf-mod-google-dataproc-cluster/tf-mod-google-dataproc-cluster_v3.0.2.tar.gz"", name=""tmori-test-cluster"", project="var.gcp_project_us", wayfair_vpc="var.dataproc_subnet["subnet_vpc_name"]", subnetwork="var.dataproc_subnet["subnet_name"]", region="local.region"<EMAIL_ADDRESS>comment-831="comment(text="#metastore_connection is utilizing the main dataproc_prod connection")", hive_metastore_connection_name="var.dataproc_metastore_connection", hive_warehouse_bucket="var.dataproc_warehouse_bucket", comment-1033="comment(text="#kms_key taken from ds_catalog_dataproc; will obviously need to be replaced")", hive_credential_kms_key=""projects/wf-gcp-us-ae-dataproc-prod/locations/us-central1/keyRings/hadoop-key-ring/cryptoKeys/hive-key"", labels="{}", history_bucket="var.dataproc_history_bucket", staging_bucket="var.dataproc_staging_bucket", autoscaling_enable="false")
<class 'list'>
['comment-1392', comment(text="# granting this custom SA ephemeral dataproc worker role; as it is the only custom role with the correct perms")]
<class 'list'>
['comment-1503', comment(text="# this should be changed to dataproc.worker when possible")]
<class 'pyterraformer.core.resources.resource_object.ResourceObject'>
google_project_iam_member(project="var.gcp_project_us", role=""organizations/825417849120/roles/tf_wf_ae_svc_ephemeral_dataproc_worker""<EMAIL_ADDRESS><class 'pyterraformer.core.resources.resource_object.ResourceObject'>

The error I receive is:
  File "C:\Users\tmori\AppData\Roaming\JetBrains\PyCharmCE2021.1\scratches\tmori_test_scratch.py", line 31, in <module>
    namespace.add_object(GoogleStorageBucket(
  File "C:\Users\tmori\wayfair-incubator\pyterraformer\pyterraformer\core\namespace.py", line 151, in add_object
    indexes = [
  File "C:\Users\tmori\wayfair-incubator\pyterraformer\pyterraformer\core\namespace.py", line 152, in <listcomp>
    idx for idx, val in enumerate(self.objects) if val._type == object._type
AttributeError: 'list' object has no attribute '_type'

For example, here is a list from an existing resource.tf file I have (each object is displaying by printing the type() and then the value)
['comment-0', comment(text="//")]
<class 'list'>
['comment-3', comment(text="//   Any of the Terraform modules can be utilized here as well as Terraform")]
<class 'list'>
['comment-79', comment(text="//   `resource` and `data` statements.")]
<class 'pyterraformer.core.resources.resource_object.ResourceObject'>
google_service_account(account_id="var.dataproc_service_account_name", display_name="var.dataproc_service_account_name", project="var.gcp_project_us")
<class 'pyterraformer.core.modules.ModuleObject'>
module(source=""https://artifactory.service.bo1.csnzoo.com/artifactory/terraform/modules/tf-mod-google-dataproc-cluster/tf-mod-google-dataproc-cluster_v3.0.2.tar.gz"", name=""tmori-test-cluster"", project="var.gcp_project_us", wayfair_vpc="var.dataproc_subnet["subnet_vpc_name"]", subnetwork="var.dataproc_subnet["subnet_name"]", region="local.region"<EMAIL_ADDRESS>comment-831="comment(text="#metastore_connection is utilizing the main dataproc_prod connection")", hive_metastore_connection_name="var.dataproc_metastore_connection", hive_warehouse_bucket="var.dataproc_warehouse_bucket", comment-1033="comment(text="#kms_key taken from ds_catalog_dataproc; will obviously need to be replaced")", hive_credential_kms_key=""projects/wf-gcp-us-ae-dataproc-prod/locations/us-central1/keyRings/hadoop-key-ring/cryptoKeys/hive-key"", labels="{}", history_bucket="var.dataproc_history_bucket", staging_bucket="var.dataproc_staging_bucket", autoscaling_enable="false")
<class 'list'>
['comment-1392', comment(text="# granting this custom SA ephemeral dataproc worker role; as it is the only custom role with the correct perms")]
<class 'list'>
['comment-1503', comment(text="# this should be changed to dataproc.worker when possible")]
<class 'pyterraformer.core.resources.resource_object.ResourceObject'>
google_project_iam_member(project="var.gcp_project_us", role=""organizations/825417849120/roles/tf_wf_ae_svc_ephemeral_dataproc_worker""<EMAIL_ADDRESS><class 'pyterraformer.core.resources.resource_object.ResourceObject'>

The error I receive is:
  File "C:\Users\tmori\AppData\Roaming\JetBrains\PyCharmCE2021.1\scratches\tmori_test_scratch.py", line 31, in <module>
    namespace.add_object(GoogleStorageBucket(
  File "C:\Users\tmori\wayfair-incubator\pyterraformer\pyterraformer\core\namespace.py", line 151, in add_object
    indexes = [
  File "C:\Users\tmori\wayfair-incubator\pyterraformer\pyterraformer\core\namespace.py", line 152, in <listcomp>
    idx for idx, val in enumerate(self.objects) if val._type == object._type
AttributeError: 'list' object has no attribute '_type'

This almost certainly relates to some original hacky handling to make sure that comments were re-serialized back to the correct location, since they can appear in arbitrary locations within other terraform objects. Do you have a basic sanitized example you could add on here to show the problem? Almost certainly a better to handle it, I think the addition of the metadata context on each object is where we want this.

@TMo484 can you approve the PR?

Approved
On Fri, Mar 24, 2023, 8:47 PM Ethan Dickinson @.***>
wrote:

@TMo484 https://github.com/TMo484 can you approve the PR?
—
Reply to this email directly, view it on GitHub
https://github.com/wayfair-incubator/pyterraformer/issues/14#issuecomment-1483641005,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJQLE7DTQIJX62DRYMNMN7TW5Y6CDANCNFSM6AAAAAAS5JNKFA
.
You are receiving this because you were mentioned.Message ID:
@.***>


chore: update to electron 13

[x] Build nightly exploration builds

Re smoke tests, make sure to merge in https://github.com/microsoft/vscode/commit/4cda850edf57f244260b7b534cbb8d7c3717f58a

They do fail after taking in the above commit, I will give this a try once internal builds are available for triggering insiders

Ready for review, plan is to merge next week to get coverage in July iteration insiders.

Weird that node.js still seems to be at 14.16.0 even though https://github.com/electron/electron/releases/tag/v13.0.0 indicates otherwise.

With E13 submitURL cannot be empty when uploadToServer is true, in smoke tests we disable crash reporter via the cli switch https://github.com/microsoft/vscode/blob/main/test/automation/src/code.ts#L143 but still end up calling crashReporter.start. I have cleaned up the logic by moving the deciding conditions earlier so that we make call to the crashReporter if it is disabled.
We only enable crash reporter if enable-crash-reporter is set or crash-reporter-directory argument is provided.

Yeah it is a typo in the release notes wrt node version, there is no version change in E13

But doesn't that mean we have to update all our launch scripts to set enable-crash-reporter if we want to enable it for our users?

enable-crash-reporter runtime argument defaults to enabled for all our users. Also there is no change in the conditions for crash reporter enablement in this PR, I have just moved the callsites earlier to avoid unnecessary call to crashreporter API.
Are you thinking of another case that breaks with this change ?

Sorry, I totally forgot we had this in argv.json.


I wish to know if exposing code in public projects can be whitelisted

[Enter feedback here]
Hi, I wish to know if code in public projects can be exposed on need basis since we want the project itself to be public and accessible to everyone, in particular Wikis and such, but the code we want to keep hidden. Is this possible to have this fine grained control?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 35cd88fb-ff0f-5c7a-4c69-e3bb850836b4
Version Independent ID: d29e4f48-37d9-c581-0fc5-91cc6829df1b
Content: Browse repos, files, branches, pushes, commits, pull requests - Azure DevOps Services
Content Source: docs/organizations/public/browse-code-public.md
Product: devops
Technology: devops-public-projects
GitHub Login: @steved0x
Microsoft Alias: sdanie

@Eldar1205 Thank you for the question.
For now you are required to use public projects, but this issue is currently under consideration with the product team.
Please refer to this post for more information.


"Transaction not found" during wallet sync

➜  hermes git:(failing-actions) ✗ cargo run --bin taker testnet
    Finished dev [unoptimized + debuginfo] target(s) in 0.26s
     Running `target/debug/taker testnet`
2021-11-02 09:03:28  INFO Initialized logger
2021-11-02 09:03:28  INFO Running version: 0.1.0-91-g854eec9
Error: Failed to sync wallet

Caused by:
    TransactionNotFound

Nothing can be done except clearing the database if the taker got itself to this state.

on another occasion, I got maker to fail this way too.

Logs from trying to setup the CFD that failed:
2021-11-03 16:18:36  INFO Taking current order: Order { id: OrderId(Hyphenated(205bb128-7e50-466a-8082-6a69bcabcf21)), trading_pair: BtcUsd, position: Short, price: Price(63870.89), min_quantity: Usd(10), max_quantity: Usd(100), leverage: Leverage(2), liquidation_price: Price(42580.593333333333333333333333), creation_timestamp: Timestamp(1635918504), settlement_time_interval_hours: Duration { seconds: 86400, nanoseconds: 0 }, origin: Theirs, oracle_event_id: BitMexPriceEventId { timestamp: OffsetDateTime { utc_datetime: PrimitiveDateTime { date: Date { year: 2021, ordinal: 308 }, time: Time { hour: 6, minute: 0, second: 0, nanosecond: 0 } }, offset: UtcOffset { hours: 0, minutes: 0, seconds: 0 } }, digits: 20 } }
2021-11-03 16:18:36 DEBUG Taker wants to take an order taker_id=15bb39a3-f8be-4920-9601-801678b0ab69 quantity=10 order_id=205bb128-7e50-466a-8082-6a69bcabcf21
2021-11-03 16:18:41  INFO POST /api/cfd/205bb128-7e50-466a-8082-6a69bcabcf21/acceptOrder:
2021-11-03 16:18:41 DEBUG Maker accepts an order order_id=205bb128-7e50-466a-8082-6a69bcabcf21
2021-11-03 16:18:41  INFO Order got accepted order_id=205bb128-7e50-466a-8082-6a69bcabcf21
2021-11-03 16:18:43  INFO Exchanged setup parameters
2021-11-03 16:18:47  INFO Created CFD transactions
2021-11-03 16:18:47  INFO Exchanged setup parameters
2021-11-03 16:18:50  INFO Created CFD transactions
2021-11-03 16:18:50  INFO Exchanged CFD transactions
2021-11-03 16:18:56  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:19:06  INFO Verified all signatures
2021-11-03 16:19:06  INFO Exchanged CFD transactions
2021-11-03 16:19:09  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:19:22  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:19:22  INFO Verified all signatures
2021-11-03 16:19:34  INFO Exchanged signed lock transaction
2021-11-03 16:19:35  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:19:35 ERROR Message handler failed: Broadcasting transaction failed. Txid: 5b8aa7a30faa7990934643d03c77f2df4b599a4f3d1cb610113c411c6afb848d. Raw transaction:<PHONE_NUMBER>0102ae7d66ee5b112e4f36a97d8a953dc46b174c974bc98901cb837d5c7c8425d53e0000000000ffffffff1a2ae6fcd6f1e89c2b31ee99bec1b496d8d91d229e2ccda8e37641250e61d04f0200000000ffffffff03bd5b000000000000220020364148ab8cfc7f9d2450deae9b9b4360384449cd52f9f88f3b3a2bdf4cf352b1704801000000000016001408d6888fb407085dd980263d108c8f2ffb40748e58a60e0000000000160014e63130c4005cd765b5a17399f832b637e3b5d9150247304402202e34d7508deb7875a304561ef0eeb44a7f7b54f4e3b9a16fadd115503eeab624022047d73259de8824feb2ff845768b527b9b647a82d55f645b56cc42b2813058fc301210260f5006a8a78edbd59e160ad3dad1ce6316eb15a9e66940d6339c203d5bd367a02483045022100d7cdeb5fe6b1e21642e3ef86b89a4238cf1272c1991e3eea00dcec2781b8e8570220258af1b3828bf89549c6008b7138510ce2fe92a990790d8784cf75d02a8c602c0121034697212cf729d2e97a0b578fabee6ad48fb5eff99ee0d48a632e3ae21a77fff600000000: Electrum(Protocol(String("sendrawtransaction RPC error: {\"code\":-25,\"message\":\"bad-txns-inputs-missingorspent\"}")))
2021-11-03 16:19:46  INFO Exchanged signed lock transaction
2021-11-03 16:19:46  INFO Setup complete, publishing on chain now
2021-11-03 16:19:48  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:19:49 ERROR Message handler failed: Broadcasting transaction failed. Txid: 5b8aa7a30faa7990934643d03c77f2df4b599a4f3d1cb610113c411c6afb848d. Raw transaction:<PHONE_NUMBER>0102ae7d66ee5b112e4f36a97d8a953dc46b174c974bc98901cb837d5c7c8425d53e0000000000ffffffff1a2ae6fcd6f1e89c2b31ee99bec1b496d8d91d229e2ccda8e37641250e61d04f0200000000ffffffff03bd5b000000000000220020364148ab8cfc7f9d2450deae9b9b4360384449cd52f9f88f3b3a2bdf4cf352b1704801000000000016001408d6888fb407085dd980263d108c8f2ffb40748e58a60e0000000000160014e63130c4005cd765b5a17399f832b637e3b5d9150247304402202e34d7508deb7875a304561ef0eeb44a7f7b54f4e3b9a16fadd115503eeab624022047d73259de8824feb2ff845768b527b9b647a82d55f645b56cc42b2813058fc301210260f5006a8a78edbd59e160ad3dad1ce6316eb15a9e66940d6339c203d5bd367a02483045022100d7cdeb5fe6b1e21642e3ef86b89a4238cf1272c1991e3eea00dcec2781b8e8570220258af1b3828bf89549c6008b7138510ce2fe92a990790d8784cf75d02a8c602c0121034697212cf729d2e97a0b578fabee6ad48fb5eff99ee0d48a632e3ae21a77fff600000000: Electrum(Protocol(String("sendrawtransaction RPC error: {\"code\":-25,\"message\":\"bad-txns-inputs-missingorspent\"}")))
2021-11-03 16:20:01  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound
2021-11-03 16:20:04 DEBUG Got notification for new block block_height=2102303
2021-11-03 16:20:05 DEBUG Got notification for new block block_height=2102303
2021-11-03 16:20:14  WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound

After successfully opening a CFD and restarting I'm forever stuck on:
~/github/itchysats/itchysats(master ✔) cargo run --bin taker -- --maker testnet.itchysats.network:9999 --maker-id 69a42aa90da8b065b9532b62bff940a3ba07dbbb11d4482c7db83a7e049a9f1e testnet
    Finished dev [unoptimized + debuginfo] target(s) in 0.10s
     Running `target/debug/taker --maker 'testnet.itchysats.network:9999' --maker-id 69a42aa90da8b065b9532b62bff940a3ba07dbbb11d4482c7db83a7e049a9f1e testnet`
2021-12-06 09:14:21  INFO daemon::logger: Initialized logger
2021-12-06 09:14:21  INFO taker: Running version: preview-241-g8914c38
Error: Failed to sync wallet

Caused by:
    TransactionNotFound

I don't have the logs of the original run, but I did get warnings like these:
WARN Failed to sync wallet: Failed to sync wallet: TransactionNotFound

If we're aware of this, is there any way to recover from this state?

Nothing can be done except clearing the database if the taker got itself to this state.

I guess we can't do anything :( This seems like a big problem, especially since nothing went wrong on this previous run as far as I could tell.


TempDir Compilation for class.

Use a tempdir compilation for classes to avoid parent rcode in destdir

Not merge for now,
It's a best way to use "present" selector with mapper.


Initial build fails with column size errors

Describe the bug
When running dbt build --select package:dbt_project_evaluator, I get a few errors when the package tries to run staging models related to column size.

Expected results
All models run.
Actual results
Models do not run.
Screenshots and log output

System information
The contents of your packages.yml file:
  - package: dbt-labs/dbt_project_evaluator
    version: 0.6.0

Which database are you using dbt with?

[ ] postgres
[x] redshift
[ ] bigquery
[ ] snowflake
[ ] trino/starburst
[ ] other (specify: ____________)

The output of dbt --version:
Core:
  - installed: 1.3.1
  - latest:    1.8.2 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Plugins:
  - postgres: 1.3.7 - Update available!
  - redshift: 1.3.1 - Update available!

  At least one plugin is out of date or incompatible with dbt-core.
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Are you interested in contributing the fix?
No

Thanks for the report.
I can see that you are on a pretty old version of the package though and we did quite a few changes recently on how we handle string/varchar in Redshfit (for example like here)
Would you be able to upgrade the package version? This will require also upgrading your dbt version most likely so I don't know if it is possible for you.

I'm also running into this error, and made a work-around by changing the dbt.type_string() calls to dbt.type_large_string() in models base_node_columns and base_source_columns

Should be closed with #475 I believe. We can reopen it otherwise.


Evaluator performance regression

Depending on the project the evaluator execution time increased by about 150% to 250% between 2021-08-19 and 2021-09-03.
This matches the time when the analyzer result switched from the legacy dependency representation to the dependency graph, so it might be related. For larger projects I observed execution times of 35+ minutes.
This tremendously increases the feedback cycle for making changes to the configuration and thus negatively affects scan support work.

Can you still reproduce the issue @fviernau, or can we close this?

@sschuberth this was reproducible with any larger ORT file and as there haven't been much changes in the dependency graph I guess this is still valid. Shall we keep it open to at least have some reminder on where to look for performance gains / problems?


Where is the DINO version？

I skipped including and testing it in this repo. You can refer to the paper and https://github.com/ShirAmir/dino-vit-features.


Create Translator HiNDI to English

📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

Dear @Abhishek-090 , thanks for your contribution. We found some issues in your app, as the screenshots show. Could you please update your app and reply to this comment?

"Dear Team,
🙏@wenzhuo2022
I'm pleased to inform you that I have successfully addressed and resolved the issues mentioned in the previous comments and attached screenshots. The necessary updates have been made to the app, ensuring its improved functionality and performance.
I invite you to review the latest version and provide any additional feedback or suggestions if needed. Your input is highly valuable, and I'm committed to delivering a high-quality contribution to the project.
Please feel free to reach out if you have any further questions or require additional information.
🙏Thank you for your guidance and support.
Best regards,
@Abhishek-090

I have fixed the issue.
On Thu, Oct 19, 2023 at 9:05 AM Mikiko @.***> wrote:

Dear @Abhishek-090 https://github.com/Abhishek-090 , thanks for your
contribution. We found some issues in your app, as the screenshots show.
Could you please update your app and reply to this comment?
[image: image]
https://user-images.githubusercontent.com/112603073/276472839-6c39809f-793c-45d3-8ab9-b1b3d325775f.png
[image: image]
https://user-images.githubusercontent.com/112603073/276472983-52644436-f771-463c-b70f-0dfb888eacc8.png
—
Reply to this email directly, view it on GitHub
https://github.com/illacloud/illa-builder/pull/2874#issuecomment-1769838011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXUHFBLEIS6JABKMP4UKL5LYACNYXAVCNFSM6AAAAAA6BCE32CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRZHAZTQMBRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Skip flaky shuffle tests

Also upgrades the nightly version on Travis - the old one has a compiler bug that has since been fixed (https://travis-ci.org/mit-pdos/distributary/builds/358092643?utm_source=github_status&utm_medium=notification).

Seems good! We shouldn't forget about these tests though, as getting them to pass eventually is important.


#2308 Go-to only numbers fix

Checking whether or not the line is a positive integer if yes then move cursor and center it. Otherwise log an error and show the console

Awesome, thanks for the PR!
I'm hesitant to open the console on an error, but should be able to take a look at everything over the weekend. Worst case we leave the console in whatever state the user had it in and log the less verbose error.

Ouuu. I didn't know that there is such thing as "notifos". I find notifos a better approach here than writing it to the console, as I think console should be used for more "important" things.

This looks great @denis631! Notifos seems like a perfect fit here.
Just the two minor changes and we should be all set.


Fix for picker showing and hiding again

Summary
I am proposing a reversion of the merge #2127, which addressed issue #2126. This commit made flatpickr use 'click' events instead of 'mousedown' events in order to be compatible with WCAG 2.1, specifically 2.5.2 Pointer Cancellation.
Reasoning
This change has introduced some strange behaviour in flatpickr because of the way that 'click' works. The fact that flatpickr runs identical events for both focus and click events ensures that compatibility with WCAG 2.1 - 2.5.2 is not possible. Think of the following scenario:
A user 'clicks' into a flatpickr input field:

The browser fires a Focus event - flatpickr runs self.open (therefore invalidating WCAG 2.1 compatibility as there is no way of cancelling this event)
The browser fires a Mousedown event - the element under the pointer is 'self._input'. The target of the event is registered as such. Flatpickr does nothing, but is still drawing the calendar since self.open got called in the Focus event
Flatpickr finishes drawing the calendar, and the div is visible underneath the pointer
The user releases the pointing device,  triggering a 'mouseup' event. Because the user took ~50ms to release the 'click', the 'mouseup' target element is now the flatpickrcalendar div.
The 'click' event is fired, but with the document as the target instead of the self._input, because the overall target of the combination of 'mousedown' and 'mouseup' was the document, not self._input. This means that self.open is not run a second time, but the documentClick method is run (because this is registered to run on a document.onClick).
documentClick sees that the picker is open, figures that the 'click' event occured outside of the flatpickr element and forces the flatpickr to close.

TL;DR - If you click too slowly, and flatpickr draws under your mouse, the browser gets confused and thinks you're clicking somewhere else, closing flatpickr.
Solution
The solution to this is to move back to using 'mousedown' events. This ensures the correct event target is being used every time. While this does mean that in theory flatpickr isn't compatible with WCAG 2.1 - 2.5.2, the reality is that this is already moot as the plugin does a draw on the Focus event, meaning that the actual behaviour of the plugin is as if it is using 'mousedown' anyway, and it doesn't allow for cancellation of that event (except in the exact case detailed above, which isn't desired behaviour).
This PR directly addresses issue #2286. I thought of making this more complicated by using some sort of latestMousedown Target buffer on the click event or something, but it all ends up in the same place; in my opinion moving back to 'mousedown' is the most elegant solution.

@edwardsteven good points, will definitely consider this.
Some issues that came up after changing to click (e.g. the macOS bug) were totally unanticipated.

Thank you @edwardsteven @chmln
Do you have a date when these changes are going to be released?

@Hramkanvas around this weekend

@chmln do you need some help for releasing new version? I really need this fix


Model size mismatch error during prediction

There is a stem size mismatch error when running the prediction. I have trained the model using the phase images only.
You can find my training config here:
/hpc/projects/comp.micro/infected_cell_imaging/Single_cell_phenotyping/code_testing_soorya/fit.yml
Prediction config is here:
/hpc/projects/comp.micro/infected_cell_imaging/Single_cell_phenotyping/code_testing_soorya/output/predict/predict.yml
The slurm output is here:
/hpc/projects/comp.micro/infected_cell_imaging/Single_cell_phenotyping/code_testing_soorya/output/predict/slurm-15752038.out
If required the training output including the checkpoints are here:
/hpc/projects/comp.micro/infected_cell_imaging/Single_cell_phenotyping/code_testing_soorya/lightning_logs/vanilla_new/

@Soorya19Pradeep this error is likely due to a mismatch in the number of z-slices you specify during model construction.
Your fit.yml has this config:
model:
  backbone: convnext_tiny
  in_channels: 1
  log_batches_per_epoch: 3
  log_samples_per_batch: 3

predict.yml:
model:
  backbone: convnext_tiny
  in_channels: 1
  in_stack_depth: 5
  stem_kernel_size: [5, 4, 4]

This means that your model was trained with 15 z-slices, which is the default in ContrastiveModule
class ContrastiveModule(LightningModule):
    """Contrastive Learning Model for self-supervised learning."""

    def __init__(
        self,
        backbone: str = "convnext_tiny",
        loss_function: Union[
            nn.Module, nn.CosineEmbeddingLoss, nn.TripletMarginLoss
        ] = nn.TripletMarginLoss(),
        margin: float = 0.5,
        lr: float = 1e-3,
        schedule: Literal["WarmupCosine", "Constant"] = "Constant",
        log_batches_per_epoch: int = 8,
        log_samples_per_batch: int = 1,
        in_channels: int = 1,
        example_input_yx_shape: Sequence[int] = (256, 256),
        in_stack_depth: int = 15,
        stem_kernel_size: tuple[int, int, int] = (5, 4, 4),
        embedding_len: int = 256,
        predict: bool = False,
        drop_path_rate: float = 0.2,
    ) -> None:

If you don't override any the model construction parameters via config, they will be assigned like above. Deleting the   in_stack_depth: 5 from predict.yml could resolve this.

The channel dimensions are mismtached by a factor of 3 (96/32), matching the ratio of z-slices in two configs.
RuntimeError: Error(s) in loading state_dict for ContrastiveModule:
        size mismatch for model.stem.conv.weight: copying a param with shape torch.Size([32, 1, 5, 4, 4]) from checkpoint, the shape in current model is torch.Size([96, 1, 5, 4, 4]).
        size mismatch for model.stem.conv.bias: copying a param with shape torch.Size([32]) from checkpoint, the shape in current model is torch.Size([96]).

Thanks @mattersoflight ! Yes, it was due to the mismatch in the number of z slices. I corrected it and the predictions are running now.


Install tools from knative/toolbox repo and bump stuff

/cc @dprotaso @kvmware @cardil
Part of https://github.com/kubernetes/k8s.io/issues/2396

/lgtm
we can revist the toolbox versioning when @cardil  can take a look


[BEAM-10812] fix(beam/sdks/go): fix buffer limit on textio

bufio.Scanner is limited to 4kb by default, for arbitary line length it
is recommended to use bufio.Reader and split by '\n'
Ref: https://github.com/golang/go/blob/master/src/bufio/scan.go#L27
issue identifyed by @salrashid123
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[X ] Choose reviewer(s) and mention them in a comment (R: @username).
[ X] 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: @youngoli
cc: @TheNeuralBit

Run Go PostCommit

Thank you for contributing to Beam! Welcome to the community! :)


Add SelfServiceRegistrationValidation lambda type

Added SelfServiceRegistrationValidation lambda type

@robotdan can you review this please.

@matthewhartstonge can you review this please.

Hi @hamxabaig thanks for the contribution. Please do note that we will review this PR, but it is not necessary to add mentions, especially within the first day or two the PR is opened.
Thanks!

@hamxabaig I can see the PR merged!


Gitignore _ instead of .

Describe the bug
The git ignore file is called _gitignore rather than .gitignore in the templates which is causing changes to be pushed to GitHub that don't need to be.
I'm not sure if this is just a windows, vscode issue or why it isn't working, but the _ is not catching the gitignore until it is named the .gitignore
Reproduction
n/a
System Info
windows

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

npm init vite my-app -- --template svelte (or any other init) should automatically rename _gitignore to .gitignore.
see  https://github.com/vitejs/vite/blob/main/packages/create-vite/index.js#L128 https://github.com/vitejs/vite/blob/main/packages/create-vite/index.js#L248-L262
This is working for me on linux with<EMAIL_ADDRESS>Is it possible that this is a windows bug?

It is working for me on windows, too.


When focusing hover, I don't hear a hint

Testing #188537
I cannot verify this step:

You should hear a hint about opening the notification in the accessible view.

When focusing the hover, the screenreader announces this:

(property) TrackingTokenizationStateStore._invalidEndStatesLineNumbers: RangePriorityQueueImpl  tool tip

Sometimes I get text like that, and sometimes I only get "you are currently on a tooltip" and nothing else

Do you both of screen reader mode enabled?

Yes

I believe you are trying this via mouse hover and click.
I should've specified that this will only work when navigating via keyboard - as that is what most screen reader users do.
Screen reader users will access the hover via ctrl + k + i or cmd +k +i. that's when you'll hear this.

For low vision users, maybe it would be helpful to support this for tooltips as well, but it's not a big priority since most SR users navigate via the keyboard

I was using the keyboard shortcut to show the hover

Maybe I'm setting the wrong aria label


Is this project still maintained?

Hi there,
thank you for such work. It's stunning and I'm very interested in playing with it. Still, before starting I checked a bit the activity around and it looks like the projet is not really moving forward at the moment. Was it a one-shot project or is there any community around maintaining it?
cheers

Wondering as well.
I am looking for a tool/framework to create a small but fun and engaging questionaire this popped in google. Looks very promising

Looks to be one shot at this point.

I just implemented it and it works great for what i needed.

I guess we all have the answer now. I close the ticket.


make lang file available to fork and translate

Plugin Version:

Request:

Make the lang files available on the GitHub page so users can fork the project and translate the files and create a pull request.
How/Why:

Because it would make adding support for new languages easier.
Additional Context:

Sorry for the long delay O-O
Haven't looked on Github for a while, I did just add the lang file on here for ya :)