Permissions

Hey, cool project!
I tried the application and had some issues with permissions.
The first issue is that I couldn't download Linux Kernel 5.9-rc2 that is listed in the Asset section of the README.
The error was:
<Error>
AccessDenied
<Message>Access denied.</Message>
Anonymous caller does not have storage.objects.get access to the Google Cloud Storage object.
</Error>

So I grabbed the kernel and rootfs from Getting Started with Firecracker instead (and Firecracker v0.23.0).
Then I started the app with cargo run and sent this request:
~/.../firesquid >>> curl --header "Content-Type: application/json" \
  --request POST \
  --data '{"vm_name":"test"}' \
  http://localhost:3000

The response was:
{"success":true}%

But this was printed to the console:
thread 'tokio-runtime-worker' panicked at 'Failed to spawn vm child process: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }', src/vm/child.rs:29:10

Any idea why I got a permission denied error?
I would be happy to provide more information if needed.

Heya! Thanks for the interest haha
The download permission thing was my bad, should be fixed now :D (forgot to check a box in GCP)
The execution permission I think is related to the firecracker binary not being executable
Try executing
chmod +x firecracker

I'll most certainly include this in the docs that I'm working on haha 👀

Thank you, the issues are now solved :+1:


Changing directions quickly causes uncontrollable driving

When using the slew rate limiter, if the driver quickly changes direction from forward and back, multiple times per second, after a few seconds the robot drives at full speed in one direction uncontrollably.
We've tested with both an older version of the template without the rate limiter, as well as disabling the slew rate limiter via the rateLimit parameter of the drive function. All test have been done on a completely vanilla copy of the template, with both WPILib 2023.2.1 as in the template, and 2023.4.2.
https://user-images.githubusercontent.com/9936886/224512561-5b24d621-76af-4ef3-8e32-0af5ff21d4d2.mp4

We haven't been able to reproduce this, so for now, we'll handle this within our email system. We've asked you some further questions there to try to help figure out what's going on. If it does turn out to be an issue with the MAXSwerve template, we'll reopen this issue.


[node-feature-discovery] Deprecate chart after PR is merged upstream

Details
Helm chart name:
node-feature-discovery
Describe the solution you'd like:
Deprecate node-feature-discovery chart after the following is merged
https://github.com/kubernetes-sigs/node-feature-discovery/pull/423

Upstream chart still not released

Upstream has been merged but not released yet. Release v0.8.0 tracking is in kubernetes-sigs/node-feature-discovery#437

Two more PR's to go according to https://github.com/kubernetes-sigs/node-feature-discovery/issues/437#issuecomment-799525058 🙂

It's released now


answer for 5.7 - Exercise 6

Although the question asks us to "Look at each destination", I think it weird not to take origin into account.
flights %>%
    filter(!is.na(air_time)) %>%
    group_by(dest, origin) %>%
    mutate(med_time = median(air_time),
           fast = (air_time - med_time) / med_time) %>%
    arrange(fast) %>%
    select(dest, origin, air_time, med_time, fast, dep_time, sched_dep_time, arr_time, sched_arr_time) %>%
    head(15)

Fortunately, the results seems not to be influenced too much.
# A tibble: 15 x 9
# Groups:   dest, origin [13]
   dest  origin air_time med_time   fast dep_time sched_dep_time arr_time sched_arr_time
   <chr> <chr>     <dbl>    <dbl>  <dbl>    <int>          <int>    <int>          <int>
 1 BOS   LGA          21       37 -0.432     1450           1500     1547           1608
 2 ATL   LGA          65      112 -0.420     1709           1700     1923           1937
 3 GSP   EWR          55       92 -0.402     2040           2025     2225           2226
 4 BNA   EWR          70      113 -0.381     1914           1910     2045           2043
 5 BOS   LGA          23       37 -0.378     1954           2000     2131           2114
 6 MSP   EWR          93      149 -0.376     1558           1513     1745           1719
 7 CVG   EWR          62       95 -0.347     1359           1343     1523           1545
 8 RIC   EWR          35       53 -0.340     1812           1639     1942           1812
 9 BUF   JFK          38       57 -0.333     2307           2250       34              8
10 BOS   JFK          26       38 -0.316     1200           1200     1254           1313
11 ROC   JFK          35       51 -0.314     2340           2250      120              5
12 ORF   JFK          36       52 -0.308     1720           1645     1820           1820
13 PIT   LGA          40       57 -0.298     1557           1610     1723           1755
14 BOS   LGA          26       37 -0.297     1711           1700     1827           1813
15 DCA   JFK          34       48 -0.292     1104           1105     1158           1215

You're right. Fast flights should considered by comparing the flights to other flights to the same destination from the same origin. I drafted a new answer. What do you think?

I didn't end up putting that code in, though I expanded some of the discussion on the question. If you really think it goes in, open a PR and I'd incorporate it.


Weedfs Volume Read Timeout !!

Regards.

Regards.
Shahzaib
—
Reply to this email directly or view it on GitHub
https://github.com/chrislusf/seaweedfs/issues/175.

Volume server has an idleTimeout option
Alright, it means this is not a serious threat. Though i have increased idleTimeout from 10 to 15seconds.

Thanks !!


Mongoid 5 throws "uninitialized constant ***::Country"

Hi.
After upgrading to mongoid 5, models with Country field defined become broken with this error.

I don't have a good environment to test this in.  Can you provide a sample ?

I found, that changing type: Country to type: ISO3166::Country helps.
I suppose it is all about namespace and it is not a bug at all, but reqiures some chenges in models' code to be compactable with new version of the gem.

Sounds good, i'll close this for now.


Cannot set batchSize for aggregation cursor

ReactiveMongo Version (0.12.7)
MongoDB version (3.6.0)
Operating System (Windows 10)
JDK (Oracle 1.8.0_152)
Expected Behavior
It should be possible to configure batch size for getMore commands of an aggregation.
Actual Behavior
None of the existing GenericCollection methods (aggregate, aggregateWith1, aggregatorContext) allows to specify the size of getMore batches, only the initial batch size. It results in performance degradation when all aggregation results do not fit in the first batch. So, if an aggregation query produces, say, 110 documents and the initial batch size is 100, collecting it into a List will generate 11 commands to Mongo: one first batch of 100 and 10 batches of 1 each.
There is a discussion about this at https://groups.google.com/forum/?fromgroups#!searchin/reactivemongo/getmore|sort:date/reactivemongo/hXJgJld71nU/1hp4GkXiDgAJ but the responses seem to suggest there is no way currently to pass desired QueryOpts.batchSize for the getMore batches.

See #733


ImportError: cannot import name 'div'

i cannot use the component , and get this error, i am working on python 3.6.9 and ubuntu 18.04 with virtualenv
during pip install i saw this
"Failed  building wheel for htbuilder"
"Failed to build st-annotated-text-htbuilder"
Installing collected packages: iteration -builder..................................
When running the app, it shows
error:
line 4, in
from annotated_text import annotated_text
error:
in
from htbuilder import HtmlElement, div, span, styles
thanks if can resolve

same issue here..

same issue here..

same issue here..

same issue here..

issue resolved by updating the python version

issue resolved by updating the python version

Merged fix.


Support MSVC name unmangling

It would be nice to support MSVC name unmangling. It's easy to do: just call the undname tool. It works well with wine.
$ wine undname '?h@@YAXJ@Z' 2> /dev/null
Microsoft (R) C++ Name Undecorator
Copyright (C) Microsoft Corporation. All rights reserved.

Undecoration of :- "?h@@YAXJ@Z"
is :- "void __cdecl h(long)"

CC @andrewpardoe - is there an unmangler for msvc?

Yes, the undname tool: https://docs.microsoft.com/en-us/cpp/build/reference/decorated-names#Undecorated

Perfect. I should learn to use a search engine! (Bing maybe? :))

That is what I used, as a matter of fact!  But it helps to know what you're looking for a priori.

To be closed?
It does not demangle lambdas correctly, but I guess that is a problem with undname.
Or maybe a newer version of the tool is required.

there are some exceptions because of the Wine environment, otherwise undname is used, so yes, this can be closed.


Classes inheritance not reproduced in generated schema

I am using io.swagger.core.v3:swagger-core:2.1.3 to generate OpenAPI specification. My Java classes inheritance does not appear in the generated specification (the subtype does not references the supertype and duplicaste properties instead), which is annoying (not blocker though) for my consumers. Am I missing something ?
My java models
@Path("dummy")
public interface DummyApi {
	@GET
	MyDto dummy();

	class MyDto  {
		public Address address;
		public SubAdress subAdress;
	}

	class Address {
		public String street;
	}

	class SubAdress extends Address {
		public String extraInfo;
	}
}

Actual generated schema
  "components" : {
    "schemas" : {
      "Address" : {
        "type" : "object",
        "properties" : {
          "street" : {
            "type" : "string"
          }
        }
      },
      "MyDto" : {
        "type" : "object",
        "properties" : {
          "address" : {
            "$ref" : "#/components/schemas/Address"
          },
          "subAdress" : {
            "$ref" : "#/components/schemas/SubAdress"
          }
        }
      },
      "SubAdress" : {
        "type" : "object",
        "properties" : {
          "street" : {
            "type" : "string"
          },
          "extraInfo" : {
            "type" : "string"
          }
        }
      }
    }
  }

Expected generated schema
  "components" : {
    "schemas" : {
      "Address" : {
        "type" : "object",
        "properties" : {
          "street" : {
            "type" : "string"
          }
        }
      },
      "MyDto" : {
        "type" : "object",
        "properties" : {
          "address" : {
            "$ref" : "#/components/schemas/Address"
          },
          "subAdress" : {
            "$ref" : "#/components/schemas/SubAdress"
          }
        }
      },
      "SubAdress" : {
        "allOf": [{
          "$ref": "#/components/schemas/Address"
        }],
        "type" : "object"
      }
    }
  }

Inheritance can be declared using annotations, the following code would be resolved as your last example (with missing extraInfo):
@Path("dummy")
public interface Ticket3615Resource {

    @GET
    MyDto dummy();

    class MyDto  {
        public Address address;
        public SubAdress subAdress;
    }

    @Schema(subTypes = SubAdress.class)
    class Address {
        public String street;
    }

    @Schema(allOf = Address.class)
    class SubAdress extends Address {
        public String extraInfo;
    }
}

Closing ticket, please reopen  if you still have questions or issues.

Many thanks for your answer.
For those asking for the same question, I also had to define the discriminatorProperty attribute in the Schema annotation on the superclass (Address) and add this property in the class so that my consumers can have inheritance in their generated classes.
Another question, though : is there a way to customize the schema generation without modifying the Address/SubAdress classes ? In our projet, we actually use domain classes (such as Address and SubAdress) but we'd rather not introduce third-party dependencies in those domain classes.


CORE-3003: Index-name without schemaName

SchemaName is not allowed in Index names

Already merged in. Thanks!


Pausing within text

Is there a way that I can get it to pause for a very brief period of time between sentences while writing my text.

Yeah u can use the pauseTyping() method


Fonts not correctly loaded if Worker enabled on Wordpress site

Even without any string replacement, if I enable the Worker on Cloudflare, it breaks the Wordpress/Elementor website.
The errors in the browser console are :

Failed to decode downloaded font: < URL >
(38 lines like : Failed to decode downloaded font: https://site.com/wp-content/themes/my-listing/assets/fonts/GlacialIndifference/Bold.otf)
OTS parsing error: incorrect entrySelector for table directory
OTS parsing error: Size of decompressed WOFF 2.0 is less than compressed size
OTS parsing error: Failed to convert WOFF 2.0 font to SFNT
OTS parsing error: incorrect file size in WOFF header
The errors are supposed to happen at /index:1

The website works well without the Worker.

Disabling the proxying and using DNS only from the dashboard solved this one for me.


Different colors for Result and Result Type

when trying to see what are operands, it would be nice to have a faster way to know what is a Result or Result Type ID

https://github.com/KhronosGroup/SPIRV-Visualizer/commit/14b4094ad900d96944a4029f0f3de016b9331d4f


Missing Output for Parallel Loop Conditions

In parallel Loop Conditions it is possible to hand over a specific position of an array variable to a task/service. E.g.
Parallel Loop i To cr.parts_count
        paintingAndCuttingTask
            In
                cr.sheet_parts[i]

However, it is not possible to specify such a field position as output. As a consequence, it is not possible to specify an output for parallel loops since there will be a loss of data if the whole array is defined as input and output, e.g.
cr{
 sheet_parts = [0,0,0],
 parts_count = 3
}

Parallel Loop i cr.parts_count
    paintingAndCuttingTask
       In
          cr.sheet_parts
       Out
          cr.sheet_parts

Here, each of the parallel tasks get the initial input value sheet_parts = [0,0,0]. Since the output of each task writes the variable cr.sheet_parts, the results of the first two paintingAndCuttingTasks that are finished are lost and only the one that finished last is saved.

From my understanding, you would like you store the return values of each service/ task call instead of keeping the last one?

Partial. The main issue is that when individual array elements of a variable are passed as input to a service or task within a parallel loop, these array elements should be able to be placed at the same array position as output of a service/task. Currently, it is only possible to return the entire array and thus the values from individual tasks are lost since the entire array is defined as output and stored in the same variable. Therefore, for each individual output of the parallel loop, a new value is assigned to the variable instead of just a single array position within the variable


Server Start-Up Instability

Ever since Carl and Garhan fixed the old server start-up issue, I've had intermittent issues with the servers. These seem to come and go throughout the day, but they've been around every day since Thursday night (Feb. 11). Even after getting a server to start-up, it seems like sometimes everything in it starts to hang, and on restart the start-up issues return. Maybe it's a network problem, but if so, it's been especially prominent this week.
The log I always get seems fine except for the last step:
Event log
Server requested
2021-02-14 19:16:57+00:00 [Normal] Successfully assigned cmsaf-prod/jupyter-matousadamec-40gmail-2ecom to red-kube-c6929.unl.edu
2021-02-14 19:16:57+00:00 [Normal] AttachVolume.Attach succeeded for volume "pvc-e060c809-3153-4384-b62b-70a3f4a2bc8e"
2021-02-14 19:17:05+00:00 [Normal] Container image "jupyterhub/k8s-network-tools:0.9.0" already present on machine
2021-02-14 19:17:05+00:00 [Normal] Created container block-cloud-metadata
2021-02-14 19:17:05+00:00 [Normal] Started container block-cloud-metadata
2021-02-14 19:17:05+00:00 [Normal] Container image "coffeateam/coffea-casa:0.3.23" already present on machine
2021-02-14 19:17:06+00:00 [Normal] Created container notebook
2021-02-14 19:17:06+00:00 [Normal] Started container notebook
2021-02-14 19:17:06+00:00 [Normal] Container image "coffeateam/coffea-casa-analysis:0.3.23" already present on machine
2021-02-14 19:17:06+00:00 [Normal] Created container dask-worker
2021-02-14 19:17:06+00:00 [Normal] Started container dask-worker
Spawn failed: Server at<EMAIL_ADDRESS>didn't respond in 30 seconds

This is likely me restarting the service to get html updates taken up.  I'll find a better way.

Dev/prod instances for test deploys will eliminate this issue.


username and password are ignored (hardcoded)

The username and password provided should be used in this line: pynetio.py#L39
i. e.
self.get('login {} {}'.format(self.username, self.password))
I am using pynetio via home assistant module which currently references the Version 0.1.6

Hi Andre,
indeed, just uploaded a fix (<IP_ADDRESS>) to pypi.
Anyway I will deprecate it as the http interface is a lot faster and less problematic (connection broken 130 etc..)
There will be a PR for HA for the next release.

Awesome, thank you!
I didn't find a documentation or an implementation for the http interface.
Not sure if it is the same for 230B and 4.


Manage Channel Rewards button goes to Access Denied page

Clicking the Manage Channel Rewards button in the Sounds tab when a username is not all lowercase goes to twitch error page Access Denied You do not have permissions to access this page. Username needs to be set to lowercase in URL string.

Fixed in 25d9abe6c76b631a4656e971951a620c96f40dec


Update 'class-transformer' to version '^0.5.1'

I have problem similar with Issue #20, but with package 'class-transformer'
Please, update 'class-transformer' to version '^0.5.1'
<PROJECT-DIR>$ npm install --save-exact nestjs-form-data
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: <PROJECT-NAME>
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/class-transformer
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peerOptional class-transformer@"*" from<EMAIL_ADDRESS>npm ERR!   node_modules/@nestjs/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>|| ^8.0.0" from<EMAIL_ADDRESS>npm ERR!     node_modules/nestjs-form-data
npm ERR!       nestjs-form-data@"*" from the root project
npm ERR!     5 more (@nestjs/core, @nestjs/microservices, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/nestjs-form-data
npm ERR!   nestjs-form-data@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See <HOME-DIR>/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     <HOME-DIR>/.npm/_logs/<DATE>-debug-0.log

Many thanks for your wonderful work!

Same here, it would be nice to have it updated.
Thanks for your work, it really helps a lot! :)

Hi, thanks for your positive feedbacks. I have published version<EMAIL_ADDRESS>check if the issue is fixed.
Best Wishes


<EMAIL_ADDRESS>incompatible with react 17.0.2

Expected Behavior
I deleted node_modules and package-lock.json then run npm install. I am waiting that successfully install packages.
Actual Behavior
But a dependency error occurred while installing packages.
peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>node_modules/@kenshooui/react-multi-select <EMAIL_ADDRESS>from the root project
Steps to Reproduce the Problem

Delete node_modules and package-lock.json
rm -rf node_modules
rm package-lock.json
npm install

Specifications
react 17.0.2
<EMAIL_ADDRESS>node v16.13.0
npm 7.24.2

Same dependency error here:
<EMAIL_ADDRESS>from the root project

Could not resolve dependency:
peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>node_modules/@kenshooui/react-multi-select
@kenshooui/react-multi-select@"*" from the root project

Same dependency error here:
<EMAIL_ADDRESS>from the root project

Could not resolve dependency:
peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>node_modules/@kenshooui/react-multi-select
@kenshooui/react-multi-select@"*" from the root project

I have a similar issue. Did you find a solution for it?


docs: add project captains to contribution

Supersedes: https://github.com/expressjs/express/pull/4210
I believe I addressed the issues from #4210, specifically the "participate as a committer". I had one comment about "the other captains getting a vote", my intent was that if there is a disagreement from existing captains on a new member, that should be taken into account. Do folks feel like this is important? Or can we land as is?
Also of note, I did end up opening a new PR because there was quite a bit of noise in there.

@wesleytodd this is not ready yet to merge, right? I think you said you were going to make one more change?

Ok, updated with that last change. I think this should be good to merge. Not I stacked this on top of the other TC changes, so might be easiest to merge this to that then merge that one, but up to you @dougwilson as you are going to merge them up tonight for the patch.


[FEATURE] Support changing the ipam mode of the harvester load balancer

Is your feature request related to a problem? Please describe.

By harvester 1.0.0 version, it could not modify the ipam mode of the harvester load balancer because the kube-vip doesn't support changing the allocated load balancer IP.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Suggest can test with #1762

Test Plan
Purpose:
Can set the ipam mode of load balancer on multi-cluster environment for

DHCP mode, write the setting in to yaml and retrieve correct ip address
Pool mode, write the setting in to yaml and retrieve correct ip address

Test Steps

Import Harvester into external or embedded Rancher
Provision an RKE2 guest cluster
Access Harvester in Rancher virtualization management page
Create a IP Pool,
In selector, use All project, All namespace and All guest cluster
Create a nginx deployment on Rancher RKE2 guest cluster page
Create a load balancer on RKE2 guest cluster page
Set the ipam mode to Pool
Check can create load balancer into Running state and route correctly
Check the yaml content of load balancer
Check the ipam mode set to Pool and retrieve correct ip address under status
Edit the load balancer
Change the ipam mode back to DHCP mode
Check the yaml content of load balancer
Check the ipam mode set to DHCP and retrieve correct ip address under status

Verified not fixed on v1.2-6ff754b8-head with Rancher v2.7.5-rc4. Move to implement stage.
Result

When we change the Rancher load balance ipam mode from DHCP to Pool and vise versa

The retrieved load balancer IP address did not change accordingly. Still keep using previous one

Test Information

Test Environment: Single nodes Harvester on local kvm machine
Harvester version: v1.2-6ff754b8-head (06/12)
Rancher version: v2.7.5-rc4

Test steps
https://github.com/harvester/harvester/issues/1834#issuecomment-1586979433

@starbops @torchiaf Please help check the status of the issue. Thanks.

Per issue https://github.com/harvester/harvester/issues/6684:
Change the IPAM from IPPool -> DHCP, the load-balancer-harvester has chance to release the allocated IP;
Change the IPAM from DHCP -> IPPool, the DHCP based IP needs to be released by the kube-vip &/ service object in the guest cluster, this is out of the control and it may cause IP leaking &/ other issues.
Back to the issue itself, change LB IPAM from DHCP <-> Pool means a fundmental change, it is not happending frequently in the production, deleting the LB and re-creating the LB seems not an un-acceptable solution.
We will adopt the change on PR https://github.com/harvester/load-balancer-harvester/pull/37 to block the change on LB IPAM. And update to document to explain this limitation. thanks.
cc @starbops @FrankYang0529 @bk201


Add <%=CURRENT_QUARTER%> template variable

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[ ] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[x] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
N/A
A clear and concise description of what the problem is. E.g. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hey, @HopperTech, thank you for the request. There is already a similar issue asking for more date-related template variables:

https://github.com/dendronhq/dendron/issues/2241

You might want to add leave a comment there as well to consolidate similar requests.

also adding this as a crop candidate

echoing a discussion from discord: <%= YESTERDAY %> would also be a very useful one we could support. Especially in the context of carry-over daily todos.


Support multiple roles for setadminrole and setmodrole

Due to Discord's role structure (as well as other unified comm apps that may be supported in the future), some servers have more than 2 tiers of moderation roles, such as Moderator, Administrator, and Super Admin.

Related to #45

I don't know that this is an appropriate solution for the problem. I'll need to think about this more, but this feels wrong in a few ways.

If all of those roles have permission to do things manually, then the bot won't even need to look at if they are a bot mod/bot admin. The permissions the roles are granting will allow access.
If the roles are being used as tiering and do not directly grant any permissions, then the simple solutions are either add a role that all mods have or make the existing mod roles cumulative on the user. (same for admin)

Additionally, adding multiple mod/admin roles as a setting encourages things about discord permission layouts which I do not want to encourage.
There are a few types of roles.

Cosmetic
Those that grant/deny access via channel settings
Those that grant server permissions.

Cosmetic ones should always be 0 permissioned and sorted below all non-cosmetic roles.
Access granting/denying roles should be just above those and also be zero permissioned at the server level.
Those that grant permissions (mods and such) should be sorted as a hierarchy.
Assuming such a hierarchy from the discord side (which is definitively the only layout that does not experience any permission hierarchy issues in any setting and which discord optimizes their event dispatch for on the gateway & push notification side) It's safe to add zero [discord] permission roles at the appropriate places (preventing hierarchy based privilege escalation with assigning yourself a role) within the granting permission section which grant bot access.

This is going to be obsolete with slash command permissions anyhow, leaving as-is until that switchover.


Option to reverse behavior and make moving all endpoints default without holding Alt.

Can we have an option to not need ALT to move all connected points, or a reverse of the current situation where ALT is needed only to move individual point but without it all endpoints are moved.
It is a bit hard to get how to use ALT (not when moving the endpoint but while an endpoint is moved) otherwise you move all the walls. It also is often the case where you want to move all connected endpoints together, and should be optionally a way to make this the default.

Will make this an option in settings.

This is a setting now in 0.3.0


Can't open the new .stl file with the quickstart example

Hi thanks for sharing the code and writing the function, I am using the quickstart example to create a new .stl file, I am able to create the new .stl file but cant open the same, maybe it is getting corrupt or so (showing 5KB file). Can someone please help to resolve the same

You'll need to give a bit more detail before I can help.

Which exact example did you use? (adding your entire code works too)
What's the exact error you're getting


Add current time/percentage while reading

Describe your suggested feature
It would be nice to have the option to see the current time/percentage and even the chapter (not only the page) while reading in full screen. Maybe even set the position of it on screen. Example:

Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.17.1.
[X] I will fill out all of the requested information in this form.

Tap the center of your screen while reading? Bring up the reader overlay, which should show your status bar & the current chapter?

I mean, how about offer the option at least. Many Reader would benefit from it.

I remembered seeing this/similar issue before, duplicate of closed #861, same response/stance

Then it wouldn't be much of fullscreen now would it? Case in point: You can disable fullscreen reader in the settings and it will display the system tray that has... Clock and Battery!


state_dict keys don't match load_state_dict

I'm using the AutoModelForCausalLMWithValueHead with a custom training loop, I was trying to copy the weights of the model to a target model with the same architecture when I noticed that the keys from state_dict don't match those for load_state_dict
model.load_state_dict(model.state_dict())

RuntimeError: Error(s) in loading state_dict for AutoModelForCausalLMWithValueHead:
	Missing key(s) in state_dict: "pretrained_model.base_model.model.model.decoder.embed_tokens.weight", ....
	Unexpected key(s) in state_dict: "base_model.model.model.decoder.embed_tokens.weight", ....

apparently it's just a key error, and modifying the state_dict function solves it

def state_dict(self, *args, **kwargs):
    r"""
    Returns the state dictionary of the model. We add the state dictionary of the value head
    to the state dictionary of the wrapped model by prepending the key with `v_head.`.
    """

    state_dict = {}

    if not self.is_peft_model:
        pretrained_model_state_dict = self.pretrained_model.state_dict(*args, **kwargs)
    else:
        # if it is a peft model, only save the v_head
        pretrained_model_state_dict = {}
    for k, v in pretrained_model_state_dict.items():
        state_dict[f"pretrained_model.{k}"] = v

    v_head_state_dict = self.v_head.state_dict(*args, **kwargs)
    for k, v in v_head_state_dict.items():
        state_dict[f"v_head.{k}"] = v

    return state_dict

I don't know if this was done to make some high level api work, but I feel like the basic api from pytorch should still be made compatible

I remember @younesbelkada had to work on that to make the models work in trl. Maybe this could be fixed, what do you think?

It looks like you are trying to load from a peft model state dict? in that case you only need to load the v_head as all other parameters are kept untouched right?
model.load_state_dict(model.state_dict(), strict=False)

Should do the trick

Actually I think all parameters are trained, not only the v_head, since I'm using the language modelling as the policy (im not doing RLHF following the tutorials, I'm doing research on RL with LLMs, I have a custom training loop).
To make it work I ended up completely overwriting the method substituting it with the original state_dict() from the torch Module
import torch
from trl import AutoModelForCausalLMWithValueHead

AutoModelForCausalLMWithValueHead.state_dict = torch.nn.Module.state_dict

I'm now wondering what was the reason to change it in the first place.


Execution failed for task ':react-native-webview:compileDebugKotlin'

Bug description:

After installing the lib and building the project it fails with the following error:
To Reproduce:

Just install the lib with yarn, run react-native link and run the project with react-native run-android
Expected behavior:

Just build the application in the smartphone.
Screenshots/Videos:

Environment:

OS: Ubuntu
OS version: 20.04
react-native version: 0.63.4
react-native-webview version: 11.2.1

same still got Execution failed for task ':react-native-webview:compileDebugKotlin'.

I just run the command in the windows 10 , npm install --save react-native-webview already auto linking enabled just run the command below,
npx react-native start
npx react-native run-android
got the following error
react-native-webview:compileDebugKotlin FAILED

The solution in https://github.com/react-native-webview/react-native-webview/issues/2568#issuecomment-1172557823 worked for me


[BUG] Loading BG in Darkmode

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Description
There's a white background which is visible in darkmode when the screen loads. We need to make it transparent.
Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

Kindly give me this issue. I can work on it.

Kindly give me this issue. I can work on it.

done.

Update? @AliHassan245

Kindly see is this feature you want.

Make this theme based

@sristy17 @pooranjoyb  I’ve implemented a theme-based UI for the modal box, but I’m encountering issues with it not applying correctly. If I generate a pull request with the changes, would you be able to take a look and assist in resolving the issue? Your expertise would be invaluable in helping the theme work smoothly.

Ok will check and get back to you

@AliHassan245 Thanks, we truly appreciate, although we'd suggest that donot limit yourself with event. You are always welcome to contribute to MindDrive :)

@pooranjoyb Bhai Could you kindly share your Instagram? I'd love to connect with you there! ❤

You can connect with me in Linkedin


Update .rubocop.yml

Make changes to support newer versions of RuboCop.

Coverage decreased (-0.01%) to 96.335% when pulling 71cc1a0f7851ae76dcccc8e7a8de1603ef35f6cb on salbertson:patch-1 into 3b4b5ee833fc5164d5f7d7a88313a47281528ee2 on geokit:master.

Coverage decreased (-0.01%) to 96.335% when pulling 71cc1a0f7851ae76dcccc8e7a8de1603ef35f6cb on salbertson:patch-1 into 3b4b5ee833fc5164d5f7d7a88313a47281528ee2 on geokit:master.

Coverage decreased (-0.01%) to 96.335% when pulling 71cc1a0f7851ae76dcccc8e7a8de1603ef35f6cb on salbertson:patch-1 into 3b4b5ee833fc5164d5f7d7a88313a47281528ee2 on geokit:master.

Thanks @salbertson


Remove color support from "Include" filter.

I would very much like to use the "include" filter but the fact that it has precedence over dynamic process colors has been a deal breaker.
Now include filter does not apply text coloring either, very much like the exclude filter. Not a big functionality loss because it is still possible to recolor included lines by defining an additional highlight rule. Perhaps more work, but definitely more flexibility.
Accept PR if it makes sense. Reject if not compatible with project vision.

thanks for the suggestion, I can see your point, you mean that if you use the include filter, it will apply its color to the included line, and overwrite the 'process color' ?

Yes, process coloring stops working as soon as an include filter is added.

Thanks for the suggestion, I will have another look at feature requests.

I think you suggestion is good, thank you for the contribution!


Training time

How long did the training take？

Good question. The pretraining for the 'Base' SSAMBA took around 6 days on 2x L40 NVIDIA GPUs.


Cannot read property 'watch' of undefined

🐛 Bug Report
On an ejected Create-react-app project, I installed puppeteer and jest-puppeteer and ran a simple script to run jest, but it's immediately returning
TypeError: Cannot read property 'watch' of undefined
    at setup (/node_modules/jest-environment-puppeteer/lib/global.js:30:18)

My jest config is inside my package.json file so I added it in there as mentioned in docs:
"jest": {
  "preset": "jest-puppeteer"
}

I ran two jest commands below, but still get the same result above:

"test": "node ./scripts/test.js --env=jsdom"
"test": "jest"

Jest Configuration
"jest": "^22.4.2"
"jest-puppeteer": "^3.9.0"
"puppeteer": "^1.12.0"
Expected behavior
I was hoping the test would run successfully without any internal errors.
Run npx envinfo --system --binaries --npmPackages expect-puppeteer,jest-dev-server,jest-environment-puppeteer,jest-puppeteer,spawnd --markdown --clipboard
Paste the results here:
## System:
 - OS: macOS 10.14.2
 - CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
 - Memory: 676.66 MB / 16.00 GB
 - Shell: 3.2.57 - /bin/bash
## Binaries:
 - Node: 8.9.4 - ~/.nvm/versions/node/v8.9.4/bin/node
 - Yarn: 1.13.0 - /usr/local/bin/yarn
 - npm: 5.6.0 - ~/.nvm/versions/node/v8.9.4/bin/npm
 - Watchman: 4.9.0 - /usr/local/bin/watchman
## npmPackages:
 - jest-puppeteer: ^3.9.0 => 3.9.0

hi,@binhxn
I tried to create an app using CRA and tested it with jest-puppeteer. But it works fine.
Could you provide a demo for us to check?

Hi @xiaoyuhen,
Sorry I forgot to clarify, CRA is version 1, not 2. Unfortunately, I cannot share the code =/ .
Here's my original (related) dependencies:
"babel-core": "6.26.0",
"babel-jest": "20.0.3",
"babel-loader": "7.1.2",
"babel-preset-react-app": "^3.1.1",
"react": "^16.2.0",
"react-dom": "^16.2.0",
"webpack": "3.8.1",
"webpack-dev-server": "2.9.4",
"jest": "^22.4.2",
"jest-puppeteer": "^3.9.0",
"puppeteer": "^1.12.0"

jest configs:
"jest": {
    "setupTestFrameworkScriptFile": "<rootDir>/config/jest/enzymeSetup.js",
    "collectCoverageFrom": [
      "src/**/*.{js,jsx,mjs}"
    ],
    "setupFiles": [
      "<rootDir>/config/polyfills.js",
      "jest-localstorage-mock"
    ],
    "testMatch": [
      "<rootDir>/src/**/__tests__/**/*.{js,jsx,mjs}",
      "<rootDir>/src/**/?(*.)(spec|test).{js,jsx,mjs}"
    ],
    "testEnvironment": "node",
    "testURL": "http://localhost",
    "transform": {
      "^.+\\.(js|jsx|mjs)$": "<rootDir>/node_modules/babel-jest",
      "^.+\\.css$": "<rootDir>/config/jest/cssTransform.js",
      "^(?!.*\\.(js|jsx|mjs|css|json)$)": "<rootDir>/config/jest/fileTransform.js"
    },
    "transformIgnorePatterns": [
      "[/\\\\]node_modules[/\\\\].+\\.(js|jsx|mjs)$"
    ],
    "moduleNameMapper": {
      "^react-native$": "react-native-web",
      ......
    },
    "moduleFileExtensions": [
      "web.js",
      "mjs",
      "js",
      "json",
      "web.jsx",
      "jsx",
      "node"
    ]
  }

I've been trying to upgrade my Jest dependencies to the latest version, but I keep going down this rabbit hole of upgrading every single babel package (starting with babel-preset-react-app and babel-core. Now I keep seeing this issue and cannot resolve it after trying numerous solutions on it:
Test suite failed to run

Plugin/Preset files are not allowed to export objects, only functions.

If I can't get any further, I may try just setting up a custom setup/teardown config without jest-puppeteer. Thanks for your time, and please let me know if you need additional info.

@binhxn
Thank you very much for your information. I will try to use CRA v1 to investigate this issue, it may take one day or two.

@binhxn  @petekp
It seems that in the jest v22 version we can't get jestConfig in jestSetup, we will be compatible as soon as possible
Now, you can choose to use jest v23.6 to solve this bug.

Using Jest 23.6 works, thank you.


Plug-in cannot customize CDN server address

Due to poor access to unpkg.com in some areas of mainland China's network, I need to change my CDN address to cdn.bootcss.com, but this configuration does not seem to work. Is my method incorrect?
    const WebpackCdnPlugin = require('webpack-cdn-plugin');
    options.plugins.push(new WebpackCdnPlugin({
        modules: {
            'vue': [{
                name: 'vue',
                cdn: 'https://cdn.bootcss.com/',
                path: 'dist/vue.runtime.min.js'
            }]
        }
    }))

Please give me some advice.

Hi @umbrella22 I think what you're looking for is prodUrl https://github.com/shirotech/webpack-cdn-plugin#produrlstring--unpkgcomnameversionpath rather than cdn (perhaps thats's not the best name for it). The cdn option is for changing the package name if it's different to npm.

Oh, I should have read the whole document. I misunderstood the meaning. Thank you for your busy reply. @shirotech


Tests failing in Editor, passing in CI

I assume this is a race / timing issue, where my local machine is faster than the CI environment.
All these failures relate to Network Identity's NetworkBehaviours hashset is null.

can you post the exact stack trace please?

StopClientCallsOnServerDisconnectForHostClient (0.005s)
---
System.NullReferenceException : Object reference not set to an instance of an object
TearDown : System.NullReferenceException : Object reference not set to an instance of an object
---
at Mirror.NetworkIdentity.OnStartServer () [0x0002f] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkIdentity.cs:635
  at Mirror.NetworkServer.SpawnObject (UnityEngine.GameObject obj, Mirror.NetworkConnection ownerConnection) [0x0014c] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:1394
  at Mirror.NetworkServer.Spawn (UnityEngine.GameObject obj, Mirror.NetworkConnection ownerConnection) [0x00001] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:1320
  at Mirror.NetworkServer.SpawnObjects () [0x00097] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:1275
  at Mirror.NetworkManager.FinishStartHost () [0x00015] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkManager.cs:520
  at Mirror.NetworkManager.StartHost () [0x00054] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkManager.cs:461
  at Mirror.Tests.NetworkManagers.NetworkManagerStopHostOnServerDisconnectTest.StopClientCallsOnServerDisconnectForHostClient () [0x0001d] in C:\Unity\Mirror2021\Assets\Mirror\Tests\Editor\NetworkManager\NetworkManagerStopHostOnServerDisconnectTest.cs:46
  at (wrapper managed-to-native) System.Reflection.RuntimeMethodInfo.InternalInvoke(System.Reflection.RuntimeMethodInfo,object,object[],System.Exception&)
  at System.Reflection.RuntimeMethodInfo.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x0006a] in <4a4789deb75f446a81a24a1a00bdd3f9>:0
--TearDown
  at Mirror.NetworkIdentity.OnStopServer () [0x0002f] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkIdentity.cs:655
  at Mirror.NetworkServer.DestroyObject (Mirror.NetworkIdentity identity, Mirror.NetworkServer+DestroyMode mode) [0x000e0] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:1503
  at Mirror.NetworkServer.CleanupSpawned () [0x0003d] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:240
  at Mirror.NetworkServer.Shutdown () [0x00071] in C:\Unity\Mirror2021\Assets\Mirror\Core\NetworkServer.cs:198
  at Mirror.Tests.MirrorTest.TearDown () [0x00007] in C:\Unity\Mirror2021\Assets\Mirror\Tests\Common\MirrorTest.cs:36
  at Mirror.Tests.MirrorEditModeTest.TearDown () [0x00000] in C:\Unity\Mirror2021\Assets\Mirror\Tests\Common\MirrorEditModeTest.cs:12
  at (wrapper managed-to-native) System.Reflection.RuntimeMethodInfo.InternalInvoke(System.Reflection.RuntimeMethodInfo,object,object[],System.Exception&)
  at System.Reflection.RuntimeMethodInfo.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x0006a] in <4a4789deb75f446a81a24a1a00bdd3f9>:0

Fixed now.


Consider supporting a later version of the bindgen

In croaring-sys Cargo.toml, it require bindgen's version cannot exceed 0.49.
bindgen = ">= 0.37, <= 0.49"

If a project depends on other C wrappers, a compilation error will occur.
error:  multiple packages link to native library `clang`, but a native library can be linked only once

Please commit the following and it should build with the latest rust nightly.
Essentially using the latest bindgen within the Cargo.toml with the latest nightly and you should be good to go.
Please remember when you build, you need to build the croaring-sys subdirectory first
Also add two rust-toolchain files pointing to "nightly-2019-12-06" to both croaring-rs and croaring-sys.
diff --git a/Cargo.toml b/Cargo.toml
index 314fd9e..047e86f 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -15,5 +15,5 @@ roaring = "0.5.2"

 [dependencies]
 libc = "0.2.42"
-croaring-sys = { path = "croaring-sys", version = "0.4.1" }
+croaring-sys = { path = "croaring-sys" }
 byteorder = "1"
diff --git a/croaring-sys/Cargo.toml b/croaring-sys/Cargo.toml
index 695b431..bdd2b5a 100644
--- a/croaring-sys/Cargo.toml
+++ b/croaring-sys/Cargo.toml
@@ -14,5 +14,4 @@ libc = "0.2.42"

 [build-dependencies]
 cc = "1.0"
-bindgen = ">= 0.37, <= 0.49"
-
+bindgen = "0.52.0"

oh yes, please ensure you did install clang, llvm, lld and that should suffice.


conversation.struct={'result': {'value': 'UnauthorizedRequest', 'message': 'Sorry, you need to login first to access this service.'}}

I ran normally on my personal computer, when i change a computer, it reported a bug. After checking it, I found that
conversation. struct={'result ': {'value': 'UnauthorizedRequest', 'message': 'Sorry, you need to login first to access this service.'}}
it led to
self. request=ChatHubRequest（
conversation_ signature=conversation.struct["conversationSignature"],
client_ id=conversation.struct["clientId"],
conversation_ id=conversation.struct["conversationId"],
)
Error.
Excuse me, is the version I downloaded too low or does it need other configurations?

pip3 show EdgeGPT

Name: EdgeGPT
Version: 0.0.10
Summary: Reverse engineered Edge Chat API
Home-page: https://github.com/acheong08/EdgeGPT
Author: Antonio Cheong
Author-email<EMAIL_ADDRESS>License: GNU General Public License v2.0
Location: /home/work/local/python3.8.6/lib/python3.8/site-packages
Requires: asyncio, requests, websockets
Required-by:

Current version is https://github.com/acheong08/EdgeGPT/releases/tag/0.0.20

Current version is https://github.com/acheong08/EdgeGPT/releases/tag/0.0.20

Thank you. I'll try again with a new version.

pip3 show EdgeGPT

sorry，it dosen't work. It still have the same problem when I switch the version to Version: 0.0.20.

It may not be a version problem, because I think another computer that can run successfully is installed with 0.0.10

Did you try to define your own cookie?

How are you running it?

Did you try to define your own cookie?

I'm using the default running method. I'm sorry that the wrong computer can't install a browser. So I can't get the cookie. Is it possible that this is the cause of the error?

It should work out of the box. Not sure where the error comes from. Try again a few times

It should work out of the box. Not sure where the error comes from. Try again a few times and add some debugging statements
sorry, excuse me, there is a new bug.

requests.exceptions.ConnectionError: HTTPSConnectionPool(host='bing.kpham.workers.dev', port=443): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fda71bc8370>: Failed to establish a new connection: [Errno 101] Network is unreachable'))

https://github.com/acheong08/EdgeGPT/releases/tag/0.0.25


Getting 5250 terminal is not supported. Please install TN5250 via yum on the re....

I updated my applications using the ACS Open Source Package Management and installed TN5250 from there.
When I run the 5250 app, I get that message.
Is installing TN5250 using ACS correct, or do I need to install it another way?
What else could be wrong?
TIA, Darryl Freinkel.

⚠ REMOVE THIS LINE AND ANY SENSITIVE INFORMATION BELOW! ⚠

Context
Version

Code for IBM i version
2.4.0

Visual Studio Code version
1.84.2

Operating System
win32_x64

Active extensions
Code Coverage for IBM i (code-coverage-ibmi): 0.1.7
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.4.0
Db2 for IBM i (vscode-db2i): 0.5.2
Emmet (emmet): 1.0.0
Error Lens (errorlens): 3.15.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
GitHub (github): 0.0.1
IBM i Notebooks (vscode-ibmi-notebooks): 0.0.6
JSON Language Features (json-language-features): 1.0.0
Merge Conflict (merge-conflict): 1.0.0
Node Debug Auto-attach (debug-auto-launch): 1.0.0
TODO Highlight (vscode-todo-highlight): 1.0.5
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0
WSL: Recommender (remote-wsl-recommender): 0.0.19

Remote system

Setting
Value

IBM i OS
V7R4M0

Tech Refresh
7

CCSID
65535

SQL
Enabled

Source dates
Enabled

Enabled features

/usr/bin
/QOpenSys/pkgs/bin
/QSYS.lib/ILEDITOR.lib
/QSYS.LIB
/QIBM/ProdData/IBMiDebugService/bin

attr
bash
GENCMDXML.PGM
QZDFMDB2.PGM
startDebugService.sh

iconv
chsh
GETNEWLIBL.PGM

setccsid
ls

tar
md5sum

sort

stat

Shell env
BUILDLIB=QGPL
CURLIB=QGPL
HOME=/home/dfreinkel
HOST=<IP_ADDRESS>
LIBLS=DPOBJLIB DPPDATA DPSRCLIB LIBHTTP YAJL QTEMP QGPL OPRUTLLIB DFREINKEL
LOGIN=dfreinkel
LOGNAME=dfreinkel
MAIL=/var/spool/mail/dfreinkel
OLDPWD=/home/DFREINKEL
PASE_USRGRP_LIMITED=N
PATH=/QOpenSys/usr/bin:/usr/ccs/bin:/QOpenSys/usr/bin/X11:/usr/sbin:.:/usr/bin
PWD=/home/dfreinkel
SHELL=/QOpenSys/pkgs/bin/bash
SHLVL=1
SSH_CLIENT=<IP_ADDRESS> 52402 22
SSH_CONNECTION=<IP_ADDRESS> 52402 <IP_ADDRESS> 22
TZ=<EST>5<EDT>,M3.2.0,M11.1.0
USER=dfreinkel
USERNAME=dfreinkel
_=/QOpenSys/usr/bin/env

Variants
{
  "american": "#@$",
  "local": "#@$"
}

Errors
[
  {
    "command": "system \"CRTLIB LIB(ILEDITOR) TEXT('Code for i temporary objects. May be cleared.')\"",
    "code": 255,
    "stderr": "CPF2111: Library ILEDITOR already exists.",
    "cwd": "/home/dfreinkel"
  },
  {
    "command": "system \"DLTOBJ OBJ(ILEDITOR/O_*) OBJTYPE(*FILE)\"",
    "code": 255,
    "stderr": "CPF2125: No objects deleted.",
    "cwd": "/home/dfreinkel"
  }
]

Disconnect and the reconnect once using this option:

It will detect that TN5250 has been installed and updated the installed features cache. Then you'll be able to open a 5250 terminal.

Thank you


Unified approach to string formatting.

Version: 3.15.0
Python: 2.7/3.4/3.5

What was wrong?
Web3 handles user provided strings in the following APIs (there are others but these are the ones that have been problematic)

web3.sha3
web3.eth.sign
Contract function calls that take any of bytesXX, bytes, or string types.

Our users expect to be able to pass in hex encoded data to these methods.
For bytesXX types this fails because the hex strings are too long.
For bytes and string this silently succeeds but doesn't behave as the user expects because it doesn't decode the hex string.
How can it be fixed?
For web3.sha3 and web3.eth.sign we have a workable solution.  See https://github.com/pipermerriam/web3.py/issues/289
For bytesXX we should be able to be smart and detect hex encoded values.
For bytes and string we need a better solution.  I propose the following.

if the type of the value is bytes, pass it through as-is.  This will allow certain silent failures in python2 but we're about to remove support so I'm ok with sweeping that under the rug.
if the type of the value is text, it is required to be valid hexidecimal.

if the type of the value is bytes, pass it through as-is. This will allow certain silent failures in python2 but we're about to remove support so I'm ok with sweeping that under the rug.

Absolutely on board

if the type of the value is text, it is required to be valid hexidecimal.

Mostly on board.
So lets look at all these cases for Python 3: (am I missing any critical ones for this discussion?)

python type of an argument to a contract function call -->

ABI type for the matching function call

bytes

bytes: pass through
bytesNN: pass through, after validating the length
string: pass through, after validating that the bytes can be UTF-8 decoded, else raise a validation error.

str

bytes: to_bytes(hexstr=arg)
bytesNN: to_bytes(hexstr=arg), and validate length
string: to_bytes(hexstr=arg)

Only the last bullet is problematic.
I would be very surprised for my str in python to be treated like hex when being sent to a string ABI type. Instead, I would expect:

str

string: to_bytes(text=arg) aka arg.encode('utf-8')

If I have a contract that has a setName(string) function in solidity, I expect to be able to call contract.setName("my name"). I expect it because I can send in the built-in python type for every other ABI type (int, bytes, bool, etc).
PS~ I don't have a strong opinion about what we do with Python 2, as long as it doesn't add a ton of bloat.

I'm convinced.  We treate string and bytes differently.  And we need to have a big well written, easy to understand, easy to find place in the docs that explains this behavior and good error messages that point people int he right direction.

(Perhaps tangent, since bringing Populus and Solidity into the mix.)
If, for example, there was a contract called FormattingDemo on-chain with a function setString(string), and I wanted to call that function with argument "0xtentacle"; using web3.py and having an object demo for that contract's instance, - what would my Python code look like?
@carver's suggesting it should be
demo.transact().setString("0xtentacle");

What would your approach require, @pipermerriam?

To be clear on what Piper says in OP:

Our users expect to be able to pass in hex encoded data to these methods.

This might be true (pending a census on all users q:p), and I do agree myself.
However, when calling a contract that has a function in its ABI specified as taking a string argument, I also expect to be able to pass it a Python str.

@veox I believe what we are trying to support is both of the following:

demo.transact().setString("0xtentacle");  # that must be a unicode string (so in python2 u"0xtent...")
demo.transact().setString(b"\x30\x78\x74\x65\x6e\x74\x61\x63\x6c\x65");

Ah. Your laconic

I'm convinced.

got me thinking otherwise: that only the latter would be valid, and that the former would raise a Python exception.

I've updated the initial issue description to reflect this change.

Merged, and will be released with v4 beta, soonTM


Bump expensify-common - quotes are required to be followed by space

Details

Bumping expensify-common is connected with change in quotes behaviour - now it's required that quote sign is followed by at least one space
Related Issues

https://github.com/Expensify/App/issues/41696
Manual Tests

Linked PRs

Forgot about web tests - fixed!


error when i tried to pull docker cloudera quickstart from kitematic. os: windows 7 ultimate,

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Please make sure to read the DOCS on the required fields for the cloudera/quickstart image:
https://hub.docker.com/r/cloudera/quickstart/


DAG API 2023-08-01 for Chimera DRv2

ARM (Control Plane) API Specification Update Pull Request

Tip: overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

 Click here to see the diagram footnotes
Diagram footnotes
[1] ARM review queue (for merge queues, see [2])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[X] New API version. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[X] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[X] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the Diagram Step 2, "ARM API changes Review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

we could ignore ❌ MISSING_APIS_IN_DEFAULT_TAG 'cause V5 SDK tag with this latest API version will be updated after 2023-08-01-preview releases.

          "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"

YAY!!!:rocket: :heart: :fire:

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2023-08-01-preview/DistributedAvailabilityGroups.json:50 in d1ed6a8. [](commit_id = d1ed6a8fa2fdcb346572fed894861897cbef1678, deletion_comment = False)

        "$ref": "../../../common/v1/types.json#/parameters/ResourceGroupParameter"

v5? (same for two others below)

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2023-08-01-preview/DistributedAvailabilityGroups.json:302 in d1ed6a8. [](commit_id = d1ed6a8fa2fdcb346572fed894861897cbef1678, deletion_comment = False)

          "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"

YAY!!!:rocket: :heart: :fire:
Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2023-08-01-preview/DistributedAvailabilityGroups.json:50 in d1ed6a8. [](commit_id = d1ed6a8, deletion_comment = False)

SQL RP has not supported v5 common types. the author @ms-mjovkovic  has reverted to v1.
@mentat9 can you help to take a look again? Thanks!

https://github.com/Azure/azure-rest-api-specs/pull/27236/checks?check_run_id=20276601471
For the breaking change. It compares with the old stable version instead of latest preview version. we don't have breaking change on this group of swagger file/API paths.

/pr RequestMerge


[ISSUE #8340] RuntimeInfo and ClusterListSubCommand show ackThreadPoolQueueSize and ackThreadPoolQueueHeadWaitTimeMills

Which Issue(s) This PR Fixes

Fixes #8339
Brief Description

RuntimeInfo and ClusterListSubCommand show ackThreadPoolQueueSize and ackThreadPoolQueueHeadWaitTimeMills
How Did You Test This Change?

Codecov Report
Attention: Patch coverage is 41.66667% with 7 lines in your changes missing coverage. Please review.

Project coverage is 43.11%. Comparing base (b31e0ce) to head (777797a).

Files
Patch %
Lines

...q/tools/command/cluster/ClusterListSubCommand.java
0.00%
7 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8339      +/-   ##
=============================================
- Coverage      43.20%   43.11%   -0.09%
+ Complexity     10524    10509      -15
=============================================
  Files           1274     1274
  Lines          88940    88947       +7
  Branches       11434    11434
=============================================
- Hits           38428    38353      -75
- Misses         45808    45874      +66
- Partials        4704     4720      +16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Iterating through JSON

I made a little JSON file to test
var json: JSON =  ["name": "Jack", "name": "Travis"]
and I want to append each name to my Array that feeds into a table view.
this works to load one key to the table

if let name = json["name"].string {
            self.shows.append(Show(name: name))

        } else {
            print(json["name"].error)
        }

but eventually I want to be able to add large json files with lots of names so I attempted to loop.
if let name = json["name"].string {
            for name in json {
                self.shows.append(Show(name: name))
            }
        }else {
                print(json["name"].error)
}

However I get an error on the self.shows.append(Show(name: name)) line
Cannot Conver value of type '(String,JSON)' to expected arguement type 'String'

Not exactly sure if it helps
if let *name* = json["name"].string {
            for name in json {
                self.shows.append(Show(name: name))
            }
        }else {
                print(json["name"].error)
}

Change that if let name to something else?

feel free to reopen this if needed

@zhigang1992 that didn't help I get the same error, I changed it to title but no difference. Also I am not sure how to open it back up hahah

so what exactly is json, is it a dictionary or a arry?
If it's a dictionary, then there should only be one key named name.
If you're trying to loop through key and values, it should be
        for (name, value) in json {
            self.shows.append(Show(name: name))
        }

@zhigang1992 well it needs to be an array hahaha, but your solution used with ["name": "Jack"] just prints out name not the value of name Jack

if let JSON = response.result.value {                 for title in JSON as! [AnyObject]{                     self.shows.append(Show(name: title as! String, description: "Is the best"))                 }
this worked for me

Closing. Feel free to reopen if need.


Update readme with instructions for firefox

Related - https://github.com/codecov/codecov-browser-extension/issues/50

@thomasrockhu-codecov Please merge this! (Or fix the extension to not require manual configuration)

N


Write a Kotlin program to convert string to snakecase

Description
Write a Kotlin program to convert string to snakecase
Snakecase is a style of writing in which all letters are lowercase and spaces are replaced with underscores.
Input  : "hello world"
Output : "hello_world"

How to contribute

Save the solution in program/ConvertStringToSnakecase.kt file
Add ConvertStringToSnakecase.kt file in convert-string-to-snakecase folder

!assign

!assign


Command type argument

By default all arguments are entered as strings in the Command Line so having an argument to turn a string into a requested type or give a failure message if conversion failed would be very nice.

@thatstoasty Any progress ;)?

@Moosems I spent most of the last month fighting against the changes in nightly trying to keep the library working haha. But now I should be able to look into it.

Great job keeping up with nightly, thats quite a handful

I check in on this every now and then, but still waiting for the type system to improve a bit before trying to tackle it. Too many roadblocks at the moment.


Implement bulk Identity Create/Update in Core API

https://docs.flagsmith.com/api/bulk-insert-identities-update
Currently exists in Edge API. We should implement it in the core API too. Related to https://github.com/Flagsmith/flagsmith/issues/1667

Closing as duplicate of https://github.com/Flagsmith/flagsmith/issues/1212


Fix bug in gitlab CI file

Pipeline now works, but we have a different issue, since the mirror+CI action doesn't support passing over the environment variables in the API call.  I opened an issue: https://github.com/SvanBoxel/gitlab-mirror-and-ci-action/issues/26
but until something is done, this won't work. We could do our own implementation, but probably it would be faster if I open a PR to the action repo


window.cep is always undefined in react template

Hey, Thanks for your great project.
When i try to use evalTS or evalES i get an error:
csinterface.js:608 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'evalScript') at CSInterface.evalScript.
So the interface wasn't loaded right.
When i try to log window.cep it's always log undefined.
How to fix it ?
Thanks in advance!

Sounds like you're testing in the browser if window.cep isn't defined. Try testing in your host app instead. If you have more questions on getting started with Bolt CEP, join our Discord:
https://discord.gg/PC3EvvuRbc


FLAPS: Does not work in 1.6.0 RC1

Hi guys,
I'm just setting up a v-tail airplane with Albatros mixer and the Flaps does not work. I preset RC channel for flaps and nothing happens when switched.
The flaps are working because I test it with servo tester but no functionality from Main Out 7,8.

I tried in Manual and Stabilized on the bench... Nothing happens. Before there was a problem with flaps not fully extending https://github.com/PX4/Firmware/issues/4438. But now they just don't move at all.

To add to the problem I tried to make a workaround for the Flaps and use Aux1 and Aux2 with RC pass-through to set it up with no luck.
There is no RC pass-through with this mixer lines:
Flaps mixer
Flap servos are physically reversed.
M: 1
O:      10000  10000      0 -10000  10000
S: 3 5      0   5000 -10000 -10000  10000
M: 1
O:      10000  10000      0 -10000  10000
S: 3 6      0  -5000  10000 -10000  10000
Nothing moves! I see there are Aux1 and Aux2 input to PX4 because in QGC the sliders move. But nothing is passed to the servos.
Probably there is something wrong with the mixer in general.

same issue here - also not getting my nose wheel moving either - going back to stable and all works fine. probably something to do with mixer size which was an issue a long time back. were you able to make a work around?

@see3peoh I was able to control the nose wheel, this might be something more recent. But the flaps still does not work neither the rc forwarding. The workaround is you use a spare channel on the RC receiver... but you cannot make automation this way except you make the automation in the transmitter mixer, in my case Taranis.

Did you test with current master?

@tubeme  Ok - thanks. Too bad this still isn't working. Thanks for the workaround but chaos for mission auto. Likely something with current master re the nose gear. Loaded up stable and everything worked again just to check. Also on a Taranis.
@LorenzMeier Yes, no go.

How does one systematically look through commits to ascertain which/what broke this? If there isn't a methodical approach, I'm happy to just jump large amounts backwards (of commits to RC1), build and test but something more targeted would be ideal. This is actually more of an issue than I thought as without flaps it causes some takeoff/landing concerns but without the nose wheel, there is no real active auto mission that is working.

There is this one: https://git-scm.com/docs/git-bisect but I suspect that its rather the result of an architectural cleanup on the PWM outputs. Can you provide this output:
pwm info -d /dev/pwm_output0

and
pwm info -d /dev/pwm_output1

and
px4io status

You can go in QGC to the log download tab on the far right and start the MAVLink shell there to launch these commands via USB.

Ping - can anyone provide these outputs with an armed system? I cannot debug with no data.

In PX4 1.6.3 flaps are working perfectly well. But I had to make my own mixer rot the AETR airplane. The same holds for the AERT mixer. Both mixer files lack the line:
Flaps  mixer for CH5,
using the payload control group
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  10000  10000      0 -10000  10000
Once I added this line to the mixer the flaps were alive.

Should be fine as we merged the definitive fix recently.


kdigger: 1.2.1 -> 1.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/quarkslab/kdigger/releases.
meta.description for kdigger is: "An in-pod context discovery tool for Kubernetes penetration testing"
meta.homepage for kdigger is: "https://github.com/quarkslab/kdigger"
meta.changelog for kdigger is: "https://github.com/quarkslab/kdigger/releases/tag/v1.3.0"
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.0 with grep in /nix/store/mxqk87b51bxln6l6pgd44mf68irdsnl7-kdigger-1.3.0
found 1.3.0 in filename of file in /nix/store/mxqk87b51bxln6l6pgd44mf68irdsnl7-kdigger-1.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
kdigger

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mxqk87b51bxln6l6pgd44mf68irdsnl7-kdigger-1.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kdigger https://github.com/r-ryantm/nixpkgs/archive/32577c51330cb1c46cacb6c5439deff2351da446.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mxqk87b51bxln6l6pgd44mf68irdsnl7-kdigger-1.3.0
ls -la /nix/store/mxqk87b51bxln6l6pgd44mf68irdsnl7-kdigger-1.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kdigger

Maintainer pings
cc @06kellyjac for testing.

Result of nixpkgs-review pr 183412 run on x86_64-linux 1

1 package built:

kdigger


URL in description and README broken

The URL in the repository description and the one in the README are pointing to githubtrainingwithsahar's copy of the game instead of yours.
Please fix both so they point to your copy of the game at https://githubtrainingwithsahar.github.io/github-games-pgorazda

fixed


DepthMap (StereoBM) acceleration

Hello @doe300!
User Gavinmc42 on Raspberry Pi forum (https://www.raspberrypi.org/forums/viewtopic.php?f=37&t=216940&p=1408611#p1408600) recommended me to contact you.
I'm a hardware developer, and our team did a stereoscopic camera based on Raspberry Pi Compute Module 3 (http://stereopi.com).
For OpenCV and ROS implementation Depth Map is a critical thing, so I'm looking for ways to increase Pi performance in this task. So I have some questions:

How do you think, if VC4CL is applicable for this function (StereoBM, FindStereoCorrespondenceBM)? As for StereoSGBM it looks like this function is far from real-time, but may be you have some ideas or suggestions here.
I plan to use H264 encoder for depth map acceleration. Yes, DM resolution will be smaller in this case (~16 times), but the main ARM kernels will not be loaded with this approach. Have you plan to use H264 encoder in some ways in your VC4CL?
StereoPi can be used not only in stereoscopic setup, but also for work with two independent cameras simultaneously (like back-to-back for 360 panoramas with 200 degree fisheye cameras, or with two different cameras like classic Pi camera and NoIR edition). If you think these tasks can be accelerated too - I will be appreciate for your ideas.
If these tasks are interesting for you, I can send you one of our boards for you to be able to conduct all experiments you need.

Regards,
Eugene

At this time, VC4CL will probably not be able to compute those (or any) OpenCV functionalities correctly, since it still lacks accuracy especially for mathematical functions.
H264 is a video codec and OpenCL (which VC4CL implements) does not support videos at all. Even if I deviate from the OpenCL standard, I have no idea how to talk to any video-encoding hardware parts. You may be able to do this yourself using the MMAL or OpenMAX interfaces, since this is - AFAIK - the way to go.
Stitching together those images (e.g. to a 360° view for panorama or to some kind of enhanced image for "normal" and NoIR camera) should most definitively be done (at least partially) on the GPU. But the software to use here depends on the performance needed and the exact tasks to perform. Maybe you'll need to write direct GPU assembler or can use the existing OpenGL implementation.
I don't think VC4CL as a software will be particularly useful for your use-case (I could be wrong though), but I will try to be of help where I can.


Read and map values in a single pass

Currently, the JSON mapper first reads the JSON string into a JsonValue structure, and then copies those values into an instance of the specific type it's mapping to.  This is obviously inefficient but makes the implementation simpler.
Instead the mapper should combine the reading/parsing steps with the allocation and assignment of specific values in the target type to remove this redundancy.
e.g.
Currently, when mapping the JSON string "[1,2,3]" to the type int[],  it is first converted into the equivalent of the following JsonValue structure:
var parsed = new JsonValue { new JsonValue(1), new JsonValue(2), new JsonValue(3) };

and then converted from a JsonValue into an int[].
This feature would remove the intermediate JsonValue step and convert directly from string -> int[]

Completed in ee09bb4