Add option to include border around form groups

Some of our settings pages in dotcom contain more than one form, each with its own submit button. To identify more visually which inputs correspond to which submit button, @vdepizzol proposed adding a border around the inputs and the button. Here's an example form in Repo Settings -> Actions -> General:

The idea would be to add a border here. Note that we may also be able to remove the "Save" button's left margin in this case since it will align with the new border. Please reach out to @vdepizzol or anyone on the design team for colors, border radius, etc. The changes will need to occur in the Group component, which is defined in lib/primer/rails_forms/group.rb and lib/primer/rails_forms/group.html.erb.
See also this Figma.

We decided not to do this because it's part of a larger set of patterns we're working on for our settings pages that will be done all at once.


Handle cgroup v1 or v2 to enable resource updates

Part of #122
This test was failing in 22.04  because it has cgroups v2.  This follows same pattern as the metrics to select either v1 or v2.

same go get error again (but other integration tests passed just fine :-/)

go get is stuggling today, I will try again Monday morning.

Same go get issue today.  I will need to look deeper :-/


Text Cursor Handle Color stays Blue

I am not sure if I should open this issue, but I have tried out all Theme and InputDecoration attributes I can think of and it just does not change.
Relevant StackOverflow question

Please post the code you are trying this with.
Sample working code
import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(
      theme: ThemeData.dark().copyWith(cursorColor: Colors.red),
      home: Scaffold(
        body: MainApp(),
      ),
    ));

class MainApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Center(child: TextField(

    ));
  }
}

@Kartik1607 Not for me. Android 28, Flutter v1.3.14-pre.64.

@Kartik1607 The underlying problem was addressed in an answer on StackOverflow.


Expand documentation suggestion

Hello,
I'm loving Pocketbase, such a fast and lightweight way to make data accessible via API.
A minor documentation suggestion.  I got really confused by the expand examples in the "API Preview".  Where it says:

Auto expand record relations. Ex.: ?expand=rel1,rel2.subrel21.subrel22

I thought that rel1 was the collection, not the column. Took me ages to realise what I was doing wrong. It would have been helpful for me if the example was something like:

Auto expand record relations. Ex.: ?expand=columnA,columnA.columnC

Anyway, thanks again for the great project!
Adam.

Thanks, I'll have it mind when redesigning the collections API preview UI in relation of #376.


Multiple files for Yaml resources

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
I would like to avoid files getting too long, so I would like to split my resources over multiple files, either by using some kind of import statement in the main file, that points to other files like:
Pulumi.yaml:
name: github-actions-runners-cluster
runtime: yaml
description: GitHub Actions Runners Cluster

resources:
  resourceFile: container.yaml
  resourceGroup:
    type: azure-native:resources:ResourceGroup

Container.yaml
resources:
  registry:
    type: azure-native:containerregistry:Registry
  workspace:
    type: azure-native:operationalinsights:Workspace
  managedCluster:
    type: azure-native:containerservice:ManagedCluster

Or implicitly by including everything from all files in a folder called "resources", in addition to the main yaml file.
Or both?
Affected area/feature

Is it possible to do this today ?

Is it possible to do this today ?

Doesn't appear so per https://pulumi-community.slack.com/archives/C03E69EQRD2/p1664981326357059
+1 to feature request.

+1
@EronWright

I would really like this. My Pulumi.yaml file is enormous and unwieldy

@terekete @nstires-ctgx @desteves @jamest-pin et al., I regret that I wrote something for this in the community Slack and neglected to update this issue. Using the compiler runtime option, a script can be used to merge multiple YAML files. See this repository for an example.
https://github.com/AaronFriel/pulumi-yaml-multi-file
I'm quite partial to this style myself, as I can put the relevant configuration alongside each resource. It's definitely a "Better Helm template", in my opinion. Hopefully this workaround helps manage the complexity of your YAML configurations.
At the same time, please continue to upvote this issue though if an "in-box" solution is important to you!

@AaronFriel very cool. At first glance looks like it will do what I need.
Thank you


AttributeError: 'list' object has no attribute 'shape'

[ ] I have marked all applicable categories:

[x] exception-raising bug
[ ] RL algorithm bug
[ ] system worker bug
[ ] system utils bug
[ ] code design/refactor
[ ] documentation request
[ ] new feature request

[ ] I have visited the readme and doc
[ ] I have searched through the issue tracker and pr tracker
[x] I have mentioned version numbers, operating system and environment, where applicable:

  import ding, torch, sys
  print(ding.__version__, torch.__version__, sys.version, sys.platform)
v0.4.3 1.12.1+cu116 3.9.7 (default, Sep 16 2021, 16:59:28) [MSC v.1916 64 bit (AMD64)] win32

出错提示如下：
Traceback (most recent call last):
  File "D:\Anaconda3\lib\site-packages\ding\worker\learner\base_learner.py", line 165, in wrapper
    ret = fn(*args, **kwargs)
  File "D:\Anaconda3\lib\site-packages\ding\worker\learner\base_learner.py", line 205, in train
    log_vars = self._policy.forward(data, **policy_kwargs)
  File "D:\Anaconda3\lib\site-packages\ding\policy\sac.py", line 843, in _forward_learn
    loss_dict['critic_loss'], td_error_per_sample0 = v_1step_td_error(q_data0, self._gamma)
  File "D:\Anaconda3\lib\site-packages\ding\rl_utils\td.py", line 320, in v_1step_td_error
    if len(v.shape) == len(reward.shape):
AttributeError: 'list' object has no attribute 'shape'

reward的值为：
reward[0].shape
Out[4]: torch.Size([4096])

reward
Out[5]:
[tensor([ 0.0170, -0.0027, -0.0095,  ...,  0.0099, -0.0055, -0.0047],
        device='cuda:0')]

你的reward为什么还被list包了一层呢，检查下从环境返回的reward是不是标准格式吧，应该为np数组，且维度为(1, )

找到了，我的rew转换为np.array执行了两次：
        rew = to_ndarray([rew])


perf: Fail fast on empty query for BatchPrefillWithPagedKVCacheKernel

Small optimization for CUDA graph use cases. According to profiling, this shaves off ~10% of kernel execution time for empty queries.

Actually it looks like setting query length to 0 causes illegal memory access under some circumstances...

Thanks for doing this, I'm now looking at the illegal memory access issue.


Suggestion for "done" language

Here's a proposal for language and markup for a generic "done but not archived" message:

Nice! I think that gets the job done.


Development/native audio

Hi guys,
In this PR, I added the logic to define the volume format as we are doing in webkit. When there is native-audio, use linear otherwise cubic. This is needed for amlogic audio sink to get cubic. I guess it is ok to have this logic in cobalt too where it is already in webkit.
@neeraj-deshpande I have seen that in cobalt 20 we were using for both brcm and amlogic undef SB_HAS_NATIVE_AUDIO but migrating cobalt 21, brcm dropped that flag, so basically I aligned amlogic integration based on brcm changes. Do we need to explicitly undef NATIVE-AUDIO for broadcom too?

@pawel-stanek-ml @modeveci  Can we merge this or is there still discussion ? If I understand correctly the volume control seems to have different scales in different applications. Can we make this configurable, Picking up an environment variable we set or can we make it generic and every where the same ?

Switching to CUBIC volume looks wrong to me. Moreover it should not affect sink. Sink implementing STREAM VOLUME properly should expect values in linear scale and should do the transform it needs internally (e.g. if its scale is from 0-500 as opposed to 0.0 - 1.0). By switching to CUBIC we say to GST the passed in value is in CUBIC and it translates it to LINEAR before passing to sink. This means that passing 0.8 will result in ~0.5 in linear. But isn't YT passing 0.8 expecting 80% in linear?We can just pass whatever we see fit here in other words. Current WPE is wrong if anything (especially Webkit upstream uses linear)

To me there is nothing here to address as wrong or correct, it depends on how you implemented; either you can accept as Cubic to clamp, or leave that to sinks to decide what to do with this Linear value.
Anyway, @pwielders @pawel-stanek-ml i do not need this change now, AML changed their impl. We can remove this PR


remove a mutable variable from Molecule

Current coverage is 97.41% (diff: 100%)

Merging #50 into master will increase coverage by 0.27%

@@             master        #50   diff @@
==========================================
  Files            12         12
  Lines           663        657     -6
  Methods         583        578     -5
  Messages          0          0
  Branches         80         79     -1
==========================================
- Hits            644        640     -4
+ Misses           19         17     -2
  Partials          0          0

Powered by Codecov. Last update 54278c5...eb37839


Clean up https://github.com/mythmon/wok/pull/153

I accidentally bitrotted https://github.com/mythmon/wok/pull/153 so here's a cleaned up version.

no, wait, I did that totally wrong.

removed spurious empty commit

wait actually I broke the branch that this was from. "Yay".


Extract shared API client classes

It could be useful to extract the classes in \DoSomething\Northstar\Common so that we could use them independently or as part of other API clients.

I'm actually going to close this, since I think we'd always want this as a dependency for any Northstar-powered API clients (for the RestApiClient and the AuthorizesWithNorthstar trait).


Implementation of new mangling (but not enabled yet)

This PR contains several commits which add the implementation of the new mangling scheme.
It's mostly a NFC since the new mangling is not enabled yet (and it still needs to be wired up in several places in the compiler).
The only functional change is that the demangler can already demangle new symbol names.
For details see the individual commit messages and docs/ABI.rst, which contains the full definition of the new mangling scheme.
A quick experiment showed that the new scheme gives ~20% reduction of the trie size and ~10% reduction of the string table (nlist) for the stdlib dylib.
About 10% of the 20% trie-size-reduction are due to a more compact mangling (e.g. using word substitutions).
The other 10% are because of the reversed order of the mangling ("post-fix"). This result in more common pre-fixes in symbol names.

@swift-ci Please smoke test


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210717153547362 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210717153547362 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.569%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26721

Relevant Lines:
56890

💛  - Coveralls


VPC: serviceIpv4Cidr not being honored when deploying new cluster

Describe the bug
When deploying a new cluster and specifying a CIDR block for the new EKS cluster, the VPC defaults to a <IP_ADDRESS>/16 range instead of the specific one.
Expected Behavior
The specified CIDR block should be used for the new VPC.
Current Behavior
The cluster was created correctly, but the VPC has a CIDR block of <IP_ADDRESS>/16. (The cluster, however, has the correct service IPv4 range.)
Reproduction Steps
Set up a cluster add-ons definition like so, for example:
const addOns: Array<blueprints.ClusterAddOn> = [
      new blueprints.addons.MetricsServerAddOn(),
      new blueprints.addons.ClusterAutoScalerAddOn(),
      new blueprints.addons.AwsLoadBalancerControllerAddOn(),
      new blueprints.addons.VpcCniAddOn(),
      new blueprints.addons.CoreDnsAddOn(),
      new blueprints.addons.KubeProxyAddOn(),
      new blueprints.addons.EbsCsiDriverAddOn(),
      new blueprints.addons.EfsCsiDriverAddOn(),
      new blueprints.addons.CertManagerAddOn(),
      new blueprints.addons.AdotCollectorAddOn()
    ];

    const clusterProps: MngClusterProviderProps = {
      version: KubernetesVersion.V1_26,
      clusterName: 'foo',
      forceUpdate: true,
      nodegroupName: 'foo-ng'
      serviceIpv4Cidr: '<IP_ADDRESS>/16' // <----- THIS
    };

    const clusterProvider = new MngClusterProvider(clusterProps);

    console.log(`Deploying to cluster using CIDR block ${clusterProvider.props.serviceIpv4Cidr}`);

    blueprints.EksBlueprint.builder()
      .account(props?.env?.account)
      .region(props?.env?.region)
      .clusterProvider(clusterProvider)
      .addOns(...addOns)
      .useDefaultSecretEncryption(false)
      .build(this, `${id}-stack`);
  }

  Then deploy that cluster.

### Possible Solution

_No response_

### Additional Information/Context

_No response_

### CDK CLI Version

2.77.0 (build 06a0b19)

### EKS Blueprints Version

1.7.2

### Node.js Version

v16.13.2

### Environment details (OS name and version, etc.)

Mac 10.15.7

### Other information

_No response_

@nmonasterio We should add support for passing custom VPC range to VpcProvider which is not there today to support your custom VPC range need. You are passing service IP range and it works right by design to create services in that range. It will be a design change to support this request. This is not a bug. We can work on this change for this release or next release.

Thank you @elamaran11! Makes perfect sense. Looking forward to seeing this in action soon.

@nmonasterio The PR is merged. Please reachout if you have any issues.


Update landing page UX

Add the following optimizations for UX:

Move suggested sale items to bottom of search result list
Allow user to press "enter" to navigate to the first matching result
Move search box closer to top of page

/gcbrun


Missing classes ? when generating Java from a JSON document (not schema)

I concatenated all my JSON instances into a single file and ran it through:
sh jsonschema2pojo.sh -s all.json -T JSON -t X3D
I looked at the resulting class files and noticed that some stuff was missing.  Could you have a look?  One thing that was missing was Sphere.  JSON is here: https://github.com/coderextreme/x3djson/blob/master/all.json.zip?raw=true
Thanks!

Hi John. When jsonschema2pojo encounter an array inside an example JSON file, it uses the first array value to decide the array's type. I notice that you seem to have concatenated different JSON objects into a single array. Only the first object will be read.
We don't support arrays that contain varying types, since these would require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example that you don't think is explained by the above, it would be useful if you could reduce this to a minimal example (rather than the full 0.5Gb json file).

It’s easy to create examples, you can just take elements out of the array.  But here’s my original source for the JSON files I concatenated together. http://www.web3d.org/x3d/content/examples/X3dExampleArchivesJsonScenes.zip http://www.web3d.org/x3d/content/examples/X3dExampleArchivesJsonScenes.zip  I had to filter the JSON files that wouldn’t parse. I can provide a zip with only the parseable files.   I think however, there is a issue that you bring up about polymorphic deserialization.  We might have a Geometry class which contains a IndexedFaceSet, Box, Sphere, Cone, or Cylinder, but only one of them at a time.  I see from the generated Java code that it only has IndexedFaceSet, so your explanation is good.  Do you know any JSON->JSON schema tools which support this?  Also, I’m interested in recursive JSON schemas, if you know a tool which does both.  I have Transforms within Transforms, and I only want one result Transform class.
Thanks,
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON file, it uses the first array value to decide the array's type. I notice that you seem to have concatenated different JSON objects into a single array. Only the first object will be read.
We don't support arrays that contain varying types, since these would require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example that you don't think is explained by the above, it would be useful if you could reduce this to a minimal example (rather than the full 0.5Gb json file).
—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406.

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks.  If the attachment doesn’t go through, I can put it on github with a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON file, it uses the first array value to decide the array's type. I notice that you seem to have concatenated different JSON objects into a single array. Only the first object will be read.
We don't support arrays that contain varying types, since these would require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example that you don't think is explained by the above, it would be useful if you could reduce this to a minimal example (rather than the full 0.5Gb json file).
—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406.

Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064
.

https://raw.githubusercontent.com/coderextreme/x3djson/master/SeaStarGroup.json https://raw.githubusercontent.com/coderextreme/x3djson/master/SeaStarGroup.json
Thanks,
John

On Nov 6, 2015, at 4:47 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064
.

—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154567038.

Multiple Child*.java classes might be acceptable.  I am not so sure about other repeated classes, and not Transforms.
I can try to find an example that generates multiple Transforms, if that will help.
John

On Nov 6, 2015, at 4:47 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064
.

—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154567038.

Looks like the first example in the file has multiple transforms.  Here it is:
https://github.com/coderextreme/x3djson/blob/master/CleatClamp.json https://github.com/coderextreme/x3djson/blob/master/CleatClamp.json
John

On Nov 6, 2015, at 5:09 PM, John Carlson<EMAIL_ADDRESS>wrote:
Multiple Child*.java classes might be acceptable.  I am not so sure about other repeated classes, and not Transforms.
I can try to find an example that generates multiple Transforms, if that will help.
John

On Nov 6, 2015, at 4:47 PM, Joe Littlejohn<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406 https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
<https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064 https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064>
.

—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154567038.

Here’s an example of the worse form of nesting Transforms we have:
https://raw.githubusercontent.com/coderextreme/x3djson/master/RedPlayer.json https://raw.githubusercontent.com/coderextreme/x3djson/master/RedPlayer.json  probably way too many Child.java and Transform.java classes.
Even if you don’t handle polymorphism, can you notify me in the Java code or in an error log and I can deal with it?  Also, generate the subclasses of the unknown superclass without generating the inheritance hierarchy if possible.  I need the classes generated, please.  That will help immensely to have the classes.  I know what the superclass is, it comes from the XSD and the standard specification.  Unfortunately, so far, we are not generating the JSON schema from the XSD as there are differences in the JSON from the XML.  We prefix attributes, comments and arrays with @, #, - in the JSON
John

On Nov 6, 2015, at 5:38 PM, John Carlson<EMAIL_ADDRESS>wrote:
Looks like the first example in the file has multiple transforms.  Here it is:
https://github.com/coderextreme/x3djson/blob/master/CleatClamp.json https://github.com/coderextreme/x3djson/blob/master/CleatClamp.json
John

On Nov 6, 2015, at 5:09 PM, John Carlson<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Multiple Child*.java classes might be acceptable.  I am not so sure about other repeated classes, and not Transforms.
I can try to find an example that generates multiple Transforms, if that will help.
John

On Nov 6, 2015, at 4:47 PM, Joe Littlejohn<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406 https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
<https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064 https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064>
.

—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154567038.

Can you suggest a good JSON Schema generator from multiple JSON files suitable for your jsonschema2pojo program?
As you can see from my issues (coderextreme) I’m not having much luck with the Schema files I try. Either no Java files are generated or one file is generated
Thanks,
John

On Nov 6, 2015, at 4:47 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Yes, no attachment I don't think
On 6 Nov 2015 10:36 pm, "John Carlson"<EMAIL_ADDRESS>wrote:

Attached is a single example that doesn’t seem to do the right thing:
…no Shape, no Sphere, multiple Child classes (with underscores).
Thanks. If the attachment doesn’t go through, I can put it on github with
a link.
John

On Nov 6, 2015, at 3:22 PM, Joe Littlejohn<EMAIL_ADDRESS>wrote:
Hi John. When jsonschema2pojo encounter an array inside an example JSON
file, it uses the first array value to decide the array's type. I notice
that you seem to have concatenated different JSON objects into a single
array. Only the first object will be read.
We don't support arrays that contain varying types, since these would
require polymorphic deserialization.
This is an assumptions about what's wrong here. If you have an example
that you don't think is explained by the above, it would be useful if you
could reduce this to a minimal example (rather than the full 0.5Gb json
file).
—
Reply to this email directly or view it on GitHub <
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154545406
.

—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154564064
.

—
Reply to this email directly or view it on GitHub https://github.com/joelittlejohn/jsonschema2pojo/issues/439#issuecomment-154567038.

JSONs:
{
"person" : {
"name" : "sandeep",
"id" : 12,
"address" : {
"add1" : "ab",
"add2" : "cd",
"state" : "up",
"country" : "India"
}
}
}
{
"employee" : {
"name" : "rajesh",
"id" : 13,
"address" : {
"add1" : "xy",
"state" : "ap"
}
}
}
classes:
public class Address{
private String add1;
private String add2;
private String state;
private String country;
//setters and getters
}
public class Address_ {
private String add1;
private String state;
}
rather than creating multiple classes(with "_" symbol) is there any way to merge the existing class If we get another "address" json object in another json file.
JSONFiles.pdf

Lets discuss this on #441


Fix assault kneel check

Fixes #344

@ystros Already patched this one up in PR https://github.com/throneteki-playtesting/throneteki/pull/345. I'd rather pull that, as it tweaks the assault section as well.
Welcome to review that if you have the time :)


Fit & finish - UI Copy | Create/connect to a skill incl. orchestrator

Fit & finish - UI Copy | Create/connect to a skill incl. orchestrator
White cells = existing copy
Yellow/blue cells = new copy
Link to 'live' copy on ' Add a skill ' tab:
https://microsoft.sharepoint.com/:x:/r/teams/ConversationalAI785/_layouts/15/Doc.aspx?sourcedoc={6B1FACBD-B5FE-4AD2-A35B-995635C1696B}&file=glossary.xlsx&action=edit&mobileredirect=true&wdPreviousSession=36539654-a42b-4b87-be1b-de96507b4422&wdOrigin=TEAMS-ELECTRON.teams.undefined&cid=676cf454-61e0-4182-bd27-39044bb51cea

Shiproom - approving as today is fit and finish day.  must be merged by EOD today

Shiproom - approved.  @beyackle can you get this merged ASAP please?  thank you


Handheld Game Console Controller Support

Integrate in a generic-handheld image handygccs

Started the work fork drokath/bazzite Would love some assist :-)


tour: code or compiler bug at page flowcontrol/10

Context: http://<IP_ADDRESS>:3999/flowcontrol/10
It is Friday. And if I change code to
switch time.Sunday { ... }
it returns "Too far away." ... but correct answer is "In two days."

The time is fixed on go tour, see the note

Note: Time in the Go playground always appears to start at 2009-11-10 23:00:00 UTC, a value whose significance is left as an exercise for the reader.


Update to include Quantum Computing

Needed for open-source Quantum Computing R&D code development.

Neat, thanks for adding this. I get a 404 error when I try to access the https://github.com/CDCgov/QuantumComputing project. Would you please confirm that the project is configured properly for public access and update the pull request?


docs: update github_repo usage and example

This fixes a few outdated calls and adds a little extra detail.

Closes #136


Compile from the Xcode error - library not found for -lAFNetworking

Hello,
The react-native-lock is working fine in simulator only when we open the from react-native run-ios.
Once we open the project Xcode. And try to compile from the Xcode. we have this error.
ld: library not found for -lAFNetworking
Please advise.
Thanks

Anyone still support this? :)

You  are supposed to use it from react-native cli since using it from Xcode requires a bit of knowledge of how cocoapods and Xcode works. If you really need to use it please try opening the .xcworkspace file since I think you opened the .xcodeproj.

Hello @hzalaz ,
Thanks for the advice. :)

Another way that worked for me so that I could compile in xcode was to add the ios/Pods project to the Library folder.

I'd avise to avoid adding the CP folder to the Xcode project since it could break it if you are not careful enough. The safe way is to use workspace file

@hzalaz thanks for the advice. i'm very new at this stuff and don't want to lead anyone down a bad path :)


Elm Online Editor indicates Error for  both Graphics.Element and Graphics.Collage Libraries

Problem:
Same error below for both Graphics.Element & Graphics.Collage
Error Message indicate in Elm Online Editor;
I cannot find module 'Graphics.Element'.

Module 'Temp1473230001580255' is trying to import it.

Potential problems could be:
  * Misspelled the module name
  * Need to add a source directory or new dependency to elm-package.json

Expected Behavior:
Expected the code example from gitbook: Elm Example Graphics API Page to work.  There is even a link on the page to the Elm Online Editor.
Request:
Please add a FAQ covering Online Editor Limitations/Restrictions.
If this reported issue is a bug please fix it accordingly.
(I searched for info on Elm Online Editor regarding Graphics API and could not find any info on the subject).
Environment Info:
OS: Win 7
Browser; Chrome Version 52.0.2743.116 m
Elm Online Editor: via both link on indicated in link above and going directly to http://elm-lang.org/try
Elm: 0.17,1

This pretty old, any update on this issue

It appears that whatever book you are using has not been updated for 0.17, so it assumes the /try page works just like it did previous to that release.
At this time, the /try page has a couple packages available by default, but it doesn't have elm-graphics. You can use it from here though.


[Feature] support scan_rows limitation for external table

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #8028
Problem Summary(Required) ：

@Mergifyio backport branch-2.3


SSIM / MS-SSIM implementation

This is not really an issue, but might be helpful for you:
While checking out your (awesome) code, I just discovered this TODO:
# TODO: Add SSIM error http://www.cns.nyu.edu/pub/eero/wang03-reprint.pdf
I was looking for an SSIM/MS-SSIM implementation in TensorFlow for a while. Because I could not discovery anything, I ported some Matlab/C++ code to TensorFlow. I've posted the code on StackOverflow, so feel free to use and improve it:
Stack Overflow: SSIM / MS-SSIM for TensorFlow

Hey @bsautermeister, thanks for the link! I'll check that out and integrate it when I have a chance. Or, if you'd like to add it in yourself and submit a pull request, I'd be happy to accept.

I will, as soon as I find time for that.
By the way: I'm not sure, but after comparing your PSNR implementation with it's formula in the paper and the PSNR Wikipedia-Definition for colored images:

For color images with three RGB values per pixel, the definition of PSNR is the same except the MSE is the sum over all squared value differences divided by image size and by three.

I think that you miss the by three. As you are using the formula for colored images and take the sum over all 3 dimension (tf.reduce_sum(...,[1, 2, 3]), I think you should consider the 3rd dimension for your num_pixels as well:
num_pixels = tf.to_float(shape[1] * shape[2] * shape[3])
Or am I wrong? :)

Good catch. Fixed!

I know this is very very late but I wanted to get the SSIM @dyelax or @bsautermeister, did one of you got the chance to integrate it with the code, or Is there any other way to calculate the SSIM for this paper like for example, compare_ssim. I would really appreciate a reply. Thank you for this helpful code @dyelax


Examples link broken

Getting 404 clicking on Examples link in the home page.
I saw this commit https://github.com/Bottr-js/Bottr/commit/c692bc8ed8378b9ab59bc8c1e0ca4aaf469c1551, but it is not published to middleman/build/.

@albertosantini Good catch, will fix this straight away.

still broken dec4
https://bottr.co/examples
404
https://goo.gl/photos/3ULoYRFSU2RnDXn16

Still broken. Dec 12.

Closing this, because it's a duplicate of #44.


If we have space for a second meme that triggers for Builder's TECH (%) above 66%

New meme

https://imgflip.com/i/573hmz
Can anyone make a better meme?

Some ideas:


bugfix in ServerRequest::getUriFromGlobals() method for url https://site.com/path?continue=https://site.com/widget?param=1

Fixes #151

could you please add a test case. then it's good to merge.

Nvm, I add the test. Thanks for the PR.


note that usage requires import polars_ds.extensions

I was expecting to be able to do import polars_ds - instead, I need to do import polars_ds.extensions to be able to use .num_ext

fixed. thank you for dropping by!


Update to use page.emulateNetworkConditions

This patch updates all call sites of the
Network.emulateNetworkConditions CDP command to the
new page.emulateNetworkConditions api on page.

@mathiasbynens I addressed your comments.

@mathiasbynens Awesome, thanks! Can you merge these changes? I don't have write access...

I think @addyosmani might want to take a look


Favicon

Add favicon as per https://crn.com/
See https://www.hlx.live/developer/favicon

Hi I would like to take this up, please assign it to me


CompositeByteBuf.addComponent always assume reference count ownership

Motivation:
The current interface for CompositeByteBuf.addComponent is not clear under what conditions ownership is transferred when addComponent is called. There should be a well defined behavior so that users can ensure that no leaks occur.
Modifications:

CompositeByteBuf.addComponent should always assume reference count ownership

Result:
Users that call CompositeByteBuf.addComponent do not have to independently check if the buffer's ownership has been transferred and if not independently release the buffer.
Fixes https://github.com/netty/netty/issues/4760

@normanmaurer @trustin - PTAL

@trustin WDYT ?

@trustin WDYT ?

@trustin - I'll pull in tomorrow if there are no objections / concerns.

@Scottmitch ship it.

Cherry-picked 4.1 (6312c2f) 4.0 (998b8a8)


Should admin lock out timer also reset 2fa?

Feels slightly weird to get prompted to re-login but no 2fa anyway (that said I notice a lot of sites are like this ex. DO, though not all)
The password is likely autofilled as well so it's basically an exercise in pressing a button to log back in, what's the point :P
Some sort of hasSolvedTFA: 30 days kinda option in ACP maybe?

This sounds familiar actually. I'd think a smoother flow would be to have the plugin bypass relogin and challenge with 2FA, since I imagine 2FA is more secure

Sounds good, just rechallenge.


Loaded datetimes with timezone offset cannot be pickled correctly

I've tried pickling an object loaded from a toml file that contains a datetime with a timezone offset.
Here is a minimal example:
>>> import toml, pickle
>>> toml_content = """
dt_with_tz = 2001-01-01 00:00:00+01:00
"""
>>> loaded_from_toml = toml.loads(toml_content)
>>> pickled = pickle.dumps(loaded_from_toml)
>>> print(pickle.loads(pickled))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: TomlTz.__init__() missing 1 required positional argument: 'toml_offset'

I have not looked deeper into the pickle data or how pickle accesses sub-objects. And yes, I know that this is a great combination, TOML+pickle.

I'm encountering the same error message when trying to read a toml file with Ansible.
To install both ansible and toml I use a simple requirements.txt in a Dockerfile:
ansible==6.5.0
toml==0.10.2

FROM python:3-slim
COPY requirements.txt .
RUN pip install -r requirements.txt

With this file at filter_plugins/from_toml.py I define Ansible modules called to_toml and from_toml:
#!/usr/bin/python

DOCUMENTATION = '''
---
module: to_toml, from_toml

version_added: "2.8"

short_description: Converts Python data to TOML and TOML to Python data.

author:
    - "Samy Coenen (contact@samycoenen.be)"

'''

import toml

def to_toml(data):
    ''' Convert the value to TOML '''
    return toml.dumps(data)

def from_toml(data):
    ''' Convert TOML to Python data '''
    return toml.loads(data)

class FilterModule(object):
    ''' Ansible TOML jinja2 filters '''

    def filters(self):
        return {
            # toml
            'to_toml': to_toml,
            'from_toml': from_toml
        }

Then I try to read a toml file and set its content as a fact with the following tasks using my custom from_toml module:
- name: Slurp existing runner config again after registration
  slurp:
    src: "/etc/gitlab-runner/config.toml"
  register: config_file
  when:  runner_registration.changed

- set_fact:
    runner_config: "{{ config_file.content | b64decode | from_toml }}"
  when:  runner_registration.changed

The file I'm trying to read (/etc/gitlab-runner/config.toml) looks like this:
concurrent = 1
check_interval = 0

[session_server]
  session_timeout = 1800

[[runners]]
  name = "zenyatta"
  url = "[REDACTED]"
  id = 176
  token = "[REDACTED]"
  token_obtained_at = 2022-11-03T14:23:18Z
  token_expires_at = 0001-01-01T00:00:00Z
  executor = "shell"
  [runners.custom_build_dir]
  [runners.cache]
    [runners.cache.s3]
    [runners.cache.gcs]
    [runners.cache.azure]

I assume token_obtained_at and/or token_expires_at to be the culprit.
If desired I can build a repository reproducing the issue.

Funny, I hit the same issue with the Salt-stack and gitlab-runner/config.toml file. Well, as the parser underneath is the same...
The biggest question is - when we can expect the release of the library that addresses this issue(being there for more, than 6 years, thanks GitLab for exposing it).


fix(twitter): update package and follower response type to Profile

Title: Update TwitterScraper Package Follower Response Type
Description:
Updates the Twitter Scraper package to use: https://pkg.go.dev/github.com/imperatrona/twitter-scraper and updated the follower scraping functionality to use the correct return type from the imperatrona/twitter-scraper package. This change:

Fix login error with changes to Twitter verification endpoint
Switches from masa-finance to imperatrona twitter-scraper package
Updates return type to []*Profile to match the package API
Maintains backward compatibility with existing functionality

Testing:

Verified follower scraping works with new type system
Confirmed compatibility with existing code paths

@mudler this affects https://github.com/masa-finance/tee-worker/pull/13


Installation variant ?

Hi there,
On macOS 10.12.2, , I had to use the following command to install automake with brew instead of autoreconf. Don't know if it's specific to the platform.

Unfortunately I am not sure which version is more faithful. I just find the installation instructions for my OS version on the Internet.
In your system compilation working with installed packages?

Closing while no new information provided. Not an issue


Add unit test to mainAsync

Adds a simple unit test for the mainAsync function.
Mostly mocks the dependencies on the file system and utility modules. This allows to easily test the mainAsync with a snapshot during local development.
Please note that I was not able to accurately verify that this is the expected result. I appreciate if any reviewer put special emphasis on the accuracy of the output.

Seems like the tests fail; you're mocking the FS but also using the FS.

Fixed. I didn't occur to me to run the rest of the tests, I think it will work now.

Seems like I was wrong. path is showing as null but I don't understand why. I'm not mocking path.


Making Rule Object Updatable

There was a certain amount of clunkiness with modifying a cube's rule. The only way to do so is to get the entire text of the rule, make the changes, and then pass the text back to the rule. That method is especially difficult for adding rule statements when there are feeder statements present.
So I added functions "add_rule_statements" and "add_feeder_statements" to the Rule Object so that the Rule file can be modified, rather than having to be completely redefined. In the pull request, there are two options of functions to create this feature, I included both and leave it up to the maintainers to decide which one (or create a new one) is better.
Thank you!
Ben Dunleavy

Thanks for the contribution @bennybooo22.
I think we can use the first version of the function. Calling the init_analytics function should be quick given that must rules have < 1000 lines of code.
I only have two small change requests.

It would be nice if the function only adds statements that don't already exist in the rule. This check would have to be case and space insensitive due to TM1 being case and space agnostic on element names.

It would be nice to have a test case for the function. You can add some test cases here: https://github.com/cubewise-code/tm1py/blob/master/Tests/Cube_test.py

Closed: Subject to Future Re-evaluation


Allow configuration of CSV format validators

This PR  allows a user to configure high level CSV validators, and override/remove the existing default CSV header validator.
In order to not introduce a breaking change, the existing default CSV header validator is configured by default.

Just updated empty_headers.rb to address a linter error

The specs are passing on all released solidus versions - just failing on solidus master!
The linter is failing on files unrelated to my changes.


rotation strange bug ?

Hum why when
sprite.rotation = "1" ;
it's not same than
sprite.rotation = 1;
but
sprite.rotation = "1.1" ;
its same like
sprite.rotation = 1.1
thanks

Its not a bug, its js. please dont pass strings there.

ok good to know so i nned to do Number(valur) to be shure

rotation = +"1.1" also works.


How does the plugin get access to the endpoint

I tried to manually make a GET request via Postman, but my access is forbidden by Cloudflare. So I was wondering how the plugin manages to gain access.

the plugin manages to gain access because the server is tailored and hardened to accept requests from a specific range of acceptable runelite client versions that are present in the user agent of the get request, if you want to see a sample of the returned data I can show it to you here.

the plugin doesn't do anything special to gain access, but the server decides whether or not the GET request is legit

I tried with the user-agent Runelite Wiki Scraper/1.0 (+abex@runelite.net), but that didn't work so probably it is not in the list.
I was wondering why the cloudflare is protecting. Can't every machine make requests with a custom user agent?

requests cost money, cloudflare is limiting the number of requests and monitoring the authenticity of the requests to save me money since the server isn’t free


Chat messages are posting days after I originally sent them

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

I sent several messages in my NewDot chat with Chronos on Thursday, Sept 14 (The messages I sent included Help, Ooo help, and a few others.
I saw the messages show up in the chat with a timestamp, but I didn't get responses from Chronos right away

Expected Result:
I either should have gotten responses from Chronos right away, or if I wasn't online, the app should have shown my messages at 50% opacity.
Actual Result:
Looking back at the chat, I can see that the messages no longer show up on the day I originally sent them, and now they show as posting over the next few days with a later timestamp. Ooo help was posted on Sept 17 and Help was posted on Sept 18.
I noticed this same behavior a week or two ago in another chat (either a group DM or an #admins chat), but thought I may have been imagining things.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @sakluger
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695028984668469
View all open jobs on GitHub

Happened to me too

Aren't these associated with the range of performance issues we're working on? I cannot specifically reproduce this at any given point in time so I don't think it makes sense to create a bug report for it. @sakluger do you agree?

I cannot specifically reproduce this at any given point in time so I don't think it makes sense to create a bug report for it.

It happened as recently as two weeks ago and it's happened to two of us, at least. I posted in the Slack thread and shared with the #expensify-bugs room, let's see if anyone else has experienced this more recently

Ok perhaps I can assign this to you to manage then @mallenexpensify cause it keeps going overdue in my queue and I am honestly not sure how we're going to reproduce it. Ideally, you reopen when you experience it and get someone to look in short order and maybe the logs will help them then.

Sounds good, I'll likely keep 👀 on it, not be able to repro then ????

Tried to do some testing, with no luck.  ie. sending messages then backgrounding app quickly.

Argh, only way I can think to test this is to be out n' about and sending a bunch of messages, like during offshore.  That's not likely to happen soon so I'll close for now


Add method MessageEntity::user_mention

Issue: https://github.com/teloxide/teloxide/issues/547#issuecomment-1107530302

I think we should also add shortcuts for other kinds of entities, so you don't have to write MessageEntityKind every time. Would you mind implementing them?

No problem, I'll open a PR later.


Broken: .Results = 0 with SearchCollection (always)

Hi
looks like search movie collections is broken. If i search a movie collection like "James Bond", the
SearchContainer.TotalResults = 1, but
SearchContainer.Results = 0.
This has worked in older releases.

Which version are you using?
It works on my end, using the master branch (as well as async).

I'm using the Nuget version. It has been worked a long time, but now is broken.

Which version. 0.8.2 ?
Mvh.
Michael
On Mon, Jul 13, 2015 at 8:07 PM, DanCooper<EMAIL_ADDRESS>wrote:

I'm using the Nuget version. It has been worked a long time, but now is
broken.
—
Reply to this email directly or view it on GitHub
https://github.com/LordMike/TMDbLib/issues/139#issuecomment-121009117.

The lastet available on Nuget.

And is that 0.7.7 (last stable) or <IP_ADDRESS>-alpha (latest prerelease)?

0.8.x alpha

I am unable to reproduce with <IP_ADDRESS>-alpha. Can you provide example code?
I tried with:
TMDbClient client = new TMDbClient("c6b31d1c....");
SearchContainer<SearchResultCollection> result = client.SearchCollection("James Bond");
Console.WriteLine(result.Results.Count);

Sorry, it works now. I have no idea why the same code has nod worked today.


bug: Chiado Reset Warning Should Be On Chiado Page, plus two other suggestions

What
See https://docs.gnosischain.com/about/networks/
At the bottom of the page, there is a "Chiado Reset" warning.  Shouldn't that be on the Chiado page?:
https://docs.gnosischain.com/about/networks/chiado
If somebody goes directly to the page (I've done this), they won't see the warning.
Also, would it be helpful if the reset warning gave the old Chain ID and the new Chain ID?
And, right now it says to point your node to the new RPC endpoint.  Would it be a good
idea to give the old RPC endpoint?  For example, "change your RPC endpoint from rpc.foo.bar
to rpc.chiadotestnet."

Added here: https://docs.gnosischain.com/about/networks/chiado#previous-iterations


ImportError: Could not import IENetwork

I followed the build instructions on the dldt Github page: https://github.com/opencv/dldt/blob/2019/inference-engine/README.md for building with OpenMP threading.
This was the cmake  command used:
cmake -DCMAKE_BUILD_TYPE=Release -DTHREADING=OMP ..
When I try to run the inference script, I run into the following error:
Traceback (most recent call last):
from openvino.inference_engine import IENetwork, IEPlugin
ImportError: cannot import name 'IENetwork' from 'openvino.inference_engine' (unknown location)
I made sure the built inference engine libraries and bin files were in LD_LIBRARY_PATH, PATH, and PYTHONPATH env variables, but still run into this error. This is on a Linux system and using python.
Any help is appreciated!

Dear @karkadad
Sorry you're having trouble. I will attempt to reproduce your error and report back on this forum.
Thanks,
Shubha

While i am concerned with your unknown location error. Can you tell me if you were able to get a  python sample to run ?
I built with OpenMP enabled and was able to get openvino running correctly.

I was able to get a python sample to run with the pre-built OpenVINO libraries which I believe is built with TBB by default.
I then tried to build OpenVINO myself with OpenMP and these are the commands I used:
cmake -DCMAKE_BUILD_TYPE=Release -DTHREADING=OMP ..
make -j
I then made sure the built inference engine libraries and bin files were in LD_LIBRARY_PATH, PATH, and PYTHONPATH env variables. I'm not sure if I'm missing any steps here.
But when I then try to run the python sample, it exits with the error :
Traceback (most recent call last):
from openvino.inference_engine import IENetwork, IEPlugin
ImportError: cannot import name 'IENetwork' from 'openvino.inference_engine' (unknown location)

Inside of the ie-bridges folder there should a python folder that has some samples. You should be able to run those samples.
There is also a readme in that folder that shows what directory to add to your pythonpath.

also did you build with the -DENABLE_PYTHON flag ?

I had tried both with and without the -DENABLE_PYTHON flag while building with the following commands:

cmake -DCMAKE_BUILD_TYPE=Release -DTHREADING=OMP ..
cmake -DCMAKE_BUILD_TYPE=Release -DTHREADING=OMP -DENABLE_PYTHON=ON -DENABLE_MKL_DNN=ON ..

But I think my issue was that I wasn't adding the right directory to the PYTHONPATH.  Adding /bin/intel64/Release/lib/python_api/python3.7 to the PYTHONPATH env variable seems to have fixed the issue. Earlier I was only adding /bin directory to the PYTHONPATH which seems to be the issue?
Thank you @shubha-ramani and @KMC-intel for your help! I will be closing the issue now.


Update README.md

Website URL fixed. You had put repo URL in place of deployed website gh page.

@ahmadsohail404

Thanks :)


[MINOR] fix(maintainers): use real names in Maintainer table

What changes were proposed in this pull request?
use real names in Maintainer table
Why are the changes needed?
use maintainers' real name
Does this PR introduce any user-facing change?
no
How was this patch tested?
no need

@justinmclean please help to review.


TypeError: Cannot read property 'padEnd' of undefined

Bug description
This error happens a few dozen times a day on prod, so I can't easily reproduce it locally.
Example stack trace:
TypeError: Cannot read property 'padEnd' of undefined
    at Object.Jf [as getErrorMessageWithLink] (/project/webpack:/engine-core/src/Engine.ts:109:32)
    at yw.handleRequestError (/project/webpack:/engine-core/src/NodeEngine.ts:1209:11)
    at /project/webpack:/engine-core/src/NodeEngine.ts:1007:36
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at $w.request (/project/webpack:/home/circleci/project/node_modules/@prisma/client/runtime/src/runtime/getPrismaClient.ts:1225:24)

We're using webpack, and looks like os.platform is undefined.
The actual error is not included in the stack, as the error is in the error handling :)
It would be nice if general errors like this could be caught in the error handling flow so the original error stack can be seen.
Prisma information
Prisma 2.9.0 on Node 12.18 (AWS lambda).

Hi @royra
Thanks for reporting this. I attempted a simple reproduction with webpack and wasn't able to reproduce this.
So I would like to wait for someone who encounters the same and have a proper reproduction so that we can identify the bug.

@pantharshit00 thanks for the quick response!
Looking at the code again, this may not be related to webpack.
The code at NodeEngine.ts (line 1209) is calling getErrorMessageWithLink before this.platform is initialized - looks like a bug in the error handling. It's masking the actual error which is intermittent.

cc @williamluke4

Will have a look tomorrow :)

@williamluke4 any progress? we're still affected by this - it masks real errors making debugging on our side difficult. LMK if you want me to try and open a PR to fix.

Hey @royra, Thank you for reporting this. I've implemented a quick fix which should make it into the next release. Please ping me if this persists after 2.13 is released :)


Allow use rule with multiple files output for labels and attributes image attributes

I want to add two kind of labels:

labels for image lifecycle management - on this labels change image must be rebuild;
labels for image stamping (from which Git revision I can build same image) - this lables change everytime and should be excluded from cache key.

This modification allow to me create two labels file and pass it by custom or filegroup rule:

labels.txt - with labels for image lifecycle management;
labels_volatile.txt - with stamping labels.

After that I can add labels_volatile.txt to omitted_inputs srubbing rule and cache will ignore every labels_volatile.txt file change.

I am not sure i follow, I don't think this change matches the description. Can you please file an issue first so we can discuss it before agree to the change?


Jikkyou Powerful Pro Yakyuu 2013 [PCSG00207]

App summary

App name: Jikkyou Powerful Pro Yakyuu 2013
App serial: PCSG00207
App version: 1.00

Vita3K summary

Version: v0.1.5
Build number: 2789
Commit hash: https://github.com/vita3k/vita3k/commit/[0cc549e1]
CPU backend: Dynarmic

Test environment summary

Tested by: nishinji
OS: Windows 10
CPU: Ryzen 9 3950x
GPU: RTX 3060ti
RAM: 32 GB

Issues
Game displays a black screen
Screenshots

Log
vita3k.log
Recommended labels
Bootable

some sound missing and graphics bugs


Latex extractor

Description of the new feature / enhancement
I would like to have the function of highlighting a mathematical formula and being able to extract the corresponding LaTeX code spontaneously and easily. I think that it  works as a text extractor but for mathematical formula
Scenario when this would be used?
I use it to create my summary from Engineering and math books. I use always in my relation . In my project reports so that I don’t have to copy and paste and manage image formatting within the text.  In the end i would use the latex extractor function to make excellent flash cards.
Supporting information
https://github.com/lukas-blecher/LaTeX-OCR

Implementation Route That Might Align With Microsoft Goals
LaTeX-OCR could easily be a custom user-defined menu item added to the bottom of the extendable menu in my #25197 proposal:
 or

Yes, I think it's a great idea

Yes, I think it's a great idea

I formally protest being marked as spam.
I plead guilty on that I wrote extremely large walls of text -- it indeed definitely produced confusion.  As a person born deaf, I type 140WPM and often write excessively long walls of text.

However -- I think most at Microsoft appreciates that I'm linking "related feature requests" together.

So as a refinement, how should I adjust my public service of linking potentially related issues together?  This is one of my shining abilities; this is something that most venues (outside of here) tends to encourage.

Appreciated on refining my etiquette.  What I do in other venues is highly appreciated by some of the companies I work with (e.g. collaborating on research papers that I am credited in), due to my insight on being able to link issues together --

Next time I see a potentially related stuff that involves capturing parts of the screen; how should I string these together?  Post just one sentence, instead of quoting a screen?  Etc.

Can you advise me?

(I do plead guilty for writing excessively-long texts -- that is a habit I have)

Yes, I think it's a great idea

See above;
@crutkas I am extremely dissapointed at being marked as spam, because here, there is a positive response that was related to their very need (see above by @simvig01)
Due to my high reputation in my industry (more than 25 peer reviewed research papers credited), I have sent you an email -- please check it -- to negotiate proper github etiquette and retroactive edits necessary to remove the Spam marker -- to protect my reputation;
(EDIT -- longer post removed and forwarded to @crutkas -- need to negotiate github etiquette going forward, as a deaf person who naturally writes giant walls of text -- and was not aware of having annoyed @crutkas -- despite users enthusaic about my posts on githubs -- including over 100 upvotes for #25595 of my suggestion.  I would like to be given a chance to edit / etc before being marked as spam as I don't think the wide audience thinks I am spam -- please reply to my email and let me know how to edit my posts to string them together better)

This issue is with the underlying OCR Engine and fixing it is beyond the scope of this project. Working on adding the ability to use Tesseract, which has an equation detector language.
/dup #20899


Improvements around setting up development environment and running nodes

add hardware requirements and details for fast catchup, including requirement for non-archival node and troubleshooting in case of failure
Slight refactoring of the devenv setup

Split sandbox on a new page
Recommend installing the Algorand software even when not running a node
Link to the list of API services

minor:

add missing blank lines in AVM document

Looks good. I'm a fan of introducing the Sandbox sooner as it's pretty key these days. Also encouraging to install locally is a good idea! I'll wait for another pair of eyes before merging.


Add translated attributes to arrayable attributes

This includes the translated attributes in any Array or JSON representation of a translated model. This means that returning a translated model will render including the translated attribute.
Fixes https://github.com/vinkla/translator/issues/1

LGTM. Thanks!


test: add more system test of instance

Add system test for following Instance APIs methods :

insertInstance
deleteInstance
getInstance
listInstances
updateInstance
stopInstance
startInstance
setShieldedInstanceIntegrityPolicyInstance
simulateMaintenanceEventInstance
setDeletionProtectionInstance
setLabelsInstanceTest
setMachineResourcesInstanceTest
setMachineTypeInstanceTestsetMachineTypeInstanceTest
setMinCpuPlatformInstanceTest

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@07a1626). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #3   +/-   ##
=========================================
  Coverage          ?   37.18%
  Complexity        ?    15148
=========================================
  Files             ?     1660
  Lines             ?   192858
  Branches          ?    28404
=========================================
  Hits              ?    71716
  Misses            ?   112814
  Partials          ?     8328

Continue to review full report at Codecov.

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


Add dependencies to setup.py

You kind of assume that when you install a released version through pip all dependencies get pulled in as well

I completely agree.

Fixed in v0.1.3.

Thanks @Tyrdall !


Add canonicalize loader

Q
A

Type
improvement

BC Break
no

Fixed issues
https://twitter.com/CiaranMcNulty/status/1384133062179520519

Summary
This PR provides a canonicalize loader that transforms the content of an XML document to a normalized version.
This canonicalized version can be used to check if 2 XML files are the same.

This PR got messy. So I've split it up in some sub issues:

#13 : DOM traverser
#15 : Improved xmlns handling
#16 : Basic canonicalize loader
#17 : Optimize namespaces traverser.

Keeping the PR for reference until it's done


Declared attributes aren't supported

I'm using SQLAlchemy's declared_attr decorator to add columns and relationships.
However, the decorated columns and relationships are completely ignored.
Here is a very simplified reproducer:
from sqlalchemy.ext.declarative import declarative_base
from sqlalchemy.ext.declarative.api import declared_attr
from sqlalchemy.orm import relationship
from sqlalchemy.schema import Column, ForeignKey
from sqlalchemy.types import Integer

Base = declarative_base()

class Person(Base):
    __tablename__ = 'people'

    id = Column(Integer, primary_key=True)

class EmailAddress(Base):
    __tablename__ = 'emails'

    @declared_attr
    def owner_id(self) -> Column:
        return Column(Integer, ForeignKey('people.id'), nullable=False)

    @declared_attr
    def owner(self) -> relationship:
        return relationship(Person)

me = Person()
email_address = EmailAddress(owner=me)

This is what mypy says:
$ pipenv run mypy decl_attr.py
decl_attr.py:30: error: Unexpected column "owner" for model "EmailAddress"

@ckarnell Yes, this would require some updates to the plugin. You can try submitting a PR.

Any updates on this feature request?

Any updates on this feature request?

Is there at least a workaround to suppress these errors other than # type: ignore for every single line?

@aydumoulin not sure how your example was supposed to work, if you try to set the @declared_attr property you get an error.
class Account(Base, BaseModel):
    __tablename__ = "accounts"

    name = Column(String(512), nullable=False)

    def __init__(self):
        self.external_id = "test"
             ~~~~~~~~~~~
             Cannot assign member "external_id" for type "Account"
               Property "external_id" has no defined setter - Pylance(reportGeneralTypeIssues)

@wbobeirne thanks for trying it out and your feedback
As a sidenote In editor Pylance type checking sometimes produces errors due to lack of mypy plugin support. Is mypy producing the same errors when ran on this file?
This is an expunged version of the solution I use in our codebase. The following code runs in production and passes mypy and pylance.
from typing import TYPE_CHECKING, cast
from uuid import UUID

from sqlalchemy import Column

if TYPE_CHECKING:
    CStr = Column[str]
    CUUID = Column[UUID]
else:
    CStr = str
    CUUID = UUID

class _TypedBase:
    """typed base for mypy purposed"""

    external_id: CStr
    revision_id: CUUID

class _BaseModel(_TypedBase):
    """
    In order to defined common foreign key we need to declare it in a mixin class
    -> Columns with foreign keys to other columns must be declared as @declared_attr callables on declarative mixin classes.
    """

    @declared_attr
    def external_id(self):
        return Column(String(512), nullable=False)

    @declared_attr
    def revision_id(self):
        return cast(CUUID, Column(UUID, ForeignKey("revisions.id", ondelete="CASCADE"))

class User(Model, _BaseModel):
    """user"""

    __tablename__ = "users"

    email = Column(String, nullable=False)
    first_name = Column(String(254), nullable=False)
    last_name = Column(String(254), nullable=False)
    deleted_at = Column(DateTime, nullable=True)

    def __init__(self):
        self.external_id = "test"

See the hover tooltip


Post action body must be JSON

Post to callback url does not succeed due to InvalidPostActionBody. However the posted body is valid JSON and the Content-Type header is set to application/json;charset=utf-8.
It seems like the problem is a strict match of the Content-Type to application/json, which fails when the charset is specified in the Content-Type header.
Could this comparison be the problem?
https://github.com/benkehoe/sfn-callback-urls/blob/f82c6039c794094c3da0bd4e91935317a0a40ca6/src/sfn_callback_urls/post_actions.py#L49
Example Post and log message:
Post to Callback:
Content-Type: application/json;charset=utf-8
{
    "source": "foo",
    "id": "bar",
    "state": "success",
}

Log of ProcessCallbackFunction:
{
    "timestamp": "2020-12-11T09:56:14.576580",
    "decode_time": 0.29622951199996805,
    "transaction_id": "xxx",
    "action": {
        "name": "post",
        "type": "post"
    },
    "error": {
        "type": "RequestError",
        "error": "InvalidPostActionBody",
        "message": "Post action body must be JSON"
    }
}

Thanks, I'll take care of this. Also a good time to update the Python runtime and dependencies.

This is fixed, both in the source and in the SAR app. Funnily enough, I was already parsing this correctly for Accept headers.

This is fixed, both in the source and in the SAR app. Funnily enough, I was already parsing this correctly for Accept headers.

Cool, thank you! 🙌

Cool, thank you! 🙌


Add Gradio authors and Blocks event

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Table: improve typing for columns

https://github.com/iTwin/iTwinUI-react/pull/940 was a hacky workaround to fixing build errors for users, and it even resulted in our own vite playground failing when running tsc. Need to investigate this issue and have proper types (without any).

Build must pass even if user doesn't have a strong type for columns.
The user must NOT be required to cast their columns using as Column[].
Vite playground build should ideally reintroduce tsc (done) and pass (playground build currently does not run in CI done).

Same with storybook ideally.

Table stories should remove {...args} because it hides TS errors.
There must be a proper test for more complicated permutations of the Table in a testing app (e.g. nextjs or vite, via playwright).

This issue has multiple parts:

Give a type error when a top-level header is passed (to reinforce that we don't support multi-level headers) (https://github.com/iTwin/iTwinUI/issues/886#issuecomment-1362950474). I worked mostly on this part.
The other parts are listed in the issue description and other comments.

I considered a couple of approaches for part #1:

To cut-paste the code from @types/react-table to itwinui-react (base-react-table.d.ts) and remove dep on @types/react-table

Users' apps don't depend on react-table; but they download it since we depend on it. So if we don't depend on it, they won't download it. Thus, Column will not be available and they cannot typecast their columns to Column[]. Thus, we can only remove this dep when/if we don't need users to use types from react-table.
Also, if we depend on @types/react-table, then users also indirectly depend on it. So when they download the dependency, their version includes the columns in ColumnGroupInterface even though we exclude it in our version. So I'm not sure how to exclude something that has been declared in the base package. Or maybe I misunderstood how TS works.

In react-table-config.ts, override columns to type of undefined or similar by overriding ColumnGroupInterface inside of react-table-config.ts

This didn't work since same-name variables should have the same type.

Add override.d.ts file

This allows to add on to the types in @types/react-table. It doesn't allow to exclude declarations already mentioned in @types/react-table.

Having a local dependency of @types/react-table on an edited folder version of the @types/react-table that excludes columns from ColumnGroupInterface (using yarn add file:src/types/react-table)

But local dependencies are not accessible by the users of itwinui-react. So they can't import Column to cast columns as Column[]

There might be a way to export react-table types from itwinui-react, but didn't try that too much. So we export Column, FieldType, etc. directly, outside any namespace, similar to how we export component props using the barrel. (not sure if this require lots of changes to the index.d.ts type file).

As mentioned, I did not work too much on part #2. But here is one observation I made:

Not passing accessor in all columns seemed to eliminate the need for casting columns to Column[]. I don't know how that worked before because I was not able to reproduce it now.

I am unassigning

reopening because:

we will not be able to migrate to tanstack table in time for v3
we can make breaking changes to types in v3, so we can duplicate react-table types into our package and fix any issues.
there are issues with old columns format (#1428)

there is a new satisfies keyword in typescript 4.9 that might help us satisfy the column types.
e.g. we could change our types so that it works with [] satisfies Columns; users on older typescript versions can forcibly fallback to [] as Columns.

Table stories should remove {...args} because it hides TS errors.

@mayank99 To confirm, what kind of type errors could be hidden because of {...args}?

what kind of type errors could be hidden because of {...args}?

spreading {...args} can hide any errors that may be caused by the props that come before (for example columns and data) because args is assumed to override it

i think this is fine to keep closed, even with the minor pending issues that may be addressed in future PRs.

i think this is fine to keep closed, even with the minor pending issues that may be addressed in future PRs.

Sounds good, closing this issue for now with #1514. Will solve https://github.com/iTwin/iTwinUI/pull/1514#discussion_r1323515670 in another PR and reference this issue.


Files downloaded have half/less size than files downloaded manually

Hello again,
I recently used the app and I noticed the downloaded files have a different (smaller) size than files that are manually downloaded on the website. They are usually half the size of the original file.
Although at first glance it doesn't appear there is a difference or loss of quality, I'm sure there is something being lost here that may be important.
The attached file has an example of a file but this happened with all of them.
Files.zip
Atelier.jpg is the copy downloaded with the app and website.jpg was manually downloaded.

Sorry for my delay.
Regarding this phenomenon, a bug has been modified at V1.3.17.
This bug will affect all lossy compressed pictures, like *.jpg files.
And the two pictures you had uploaded are different, maybe the source file is Atelier.jpg bigger then website.jpg


Add support for xpath

Xpath libraries:

https://github.com/antchfx/jsonquery
https://github.com/go-xmlpath/xmlpath/tree/v2

Implemented: https://github.com/linksmart/thing-directory/compare/ed0b8c39c44e504d291661b78e22d125ba4a00bb..3713365c2fb6e16aca1278df33be9743efb7fec7

Query examples: https://github.com/linksmart/thing-directory/wiki/Query-Language


TexMathZone Questions

HI, how to add \begin{align} as one of the math context in the define math() functions? THank you!

I also encountered this problem, and I find this website https://tex.stackexchange.com/questions/416030/how-do-i-make-vim-highlight-math-properly-in-the-align-environment.
I did it according to this website, but it didn't work. I find out it's because vim con't identify align environment, so I add the follow code to~/.vim/bundle/vimtex/after/ftplugin/tex.vim last line.
call TexNewMathZone("E","align",1)

then it works.


⚠ wakapi has degraded performance

In 93c0726, wakapi (https://waka.mcha.cloud/api/health) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: wakapi performance has improved in 84360da after 1 hour, 47 minutes.


add test for ConfigException

Adding a test ensuring the Exception is raised and hopefully avoiding future misses of the Exception.
adds a test for what was fixed by @marrrcin #19  and mentioned in #20

Merged, thanks for your contribution!


Added Docker to development environment

#73
Command to run the server:
docker-compose -f docker-compose-development.yml up
Command to run the tests:
docker-compose -f docker-compose-test.yml up
Docker version:
Docker version 17.12.0-ce, build c97c6d6
Docker-compose version:
docker-compose version 1.20.0-rc2, build 8c4af54

👍


SPIKE: try VCRpy

https://github.com/packit-service/research/pull/41

I am closing this in the meantime before we come to our decision wrt testing.


fix: don't change the systemd default from shared

If the default is private, then if namespaced services (like auditd/chronyd) are restarted after the persistent disk is mounted, then it is not possible to fully unmount that disk until those services are stopped. The system will appear to let you do so, however the device remains busy and subsequent attempts to remount (as read-only) will fail as the device is busy.
Fixes #310
There's probably a test that could be written to confirm it this is not added, however I don't know the DSL that Ruby is using for that and probably won't pick up for this one change.

CLA

This is a trivial change, and getting an authorized person to sign a CLA on my side will likely be onerous. Can this please be accepted as trivial and not requiring a CLA, or can a maintainer kindly reimplement the relevant deletion?

CLA is unfortunately required and I don't think there is much flexibility on that. Could you do this in your "free time" and sign the CLA just as yourself?
Bionic is also not built from the jammy branch, so the bionic guard shouldn't be needed.

Have updated the PR to drop the bionic guard, and it's now a delete only PR.

CLA is now signed.

@ae-govau Thanks for the contribution!

Just a quick testimony: with the Stemcell v1.351 that ships this PR, the blocking issues I had with Concourse on Bosh-Lite seems to be fixed.
Thank you all 👏


[WIP] Cross Oni Movement

Potentially fixes #67.
I've not actually got 2 monitors, so I've written this such that it should work broadly moving across any Oni instances. Plus I don't think the screen number actually matters since I think if you have 2 1080p monitors, the top left of your primary is 0,0 and then the other ones are either -1920,0 or 1920,0, so this approach (ie using the current location) should work.
That said! I don't want to put a bunch of effort in based on vague assumptions I've had from reading StackExchange and GitHub issues, so if someone with 2 monitors could test it, that would be great. Currently, I've not bothered hooking up the movement keys, I'm just using the Ctrl+Shift+P menu to add Up, Down, Left, Right movements.
Assuming that works on multiple monitors, since it works for me on one at least:

[ ] Check issue with windows being correctly emptied, since currently if you close a Window, this stops working due to an object being destroyed.
[ ] Actually link this up up the to unhandled movement option.
[ ] Move the code out of main and into a separate file (+ tidy it)
[ ] Unit test it, and spend a little time seeing if CI tests are feasible.
[ ] Deal with minimised windows? Ie currently if you have 3 windows of oni, with the middle minimised, the middle will open up again.

If not, well at least I only spent an hour or so on this to avoid other work.

This is awesome, @CrossR ! Once we have this I might never need to leave Oni... I'll put browser and terminal in one window and code in the other haha

so if someone with 2 monitors could test it, that would be great.

Tested it out, worked perfectly moving right/left between my two monitors 👍

Unit test it, and spend a little time seeing if CI tests are feasible.

Thanks for thinking about the tests!
The CiTests may be tricky - we don't have any tests that launch multiple Oni instances today, and in automation, we don't force a single instance of the main process (so this actually wouldn't work, unless we made further changes to add a setting to allow that). I think a unit test for the window-switching logic would be sufficient, and we could document a manual test case here:  https://github.com/onivim/oni/blob/master/test/Manual.md#manual-test-cases
It would be nice to have a full CiTest eventually, but I'm not sure it's the right tradeoff of value vs time at the moment (up to you though).
Other bullet points look great - excited to have this in!

It looks like the error I get isn't due to the windows list not being correctly updated on close, but rather when the initial Oni window (that is, the first Oni window to be opened) is closed, the windows list gets deleted entirely.
Need to look into why that is happening, and how to fix it...

It seems that for some reason, when the main Window is closed, the first item in the windows array becomes unreferenced...for some reason. Which then causes issues since I need to check over all the values in that array. I'll poke around it for a bit tomorrow, since I've not had that much time today to check.

I've added tests for all the easily testable parts. The remaining function that hasn't been tested doesn't return anything, so we'd need a CI test that can check the focus of a window, which sounds pretty awkward.
Only thing left here I think is I want to check is that where I've added the subscription makes sense in OniEditor?
Oh and the original ticket mentions adding commands for Expand Up/Down etc, but I think that is distinct enough to be done in a separate PR. Could probably also do with making the movement into an action that users can bind, such that a user can use it without having to move to the edge of the window with their own keybinds.

I've added tests for all the easily testable parts.

Awesome! Thanks for wiring up unit tests for our main process - big step forward to have some of that code covered under test. This is a great example of a change that improves the quality of the code while implementing new functionality.

The remaining function that hasn't been tested doesn't return anything, so we'd need a CI test that can check the focus of a window, which sounds pretty awkward.

Agree 100% - I don't think a CiTest is necessary for this.

Only thing left here I think is I want to check is that where I've added the subscription makes sense in OniEditor?

Ah ya, good catch - there might be an issue in the 'multiplexing' case - I left a more detailed comment on the file. But we probably will want to move it to a place that only gets initialized once.

Oh and the original ticket mentions adding commands for Expand Up/Down etc, but I think that is distinct enough to be done in a separate PR. Could probably also do with making the movement into an action that users can bind, such that a user can use it without having to move to the edge of the window with their own keybinds.

Yep, makes sense to have that be a separate PR 👍
Nice work! 💯

Assuming no failures on the CI side of things, this should be done now.

Awesome! Code changes look great 👍 Approving now


Read battery voltage and clean up logged text  and comments

On Clima V3 hardware, the battery is connected to an analogue input through a voltage divider. Copy the existing code that reads solar voltage and adjust to read and output the battery voltage on V3 Clima.
Fix typos in Log text.
Set V2 & V3 interface comments to have the same text.

Thx @doingnz  - really appreciated!


Extend #234 (Goto) to CreateMachine etc.

Copied from #234:
PSharpRuntime has several methods that take (Type type, ...):

the CreateMachine variants
RegisterMonitor
InvokeMonitor This is the only one that also has a generic overload: InvokeMonitor<T>(Event e).
also INetworkProvider.RemoteCreateMachine

Add the generic form for these and deprecate the non-generic form.

Closing as I accidentally opened a duplicate (#244)


IntegrationTest use symfony.phar build

I think is not necessary to use a build of symfony.phar to launch the integration tests.
 public function setUp()
    {
        $this->rootDir = realpath(__DIR__.'/../../../../');
        $this->fs = new Filesystem();

        if (!$this->fs->exists($this->rootDir.'/symfony.phar')) {
            throw new \RuntimeException(sprintf("Before running the tests, make sure that the Symfony Installer is available as a 'symfony.phar' file in the '%s' directory.", $this->rootDir));
        }
    }

If we use the symfony script file is better than the build of phar, because the symfony.phar not must be in the repository, and is not a good idea to build the phar file in the setUp of test.
Of this way we can work and launch integration test of project, without build the project to launch integration test. Build the symfony.phar for integration test I think is redundant.
If you are agree I can make a simple PR.
Regards,

The phar file is built on travis-ci only (check .travis.yml file for more info). It sounds relevant to me because it pushes the tests very close to "production" tests, because everyone will use the phar in fine, not the repo binary.

well, using the phar is done on purpose, to ensure that the phar itself is working (it also catches mistakes in the phar building itself, which happened in the past).
So I'm closing this issue


Gh action all in one testing

Which problem is this PR solving?

Short description of the changes

Codecov Report

Merging #10 (5a21db6) into master (556e3ba) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   95.13%   95.11%   -0.03%
==========================================
  Files         213      213
  Lines        9491     9491
==========================================
- Hits         9029     9027       -2
- Misses        386      387       +1
- Partials       76       77       +1

Impacted Files
Coverage Δ

plugin/storage/badger/spanstore/reader.go
95.37% <0.00%> (-0.72%)
:arrow_down:

Continue to review full report at Codecov.

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


High CPU usage in VS Code extension

Type: Bug
VS Code stopped showing code suggestions. A Code Helper (Plugin) process consumes 100% CPU, and can't be killed when reloading the window. I used VS Code's Extension Bisect feature and located UnoCSS plugin.
Maybe related: https://github.com/vuejs/language-tools/issues/4239
Extension version: 0.59.1
VS Code version: Code 1.88.0 (5c3e652f63e798a5ac2f31ffd0d863669328dc4c, 2024-04-03T13:28:09.495Z)
OS version: Darwin arm64 23.4.0
Modes:

System Info

Item
Value

CPUs
Apple M2 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
2, 3, 4

Memory (System)
32.00GB (5.00GB free)

Process Argv
--crash-reporter-id 33f2db18-d6d2-4571-bcf7-5ce1184f51fc

Screen Reader
no

VM
0%

I have the same issue, vscode is completely frozen and unresponsive for me.
The current workaround is to downgrade to v0.59.0.

/cc @Simon-He95 do you have some idea? Might be https://github.com/unocss/unocss/pull/3696 ?

I'll check it later.

I also encountered VsCode freeze in version 1.88.1 and unocss 0.59.1. The current solution is to downgrade it to unocss 0.59.0,
PS: Operating System Win10

I'm facing the same issue where my VSCode is unable to load any functionality other than typing. Using Start Extension Bisect, I found out the issue is with this extension, so I've now downgraded to v0.59.0 for use

I have found the reason and will submit a PR to fix it later.

Morning, can confirm something happening - blocking extension initialisation from finishing, git etc also not resolving

Update to v0.59.2 and seems fine now

I am running v0.59.4 and the issue happens when I'm editing a typescript file that has this comment in it (needed in the codebase):
// @unocss-include

Sticking to v0.59.0 fixes the issue.
Maybe I need to update my project's unocss version ?

can you provider your file to help me test


Unable to download spanish course

When trying to download the following course, i get errors (its a free course)
https://www.udemy.com/draft/58026/learn/v4/content
/data/other/udemy$ udemy-dl https://www.udemy.com/draft/58026/learn/v4/overview
Username / Email : <hidden>
Password :
[INFO-835] Downloading to: /data/other/udemy/58026

[INFO-107] Trying to log in ...
[INFO-137] Login success.
[INFO-144] Found draft...id: 58026
Getting course data: 1.45% [--------------------] [CRITICAL-407] Cannot download lecture "Introduction": "'view_html'"
Getting course data: 4.35% [=-------------------] [CRITICAL-407] Cannot download lecture "Basic Introductions": "'view_html'"
Getting course data: 7.25% [=-------------------] [CRITICAL-407] Cannot download lecture "Basic Greetings": "'view_html'"
Getting course data: 10.14% [==------------------] [CRITICAL-407] Cannot download lecture "Pronombres personales": "'view_html'"
Getting course data: 13.04% [===-----------------] [CRITICAL-407] Cannot download lecture "Present tense--Regular verbs": "'view_html'"
Getting course data: 15.94% [===-----------------] [CRITICAL-407] Cannot download lecture "Singular and Plural Nouns": "'view_html'"
Getting course data: 18.84% [====----------------] [CRITICAL-407] Cannot download lecture "Noun Gender - MasculineFeminine": "'view_html'"
Getting course data: 21.74% [====----------------] [CRITICAL-407] Cannot download lecture "The verb ser": "'view_html'"
Getting course data: 24.64% [=====---------------] [CRITICAL-407] Cannot download lecture "Adverbios de lugar MuyMucho": "'view_html'"
Getting course data: 27.54% [======--------------] [CRITICAL-407] Cannot download lecture "Question Words": "'view_html'"
Getting course data: 30.43% [======--------------] [CRITICAL-407] Cannot download lecture "Estar - To be": "'view_html'"
Getting course data: 33.33% [=======-------------] [CRITICAL-407] Cannot download lecture "Definite and Indefinite Articles - The Basics": "'view_html'"
Getting course data: 36.23% [=======-------------] [CRITICAL-407] Cannot download lecture "Tener - To have": "'view_html'"
Getting course data: 39.13% [========------------] [CRITICAL-407] Cannot download lecture "Quantifiers": "'view_html'"
Getting course data: 42.03% [========------------] [CRITICAL-407] Cannot download lecture "Llamarse - To call oneself": "'view_html'"
Getting course data: 47.83% [==========----------] [CRITICAL-407] Cannot download lecture "Demostrativos": "'view_html'"
Getting course data: 50.72% [==========----------] [CRITICAL-407] Cannot download lecture "A personal": "'view_html'"
Getting course data: 53.62% [===========---------] [CRITICAL-407] Cannot download lecture "Posesivos 1": "'view_html'"
Getting course data: 56.52% [===========---------] [CRITICAL-407] Cannot download lecture "Posesivos 2": "'view_html'"
Getting course data: 59.42% [============--------] [CRITICAL-407] Cannot download lecture "Stem change o-- ue": "'view_html'"
Getting course data: 62.32% [============--------] [CRITICAL-407] Cannot download lecture "Stem change e-- ie e -- i": "'view_html'"
Getting course data: 65.22% [=============-------] [CRITICAL-407] Cannot download lecture "Verbo -ir": "'view_html'"
Getting course data: 68.12% [==============------] [CRITICAL-407] Cannot download lecture "Ser vs. Estar 1": "'view_html'"
Getting course data: 71.01% [==============------] [CRITICAL-407] Cannot download lecture "Verbo - haberhay": "'view_html'"
Getting course data: 75.36% [===============-----] [CRITICAL-407] Cannot download lecture "Likes and dislikes with the verb GUSTAR": "'view_html'"
Getting course data: 78.26% [================----] [CRITICAL-407] Cannot download lecture "Muy Mucho": "'view_html'"
Getting course data: 81.16% [================----] [CRITICAL-407] Cannot download lecture "BellÃsima The intensifier -isimo": "'view_html'"
Getting course data: 84.06% [=================---] [CRITICAL-407] Cannot download lecture "Adverbs - mente": "'view_html'"
Getting course data: 86.96% [=================---] [CRITICAL-407] Cannot download lecture "Go future ir  a  infintive": "'view_html'"
Getting course data: 89.86% [==================--] [CRITICAL-407] Cannot download lecture "Expressing reflexive actions": "'view_html'"
Getting course data: 92.75% [===================-] [CRITICAL-407] Cannot download lecture "Necesito dormir Verbo  infinitivo": "'view_html'"
Getting course data: 95.65% [===================-] [CRITICAL-407] Cannot download lecture "Verbos irregulares y g z": "'view_html'"
Getting course data: 100.00% [====================] Done
[CRITICAL-407] Cannot download lecture "Conclusion": "'view_html'"

Could you tell me how you resolved this issue please.
Thanks

I ended up using udeler (desktop GUI app)
On Apr 4, 2019 2:47 AM, reachoutsanjay<EMAIL_ADDRESS>wrote:
Could you tell me how you resolved this issue please.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/r0oth3x49/udemy-dl/issues/284#issuecomment-479772057, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYp33bHL3nkwn9C7pdBUfKDLIwzppFPNks5vdZ_jgaJpZM4XUFvE.

Thank you very much for your reply. I have done the same as well.
On Thu, Apr 4, 2019 at 4:57 PM Patrick Picard<EMAIL_ADDRESS>wrote:

I ended up using udeler (desktop GUI app)
On Apr 4, 2019 2:47 AM, reachoutsanjay<EMAIL_ADDRESS>wrote:
Could you tell me how you resolved this issue please.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/r0oth3x49/udemy-dl/issues/284#issuecomment-479772057>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AYp33bHL3nkwn9C7pdBUfKDLIwzppFPNks5vdZ_jgaJpZM4XUFvE

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/r0oth3x49/udemy-dl/issues/284#issuecomment-479858867,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANbLJBhXN1L4YZxv98ghzftiv2bB0MJmks5vdeGOgaJpZM4XUFvE
.

Getting the same error. How to resolve this


Bump the cobra requirement, since they have some nice usage printing fixes

@diogomonica and @riyazdf LGTM'ed IRL


Missing rights to execute sh scripts

Fixed with:
RUN chmod +x /run.sh
RUN chmod +x /set_mongodb_password.sh

Where do you see this issue? Both scripts in both versions (2.6 and 3.0) already have +x permissions


changes in hiera.yml

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


chore: Switch track to 0.2

Description
Please include a summary of the change. Please also include relevant motivation and context. List any dependencies that are required for this change.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have made corresponding changes to the documentation https://github.com/canonical/charmed-5g/pull/201

Blocked by https://github.com/canonical/sdcore-github-workflows/pull/44


fix to add pool members with the specific port and to the virtual listener IP address after the creation process

fix to add pool members with the specific port and to the virtual listener IP address after the creation process
Previously when we add the pool members with the port number, it ignores the port number and always sets as default value 'ANY'. As per the API, we don't have pass any value for the port if it needs to be 'ANY'.
Changed the virtual_lisetener resource to set the virtual listener IP address once it has been created so that we can use it any other resources

Hi - thanks for the fix! I'll review this ASAP.

Will the documentation for ddcloud_vip_pool_member (relating to the ipv4 property) need to be updated?

BTW, I've written some acceptance tests for this scenario; once the PR is merged, I'll add it in so you'll be able to see how those are written.

@kumarappanc - thanks for putting this together!
I'll include this fix in the next release, but feel free to use your own build for now if necessary.
Let me know if you need help doing a local build.
Cheers,
Adam.
PS. I've added some acceptance tests to cover this scenario.