[New Component] - Calendar

WIP

hello @huntabyte!
I have used the calendar added in this patch and wanted to give you some feedback (if you are interested in that). I know that this is a draft so maybe these issues are known, but hopefully some of this could be useful.
overall, great work, really nice implementation.
I did run into some issues:

it seems like passing for example class="p-0" to CalendarGridHeadCell and CalendarGridBodyCell does nothing. I am not sure why! I had to hardcode the classes I wanted to these components to make it work (see diff below)

for my application, I found it necessary to access the value from the root component. I added value to slotProps for the root component and that seems to have worked.

I wanted a more custom CalendarDate component, so I used asChild and followed the render delegation docs. however, I still get a lot of squiggly lines:

here is my custom date component:
https://github.com/ollema/skiftesgatan/blob/52bc51113b97813a84ce4c60a19961a59b22fe84/src/routes/laundry/calendar/LaundryDate.svelte#L12
(side note - the bits-ui docs tells you to use the builderAttrs function but that function did not exist. I assumed getAttrs should be used instead. maybe that assumption was wrong and that is the culprit!)
I have not found a fix to this third issue yet, so that is not part of the diff below.

finally, here is the git diff between the components in this patch and what I am using in my site:
diff --git b/src/lib/components/calendar/components/Calendar.svelte a/src/lib/components/calendar/components/Calendar.svelte
--- b/src/lib/components/calendar/components/Calendar.svelte
+++ a/src/lib/components/calendar/components/Calendar.svelte
@@ -86,6 +86,7 @@
        $: slotProps = {
                builder,
                attrs,
+               value: value,
                months: $months,
                daysOfWeek: $daysOfWeek
        };

diff --git b/src/lib/components/calendar/components/CalendarGridBodyCell.svelte a/src/lib/components/calendar/components/CalendarGridBodyCell.svelte
--- b/src/lib/components/calendar/components/CalendarGridBodyCell.svelte
+++ a/src/lib/components/calendar/components/CalendarGridBodyCell.svelte
@@ -21,7 +21,7 @@
 {#if asChild}
        <slot {attrs} />
 {:else}
-       <td {...attrs}>
+       <td {...attrs} class="p-0">
                <slot {attrs} />
        </td>
 {/if}

diff --git b/src/lib/components/calendar/components/CalendarGridHeadCell.svelte a/src/lib/components/calendar/components/CalendarGridHeadCell.svelte
--- b/src/lib/components/calendar/components/CalendarGridHeadCell.svelte
+++ a/src/lib/components/calendar/components/CalendarGridHeadCell.svelte
@@ -11,7 +11,7 @@
 {#if asChild}
        <slot {attrs} />
 {:else}
-       <th {...$$restProps} {...attrs}>
+       <th {...$$restProps} {...attrs} class="bg-background p-0 text-foreground">
                <slot />
        </th>
 {/if}

oh and here is the calendar in action if you are interested:
https://skiftesgatan.com/laundry/calendar
seems to work great both in desktop and in mobile


fix syntax error

except Exception, e: is python2-only syntax

LGTM, I think we'd rather support python versions > 2.5 than versions < 3 :)
http://stackoverflow.com/a/2535770

though I just noticed that it only worked with py3 here because I renamed rplugin/python to rplugin/python3. Is there a secret to making this work that I'm not aware of?

I'm happy to land this based on the syntax change, although I'm curious what you're seeing between rplugin/python & rplugin/python3

well, having python3 as python_host_prog, :UpdateRemotePlugins fails with Vim(let):E903: Could not spawn API job, with the debug log not mentioning anything about it. Unless I rename the rplugin dir to python3 and fix the path in ensime_init_path(), then it just works. Not sure what the right course of action would be here.

Could you please test if a symlink from python3 to python would work @tek? I've got the feeling that if we rename python to python3 directly, support for python2 would vanish.

if I replace path.startswith with a regex match test, it works, but the python2 plugin will still be attempted to be loaded. I guess this is a problem with neovim's plugin host initialization routine, I'll have a closer look at it to see if this can be avoided.
FWIW, for my plugins, I publish the python part on pypi and include only a dummy adapter in the neovim plugin, which makes handling deps and pythonpath trivial.

alright, it was just a configuration error after all. I had set python_host_prog = 'python3', but that variable is intended for the python2 interpreter. Unsetting it solved the problem. I apologize.


SQL not updating

As the title says, SQL isn't updating without me running /pex reload.
If I add myself into a group on one server and then go to another, I won't be in the group until I run /pex reload. Can this be fixed?..

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


⚠ Bofrost.be has degraded performance

In 27786ee, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1992 ms

Resolved: Bofrost.be performance has improved in aaff837.


Unable to compile when 'offscreenCanvasPonyFill' is imported

Hello Devs,
I have been trying to follow the tutorial on creating renderer plugins here: https://jbrowse.org/jb2/docs/developer_guide#creating-custom-renderers
When the clone this repo and attempt to create a renderer that uses 'offscreenCanvasPonyFill' as is used in the tutorial, I get the following error (I believe from tsdx):

✖ Failed to compile
Error: Unexpected character '' (Note that you need plugins to import files that are not JavaScript)
at /var/www/html/jbrowse-plugin-2/node_modules/canvas/build/Release/canvas.node:1:0

and a slew of import errors such as:

'fs' is imported by fs?commonjs-external, but could not be resolved – treating it as an external dependency

It seems that tsdx is attempting to compile the serverside version of the canvas library and the 'canvas.node' binary file. Perhaps something in the compiler configuration must be updated to ignore such files?
Also, when I remove the 'offscreenCanvasPonyFill'  code and just return 'null' in 'imageData' it compiles fine.
Here is the code from my MyRenderer.ts to reproduce the error:
import React from "react"
import { string } from "mobx-state-tree/dist/internal"
import {
  createCanvas,
  createImageBitmap,
} from '@jbrowse/core/util/offscreenCanvasPonyfill';
import PluginManager from '@jbrowse/core/PluginManager';
import { MyReactComponent } from "./components/MyReactComponent";
import ServerSideRendererType from "@jbrowse/core/pluggableElementTypes/renderers/ServerSideRendererType";

export { MyReactComponent }

export function MyRendererFactory(pluginManager: PluginManager) {
  return class MyRenderer extends ServerSideRendererType {
    render(props: Record<string, any>): Promise<Record<string, any>> {
      const { width, height, regions, features } = props
      const canvas = createCanvas(width, height)
      const ctx = canvas.getContext('2d')
      ctx.fillStyle = 'red'
      ctx.drawRect(0, 0, 100, 100)
      const imageData = createImageBitmap(canvas)
      return new Promise<Record<string, any>>((resolve, reject) =>
        {
          resolve(
            {
              reactElement: React.createElement(this.ReactComponent, { }),
              imageData: imageData,
              height: 100,
              width: 100,
            }
          );
        }
      );
    }
  }
}

it is a bit tricky to know exactly what's goin on without seeing the setup.
perhaps, you could checkout this plugin which includes a custom renderer (it also includes a custom display type that extends from the "linearwiggledisplay" so it is conceptually similar to a wiggle, but draws the little dots like a normal manhattan plot)
https://github.com/cmdcolin/jbrowse-plugin-gwas
you could maybe adapt such a thing to your needs

Hey cmdcolin,
Thanks for your message. I have looked at the GWAS renderer in some depth and I see that it extends the WiggleBaseRenderer. I'll see if I can adapt this for my needs. I am trying to extend a regular feature renderer like BoxRenderer/SVGFeatureRender to customize the way elements are “bumped” to allow certain overlapping features to have the same y-coordinate. Using a offscreen canvas is probably not  required for this but I was attempting to follow the plugin development tutorials (which are probably a bit outdated now).
After some testing, it seems to be some configuration issue with Rollup. I made a zero configuration with just Rollup (no tsdx) and ran into the same error after quite a bit of testing. Though I am building the plugin for the browser/umd, the 'canvas' module is being required and concatenated for the node-js version, and the compiler fails trying to compile the ‘canvas.node’ binary. A workaround I found was to modify ‘tsdx.config.js’ to make Rollup treat the ‘canvas’ module as an external with some dummy global name (only for umd/iife builds). Since the module will never be used in these builds it won’t be an issue.
If this is not a bug with the plugin-template but a user error on my part I can close this issue. However, I got the error using a more of less fresh clone of this repos with just a bit of code for my renderer (no modification of the compiler config).

you may have found a real issue then, we have indeed had a bit of trouble where we try to switch logic based on whether we think we are in the browser, where sometimes we have assumed that means webpack_require exists. since this is indeed a rollup based config, this would have issues being used. that said, if you are using Svg based renderers, you shouldn't have to use the offscreenCanvasPonyfill
Here is maybe a starter guide for creating a svg type renderer https://jbrowse.org/jb2/docs/developer_guide#rendering-svg

note that the "rendering" and the "renderer" are the same thing in this case so SvgFeatureRendererReactComponent==SvgFeatureRendering
with offscreen canvas based renderers, the "rendering" component is a react component that puts a canvas with the results of the offscreen canvas polyfill.
for a "svg" or react component based renderer, the "rendering" is just the react component, no offscreen canvas polyfill needed

Thanks for the explanation of how SVGFeaturerRenderer works. You're right that I probably won't need the offscreen canvas for my purposes, so I'll go about figuring out how to hack the react components of SVGFeaturerRenderer.
Since this seems to be a compiler configuration issue with little bearing on the application logic I'll go ahead and close this for now. I'm fairly new to web-dev but it seems that Javascript nowday's has it own verision of 'make-file hell' from good old C++, with all its transpilation, bundling, and whatnot.

I made a new issue number just to make a clean slate to track this because I do believe it is a valid issue https://github.com/GMOD/jbrowse-plugin-template/issues/15
also really interesting to see how far you are diving into the code! it is certainly a fairly complex system so do let us know if you run into anything else if you have ideas to make things easier

If I’m able to discover anything useful I’ll definitely let you know. It's an exciting project and honestly I’ve learned quite a bit by diving into the code.
If I have time would be to figure out how to get Webpack to work with this plugin template and make a PR. It would be good to have the consistency with the make jbrowse-components repo.


Asset color de-saturated look?

Description
We started noticing some assets when rendered in the Model Viewer they tend to look more de-saturated/ washed out...which is not the same as the default exposure value being too hot, since we already calibrated it, and the de-saturated look remains in both defaultLight as well as neutral ligthing environment.
Lighting Environment:
Default Light: exposure value 0.6
Neutral Light: exposure value 1.0
Asset
Only with albedo map
metalness: 0.0
roughness: 1.0
To replicate the result:

I screen-grabbed only the front-view in the default lighting setup
Overlay the  screen grabs on the Albeodo maps to see compare the differences.
GLB example here:
satuartion_checker.zip

Live Demo

https://glitch.com/edit/#!/model-viewer

Version

model-viewer: v1.7.1, also v1.6.0

Browser and OS

[X] Chrome, version: Version 84.0.4147.135
[X] Windows 10 64-bit

I don't quite understand what looks desaturated. What do you expect the render to look like?

Oh, maybe I do see. The shading varies based on the precise angles of the lighting (you can see how the saturation changes around the sphere). If you are comparing to the front face of the cube, that is just one arbitrary direction, and with slight changes to the camera you will find the colors all change accordingly. The albedo jpeg is a baseColor, so it represents the % of light reflected (material color), which is not the same as the rendered output color, since that is mostly driven by the details of the lighting and roughness.

Use this picture color block RGB values as an example.
The desire is to have the color rendered as close as possible to original albedo.
Rendered Asset RGB 226,211,181
Asset Albedo     RGB 238,201,148
The front view is the easiest way to explain it, we do see the desaturated look consistent from various of angles when reviewing production assets. To help simplify the variables in discussion, I put the metal to 0.0 and rough to 1.0.
Thomas

As I've discussed with the 3D Commerce Certification TSG, this desire for baseColor to match output color is fundamentally at odds with tone mapping, which we apply in order to avoid clipping artifacts and blown-out renders around bright reflections. This is a standard rendering practice since the results are so poor and unrealistic without it. However, to accomplish this, the gamut must be compressed. This is to help match the nonlinear behavior of the human eye. Please take samples from across your sphere to get an idea of the range of values to expect from a single base color under this neutral lighting. I think you'll find they vary quite a bit, but if they didn't the result would be worse. If you want base Color to be faithfully reproduced in the render (for instance with photogrammetry scans) you should export those using the unlit extension. However, we do currently have a bug where tone mapping is applied when it shouldn't be to unlit objects.

Does Model Viewer use ACES tone mapper? What the derivation of this coloration issue may result from the workflow?
If the Model Viewer uses ACES tone mapper, then maybe the content creation phase (e.g. using Adobe tools) need to also apply ACES LUTs?

Yes, we use ACES (technically it's the modified version, since our users are not in a darkened theater) and yes, I think applying that in the authoring tools is exactly the right step.


pallet_uniques : Impl set_collection_attribute and set_attribute

Changelog:

Extract set_attribute logic to do_set_attribute fn
Add set_collection_attribute and set_attribute function to non_fungibles::Mutate implementation

Hi @stanly-johnson 👋
Could you please explain why this change is necessary and why you want to do it?

Hi, want to manage creation/management of Nonfungible tokens from another pallet. Expecting a config that implements non_fungibles::Mutate and using pallet-uniques but realised that uniques does not implement the functions that allow setting the collection/item attritbutes. Thought this is the better way rather than tightly coupling pallet-uniques.


With async atomic select is ok but get failed

    async with misc.aio_dbm.atomic() as t:
        item = await misc.aio_dbm.execute(AdUnit.select(AdUnit).where(AdUnit.id == 1))  # Is ok
        item = await misc.aio_dbm.execute(AdUnit.get(AdUnit.id == 1)) #  is wrong
        breakpoint()

the error is such like
Traceback (most recent call last):
  File "/Users/jiamo/data/test/.venv/lib/python3.10/site-packages/peewee.py", line 3158, in execute_sql
    cursor = self.cursor(commit)
  File "/Users/jiamo/data/test/.venv/lib/python3.10/site-packages/peewee.py", line 3144, in cursor
    raise InterfaceError('Error, database connection not opened.')
peewee.InterfaceError: Error, database connection not opened.

So why  get is so different with the select.

Because get method is a shortcut to run query using synchronous peewee. Use
await misc.aio_dbm.get(AdUnit, id = 1)

instead


added manuals

The links were incorrect, added the manuals in the static/files and linked the manuals there.

Fixes #43

What about http://kwoc.kossiitkgp.in/mentor_form?

@pranitbauva1997  I didn't get that. The manual mentions the steps to add projects!

Thanks!


io.EOF error when getting successful HTTP 200 response

Problem statement
Getting an io.EOF error that is not properly checked when unmarshalling a discriminator field that is empty at the time of the GET request. I've modified the generated client code to resolve this as follows:
Before:
<myDiscriminator>, err := Unmarshal<myDiscriminator>(bytes.NewBuffer(<myDiscriminator>), runtime.JSONConsumer())
if err != nil {
  return err
}

After:
<myDiscriminator>, err := Unmarshal<myDiscriminator>(bytes.NewBuffer(<myDiscriminator>), runtime.JSONConsumer())
if err != nil && err != io.EOF {
  return err
}

Swagger specification
2.0
Steps to reproduce

Use v0.7.4 of CLI
Generate client from existing spec with references to discriminator
Execute client operation that returns HTTP 200 with an empty response in the discriminator field

A discriminator field can't be empty. https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#schema-object

Adds support for polymorphism. The discriminator is the schema property name that is used to differentiate between other schema that inherit this schema. The property name used MUST be defined at this schema and it MUST be in the required property list. When used, the value MUST be the name of this schema or any schema that inherits it.

Is there any chance you can actually submit swagger specifications to reproduce your issues?

Hmm, good point. So my explanation of the issue may be incorrect. I'm using the Swagger spec here -> https://westus.dev.cognitive.microsoft.com/docs/services/TextAnalytics.V2.0/export?DocumentFormat=Swagger&ApiName=Azure Machine Learning - Text Analytics
More specifically, when I call the /operations/{operationId} endpoint, the operationProcessingResult property of the OperationResult is empty on the first few HTTP 200 responses. The  placeholder in my code snippet is actually the OperationProcessingResult type.


feat: extend #[generate_trait] part

reference: https://github.com/starkware-libs/cairo/blob/main/crates/cairo-lang-plugins/src/test_data/generate_trait
est. 2 to 3 hours

@enitrat i can do this :)

@rkdud007 Cool! Please take your time, explain things clearly and provide comprehensive and meaningful examples 😄
There is a very limited part about generate_trait right now, we want to show what happens under the hood, what code is being generated, how to use it for generics, etc.

can I know where is generate_trait currently written? I cannot find it
I'm planning to add it on 7.2 traits in cairo bottom

actually I suggest to make another chapter or appendix for Plugins.

https://github.com/cairo-book/cairo-book.github.io/blob/cairo-2.0/src/ch07-02-traits-in-cairo.md#implementing-a-trait-without-writing-its-declaration

oh wait its on cairo2.0 branch, so that means i better push it on cairo2.0 branch right?

yes


No intellisense for .tsx files

Version: 1.6.0, OS X
I do get intellisense for .ts, but not .tsx.

Please read Readme https://github.com/neoclide/coc-tsserver#coc-tsserver

I was bit by this as well. Had been fine for months though.

Make sure your filetype is expected

@chemzqm Just added an autocommand to set it to typescript.tsx and it's fine now. Seems like I didn't have this issue until today though. Maybe something changed? Oh well, working now.

Use :verbose set filetype to check

@brandonpittman how did you manage to do it? It was working in another computer setup but is not working in this one

@romariolopezc Sorry, I can't remember what I did back then. I'm not using the autocommand anymore so I figured something else out.


Persist audit logs for GraphQL API requests

Related Issue or Background Info

#522
#797
#798

Changes Proposed

create models and database table for audit logging
create service and repo for storing the desired information
create Instrumentation component that reads the desired information from each graphql request, and stores it

(The model probably is adequate for MagicLink-based patient authentication—the service layer obviously is not aimed that direction.)
For further discussion

the expected approach here is that we will hook into a graphql-java Instrumentation object that we then inject into the graphql context, much as we have already done for conventional request logging. QUESTION: do we want to expand the scope of the existing QueryLoggingInstrumentation, or create a separate one for this?

 QUESTION: Should we be suppressing those from the logs? Or not worth it considering the number of people who ever use those?

ANSWER SO FAR: worth thinking about but not worth delaying this feature

QUESTION: how much is it worth to us to know what Facility things are happening at in a given query?

NO ANSWER YET (so the answer is "nothing" for this PR)

Punchlist

[x] unit and serialization tests for the new models
[x] round-trip test in the API using different users, that having been the point of that entire refactor in #1001
[x] some mechanism for manually verifying that audit logs are being created other than going into the DB (probably only for non-prod environments, though)

some mechanism for manually verifying that audit logs are being created other than going into the DB (probably only for non-prod environments, though)

We can use metabase for this

Can't use metabase to verify things in dev, though. 🙂
I have a thing built that works locally (it's how I did the demo last week), I'll put it in here (tomorrow) and we can debate if it should stay.

Tested in test, going to start rolling this out now. If people have problems with or questions about the actuator trick for viewing recent data, ping me by whatever channel seems most appropriate.


Add "✅ Successfully mined block" log message

Simply adds a log line to notify miners about found blocks

bot merge

bot merge


fix: add the signer header when using aa-alchemy

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
This PR enhances Alchemy client functionality by adding headers update capability and improving smart account client handling.
Detailed summary

Added updateHeaders method to modify headers in fetchOptions
Updated headers in lightAccountClient.test.ts
Improved handling of headers in bundlerClient.ts
Enhanced rpcClient.ts to update headers with fetchOptions
Added isSmartAccountWithSigner and getSignerTypeHeader in smartAccountClientFromRpc.ts
Updated tests in smartAccountClient.test.ts for header setting

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Overall, LGTM. !


Fixes #121 Added Ask.com support

Fixes #121 , #165

[x] Added Ask.com support
[x] Added Ask.com icon
[x] Python 3 compatibility
[x] Taken care of all the spaces and other specifications

@Pk13055 @vaibhavsingh97 @nikhilrayaprolu Please review. Sorry for inconvenience. My local repo got messed up. Sorry again.

@sayak119 Why the commits are mixed up?


cordova build error

it doesn't work when i build cordova-android 9 and cordova 9 version

This should be resolved in the latest version published to npm (0.3.5).


General Issue: Typo in the example impute_dtc_dt

Background Information
The code in line 707 may be impute_dtc_dt
https://github.com/pharmaverse/admiral/blob/a2b224a1d54afcc58ad7f06cfa60011415b02d4e/R/derive_date_vars.R#L701-L707
Definition of Done
No response

Wow!!  Thanks @ynsec37 !!  Are you on pharmaverse slack?

Also - do you mind fixing this?


Bug/tr 46 fix pypi install

Fix for Pip installations. Should be working as intended now, from what I understand after completing this task, it appeared the problem you were mainly facing was a lack of MANIFEST.in file, which is required to tell Pypi which files to include. This problem may also be causing the issue with long_description, but as explained inline, I felt that solving this problem now would distract from the PR itself and muddy it a bit. So that should be kept separate. End of the day, Pip install now works. That's what it's all about.

Coverage remained the same at 53.386% when pulling 0299a45cd18c39b9bb62395c83b36417c6590ec1 on JohnVonNeumann:bug/TR-46-fix-pypi-install into fe8f4c9b57c1fb0dff0fc3866cda3aff9cc2ffdd on Netflix:master.


Error in console. Crash on command execution

What is the output link of /viaversion dump?
Unable to get viaversion dump due to crash on command execution.
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
Latest version of bungeecord.
Plugins:
Advanced-portals
Animatronics
ArmorStandTools
Multiverse-Core
ProdigyGadgetDiamon
Protocollib
ServerSelectorX
Viaversion
How does this error happen? login? Using an item?
When starting the server, once everything enables and loads, the error appears.
Is there an error in the console? Use pastebin.com. Is there a kick message?
https://pastebin.com/U8V278Fb

Redownload ViaVersion and try again?

I've done that. Same thing.

So what's the full console output? To me it looks like file corruption but it could be something else I guess.

I would assume you've probably solved this as I've seen your posts on spigot, considering you're having it with other plugins i'd assume it's a java / network issue.


Add rowwise_mean to linalg using Eigen3

Refer #3051 and PR #3070, #3125

I didn't implement colwise_mean at the same time because I'm not sure how this implement looks like. Apparently I cannot copy SGVector<T> to SGVector<int2float type> so I just used for-loops. Also, I think we are supposed to use EIGEN3 methods, which I have not tried (but used for-loop instead).

According to #3100, I think we can get rid of <enum Backend> generic type? I can re-factor this after rowwise/colwise are both done.

There are also Block and symmetric matrix calculation for matrixsum. I am wondering whether we need both for matrixmean as while?

I saw the comparison highlighted the changes I made in PR #3125 (merged). I had no idea why this is happening...

Regarding last point , I think you are missing a rebase on updated develop. https://github.com/shogun-toolbox/shogun/wiki/README_developer , its explained under "versioning scheme"

I think for now we dont need to blocks. We might use later @lambday when we work more on the block mmd

@Saurabh7 is correct, you just have to rebase against develop.
Whenever you pull, use the rebase flag, so avoid this

Thanks I'll try rebasing. I am not familiar with SIMD so I might need a little bit reading. I was out of town for the last few days so I was not able to work on this.

Just use Eigen operations rather than loops.
On Monday, 4 April 2016, OXPHOS<EMAIL_ADDRESS>wrote:

Thanks I'll try rebasing. I am not familiar with SIMD so I might need a
little bit reading. I was out of town for the last few days so I was not
able to work on this.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/shogun-toolbox/shogun/pull/3136#issuecomment-205145078

Update: I cannot bypass loops now because of the type convert problem during mean calculation.
I tried:
/** int2float type */
typedef typename int2float<T>::floatType ReturnDataType;
 /** Eigen vector type - T */
typedef Eigen::Matrix<T,Eigen::Dynamic,1> VectorXt;
 /** Eigen vector type - int2float Type */
typedef Eigen::Matrix<ReturnDataType,Eigen::Dynamic,1> VectorXr;

SGVector<T> sum = rowwise_sum<Backend::EIGEN3, SGMatrix<T> >::compute(mat, no_diag);
Eigen::Map<VectorXt> s = sum;
Eigen::Map<VectorXr> r1 = result; // SGVector<ReturnDataType>
r1 = s / mat.num_cols; // **Error: MIX DATA TYPE**
Vector Xt r2;
r2 = s / mat.num_cols; // Correct
r2 = s / ReturnDataType(mat.num_cols); // **Error: MIX DATA TYPE**

I also tried using SGVector, but it doesn't support copy from one type to another
SGVector<T> sum = rowwise_sum<Backend::EIGEN3, SGMatrix<T> >::compute(mat, no_diag);
SGVector<double> mean(sum); // **Error:  no matching constructor for initialization of 'SGVector<double>'**

Ok then, I see.
Sorry I did not realise this problem when I asked for that.
Well then -- if you rebase, squash, and address the comments above, we can merge this
Thanks!

@karlnapf Learned something more about cast...Thought it would be easy in the first place. But I guess it's nice to have int -> float copy method in SGVector and 'SGMatrix` ?
I'll see how things work out this time. I'll squash once everything looks good.
And thanks for the tips on rebasing! @Saurabh7 Cute invoker : )

@OXPHOS Glorious invocation :) )

@OXPHOS I think such a method should be in linalg itself. Feel free to add....

Fine to merge apart from the data version

...May I ask how to access that data version thing? I cannot even find it. And the change was in my very first commit on this branch so I could not revert/reset. Rebase didn't seem to work either..

Did it..But still no idea where that data version change came from.

Data is changed whenever the git submodule data is changed (might happen if you run tests locally, or otherwise). It just stored the sha1 hash of the current version of the data submodule that shogun depends upon

Thanks for the patch! :)


Compile individual component Sass files

This PR

installs express-compile-sass to compile individual componet scss files on request
configure middleware in app.js and pass component name to component-preview view
modify component-preview view to not inherit from layout  view and load in component-specific scss file
move app iframe styles to app.scss file
modify gulp dev task so that it no longer compiles scss to css

This is to aid https://trello.com/c/7ROgYy2X/828-ensure-sass-imports-match-npm-dependencies
and  https://trello.com/c/unO51Rno/830-make-sass-imports-dependencies-less-broad, so we can check if components scss file imports all required dependencies

hmmm
FAIL src/button/button.test.js (16.518s)
  ● /components/button › /components/button/link › triggers the click event when the space key is pressed
    Timeout - Async callback was not invoked within the 5000ms timeout specified by jest.setTimeout.

      at node_modules/jest-jasmine2/build/queue_runner.js:72:21

Does this mean the styles in the review app are no longer going through postcss?

they do now. autoprefix and postcsspseudo

I won't be able to make this work for IE8, e.g. compile .scss with ie8 settings
$mq-responsive: false;
$govuk-is-ie: true;
$govuk-ie-version: 8;


Zcpy API: Remove unnecessary for loop from getSrcPe() method

This commit has been cherry-picked on top of charm-6.10.0


orbitdb is not replicated after a while

This may be hard to reproduce but creating this issue now -
Steps to reproduce

Setup Box/IPFS/IPFS-Cluster on two separate machines (in my case it was two rPis)
Connect fsync to Box0
Connect fuludocs to Box1 and see change events propagated to it
After a while the events stop getting propagated to Box1 (even if I reload the browser)
The only fix was to restart the nodejs 'Box' app on Box0.

I created two instances of Box on two ubuntu VMs using ansible playbooks (ipfs + ipfs-cluster + box). After doing the steps above, everything works and I can see the changes I make in the file reflected in the browser.
However, after a while it seems that fula-client gets disconnected. The cloud icon changes to a loading animation and it can't reconnect to the box again. Even when I refresh the browser it can't connect. I restarted Box service (the one fuludocs was trying to connect to) and the browser gets connected again.
@gitaaron is it possible that the issue is about fula-client getting disconnected and unable to connect again? Are you sure that fula-client was connected when you saw that changes aren't getting reflected in the browser?

Hi Mehdi,
I tried refreshing the browser and reconnecting and that did not work.  It
was reloading the previous CID.
If you want you can reassign this to me and I can come up with a more
reliable way of reproducing it.
Thanks,
Aaron
On Sat., May 28, 2022, 5:54 p.m. mehdibalouchi, @.***>
wrote:

I created two instances of Box on two ubuntu VMs using ansible playbooks
(ipfs + ipfs-cluster + box). After doing the steps above, everything works
and I can see the changes I make in the file reflected in the browser.
However, after a while it seems that fula-client gets disconnected. The
cloud icon changes to a loading animation and it can't reconnect to the box
again. Even when I refresh the browser it can't connect. I restarted Box
service (the one fuludocs was trying to connect to) and the browser gets
connected again.
@gitaaron https://github.com/gitaaron is it possible that the issue is
about fula-client getting disconnected and unable to connect again? Are you
sure that fula-client was connected when you saw that changes aren't
getting reflected in the browser?
—
Reply to this email directly, view it on GitHub
https://github.com/functionland/fula/issues/189#issuecomment-1140334330,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFDZZFZUEM5ZSQPROZ6Q3VMKIZ3ANCNFSM5WPVXZIQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


operator/network: add standard OperatorStatus/Spec members

This adds the standard OperatorStatus and OperatorSpec members to the struct, allowing the CNO to act a bit more like a "normal" operator.

/cc @danwinship

Are we actually planning to implement all of those fields?

We get most of them for free when we switch to the library-go tooling. The biggest open question would be ObservedConfig, which we should use for things like MTU that we infer unless overriden. But we don't need to use that just yet.

Should we have a CNO PR ready to go before merging this?

Generally yes, but I expect it to be a few PRs just for logistical purposes (unblocking people). I can hold this until the first one is filed.

Rebased, and I have a PR implementing this. This is ready for final lgtm.

/lgtm

/approve


Fix docs examples that use rowRenderer

Small thing i found while reading docs.
Great Repo. Keep it up. :beer:

Thank you! It's so difficult to keep docs updated properly. :)


Dialog: add undecorated

Missing method (peer.setUndecorated)

I guess this is supposed to be supported as new Dialog with RichWindow.Undecorated. Closing for now.


I have limited trial but no Move to Azure button?

What should I do?

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

ID: b244d131-ba7c-30e5-7dbb-9edd22df3957
Version Independent ID: 6932d029-8313-e4f7-1983-5ec25e1ad0b9
Content: Move a limited trial project to Azure
Content Source: articles/cognitive-services/Custom-Vision-Service/move-your-project-to-azure.md
Service: cognitive-services
GitHub Login: @anrothMSFT
Microsoft Alias: anroth

@byatis Thanks for the feedback. We are investigating into the issue and will update you shortly.

Hi @byatis :
Do you have an Azure subscription attached to the account you used to log into Custom Vision?

(the button will only appear if you have completed this prereq)

@anrothMSFT Thanks for your inputs. @byatis Could you please check if the correct subscription is attached to your custom vision login?

I have the same problem, do I need specific permissions on the attached subscription? ... this is part of a Company subscription so I am not an admin

I might have had two different subscriptions. I solved this by creating this again, so it's not actually a solution but was enough for me. I don't have the original solution anymore so I can't check anymore. :(

I have the same problem. No move to azure button.

I have the same issue. No move to azure button. Only a sign up button but with that email address I have a valid pay as you go subscription in azure.

@thefumigator @DClark-Landmark @byatis @byatis
Thanks to your reporting, we found and edge case where the Move to Azure button might not appear on limited trial projects. We've just deployed a fix. Please ping me if you're still unable to see it (after signing in and out again)

@anrothMSFT Still the same. Please see the screenshot. It's the casee when the model was created in Personal Account and Azure Subscription is in another default directory with same email address.

@thefumigator @DClark-Landmark @byatis @byatis
Thanks to your reporting, we found and edge case where the Move to Azure button might not appear on limited trial projects. We've just deployed a fix. Please ping me if you're still unable to see it (after signing in and out again)

Works for me now ... thank you for the quick response (New Microsoft :) )

@byatis @DClark-Landmark Thank you for the confirmation.
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

@ericbrunner sorry to ask for the favor. can you contact us via the "contact us" link in the customvision.ai porta. This generates an email directly to me + the rest of the engineering team. This is unexpected. We might have a few more questions. Will report back to this thread what the outcome was.

@anrothMSFT filed an initial request from the customvision portal

@anrothMSFT I filed an initial request in the CV portal

Outcome on this thread was Project Settings page vs overall Settings page.
Please close.


event.stopPropagation() required on default tour button clicks?

Clicking the default tour button fires the click event on containing elements, which isn't ideal.

Or maybe even event.stopImmediatePropagation()...? At the moment, if you have an anno on an element inside a form, clicking an AnnoButton submits the form.

In my tests this worked:
	  AnnoButton.prototype.buttonElem = function(anno) {
	    return $("<button class='anno-btn'></button>").html(this.textFn(anno)).addClass(this.className).click((function(_this) {
	      return function(evt) {
	        evt.preventDefault(); //<--Stop event from bubbling up to elements it shouldn't be fiddling with
	        return _this.click.call(anno, anno, evt);
	      };
	    })(this));
	  };


Terminal crashing before open

Type: Bug
Example of error
Inappropriate ioctl for device

https://user-images.githubusercontent.com/3511344/229139432-3351afa5-793b-481f-83af-85b67539659d.mp4
VS Code version: Code - Insiders 1.78.0-insider (f31dffc7f9b93a1cdef21243ec30dea7d489a615, 2023-03-31T01:49:42.343Z)
OS version: Darwin arm64 22.4.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
5, 6, 6

Memory (System)
64.00GB (19.24GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (73)

Extension
Author (truncated)
Version

svgo
100
3.0.0

vscode-nginx-conf
ahm
0.1.3

vscode-fish
bma
1.0.33

vscode-intelephense-client
bme
1.9.5

npm-intellisense
chr
1.4.4

vscode-css-modules
cli
0.4.2

gitignore
cod
0.9.0

vscode-markdownlint
Dav
0.49.0

vscode-eslint
dba
2.4.0

gitlens
eam
13.4.0

EditorConfig
Edi
0.16.4

LogFileHighlighter
emi
2.16.0

prettier-vscode
esb
9.10.4

code-runner
for
0.12.0

platformio
for
0.3.0

shell-format
fox
7.2.5

vscode-github-actions
git
0.25.3

vscode-pull-request-github
Git
0.62.0

go
gol
0.38.0

todo-tree
Gru
0.0.224

terraform
has
2.25.4

vscode-htmlhint
HTM
1.0.4

output-colorizer
IBM
0.1.2

r-pack
Iku
0.0.2

reason-vscode
jar
1.7.13

svg
joc
1.5.2

vscode-phpfmt
kok
1.0.31

bash-ide-vscode
mad
1.36.0

rainbow-csv
mec
3.6.0

vscode-language-babel
mgm
0.0.38

git-graph
mhu
1.30.0

dotenv
mik
1.0.1

vscode-apache
mrm
1.2.0

vscode-scss
mrm
0.10.0

vscode-docker
ms-
1.24.0

gather
ms-
2022.3.2

isort
ms-
2022.8.0

python
ms-
2023.6.0

vscode-pylance
ms-
2023.3.40

jupyter
ms-
2023.3.1000892223

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.288.0

remote-ssh
ms-
0.98.0

remote-ssh-edit
ms-
0.84.0

cpptools
ms-
1.14.5

remote-explorer
ms-
0.2.0

sftp
Nat
1.16.1

color-highlight
nau
2.5.0

proto
pet
0.0.2

material-icon-theme
PKi
4.25.0

polacode
pnp
0.3.4

gitlink
qez
1.2.4

inline-sql-syntax
quf
2.16.0

quick-lint-js
qui
2.12.0

quicktype
qui
12.0.46

java
red
1.16.0

vscode-commons
red
0.0.6

vscode-yaml
red
1.12.2

r
REd
2.7.2

shellman
Rem
5.6.1

postcss
ric
2.0.0

rome
rom
0.22.0

rust-analyzer
rus
0.3.1451

gitconfig
sid
2.0.1

vscode-styled-components
sty
1.7.5

vscode-stylelint
sty
1.2.4

svelte-vscode
sve
107.2.5

hlsltools
Tim
1.1.303

vscodeintellicode
Vis
1.2.30

viml
Xad
2.1.2

I'm also getting this on my macOS 13.2.1, arm64, fish after auto-update to insiders 1.78.0

can you reproduce this with all extensions disabled?

can you reproduce this with all extensions disabled?

Yup, all extensions disabled results in the same issue.

does setting terminal.integrated.shellIntegration.enabled:false fix it?

I added
"terminal.integrated.shellIntegration.enabled": false,
to the end of my settings.json. Relaunched, no difference.

@meganrogge I am googled a little and found this comment, maybe helps

I have the same issue. Also using Fish, version 3.6.0 and Visual Studio Code Insiders version
Version: 1.78.0-insider (Universal)
Commit: f31dffc7f9b93a1cdef21243ec30dea7d489a615
Date: 2023-03-31T01:49:03.052Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.3.0
Sandboxed: Yes

@dalisoft @meganrogge in particular this bit?

The basic issue is that fish checks if stdin is a tty via isatty; after confirming it is fish attempts to activate job control via the tc{get,set}pgrp family of functions and here receives ENOTTY: stdin is not a tty.

nushell has the same issue.

Currently workaround is using tmux + fish. On Terminal tab, click to + and choose tmux and you can use as workaround

NB, csh also warns of the problem but will start.

On latest Insider version it works on macOS. Try upgrade both fish and VSCode Insider to latest. Thank you to all team who worked on this fix
Info:
Version: 1.78.0-insider
Commit: cbf35328c8357175b9cd747823f39ade987c3d3c
Date: 2023-04-04T19:09:48.072Z (8 hrs ago)
Electron: 22.4.3
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.4.0
Sandboxed: No

For anyone interested, my linked and expanded writeup for this issue is here https://github.com/microsoft/vscode/issues/178999


Issue 3: Implement JSON Import and Export Backup Functionality - UI (Android)

This PR introduces the feature of restoring and creating a backup from the Hive DB by parsing the JSON file content to/from hive models. This PR includes all the UI part for Android.
Closes issue #3
Evidence

You don't need to write platform-specific code in order to save a file. There is already a plugin provided by google that does that since day one.
https://docs.flutter.dev/cookbook/persistence/reading-writing-files
Use this plugins in order to provide a common solution for all platforms. I don;t see any edge case where you will need a platform specific implementation.


Support for gpt2

Hi,
GPT2 is supported based on the documentation and previous working version of mindrlhf. However, in the latest code, there is a hardcoded check allowing only pangu and bloom models. How can we enable gpt2 training?
Thanks
Commit: 5fb1273

GPT2：https://github.com/mindspore-lab/mindrlhf/pull/33


🆙📅 Update API names 📛

Fixes https://github.com/microsoft/vscode-cosmosdb/issues/1237

Is it worth renaming the events too? I know it'd probably mess with dashboards?

Is it worth renaming the events too? I know it'd probably mess with dashboards?

User-facing strings are higher priority. No strong preference on changing the event names (but if you change them - I expect you to also update the dashboard and re-categorize all gdpr stuff)

@fiveisprime Said he wants it to remain SQL 🤷♂


Floating headers in HTML output

Hi there.
I recently ran Rubocop over a decent sized existing project and it worked great, but I found it a minor pain trying to figure out what file a particular offence related to. A lot of offences are similar, for example favoring . over :: for static method calls, and I found myself searching the HTML output for files which committed that offence and if a file had (say) 30 offences (not uncommon), I would have to scroll up a long way to figure out which file I needed to change.
This PR is my first attempt and I'm aware it's not complete/polished, but I have found it very helpful and I was hoping to start a discussion about the merits of this kind of feature.
Thanks!

I think this would be useful, so I'd accept a polished version of this PR.

Great, thanks for that. Any hints on what kind of polish? Obviously making the test suite pass would be step 1, but what else? Have you tried it in a browser? What did you think of the UI?

Also, what level of browser support do you deem acceptable? For example, the classList API does not work in IE9 and below. I will try to make it only progressive enhancement, ie it should retain the current behaviour in all browsers.

Great, thanks for that. Any hints on what kind of polish? Obviously making the test suite pass would be step 1, but what else? Have you tried it in a browser? What did you think of the UI?

A changelog will be needed as well. Haven't had time to play with it, so you can just post a screenshot/animated gif.
I'm fine with not supporting IE9 as I doubt many devs are using it.

Alright, I've tidied up the formatting, added some comments, tested in IE (it works), and updated the test suite. Pull?

You'll have to rebase on top of the current master.


Keras Model.compile() exception text is unclear when a loss method is not specified during compile()

System information

TensorFlow version (you are using): 2.3.1
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state. User is not given an obvious error message when  tensorflow.keras.models.Sequential.compile() is called and a loss function is omitted.
# Build the model
tf_model = Sequential()
tf_model.add(LSTM(
    units=32,
    input_shape=[window, 1],
))
tf_model.add(Dense(units=1))
tf_model.compile()
tf_model.fit(rolling_x, rolling_y, epochs=100)

Triggers the following exception:
\env\lib\site-packages\tensorflow\python\keras\optimizer_v2\optimizer_v2.py:1270 _filter_grads
        raise ValueError("No gradients provided for any variable: %s." %

Will this change the current api? How? No
Who will benefit with this feature? New users

Can you try with tf-nighlty?

Yes, this still occurs with nightly. PS. Due to Nvidia software mismatch this was run on CPU and not GPU. I don't think that matters.

Can you fill your variable in you code example so that I could run your same code quickly?


Upgrade scroll area

There are a bunch of other changes in that primitives release too, I've yet to document the release/changelog there.

There are a bunch of other changes in that primitives release too, I've yet to document the release/changelog there.

Oh, didn't realise there're other new primitives versions. Should I upgrade them too?

Pushed upgrades to other primitives and all PostCSS packages. Confirmed that this had no effect on the built CSS.


[AutoPR eventhub/resource-manager] EventHub: added $skip and $top to list calls

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2958

(message created by the CI based on PR content)


Cargo pulls older version (0.8)

Feature Description
Cargo add ...
[dependencies]
duckscript = "0.8"
duckscriptsdk = "0.9.3"

Shouldn't this be:
[dependencies]
duckscript = "0.9.3"
duckscriptsdk = "0.9.3"

Attempting to manually pull a newer version of duckscript reveals that 0.8 is the newest version available.
If this is a case of using different semver for package types, then close this issue as I assume you probably have an internal reason for doing so.

Realized each component has it's own version. I was thrown off by the github release version.


Reorganize user options into nav dropdown menu

Fixes #446 - the UI should cover all the scenarios described by the ticket, worked with @thatbudakguy
Looking for input on:

The way we found the user's organizations and if there's a better Rails magic way
Tests, how to cover the cases of being logged in with different roles (admin, and user who is a member of an organization)

For the UI colors, we kept things in default dark mode for now until future design work specifies any changes.

For testing users with different roles/attributes you'll need to create them using factorybot. You're already using factory bot when you set the current_user to :user:  let(:current_user) { create(:user) }. You probably already found it, but that :user is defined here: https://github.com/pod4lib/aggregator/blob/main/spec/factories/users.rb#L4-L9.
You can create a new user or add traits to this user to test various scenarios. Some links to docs about how to define and use factories:

https://github.com/thoughtbot/factory_bot/blob/master/GETTING_STARTED.md#defining-factories
https://github.com/thoughtbot/factory_bot/blob/master/GETTING_STARTED.md#using-factories

Here are some updates and more tests which may be overkill!
I was curious about the preferred place to test the links in the menu - in the template spec vs. feature spec like we discussed this morning. Right now it's sort of split between spec/features/nav_dropdown_spec.rb and the existing login/logout link tests in spec/views/shared/nav.html.erb_spec.rb


Fix "Error: init() not called, Hydra requires a configuration object."

Generate the hydra service project with logger, npm install and npm start, following ERR throw
    at IHydra._getServiceName (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:379:13)
    at IHydra.getServiceName (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:1818:18)
    at hydraOpts (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/fwsp-logger/lib/HydraLogger.js:7:37)
    at HydraPinoLogger (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/fwsp-logger/lib/HydraLogger.js:22:11)
    at HydraLogger.initLogger (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/fwsp-logger/lib/HydraLogger.js:54:20)
    at HydraLogger.setConfig (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/fwsp-logger/lib/HydraLogger.js:40:10)
    at Promise.series (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:107:74)
    at Array.map (native)
    at Function.Promise.series (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:4:14)
    at loader (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:107:24)
    at Promise (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:212:16)
    at Promise._execute (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/bluebird/js/release/debuggability.js:300:9)
    at Promise._resolveFromExecutor (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/bluebird/js/release/promise.js:483:18)
    at new Promise (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/bluebird/js/release/promise.js:79:10)
    at IHydra.init (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:105:25)
    at IHydra.init (/Users/khoinqq/Working/opensource/hydra-event-bus-service/node_modules/hydra/index.js:1779:18)```

Why?
`HydraPinoLogger` init with a call to `hydra.getServiceName` which require hydra fully initialized, but it is not at this time.

Solution: Use hydra config service name instead

For now hydra.sendToHealthLog emit log event, HydraLogger capture it to do the log.
However, it not works if the message is an object, this will be required in case of an Error object in the catch().
This pull request contains the fix for that, I set Error.message as entry.msg and Error.stack as entry.stack

New PR for use errSerializer


Create Test Environment

FSM readers
Create FSM Class  (faulty and correct)
i) transitions are stored in a useful data structure
ii) a funtion to apply test sequences  with output sequence
implement suspected faulty final states for hypothesis
implement suspected faulty final states for paper

step completed

step completed

step completed

and 4. step completed


tests/lean/inductive_resultant_level_inference.lean failing in clang release builds

according to Travis

introduced by 7dcc36277ab179c23a22b9cc008b4689dc028a18

I know about this and will fix it tomorrow.

Fixed by
d37fd17725b450ea12
89d0f1773e9fb6


The authorization mechanism you have provided is not supported. Please use AWS4-HMAC-SHA256.

Below Sample returns subjected error, using latest version of java sdk
package com.amazonaws.samples;
import java.io.IOException;
import java.net.URL;
import com.amazonaws.AmazonClientException;
import com.amazonaws.AmazonServiceException;
import com.amazonaws.HttpMethod;
import com.amazonaws.auth.profile.ProfileCredentialsProvider;
import com.amazonaws.services.s3.AmazonS3;
import com.amazonaws.services.s3.AmazonS3Client;
import com.amazonaws.services.s3.model.GeneratePresignedUrlRequest;
public class GeneratePreSignedUrl {
private static String bucketName = "bucketforwso2";
private static String objectKey  =  "/HTML-CHEAT-SHEET-768x8555.pdf";
public static void main(String[] args) throws IOException {
	@SuppressWarnings("deprecation")
	AmazonS3 s3client = new AmazonS3Client(new ProfileCredentialsProvider());

	try {
		System.out.println("Generating pre-signed URL.");
		java.util.Date expiration = new java.util.Date();
		long milliSeconds = expiration.getTime();
		milliSeconds += 1000 * 60 * 60; // Add 1 hour.
		expiration.setTime(milliSeconds);

		GeneratePresignedUrlRequest generatePresignedUrlRequest =
			    new GeneratePresignedUrlRequest(bucketName, objectKey);
		generatePresignedUrlRequest.setMethod(HttpMethod.GET);
		generatePresignedUrlRequest.setExpiration(expiration);

		URL url = s3client.generatePresignedUrl(generatePresignedUrlRequest);

		System.out.println("Pre-Signed URL = " + url.toString());
	} catch (AmazonServiceException exception) {
		System.out.println("Caught an AmazonServiceException, " +
				"which means your request made it " +
				"to Amazon S3, but was rejected with an error response " +
		"for some reason.");
		System.out.println("Error Message: " + exception.getMessage());
		System.out.println("HTTP  Code: "    + exception.getStatusCode());
		System.out.println("AWS Error Code:" + exception.getErrorCode());
		System.out.println("Error Type:    " + exception.getErrorType());
		System.out.println("Request ID:    " + exception.getRequestId());
	} catch (AmazonClientException ace) {
		System.out.println("Caught an AmazonClientException, " +
				"which means the client encountered " +
				"an internal error while trying to communicate" +
				" with S3, " +
		"such as not being able to access the network.");
		System.out.println("Error Message: " + ace.getMessage());
	}
}

}
Generating pre-signed URL.
Pre-Signed URL = https://bucketforwso2.s3.amazonaws.com/%2FHTML-CHEAT-SHEET-768x8555.pdf?AWSAccessKeyId=AKIAJH5K4ZJ6ZQB2F4XB&Expires=1514962084&Signature=rY7RjDwJkck8ZCEGWUr4lSm%2Bkuw%3D

InvalidRequest

The authorization mechanism you have provided is not supported. Please use AWS4-HMAC-SHA256.

2947D4830B323D3A

4JK5mlUk/a7oOd6WJT5ol5X0prJY+0buLKr9zhnOcoz0mxAk/oc9A/nWHgIC8lggT11xneCtRLU=

have you find solution for this issue ?


Implemented the SwaggerUI urls config parameter which displays a specs dropdown

When the urls parameter is specified in the SwaggerUI config, a dropdown of API specs is displayed instead of an editable textbox. This can greatly help organize the multiple specs in a project, as suggested in this StackOverflow answer.
This behavior is currently triggered by simply adding a name parameter to the specs object, e.g.:
{
    "specs": [
        {
            "version": "0.0.1",
            "title": "Api v1",
            "name": "v1",
            "endpoint": "v1_spec",
            "description": "This is the version 1 of our API"
        }
    ]
}

When this parameter is not included in the specs object, the editable textbox is displayed as usual. It might be more intuitive to have this behavior triggered by an external flag or something, but I'm not sure how this interface would look like.
Let me know if you have any ideas for this!

@rochacbruno Do you plan to have a new release for the flasgger package anytime soon? Seems like it's been over a year since the last one and there have been some good features merged.


When I configured batch size 4 on v3-8, it was normal. But when I configured batch size 128 on v3-256, it reported OOM.What is the reason?

When I configured batch size 4 on v3-8, it was normal. But when I configured batch size 128 on v3-256, it reported OOM.
What is the reason?
...
--mp_mesh_dim='-1, 1'
--train_dataset.json_dataset.batch_size=128
...

--initialize_jax_distributed=True \

"For example, if we have 8 accelerators for each host and 32 hosts in total, this gives us a total of 256 accelerators. We can use a 2D mesh of shape (8, 32) to specify that one model is partitioned across 32 accelerators for model parallelism, and each parition has 8 replicas for data parallelism."
Does this sentence mean --mp_mesh_dim='8,32'?

The --mp_mesh_dim='-1, 1' indicates that you are using a 3D mesh size of 1x256x1 in a v3-256 configuration. To replicate your v3-8 setting and only expand the data parallel axis, you'll need to specify --mp_mesh_dim='8,1'.


npm 3+ peer

npm WARN peerDependencies The peer dependency yo included from generator-kickstart will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.

will be fine. http://blog.npmjs.org/post/110924823920/npm-weekly-5


feat: Introduce v1alpha2 API version in preparation for changes

What type of PR is this?
/kind feature

What this PR does / why we need it:
In preparation for the API changes proposed in #361, this PR introduces and switches to API version v1alpha2.  The changes proposed in #361 will be in a subsequent PR.
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 4 (50.0%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.5%) to 45.354%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/azdiskschedulerextender/azdiskschedulerextender.go
2
4
50.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/azdiskschedulerextender/azdiskschedulerextender.go
7
57.58%

Totals

Change from base Build<PHONE_NUMBER>:
-1.5%

Covered Lines:
205

Relevant Lines:
452

💛  - Coveralls


Unable to determine the workspace. - Error after update to Rider 2018.2

Can no longer retrieve latest from TFS after updating Rider to 2018.2. Worked fine prior, during the same day. On latest 1.1.150 version of the plugin. Not sure if the VSO plugin needs an update or if this is a bug caused by Jetbrains.

Error:An argument error occurred: Unable to determine the workspace. You may be able to correct this by running 'tf workspaces /collection:TeamProjectCollectionUrl'.

Log details show:
16:38:32.141 | INFO  | TFSUpdateEnvironment           | Update on files initiated...
16:38:32.141 | INFO  | VcsHelper                      | getRepositoryContext: cache hit: C:/RiderTFS/Source
16:38:32.141 | INFO  | TFSVcs                         | TFSVcs.getServerContext repositoryContext is null: false
16:38:32.141 | INFO  | ToolRunnerCache                | getRunningToolRunner: toolLocation={0}
16:38:32.142 | INFO  | ToolRunnerCache                | getRunningToolRunner: fast version -
<IP_ADDRESS>709051449
16:38:32.142 | INFO  | ToolRunnerCache                | getRunningToolRunner: key=c:\tf\tf.cmd|
16:38:32.142 | INFO  | ToolRunner                     | sendArgsViaStandardInput: proceedWithArgs: get
-noprompt C:/RiderTFS/Source C:/Users/mhenderson/.Rider2018.2/config/scratches C:/Users/mhenderson/.Rider2018.2/system/extResources -recursive
16:38:32.142 | INFO  | ToolRunnerCache                | updateCachedInstance: caching a new runner:
key=c:\tf\tf.cmd|
16:38:32.142 | INFO  | ToolRunner                     | ToolRunner.start: toolLocation = C:\tf\tf.cmd
16:38:32.142 | INFO  | ToolRunner                     | ToolRunner.start: workingDirectory = null
16:38:32.142 | INFO  | ToolRunner                     | arguments: @
16:38:32.319 | INFO  | Command                        | ERROR: An argument error occurred: Unable to
determine the workspace. You may be able to correct this by running 'tf workspaces /collection:TeamProjectCollectionUrl'.
16:38:32.321 | INFO  | Command                        | CMD: Team Explorer Everywhere Command Line Client
(version <IP_ADDRESS>709051449)
16:38:32.321 | INFO  | Command                        | CMD:
16:38:32.322 | INFO  | Command                        | CMD:  get command:
16:38:32.322 | INFO  | Command                        | CMD:
16:38:32.323 | INFO  | Command                        | CMD:   Retrieves a read-only copy of one or more files
from the source control
16:38:32.323 | INFO  | Command                        | CMD:   server to the local disk. Any intermediate folders
are created if necessary.
16:38:32.323 | INFO  | Command                        | CMD:
16:38:32.323 | INFO  | Command                        | CMD:   The version to retrieve may be specified
through the 'version' option or as
16:38:32.323 | INFO  | Command                        | CMD:   a version specification suffix to the item
specification (example:
16:38:32.323 | INFO  | Command                        | CMD:   '$/file.txt;C34').
16:38:32.323 | INFO  | Command                        | CMD:
16:38:32.323 | INFO  | Command                        | CMD:  Valid option sets:
16:38:32.323 | INFO  | Command                        | CMD:
16:38:32.341 | INFO  | Command                        | CMD:   get [/version:<value>] [/recursive] [/preview]
[/force] [/all] [/overwrite]
16:38:32.341 | INFO  | Command                        | CMD:       [/noautoresolve] [<itemSpec>...]
16:38:32.341 | INFO  | Command                        | CMD:
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command: Team
Explorer Everywhere Command Line Client (version <IP_ADDRESS>709051449)
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:  get
command:
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:
Retrieves a read-only copy of one or more files from the source control
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:   server
to the local disk. Any intermediate folders are created if necessary.
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:   The
version to retrieve may be specified through the 'version' option or as
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:   a
version specification suffix to the item specification (example:
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:
'$/file.txt;C34').
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:  Valid
option sets:
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:   get
[/version:<value>] [/recursive] [/preview] [/force] [/all] [/overwrite]
16:38:32.385 | WARN  | SyncCommand                    | Unknown response from 'tf get' command:
[/noautoresolve] [<itemSpec>...]
16:38:32.385 | INFO  | Command                        | 243662200(ns) - elapsed time for get -noprompt
C:/RiderTFS/Source C:/Users/mhenderson/.Rider2018.2/config/scratches C:/Users/mhenderson/.Rider2018.2/system/extResources -recursive
16:38:32.385 | WARN  | RiderApplicationLoadListener   | [STDERR] : 243662200(ns) - elapsed time for get
-noprompt C:/RiderTFS/Source C:/Users/mhenderson/.Rider2018.2/config/scratches C:/Users/mhenderson/.Rider2018.2/system/extResources -recursive
16:38:32.387 | INFO  | AbstractCommonUpdateAction     | Calling refresh files after update for roots:
[C:/RiderTFS/Source/, C:/Users/mhenderson/.Rider2018.2/config/scratches/, C:/Users/mhenderson/.Rider2018.2/system/extResources/]

It seems with the size of the repository I am dealing with the java process started up to try to take care of this gets stuck in some infinite loop; at this point I have simply had to disable the plugin entirely for Rider to be usable again.

I am also getting this error on rider.

Sometimes this may be caused by Visual Studio TFS workspace being imported into a portable (TFC-CLC) one. The recommendation here would be to try running tf workspaces /collection:TeamProjectCollectionUrl as the command-line cliend suggests, e.g.
tf workspaces /collection:http://myserver:8080/tfs/DefaultCollection

It could help with remapping the workspace.

While it was unlikely the original way or reproducing the issue, I've found at least some way to reproduce that.
Stage 1: Reproduce #201
First of all, you'll need to obtain version of the plugin where #201 was easily reproducible. For example, install Rider 2019.1.1 where version 1.151 of the Azure DevOps plugin is bundled.

Checkout any TFVC project containing .NET solution
Add an xunit test project
Press the Refresh button in the local changes list
Make sure no changes is shown: here you go, you've reproduced #201

Stage 2: Reproduce #173
Now, even if you update the plugin to 1.152 (the latest version currently), and then go to the Incoming tab in the Version control tool window, you'll be unable to get any changes: pressing the Update Changes button will only show the Messages window with the error message:

The issue won't be fixed by tf workspaces, because it has other root cause. We should port the same fix we used for #201, but not only for tf status command, but for tf get, too.


Lab 2 - peer review

Hello, I have read your solution for lab 2 and want to leave you some thoughts:

Your code is well structured and fairly readable, despite lacking explicit explanations of the thought process behind it;
Your mutation rate is on the lower side, and you are not checking for premature convergence, which may cause the algorithm to get stuck when a suboptimal is found;
You left too many prints/logs showing the game state but no visualization of how the agents are evolving other than printing the fitness value;

Nice job, and good luck with the next lab

Thank you for the review, you are right, I think the the algorithm get stuck in a local optimal and I should have make better use of the logs,


RPC to get Messages from IPFS Payload Locations

Given that a schema can be created with Payload Location IPFS (#190)
Unblocked by building on top of https://github.com/LibertyDSNP/frequency/pull/247 or main if that is merged.
As a consumer of the network, I want to be able to read the messages from schemas with a payload location of IPFS, so that I can read those messages
Acceptance Criteria

[ ] I can call an RPC that to retrieve previously added messages to that IPFS location schema
[ ] I should be able to read enough data from each message to be able to retrieve the file from IPFS
[ ] I should receive the payload length data so I can validate the file I receive from IPFS
[ ] (Pagination) I can receive a large number of messages over several calls if needed

I should receive the payload length data so I can validate the file I receive from IPFS

currently it seems like we are not storing payload_length or the ipfs file location in messages.
we have generic rpc to get messages with schema id, I'm not sure what we are trying to achieve here?

@aramikm question ipfs file location?
I think we just have to know the cid and any gateway gives file back right?
We can store the payload_length that way the get from IPFS gateway content length can be matched with stored payload_length
So as long as we are storing the cid and length we are good unless we run own IPFS gateway then we can specify a location per se

@aramikm / @saraswatpuneet The PR #247  was merged. So add_ipfs_message is the extrinsic for adding an IPFS message.

@saraswatpuneet yes by location I meant cid
@wilwade There is an extrinsic but it's not storing any of mentioned data


replace tab before authors with two spaces

this makes authors consistent with everything else

Superseded by #200

Thank you though!


Can (ev/with-params {...} func) accept functions as parameters?

ev/with-params seems unable to process parameters that are actually functions, rather than just fixed values.
I appreciate that this is due to the serialization that clerk has to perform, but is there a workaround?
One example where this becomes relevant when you need to plot a number of points with an emmy-viewer in a way that is determined by a function that might only be specified at run time.

Can you show what you're trying to do?

I am using mafs trying to plot a slope-field as a background to a future ODE solver demo.
The functions that will determine the gradient at each point will be supplied by the user, so they are not known beforehand.
I tried to pass them using ev/with-params but this was not accepted.
(def fx1 (fn [x y] 1))
(def fx2 (fn [x y] (* 2 x)))

^{::clerk/visibility {:code :hide :result :show}}
 (ev/with-let [!fns [fx1 fx2]]
             (mafs/mafs
               {:zoom {:min 1 :max 5}}
               (mafs/cartesian {:subdivisions 5})
               (mafs/vector-field
                 {:step 0.2
                  :xy   (ev/with-params {:atom !fns :params [0 1]}
                                        (fn [af1 af2]
                                          (fn [[x y]]
                                            (let [x1 (af1 x y) x2 (af2 x y)
                                            ;     (let [x1 1 x2 (* 2 x)
                                                  h (sqrt (+ (square x1) (square x2)))]
                                              [(/ x1 h)  (/ x2 h)]
                                              ))))
                  :xy-opacity (constantly 0.4)})))

2024-05-26T22:56:41.197Z alex-HP-2570p INFO [emmy.expression.compile:661] - compiled function in 796.963 μs
Clerk evaluated 'src/demo.clj' in 257.761635ms.
Execution error (ExceptionInfo) at taoensso.nippy/throw-unfreezable (nippy.clj:1005).
Unfreezable type: class demo$fx1

OK, I worked it out, I was overcomplicating it my mind
I didn't have to use 'ev/with-params', as there are no params, only the x and y, which are supplied by the system through the :xy key.
So the function can be defined outside the mafs/vector-field call!
This is all I had to do in the end
(defn fx [[x y]]
  (let [x1 1  x2 (* 2 x)
        h (sqrt (+ (square x1) (square x2)))]
    [(/ x1 h) (/ x2 h)]))

(mafs/mafs
  {:zoom {:min 1 :max 5}}
  (mafs/cartesian {:subdivisions 5})
  (mafs/vector-field
    {:step 0.2
     :xy   fx
     :xy-opacity (constantly 0.4)
     }))

Issue can be closed now.


add alternate detection methods

I have created this PR to allow this component to function with multiple prerendering libraries. The ones I have tested so far are below:

Library
Working
Notes

react-snap
✔
details

prerender-spa-plugin
not tested yet
details

presite
not implemented yet
awaiting response from library author

snapshotify
not implemented yet
awaiting response from library author

prerenderer
not implemented yet
awaiting response from library author

prep
not implemented yet
awaiting response from library author

I will update this PR as I hear back from the library authors. I will also happily add detection methods for other libraries as requested if I can find out how to detect that the app is being prerendered.

I prefer exposing a prop like shouldRender over directly support every possible use case in this library.

Ah ok. By this do you mean a shouldRender prop that you pass whatever expression that your prerendering library uses to signify that it is being used? If so, I can definitely see why, as I am also usually a fan of a general solution instead of numerous hard-coded solutions.
This being said, I still have a few concerns with just using a shouldRender prop:

Wouldn't this result in a bunch of boilerplate code? Having to add the exact same prop to each instance of the component throughout a project seems rather repetitive. A single prop may not be much on its own, but it's still something extra that you have to remember and add each time you use it.
Since this currently adds very little complexity, wouldn't it make sense to choose the "zero config" option? If it was more complex to hard-code a few libraries by default I would definitely agree, but at the moment it is not.

Perhaps including a few hard-coded options for some of the more popular libraries while also including a shouldRender prop would be a solution that combines the best of both worlds?


Capabilities Deprecated MSC API Cleanup

This PR migrates all capability providing services to non deprecated MSC API.
https://issues.jboss.org/browse/WFCORE-4477
https://issues.jboss.org/browse/WFCORE-4478

Core - Full Integration Build 8640 outcome was FAILURE using a merge of 9a29892b552f8ecfab1294c247696489717e994f
Summary: Tests passed: 294, ignored: 22; compilation error: Compiler (new) Build time: 00:04:44

Who is volunteering @jmesnil @darranl @bstansberry to review this PR ?

All fixed @bstansberry

@bstansberry due to your LGTM comment @ropalka has applied a couple more fixes so I will include this in today's merge.


Cannot build gobot

After go get ... or go build i got this messages:
# github.com/gabeguz/gobot/plugins/dice
../../plugins/dice/dice.go:76:6: p.Attachments undefined (type "github.com/nlopes/slack".PostMessageParameters has no field or method Attachments)
../../plugins/dice/dice.go:86:33: cannot use "" (type string) as type "github.com/nlopes/slack".MsgOption in argument to c.Client.PostMessage
../../plugins/dice/dice.go:86:33: cannot use p (type "github.com/nlopes/slack".PostMessageParameters) as type "github.com/nlopes/slack".MsgOption in argument to c.Client.PostMessage
# github.com/gabeguz/gobot/plugins/jira
../../plugins/jira/jira.go:44:6: p.Attachments undefined (type "github.com/nlopes/slack".PostMessageParameters has no field or method Attachments)
../../plugins/jira/jira.go:55:7: p.Attachments undefined (type "github.com/nlopes/slack".PostMessageParameters has no field or method Attachments)
../../plugins/jira/jira.go:58:31: cannot use "" (type string) as type "github.com/nlopes/slack".MsgOption in argument to c.Client.PostMessage
../../plugins/jira/jira.go:58:31: cannot use p (type "github.com/nlopes/slack".PostMessageParameters) as type "github.com/nlopes/slack".MsgOption in argument to c.Client.PostMessage

Hi, looks like the slack library I'm using has changed its API.  I'll take a look this week.


Wallet Page Show Links To Files - Show more of claim name

PR Checklist

Please check all that apply to this PR using "x":

[x ] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x ] Bugfix
[ x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: https://github.com/lbryio/lbry-desktop/issues/2701
What is the current behavior?
In wallet page, the recent transactions have the same width in all columns
What is the new behavior?
Now the column details is more longer than others
Other information
I fixed a bug generated by className that receive a Boolean value

Yes, I will do today
Obtener Outlook para Androidhttps://aka.ms/ghei36

From: Thomas Zarebczan<EMAIL_ADDRESS>Sent: Tuesday, August 27, 2019 11:31:13 AM
To: lbryio/lbry-desktop<EMAIL_ADDRESS>Cc: nestordominguez<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [lbryio/lbry-desktop] Wallet Page Show Links To Files - Show more of claim name (#2787)
@nestordominguezhttps://github.com/nestordominguez - Thanks for the PR! Can you take another shot based on the feedback above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lbryio/lbry-desktop/pull/2787?email_source=notifications&email_token=ABAIW6QNHMZDFFTPIQCBBADQGU3DDA5CNFSM4IPEKWW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5H6AKI#issuecomment-525328425, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABAIW6REPP2EZZIFZT4B7ZLQGU3DDANCNFSM4IPEKWWQ.

@nestordominguez a friendly poke. I know you had trouble running the app - this should be fixed now. If not, try clearing out your dist folder.

Thanks a lot, this solve my problem, I will continue fixing this issue after a break, I was working all day.

De: Thomas Zarebczan<EMAIL_ADDRESS>Enviado: miércoles, 4 de septiembre de 2019 19:05
Para: lbryio/lbry-desktop<EMAIL_ADDRESS>Cc: nestordominguez<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Asunto: Re: [lbryio/lbry-desktop] Wallet Page Show Links To Files - Show more of claim name (#2787)
@nestordominguezhttps://github.com/nestordominguez a friendly poke. I know you had trouble running the app - this should be fixed now. If not, try clearing out your dist folder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/lbryio/lbry-desktop/pull/2787?email_source=notifications&email_token=ABAIW6UILV3YDRYBMI627C3QIAWJXA5CNFSM4IPEKWW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55E2SQ#issuecomment-528108874, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABAIW6RONW757BDLG3BGBHTQIAWJXANCNFSM4IPEKWWQ.

Hi, I did not find other issue related to boolean value in className on the whole page, I have searched in Transaction history too and nothing, if you see this issue in this page then you must press f5 the error is in the previous page

Closing this for now. Feel free to re-open if you have a new solution!


Layers drawn differently?

Is there a way to have certain layers drawn above the player sprite?

Not sure. Take a look at the code in the main TMX library. You might be able to draw layers after drawing the sprite, but I'm not sure since I didn't have to do that for this example.


merge package template from arlac77/npm-package-template

.travis.yml

chore(travis): merge from template .travis.yml

Coverage remained the same at 97.143% when pulling 867d4d17504eaf7096fab042b92d8490a01f2eba on template-sync-1 into f03a8d836245432d2c115b38faf61c22ed685681 on master.

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          29     29
=====================================
  Hits           29     29

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 f03a8d8...867d4d1. Read the comment docs.


[BUG] empty dict in array list will not be parsed

a1 = {"a": [{"name": "bob", "kwargs": {}}]}
print(toml.dumps(a1))

What I got
[[a]]
name = "bob"

I am expecting to have
[[a]]
name = "bob"
[[a.kwargs]]

Latest toml and python=3.12
Not sure if this is the intended behavior

I don't think this one can be put down to toml's strict TOML 0.5.0 support toml-0.5.0.zip
This isn't the only issue with toml's code for array tables  - it needs work.  See #439
In the mean time, other toml writers are available.
Reproduced:
>>> import tomli_w, tomlkit, toml_tools, toml
>>> a1 = {"a": [{"name": "bob", "kwargs": {}}]}
>>> for module in (tomli_w, tomlkit, toml_tools, toml):
...   print(module.__name__); print(module.dumps(a1))
...
tomli_w
a = [
    { name = "bob", kwargs = {} },
]

tomlkit
[[a]]
name = "bob"

[a.kwargs]

toml_tools
a = [
    { name = "bob", kwargs = {} },
]

toml
[[a]]
name = "bob"

Having said that, toml will corretly parse a sub array table, and give your intended output:
>>> a2 = {"a": [{"name": "bob", "kwargs": [{}]}]}
>>> for module in (tomli_w, tomlkit, toml_tools, toml):
...   print(module.__name__); print(module.dumps(a2))
...
tomli_w
[[a]]
name = "bob"
kwargs = [
    {},
]

tomlkit
[[a]]
name = "bob"

[[a.kwargs]]

toml_tools
[[a]]
name = "bob"
kwargs = [
    {},
]

toml
[[a]]
name = "bob"
[[a.kwargs]]


Some rules are applied to wrong type of request

@AdamWr commented on Wed Feb 06 2019

Steps to reproduce

Enable AdGuard Base filter
Go to https://www.wp.pl/

Expected behavior

Rules with ~script modifier shouldn't be applied to script requests, for example this rule:
||wp.pl^$replace=/(<VAST[\s\S]*?>)[\s\S]*<\/VAST>/\$1<\/VAST>/,xmlhttprequest,other,~script
Actual behavior

It's applied to script request.
Screenshot:

Your environment

Adguard version: 3.0.229 (1.3.126cl)
Adguard filtering mode: VPN

core/pull-requests/1032


if found a bug for flask 1.0.2

in  frontend/src/components/Home.vue   http://localhost:5000/api/random   and run.py  @app.route('/api/random') the route looks like can not run report a bug :
Access to XMLHttpRequest at 'http://localhost:5000/api/random' from origin 'http://localhost:8080' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
i modify it   http://localhost:5000/api/random/ and  @app.route('/api/random/')  this be ok

Indeed, it is a bit confusing in Flask that you have to follow this slash strictly.
I recommend just adding a trailing slash to the route definition and use it in your AJAX request.
(Another solution is probably disable CORS for whitelisted domains in Flask, but this can be tricky/impossible sometimes.)
Thanks!


fix: make email validation regex match full text

@zovt Looks good, but could we add some test coverage? Like testing that a user with email bad<EMAIL_ADDRESS>is not considered valid.

Looks like you've found your first live-data test failure 😅
Side note, I like to use the conventional commit prefix test: for commits that exclusively change/add/remove tests; here's one example. Other good prefixes to use are ci: and docs: for commits that only change CI workflows and documentation, respectively, and refactor: for commits that change code but don't change any app behavior. I find these improve the "skimmability" of the Git history, personally, but it's not at all essential.


Pre-commit hook doesn't recognize internal packages

Pre-commit hook doesn't recognize internal packages. When I lint project using ruff check inside superproject directory everything works fine. But when I run pre-commit, imports from external and internal packages (eg. flask and superpackage) are not separated. This happens because pre-commit doesn't recognize internal and external packages. I tried to cd superproject && ruff check, but this doesn't seem to work. Other solutions with bash -c 'cd ...' seems way too hacky. I also don't want to store inside pyproject.toml list of all internal packages. Is there any elegant solution?
Project structure:
.
└── superproject
    ├── main.py
    ├── pyproject.toml
    └── superpackage
        ├── __init__.py
        ├── __pycache__
        └── test.py

main.py
from flask import json

from superpackage.test import hello_world

# this doesn't matter
print(hello_world())
print(json.dumps({"hello": "world"}))

pyproject.toml
[tool.ruff.lint]
extend-select = ["I"]

[tool.ruff.lint.isort]
combine-as-imports = true

Python 3.12.3
Ruff 0.7.3

Can you specify where your .pre-commit-config.yaml file lives? Is it in a directory above superproject or is it inside superproject?

Can you specify where your .pre-commit-config.yaml file lives? Is it in a directory above superproject or is it inside superproject?

Above superproject

running into the same issue, where the pre-commit yaml also lives above the root of our logic.

@skullyhoofd if you want an obscure solution:
repos:
  - repo: https://github.com/astral-sh/ruff-pre-commit
    rev: v0.7.3
    hooks:
      - id: ruff
        name: Python Ruff linter for superproject
        entry: bash -c 'cd superproject && ruff check . --config pyproject.toml --fix'
        language_version: python3.12
        types: [python]
        files: "superproject"
      - id: ruff-format
        name: Python Ruff formatter for superproject
        entry: bash -c 'cd superproject && ruff format . --config pyproject.toml'
        language_version: python3.12
        types: [python]
        files: "superproject"

That's the best solution I'm aware of. Hate this, but it works

So, just to confirm that this is your project structure?
.
├── .git
|   └── ...
├── .pre-commit-config.yaml
└── superproject
    ├── main.py
    ├── pyproject.toml
    └── superpackage
        ├── __init__.py
        └── test.py

Can you specify how you're running pre-commit? Which directory are you running it from?
For me, it doesn't seem to be complaining:
/tmp/ruff-pre-commit-test
$ pre-commit run --all-files
ruff.....................................................................Passed

/tmp/ruff-pre-commit-test
$ cd superproject

/tmp/ruff-pre-commit-test/superproject
$ pre-commit run --all-files
ruff.....................................................................Passed

@dhruvmanila
NOTE: pre-commit-config in the comment above is my current workaround
Your project structure is correct. To be more precise:
test.py
def hello_world():
    print("Hello, world!")

.pre-commit-config.yaml
repos:
  - repo: https://github.com/astral-sh/ruff-pre-commit
    rev: v0.7.3
    hooks:
      - id: ruff
        name: Python Ruff linter for superproject
        args: ["--config", "superproject/pyproject.toml", "--fix"]
        language_version: python3.12
        types: [python]
        files: "superproject"
      - id: ruff-format
        name: Python Ruff formatter for superproject
        args: ["--config", "superproject/pyproject.toml"]
        language_version: python3.12
        types: [python]
        files: "superproject"

❯ pre-commit run --all-files
Python Ruff linter for superproject......................................Failed
- hook id: ruff
- files were modified by this hook

Found 1 error (1 fixed, 0 remaining).

Python Ruff formatter for superproject...................................Passed

This is how ruff formatted main.py
from flask import json
from superpackage.test import hello_world

# this doesn't matter
print(hello_world())
print(json.dumps({"hello": "world"}))

I want this file to be formatted as in my issue description. Ruff doesn't recognize here external (flask) and internal (superpackage) libraries.


Setup clap to generate man pages

clap cli parser supports generating man pages from the long form documentation. We should use this feature so the docs can all be maintained in one place and simply generated in different forms.

Ok, so I setup the initial generation following the docs for clap_manpage and it seems to only generate the man page for the top level command. It doesn't generate the man pages for the subcommands.
I dug into issues and stuff around it and I found https://github.com/clap-rs/clap/discussions/3603 which doesn't have a great answer as all the attempts have a number of problems.

Ok, @Alizter sent some some starter code to try and get it working for the subcommands. It needed some love to get it building but it got me going enough to get something working in my explorations. So task is back on.

This issue has been resolved and integrated into main. Therefore, I am closing this issue.


warp 0.13.0 License clarification

Hello warp authors, thanks a lot for working on warp and changing license to allow commercial usage (see https://github.com/NVIDIA/warp/releases/tag/v0.13.0).
Together with @flferretti we are mantaing the conda-forge package of warp, and we wanted to understand how to properly document this new license, in particular using SPDX
I see that the new license is reference as "NVIDIA Software License" (see for example https://github.com/NVIDIA/warp/blob/9b2a57f7268e8523c24eec1625c6d08548e5c60c/pyproject.toml#L12). However, I am not finding any reference to this "NVIDIA Software License". Is that a "generic"  license used by NVIDIA, or it is something warp-specific? If it is warp specific, could it make sense to call it with a warp-specific name, to avoid ambiguity wit the software license of other NVIDIA libraries?
Thanks in advance for the answer!

Hi @traversaro, looking at our internal guidance, I think SPDX-License-Identifier: LicenseRef-NvidiaProprietary is the appropriate identifier to use here.

Thanks a lot @shi-eric ! I just have a comment on this.

Hi @traversaro, looking at our internal guidance, I think SPDX-License-Identifier: LicenseRef-NvidiaProprietary is the appropriate identifier to use here.

Based on https://spdx.github.io/spdx-spec/v2.3/other-licensing-information-detected/, ""LicenseRef-"[idstring] where [idstring] is a unique string containing letters, numbers, . and/or -." it seems that the license used by warp is quite warp-specific, and Nvidia employies for its source available software many prorietary licenses, that are all different. Using LicenseRef-NvidiaProprietary seems a bit risky to me, as it seems to be highly probably that people will use it to refer to other (but different) licenses used for proprietary Nvidia software. What do you think on this? Could it make sense to use a more warp-specific idstring?

Hi @traversaro, I just double-checked with Legal after your last question. I'm told that LicenseRef-NvidiaProprietary is used as a "catch-all" and is indeed the correct one to use for Warp.

Thanks @shi-eric! Personally I am not sure the use of a "catch-all" code for referring to different license is compatible with Clause 10 of SPDX v2.30 spec, but I guess the Nvidia position is now clear, thanks for the clarification!


Unable to update files dynamically from javascript

Out of scope for this package you need to implement a filesystem Provider there are endless possible.


[23.11] pandoc: apply patch removing the usage of polyfill.io in the templates

Description of changes
If you output HTML with MathJax content Pandoc might uses a JS library provided by cdn.polyfill.io which is now considered to be a bad actor.
https://sansec.io/research/polyfill-supply-chain-attack
haskellPackages.pandoc is not impacted, the concerned domain is not used
To reproduce the issue:

Create a file math.tex with the following content $a^2 + b^2 = c^2$
Call pandoc with pandoc math.tex -s --mathjax -o ex.html
Look at the injected scripts in ex.html

This is the 23.11 version of #322743

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Looks like I failed the rebase... Anyway, as NixOS 23.11 reaches end-of-life today, this PR is closed as part of the EOL cleanup.


Expects macro clashes with MS's GSL

How could I go about using both libraries? See https://github.com/microsoft/GSL/blob/master/include/gsl/gsl_assert#L126 and https://github.com/mpusz/units/blob/master/src/include/units/bits/external/hacks.h#L35-L38.

Another difference is that unit's Expects is a statement (with semicolon included) whereas GSL's is an expression.

Yes, it might be an issue. Dou you have any suggestions? I could consider switching it to direct GSL usage but still, there are some competing solutions (i.e. https://github.com/gsl-lite/gsl-lite).

gsl-lite uses gsl_Expects, so you could either use MS's GSL or rename Expects.

Thanks!


Allow Trash to copy properties from other properties

Trash should be able to copy properties from other properties without
causing a problem. Trash should also, when doing this, not set any
properties that it doesn't define.
Note: This also includes a refactor of Dash::PropertyTranslation to make it more understandable.
Fixes #456

Bugger, good call. I'm going to have to make a few other changes for that. Dash currently isn't aggressive about creating getters when a setter is already set.

I pushed another commit that makes sure they are not order-dependent. What do you think about the change?

I think it's brilliant. Merged.

Cool, I think I will roll a release today. We're up to five bug fixes and three miscellaneous, which seems like a good point to cut a release.


Add support for executable files without a .sh extension

As can be seen in the logs for 1473a6b (see build logs linked below), the current script doesn’t find the shell-script without the .sh extension. As can be seen in the following log after I added the file tests/test-custom-no-extension and set its file mode to +x:
Linting all executables and .sh files, ignoring files inside git modules...
[ OK ] Linting ./tests/ksh.sh...
[ OK ] Linting ./tests/sh.sh...
[ OK ] Linting ./tests/bash.sh...
[ OK ] Linting ./tests/test-custom-check.sh...
[ OK ] Linting ./build.sh...
[ OK ] Linting ./install.sh...

This is the same for both for the Linux and OS X builds:

Linux build log
OS X build log

This PR resolves this issue by reverting PR #20’s change to the find command arguments and to use +111 instead of +x for the -perm argument. Further it resolves compatibility with modern GNU find versions (4.5.12+, released in 2013) as it expects the value to be prepended by / instead.
I’ve also added a very rudimentary test that makes sure the built find_cmd fins the correct number of files.
@caarlos0 let me know if this looks good to you, or if there’s something you’d like me to modify.

@rastersize thanks for the PR, I have only one single comment..

@caarlos0 I’ve updated the PR with a commit that hopefully resolves your comment. Let me know what you think.

Simplified it as suggested, a lot better now :smile:
Still piping the output though, so that it doesn’t get printed in the log.

Nice, thanks!


Move the TransportCompliance to a new "WolverineFx.ComplianceTests" library

This is to get ready for adding more tech to Wolverine

Done


Add config option for focusOnInit

Suggested config / functionality to limit focus on initial editor rendering.

@andrewsprinz got any thoughts on this and if you still use whether it would work for you?


Fix submenu bug (not register hide event)

see https://github.com/vkbansal/react-contextmenu/issues/143

Can you explain what your change does exactly? How does adding a listener fix the reported issue?

When click on menu item in sub menu, MENU_HIDE event is fired. But the SubMenu component haven't register hideMenu() to the event. So the visible state is still true, witch will cause user can click multi times
hideMenu = () => {
    if (this.props.forceOpen) {
        this.props.forceClose();
    }
    this.setState({ visible: false, selectedItem: null });
}

Oh got it. Thank for the fix!


Fix mocking aws service

Since moto 5.x service specific mock decorator is removed. To fix this use mock_aws instead.

Thanks for the fix! will release a new version shortly