text:
Support for marked wait

Often times retrying is done in a stepping manner whereupon
the first N attempts wait X ms, the next M attempts wait for Y ms,
etc. This patch implements support for such waiting by introducing
the notion of 'wait markers'. Wait markers are simply tuples that
define a set of attempts and their corresponding waits to go into
effect.
A handful of unit tests are also included.

@jd @harlowja @rholder
I realize this PR has conflicts now, but I'd like to understand your initial impressions of this code to determine if it's worth my time continuing here.
Thanks

This looks simple enough to be done with a custom wait_func so I wonder if it's worth it?

@jd well almost anything in this respect could be done with a custom wait func couldn't it? : )
The idea was to enhance the base functionality of retrying by introducing this flexible mechanism.
IMHO it could be pretty useful.

You're right. I think we should just stop adding more kwargs to init, and just use partial to provide pre-defined wait_funcs in this regard.

@jd sorry I'm not sure I completely follow your last comment.
Are you inferring we should update how retrying supports waiting, or are you suggesting in cases like this (add new wait behavior) we house custom the logic in the consumer and just use a consumer's custom wait func?

My suggestion is to do this
@retrying.retry(wait_func=retrying.wait_markers((2, 10), (4, 20)))
def foobar(): pass

and in retrying.py
def wait_markers(*args):
   @wraps
    def _wrap(f):
         sleep(using args)
   return _wrap

See what I mean? :)

Yeah so what I referred to (perhaps poorly) as housing the logic in the consumer. I'm fine with that, but does your comment about not adding any more args to init infer that maybe retrying could use a little redesign/refactor :)

We all know it needs some refactor. We're waiting for @rholder to merge PR and release (more often) :)

I'll close this one out then; as we agreed for now the custom wait func won't live in retrying.
If I'm incorrectly closing this, please feel free to reopen.
Thanks

Don't get me wrong: I'd be ok to have it living in retrying, just not added in init kwargs.
My example above shows that :)

authors: bodenr jd

repo: rholder/retrying


text:
Move up to Bevy 0.14

Moved up the version for anyone looking to use it on the latest. However, there are a couple of notable things.
The output isn't what I'd expect so I suspect something is off (I'm not familiar with this level of bevy so just moving up what I know) i.e.

Additionally voxel_renderer.rs:36
-  .add_render_command::<Opaque3d, DrawVoxel>()
+ .add_render_command::<Transparent3d, DrawVoxel>()

This change may want to go back to using Opaque for batching and performance, but given the artifacts above I wasn't sure. I tried both, but no result.
Otherwise leme know if anyone makes any progress on this as it would massively save me time writing my own or making changes I really don't understand enough yet.

Thanks for the pull request! I'll have to update the voxelization at some other point. The issue with the rendering is just because bevy changed the cuboid mesh to only have a half size instead of two corners. This means the vertex coordinates weren't going from 0 to 1 like the shader was expecting.

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
Bug: TZOFFSETTO in VTIMEZONE is incorrect for DAYLIGHT component

Hi,
This issue seems to affect timezones with a positive timezone offset, in my specific case it's affecting daylight savings time differences between AEST+10 and AEST+11(DST).
Instead of having TZOFFSETTO set to +11 as expected, it's actually +09. This is causing some Outlook calendars to record the event two hours earlier than expected.
I believe the cause of this issue is in VTimeZone.cs on line 156:
if (isDaylight)

            {
                timeZoneInfo.Name = "DAYLIGHT";
                timeZoneInfo.OffsetFrom = new UtcOffset(utcOffset);
                timeZoneInfo.OffsetTo = new UtcOffset(utcOffset - delta);
            }

By subtracting the difference in offsets, rather than adding it, the time is moved further back instead of being pushed forward.
I have taken a fork of this project and reversed the operation on the same line of code. This seems to have resolved the problem for my case, but I'm not confident this will work for all other cases.
When running the unit tests in debug mode I noticed several cases of delta being negative, which caused a similar issue as I am experiencing, but for timezones with a negative offset instead.
Do you have any advice on how to resolve this? I am considering just forcing delta to always be negative which means the current subtraction operation will give a correct result.
E.g., adding this in just before subtracting delta to set timeZoneInfo.OffsetTo:
if (delta > new TimeSpan()) delta = delta.Negate();

I've had an issue too with "Australia/Melbourne" timezone and tried negating the delta at first, but calendars with older records still had issues.
I haven't quite got to the bottom of this but what I have below seems more stable.
This moves the isDaylight declaration up a few lines and the initial delta is created based on that.
It actually seemed to be related to previousInternal being null as well, but I opted not rewrite the entire if...else block.
var previousInterval = intervals.SingleOrDefault(x => (x.HasEnd ? x.End : Instant.MaxValue) == oldestInterval.Start);

var isDaylight = oldestInterval.Savings.Ticks > 0;

var delta = new TimeSpan(isDaylight ? -1 : 1, 0, 0);

if (previousInterval != null)
{
    delta = (previousInterval.WallOffset - oldestInterval.WallOffset).ToTimeSpan();
}
else if (isOnlyInterval)
{
    delta = new TimeSpan();
}

I had a similar issue. To create the timezone object I was trying to be efficient by using the overload VTimeZone.FromLocalTimeZone(DateTime earlistDateTimeToSupport, bool includeHistoricalData), with earlistDateTimeToSupport set to 2020-08-17, includeHistoricalData set to false, and my timezone being "Europe/Amsterdam". After pulling out my hairs I discovered that the issue did not occur for me for overload VTimeZone.FromLocalTimeZone(). Turns out that VTimeZone.FromLocalTimeZone() is internally using january 1st of the current year as value for earlistDateTimeToSupport, in my case 2020-01-01, as seen in VTimeZone.cs line 21.
For my timezone in the northern hemisphere 2020-08-17 has DST enabled while 2020-01-01 has not. And for @bjornlyngwa timezone AEST and @Y90SMH "Australia/Melbourne" in the southern hemisphere it's just the other way around. I tried these timezones with earlistDateTimeToSupport set to 2020-08-17 and this made the issue disappear.
The underlying issue seems to be that VTimeZone.cs line 133 fails to lookup previousInterval when a DST date is used for earlistDateTimeToSupport because the previous non-DST interval has not been provided by the calling method, and because of that the proper delta cannot de determined in line 139.
Hope this will others as a workaround, and the dev to fix the issue.

source: gharchive/issue

license: mit

license_source: bigquery


text:
microservice and docker

Hi, please introduce my name is Naray Citra from Indonesia. I learned Golang since 2 weeks ago, same as you my programming experience is Java with SpringBoot :). I found your started kit by searching google :). I like your stater kit, but i still newbie in Golang, i have some questions for you. How we separate the pkg into microservice and containerized it? Can you give an example for that?

If you're quite new to Go, I'd recommend staying with monolith. If you want to make microservices and use monorepo, you would have multiple binaries located in cmd (e.g. users, payment) and the  same directory in pkg (e.g. /pkg/users/, /pkg/payment).

Thank you for your recommendation. If you don't mind, can you give us example on the read me how to make monorepo or microservice. The purpose is to enrich your stater kit.

https://en.wikipedia.org/wiki/Microservices
https://en.wikipedia.org/wiki/Monorepo

authors: naraycitra ribice

repo: ribice/gorsk

license: MIT


text:
Trouble with Arduino slave

I saw a similar resolved issue to this but I can't figure this one out. When uploading the docs to make my Arduino nano a slave I get this error. "Fatal error Arduino.h no such file or directory .... Include arduino.h"
I'm assuming that's a library I need to add to the ide but I can't find it. I found a file arduino.h in the Arduino download but it's where it was supposed to be in files and I can't upload it as a library on ide so I'm stumped.
Any help appreciated.

arduino.h is related to the arduino board. Before upload the code you need to select the correct board. Select arduno nano and then select the based microcontroller ATMega328 for the original or ATMega328 (old bootloader) for the copy.

arduino.h is related to the arduino board. Before upload the code you need to select the correct board. Select arduno nano and then select the based microcontroller ATMega328 for the original or ATMega328 (old bootloader) for the copy.

Yeah I did select the nano and I started with the old bootloader per the instructions that came with my Arduino and when that didn't work I tried the new version just to be sure


text:
Canceling the confirm prompt produces error

Canceling the prompt causes the below error...
macOS:shell-ai user$ shai terraform check
? Select a command: terraform validate
? Confirm: terraform validate
Traceback (most recent call last):
  File "/opt/homebrew/bin/shai", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/shell_ai/main.py", line 171, in main
    ).execute()
      ^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/InquirerPy/base/simple.py", line 336, in execute
    raise KeyboardInterrupt
KeyboardInterrupt

Ah good catch, cleaned it up https://github.com/ricklamers/shell-ai/commit/26f8e19526b85b1108a765273555ca56b2ab7979

authors: peteallport ricklamers

repo: ricklamers/shell-ai


text:
Would you please update this pannel？

there is none server recoder after today‘s update，only left Server Info  Online Players and Offline Players these three parts

Can you provide a screenshot of what you mean? I'm not getting this issue on my server

it just like this.

Login

repo: ricky-davis/AstroLauncher


text:
[Android] FreshTabbedNavigationContainer "jump" home on tab click

On iOS, regardless of where i am in the navigationstack, when i click on the tab that "represents" the top level page for my current navigation stack i am popped to the top level page. This is not the case with android, instead i have to pop multiple times to get to the mainpage, is this intended or am i doing something wrong?

@ivarlovlie I don't know if you have already tried but you are more likely to get a response to this question on StackOverflow. If I knew the answer I'd offer to answer on SO but I've never used the FreshTabbedNavigationContainer.

Thanks @munkii, asked on SO here: https://stackoverflow.com/questions/63567176.

authors: ivarlovlie munkii


text:
Embed Tcl directly into app

[mostly notes for my future self]
From https://developer.apple.com/documentation/macos_release_notes/macos_catalina_10_15_release_notes?language=objc

Scripting language runtimes such as Python, Ruby, and Perl are included in macOS for compatibility with legacy software. Future versions of macOS won’t include scripting language runtimes by default, and might require you to install additional packages. If your software depends on scripting languages, it’s recommended that you bundle the runtime within the app.

While they don't mention Tcl, it might be removed in a future macOS version (ARM macOS?). If that's the case, we'll need to embed the Tcl framework directly into the app. This also benefits us by targeting a single version of the language/runtime, instead of whatever is built-in to the OS.
To compile:

Download source from https://www.tcl.tk/software/tcltk/download.html
Unarchive like so: tar -xvf tcl8.6.9-src.tar.gz
Move into the macosx directory: cd tcl8.6.9/macosx
Configure with ./configure
Build with make embedded

Alongside the archive will be build/tcl/Tcl.framework which we can embed into the app.

Will reopen if/once this becomes a necessity.

repo: ridiculousfish/HexFiend


text:
Integrate new code from Brightscout into Riff EDU

This task is for integrating new code from Brightscout to support surveys into Riff EDU. This is an ongoing task as Brightscout continues to release new code.

@mlippert - can I close this story or do you want to use it to track taking ownership of the Repo?

I'd say even though this was just for work in the previous iteration, we might as well hang on to this story until we've taken ownership of the repo. Silly to create another story

ok cool, just stick in play when you start the work.

The repo is transferred https://github.com/rifflearning/mattermost-plugin-survey
However, I'm having some issues adding it to zenhub that I have to figure out or contact support about.

OK, that worked itself out and the repository has been added to this workspace successfully.

Actually before this gets closed out perhaps @jaedoucette wants to review the documentation they wrote?

Overview
Survey Schema Design

Thanks Mike. I've submitted a PR to address the one ambiguity I saw.
Otherwise, looks good.
John Doucette, PhD
Principal Scientist, Riff Learning
john@rifflearning.com
760-642-9199
On Wed, Jun 12, 2019 at 7:27 PM Mike Lippert notifications@github.com
wrote:

Actually before this gets closed out perhaps @jaedoucette
https://github.com/jaedoucette wants to review the documentation they
wrote?

Overview
https://github.com/rifflearning/mattermost-plugin-survey/blob/master/docs/overview.md
Survey Schema Design
https://github.com/rifflearning/mattermost-plugin-survey/blob/master/docs/db_schema.md

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rifflearning/mattermost-webapp/issues/91?email_source=notifications&email_token=AL4D2X6B2EIO3K52RIGA6S3P2GA4ZA5CNFSM4HP7LCR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXSCHJQ#issuecomment-501490598,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL4D2X63DBIOF757MLUF6Q3P2GA4ZANCNFSM4HP7LCRQ
.

authors: adonahue ebporter jaedoucette mlippert

repo: rifflearning/mattermost-webapp


text:
🛑 Canada Server is down

In afdd124, Canada Server (1.1.1.1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Canada Server is back up in 6bf2377.

repo: rifzkiadiyaksa/upptime


text:
QASM benchmarks

Contains a new routine (and make target) for benchmarking quilc's performance against the standard QASM benchmarking suite. Partially? addresses #465.
Adds the QASM benchmarks from the IBM QX Mapper as a git submodule.
I'm a little uncomfortable adding yet another benchmarking code path to quilc. This could probably be implemented within the framework set up in rewiring-analysis.lisp, but I also don't feel that that's much of an improvement over this minimalist procedure. On the other hand, one way that this implementation falls short is that it prints its results straight to *standard-output*, without opportunity to manipulate the results as data. I'd support at least opening an issue to smooth out and standardize the benchmarking techniques within quilc.
Example output:
+------------------+----------+-------+----------+
|       NAME       | TIME (s) | SWAPS | 2Q DEPTH |
+------------------+----------+-------+----------+
| 0410184_169      |    4.292 |   115 |      185 |
| 3_17_13          |    0.778 |    11 |       35 |
| 4_49_16          |    3.578 |    84 |      205 |
| 4gt10-v1_81      |    2.775 |    53 |      128 |
...
| z4_268           | >>>>>>30 | ????? | ???????? |
+------------------+----------+-------+----------+

One other thing: It might be nice to have a function describe-quilc-configuration which describes the set of options, parameters, etc. that quilc is configured to use, and that can be printed out before the b-marks are run

One other thing: It might be nice to have a function describe-quilc-configuration which describes the set of options, parameters, etc. that quilc is configured to use, and that can be printed out before the b-marks are run

Is there a way to get a list of parameter definitions within a package, rather than maintaining a list of all possible config variables?

One other thing: It might be nice to have a function describe-quilc-configuration which describes the set of options, parameters, etc. that quilc is configured to use, and that can be printed out before the b-marks are run

Is there a way to get a list of parameter definitions within a package, rather than maintaining a list of all possible config variables?

here's a lil somethin somethin for u
(defun idiomatic-package-variables (package &key include-internal)
  (let ((symbols nil))
    (labels ((bookended-by (char symbol)
               (let* ((string (symbol-name symbol))
                      (len (length string)))
                 (and (<= 2 len)
                      (char= char (char string 0) (char string (1- len))))))
             (variable-looking-symbol-p (s)
               (and (boundp s)
                    (or (bookended-by #\+ s)
                        (bookended-by #\* s))))
             (ponder-symbol (s)
               (when (variable-looking-symbol-p s)
                 (push s symbols))))
      (do-symbols (s package (sort symbols #'string<))
        (multiple-value-bind (s status)
            (find-symbol (symbol-name s) package)
          (case status
            (:internal
             (when include-internal
               (ponder-symbol s)))
            (:external
             (ponder-symbol s))))))))

Examples:
CL-USER> (idiomatic-package-variables ':qvm)
(QVM:*COMPILE-BEFORE-RUNNING* QVM:*QUBITS-REQUIRED-FOR-PARALLELIZATION*
 QVM:*TRANSITION-VERBOSE*)

CL-USER> (idiomatic-package-variables ':qvm :include-internal t)
(QVM::**ALIGNMENT** QVM::**APPLY-MATRIX-OPERATOR-FUNCTIONS**
 QVM::**CLASSICAL-MEMORY-SIZE-LIMIT** QVM::**CLIFFORD-OPERATIONS**
 QVM::**CLIFFORDS** QVM::**DOTIMES-ITERATOR** QVM::**INTEGER-BITS**
 QVM::**REAL-BITS** QVM::**SHARED-MEMORIES** QVM::*AVAILABLE-KERNELS*
 QVM:*COMPILE-BEFORE-RUNNING* QVM::*COMPILE-MEASURE-CHAINS*
 QVM::*FUSE-GATES-DURING-COMPILATION*
 QVM::*INLINE-STATIC-GATES-DURING-COMPILATION* QVM::*OPTIMIZE-BRISKLY*
 QVM::*OPTIMIZE-DANGEROUSLY-FAST* QVM::*OPTIMIZE-SAFELY*
 QVM::*QUBIT-LIMIT-FOR-USING-SERIAL-KERNELS*
 QVM:*QUBITS-REQUIRED-FOR-PARALLELIZATION* QVM:*TRANSITION-VERBOSE*
 QVM::+DEFAULT-NOISE-PREDICATE-NAME+ QVM::+MAX-NAT-TUPLE-CARDINALITY+
 QVM::+MAXPRIORITY+ QVM::+MINPRIORITY+ QVM::+OCTETS-PER-CFLONUM+
 QVM::+OCTETS-PER-FLONUM+)

authors: ecpeterson stylewarning

repo: rigetti/quilc


text:
Provide variant of QVMConnection.expectation that accepts  PauliTerm's

PauliTerm's make for a better basis of observables than "observable programs"

cc #261

authors: mpharrigan ntezak

repo: rigetticomputing/pyquil


text:
Fix typo in README.md

Sorry, I made a typo. I fixed it.

Merged

authors: ngmy riktar

repo: riktar/jkanban


text:
fix: flip closed connection bool

Closes: #4999

Just did a quick review and the change looks good. @AdityaHegde : Can you also do a quick review and merge if it seems good to you ?

repo: rilldata/rill


text:
AoT compilation fails

When running AoT compilation ( node_modules/.bin/ngc -p tsconfig-aot.json ) it fails with the following error:
Unexpected value 'VirtualScrollModule in /node_modules/angular2-virtual-scroll/dist/virtual-scroll.d.ts' imported by the module 'AppModule in /app/app.module.ts'
tsconfig-aot.json
{
  "compilerOptions": {
    "module": "es2015",
    "moduleResolution": "node",
    "target": "es5",
    "noImplicitAny": false,
    "sourceMap": true,
    "mapRoot": "",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "lib": [
      "es2015",
      "dom"
    ],
    "outDir": "lib",
    "skipLibCheck": true,
    "rootDir": ".",
	"typeRoots": [
      "node_modules/@types"
    ]
  },
  "angularCompilerOptions": {
    "genDir": "./app/ngfactory",
    "entryModule": "./app/app.module#AppModule"
  }
}

app.module.ts (excerpt)
import { VirtualScrollModule } from 'angular2-virtual-scroll';

@NgModule({
  imports: [
	VirtualScrollModule,
  ],
  declarations: [

  ],
  bootstrap: [ AppComponent ]
})

Seems like AOT does not like the distributed .js file, but requires the original .ts file.
This seems to work...
// FIXME: quick-hack to fix AOT
import { VirtualScrollModule } from "../../resources/angular2-virtual-scroll/src/virtual-scroll"

Notice that if you provide the metadata.json files alongside the .js files (meaning you need to AOT compile this library in advance) the .ts files are no longer needed.
https://medium.com/@isaacplmann/getting-your-angular-2-library-ready-for-aot-90d1347bcad#.wxi37ud13

Fixed with v0.1.0. Now this library is AoT enabled.

authors: Rob-Post jsen- rintoj ronzeidman


text:
Cannot run in mobile browser

hi ，I cloned this project and ran it. There was no problem accessing the web page on the PC, but it was found that it could not be used normally when accessing the web page on the iphone. Do you know whether this asset supports mobile browsers, or do you know how to solve this problem?

Because I want to use it to build a multi-terminal metaverse, I have to confirm that it is fully work.

In order to run in the mobile browser, I made some changes to the code

very good! I hadn't tested it on the iphone yet, so glad you found a way to
make it work!
Em sex., 3 de fev. de 2023 às 08:00, wei-kris @.***>
escreveu:

Closed #2 https://github.com/rio3dstudios/metaverse/issues/2 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/rio3dstudios/metaverse/issues/2#event-8430955427, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCREJQCMTF7NAM7OPFSEETWVTQMFANCNFSM6AAAAAAUQCMMAU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Sebastião Lúcio
Lead Developer | Rio 3D Studios
Website:  www.rio3dstudios.com
Email:  @.***


text:
Improve incremental source generator support

With the current approach, almost everything is regenerated all the time. This should be better abstracted to make better use of the incremental source generator. Approaches to be discussed.

You'll want to use ForAttributeWithMetadataName (requires Roslyn 4.3.1 or greater) to target uses of [Mapper] uses. Then you'll need to build a fully comparable model to pass down in the pipeline. Watch out if you have collection types in your model, as eg. ImmutableArray<T> is not equatable. If you need references, an example you can look at is our generators in the MVVM Toolkit, you can find them here: https://aka.ms/mvvmtoolkit/dotnet 🙂

@Sergio0694
Thank you for the inputs / help! I started to work on this here. However, it needs some more effort to complete.
In the meantime, since Mapperly only generates the implementation of user defined partial methods, couldn't RegisterImplementationSourceOutput be used instead of RegisterSourceOutput (which should lead to similar performance improvements)?

I'm implementing a couple of source generators myself and came across the same problem, that I was re-generating the source on any change within the same project. One way I found to drastically improve this is to use a comparer that will only evaluate Equals to false if the syntax nodes aren't equal (and ignore the compilation). Sample code is below...
    public void Initialize(IncrementalGeneratorInitializationContext context)
    {
        var provider = GetIncrementalValuesProvider(context);

        // Combine the CompilationProvider with the transformed data, but add a custom comparer so that
        // differences can be intelligently determined in order to avoid unnecessarily invalidation of
        // the cached generated code. This is vital to avoid lots of unneeded generation in the IDE, as
        // without the comparer any change to any file in the same project will cause a re-generation.
        // See performance tips at https://www.thinktecture.com/en/net/roslyn-source-generators-performance/
        var combined = provider
            .Combine(context.CompilationProvider)
            .WithComparer(GetComparer())
            .Collect()
            .SelectMany((array, _) => array.Distinct());

        context.RegisterSourceOutput(combined, (ctx, data) => Execute(ctx, data.Item2, data.Item1));
    }

/// <summary>
/// Compares a syntax node and compilation combination by ignoring the compilation and only comparing the
/// syntax nodes for equality. This is used for comparing a syntax node returned from a syntax provider
/// with a combined compilation.
/// The reason we ignore the compilation in the comparision is because the syntax node is immutable, but the
/// compilation is not, and therefore if there are no changes to the syntax, <c>Equals</c> will evaluate to true.
/// If we included the compilation in the equality check also then we would always return false, which would
/// result in the generated source cache frequently being invalidated when there are no changes to the syntax.
/// </summary>
public class SyntaxNodeComparer<TSyntax> : IEqualityComparer<(TSyntax Node, Compilation Compilation)>
    where TSyntax : SyntaxNode
{
    public virtual bool Equals((TSyntax Node, Compilation Compilation) x, (TSyntax Node, Compilation Compilation) y)
    {
        return x.Node.Equals(y.Node);
    }

    public virtual int GetHashCode((TSyntax Node, Compilation Compilation) obj)
    {
        return obj.Node.GetHashCode();
    }
}

Note that I go this idea from https://www.thinktecture.com/en/net/roslyn-source-generators-performance/, which was a really great source of info for me.
The issue with the solution above though is that the re-generation will only happen if the mapper file changes in some way, but not when one of the models change. In order to include the models in the comparison, you'd need to retrieve their syntax nodes and compare them too (which is more expensive... but probably a fair bit cheaper still than re-generating every time)
I have found an issue in my own generation logic that I can't (easily) check the syntax of models that live in a different assembly. I know this is possible, but I haven't gotten around to working on it yet. I found that mapperly has the same limitation (i.e. if I update a model from a different project the mapperly generated file doesn't automatically re-generate). I think this is potentially a pretty common use-case for mappings (i.e. it's common for entities to live in a different project to DTO models), so would be great if this could be improved.

Mapperly now uses RegisterImplementationSourceOutput which doesn't live update the generated build output but only runs the generator when building the solution. This is okay, as long as Mapperly only provides implementations for user defined methods.
When I tried to implementa fully incremental source generator approach for Mapperly, the main problem I encountered was the ClassifyConversion on the compilation used by Mapperly to determine whether one type can be converted to another.
Have you @nieldy encountered any performance issues while using Mapperly?

"use a comparer that will only evaluate Equals to false if the syntax nodes aren't equal (and ignore the compilation)"

@nieldy absolutely do NOT do this. It will root Compilation objects and cause significant greater memory use.

Ok, thanks for the tip @Sergio0694. I read about the WithComparer usage in the article I linked in my original post, and it seemed to give me the desired effect, but I had no idea it would cause memory issues. I'm very new to source generation (starting looking at it only a few weeks ago), so still trying to get my head around the best practices. What would you recommend instead of using a comparer if I want to avoid re-generating for every file touched in the project, but still have access to the Compilation?
I'm still quite confused about the effect of RegisterImplementationSourceOutput @latonz, because I tried using it, but it is still re-generating my source live (as opposed to just on the build). When I test with the latest stable version of Mappery (2.8.0), I find the mapper source live re-generates when the mapper file is touched, or one of the model files (providing it's in the same proeject) without requiring a rebuild. If a model is touched in a different project then I'm finding the mapping source isn't re-generating at all, even on a re-build... having said that, I am using Rider, so it's possibly an issue with the way it's handled in Rider. I'll test in VS and see what happens.

"What would you recommend instead of using a comparer if I want to avoid re-generating for every file touched in the project, but still have access to the Compilation?"

You just cannot do that. Gather all the info you need from the first transform step and make it equatable. No symbols or compilation objects or anything that's not equatable should get past the first transform step, generally speaking.

"I tried using it, but it is still re-generating my source live"

RegisterImplementationSourceOutput is not wired up yet in the compiler. Right now it's identical to RegisterSourceOutput.

Ah right, that makes sense @Sergio0694, thanks. I think RegisterImplementationSourceOutput will be what I want to use (when it's implemented). Do you have any idea of when that might be?
Regarding Rider behaviour vs VS. I just tried in VS and can see it is re-generating the source code when a model from a different assembly/project is modified, so it's clearly a quirk/bug in Rider. The generation also seems to be smoother and faster in VS (in Rider the code jumps about a little before it updates). Seems like JetBrains have a bit of work to do to get their implementation up to scratch.

"I think RegisterImplementationSourceOutput will be what I want to use (when it's implemented). Do you have any idea of when that might be?"

You don't have to wait, if that makes sense for you just use that. It'll just "get faster" when Roslyn starts supporting it 🙂

Looking at MVVM Toolkit it looks like it would be possible to convert MapperGenerator.Execute up to NormalizeWhitespace in SourceEmitter.Build into a pipeline step. This step would output (CompilationUnitSyntax,ImmutableArray<DiagnosticInfo>). The errors can be reported before each CompilationUnit is formatted with NormalizeWhitespace and then added.

Closing this for now, as @TimothyMakkison implemented great improvements by caching before normalizing whitespace which seems to work great. We'll probably revisit this later.

I can think of two possible improvements:

Stop using roslyn and instead emit strings, NormalizeWhitespace is the bulk of time and memory usage.
Make *Mapping cacheable, this is a little tricky due to SourceType and TargetType and the existence of self referential method loops. This could make possible nested configurations easier as it would be easy to cull identical mappings and methods.

Remove the source/target types and make *Mapping a pure value, MethodMapping might become a separate object.
Not ideal but each *Mapping could implement its own custom comparer

authors: Sergio0694 TimothyMakkison latonz nieldy

repo: riok/mapperly


text:
Proper handling of array hashs

Current we have an array of hashs in our yaml file
people:
- name: "Trevor"
  id: "101"
- name: "Tim"
  id: "202"

Using thearray option with hiera it tries to flatten, uniq, compact and sort the data. I believe it goes fine until the sort.

./lib/hiera/backend.rb:132:in sort': undefined method <=>' for {"name"=>"Trevor", "id"=>"101"}:Hash (NoMethodError)
from ./lib/hiera/backend.rb:132:in resolve_answer'         from ./lib/hiera/backend.rb:166:in lookup'
from ./lib/hiera.rb:59:in `lookup'
from bin/hiera:208

I am not sure of the best approach for a fix.  I am using ruby 1.8.5.  I heard ruby < 1.9 don't support sorting hashs.  Using ruby 1.9.2 with hiera seems to have it's own issues though (https://github.com/ripienaar/hiera/issues/19)  .
Thoughts?

Do you need to use this array with the dynamic array building feature of hiera_array? To get just your array simple hiera("foo") will suffice.
--
R.I.Pienaar
On 18 Oct 2011, at 01:33, CatDaaaady reply@reply.github.com wrote:

Current we have an array of hashs in our yaml file
people:
- name: "Trevor"
 id: "101"
- name: "Tim"
 id: "202"

Using thearray option with hiera it tries to flatten, uniq, compact and sort the data. I believe it goes fine until the sort.

./lib/hiera/backend.rb:132:in sort': undefined method <=>' for {"name"=>"Trevor", "id"=>"101"}:Hash (NoMethodError)
from ./lib/hiera/backend.rb:132:in resolve_answer'        from ./lib/hiera/backend.rb:166:in lookup'
from ./lib/hiera.rb:59:in `lookup'
from bin/hiera:208

I am not sure of the best approach for a fix.  I am using ruby 1.8.5.  I heard ruby < 1.9 don't support sorting hashs.  Using ruby 1.9.2 with hiera seems to have it's own issues though (https://github.com/ripienaar/hiera/issues/19)  .
Thoughts?
--
Reply to this email directly or view it on GitHub:
https://github.com/ripienaar/hiera/issues/22

That is smooth. I see what you mean. But we have different info stored in different yaml files that we would like to "glue" together.  So in one file we have our common users and in another we could grab out users by environment.  I think hiera_array would get us that? Using hiera simple it stops the search after it finds info inside our ${env} file.
We have something like this

%{env}
common

OK kewl, so in that case yes you want hiera_array and as you say it barfs when it encounters complex data.
I can't see why sorting the answer is strictly needed, its probably just some OCD neatness thing I had while writing it, if you edit lib/hiera/backend.rb line 133 and remove the .sort its all good right?

Yup. It's all happy then.  Nothing is wrong with neatness every now and then :-)

OK I pushed a fix for that

Smooth!

authors: CatDaaaady ripienaar

repo: ripienaar/hiera


text:
Emulator: Improve usability by restricting memory region access for guest programs

Memory regions outside of the address range TEXT_START to SYSTEM.start() - 1 is
used to store information related to the zkVM. Tampering with this region will
cause the prover to crash. Restrict the guest from writing to this region.
Fixes: #672

I'm not sure if we want this commit https://github.com/risc0/risc0/pull/727/commits/bd843b837ff35e96f693f0ab8ecc502e82ec6bcf
I realized while adding all of those changes that returning Result on everything is a related but separate task


text:
How to run an OS on e203_hbird?

Would you please publish a tutorial about how to run an OS on e203_hbird? Or please let me know where there is a such tutorial?

See https://doc.nucleisys.com/hbird_sdk/ and https://github.com/riscv-mcu/hbird-sdk

authors: fanghuaqi fantasysee


text:
Add a section with examples

The examples are non-normative. I've taken a subset of the examples in the examples directory of the repository.
This fixes #319

@nick-knight can you take a closer look at the matrix multiplication? I took the existing example but it presumes a transposed matrix in B (I think) so I reimplemented it for a more naive approach so we can show a strided access. Also I understand partial accumulations should use tail undisturbed (regardless of the fact that this is a vfmacc with already 3 input operands).

@rofirrim I've always strongly disliked the matmul example in this repo: it's not how a sane person would implement it.

I agree with @nick-knight, I think we should stick with code we can confidently say will portable perform at close to the peak performance.

Hi @kito-cheng thanks a lot for merging this for me.
I will update v1.0.x to the current main so not to stall further work (such as bf16).

authors: camel-cdr nick-knight rofirrim

repo: riscv-non-isa/rvv-intrinsic-doc


text:
Non-zero D/A/U bits in a non-leaf PTE

Section 4.3.1, has the following:

For non-leaf PTEs, the D, A, and U bits are reserved for future standard use. Until their use is defined by a standard extension, they must be cleared by software for forward compatibility.

What should an implementation do if the D, A, and U bits are found to be non-zero in a non-leaf PTE? Raise an exception? If so, what kind of exception?

"If pte.v = 0, or if pte.r = 0 and pte.w = 1, or if any bits or encodings that are reserved for
future standard use are set within pte, stop and raise a page-fault exception corresponding
to the original access type."

Thanks

authors: jrahmeh pdonahue-ventana

repo: riscv/riscv-isa-manual


text:
Clarify that compares only AND in the mask if vd == v0

This PR clarifies that masked compares AND in the mask only if the destination register is v0, as discussed in #900

While we're here, I think "when mask-undisturbed" warrants a further clarification. Mask-producing instructions like these ignore vtype.vma and always have mask agnostic behavior. I think we should say something more explicit, like "on implementations that implement mask-agnostic as mask-undisturbed". Otherwise the following example is wrong (consider a == b and VL = 1).

Is that really true? I know it’s the case for mask-logical instructions (which correspondingly aren’t allowed to be masked) but I don’t recall this being the case for comparisons.

My reading of
https://github.com/riscv/riscv-v-spec/blob/master/v-spec.adoc#343-vector-tail-agnostic-and-vector-mask-agnostic-vta-and-vma
is that inactive destination (mask) bits can be asserted.

What clause?

Could you use some quotation marks? All I see that’s potentially relevant (but isn’t actually) is “Mask destination tail elements are always treated as tail-agnostic, regardless of the setting of vta.”

A little further down:

When a set is marked agnostic, the corresponding set of destination elements in any vector destination operand can either retain the value they previously held, or are overwritten with 1s. Within a single vector instruction, each destination element can be either left undisturbed or overwritten with 1s, in any combination, and the pattern of undisturbed or overwritten with 1s is not required to be deterministic when the instruction is executed with the same inputs.

Maybe I'm making a logical leap that I shouldn't?

What I am missing is what marks this particular set agnostic. I agree that writing 1 is legal for an agnostic mask bit, but what mandates the result of comparisons be agnostic?

Mask-producing instructions like these ignore vtype.vma and always have mask agnostic behavior.

No. Mask-producing instructions are always vta but not vma.

Sorry. Please disregard my comments.

authors: aswaterman michael-platzer nick-knight

repo: riscv/riscv-v-spec


text:
Added Challenge for CodeSpace Community Website

I have worked on the #72 issue.
I have Designed a Quiz Challenge for the Codespace Community Website.
Figma Design : Design Link
#72

@rishipurwar1 Please check my PR for this issue

As I already told you, you need to improve the UI of this challenge so that I can merge this PR.

As I already told you, you need to improve the UI of this challenge so that I can merge this PR.

I have made those changes....please check it once

It looks OK, you need to make some more changes so that It looks professional. Please see the design that I shared with you.


text:
Draft of section 2

WIP currently

The three sections seem to actually be part of an entire offloading scenario. Can we describe an overall explanation it in the readme.md and draw some architectural diagrams? In addition, we can explain to users how it was used without RisingWave. For example, compare:

Executing aggregate queries directly within the user's TP database. In this case, valuable TP resources are consumed. You can refer to the section 'The Problem: Running Complex Queries on OLTP' in this article https://materialize.com/blog/oltp-workloads/.
Offloading to an OLAP system, such as ClickHouse. We might temporarily not complete this part. Because explaining the advantages in this situation is quite complex. Simply put, because RisingWave utilizes the pre-defined computations provided by the user, completing the computation at the time of data writing rather than at the time of querying, this allows for the reuse of repeated computations between multiple queries.

Furthermore, I think we can add a section on sinking the materialized view results of RisingWave back into PG. Although we do not recommend this approach because it incurs additional overhead compared to querying RisingWave directly. I believe there are two main advantages to this.

Compared to FDW, storing the materialized view results as a table in PG makes the performance of querying this result table more predictable.
Sometimes, with the help of PG's ecosystem, it is possible to achieve some functionalities that RisingWave does not support. For example, PG is compatible with more upper layers (such as Hasura), or special indexes can be supported at the storage layer through PG's plugins."

I've added an extended overview to this tutorial section. I don't know if an architectural diagram would be that helpful in this case since we're only using RisingWave and PostgreSQL.
I have a section on sinking data to PostgreSQL in part 3 and expanded on the benefits.
Thanks for the quick feedback!


text:
feat: Promote the metaStore and stateStore

What's changed and what's your intention?

As title

Checklist

[x] I have written the necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Update the branch to trigger the new CI.

Besides, I think we need test coverage on those reference fields.

repo: risingwavelabs/risingwave-operator


text:
perf(streaming): compact useless intermediate result in stream chunk

Background
Some streaming operators can produce useless intermediate results in a stream chunk. And because under out process model, we can compact ops with the same stream key in the same epoch. We can reduce them. Even for those slightly reducing could matters before some other operators with amplification such as join and HopWindow( 2x amplification does not matter but 100x  amplification is far better than 200x). here are some examples.

Project update some field

create table t(a int, b int, k int primiary key);
create materialize view mv as select a, k from t;
insert into t values (1, 1, 1), (2, 2, 2);
UPDATE SET b = 10 FROM T;

even the update will not change the value of the mv, the changes will still be sent to the down stream (IIRC it is found by @TennyZhuang )

In discussion with some users, the first case can appear in some source or cdc connector(update a extra filed and give update with the same old value and new value)

outer join
if outer join does not do any optimization. It will amplify anyway. the outer's two op (U-, old),(U+,new) change to (U-, old), (U+,NULL), (U-, NULL), (U+,new)

feature
we should have common utils to compact ops with the same stream key in the same epoch, even only on the chunk based.

#10853 can also be resolved by this. cc @fuyufjh

repo: risingwavelabs/risingwave


text:
bug: temporal join on CTE is lost silently

We need return an error to tell user it is not supported
dev=> create table stream(id1 int, a1 int, b1 int) APPEND ONLY;
create table version(id2 int, a2 int, b2 int, primary key (id2));

NOTICE:  APPEND ONLY TABLE is currently an experimental feature.
CREATE_TABLE
CREATE_TABLE
dev=> explain
create materialized view mv as
select id1, a1, id2, a2 from stream left join version FOR SYSTEM_TIME AS OF PROCTIME() on id1= id2
dev-> ;
                                                                              QUERY PLAN

------------------------------------------------------------------------------------------------------------------------
-----------------------------------------------
 StreamMaterialize { columns: [id1, a1, id2, a2, stream._row_id(hidden)], stream_key: [stream._row_id, id1], pk_columns:
 [stream._row_id, id1], pk_conflict: NoCheck }
 └─StreamExchange { dist: HashShard(stream.id1, stream._row_id) }
   └─StreamTemporalJoin { type: LeftOuter, append_only: true, predicate: stream.id1 = version.id2 }
     ├─StreamExchange { dist: HashShard(stream.id1) }
     │ └─StreamTableScan { table: stream, columns: [id1, a1, _row_id] }
     └─StreamExchange [no_shuffle] { dist: UpstreamHashShard(version.id2) }
       └─StreamTableScan { table: version, columns: [id2, a2] }
(7 rows)

dev=> explain
create materialized view mv as
with version as (select * from version)
select id1, a1, id2, a2 from stream left join version FOR SYSTEM_TIME AS OF PROCTIME() on id1= id2;
                                                                              QUERY PLAN

------------------------------------------------------------------------------------------------------------------------
-----------------------------------------------
 StreamMaterialize { columns: [id1, a1, id2, a2, stream._row_id(hidden)], stream_key: [stream._row_id, id1], pk_columns:
 [stream._row_id, id1], pk_conflict: NoCheck }
 └─StreamExchange { dist: HashShard(stream.id1, stream._row_id) }
   └─StreamHashJoin { type: LeftOuter, predicate: stream.id1 = version.id2 }
     ├─StreamExchange { dist: HashShard(stream.id1) }
     │ └─StreamTableScan { table: stream, columns: [id1, a1, _row_id] }
     └─StreamExchange { dist: HashShard(version.id2) }
       └─StreamTableScan { table: version, columns: [id2, a2] }
(7 rows)

Actually, you should try because FOR SYSTEM_TIME AS OF PROCTIME() is binded to scan. 🥵
explain
create materialized view mv as
with version as (select * from version  FOR SYSTEM_TIME AS OF PROCTIME())
select id1, a1, id2, a2 from stream left join version on id1= id2;

So we should report an error when trying to use any relation we can't scan as the reference table in temporal join?


text:
Decouple show jobs from CreateMviewProgressTracker

We can decouple rw_ddl_progress from meta’s materialized view progress tracker, and maintain it adhoc:

Make internal backfill state tables visible. They contain the backfilled row_count, and whether the backfill is finished or not.
a. Query all the internal backfill state tables for an MV to fetch row_count and finished status.
b. Query internal tables of the MV.
c. Regex the name for backfill tables.
Query the hummock version stats, so we can get the upstream row count.
Calculate the estimated progress.

This will:
Simplify logic of materialized view progress tracker. We no longer need to maintain state for counts.
Allow us to track the backfill progress of created sink jobs.
In general, I think the same pattern can be applied to other forms of backfilling, snapshot backfill and shared source backfill.

Implementation-wise, we need to consider querying permissions for the internal table.

This also has the added benefit where we can display finished jobs.


text:
bug: "Timestamp cmp Varchar" error when comparing timestamp field with "to_timstamp"

Describe the bug
When testing risingwave (commit: b5c154d8a62ecf646207f09f2d2792cb5c678160) integration with superset (commit 79114bcd29b0ce1685828ff9a60791d13a7459c9), it failed to create time series chart and threw " QueryError: internal error: Failed to build executor: Expr error: Unsupported function: Timestamp cmp Varchar" error for the following query:
SELECT v1 AS __timestamp,
v1 AS v1,
COUNT(*) AS count
FROM public.mv1
WHERE v1 >= TO_TIMESTAMP('2023-02-06 00:00:00.000000', 'YYYY-MM-DD HH24:MI:SS.US')
AND v1 < TO_TIMESTAMP('2023-02-07 00:00:00.000000', 'YYYY-MM-DD HH24:MI:SS.US')
GROUP BY v1
ORDER BY count DESC
LIMIT 10000

To Reproduce
dev=> create table test (v1 timestamp);

dev=> select * from test where v1 > TO_TIMESTAMP('2023-02-06 00:00:00', 'YYYY-MM-DD HH24:MI:SS');
ERROR:  QueryError: internal error: Failed to build executor: Expr error: Unsupported function: Timestamp cmp Varchar

Expected behavior
No response
Additional context
No response

We hard-coded to_timestamp to return varchar by mistake.
https://github.com/risingwavelabs/risingwave/blob/d8198fa138003e1f1431053f4f5f09e4a5fa8fd8/src/expr/src/expr/expr_to_timestamp_const_tmpl.rs#L37-L40
Should be timestamp to match current implementation.
Should be timestamptz and update implementation to align with PostgreSQL #7780.

It is being fixed to Timestamp first: #9924


text:
fix(scale): move reschedule_lock to ScaleController

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Closes #15018 .
We need to implement mutual exclusion between the recovery and the auto scaling loop. Previously, the lock was inside the stream manager, which made it unattainable for recovery. By moving it to the scale controller, we can now acquire the lock.
Additionally, we mistakenly used separate scale controllers in both the stream manager and recovery, which resulted in the lock becoming ineffective. This pull request has also addressed and fixed that issue.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Just asking some questions to better understand the problem and solution.

We need to implement mutual exclusion between the recovery and the auto scaling loop. Previously, the lock was inside the stream manager, which made it unattainable for recovery. By moving it to the scale controller, we can now acquire the lock.

During recovery, the stream manager is not online, is that why the lock is not acquirable on recovery?
Why scale controller is available during recovery, but not stream manager?
During recovery why do we need to acquire the reschedule lock? Is it to prevent any scaling from happening?
I suppose the scaling controller controls the scaling process, and so conceptually it also makes sense that it should hold the rw lock.

Additionally, we mistakenly used separate scale controllers in both the stream manager and recovery, which resulted in the lock becoming ineffective. This pull request has also addressed and fixed that issue.

Could you elaborate on how it led to the panic in #15018?

authors: kwannoel shanicky


text:
Clarify the SSH usage

At some place in the tutorial it could be great to add more informations on which ssh keys do we have to use. (Dokku / Git / Connexion...)

Thanks @julienstroheker! Closed via 7226a0b

authors: julienstroheker ritazh

repo: ritazh/devopsfun


text:
Archivos dotenv

Crear una un archivo llamado ./src/config/constants.js dentro de dicho archivo exportar un objeto que contendra las llaves de configuración tal como lo hace el curso building REST API with Nodejs / MongoDB /Passport /JWT en el tema Adding Mongoose.
El defaultConfig debera tener las siguientes llaves:

PORT: process.env.PORT || 3000
ENV: process.env.NODE_ENV

El devConfig debera tener las siguientes llaves:

MONGO_URL: process.env.MONGO_URL_DEV,
JWT_SECRET: process.env.JWT_SECRET,

El testConfig deberá tener las siguientes llaves:

MONGO_URL: process.env.MONGO_URL_TEST,

El prodConfig deberá tener las siguientes llaves:

MONGO_URL: process.env.MONGO_URL_PROD,

@Fabiolaolmos se ha creado la rama 19-archivos-dotenv para que trabajes el issue sobre esa rama

repo: rivalcoba/restApi-ci-itgam


text:
[Feature Request] Retail Only Option

Es wäre schön, eine Option zu haben, um ausschließlich Retails bei der automatischen Filmsuche zu berücksichtigen. Damit könnten Vorabversionen ignoriert und der gleiche Film würde in Kombination mit der vorhandenen Retail Option bestenfalls nur 1x heruntergeladen werden.

Behoben in Version 6.6.0


text:
D4 - Adopted Technologies

List the adopted technologies with a brief description and justification for choosing them.
Grading criteria (1 point): This section will be evaluated in terms of correctness, completeness, thoroughness, consistency, coherence, and adequate use of language.

assignment was turned in so I am closing the issue


text:
Add "Open with..." option

It would be good to have a feature to open a distro with a specific terminal (and set a default terminal) as opposed to only allwoing the default WSL terminal. For instance, I use Hyper with my WSL install and it would be great to be able to open a distro using WSL Distro Manager in Hyper.

Pull Request #12 adds an option to open a distro in Hyper.
Due to the nuanced configurations for other terminals, it may be best to wait to add the option to open in those other terminals until it is requested.
Also, it doesn't appear possible to open multiple distros at once in Hyper by shift-selecting. If it is figured out, this could be added later as well.

I added code to create a shim to make it easy to use the hyper.js terminal, whether it was installed via the Microsoft Store or not. This code can be used in all cases by simply changing the shell property in the hyper.js configuration file. I will close this issue once the feature development is complete.
https://github.com/rkttu/WSL-DistroManager/commit/89944dee582ceb2733980f459f91482816054ddf

authors: erhassettbg rkttu

repo: rkttu/WSL-DistroManager


text:
Create 0000-01-02-rkvila.md

Add rkvila's file

great

repo: rkvila/github-slideshow


text:
Strings of dashes in code blocks are removed

Related to #4 long strings of dashes in code blocks seem to become a single dash
→ node -e 'console.log(require("render-readme")("\`\`\`\n------------------------------\n\`\`\`"))'

-

they come through fine from markdown-it
→ node -e 'console.log(require("markdown-it")().render("\`\`\`\n------------------------------\n\`\`\`"))'

------------------------------

Fixed in https://github.com/rlidwka/render-readme/commit/8d40f8935ac77d3ea8b6da80c53e260663cf7189, thanks.

repo: rlidwka/render-readme


text:
🛑 Student Portal is down

In 0fb8955, Student Portal (https://myportal.cvsu.edu.ph) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Student Portal is back up in 9602ea1 after 5 minutes.

repo: rlvillacarlos/cvsu-site-monitor


text:
🛑 Library is down

In 2922d84, Library (http://library.cvsu.edu.ph) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Library is back up in f2aa502 after 5 minutes.


text:
Python 3.11

Any goals to make the bot work on python 3.11?

repo: rmmh/skybot

license: unlicense


text:
salah perintah/config

salah config

solved.


text:
Add dark mode

Bulma should make it relatively straightforward to implement a toggle for switching between light and dark mode.

With the release of Bulma v1.0, this is built-in. So, the easiest way to achieve that is probably to upgrade Bulma to v1.0.

repo: rmnldwg/lyprox


text:
Loading players

Implement getLogs method on StakeRegistry to load all players. Replay:

StakeUpdated for adding amounts to stake.
StakeSlashed for remove amounts from stake.

Additionally, we should include a stakeAmount for the Player to track this value and any subsequent slashes / updates.

I just implemented a Player.stake element and have it displaying in the All Players and Round Players boxes.   Unfortunately, the StakeSlashed events need to be processed in sequence with the StakeUpdated.   The StakeUpdated event SETS the state to an amount while the StakeSlashed event SUBTRACTS the amount from the current stake not allowing it to go negative.
I don't know how to get the pair logs from the StakeRegistry interleaved in time rather than individually monitored.   And the monitor will need to look at the state RUNNING before processing them, I think, maybe, see the next comment.
The initial sync also needs to filter both events and process them in time sequence, stopping at the starting block.   But if we stop at the starting block and the monitor waits for RUNNING, then we'll potentially have a gap of events that we miss while the blocks are loading.   Seems maybe the stakes would need to have a separate state for starting to process the events, but that might get messed up if a slash log comes in from a winner event.   I think the stakes will need to only look at the stake contract, and ignore the slashes from the redistributor.  (Yes, I probably have those contract names not quite right).

@mfw78  If you can work out the proper filter for both the initial load and a single, time-sequenced subscribe, it would be appreciated.  I'm out of my depth when it comes to more than a single filter term.
Possibly related to #10

repo: rndlabs/monSI


text:
🛑 Rova Dashboard is down

In 6aff816, Rova Dashboard (https://dashboard.dev.getrova.io) was down:

HTTP code: 503
Response time: 309 ms

Resolved: Rova Dashboard is back up in 9422df7 after 10 hours, 42 minutes.

repo: roava/rova-uptime-checks


text:
Floating point errors in .align_to()

Describe the bug
The below code:
import sectionproperties.pre.library.bridge_sections as bridge_sections
import sectionproperties.pre.library.primitive_sections as standard_sections
from sectionproperties.pre.pre import Material
from sectionproperties.analysis.section import Section
from sectionproperties.pre.geometry import CompoundGeometry

Dslab, w, t_f = 180, 2100, 75

precast = Material(
    name="65 MPa",
    elastic_modulus=37.4e3,
    poissons_ratio=0.2,
    yield_strength=65,
    density=2.4e-6,
    color="grey",
)
insitu = Material(
    name="40 MPa",
    elastic_modulus=32.8e3,
    poissons_ratio=0.2,
    yield_strength=40,
    density=2.4e-6,
    color="lightgrey",
)

super_t = bridge_sections.super_t_girder_section(girder_type=5, w=w, material=precast)
slab = standard_sections.rectangular_section(d=Dslab, b=w, material=insitu).align_center(super_t).align_to(super_t, "top")

geom = CompoundGeometry([super_t, slab])
geom.plot_geometry()
geom

Produces this...

But it SHOULD produce this...(Note the red svg denoting "invalid" geometry and the hole indicated on the plot)

Additional context
This seems to be a problem with shapely. I opened an issue with shapely after realizing that one of their examples in their docs does not reproduce as given (another example of the same problem): https://shapely.readthedocs.io/en/stable/manual.html#polygons
exter = [(0, 0), (0, 2), (2, 2), (2, 0), (0, 0)]
inter = [(1, 0), (0, 1), (0.5, 1.5), (1.5, 0.5), (1, 0)]
(Polygon(exter) - Polygon(inter)) # Creates valid MultiPolygon even though two points in common

Produces this...

Something else that's funny (odd, not 'ha ha')
If I take the geom object that was created in the bridge section code (at top) and break it out with .points as such:
geom = CompoundGeometry(MultiPolygon([Polygon(slab.points), Polygon(super_t.points)]))

# Instead of...

geom = CompoundGeometry([slab, super_t])

Then I also get this...

I don't fully understand but I think it is shapely related... Will post more on this as I learn more.

A more robust algorithm for Geometry.align_to() might be to instead replace source ordinates with the target ordinates instead of calculating offsets with them.
e.g. If aligning Geometry A to the top of Geometry B, the y-ordinates matching the minimum y-extent of Geometry A would be changed so that they are the same as the maximum y-extent of Geometry B. This way, they remain exact and no rounding error occurs.

There is also this:
https://gis.stackexchange.com/questions/188622/rounding-all-coordinates-in-shapely/188739#188739

Hi @connorferster, I agree that the precision method seems to be a good approach, i.e. offset = round(align_to_coord - self_align_coord, precision).
How do you see the stackexhange _set_precision() method working? Every time the coordinates are modified this method is called? Perhaps that's a good idea? Maybe a note to the user in the docs as well that there is a defined minimum precision e.g. 8 and that units should be chosen to avoid precision rounding issues, i.e. don't model your small section in kilometers.
Reminds of the issues early kerbal space program had with precision!

@connorferster I'm not sure if I'm 100% following the issue, but the hole is not being detected, seems like here is where it should happen. I can see that there are two sets of logic for MultiPolygon and Polygons. I can see why this is done: to catch holes interior to each Polygon. It seems to miss holes that are formed by the two polygons' exteriors though.
I would like to propose a new logic that I think will work for both MultiPolygon and Polygon. I will use your example to demonstrate the logic:
from shapely import geometry
from shapely.geometry.point import Point
import sectionproperties.pre.library.bridge_sections as bridge_sections
import sectionproperties.pre.library.primitive_sections as standard_sections
from sectionproperties.pre.pre import Material
from sectionproperties.analysis.section import Section
from sectionproperties.pre.geometry import CompoundGeometry, Geometry
from shapely.geometry import box, Point

Dslab, w, t_f = 180, 2100, 75

precast = Material(
    name="65 MPa",
    elastic_modulus=37.4e3,
    poissons_ratio=0.2,
    yield_strength=65,
    density=2.4e-6,
    color="grey",
)
insitu = Material(
    name="40 MPa",
    elastic_modulus=32.8e3,
    poissons_ratio=0.2,
    yield_strength=40,
    density=2.4e-6,
    color="lightgrey",
)

super_t = bridge_sections.super_t_girder_section(girder_type=5, w=w, material=precast)
slab = standard_sections.rectangular_section(d=Dslab, b=w, material=insitu).align_center(super_t).align_to(super_t, "top")

geom = CompoundGeometry([super_t, slab])

# a proposed fix to hole detection in CompoundGeometry
b_box = box(*geom.geom.bounds)
sub = b_box - geom.geom
hole_polys = [ii for ii in sub if not b_box.exterior.intersects(ii)]
holes = [hole.representative_point() for hole in hole_polys]
[print(hole) for hole in holes]

Essentially, it takes a bounding box and cuts out the geometry, then there is some filtering to find out which cutouts are interior to the section. Let me know your thoughts, there may be some issues I'm not seeing.

"It seems to miss holes that are formed by the two polygons' exteriors though"
Correction: it seems unary_union is not working as expected in this case. For this case we should expect a merged Polygon... and it is producing a Multipolygon, and that is why the hole formed by the two exteriors is not being picked up.

Unfortunately, I'm now beginning to see the same issues with this method:
geom = CompoundGeometry([super_t, slab])
geom = geom.rotate_section(30)

# try proposed fix to hole detection in CompoundGeometry
b_box = box(*geom.geom.bounds)
sub = b_box - geom.geom
hole_polys = [ii for ii in sub if not b_box.exterior.intersects(ii)]
holes = [hole.representative_point() for hole in hole_polys]
[print(hole) for hole in holes]

Scratch that idea.

@robbievanleeuwen
So, it seems that just rounding the offset some decimal places does not work: it seems like it can create a small overlap between two geometries which causes triangle to spiral out of control.
I think that numpy.clip could hold a solution but it will be a bit tricky to implement. Started playing with it this evening. Will take some thinking.

Scratch that...numpy.clip can only handle floating point errors in one direction for each of min and max. Back the drawing board.

Gah! Simple solution: round the resulting coordinates, not the offset.
In Geometry.compile_geometry(), which generates the .points, .facets, etc. the coordinates are already being rounded to self.tol which is set to 12 decimal places. This is why the geometry can be created without holes but when using the generated points in a new geometry with .from_points(), we get what we want.
Totally forgot about that tolerance thing in the creation of .points. Going to re-order Geometry.__init__() so that the shapely object is exactly the same as the generated points because, right now, there is a a slight difference between the geometry you see and the points that are generated from it.

Fixed in #167

repo: robbievanleeuwen/section-properties


text:
Add README for Kubectl in #7175

This adds a readme file for the kubectl plugin as requested in #7175.

Thank you!

authors: kaltepeter mcornella

repo: robbyrussell/oh-my-zsh


text:
Update 10.09.24 fehlt

Ebenso bei COVID-19_7-Tage-Inzidenz_in_Deutschland

Hallo,
die Daten für heute sind nun verfügbar, vielen Dank fürs Melden.
Noch ein kurzer Hinweis: Wir haben ein internes Monitoring für die Datenbereitstellung, insofern ist es normalerweise ausreichend, uns erst dann Bescheid zu geben, wenn die Daten länger als 12 Stunden nicht verfügbar sind. Davor ist es nicht zwingend nötig, da wir uns normalerweise zeitnah um aufkommende Probleme kümmern.
Danke und herzliche Grüße
Simon | RKI Open Data Team

authors: HorstErnst RKIOpenData


text:
Swift Package Manager Support

Support for Swift Package manager

Add SPM Support

repo: roberthein/BouncyLayout


text:
Slow query at scale

First off, thanks a ton @robinpyon for the plugin. Saves us a lot of work!
In the process of migrating over our assets and tag to Sanity (7k assets and 2400 tags) our users' browsers were consistently crashing when accessing the media library. With some digging on the the network tab it looks like this query here is the main issue:
https://github.com/robinpyon/sanity-plugin-media/blob/cfa18eaf3b63828d63df8c5fb962e5877df57a5b/src/modules/tags/index.ts#L155
Pulling back 2400 tags from sanity via GROQ multiple times on page appears to be the crashing point.
I'm wondering if there's a better way here to display the tags to the end user while limiting the amount of tags we're pulling in a single query.
If our team can be of help at all in thinking about a solution don't hesitate to reach out.

That fix is now live in 1.4.0!

authors: dvmonroe robinpyon

repo: robinpyon/sanity-plugin-media


text:
Update FtpClientProxy reference

PR Summary
Commit e2633a5bd4f0a4f43ebdd971d89eaac9ad1898ff moved the location of FtpClientProxy.cs. This PR adjusts sources to changes.

Thanks for this.

authors: FanDjango emmanuel-ferdman

repo: robinrodricks/FluentFTP


text:
Copy initial docs

@brettpac as discussed earlier here copy-paste from ROS1

My 12 commits should have been one.  The changes in eb2de75 were meant to be the only commit. Edited README.md to update mentions of SMACC/ROS to SMACC2/ROS2

Is it possible to disregard the remaining commits?

authors: DecDury destogl


text:
interrupible ros lauch client behavior

┆Issue is synchronized with this Jira Task by Unito

duplicated


text:
Обновить модель профиля проекта

Task request
Цель задачи
Модель должна соответствовать структуре профиля на основании feature #50.

Решение задачи
Добавить новые поля в класс Project в models.py.

Сделано, однако дальнейшее изменение моделей может зависеть от других задач.

repo: robot-lab/projectsfair


text:
[ENHANCEMENT] Quick fix could be more specific than disabling while robotcode diagnostics

Is your enhancement request related to a problem? Please describe.
Robotcode recently add more granularity to disable diagnostics rule by rule rather than the whole diagnostics.
Quick fix shortcut still suggest disabling the whole diagnostics with # robotcode: ignore.
Describe the solution you'd like
When using quick fix for Variable XXX is not used, quick fix shortcuts add the # robotcode: ignore comment.
I would prefer the quick fix to add # robotcode: ignore[variable-not-used].
Describe alternatives you've considered
Not using the quick fix shortcut.
Additional context
This enhancement would help to spread the word about the new feature.

Additional options to consider:

Multiple Quick Fix Suggestions: In addition to the existing # robotcode: ignore option, it would be helpful to provide other Quick Fix suggestions. For instance, allow the user to turn the diagnostic into a warning or a hint instead of ignoring it completely.

Scope of Quick Fix Application: It would be beneficial to allow the Quick Fix to be applied not just on a single line, but also for larger scopes like the entire block, the whole file, or even globally. For the latter, options could include applying the fix via the VSCode settings or adding the rule to the robot.toml configuration file.

authors: d-biehl gohierf

repo: robotcodedev/robotcode


text:
Namespace is not removed when parse xml with lxml library

Original discussion in https://groups.google.com/forum/#!topic/robotframework-users/pNdU51HXnQc
When using lxml library and parse xml, namespaces are included when you get element from specific xpath. This does not occur when use standard ElementTree module (aka. use_lxml=False)
Can be reproduce by following data
<feed xmlns:verr="http://www.xyz.com/error" xmlns:vaext="http://www.xyz.com/atom-ext" xmlns="http://www.w3.org/2005/Atom">
<example>
  <first id="1">text</first>
</example>
</feed>

This is test case
*** Setting ***
Library  XML  use_lxml=True

*** Variables ***
${xml}  test.xml

*** Test Cases ***
test 1
   ${root}=  parse xml  ${xml}
   ${elem}=  get element  ${root}  example/first
   log element  ${elem}

When log element is executed, result is
<first xmlns:verr="http://www.xyz.com/error" xmlns:vaext="http://www.xyz.com/atom-ext" xmlns="http://www.w3.org/2005/Atom" id="1">text</first>

instead of
<first id="1">text</first>

Output of parse xml on lxml and ElementTree library should be the same.
Version information

Robot Framework 3.0
Python 2.7.11
Windows 7

First of all, sorry for ignoring this issue for so long. It was buried under a pile of other issues during summer time when I didn't have much time to follow this tracker.
Including namespace information in child elements seems to be a feature in lxml and avoiding that doesn't seem to be easy. Instead of that, I think it is better to add new strip_namespaces argument to Parse XML to allow stripping namespaces altogether and I already submitted #2534 about that.
With #2534 implemented, the problem in this issue can be avoided easily and I think this issue can be closed. Comment here if you disagreee and have use cases where the new functionality wouldn't work.

authors: Felys pekkaklarck

repo: robotframework/robotframework


text:
Support embedding Text test cases into Markdown files

Many source control systems including bitbucket support markdown natively. Adding markdown syntax will be much helpful.

Could you clarify where and how you'd like to use Markdown?

Markdown is supported in Visual Studio Team Services, Team Foundation Server and Bitbucket. There is no support for reStructuredText.
This will be more helpful to add "Acceptance Documentations" available naturally. Without which it adds additional dependencies to make documentations and scripts available as HTML, PDF documents.

@vairamsvsjdo Are talking about saving robot test as MarkDown (.md) files instead of .robot or .txt?

As we don't have unlimited history on our RF Slack channel I've saved the Slack into a Gist.

@Tset-Noitamotua that is exactly I was looking for. @pekkaklarck  please add this into robotframework support. It works excellent. I have about 50 test script files embedded with acceptance criteria documentations and all work perfect.

OK, you meant creating test cases in Markdown files similarly as you can create tests in reStructuredText files nowadays. I'm totally fine with that idea and, as @Tset-Noitamotua commented, there's already been discussion about that earlier on different forums. You could have also meant using Markdown when defining library documentation which is a valid proposal too.
What needs to be done to actually get the requested Markdown support into the framework is proving a pull request. The Gist by @Tset-Noitamotua is a good start but needs a little cleanup. A bigger task is creating tests and wring documentation. If there's anyone interested, please first take a look at the CONTRIBUTING.rst file in the project root. I'm willing to help here or on Slack too.

@pekkaklarck will do that appropriately.
Doc for libraries
Doc,
code for PEP8 standard
Support for HTML test cases
Support for text , tsv test cases
Are features in that support comes to my mind.
Identify others please.

Better to have just one feature per issue. This issue covers adding support for test cases in MD files, supporting MD in library docs requires another issue. That is likely to be a bit more complicated feature to implement too.

RF 3.0.3 has been delayed a lot. Now we want it out ASAP and decided to only include bug fixes in it. I'll thus descope this enhancement to RF 3.1 which we plant to start developing right after RF 3.0.3 is out.

@vairamsvsjdo, do you still have interest to work with this issue? Getting this into RF 3.1 would be great.

@pekkaklarck
Yes, will work on this next week. What is your timeline? I have few other tasks to take care in OSS next week.

Target for 3.1 is early June.

No interest lately. Removing from the long overdue RF 3.1 scope.


text:
CI: "Robot.Running.Timeouts With Logging.Timeouts with programmatic logging" acceptance tests fail from time to time

Related to PR.
The test Robot.Running.Timeouts With Logging.Timeouts with programmatic logging fails randomly on any combination of Python and OS. Either this is test case flakiness, or robot misbehaviour.
Logs are there: https://github.com/robotframework/robotframework/pull/3428/checks?check_run_id=365059240

Has this happened lately? Or were those tests just disabled?

I think you have fixed them here: 698fb2e51 ("Refactor tests to hopefully prevent strange issues on CI.", 2020-01-04). I haven't seen they were failing after that.

Ok, I guess we can close this then.

authors: pekkaklarck tysonite


text:
PR for FR 3240

MR for the issue 3240

Codecov Report

Merging #3813 (7218d63) into master (eb899fb) will decrease coverage by 0.68%.
The diff coverage is 1.50%.

@@            Coverage Diff             @@
##           master    #3813      +/-   ##
==========================================
- Coverage   74.61%   73.93%   -0.67%
==========================================
  Files         218      217       -1
  Lines       17961    17985      +24
  Branches     2919     2949      +30
==========================================
- Hits        13400    13296     -104
- Misses       4066     4176     +110
- Partials      495      513      +18

Impacted Files
Coverage Δ

src/robot/libraries/BuiltIn.py
26.33% <1.50%> (-1.55%)
:arrow_down:

src/robot/utils/encodingsniffer.py
66.16% <0.00%> (-21.53%)
:arrow_down:

src/robot/utils/robotinspect.py
47.62% <0.00%> (-19.04%)
:arrow_down:

src/robot/version.py
55.56% <0.00%> (-11.11%)
:arrow_down:

src/robot/utils/robotpath.py
64.16% <0.00%> (-9.43%)
:arrow_down:

src/robot/utils/robottypes3.py
90.91% <0.00%> (-9.09%)
:arrow_down:

src/robot/utils/argumentparser.py
78.02% <0.00%> (-5.67%)
:arrow_down:

src/robot/output/console/highlighting.py
69.77% <0.00%> (-5.42%)
:arrow_down:

src/robot/libdocpkg/xmlwriter.py
54.81% <0.00%> (-3.84%)
:arrow_down:

src/robot/utils/filereader.py
89.84% <0.00%> (-3.38%)
:arrow_down:

... and 6 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 eb899fb...7218d63. Read the comment docs.

I am going to close the PR and try to squash all my commits. It has a lot of them and this is not good.

I am going to close the PR and try to squash all my commits. It has a lot of them and this is not good.

authors: codecov-io vokiput


text:
De-sugar bang ! operator - part 4 - if-then-else

This PR builds upon #6613 and implements desugaring of suffixed if-then-else expressions as described in Chaining Syntax proposal
Tested using the following program(s).
app "example"
    packages { pf : "https://github.com/roc-lang/basic-cli/releases/download/0.8.1/x8URkvfyi9I0QhmVG98roKBUs_AZRkLFwFJVJ3942YA.tar.br" }
    imports [ pf.Stdout.{line}, pf.Task.{Task} ]
    provides [ main ] to pf

truthey = Task.ok Bool.true
falsey = Task.ok Bool.false

# Combination 1. NIL if_thens before the first suffixed, and NIL after
main =
    if truthey! then
        line! "SUCCESS"
    else
        line! "FAIL"

# Combination 2. NIL if_thens before the first suffixed, and SOME after
main =
    if falsey! then
        line! "FAIL"
    else if truthey! then
        line! "SUCCES"
    else
        line! "FAIL"

# Combination 3. SOME if_thens before the first suffixed, and NIL after
main =
    if Bool.false then
        line! "FAIL"
    else if truthey! then
        line! "SUCCESS"
    else
        line! "FAIL"

# Combination 4. SOME if_thens before the first suffixed, and SOME after
main =
    if Bool.false then
        line! "FAIL"
    else if falsey! then
        line! "FAIL"
    else if truthey! then
        line! "SUCCESS"
    else
        line! "FAIL"

I've added this as a draft... once #6601 is merged (if it's ready) then this should only be a single commit. I can rebase and update as required.

Closing in favour of #6634

repo: roc-lang/roc


text:
无法编译

go build -o paopao-api . build github.com/rocboss/paopao-ce: cannot load embed: malformed module path "embed": missing dot in first path element
go version
go version go1.13.8 linux/amd64

我也遇到一样的问题

go version need go1.16+

//go:embed support is since go1.16

authors: Howard-guo alimy onenov

repo: rocboss/paopao-ce


text:
Reading esv variables via cli is not machine processable

Frodo CLI version
0.23.0
Describe the issue
We have a CI pipeline which deploys all our variables to FR Identity Cloud. We iterate over the variables and use frodo to set them. However, if nothing's actually changed, we could skip this entirely, or at least rely on Frodo to skip the apply command.
Therefore, we looked at the output of this command:
npx frodo esv variable describe --variable-id my-awsome-variable https://openam-foo-euw2-staging.id.forgerock.io/am
The output is in human, not machine-readable format:

As such we couldn't process the output as part of some conditional logic to determine whether to call the command to create the variable with a different value.
Let's say there was a command line switch --read-value to just output the value of the variable, or --json to ask for output rendered in json, rather than a human-readable table, this would be significantly more useful.

@ashleyfrieze , a --json option would be handy, I agree! describe was never intended to be machine processable but I like that you are trying to use it that way. I'll look into it asap!
Feel free to put up a PR if you like to write some code yourself? :)

I agree, I have used the output and generated my own json file with {"Name":"esv-...","Value":"value"} and then did a command in bash like:
for file in $(ls -1 *.json); do
echo $file
name=$(cat $file|jq -r '.Name')
value=$(cat $file|jq -r '.Value')
frodo esv variable create -i $name --value "$value" substr
done

@jgdtech I don't know whether you're using ID Cloud or a self-installed AM. With ID cloud the consequence of deploying new variables is that there's some sort of restart required. This, in turn, makes the deployment take a long time.
If the variables haven't changed since the last time, then there's no reason to deploy. Similarly, iterating over the variables we wish to add doesn't allow unused variables to be removed.
So, I'm probably going to end up scripting the deployment against frodo lib... though the CLI should also be machine readable where possible.

authors: ashleyfrieze jgdtech vscheuber

repo: rockcarver/frodo-cli


text:
Add Wansmer/binary-swap.nvim & Wansmer/sibling-swap.nvim

Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[x] The title of the pull request is Add/Update/Remove `username/repo`  when adding a new plugin.
[x] The description doesn't start with A Neovim plugin for.., or A plugin for...
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized).

Thanks for the PR!

authors: Wansmer tversteeg

repo: rockerBOO/awesome-neovim


text:
Multiple grids in the same controller

I have a need to run about 4, maybe 5 completely different grids in the same controller, different configs, different schemas, different data.  To make matters worse, they can't all load at once.  Only when i execute the state machine function can they begin to load.  I am moving as much as i can to models to eliminate the uber controllers, but just the way this is put together, its not realistic to do it completely.
How would i pull this off?

There is no problem putting several forms in the same page/controller.
You can look at the demo app to see an example of a page that has 3 differently configured forms.
It is also possible to dynamically add more forms to the page, as they are plain directives.
Do you need the different forms to communicate with each-other? That may require some scope juggling, but other than that it should be pretty straight-forward.

authors: rodikh scphantm

repo: rodikh/angular-json-editor


text:
run.and_raise_error fails on regex mismatch

I tried writing a test, that checks weither a certain ParseError is raised by a parser function. Apparently this works fine as long as the regex is matching, but as soon as it does not, the test fails with:
wrong number of arguments (0 for 1)
This message is not really helpful and seems more like its a bug in rspec-puppet.

Most of this was rewritten in 2.2.0. I've added a test case to verify that this is not happening anymore.

authors: DavidS aptituz

repo: rodjek/rspec-puppet


text:
automatic closures?

Take as example the following code:
class foo {
    public var b = 2;

    public function fooMethod() {
	return this.b;
    }
}

class bar {
    public function barMethod(method) {
	return method();
    }
}

var a = new foo();
var b = new bar();
alert(a.fooMethod());
alert(b.barMethod(a.fooMethod));

I would expect (I know I should not expect this in standard javascript) the second alert to also display 2 instead of undefined.
Maybe add an option to add automatic closures to public methods? So "this" always executes in the correct scope, instead of me having to do a method.apply(scope,...).
It is much easier to not have to keep track of both scope/method when passing methods along different classes. After all, when working with classes, this is the way anyone would expect it to work.

Thanks!

I'm re-opening this issue because it will break with asynchronous callbacks and event handlers:
class foo {
	public function() {
		var bar = document.createElement("div");
		bar.className = "bar";
		bar.style.width = "200px";
		bar.style.height = "100px";
		bar.style.background = "#000";
		bar.onclick = function() {
			alert(this.className); //"this" should not resolve to the class instance
		};

		return bar;
	}
}

authors: CheloXL rogp

repo: rogp/jspp


text:
added error message when catalog has wrong api key

Description
This should give the proper error message in the log when the published documents are not showing on the browser without logging in. @sandeep-ps Since I am out, you can modify and push necessary changes if needed please
Fixes #923
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[x] When possible

Type of changes
Please select a relevant option:

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[x] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Not able to see published contributions when not logged in. The error remains the same: "2022-06-02T17:13:44.130Z INFO [Thread-35 ] : root - Contribution SEARCH {"search": "Contribution search performed with : None", "ep_building_block": "contributions_building_block", "ep_name": "Contribution", "ep_method": "SEARCH"} "

This PR is not the PR for showing the published contribution without logging in. If you want me to do include it, I will create an issue and include it in here.

Not able to see published contributions when not logged in. The error remains the same: "2022-06-02T17:13:44.130Z INFO [Thread-35 ] : root - Contribution SEARCH {"search": "Contribution search performed with : None", "ep_building_block": "contributions_building_block", "ep_name": "Contribution", "ep_method": "SEARCH"} "

Not able to see published contributions when not logged in. The error remains the same: "2022-06-02T17:13:44.130Z INFO [Thread-35 ] : root - Contribution SEARCH {"search": "Contribution search performed with : None", "ep_building_block": "contributions_building_block", "ep_name": "Contribution", "ep_method": "SEARCH"} "

I have checked with my local and it shows the published contributions without logging in. Are you sure it doesn't show the published ones without logging in?

Not able to see published contributions when not logged in. The error remains the same: "2022-06-02T17:13:44.130Z INFO [Thread-35 ] : root - Contribution SEARCH {"search": "Contribution search performed with : None", "ep_building_block": "contributions_building_block", "ep_name": "Contribution", "ep_method": "SEARCH"} "

@minump Can you post your published contributions Json in here to see if it is correct?

Added the message when there is no published contribution saying that there is no published contribution. Also, added the redirection to error pages when the api key is wrong from just showing main page with empty contributions.

Not able to see published contributions when not logged in. The error remains the same: "2022-06-02T17:13:44.130Z INFO [Thread-35 ] : root - Contribution SEARCH {"search": "Contribution search performed with : None", "ep_building_block": "contributions_building_block", "ep_name": "Contribution", "ep_method": "SEARCH"} "

@minump I have checked with my local and it shows the published contributions without logging in. Can we double check to see if it doesn't show the published ones without logging in?

Able to see published contributions when not logged in. Must have been a mistake from my end before.
Also able to see "There is no published contributions" in home.html when using wrong APIKey.

I will move the CHANGELOG entry to the unreleased section. The rest looks good.

I will move the CHANGELOG entry to the unreleased section. The rest looks good.

thanks

@minump, it looks like you had requested some changes in this PR. If they look good, could you please approve this PR? Thanks.

Thank you! Approved and merged.

authors: minump sandeep-ps ywkim312

repo: rokwire/rokwire-building-blocks-api


text:
++ steps to update the repo caches of deb/rh sys

I added 1 step for each OS_Family (debian + redhat) to update their repo caches before installing packages.
please check this for convience and give feedback.
may thanks

centos7 is failing with
TASK [./ : update repo-cache for debian/ubuntu] ********************************
fatal: [localhost]: FAILED! => changed=false
  cmd: dnf install -y python2-dnf
  msg: '[Errno 2] No such file or directory'
  rc: 2


text:
Capistrano3  deployments should not be blocked by intermittent SSL failures to api.rollbar.com

Yesterday https://status.rollbar.com/incidents/39rm7hwmplpm  (Intermittent SSL failures when connecting to https://api.rollbar.com ) blocked the deployment of all Rails apps.
I was able to deploy by commenting the following hooks:
https://github.com/rollbar/rollbar-gem/blob/54b32011b4f139db198ae3c32e166580d487492e/lib/rollbar/capistrano3.rb#L39-L46
Stacktrace:
SSL_connect returned=1 errno=0 state=error: certificate verify failed (unable to get local issuer certificate) (SSHKit::Runner::ExecuteError)
/var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/protocol.rb:44:in `connect_nonblock': SSL_connect returned=1 errno=0 state=error: certificate verify failed (unable to get local issuer certificate) (OpenSSL::SSL::SSLError)
	from /var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/protocol.rb:44:in `ssl_socket_connect'
	from /var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/http.rb:996:in `connect'
	from /var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/http.rb:930:in `do_start'
	from /var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/http.rb:919:in `start'
	from /var/jenkins_home/.rvm/rubies/ruby-2.6.6/lib/ruby/2.6.0/net/http.rb:605:in `start'
	from /var/jenkins_home/.rvm/gems/ruby-2.6.6/gems/rollbar-2.26.0/lib/rollbar/deploy.rb:48:in `send_request'

Can you rescue OpenSSL::SSL::SSLError / retry when it happens and add a warning maybe?

Failsafe from rollbar-gem. OpenSSL::SSL::SSLError: "SSL_connect returned=1 errno=0 state=error: certificate verify failed (unable to get local issuer certificate)" in /home/deploy/.rbenv/versions/2.5.5/lib/ruby/2.5.0/net/protocol.rb:44:in `connect_nonblock': error in process_item
me too

@truongduyng Contact support to request a credit for your quota.

Fix released in v2.26.1 https://github.com/rollbar/rollbar-gem/releases/tag/v2.26.1

authors: jmgarnier truongduyng waltjones

repo: rollbar/rollbar-gem


text:
🛑 Voxelmanip Forums is down

In 48b0dcc, Voxelmanip Forums (https://forum.voxelmanip.se) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Voxelmanip Forums is back up in d606253 after 7 minutes.

repo: rollerozxa/uptime-test


text:
Remove Evidence while keeping ABCI compatibility.

Removes Evidence from Block struct. Comments out relevant code lines with:
// Note: Temporarily remove Evidence #896
for adding back in the future when needed.
Closes: #896

LGTM, but this code needs to be tested against cosmos-sdk to ensure compatibility.

Was this tested @Manav-Aggarwal

LGTM, but this code needs to be tested against cosmos-sdk to ensure compatibility.

Was this tested @Manav-Aggarwal

yes, see https://github.com/rollkit/rollkit/pull/915#discussion_r1191152985

authors: Manav-Aggarwal nashqueue

repo: rollkit/rollkit


text:
TypeError: Cannot read property 'resolve' of undefined

Node-CommonJS Plugin Version: 10.1.0
Rollup Version: 1.27.2
Operating System (or Browser): Windows 10
Node Version: 12.13.0

How Do We Reproduce?

Reproduction steps are outlined in this reduced test case repo:
https://github.com/Dan503/BUG-rollup-plugin-node-resolve/tree/CommonJS-bug
Note this part of gulpfile.js in particular:
const rollupJS = (inputFile, options) => {
	return () => {
		return (
			rollup({
				input: options.basePath + inputFile,
				format: options.format,
				sourcemap: options.sourcemap,
				plugins: [
					CommonJS(), // <<-- This is the line that calls rollup-plugin-commonjs
				],
			})
				.pipe(source(inputFile, options.basePath))
				.pipe(buffer())
				.pipe(sourcemaps.init({ loadMaps: true }))
				.pipe(sourcemaps.write('.'))
				.pipe(gulp.dest(options.distPath))
		)
	}
}

Expected Behavior
It should not error
Actual Behavior
It throws this error when trying to run the code:
TypeError: Cannot read property 'resolve' of undefined
    at resolveId (C:\xxx\node_modules\rollup-plugin-commonjs\dist\rollup-plugin-commonjs.cjs.js:157:17)
    at C:\xxx\node_modules\rollup-stream\node_modules\rollup\dist\rollup.js:1750:32
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

I can confirm I have the same issue. Here is my gulp file:
import { dest, parallel } from "gulp";
import rollup from "rollup-stream";
import nodeResolve from "rollup-plugin-node-resolve";
import commonjs from "rollup-plugin-commonjs";
import vue from "rollup-plugin-vue";

const js = () =>
  rollup({
    input: "src/js/index.js",
    output: {
      format: "iife",
      file: "index.js"
    },
    plugins: [commonjs(), nodeResolve(), vue()]
  }).pipe(dest("dist/js"));

const build = parallel(js);

export { build };

Here is the error my console prints:
$ yarn build
yarn run v1.19.1
$ gulp build
[22:48:37] Requiring external module @babel/register
[22:48:39] Using gulpfile C:\xampp\htdocs\expense-manager\gulpfile.babel.js
[22:48:39] Starting 'build'...
[22:48:39] Starting 'js'...
[22:48:39] 'js' errored after 38 ms
[22:48:39] TypeError: Cannot read property 'resolve' of undefined
    at resolveId (C:\xampp\htdocs\expense-manager\node_modules\rollup-plugin-commonjs\src\resolve-id.js:54:15)
    at promise.then.result (C:\xampp\htdocs\expense-manager\node_modules\rollup-stream\node_modules\rollup\dist\rollup.js:1750:32)
[22:48:39] 'build' errored after 80 ms
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

However, with the same config but running native rollup, no errors:
// rollup.config.js
import nodeResolve from "rollup-plugin-node-resolve";
import commonjs from "rollup-plugin-commonjs";
import vue from "rollup-plugin-vue";

export default {
  input: "src/js/index.js",
  output: {
    format: "iife",
    file: "index.js"
  },
  plugins: [nodeResolve(), commonjs(), vue()]
};

$ yarn rollup --config
yarn run v1.19.1
$ rollup --config

src/js/index.js → index.js...
created index.js in 621ms
Done in 1.66s.

Do you know if there is something linked with rollup-stream or the plugins?

Hey folks, we've moved this repo to a new home in our plugins monorepo at https://github.com/rollup/plugins. If this issue is still pertinent, please reopen, by filling out the appropriate issue template, in the new repo. Thanks for originally opening this issue, and we look forward to seeing you in the new repo. This issue is being closed and this repo is being permanently archived.

authors: Dan503 khalyomede shellscape

repo: rollup/rollup-plugin-commonjs


text:
Version 5 still has transform hook return dependencies

Version 5, intended as the 'Rollup 1.0' version, still has dependencies return from the transform hook.
This was code written by me, to allow preprocessors to return additional watch dependencies. The API selected for the preprocessors was based on the old Rollup API for the transform hook, but probably it makes the most sense to still have dependencies be an array returned by the preprocessor?
this.addWatchFile is not mentioned in the Rollup changelog as far as I can see, but it looks like it is not present in 0.60.0, which is what's listed as a peer dependency of this plugin. So, despite rollup-plugin-svelte@5 being intended as the rollup@1 update, it should probably be considered breaking to assume this.addWatchFile is present.
So I think we can instead then just look for the truthiness of this.addWatchFile, call that if it's present, and otherwise put dependencies on the object returned by transform.
Sorry this issue description is so much longer than it needs to be.

thanks, released 5.0.1

authors: Conduitry Rich-Harris

repo: rollup/rollup-plugin-svelte


text:
Publish new version

@shellscape Updated some dependencies to solve security issues, would be nice if one of @Ashot-KR or @Swatinem could publish an updated version as you are the ones with publishing rights.

as you are the ones with publishing rights.

I just added you.
I wasn’t involved in this project for years, so I don’t really know if this is a patch/minor and how about the changelog etc…
If @Ashot-KR won’t get to it till tonight, I will take another look.

I'll do it today, but later

I don’t really know if this is a patch/minor

Technically the peerDependency version of Rollup was updated which would make it a major but I think this change is unnecessary as no code changes were included that warrant this. I will change it back to the old peerDependency version range so that we can release it as a patch.

published as 2.2.2

authors: Ashot-KR Swatinem lukastaegert

repo: rollup/rollup-plugin-url


text:
Some thoughts about adapter

Hi! Here's some thoughts about adapter interface.
I'm implement only Dataset, Relation and Gateway with my ideas, I think interface should be
simple as possible and low-level, without creating abstractions on top of native Redis data
structures. User must choose structures itself, depending on it's goals.
Calling to Redis is simple:
rom.relations.users.set(:key, :value).set('another', 42).get(:key).to_a

To get it done I'm collect all requests with creating new Dataset object and send it when we
call #view.
It gives full control, but with this solution I don't know how to be with Commands.
Maybe you can help me? :)

Ping @solnic :)

Commands receive a relation so they can use redis' interface. I'm gonna merge this in since it's a nice progress. I'll give you commit access too so feel free to continue working on this adapter. As I mentioned I don't use redis so I have no idea how this adapter should be built :)
We can chat about commands in a separate PR or just hit me on gitter.

@solnic ok, thanks! :)

authors: artemeff solnic

repo: rom-rb/rom-redis