Invitation to peer review: Razzle with loadable-components

I have given Razzle + loadable-components another try and I think this time I got pretty far. I have it good enough to continue with my work, but I have a feeling that my solution still is somewhat of a hack and I would like to invite others with more expertise to tell me what is the missing piece of the puzzle ;)
I have published a sample repo here to allow you to quickly set this up on your machine:
https://github.com/mbrochh/razzle-loadable-components
You can set it up like so:
git clone git@github.com:mbrochh/razzle-loadable-components.git my-app
cd my-app
yarn
mkdir build
yarn start
# go to localhost

At this point, you can click around and everything should work. Note the "render Home" and "render About" output. I added those to show that React has actually loaded on the client-side and I'm not just seeing the server rendered page.
Now remove the asd in Home.js and save the file. You will see that the page goes blank and the console throws a warning:
index.js:1 Uncaught Error: Iframe has not been created yet.

It's annoying but I'm OK with this, I don't mind refreshing the page after making changes in my code (after all, I want to see if the server rendered page still looks good as well).

So what is the problem? Have a look at what I did in server.js.
You can switch to the branch master_broken and now try to change something in Home.js - when you refresh the browser, your change does show up, but React does not load on the client side any more and you just seeing the server rendered page until you restart yarn start.
In the past, when I attempted this, I realised that client and server get out of sync when I change stuff in my files and I always have to restart yarn start to be able to continue to work, which is extremely annoying when compilation takes 40 seconds.
I think I now understand why this happens:

we have two dev-servers running, one for the client, and one for the server
when we change a file, both dev-servers re-compile
however, we can't control the order (and speed) in which they re-compile so it can happen that the server re-compiles faster than the client (that would be expected because the server doesn't need to create chunks)
this means, when the server re-compiles, it will sometimes read the outdated loadable-stats.json file because the client is still compiling and has not created a new file, yet
now client and server are out of sync and all kinds of crazy annoying stuff happens during development, you must run yarn start again to get them back into sync

So as my ugly hacky workaround, I am now doing this:

if we are working in local development mode (NODE_ENV === 'development'), we will simply read the stats file on every request. I assume that in a production environment, this would be overkill and put too much stress on the server (maybe I'm wrong and just worrying too much?)
So in a production environment, I will just load the file once when the server starts, after all, I'm never changing the file on my PROD server unless I make a deployment and when I do make a deployment, I will restart the server anyways and it will then load the latest file

My questions are:

Is this the best way to do it?
Is there some better trick to make sure that the server only ever compiles AFTER the client has finished compiling?
Would it be OK even in a PROD environment to read the stats-file on every request?
How can I get rid of this Uncaught Error: Iframe has not been created yet. error?

Any and all input is hugely appreciated! Many thanks in advance!

I have no problem with combining these two. The code is in some lines of code.
server.ts
const extractor = new ChunkExtractor({
      statsFile: path.resolve(__dirname, 'loadable-stats.json'),
      entrypoints: ['client'],
    });

    const markup = await ReactDOM.renderToStringAsync(
        <ChunkExtractorManager extractor={extractor}>
              <App />
        </ChunkExtractorManager>
    );
const html = `

${extractor.getLinkTags()}
${extractor.getStyleTags()}

${markup}

${extractor.getScriptTags()}

@revskill10 so that code in your case gets executed on every single request, correct?
otherwise I don't see how this could work, because as far as I understand it, the stats file will not be guaranteed to be updated by the time the server re-compiles after you change the sourcecode.

@mbrochh Yes, the whole code is inside a handler (req, res). I'm using the above code and it works normally.

@revskill10 that makes sense! As I mentioned in my initial post, I am a little bit concerned of the CPU overhead to read and parse the stats file on every single request. That file is 154k in our app. Surely, parsing that file on every single request will add a few ms to the request-response time, right?


Issues of getArmResources

I'm looking at getArmResources. There are several problems:
1. Incorrect assumption of "a model can only belong to one resource"
From the implementation of @armResourceRead and others, we could see that the model passed in to the template of ArmResourceRead<Resource> will be taken as a resource model, and all the operations using the model this template way will be taken as the corresponding operations to this resource model. In real services, we have many cases that a model can belong to several resources. For example, this client and this client share the same resource model Site.
Actual: the key of resource is model.
Expected: I think the key of resource should be request path, or at least resource type.
2. A resource can only have one operation for each lifecycle operation
For example,
model Parameter {
  @path
  @segment("slots")
  slot: string;
}

@armResourceOperations
interface WebApps {
  get is ArmResourceRead<Site>;
  getSlot is ArmResourceRead<Site, Parameters = Parameter>;
}

The second getSlot wins. But I assume if we fix the first problem, this is not a problem any more.
3. Whether an arbitrary model can be the resource model of a resource?
For the current implementation, as long as the model is used in a template, it could be the resource model of a resource. However, in .net generator implementation, there needs some conditions to make a model a resource model. For example, the request path should have even number of segments after the providers segment, there should be a get operation, etc. I believe it follows some standard. This leads to the below scenario:
interface PrivateLinks {
  listByMongoCluster is ArmResourceListByParent<PrivateLinkResource>;
}

For this case, only list exists and that's it.
Actual: the output contains one resource with PrivateLinkResource as resource model, and a list operation belongs to it.
Expected: we should come to an agreement whether there should be a resource.
4. Resource Type
This is how we calculate resource type. Is that correct? I think it should be Microsoft.Contoso/Databases for the case in comment.

4. Resource Type This is how we calculate resource type. Is that correct? I think it should be Microsoft.Contoso/Databases for the case in comment.

We use the below logic to calculate resource type when building Swagger KPI and I believe this matches the ARM definition of resource type.
https://devdiv.visualstudio.com/DevDiv/_git/openapi-swaggerkpi?path=/azure-functions/DurableScanSwagger/scanner.ts&version=GBmain&line=53&lineEnd=54&lineStartColumn=1&lineEndColumn=1&lineStyle=plain&_a=contents

@m-nash

it looks like 1 and 2 are the same thing? Do you have a concrete example of when we would want multiple life cycle operations of the same type for the same resource?

Yes, I pointed out as long as we correctly handled 1, 2 is on longer a problem. And as long as TypeSpec correctly define resource, I agree only one operation is of same type.

I assume by life cycle operation you mean CRUD only since you can have multiple POST operations?

Correct. TypeSpec uses this terminology, so I use it too.

can you run some unit tests to see if we get the right resource type out given different urls? It does appear there might be an issue but its hard to say for sure exactly which scenario comes out incorrect.

I noticed this problem right because it gives me wrong output. For example. below case gives me "/subscriptions/{subscriptionId}/providers/MgmtTypeSpec/privateLinkResources", which I think should be MgmtTypeSpec/privateLinkResources.
interface PrivateLinks {
  listByMongoCluster is ArmResourceListByParent<PrivateLinkResource>;
  get is ArmResourceRead<PrivateLinkResource>;
}

(1) Here Resource means ArmResource.  It is not possible, AFAIK for an ARM resource to be registered at different paths: different registrations == different resources, and the way resources are defined is in the resource model itself.  There may be common schema between resources at different paths, which is different than these being the same resources.  I think the issue here is how to represent those cases where two resources share the same envelope and rp-specific properties.
(2) I think this goes back to point (1), as Michael indicated.  You should not use ArmResourceRead for non-resource models.  Either these are two resources that share a schema, or one is a resource and one is not.
(3)  We should use the arm definition:  anything registered as a resource is a resource.  Anything not registsred as a resource is not.  It is possible to have non-resource operations outside the RPC, for example, 'get' on a non-resource path, and provider actions.  Currently, these would be outside.
(4)  No, this is not the same as resource Type.  We likely need to include resource type in what is here (it would just concatenate the segment values from the parent resource chain and prepend the provider name)


Added Gratipay team to fakedata so pricing page will work

Was updating my local repo and realized that http://localhost:8537/about/pricing was not working because it expected a Gratipay team to exist in fake_data.
Only needed minor changes.

Other than those two nitpicks, this looks good to me

Looks good to me. Rebasing ...

Previous head was 4aa23ba832787fdb95a7748cc5ad5e8474a6556c.

Will merge when Travis is green ...

!m @kaguillera
On to the next story! :-)


Added "ː"-suffixed chars to list of valid IPA

Fixed issues outlined in issue #272
list_phones.py was incorrectly rejecting IPA symbols such as "t̼ː" because ipapy doesn't recognize "ː" as a valid suffix for some IPA characters for whatever reason. I added all "ː"-suffixed IPA symbols to our supplementary list of IPA symbols not covered by ipapy to fix this.

[ ] Updated Unreleased in CHANGELOG.md to reflect the changes in code or data.

Hi again, could you run black -l79 on list_phones.py? That should make the remaining test pass.

Sure thing, fixed


Merging master into Intervals branch

Using this as a reference to comment on things before attempting conflict resolution

Search for all moment.add/subtract to make sure no argument transitions are lost. The number should be the first argument, so search for .add(' and replace.


[OTHER] Update Discord links

What would you like to share?
We're using a new link for The Algorithms' Discord invitation.
All Discord invitation links should point to https://the-algorithms.com/discord/.
Additional information
This is a good first issue. Please leave it open to first-time contributors and beginners. 🙂

can i work on this??

can i work on this??

I have checked all the discord link is already pointing to this link only please can you clear me what is the issue??

https://github.com/TheAlgorithms/C-Plus-Plus/search?q=c7MnfGFGa6
There are still a few instances that need to be updated.

Hello, as a first timer I would to start my contribution to open source from this issue , so can you please assign me this?

hi,I'm working on this issue!!


Complete the README.md

Complete the How To Get Started

[ ] With pre-builded UMVC.Core DLLS
[ ] Integrated UMVC.Core project with MSBuildForUnity

FAQ

[ ] How to enable / disable Automatic Model Events

Need to be done first : #21 #22


do not drop post-aggs in TimeseriesQueryToolChest.makePreComputeManipulatorFn

new behavior is consistent with TopN and GroupBy.
That said, I am changing it for following reason.
We are working on a custom query type to fulfill a specific internal use case and use DirectDruidClient which was dropping all the post-aggregations returned for Timeseries query because it passes the result through toolChest.makePreComputeManipulatorFn(..) for deserialization and removing post-aggs for timeseries as a side effect.

:+1:

:+1:


Move "run app" logic to shared module.

Currently, the "run" logic is available in the GUI application only, but it might be a good idea to move it out to a module shared between all applications. It might even make sense to move it to the Updater class, to allow other libraries to make use of its functionality.
Related to #31.

Note that just executing a binary produces different results (at least in GNOME) compared to launching an application via its desktop file or via the menu:

For applications that ship an icon but do not actively set the icon on their main window, the application icon will only show up in the dock if the application was launched via its desktop file or via the menu
Also, the context menu items in the dock are only available if the application was launched via its desktop file or via the menu

I had written some code that emulates the "proper" way of launching something through its desktop file some time back, https://github.com/AppImage/AppImageKit/issues/174#issuecomment-252491277. This is overkill for appimageupdatetool but we may want to consider this for AppImageUpdate.
Low priority though.

Hm, the problem is that code links to glib, which I could successfully avoid as a dependency so far. First of all, the "run app" code has to be refactored out, however. If we add this behavior, it should probably not be part of libappimageupdate but the standalone applications.

That is true. Ideally this would be invoked by runtime when running an AppImage. And indeed one would need to look deeper than this to understand how to re-create this without additional dependencies such as glib. So indeed probably out of scope here.

@probonopd well we could always make a separate executable with that code.


Add description of custom fields

It would be really nice if authors would be obligated to provide descriptions of all custom fields that could be then displayed in cxg during hover-over (as is now done for full name of e.g. obs columns). This could be stored in uns in format
{
obs : {
  col1 : description,
  ...
  },
var : {
  col1 : description,
  ...
  },
obsm : {
  embedding1 : description,
  ...
  }
}

@Hrovatin - Thank you very much for your proposal. During our issue triage today, there was agreement that while this would be a nice feature for data consumers, the additional MUST requirement would be onerous for data submitters and curators. Please keep sharing ideas!


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-11-19T17:44:27+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.24
c3521338885acdb8ae843bd72268d79fc8c31300
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


The host machine and virtualdsm cannot access each other.

how to solve this problem. All the devices under the LAN can access the virtualdsm, but the host machine cannot access it.

Thank you. I've understand this method, but complex for me. So I use OpenVPN as a bridge to connect these devices.


Spanish translation

Hi, so there's my Spanish translation, I hope you like it. I expect to do another translation to another language.

Thanks! Looks great. Will merge this soon

Thanks! Looks great. Will merge this soon
What other language? Just curious :)

It'll be Catalan.

Thanks! Looks great. Will merge this soon
What other language? Just curious :)

It'll be Catalan.

Actually, it'd be nice if you could wait till tomorrow to merge the pull request, tomorrow I'll probably send the Catalan translation, so you can merge it all in one.

Yes I will wait, take your time. Thank you for this
I just released a feature where you can translate the ingredients as well. It is optional but appreciated.
There is info about this in the README.md

@YottaMxt Let me know when I can merge this 🚀

@YottaMxt Let me know when I can merge this 🚀

Only the Catalan translation of ingredients is incomplete.


CLB LUT Fault Modeling

It would be nice to model failures in the LUTs of a CLB. While the current approach clearly indicates that a LUT bit has been upset, it would be nice to see how the overall LUT changed. A simple approach would be to print the complete LUT contents before and after the LUT upset. This could later be expanded to evaluate how it impacts the logic function based on the number of inputs (for example, if one of the LUT inputs is hard coded to a value it is possible that the LUT upset is a "don't care"). Some logic evaluation could help with this understanding.

Added detection for LUT initialization string modifications in commit d58aa0ae1e7150b2842860ff915e3758c84ddfe2 . I still need to add conversion to the underlying logic function.


rpi firmware copy

This copies the u-boot.bin firmware file into /bin when preparing the image.
Raspberry Pi's need that file to be able to boot, copying it through an RPM is cumbersome see #3312.
I've created a new method on the platform that can be used in the image-pipelines to place files where they need to be. This is the right place to do it, though it could also be moved to a separate pipeline.
This PR allows Fedora IoT to build bootable (on raspberry pi) images in Koji.

Ok, this got a bit hairy as we can't do * copies. This is all the firmware packages from F37. For F38 I'll still need to add the bcm2711-rpi-cm4-io.dtb.

This copies the u-boot.bin firmware file into /bin when preparing the image.

Do you mean /boot/efi rather than /bin?

This copies the u-boot.bin firmware file into /bin when preparing the image.

Do you mean /boot/efi rather than /bin?

That's indeed what I mean!


Inconsistent behavior of components in component slots 🐛

Describe the bug 🐛
Simple components like
component Container:
  `template`:
    tSpan:
      slot

should behave essentially as tags like tSpan, but there are situations with iteration and components with Container in the template that do not.
To Reproduce 👨🔬

component Container:
  `template`:
    tSpan:
      slot

var grandCount: int = 0

proc nextCount(): int =
  grandCount += 1
  result = grandCount

component Counter:
  n: int = nextCount()
  `template`:
    "{self.n} )"

component withCounter:
  n: int = 9
  `template`:
    #component Container:
    tSpan:
      for i in 1..self.n.val:
        component Counter

appRoutes("ROOT"):
  "/":
    for i in 1..10:
      component withCounter

Expected behavior 🤔
We would expect the Container's slot to behave like contents of a tSpan. This modularity would allow repeated code to move into components at all levels of the codebase. Instead each parent conponent withCounter uses identical instances of the slot. That is, the tSpan counts from 1 to 90:

But if you comment out the tSpan and use component Container instead, it merely counts from 1 to 9 repeatedly:

Please complete the following information 📃
working in the SPA end
w/Chrome, Manjaro OS
HappyX 3.0.0  @#head

Works great now, thanks for the commit!


Improve tests

Q
A

Bug fix?
yes

New feature?
yes

BC breaks?
no

Deprecations?
no

License
MIT

What's in this PR?

Use assertSame where it had meaning.
Use a clone of the objects under testing. This way we ensure that the object has not changed state.
Added some more assertions.

To Do

[x] Need to check more places about using assertEqual with a clone.

I think that's it.. hope I haven't missed anything but if I did we can do more improvements later.

i am not sure i fully get what the getClone method is supposed to do

The abstract definition of getClone method is there in order for the trait to ensure that the method exists and be able to be used by the trait tests.. check this

i think it would be cleaner to explicitly clone the original request/response in 1-2 places

Yes I can clone  the message per test case... but the places I counted are 5 not 1-2 😄 .. I have no disagreement about this.. I will make the changes...

the getCone method name is also very generic

I was pretty sure about this comment.. I have the same feeling as you...  Since I will remove it and do the clone per test case then no need to think of a name.. if you change your mind and keep it I vote for getMessageOriginalState

okay, cool! when i said 1-2 places i was thinking if we need the clone in all places we currently use it. if there is a specific reason to test that in each of the places, i am all for it, but we should not test the same thing multiple times.


Go Web Scraping CLI

Made our scraping CLI fully usable for daily scraping and adding new sources.

Furthermore; please squash your commits so we have a more readable tree.
Things like the "merge conflict" commit can be easily squashed without losing information. "some minor continuations of go web scraping scripts" should also be squashable into the previous one.
And the 7 commits for the README are a bit... yeah 😂


validateFile: fileInfo.size is always 0

Hi,
Thanks for a great package, this is proving to be really useful! I just ran into a bit of an issue, however.
I'm trying to use the validateFile: function(fileInfo) and return true if fileInfo.size < maxSize -- however, size is always set to 0.
This is the object that gets passed to the callback:
{
domain: null,
_events: {},
_maxListeners: 10,
size: 0,
path: '/tmp/.uploads/8b7cdf076c7c218a464f813e0b7abf85',
name: 'filename.JPG',
type: 'image/jpeg',
hash: null,
lastModifiedDate: null,
_writeStream: null
}

Do you have any suggestion on where to start looking for a fix?
Thanks!

I found that I could get the file size via validateRequest: function(req) -- the size is in req.headers['content-length']. It's definitely not the same thing, of course, but it's a quick and dirty workaround.
However, as I just mentioned in #155, validateRequest does not work either: return false allows uploads anyway, and return true complains about TypeError: first argument must be a string or Buffer.
Using upload-server 1.3.2 on Meteor 1.2.1

Hi,
Do you have any update on this? It would be great if you could let me know if this is already fixed, and if the fixed version is released as a new package on atmosphere. Let me know if I can help with anything!
Thanks!

@rteslaru, sorry for a very late response. There was an error in the package which is now fixed. I have also updated the documentation that states that failed validation returns a description of validation error and passed validation returns null. Asa failsafe, I have added the possibility to return null with the default text of "validationFailed".
Concerning the file size. You were right! I amd calling validation BEFORE the file has uploaded to the server, thus server did not know about the file size. As a result I have moved the validation to state when file is uploaded to the server.
This way, you can use Validate request to cancel any upload that is not valid BEFORE it starts uploading. And you can use validateFile to validate the integrity of uploaded file AFTER it was uploaded on the server.


PowerToys Run is not working, doesn´t open

Microsoft PowerToys version
0.57.2
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
open powerToys, hit PowerToys Run shortcut, nothing happens, worked finer in version 0.56.2, it broke with update
✔ Expected Behavior
expect to open the Run search box
❌ Actual Behavior
nothing happens
Other Software
No response

/bugreport

What happens is that There is no way to have documented that, it just doesn´t make anything when hiting the shortcut

Didn't it create a zip file in your Desktop? /needinfo


Bump conda-lock to 2.5.1, add fiona and h5netcdf

Some dependency updates and additions

conda-lock 2.4.2 to 2.5.1, which brings alphabetical sorting
fiona, used to read vector files in sampling script at #29
h5netcdf, used to write NetCDF4 files in #44

Merging directly again, so that I can continue work on #37.


Connecting beaker to existing ipython kernel

I need to be able to connect beaker to existing ipython kernel.
I explored the alternatives, and adding the "startup" ipython code in the default beaker notebook can't solve my problem.
Normally, you can start ipython by connecting to existing kernel like described in this post: http://stackoverflow.com/questions/31382405/ipython-notebook-how-to-connect-to-existing-kernel.
Is there a way of making beaker connect to an existing ipython kernel?

I thought about programaticaly overriding the /home/rkoziko/.beaker/v1/config/beaker.pref.json to launch to newly launched kernel, but that sounds like a bad hack.

we don't have any support for this feature, like IPython the kernel is started from the client.
i am not opposed to it, but i am not aware of widespread demand either.
what would the UI look like?

Our current use case is launching ipython from a GUI based tool. Based on the state of the GUI tool, the ipython session will have some variables added to the global scope.
You can connect to the existing kernel using command like: ipython console --existing kernel-509.json.
Are ipython sessions launched by "shell"?  If yes, then I imagine that being able to pass command line flags to ipython from the GUI could solve more problems than just this.
For future reference, end to end command line workflow for daemon/client:
In shell 1:
#!/usr/bin/env python
import os
from IPython.zmq.ipkernel import IPKernelApp

app = IPKernelApp.instance()
app.initialize([])
app.start()

This will print to stdout something like:
To connect another client to this kernel, use:
    --existing kernel-32504.json

Then in shell 2:
ipython console --existing kernel-32504.json

so you want to add a cmd line option here:
https://github.com/twosigma/beaker-notebook/blob/master/plugin/ipythonPlugins/src/dist/python3/python3PluginStart.py#L21
?
is there any other way to get that config in?  could it be done with a magic?


Test smoothquant accuracy for just fc2 layer

I'm integrating smoothquant into a different large autoregressive transformer model with a somewhat different arch.
Say I want to test smoothquant accuracy just quantizing and smoothing fc2 to int8. Could you help me with the steps for this? I'm using the real PyTorch W8A8B8O8Linear layer from torch-int.
Roughly looks like:

Get act scales, static decoder layer scales for whole model (or just fc2, but might be easier to do whole model).
Use act scales + just the fc part of smooth_lm
Call the W8A8 linear kernel

I'd appreciate some help with step 3 (or just some example code for getting scales/quantizing/calling a single linear). In particular, I'm confused by what the input and output scale should be. Also looks like some of the scaling factors are fused into previous layers so not sure if that will introduce any issues. Thanks!

Ok trying to break this down into simpler questions:

For fc2, what is input_scale? Is it the per-tensor scaling factor that you calculated on the activations in get_static_decoder_layer_scales?
As a follow-up to 1, can't you calculate per-tensor activation scaling factor using the statistics you got in get_act_scales? get_act_scales gives you a vector of per-channel scaling factors which you could average to get per-tensor activation scaling. Why do you need a separate get_static_decoder_layer_scales? Not sure if I'm missing something here.
Similarly, what are alpha and beta? Is alpha the final per-tensor scaling factor for the weight and beta the final per-tensor scaling factor for the input activation?

Example code for a single matmul would be really helpful tbh - I'm still not quite understanding how the paper maps to the implementation. For the model I'm trying to use smoothquant on it's a bit difficult to copy and paste so have to re-implement some stuff.

Hi Eric, thank you for your interest in SmoothQuant!
To answer your questions in short:

Yes, it is.
We convert the model into INT8 by converting each linear layers into INT8. You can check the from_float functions of those linear layers, e.g., from_float of the W8A8BFP32OFP32Linear.

We use the W8A8BFP32OFP32Linear to implement FC2 and out_proj since their outputs are the inputs of the residual adds and LayerNorms. You can check the from_float function of the W8A8BFP32OFP32Linear (as well as other INT8 modules) to see how we convert a FP module into an INT8 module. The input_scale always refers to quantization step size $\Delta$, which is the maximum absolute value in activations divided by 127 (see get_static_decoder_layer_scales).

Regarding question 2, my question is where does the fp16 input get quantized into int8 first on the model? I'm looking for a call like "input.to(torch.int8)". Does it happen in the tokenizer or right at the start of OPTDecoder?

It happens in the LayerNormQ.

What about in the attention though? There's no layernorm before that.

There are LayerNorms before attention modules. Quantization always happens in LayerNormQ.

Ok, I see. That quantizes for layers after the first one - are we not quantizing the first MHA?


CUDA 12.0 ARM conda support.

This PR enables CUDA 12.0 ARM workflows for building and testing RAPIDS conda packages. This PR is currently targeting the 23.12 release.

Accidentally created this from my fork, and it needs to be a branch on the upstream. Replaced by https://github.com/rapidsai/shared-action-workflows/pull/125.


Crashes when too much happening for Files to handle

Description
Whenever I quickly try to archive a zip file or quickly check the properties of something, Files freezes and closes pretty much right after. It crashed on both my laptop and desktop, but more often on my laptop.
Laptop Specs:

Desktop Specs:

Steps To Reproduce

Open Files (Preview) / Files
Select a file and then Right click followed by a quick left click

!Video has been made on laptop!
https://user-images.githubusercontent.com/100706516/224126159-7f5a2738-88cc-4d06-a230-627649606dd5.mp4
Steps to fix
I don't the fix for the issue. I know that is less likely to happen on a more powerful computer like my desktop. I did manage to crash it there too but far less often and just in more heavy loaded cases.
Files Version
<IP_ADDRESS>
Windows Version
Version:22H2 | OS Build 22624.1391
Log file
debug.log

Can you fill out the "Steps to fix" section? (eg. Fix crash that occurs when...)

I think this is specific to the context menu and we're taking several steps to address it. You can expect improvements over the next few updates.


Fix TS 2.4 Errors in rawSearchService.ts

Errors
In rawSearchService.ts:
[11:23:14] Error: /Users/matb/projects/vscode/src/vs/workbench/services/search/node/rawSearchService.ts(58,7): Argument of type 'ISearchLog' is not assignable to parameter of type 'ISerializedSearchProgressItem'.
  Type 'ISearchLog' has no properties in common with type 'IProgress'.
[11:23:14] Error: /Users/matb/projects/vscode/src/vs/workbench/services/search/node/rawSearchService.ts(126,9): Argument of type 'IRawFileMatch | IProgress' is not assignable to parameter of type 'ISerializedSearchProgressItem'.
  Type 'IRawFileMatch' is not assignable to type 'ISerializedSearchProgressItem'.
    Type 'IRawFileMatch' has no properties in common with type 'IProgress'.
[11:23:14] Error: /Users/matb/projects/vscode/src/vs/workbench/services/search/node/rawSearchService.ts(149,9): Argument of type 'IRawFileMatch | IProgress' is not assignable to parameter of type 'IProgress'.
  Type 'IRawFileMatch' has no properties in common with type 'IProgress'.

How to investigate

Install TS 2.4 in the vscode workspace by running npm install typescript@next
Make sure vscode is using the workspace version of TS for error reporting by running the Select TypeScript versions command

Those interfaces have become sort of a mess


injected text

I've pushed changes to improve the versionId: number | null situation, a slight perf improvement where we would resolve ranges all the time when removing decorations (even not interesting ones), and a change to create LineBreakData even when not wrapping.
But it looks like there is some problem with eventing when doing edits. I think perhaps ModelRawLinesInserted also needs an injectedTexts property because apparently there can be injected text on newly inserted lines (if the content itself is not new, but already had injected text).
e.g. pressing Enter and Backspace:
https://user-images.githubusercontent.com/5047891/123167942-91f6cf00-d477-11eb-87f2-9f2a6521d484.mp4

@alexdima
Looks really interesting. Is this multi-line text editor decorations?
Would it be possible to know which issues this fixes and how to use it? I can only guess that it's https://github.com/microsoft/vscode/issues/125742. But will this only be available to completion provider or https://github.com/microsoft/vscode/issues/32856 can be fixed as well?

Looks really interesting. Is this multi-line text editor decorations?

No, this allows proper wrapping of inline text decorations. This feature is not complete though, cursor positioning is buggy with such injected text.
It also potentially allows to style the injected text, which was not possible with css after/before rules.

I can only guess that it's #125742.

This is correct.

#32856 can be fixed as well?

Parts of it (only text injections, not color box injections). But more work is required for that.


Add Uninstall CTT PowerShell Profile feature

Type of Change

[x] New feature
[ ] Bug fix
[ ] Documentation update
[ ] Refactoring
[ ] Hotfix
[ ] Security patch
[x] UI/UX improvement

Description
Implements the ability to uninstall the CTT PowerShell Profile and all its related dependencies.
Testing
I have tested the profile uninstallation functionality by running it in the following test scenarios:

Baseline: Installing the CTT PowerShell Profile
Test 1: Uninstallation of OhMyPosh dependency: [✅] Successful
Test 2: Uninstallation of Nerd Fonts dependency: [❓] Conditional

Additional Context: See Notes for Test 2 in Additional Information.

Test 3: Uninstallation of Terminal-Icons dependency: [✅] Successful
Test 4: Uninstallation of Zoxide dependency: [✅] Successful
Test 5: Skipping uninstallation of in-use dependencies: [✅] Successful
Test 6: Uninstallation of CTT PowerShell Profile: [✅] Successful

Impact

Covered by ## Description, ## Testing, and ## Additional Information.

Issue related to PR

Resolves #2836

Additional Information
New Files Created

$PSProfile.hash: A file that stores the hash of the CTT PowerShell Profile; this is used to check if the profile is installed on the system.

Additional Context:

The path is derived from the $PSProfile variable with .hash being appended to the end of the file path.
Example Hash File Path: C:\Users\<user>\Documents\PowerShell\Microsoft.PowerShell_profile.ps1.hash.
This file will be cleaned up and removed when the CTT PowerShell Profile is uninstalled from the system.

$PSProfile.bak: A file that is used as a backup of the PowerShell profile prior to the CTT PowerShell Profile being installed on the system.

Additional Context:

The path is derived from the $PSProfile variable with .bak being appended to the end of the file path.
Example Backup File Path: C:\Users\<user>\Documents\PowerShell\Microsoft.PowerShell_profile.ps1.bak.
This file will be moved back to the original PowerShell profile's location when the CTT PowerShell Profile is uninstalled from the system.

Notes for Test 2

Note 1: Nerd Fonts won't always be installed due to an issue with the setup script for the PowerShell profile.
Note 2: If the font is not installed, the user will see the message: ===> Already Uninstalled: Nerd Fonts. <===.

Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no errors/warnings/merge conflicts.

Seeing that Powershell profile Installation is pulled in from a remote repo, would it maybe be a better way to create the uninstall logic also in the remote repo instead of in winutil itself?
Alternatively we could completely merge both repos and maintain the psprofile as a part of winutil in the future. All in all, no matter what's done in the end, I think we should keep the psprofile logic (install and uninstall) in the same repo

Seeing that Powershell profile Installation is pulled in from a remote repo, would it maybe be a better way to create the uninstall logic also in the remote repo instead of in winutil itself? Alternatively we could completely merge both repos and maintain the psprofile as a part of winutil in the future. All in all, no matter what's done in the end, I think we should keep the psprofile logic (install and uninstall) in the same repo

That sounds like a good idea, but wouldn't merging the PowerShell profile into WinUtil create additional work to maintain it? 🤔
In addition to that, the profile would need to fit seamlessly within WinUtil’s existing structure, and its setup script would need to play nicely within runspaces.


Allow script to set console color

Snippet (Shell/bash/etc interpreter):
printf "\033[0;31m"
echo red text
printf "\033[0m"

Expected:
red text (in red color)
Actual:
[0;31mred?
[0m

Also tested with: zsh,perl,python.
tput in sh also doesn't work.

As stated in the documentation, Platypus apps are NOT terminal emulators and never will be. If you want full terminal functionality, run the script in the terminal!


support gold coin burn transaction (4th tab in send page) in frontend

Blocked by https://github.com/threefoldtech/threefold-wallet-electron/issues/239, the backend issue has to be finished first.

Functional design I have in mind (please do style it better than my fast mockup):

As you can see it will be a new transaction tab, available in the send page as "burn transaction".
User has to define an amount, optionally the user can switch wallet (as always, to fund from), and a message.
Structured messages are now possible too. So you'll need to define a message type radio button list, so the user can choose between "free formatted" message or a "structured" message. For burn transaction I would put the default type on "structured message", as it makes the most sense for this type of transaction, given you are burning in order to convert your tokens.
When the "free formatted" message option is selected, the user can type in an optional message, as was already possible on other transaction tabs (NOTE: please name it message instead of description, makes more sense IMHO).
When the "structured message" option is selected, the user gets 3 different fields (similar to what you are used to in bank online transfers, at least for Belgian banks). Please make sure that the field automatically jumps from one field to another while you type (meaning if you finished typing in the first of three fields, as in that field is full, it should jump to the next, and than afterwards when that one is full to the third and last field.
For validation of the 3 structured message fields it goes as follows:

all fields can only represent numeric characters (so 0, 1, 2, 3, 4, 5, 6, 7, 8 and  9);
the first field can maximum contain 3 characters
the second field can maximum contain 4 characters
the third field can maximum contain 5 characters

Sending a coin transfer can be done using the API, e.g. to burn 100 coins:
msg = 'this is a free format message'
msg = [123, 4567, 89012] // example of a structured message
msg = ['123', '4567', '89012'] // also an example of a structured message
wallet.coins_burn('100', {message: msg}) // returns a promise in the same format as you are used to

As you can see the structured message can be both a list of strings as well a list integers.

Blocked on parsing the structured message in the transaction list after you burn tokens providing a structured message.

Closed by https://github.com/threefoldtech/threefold-wallet-electron/commit/816c10f7cdeb16f4d914214e0f1ce55d3d199bfd


Missing CONFIG_RPIVID_MEM kernel config for Pi 4

[richbayliss] This issue has attached support thread https://jel.ly.fish/#/e260cf26-d5f4-4b23-89ff-b665e4551b67

Where should this config with the others be added to rebuild the image with Broadcom driver?

Hi @matohavo what are you trying to achieve?

Fixed by https://github.com/balena-os/balena-raspberrypi/pull/457

Disclaimer - I'm new to all of this, so I might be totally off.
Ultimately I'm trying to make hardware accelerated decoding of HEVC video work with BalenaOS on RPi4.
If I understand correctly, the broadcom driver needs to be used for that matter, because ffmpeg patch requires video devices also from this driver:
/dev/rpivid-hevcmem
/dev/vc-mem
/dev/vcsm
/dev/vcsm-cma
/dev/vchiq
/dev/vcio

Those devices are not present in current 64bit, so I'm not sure if unavailability of vc4 on 64bit systems affect this somehow and if I have to use 32bit image.
I've tried to build 32bit image for RPi4 - the build succeeded after I added a coffee file, but it didn't boot. There was no coffee file for barys for 32bit RPi4, although the machine file is there - is there a reason for that?
guys from LibreELEC managed to get this working using V4L2, but they used newer kernel 5.4.16 - is it possible to build BalenaOS with new kernels?

Thanks

Hi, I can do a remote host OS update on a test device of yours if you wish and then you can check if the  CONFIG_RPIVID_MEM is all that's needed of if something else needs to be enabled.
We only ever tested the 64 bits OS and we have no plan going forward to add the 32 bits machine so no idea why it was not booting for you.
We will be switching to a new kernel once the BSP uses a newer kernel. I see that the newest BSP branch is zeus which has kernel 4.19.93 so 5.x is a long wait I would say.

Ok, thank you. I need these configs:
CONFIG_BRCM_CHAR_DRIVERS=y
CONFIG_BCM_VCIO=y
CONFIG_BCM_VC_SM=y
CONFIG_BCM2835_DEVGPIOMEM=y
CONFIG_RPIVID_MEM=y

Building vc_sm for 64bit is failing.

Thanks for suggested approach, it helped.
#close 453

Hi @zub0r is this still something you need?


CA module

Dear Author
I don't see the use of CA module in mctrans.py. Is the CA module useless?

hi， please check https://github.com/JiYuanFeng/MCTrans/blob/01afe3b589e779730b04964b6419b2b5afb7e909/mctrans/models/heads/mctrans_aux_head.py#:~:text=class MCTransAuxHead(nn.Module)%3A

您好，请查看https://github.com/JiYuanFeng/MCTrans/blob/01afe3b589e779730b04964b6419b2b5afb7e909/mctrans/models/heads/mctrans_aux_head.py#:~:text=class%20MCTransAuxHead(nn.Module)%3A

Thank you very much for your reply


[BUG]Can't withdraw all sits From Staking Address

Hi,
1、sit version: 1.2.0
2、on command line, input the command:
sit wallet send_from -s xxxxxx -t xxxxx , occur the following warning:
2022-01-11T15:01:25.550 wallet chia.wallet.wallet_node    : WARNING  SpendBundle has been rejected by the FullNode. {'error': 'INVALID_BLOCK_COST',  'status': 3,  'txid': '0x5fbfa1aa7b13bbb2237df100a0f8d152520b34610a87d7abff3082d115acff9b'} 2022-01-11T15:01:26.944 wallet chia.wallet.wallet_node    : WARNING  SpendBundle has been rejected by the FullNode. {'error': 'INVALID_BLOCK_COST',  'status': 3,  'txid': '0x5fbfa1aa7b13bbb2237df100a0f8d152520b34610a87d7abff3082d115acff9b'} 2022-01-11T15:01:27.225 wallet chia.consensus.block_header_validation: INFO     [debug] validate block 1.694634298946897077013977471 2022-01-11T15:01:27.488 full_node chia.full_node.full_node: INFO     ⏲ Finished signage point 54/64: CC: 1cee5c63f6479acec822ec8ff72a44af55efcb25fde59f65f3223aaf6bf45e3b RC: e19c62a45dc0127fd541cf407080d4553525a0c8d76702d8d492cdb1df621e6c
and then the sits doesn't send to the target address successfully,  the transaction status is always "pending".
how to solve this problem?

Can you provide your staking address ?

I have this problem too

Can you provide your staking address ?

sit1ep6ntmhvue9nkn7g279l964erdm049j9wr2zs47hwcprfsmvrj9qelljhd

Will fix it in the next release

@sofregaz you can try to install the dev branch. I added "amount" parameter to the "send_from" command. Since you have 3000 coins in your staking address, it may exceed the max cost of the block if you withdraw them all in one block.

@sofregaz you can try to install the dev branch. I added "amount" parameter to the "send_from" command. Since you have 3000 coins in your staking address, it may exceed the max cost of the block if you withdraw them all in one block.

just switch to the dev version，any other place need to change？

@sofregaz you can try to install the dev branch. I added "amount" parameter to the "send_from" command. Since you have 3000 coins in your staking address, it may exceed the max cost of the block if you withdraw them all in one block.

just switch to the dev version，any other place need to change？

Run the send_from command should be ok, no other place need to change.


Bundle only addons/beehave folder in releases

Is your feature request related to a problem? Please describe.
We currently bundle the entire .zip folder for the release but users of this addon do not care about all the extra files.
Describe the solution you'd like
Provide a better way of bundling releases to minimise the size.

This worked flawlessly in #83! Closing this.


[formrecognizer] Update samples

Enable type checking on samples

API change check
API changes are not detected in this pull request.


Gitignore .vagrant

.gitignore
If you use Git, add ".vagrant/" to your .gitignore file. Vagrant creates this folder when booting the VM: it contains auto-generated stuff you don't want to version.

From: http://tech.osteel.me/posts/2015/01/25/how-to-use-vagrant-for-local-web-development.html

This is not relevant to dotfizzles. Closing.


FEATURE REQUEST - add autocorrect flag

add autocorrect, enableSuggestions,  flag
https://api.flutter.dev/flutter/material/TextField/autocorrect.html
https://api.flutter.dev/flutter/material/TextField/enableSuggestions.html

The autocorrect option is already available. Try using the plugin's auotCorrect option. The enableSuggestions option is always set to true, but I'll add it if necessary.


add requirements

kafka or datahub are not necessary, so I don't think we need to add the requirements.

kafka or datahub are not necessary, so I don't think we need to add the requirements.

I found this output
[2024-11-14 20:31:52,789][WARNING] DataHub is not installed[Traceback (most recent call last):
  File "/usr/local/python3/lib/python3.8/site-packages/easy_rec-0.8.5-py3.8.egg/easy_rec/python/input/datahub_input.py", line 21, in <module>
    from datahub import DataHub
ModuleNotFoundError: No module named 'datahub'
]. You can install it by: pip install pydatahub
[2024-11-14 20:31:52,791][WARNING] kafka-python is not installed[Traceback (most recent call last):
  File "/usr/local/python3/lib/python3.8/site-packages/easy_rec-0.8.5-py3.8.egg/easy_rec/python/input/kafka_input.py", line 16, in <module>
    from kafka import KafkaConsumer, TopicPartition
ModuleNotFoundError: No module named 'kafka'
]. You can install it by: pip install kafka-python


Debian 10.10 error when I try to launch AppImage

Hello team
When I try to launch for the first time Landrop 0.4 on my debian 10.10, I receive in terminal output this message 👍
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found. This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: xcb.
First, libsodium is installed and I even try with installation of full qt platform, anyway Landrop don't start ...
could someone help me?
I saw that it works well on debian 11, but for the moment debian 11 is too new for some things I use.
Best Regard,
Mtxrider.

Was someone able to resolve this issue? Can't start the AppImage with Deepin 20.5. I thought that AppImages are supposed to resolve this kind of dependency issue. Is there a way to improve the build process so that there is no dependency on a specific glibc version?


[@thumbtack/thumbprint-react]: Disabled Label Font Color does not Comply with WCAG 2.0

Disabled Labels don't comply with WCAG 2.0 in terms of contrast to background (assuming the background is #FAFAFA, the text #D3D4D5). That means the labels are hard to read.
This can be seen here: https://thumbprint.design/components/checkbox/react/#section-disabled-checkboxes

From https://webaim.org/resources/contrastchecker/

WCAG 2.0 level AA requires a contrast ratio of at least 4.5:1 for normal text and 3:1 for large text. WCAG 2.1 requires a contrast ratio of at least 3:1 for graphics and user interface components (such as form input borders). WCAG Level AAA requires a contrast ratio of at least 7:1 for normal text and 4.5:1 for large text.

Hey @TTblee — Thanks for the issue. My understanding is that the contrast requirements technically don't apply to disabled elements. Here's an interesting thread on this. That said, I also disagree with the standard for the reasons mentioned in those GitHub comments.
While @jonpkerwin is exploring colors, I'd be curious to understand where and when we use disabled radios and checkboxes. Should we even be displaying the form elements if they are disabled?

Here's an example of a design we are using on Pro Onboarding.
A user is enrolled in a category already, and we want to display that, but not allow them to deselect from the list. The page allows you to enroll your business into more categories.

@tomgenoni — This could be another good fix-it week task to take on.

I did some research and, as you'd expect from the w3c Github issue linked to above, there is no consensus about what color a disabled "thing" should be. I think ours could be a little darker but it's in the ballpark as other sites (FWIW, Airbnb's is in fact even lighter). Thumbprint will be adding guidelines on Forms in the coming months and we'll revisit this and a number of other form related items at that time.
But one likely recommendation will be to only show disabled states when absolutely necessary. @TTblee in the example you posted above we could redesign this to avoid using the disabled checkbox. For example, we could separate the previously chosen item and use a check icon or use words to effect "You've picked Electrical and Wiring Repair..." Currently the disabled state connotes a somewhat negative "you picked this and can't undo it" rather than an affirmation of the choice.
I'm going to close this for now and add it to the items to consider during the Form guidelines project. If you'd like to discuss it further @TTblee let me know.


root: bump api-extractor & friends + recreate API reports along with previously unsupported packages

A new version of api-extractor is just out, which now supports the previously unsupported import() and import * as ... syntax, allowing us to create API reports for all packages that we want.

Would be neat to some day add a prettier step to these after generation so we ger isolated from little spacing quirk variations

I guess we can probably continue the work in this PR rather than #6380 @OrkoHunter?

@freben making that this day x), as with many things in the api-extractor script, it's not pretty 😅

I guess we can probably continue the work in this PR rather than #6380 @OrkoHunter?

Sounds good!

Let's goooo!!

@backstage/techdocs-core / @OrkoHunter need a +1 here.


Templates listed wrong under "Platform"

Describe the bug
The WinUI templates also adds a "Windows App SDK" category that incorrectly gets placed under platform:

I'm not sure what the purpose of this category is though, it already has a WinUI category which is probably what you'll be using to start a WinUI app. I'd suggest removing the category #LessIsMore
Steps to reproduce the bug

Steps to reproduce the behavior:

Install the preview 3 C#extension for VS2022
Create a new project and check the project dialog

Expected behavior
Windows App SDK isn't under the 'platform' section.
Screenshots

Version Info
NuGet package version: 1.0.0 preview 3
VS 2022 RC3.

Windows version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

Additional context

@BenJKuhn serve, return volley... :-) Forgot to tag you on the reply above. Are you asking about that? Is the 'fix' that simple?

@BenJKuhn @DrusTheAxe any updates on this? We definitely should NOT be introducing a new Platform category here, sounds like we always had this (previously as Project Reunion) though, but we shouldn't have this.
It definitely seems like it should be as simple as just removing that line from the VS templates. We already correctly have the Windows platform tag so we show up there.
@DefaultRyan would this be something you'd own? Are there any internal bugs tracking this yet?

I've created an internal bug to track this Bug 36933899: WinAppSDK VSIX incorrectly adds Windows App SDK as a "Platform" in the new project choices.
We'll fix this in a 1.0 servicing release after GA. Thank you @dotMorten for bringing this to our attention!

Thanks @andrewleader. I'm playing with the VSIXs a lot lately, so I'm happy to take it, or if @DrusTheAxe wants to keep this assigned to him, that's fine with me as well.

Thanks @andrewleader. I'm playing with the VSIXs a lot lately, so I'm happy to take it, or if @DrusTheAxe wants to keep this assigned to him, that's fine with me as well.

All yours :-)


A8 – A11 VMs getting decommissioned

Hi,
Please let me know If the plan to decommission  A8 – A11 VMs is still happening by end of this month.
Regards
Karthick

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

ID: 616242df-d3da-e9b3-73b0-fba7ae7a28a7
Version Independent ID: 33587769-6f9d-af7c-3fc1-08676adc860b
Content: Azure VM sizes - previous generations - Azure Virtual Machines
Content Source: articles/virtual-machines/sizes-previous-gen.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @mimckitt
Microsoft Alias: mimckitt

@karthick3d
Thanks for your Question! We are currently checking your ask and will get back on this.

@mimckitt
Can you please check and add your comments on this as applicable.

@karthick3d Yes. There is no change to the announced retirement on 3/1.
If you have been a user of these VM sizes in the past, you will have received (multiple) notifications by now, one as recent as 2/16. Please let us know if there are any questions.

I havent received any notification. I just noticed we have Standard_A8v2 rather than Standard_A8. Are we saying that it may not affect A8V2 VMs?

Correct. A_v2 series is not covered as part of this retirement.
If you were a user of A8 through A11, you would receive the retirement notification.

@karthick3d
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Add more cases and fix join key in Analytic Rule AzDiagSettingsDeleted.yaml

Change(s):

Add case where OperationNameValue is not uppercase (thus microsoft.insights/diagnosticSettings/delete).
Fix join key on the right side of the join.

Reason for Change(s):

AzureActivity events with the same OperationNameValues can have different parsings, so the rules must contemplate both possible parsings.
The resource column in the right side of the join was returning just the resource name like exampleresourcename, while the resource column in the left side was returning the resource path, like /subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/rg-exampleresourcegroup/providers/microsoft.sql/servers/exampleresourcename

Version Updated:

Yes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @devikamehra,
Can you please review the changes and approve the PR.
Thanks

Hi @devikamehra, @rushriva
Can you please review the changes and provide your feedback.
Thanks

Hi @devikamehra, @rushriva
Can you please review the changes and provide your feedback.
Thanks

Hi @devikamehra, @rushriva
Can you please review the changes and share your thoughts on it.
Thanks

Hi @devikamehra, @rushriva
Can you please review the changes and share your thoughts on it.
Thanks

Hi @devikamehra,
Can you please review the changes and share your feedback.
Thanks

Hi @devikamehra,
It would be great if you can review this PR and share your feedback as its pending from long back.
Thanks

@vakohl As you are already working on AzureActivity solution improvement, can you please take this up.

Hi @vakohl,
Can you please review the changes and provide your feedback.
Thanks

Hi @vakohl,
It would be great if you could spare some time to review this PR and share your thoughts.
Thanks

Hi @vakohl,
Could you please review the changes and provide your feedback.
Thanks

Hi @ep3p,
If you are done with the changes please confirm and request for re review.
Thanks


permit empty capture groups?

It seems like both Ruby and Python support empty capture groups. Their use is a little niche, but it probably isn't hard to permit them.
>>> import re
>>> m = re.match('()', 'hi')
>>> m.group(0)
''
>>> m.group(1)
''

Today, a regex with an empty capture group is forbidden, so it would be backwards compatible to add this after a 1.0 release.

What are some use cases of empty capture groups?
I'm writing some code that tokenizes an input string with regexes, and an empty capture group would have been useful. I needed my token regexes to have the same number of capture groups so that they can all be unpacked uniformly, and this was only possible in one case with an empty capture group.
I was able to work around it however, so it's kind of a weak use case.

This issue has been open for a long time, and people seem to have made do without it, so I'd like to close this issue.
Empty capture groups are still forbidden, so this is something that can be revisited if we so chose.

I just ran into this issue / wanted to have this feature for auto generated regexes.
This issue can of course be easily worked around by use non-empty capture groups which match zero length strings (eg. (\\b|\\B) or (a{0})) so this feel like a very arbitrary limitation.

I actually also ran into this issue a second time. I wrote a program which colorizes stdin according to user-supplied regexes, and there was a crash if the user supplied an empty regex. Of course, I could have checked for this, however, it seems like a unnecessary restriction.

@casey Why does it feel like an unnecesary restriction? You need to check for errors anyway.

Certainly, but it's a restriction that I don't think that users will necessarily expect.
If their knowledge of regexes is from Python, Ruby, or JavaScript, then this is an incompatibility.
Admittedly, there are other differences, but it seems like relaxing this would be nice.

There are many many incompatibilities and this still seems like a very strange thing to explicitly support.

Also, anecdotally, I also ran into this another time when programmatically generating a regex.
I was doing something like:
s = ''
for name, pattern in parts:
  s += '(<?{}>{})'.format(name, pattern)

On of the patterns was empty, and thus inadvertently created an empty capture group. Of course, I was able to fix this, but it introduced more special cases and checks in my code. Also, it was a case where it wasn't easy to visual inspect the pattern for empty groups, since it wasn't a simple string literal.
I think that the point isn't really that this is vital to support, or that it's useful (it's definitely a very strange thing!) but that people will run into this problem, so why not just make it do the right thing.
I've run into it on three separate occasions. Of course, this is anecdotal evidence, but I suspect that others will also run into it.

FWIW, I do think autogenerating regexes is the most compelling use case for
this.
On Feb 10, 2017 6:08 PM, "Casey Rodarmor"<EMAIL_ADDRESS>wrote:

Also, anecdotally, I also ran into this another time when programmatically
generating a regex.
I was doing something like:
s = ''for name, pattern in parts:
s += '(<?{}>{})'.format(name, pattern)
On of the patterns was empty, and thus inadvertently created an empty
capture group. Of course, I was able to fix this, but it introduced more
special cases and checks in my code. Also, it was a case where it wasn't
easy to visual inspect the pattern for empty groups, since it wasn't a
simple string literal.
I think that the point isn't really that this is vital to support, or that
it's useful (it's definitely a very strange thing!) but that people will
run into this problem, so why not just make it do the right thing.
I've run into it on three separate occasions. Of course, this is anecdotal
evidence, but I suspect that others will also run into it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/regex/issues/106#issuecomment-279091393,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAb34mSrfPGHPa-3z27bQziZpKT2Y23Kks5rbO36gaJpZM4FkCXl
.


Can't set port in Casdoor url

When using casdoor for local development, there is cases, when casdoor cannot be published on 7001 port.
I thought, it would work, if I define port in the Endpoint, like this:

But it didn't worked, it still uses 7001 port.
How this can be avoided?

@MadL1me hi, why close it?

@hsluoyz sorry, I mistaken that problem was fixed for me. I was tired and forget at the start of the day, that I fixed it yesterday evening by deploying Casdoor localy on 7001 port. The problem still exists

@MadL1me please make a PR

When using casdoor for local development, there is cases, when casdoor cannot be published on 7001 port.
I thought, it would work, if I define port in the Endpoint, like this:

But it didn't worked, it still uses 7001 port. How this can be avoided?

May be you should clear your browser's cookies and run it again.

@MadL1me did you restart the MVC application and reload the page? The changes in appsettings.json should have taken effect after you refresh the web page in browser

Closed as resolved


Add GFG solutions

Create a separate branch/folder and further a branch/folder labelling it as Beginner level or Easy section.
Further solutions must be in C or C++ language only.
explaining your approach in a clear way, no plagiarism/copying should be there.

Hello @MilindGupta-Creator  , I would like to work on this issue
Following thing, I would like to add in this repo

[x] Area_of_triangle using C with documentation

Thank you

Yes sure make a folder and place it

I will add this problem : Find the only repeated element in the array.

@MilindGupta-Creator Check out PR #28


Allow to resize a layer via handles on its borders/corners

A layer should allow to be resized within the page using handles on its borders/corners like graphics programs do it.
Sebastian

@ibuck would this be actually useful?
Honza

@janodvarko Short answer: no
I can't think of a valid use-case for this. In general, a developer would not want to resize an overlay because it will already be exported at the desired dimensions to match the design.

Yep, that's what I thought
Honza

So what about the scaling feature in general? Should it be removed then?
The decision on this has influence on #4 and #21 and maybe other issues.
Sebastian

It's an adept for removal, we'll discuss that on our meeting.
Honza


Updated libs to current versions

I had issues installing this on newer versions of Ruby. Updating the libs solved this issue. I also updated some of the API call examples with the newest versions (SABnzbd ver. 1.2.3).
/Users/homans/.rvm/rubies/ruby-2.3.3/bin/ruby -I/Users/homans/.rvm/gems/ruby-2.3.3/gems/rspec-core-3.5.4/lib:/Users/homans/.rvm/gems/ruby-2.3.3/gems/rspec-support-3.5.0/lib /Users/homans/.rvm/gems/ruby-2.3.3/gems/rspec-core-3.5.4/exe/rspec spec/sabnzbd_spec.rb spec/utils_spec.rb
WARNING: VCR::RSpec::Macros is deprecated. Use RSpec metadata options instead `:vcr => vcr_options`
WARNING: VCR::RSpec::Macros is deprecated. Use RSpec metadata options instead `:vcr => vcr_options`
..............

Deprecation Warnings:

Using `should` from rspec-expectations' old `:should` syntax without explicitly enabling the syntax is deprecated. Use the new `:expect` syntax or explicitly enable `:should` with `config.expect_with(:rspec) { |c| c.syntax = :should }` instead. Called from /Users/homans/code/sabnzbd/spec/sabnzbd_spec.rb:12:in `block (3 levels) in <top (required)>'.

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
`config.raise_errors_for_deprecations!`, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

1 deprecation warning total

Finished in 0.14339 seconds (files took 1.87 seconds to load)
14 examples, 0 failures

There are some deprecation warnings that should probably be examined.

Hi @kalinon thanks for your PR! I haven't done anything with this code in a long while. Your PR looks good however, 'everthing' is crammed into 1 commit. I am okay-ish with changing every double quote to single quotes, but I would like to see that in a separate commit. The same goes for updating the dependencies, and updating the mocked api responses.
So please:

[ ] 1 commit for changing double quotes to single quotes.
[ ] 1 commit for updating dependencies.
[ ] 1 commit for updating the mocked responses.

Thank you!

Sure! I'll play with that. I'm still learning to commit small changes 😁

Cool! If you need any help let me know :)


Questions about dataset_states_to_obs.py

Hi! Thank you all for the fantastic work! I am trying to look for one way to extract observations from the downloaded demo.hdf5. I found the instruction in the tutorial. I liked the idea of it. However, I don't know how to extract depth observation from it. It seems that I can only extract RGB observations only. Do you have any ideas on how to extract depth observation from the demo.hdf5?

If you're talking about robosuite datasets, that would require a few changes to the script and the EnvRobosuite class. It could also require some other changes in the codebase to make sure depth images are processed correctly for training. We could put it on our roadmap for the next release if there's interest.

Actually, I have found one way of rendering depth images. If you are interested, I could create a pull request and discuss it. If that has lower priority, we could close this issue.

Please share the pull request with us!

Thanks! I just created the pull request #99. We may discuss there.


How can I use this component to infinite scroll horizontally?

So I have an horizontal list, but how do I make it work with InfiniteScroll horizontally?

As of the current implementation it can't work horizontally. I've to tweak
some things probably to make it work. Sorry about that!
On Tue, 22 Oct, 2019, 11:22 PM Simone Aronica<EMAIL_ADDRESS>wrote:

So I have an horizontal list, but how do I make it work with
InfiniteScroll horizontally?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ankeetmaini/react-infinite-scroll-component/issues/142?email_source=notifications&email_token=ABSYHF5ADNS7QGEGHHHTZB3QP44W7A5CNFSM4JDTGJE2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HTSMKWQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSYHF6ZE6E3TDGFIMPHHSDQP44W7ANCNFSM4JDTGJEQ
.

Oh okay, thanks anyways! in the end I've managed to build my own way to create an infinite scroller!

Oh okay, thanks anyways! in the end I've managed to build my own way to create an infinite scroller!

How did you do that?

Oh okay, thanks anyways! in the end I've managed to build my own way to create an infinite scroller!

How did you do that?

I don't remember how I did that to be honest, it's too long ago. But I see that on NPM there are now modules for that!
https://www.npmjs.com/package/react-horizontal-infinite-scroll


Reliance on dnsapi.dll in Windows 7 and 10

I am trying to point the windows update hostnames to only one IP each so I can whitelist them on my firewall, otherwise they will be quite random occasionally breaking firewall whitelists.
I have tried removing and also editing out the hosts from dnsapi.dll.
Some example hosts for windows 10:
go.microsoft.com (telemetry, thankfully this can be blocked in DNSCrypt despite existing in dnsapi.dll)
sls.update.microsoft.com (for windows update)
Windows 7:
ds.download.windowsupdate.com (for windows update).
Some furthur reading here from a long time ago: https://slashdot.org/story/06/04/16/1351217/microsoft-bypasses-hosts-file
Do not overlook the small text at the top of the thread there which contains a link to www.securityfocus.com.
If I delete dnsapi.dll Firefox crashes.
If I hex edit-out the hostnames from dnsapi.dll(it is not signed surprisingly, at least not in Windows 7). The hostnames do not resolve at all.
I have DNS Client service disabled and using exclusivly DNS Crypt with no other adapters but loopback.
I would like DNSCrypt to either make irrelevant the feature Microsoft has had with forced hostname resolution or for DNSCrypt to have a plugin which acts as its own high priority hosts file which can bypass the windows crap.

This may fail anyway since they certainly use multiple hosts/IPs/load balancing/etc.

in win10 it is signed sha256 but expires in 18.11.2016!
you could pass the connection with hostnames to socks5 firewalled, and make whitelist rules by hostname instead of ip... proplem solved.

Note that unbound can serve the IP addresses you want for specific names. Most other DNS caches can probably do the same thing. That's way cleaner and more reliable than deleting DLLs.


adding follow us section

I would like to work on this feature, could you please assign me this

Please explain your approach in detail and follow the template when raising an issue.

@jinx-vi-0 I would like to add a side bar for social icons, or i can add the follow us on section in footer also,

@jinx-vi-0 Please assign me this issue I would enhance the overall UI

@jinx-vi-0 could you please add the level tag too

@jinx-vi-0 could you please add the level tag too

I'll add it after reviewing the pr.

@jinx-vi-0
I have solved this one, kindly review it


Add support for showing icons instead of text labels

Closes #4

Just wanted to say icon mode looks AMAZING!  Thank you!


[preprocessor/folder] fix missing fallthrough

wacz should be handled like generic

Note another way to do this would be case "generic", "wacz":.


Added comma

Signed-off-by: Travin Keith<EMAIL_ADDRESS>

Hey @hartm. I added the proper signoff as instructed by Ry, but it doesn't seem to be passing the DCO. Would it be ok for you to apply the changes directly to the Master branch?

Signed-off-by: Travin Keith<EMAIL_ADDRESS>


plugin does not differs fingers

Hello all. I have fingers sabe on my smartphone.
I do encrypted one time for each finger using this options
clientId - both for same fingers
username - different for each finger
After this i've tried to use decrypt function changing my fingers with another password.
For example
What i've registered
finger1
pass 123
finger2
pass456
What i did is used decrypted as well using
finger1
pass456
or
finger2
pass123
So whats is the utility of the token if the plugin does not differs each finger and those password or what i'm doing wrong?
Thanks

Android Fingerprint API does not return a Fingerprint ID.  It only returns if the probe matched one of the prints registered on the device.  The encrypted token is for securely storing the user credentials and so you can retrieve the password for a standard login with username and password.

Any fingerprint registered on the device can authorize decryption of the token.


Paper Trail not tracking Whodunnit column

We are using paper trail 4.0.0 and we make a lot of updates.
For some time now, Paper Trail does not store the whodunnit column.
We have this in application controller
def user_for_paper_trail
    current_user.try(:email)
  end

What could be responsible for this? Is there a fix?

Hi Deborah, how cool to get a bug report from Lagos!
Can you reproduce this problem in the latest version of PT (currently 6.0.2)? You may find our bug report template helpful for this. Thanks.

Closing due to insufficient information.


Angular cli error on components in spec file upon ng serve

Mac OSX Yosemite
angular-cli: 1.0.0-beta.9
node: 6.3.0
os: darwin x64
Using Angular-cli to create components is generating a consistent error on the .spec files on line 17
//let component = new TodoListComponent() // expect(component).toBeTruthy();
The Broccoli Plugin: [BroccoliTypeScriptCompiler] failed with:
Error: Typescript found the following errors:
/Users/dp/thechariot/eggAng1/tmp/broccoli_type_script_compiler-input_base_path-UVT171nw.tmp/0/src/app/todo-list/todo-list.component.spec.ts (20, 1): '}' expected.
at BroccoliTypeScriptCompiler._doIncrementalBuild (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/broccoli-typescript.js:120:19)
at BroccoliTypeScriptCompiler.build (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/broccoli-typescript.js:43:10)
at /Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/broccoli-caching-writer/index.js:152:21
at lib$rsvp$$internal$$tryCatch (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1036:16)
at lib$rsvp$$internal$$invokeCallback (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1048:17)
at lib$rsvp$$internal$$publish (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1019:11)
at lib$rsvp$asap$$flush (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1198:9)
at _combinedTickCallback (internal/process/next_tick.js:67:7)
at process._tickCallback (internal/process/next_tick.js:98:9)
The broccoli plugin was instantiated at:
at BroccoliTypeScriptCompiler.Plugin (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/broccoli-plugin/index.js:10:31)
at BroccoliTypeScriptCompiler.CachingWriter as constructor
at BroccoliTypeScriptCompiler (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/broccoli-typescript.js:27:5)
at Angular2App._getTsTree (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/angular2-app.js:329:18)
at Angular2App._buildTree (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/angular2-app.js:124:23)
at new Angular2App (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/lib/broccoli/angular2-app.js:53:23)
at module.exports (/Users/dp/thechariot/eggAng1/angular-cli-build.js:10:10)
at Class.module.exports.Task.extend.setupBroccoliBuilder (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/ember-cli/lib/models/builder.js:55:19)
at Class.module.exports.Task.extend.init (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/ember-cli/lib/models/builder.js:89:10)
at new Class (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/core-object/core-object.js:18:12)
at Class.module.exports.Task.extend.run (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/ember-cli/lib/tasks/serve.js:15:19)
at /Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/ember-cli/lib/commands/serve.js:64:24
at lib$rsvp$$internal$$tryCatch (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1036:16)
at lib$rsvp$$internal$$invokeCallback (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1048:17)
at /Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:331:11
at lib$rsvp$asap$$flush (/Users/dp/thechariot/eggAng1/node_modules/angular-cli/node_modules/rsvp/dist/rsvp.js:1198:9)

Seems like you have a syntax error /src/app/todo-list/todo-list.component.spec.ts (20, 1): '}' expected..
#1455 should make these errors much more readable.

I agree but its not my error, its the code generated by cli
 Dan Pride
1-303-800-0900 - Skype No Texting1-360-850-9322 Mobile
http://danielpride.com
www.linkedin.com/in/danielpride/http://archaeolibrary.com/https://headlessuber.com
  From: Filipe Silva<EMAIL_ADDRESS>
To: angular/angular-cli<EMAIL_ADDRESS>Cc: Daniel Pride<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Wednesday, July 27, 2016 8:07 PM
Subject: Re: [angular/angular-cli] Angular cli error on components in spec file upon ng serve (#1332)
Closed #1332.—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you show me todo-list.component.spec.ts please?

Even I am facing issue while running ng serve. I have upgraded my project to latest angular and angular-cli to @angular/cli.
When I run ng serve command, I get following error
Time: 18152ms
chunk    {0} 0.chunk.js, 0.chunk.js.map 13.9 kB {1} [rendered]
chunk    {1} main.bundle.js, main.bundle.js.map (main) 159 kB {3} [initial] [rendered]
chunk    {2} styles.bundle.js, styles.bundle.js.map (styles) 140 kB {4} [initial] [rendered]
chunk    {3} vendor.bundle.js, vendor.bundle.js.map (vendor) 5 MB [initial] [rendered]
chunk    {4} inline.bundle.js, inline.bundle.js.map (inline) 0 bytes [entry] [rendered]
ERROR in d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/connections-viewer/connections-viewer.component.spec.ts (69,9): Type '{ srcPropertyName: string; srcPortName: string; targetP
ropertyName: string; targetPortName: strin...' is not assignable to type 'Connection[]'.
Type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...' is not assignable to type 'Connection'.
Property 'uri' is missing in type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...'.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/connections-viewer/connections-viewer.component.spec.ts (83,9): Type '{ srcPropertyName: string; srcPortName: string; targetPropertyNa
me: string; targetPortName: strin...' is not assignable to type 'Connection[]'.
Type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...' is not assignable to type 'Connection'.
Property 'uri' is missing in type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...'.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/connections-viewer/connections-viewer.component.spec.ts (94,9): Type '{ srcPropertyName: string; srcPortName: string; targetPropertyNa
me: string; targetPortName: strin...' is not assignable to type 'Connection[]'.
Type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...' is not assignable to type 'Connection'.
Property 'uri' is missing in type '{ srcPropertyName: string; srcPortName: string; targetPropertyName: string; targetPortName: strin...'.)
ERROR in d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/system-dashboard/system-dashboard.component.spec.ts (93,15): Property 'systemId' does not exist on type 'SystemDashboardCompo
nent'.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/system-dashboard/system-dashboard.component.spec.ts (105,15): Property 'systemId' does not exist on type 'SystemDashboardComponent'.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/system-dashboard/system-dashboard.component.spec.ts (107,15): Property 'startSystem' does not exist on type 'SystemDashboardComponent'
.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/system-dashboard/system-dashboard.component.spec.ts (114,15): Property 'systemId' does not exist on type 'SystemDashboardComponent'.)
d:/Project/Cosym/Cosym1.2/mockup/client/src/app/system/system-dashboard/system-dashboard.component.spec.ts (116,15): Property 'startSystem' does not exist on type 'SystemDashboardComponent'
.)
webpack: Failed to compile.
There is no error actually in my spec files. I have commented code in specs and compiled it worked. Later I have uncommented the code, still it compiled successfully.
I did fresh ng serve again, facing same issue..
Could someone help me with this. Thanks in advance :)


Add clearer error message for missing project icon during Android export

Resolves https://github.com/godotengine/godot/issues/39218

Thanks!


🐛 Bug - Fails to compile in TypeScript CommonJS project with moduleResolution: node16/nodenext

🐛 Describe the bug
microdiff doesn't export types correctly for Node16+'s ESM module resolution. This means that using microdiff with a project using moduleResolution: Node16 or NodeNext, which is what TypeScript recommends for Node projects, fails to compile because of missing types.
🧑🏫 To Reproduce
When compiling a CommonJS TypeScript project with moduleResolution: node16 or nodenext, importing microdiff results in the following error:
error TS1479: The current file is a CommonJS module whose imports will produce 'require' calls; however, the referenced file is an ECMAScript module and cannot be imported with 'require'. Consider writing a dynamic 'import("microdiff")' call instead.

To convert this file to an ECMAScript module, change its file extension to '.mts', or add the field `"type": "module"` to ...

See https://arethetypeswrong.github.io/?p=microdiff%401.4.0 and you'll see that microdiff has an ESM types file but none for CJS, so TypeScript ultimately "can't find" the CJS types, so it thinks the package is ESM-only. arethetypeswrong is a great tool to investigate and validate your export map.
🧑💻 Code
I can provide a test project if needed.
✅ Expected behavior
No compilation error.
Possible fix
A possible fix would be to create a rolled-up declaration for ESM and CJS using rollup and rollup-plugin-dts. This has worked well in some other projects I've seen, but might not be appropriate here.
Another option might be to use https://github.com/isaacs/tshy to use TypeScript to produce both ESM and CJS output.
Options that are tempting but don't work
You might be tempted to make the exports map look like this and use the same declaration files for all exports. I.e.
"exports": {
  "require": {
    "types": "./dist/index.d.ts",
    "default": "./dist/index.cjs"
  },
  "import": {
    "types": "./dist/index.d.ts",
    "default": "./dist/index.js"
  },
  "default": {
    "types": "./dist/index.d.ts",
    "default": "./dist/index.js"
  }
},

This setup creates the following problems:

When importing in a CJS TypeScript project, the types are resolved as ESM but should be CJS: https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/main/docs/problems/FalseESM.md
When importing in an ESM TypeScript project, the types are not resolved properly because the import statements in the declarations don't include file extensions: https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/main/docs/problems/InternalResolutionError.md

🖥 Runtime Version
Node 18, mac OS ARM

This should be fixed in 1.5.0


Add Auto Lint

Hey @steviebeenz,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 468f58f0-c517-11e9-8712-75d78f9b457f

Hey @steviebeenz,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 4c532d70-c517-11e9-8712-75d78f9b457f

Please provide a description to understand easily what this PR does.

Fully lints, and formats HTML, CSS and JS.
See the Pocket Lint github plugin for more info.

Closing this PR since project had been migrated to Vue Nuxt  #51


AppImage available for download?

Hi, thanks for making an AppImage for Linux. Is it available for download somewhere?

Go to Actions tab. Latest is https://github.com/mryndzionek/composable-sdr/actions/runs/102738008

There is a short note in the README on the devel branch: https://github.com/mryndzionek/composable-sdr/tree/devel

Thank you very much. Unfortunately it is a zip which we cannot use for our automated tests, please see https://github.com/AppImage/AppImageKit/wiki/Creating-AppImages/cc2441518975caca23e9ce2dba6f08a22c678d1e#common-mistake for an explanation. Thank you very much for choosing AppImage as a format and for your consideration.

GitHub actions https://github.com/actions/upload-artifact works this way: https://github.com/actions/upload-artifact/issues/3#issuecomment-620722206. I think I'll do first official release in few days and upload it unzipped. I can let you know.

Thanks a lot! :+1:

Hi. I've made the release here: https://github.com/mryndzionek/composable-sdr/releases/tag/v0.1.0-alpha

I've messed around now for about an hour trying to build from source just so I could try out the helicopter decode. No luck. Would you happen to have an appImage for that app as well?

Not sure when I'll have time to do an official release, but you can try forking and then pushing master branch to build branch. This will trigger a github action that will (hopefully)  build the appImage. However this appImage will only contain soapy-sdr app, not the helidecode app needed for extracting KML files.

Okay I’ll try that, although the helicopter part is what I really wanted to try. The standalone sdrapp image works fine with at least my rtlsdr but doesn’t see the soapy 7 modules that my other programs (cubicsdr, SigDigger) sees. So I’m unable to test with the hackrf and sdrplay equipment.
I pulled down cable and required ghc, but my first problem was that I had to change the version from <=4.12 in your cable file to 4.9 before I could get cable to do anything. After that things just kind of fell apart when trying to individually build the helicopter decode. I’m not by the computer but some package called repeater something failed to build do to something with anyTell and manyTell.. I’ll probably have to wait till you release something.

Okay, yes, I know that SoapySDR version mismatches can be problematic. librtlsdrSupport.so is bundled with the AppImage and it would probably be best to do the same with the other modules. Current AppImage expects modules0.8.
Regarding building from source you might want to use stack instead of cabal. For more details on building you might want to look at https://github.com/mryndzionek/composable-sdr/blob/master/.github/workflows/build.yml

Thank you. I think I see how I can modify and build this. I figure it’s also worth asking, if you happen to have the time, check out my “DragonOS LTS” project which now includes your AppImage. If by chance you could build all your apps taking into account everything I have presintalled in DragonOS would open up your project to everyone running the distro I’ve been workin on. I’d love for it to use all the soapy7 modules I’ve included which should mean SDRplay equipment would be supported. Otherwise I’ll try and get to it myself when I have time. Thanks!

So it was the ghc version that was killing the process. I figured out I had to add the PPA to get the correct ghc referenced in your build directions. Seeing what the end result will be now that it’s all building.

Yeah, I think build.yml is ideal reference for adding the steps needed for building on other distros. It should also be possible to compile soapy-sdr for ARM just using ARM Ubuntu base image (GitHub doesn't provide one yet). Regarding the helicopter example, I think it has only educational value. Is there much sense in distributing an app capable of doing end-to-end decoding? The app is specific to this particular protocol format and isn't a streaming app - expects all samples being available upfront.

My project is just 18.04 Ubuntu for now. I was able to build your soapysdr AppImage, however, it pulled in all sorts of rtlsdr and soapymodules due to me have a variety of things preinstalled. The end result was a segmentation fault even after went into the AppImage folder and manually deleted things I didn’t think was needed and then rebuilt the AppImage. Very cool process and learned a few things along the way. I think it’s important to use a clean build system with the bare minimum dependencies installed, then add extra soapymodules. I’ll try this soon.
As far as helidecode, I couldn’t get that app to build. I ended up using just a couple of your other examples. Sorry to fill up your thread here. Look forward to whatever else you release.
https://youtu.be/tU2BXGuUU5k

I was struggling with segfaults also - this is caused by mismatch between SoapySDR and Soapy modules and this mismatch occurs in official release versions (!!!). That's why in my build.yml I use very specific versions/commits. If you have debug builds of the libraries you can use ordinary GDB to investigate the segfaults.
Regarding AppImage building, did you use cabal2appimage? This app might be worth a separate repo and its own AppImage (as opposed to helidecode :smiley:).
I'll definitely be developing soapy-sdr further. Something like this was missing from my toolbox. The UI apps (CubicSDR etc.) are not that great for automated workflows and have limited capabilities. GNU Radio is often too big. On top of that Haskell just makes so much sense for applications like this.
I don't have other SDR hardware to test, but I can add other SoapySDR modules to build.yml, so that they are bundled in AppImage.

I've added a new issue: #2. Lets move there.


Split development config from production config

in docker-compose.yml and docker-compose.override.yml. This should speed
up builds e.g. on Travis CI, which does not need to sync folders or run
yarn run dev if the docker image was built recently. Also it should
make the build more reliable as it behaves more similar to our deployment.

@roschaefer I do not get the change as you do not build the image at all but using the last image!? I'm confused.


Make tables rows clickable

Make the whole row clickable in schema registry view. All the tables should work the same way.

schema registry table: make it a list with tags

Not that important for now


Complete core formula bottling

By boneyarding these unbottled formulae. See https://github.com/Homebrew/homebrew-core/pull/2194

Whoops, didn't mean to squash these 😞

You could get out the force push light saber. :zap:

@ilovezfs an inelegant weapon for a less civilised age 😉

Well, this is the boneyard after all. Bam bam, bam bam bam.


[BUG] Copy text on an edited post copies the original text to clipboard

Describe the bug
Copy text on an edited post copies the original text to clipboard instead of the (new) edited text
To Reproduce
Steps to reproduce the behavior:

Create a new post
Edit the post and add new text
Copy text
Copied text is the original note text and not the new edited text

Expected behavior
Copied text should be the new edited content
Device (please complete the following information):

Phone Brand/Model Pixel 7 Pro
Android Version 34
App Version 89.7 google play

Works. Closing


found a color bug

yellow and red highlights turned to white color il add some photos.

this is the link to the video-https://www.youtube.com/watch?v=8MCFqGnYW4M

Try to remove the utils/chroma_correction.glsl and change gamut-mapping/jedypod.glsl to gamut-mapping/clip.glsl

ty for responding.have tried,no change.it still happens on red and yellow.

The tower is a bit orange, you can recheck this by using bt2390 or bt2446c. madvr shows orange too. "paris" and "planet" is too bright, and "ph" too colorful. so, they got pretty highly desaturation.
youtube's conversation make some blue turns purple, It can't be reference.

i have attached photos.red and yellow crushed to white on hdr toys as you can see.

Whole image of your madvr screenshot not red enough, blue not good too.
This is most ref image I think.

But yes, yellow needs some tweaking.

thanks for the detailed answer much apprecieted!
have tried your setting.much better and yet i dont get a resault like yours also i get darker shadows.added a photo.sadly yellow still white.but,the tower now much better and so as the  paris sign but again not as good as yours.do you use cutum settings perhaps?

nvm,github compresion ruin it and it basicly looks now like yours.ty for the help.i cant wait for the color adjustments.

Today's update should alleviate this issue.


[BUG] The attribute 'Version' with value '3.1.2301.0006' failed to parse.

:information_source: Info
Version: e.g. v3.7.0
:speech_balloon: Description
We have specified our msix version as "msix_version: 3.1.2301.0006" under "msix_config".
However, after running flutter pub run msix:create command, we get flutter build error as below.
'3.1.2301.0006' violates pattern constraint of '(0|[1-9][0-9]{0,3}|[1-5][0-9]{4}|6[0-4][0-9]{3}|65[0-4][0-9]{2}|655[0-2][0-9]|6553[0-5])(.(0|[1-9][0-9]{0,3}|[1-5][0-9]{4}|6[0-4][0-9]{3}|65[0-4][0-9]{2}|655[0-2][0-9]|6553[0-5])){3}'.
The attribute 'Version' with value '3.1.2301.0006' failed to parse.
Tried to run the command with extended logs : flutter pub run msix:create -v generates below output,
PS D:\StudioProjects\Client> flutter pub run msix:create -v
////////////////////////// Excluded some unused logs ////////////////////////////////////
Using "\?\D:\StudioProjects\Client\build\windows\runner\Release\AppxManifest.xml" as the manifest for the package.
MakeAppx : error: Failure at appxFactory->CreateManifestReader(manifestStream, &manifestReader) - 0x80080204 - The specified package format is not valid: The package manife
st is not valid.
MakeAppx : error: Error info: /[local-name()="Package" and namespace-uri()="http://schemas.microsoft.com/appx/manifest/foundation/windows10"]/[local-name()="Identity" and
namespace-uri()="http://schemas.microsoft.com/appx/manifest/foundation/windows10"][1]/@Version
'3.1.2301.0006' violates pattern constraint of '(0|[1-9][0-9]{0,3}|[1-5][0-9]{4}|6[0-4][0-9]{3}|65[0-4][0-9]{2}|655[0-2][0-9]|6553[0-5])(.(0|[1-9][0-9]{0,3}|[1-5][0-9]{4}|
6[0-4][0-9]{3}|65[0-4][0-9]{2}|655[0-2][0-9]|6553[0-5])){3}'.
The attribute 'Version' with value '3.1.2301.0006' failed to parse.
Cleaning up output file "\?\D:\StudioProjects\Client\build\windows\runner\Release\pos_flutter.msix".
MakeAppx : error: Failure at (CreatePackage( overwrite, hashAlgorithm, fileList, outputPath, manifestStream.Get(), forceCompressionNone, performanceOptions, encryptPackage,
encryptionOptions, cgmPath, mainPackagePathForResourceExemption, makepriExeFullPath)) - 0x80080204 - The specified package format is not valid: The package manifest is not
valid.
MakeAppx : error: Package creation failed.
MakeAppx : error: 0x80080204 - The specified package format is not valid: The package manifest is not valid.
Unhandled exception:
#0      ProcessResultExtensions.exitOnError (package:msix/src/method_extensions.dart:59:7)
#1      MakeAppx.pack (package:msix/src/makeappx.dart:28:9)

#2      Msix._packMsixFiles (package:msix/msix.dart:125:5)

#3      Msix._createMsix (package:msix/msix.dart:99:5)

#4      Msix.create (package:msix/msix.dart:57:5)

#5      main (file:///D:/Android/flutter_windows_3.3.10-stable/flutter/.pub-cache/hosted/pub.dartlang.org/msix-3.7.0/bin/create.dart:4:3)

[+34290 ms] "flutter run" took 34,489ms.
[   +5 ms] pub finished with exit code 255
[   +2 ms]
#0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
#1      _DefaultPub.interactively (package:flutter_tools/src/dart/pub.dart:418:7)

#2      PackagesForwardCommand.runCommand (package:flutter_tools/src/commands/packages.dart:249:5)

#3      FlutterCommand.run. (package:flutter_tools/src/runner/flutter_command.dart:1209:27)

#4      AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)

#6      FlutterCommandRunner.runCommand. (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)

#7      AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)

#9      run.. (package:flutter_tools/runner.dart:62:9)

#10     AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#11     main (package:flutter_tools/executable.dart:91:3)

[ +199 ms] ensureAnalyticsSent: 193ms
[   +2 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 255
:scroll: Pubspec.yaml

Pubspec.yaml file looks like below,
msix_config:
msix_version: 3.1.2301.0006
display_name: XXXXX
publisher_display_name: XXXXX
identity_name: xxxxxxxxxxxxxx
certificate_path: .\assets\CERTIFICATE.pfx
certificate_password: Welcome@123
logo_path: .\assets\images\sample.png
publisher: E = xxxx, CN = xxxx OU = xxxxxx, O = xxxx, L = xxxx, S = xxxxx, C = xxxxxx
icons_background_color: 'transparent'
architecture: x64
capabilities: 'internetClient,location,microphone,webcam'
flutter_intl:
enabled: true

Hello
You can't have build number start with <IP_ADDRESS>1, its need to start with positive number: <IP_ADDRESS>1


Rfc 454 tweaks

Hi @sklump
Can you please pluralize proposal~attach? This PR looks good otherwise.

approved on community B call on 24 Mar 2021; suggestion by @llorllale was included