Don't AddError for Rollback on ErrTxDone

When db.Rollback fails, check the error code. If it matches sql.ErrTxDone, we won't treat it as an error. This is expected and normal behavior of using Rollback with defer.
e.g:
tx := db.Begin()
defer tx.Rollback()
...
return tx.Commit().Error

See more discussion here: https://github.com/go-gormigrate/gormigrate/issues/36

Codecov Report

Merging #2434 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2434      +/-   ##
==========================================
- Coverage   80.15%   80.09%   -0.06%
==========================================
  Files          24       24
  Lines        3431     3432       +1
==========================================
- Hits         2750     2749       -1
- Misses        578      580       +2
  Partials      103      103

Impacted Files
Coverage Δ

main.go
82.58% <0%> (-0.46%)
: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 5c0f671...c17255f. Read the comment docs.

@javie2007 what do you think it would take to get this merged? The coverage could be a little tricky, not sure how to trigger an appropriate error in Rollback.

Ping @jinzhu @emirb


Fails to load on Fully Android browser since 1.3.0

I am using the Fully Kiosk Browser (https://www.fully-kiosk.com/#get-kiosk-apps) on an Android tablet with Vis.
That worked well up to version 1.2.12. However I have tried both 1.3.0 and 1.3.4 and only see the loading spinner rotating forever.
I tried that with both Firefox and Chrome as backends/webviews and apparently the problem started with the switch from 1.2.12 to 1.3.0.
Firefox and Chrome themselves work properly on Android. Just the webview used by the kiosk browser starts to break. Unfortunately I couldn't debug that any better. So all I can tell you is the versions I tried.
I would appreciate if you found time to look into it. In my opinion a kiosk browser is the perfect tool on a cheap chinese tablet. :)

Android version?

Android version | 8.1 (SDK 19)
Webview version | <IP_ADDRESS>
Webview UA | Mozilla/5.0 (Linux; Android 8.1; T906 Build/KOT49H) App...
Device model | T906 (HYD)

I have an similar issue. For my old device "new" javascript feature are not supported. for example:
formatMomentDate: function formatMomentDate(dateObj, _format, useTodayOrYesterday = false) {

Error message:
Unexpected token =; /vis/js/vis.js@L2253

So default values with syntax " = false" not allowed.

I have the problem too, can I install the Github version or should I wait?

Please test 1.3.6

@GermanBluefox Updated. Works on Fully. Thanks.


[BUG] Mantine AuthPage hooks incorrect

Describe the bug
The AuthPage exported by refine-mantine is calling the useLogin() hook for the Register, ForgotPassword, and UpdatePassword instead of their respective hooks exported from the core library. As a result; filling out the form and submitting on any of these pages will result in the login function being called in the authProvider.
Steps To Reproduce

Go to any of the pages (Register, ForgotPassword, UpdatePassword) using the refine-mantine library
Fill out the form and submit
The login() function in authProvider will be called instead of the expected function

Expected behavior
Expecting the correct functions inside of the authProvider to be called onSubmit (Register should call register, etc...)
Screenshot
https://github.com/pankod/refine/blob/2d50ce85e2907a8c2345ad26da150ad324ac544c/packages/mantine/src/components/pages/auth/components/forgotPassword/index.tsx#L64-L65
https://github.com/pankod/refine/blob/2d50ce85e2907a8c2345ad26da150ad324ac544c/packages/mantine/src/components/pages/auth/components/register/index.tsx#L63
https://github.com/pankod/refine/blob/2d50ce85e2907a8c2345ad26da150ad324ac544c/packages/mantine/src/components/pages/auth/components/updatePassword/index.tsx#L61-L62
Desktop
No response
Mobile
No response
Additional Context
No response

Hey @dgelineau ,
Thank you for reporting issue. Anyone want to do? 👀


Miri: use extern fn to expose interpreter operations to program; fix leak checker on Windows

This is the Miri side of https://github.com/rust-lang/rust/pull/74681.
Fixes https://github.com/rust-lang/miri/issues/1302
Fixes https://github.com/rust-lang/miri/issues/1318

Not sure where the best place to document these is, but we should definitely document them. I think somewhere near the stacked borrows debugging command line flags.

@oli-obk what about this?

@bors r=oli-obk

:pushpin: Commit c641fbde0223cde9fae67de3cc4892998a21cec0 has been approved by oli-obk

:hourglass: Testing commit c641fbde0223cde9fae67de3cc4892998a21cec0 with merge 91b58c93c5ab18969e7bdce3bf3e110c9b150d4b...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing 91b58c93c5ab18969e7bdce3bf3e110c9b150d4b to master...


is it possible headless mode?

HI, inspirai.
I'm trying to train my model. I wonder is it possible headless mode? because, Unity enviornment is paused frequently. I had to click client for tackling this problem always... when I trained on windows.
In case of linux version, can possible headlees mode?

Hi, Leejw,
Thank you for the question. I am not sure if you are saying that windows version of the backend periodically stops running? In my experience, I did not encounter that often. But in case you do have the problem, we recommend you switching to the linux backend, which is tested more extensively and you shall probably not have this issue any more.

Hi, FateDawnLeon
I tried to switch a backend from windows to linux. however in my case, track2 battle senario occurs a problem that the simulator is paused when I trained Agent on windows and linux.
other track is fine.
thanks.

Hi, LeejwUniverse
I guess maybe the problem is that you set more than one supply_refresh events in a game episode. As we have tested, this may slow down the game significantly. We suggest you always have no more than one supply_refresh event when running the game.
Thanks.


Message editor deletes what I'm writing

OK, I finally got to reply a note, but I can't write almost anything, the text go every time (almost at every sentence I write, the message editor erase it and make me start over.

I need to reproduce this or at least understand, how this happens on your device. Please set logging to Verbose, reproduce the problem and send me logs, created during the test.
Please use different text for each attempt,  e.g. different beginning. E.g. "Test 001...", "Test 002...", so these words could be easily found in logs.

I see that something is wrong with current implementation. Thinking, how to fix this...

Good, I haven't seen the bug again for now but I did with the other bug so I'll upload a screenshot on the computer

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>#AndStatus v.19.1 is available at the Beta testing channel, please opt in here: https://play.google.com/apps/testing/org.andstatus.app
In this release I'm fixing a bug with Message Editor, which sometimes prevents from sharing a message, sometimes causes loss of last edits to a message draft.

AndStatus v.19.2 is available at the Beta testing channel, please opt in here: https://play.google.com/apps/testing/org.andstatus.app
In this release I'm fixing a bug with Message Editor, which sometimes prevents from sharing a message, sometimes causes loss of last edits to a message draft.

Supposedly fixed in v.20.4


runcfanotify: fallback when pidfd_open fails

fanotify hook without pidfd_open
pidfd_open was introduced in Linux 5.3. Introduce a fallback mechanism
for older kernel.
How to use
The code automatically detects if pidfd_open can be used.
Testing done
TODO

This is done without interfacing with the podinformer, so that the runcfanotify package can be independent and be reused in other projects.


grammar

@tbertenshaw : Thanks for your contribution! The author, @jovanpop-msft, has been notified to review your proposed change.

@tbertenshaw great catch!! Thank you 👍
#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @jovanpop-msft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver : Thanks for your contribution! The author, @, has been notified to review your proposed change.


Parameter "country" is required

[2021-09-17 17:53:48.573 +0200] ERROR (40 on a70d1f157429):
        errorCode: "errors.com.epicgames.accountportal.parameter_required"
        message: "Parameter \"country\" is required"
        correlationId: "72505860-17cf-11ec-ad3f-f7249874c905"
[2021-09-17 17:53:48.573 +0200] ERROR (HTTPError/40 on a70d1f157429): Response code 400 (Bad Request)
        HTTPError: Response code 400 (Bad Request)
        at PromisableRequest.<anonymous> (/usr/app/node_modules/got/dist/source/as-promise/index.js:124:28)
        at processTicksAndRejections (internal/process/task_queues.js:97:5)

I first posted this in  #16 however hear is clear what is causing the error: Parameter \"country\" is required.
How to fix that?

duplicate of #137


Apply to become members of Open Astronomy

Instructions at https://github.com/OpenAstronomy/openastronomy.github.io/wiki/New-member-on-Open-Astronomy

In progress: https://github.com/OpenAstronomy/openastronomy.github.io/pull/94

Now we are members of OpenAstronomy! :tada:
http://openastronomy.org/members/


Linking to hue account online

Hi, I was able to program everything and have it working. I want to add some IFTTT applets to my google home connected to a Vera Plus thru HA-Bridge... I am able to "find" a Hue HUB with the latest ha-bridge and hue app... My issue now is when I want to log in to meethue.com and link the hub, it does not find it.
Im running the server on a windows 10 PC, Ha-Bridge 4.5.6, hue app version 2.13.0... only the meethue.com account cannot be linked as it request to "press" the button again on the hub. Not sure if I'm explaining my self clear.. but thats my question... how to link the virtual Hub to meethue.com account so I can give permission to access IFTTT applets.
Thanks

Since the ha-bridge is not a valid hue product, it just emulates the local api, the linking the ha-bridge to meethue.com never occur. It is to be used only locally.


Adapt to Create Mod

Allow to put the latex liquid into Item Drain and Fluid Tank. Maybe this's a very tiny feature I think is important:}

i think you can do that already, test before posting

i think you can do that already, test before posting

I 've already test! Create Mod version is 0.5.1g,click the Item Drain the latex bucket just jumped away!


Misc Improvements

Modified line 49 to use size parameter instead of static value
Modified line 159 to conform to PEP8
Removed unused sys library import

Thanks!


A few new domains from the website klikponudba.com

Hello,
there are some suspicious 3rd-party requests on the website klikponudba.com coming from the various domains, which might be relevant for adding them to hosts, such as:

boostercentral.com
boostifytheme.com
cdn.codeblackbelt.com
cdn.ghostmonitor.com
cdn.judge.me
cdn.pushowl.com
cdn-spurit.com
d1u9wuqimc88kc.cloudfront.net
dashboard.wheelio-app.com
disco-flipclock.netlify.app
easygdpr.b-cdn.net
monorail-edge.shopifysvc.com
multifbpixels.website
sp-seller.webkul.com
sr-cdn.azureedge.net
sy.salesmanago.pl
wheelioapp.azureedge.net
window-shoppers.azurewebsites.net

Related discussion: https://github.com/badmojr/1Hosts/issues/59

Hello @betterwebleon thank you for this.
I'm going to pass on these.  Primary reason is, to add these to my personal ad-hoc list, I need the ability to curate these out someday, as appropriate.  The assertion that these are "suspicious 3rd-party requests" doesn't give me any insight.  I won't list a site unless I can see a tangible, actionable reason.
Thank you for the suggestion, though.  Perhaps one of our curated sources would take these?

Thank you for suggestion! 👍


Create the empowerment section

Description
Implement the empowerment section as per the design. Also implement the mobile responsive design. Try to ensure the component is pixel perfect.

Scenarios
A user needs to view the empowerment section as per the design
A user can view the desktop and mobile version
Tasks

[ ] Implement the section as per the design
[ ] Use GitHub CoPilot for the project to generate code suggestions
[ ] Implement hover states on the button element
[ ] Submit PR as per the contributing guidelines

Hi @Shubhankar-1 I have assigned you the issue. Remember to still use Copilot for this section


Reset our fork to latest upstream

And reapply relevant changes

Note that this is not actually running the github actions test on zenika/alpine-chrome (unlike the upstream on gitlab). It seems happy enough on the existing behat-chrome/docker-chrome-headless image for now, since they've pinned that to chrome:127


Date Range - Add additional validations

Reviewing Production DB full reindex to Solr. These errors occur in live data and we need better validation rules to keep Solr happy.

[x] Error adding field 'gbl_dateRange_drsim'='[[1839 TO ]]' msg=str is null or blank
[x] Error adding field 'gbl_dateRange_drsim'='[[1851 TO 1851?]]' msg=Couldn't parse date because: Improperly formatted datetime: 1851?
[x] Error adding field 'gbl_dateRange_drsim'='[[1681 TO 1676]]' msg=Wrong order: 1681 TO 1676

Fixed in https://github.com/geobtaa/geomg/commit/7dbb87612eeeeb1e17ad2938898394e22267ce1f


current_app.logger.info streamhandler

Dear Miguel,
Trying log info but can not do it.
@main.route('/', methods=['GET', 'POST'])
def index():
current_app.logger.info('infoo')
current_app.logger.warning('warningg')
warning Works but not info. I think info should also work since streamhandler level is info.
Thanks in advance

You should set the level also on the logger, not just the handler.


SB-1339: Improve privileges, roles and access model

@steve-todorov, the Authorization Config API now looks like this:
GET http://localhost:48080/api/configuration/authorization

Response
{
   "roles":[
      {
         "name":"CUSTOM_ROLE",
         "description":"Custom role configuration example",
         "accessModel":{
            "apiAuthorities":[
               "CONFIGURATION_VIEW"
            ],
            "storageAuthorities":[
               {
                  "storageId":"private-storage",
                  "repositories":[
                     {
                        "repositoryId":"private-releases",
                        "repositoryPrivileges":[
                           "ARTIFACTS_VIEW",
                           "ARTIFACTS_RESOLVE"
                        ],
                        "pathPrivileges":[
                           {
                              "path":"org/carlspring/*",
                              "wildcard":true,
                              "privileges":[
                                 "ARTIFACTS_DEPLOY",
                                 "ARTIFACTS_DELETE"
                              ]
                           }
                        ]
                     }
                  ]
               }
            ]
         }
      },
      {
         "name":"ADMIN",
         "description":"Super User role with all possible privileges",
         "accessModel":null
      },
      {
         "name":"GLOBAL_CONFIGURATION_MANAGER",
         "description":"Role with all global configuration related privileges",
         "accessModel":{
            "apiAuthorities":[
               "CONFIGURATION_UPLOAD",
               "CONFIGURATION_VIEW",
               "CONFIGURATION_SET_INSTANCE_NAME",
               "CONFIGURATION_VIEW_INSTANCE_NAME",
               "CONFIGURATION_SET_BASE_URL",
               "CONFIGURATION_VIEW_BASE_URL",
               "CONFIGURATION_SET_PORT",
               "CONFIGURATION_VIEW_PORT",
               "CONFIGURATION_SET_GLOBAL_PROXY_CFG",
               "CONFIGURATION_VIEW_GLOBAL_PROXY_CFG",
               "CONFIGURATION_ADD_UPDATE_STORAGE",
               "CONFIGURATION_VIEW_STORAGE_CONFIGURATION",
               "CONFIGURATION_DELETE_STORAGE_CONFIGURATION",
               "CONFIGURATION_ADD_UPDATE_REPOSITORY",
               "CONFIGURATION_VIEW_REPOSITORY",
               "CONFIGURATION_DELETE_REPOSITORY",
               "CONFIGURATION_ADD_LOGGER",
               "CONFIGURATION_UPDATE_LOGGER",
               "CONFIGURATION_DELETE_LOGGER",
               "CONFIGURATION_RETRIEVE_LOG",
               "CONFIGURATION_RETRIEVE_LOGBACK_CFG",
               "CONFIGURATION_UPLOAD_LOGBACK_CFG"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"REPOSITORY_MANAGER",
         "description":"Role with all repository related privileges",
         "accessModel":{
            "apiAuthorities":[
               "ADMIN_CREATE_REPO",
               "ADMIN_UPDATE_REPO",
               "ADMIN_DELETE_REPO",
               "ADMIN_LIST_REPO"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"ARTIFACTS_MANAGER",
         "description":"Role with all artifact related privileges",
         "accessModel":{
            "apiAuthorities":[
               "ARTIFACTS_DEPLOY",
               "ARTIFACTS_DELETE",
               "ARTIFACTS_VIEW",
               "ARTIFACTS_RESOLVE",
               "ARTIFACTS_COPY"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"USER_MANAGER",
         "description":"Role with all user related privileges",
         "accessModel":{
            "apiAuthorities":[
               "CREATE_USER",
               "UPDATE_USER",
               "VIEW_USER",
               "IMPERSONATE_USER",
               "DELETE_USER"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"TOKEN_MANAGER",
         "description":"Role with all token related privileges",
         "accessModel":{
            "apiAuthorities":[
               "VIEW_OWN_TOKEN",
               "VIEW_ANY_TOKEN"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"LOGS_MANAGER",
         "description":"Role with all logs related privileges",
         "accessModel":{
            "apiAuthorities":[
               "VIEW_LOGS",
               "CONFIGURE_LOGS",
               "RSS_FEED"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"UI_MANAGER",
         "description":"Role with all user interface related privileges",
         "accessModel":{
            "apiAuthorities":[
               "UI_LOGIN",
               "UI_BROWSE"
            ],
            "storageAuthorities":[

            ]
         }
      },
      {
         "name":"ANONYMOUS",
         "description":"Anonymous user role",
         "accessModel":{
            "apiAuthorities":[
               "ARTIFACTS_RESOLVE",
               "SEARCH_ARTIFACTS"
            ],
            "storageAuthorities":[

            ]
         }
      }
   ]
}

Users API
GET http://localhost:48080/api/users

{
   "users":[
      {
         "username":"admin",
         "enabled":true,
         "roles":[
            "ADMIN"
         ],
         "securityTokenKey":"admin-secret"
      },
      {
         "username":"deployer",
         "enabled":true,
         "roles":[
            "ARTIFACTS_MANAGER",
            "UI_MANAGER"
         ],
         "securityTokenKey":"deployer-secret"
      }
   ]
}

There can be follow up tasks:

provide a proper REST API satisfying the UI requirements if needed
restrict modification of "System" roles

@carlspring @fuss86 @steve-todorov thanks for review
all comments now resolved

@carlspring / @fuss86  do you guys have more comments on this PR? It looks good to me, I've tried it and it works. I'll need to do some work for the role management, but we should merge this to avoid blocking other PRs. :)
I have opened strongbox/strongbox-web-ui#23 which will fix the non-working UI and will add back the functionality in the upcoming days.

I think we should merge it :)


OPR grades aren't normalized

As far as I can see in the proposal paper there's no normalization for OPR grades, meaning high dollar value assets have significantly more weight.
Assume there are only two assets with averages for the 50 highest POW OPR records as follows.
Bitcoin: $10,000
Euro: $1.10
Alice submits an OPR with the following.
Bitcoin: $10,005
Euro: $1.10
Alice's grade is 5^4 + 0^4 = 625
Bob maliciously submits an OPR with the following:
Bitcoin: $10,000
Euro: $2.10
Bob's grade is 0^4 + 1^4 = 1
Bob gets a much lower grade despite submitting a malicious entry showing the Euro value to be worth almost twice as much. To fix this the OPR grades could normalize to the average before squaring.

The grades are actually normalized in the code: https://github.com/pegnet/pegnet/blob/develop/opr/grading.go#L53
This would need to be updated in the proposal paper @PaulSnow .

Thank you.


Authenticate commands are coming from Slack

As described in validating commands in Slack docs.

Landed on master.


Review script for paper "2024ApJS..271...46M" made with predictor "DataLinkerCondenseDetailsWithTemplate"

Created by Paper Data Linking Bot
Please review the download script and instrument details for the paper.
bibcode: 2024ApJS..271...46M
predictor: DataLinkerCondenseDetailsWithTemplate

NSO/GONG is not an instrument in the VSO. Month/day for mdi_end_date is unclear, but 2010-2011 is correct. SDO/HMI instrument is not mentioned in the script. From article: "NSO/GONG and SOHO/MDI act as our source instruments and SDO/HMI as our target instrument."


Add some code verification tests to the programmatic DynamoDB module

This change adds some tests to the DynamoDB module, demonstrates testing with DynamoDB local and Gradle, and fixes some null pointer exceptions in DynamoDBProgrammaticKeyValue related to the fact that the encapsulated KeyValue was not delegated to for writeOnce, writeBulk, readOnce, and readBulk

@ipapapa this is what we discussed earlier today.

The build is failing. Would you like to check that?

@amcp Now that the other PR has been merged, would you like to move forward with this one as well?

@ipapapa @vinaykumarchella i am good to move forward now.


Mail support in Certificate Resource

Is your feature request related to a problem? Please describe.
Hello,
First of all, thanks for your wonderful work :)
I create this issue because I would like to create certificates for my users with the e-mail in the Sublect Alternative Names.
Today, we can only put DNS and URIs, and without E-mail address in the SANs, they're unable to sign and encrypt their messages.
Describe the solution you'd like
Two possibilities in the Certificate resource :

a new field (emailAddresses ?) as String[], but you'll have some other feature requests for other kinds of SANs
or a generic field (SANs ?) as String[] where the user puts whatever he wants

Describe alternatives you've considered
Putting directly the e-mail in the commonName field, but it doesn't do what I expected.
Environment details (if applicable):

Kubernetes version (e.g. v1.10.2): K3s 1.16
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): Myself
cert-manager version (e.g. v0.4.0): latest
Install method (e.g. helm or static manifests): static manifests

/kind feature

This is a dupe of #1356 and is being implemented in #2597!


feat: add account selection functionality to /sign

Adds account selection functionality to /sign wallet integration point. Closes #2399 and #2396:

I need this so bad - currently super painful as it is

Closing in favor of #2447 after discussion on #2402


Incompatibility between Flink CDC master and Tencent cloud MySQL

flink version:1.13.0
flink-cdc version: master branch
desc:
When I used version 1.4.0, when I tested the CDC on the local server and Tencent server, I found that the CDC lock time on the local server was relatively short, but it took about 10 minutes on Tencent cloud. The number of tables in the two libraries is about 270, and the data is about 1000W. When you see that there is a CDC on the CDC master branch that supports lockless tables, you download and compile it for verification.
When testing in the local database, it is found that the CDC does not lock the data table, and the data can be carried out quickly。
But later, when we did CDC on Tencent cloud, we found that data synchronization has not been carried out, and it has not been carried out for more than an hour. Finally, we started to throw exceptions directly. The specific implementation results are as follows

DDL:
CREATE TABLE tb_qy_learnonline_personal_course(
id int,
course_id varchar ,
create_time TIMESTAMP(9) ,
start_time TIMESTAMP(9) ,
end_time TIMESTAMP(9) ,
state tinyint ,
expend int ,
finish_enclosure_count int ,
user_id varchar ,
person_name varchar,
mobile varchar ,
head_pic varchar ,
wx_user_id varchar,
department_name varchar ,
department_id varchar,
recent_id varchar ,
org_id varchar ,
credit int ,
score float ,
remove tinyint,
is_pass tinyint ,
finish_rate float ,
map_id int,
finish_time TIMESTAMP(9) ,
remind_time date ,
is_mind tinyint ,
times smallint,
PRIMARY KEY (id) NOT ENFORCED
)WITH(
'connector' = 'mysql-cdc',
'scan.startup.mode' = 'initial',
'debezium.snapshot.fetch.size' = '1000',
'server-id' = '5405',
'hostname' = 'gz-cdb-7v4po3cb.sql.tencentcdb.com',
'port' = '62463',
'username' = 'root',
'password' = 'root',
'database-name' = 'test',
'table-name' = 'tb_qy_learnonline_personal_course'
);
CREATE TABLE tidb_learnonline_personal_course (
id int,
course_id varchar ,
create_time TIMESTAMP(9) ,
start_time TIMESTAMP(9) ,
end_time TIMESTAMP(9) ,
state tinyint ,
expend int ,
finish_enclosure_count int ,
user_id varchar ,
person_name varchar,
mobile varchar ,
head_pic varchar ,
wx_user_id varchar,
department_name varchar ,
department_id varchar,
recent_id varchar ,
org_id varchar ,
credit int ,
score float ,
remove tinyint,
is_pass tinyint ,
finish_rate float ,
map_id int,
finish_time TIMESTAMP(9) ,
remind_time date ,
is_mind tinyint ,
times smallint,
PRIMARY KEY (id) NOT ENFORCED
)WITH(
'connector' = 'jdbc',
'driver' = 'com.mysql.cj.jdbc.Driver',
'lookup.cache.max-rows' = '3000',
'lookup.cache.ttl' = '10s',
'lookup.max-retries' = '3',
'url' = 'jdbc:mysql://HOST/ddp_test?autoReconnect=true&autoReconnectForPools=true&useUnicode=true&characterEncoding=UTF-8&zeroDateTimeBehavior=convertToNull&allowMultiQueries=true&serverTimezone=Asia/Shanghai&rewriteBatchedStatements=true',
'table-name' = 'tb_qy_learnonline_personal_course',
'username' = 'root',
'password' = 'root'
);
INSERT INTO tidb_learnonline_personal_course SELECT o.* FROM tb_qy_learnonline_personal_course AS o;
MYSQL: INSERT
INSERT INTO test.tb_qy_learnonline_personal_course(id, course_id, create_time, start_time, end_time, state, expend, finish_enclosure_count, user_id, person_name, mobile, head_pic, wx_user_id, department_name, department_id, recent_id, org_id, credit, score, remove, is_pass, finish_rate, map_id, finish_time, remind_time, is_mind, times) VALUES (1, '8407c2b2dc9849169a88cd093b511836', '2017-08-17 10:41:11', NULL, NULL, 3, 0, 0, 'd3781ed48a2349dcb3fa9a0a029fc455', 'SDFW', '15113020772', 'http://shp.qpic.cn/bizmp/ck4hSGq23RjYQCezUpgrc2ibR1cxEOdxagrDCQMzMO2lTV6DDwDq4aQ/100', '15113020772', 'SADASD', '450d76a33899169e9d6ab8d4933322aa', NULL, 'b17efb43-292e-4cc9-ac5d-0b46bce059c4', 0, NULL, 0, 2, 0.0000, 0, NULL, NULL, 0, 0);

Later, I passed the verification in other machines. Maybe it was because of my own machine network that I failed


docs/local_docker: fix documentation  and bootstrap

Starting a PR to fix #2378

Can someone validate and merge this, please?

Hi @pires -- running kubernetes inside of docker isn't something we are testing and supporting right now.  While most components will run just fine, the kubelet itself does some host side management that isn't appropriate to run in a container all of the time.
Specifically, we'll adjust install IPtables for service redirection (which might work with --net=host) and we'll create/mount/format/manage volumes.  The volumes thing is really a sticking point.
Eventually I'd love to support testing running the bulk of kubernetes in docker containers (on kubernetes, of course) it isn't a super high priority right now.  As such, this stuff is likely to break often.
With that in mind, I'm going to remove the run-image stuff from build/* and let @proppy move some of this stuff to contrib if he likes.
Most of my effort is going to be going in to https://github.com/GoogleCloudPlatform/kubernetes/issues/2303 right now.

Also see #19 for eventually completing this work.

Thanks @jbeda for clarifying. For me, as Mac-user, it's been really a major PITA to get k8s cluster up&running. For now, I've been using GCE but it comes at a cost :-(
Will follow #2303.

@pires, it would be nice to have static VMs image (build with packer?) with a preconfigured cluster, should we start a new issue about that.

That would be great. How can I help @proppy?

@pires let's discuss that on a new issue. (care to file it?).

@proppy sure thing #3244

Closing this as we are putting "kubernetes in docker" on the back burner.  When we pick it up we'll track it in #19.


flannel add option --public-ip it will delete subnet from etcd every 24 hours

in  k8s cluster some node i add option "--public-ip" when flannel start, and some not add this option, the node add option "--public-ip" will delete subnet from etcd, other node not add option "--public-ip" not delete subnet from etcd, why???

etcd network config is '{"Network":"<IP_ADDRESS>/16", "Backend":{"Type":"vxlan"}}'
the normal flannel start option ’--etcd-endpoints="http://etcd_host_ip:4003" --iface="eth0"‘
the unnormal flannel start option  '--etcd-endpoints="http://etcd_host_ip:4003" --public-ip="node_ip"'
fail log is 'vxlan.go:349] Route for <IP_ADDRESS> not found'

@wksw Could you run with v0.7.0 which has improved logging in this area? And let me know the full output?

@tomdee issue solved,Time synchronization problem.


Question: Setting connection properties in spring application.yml

Using Spring with TomcatCP, I can do this:
spring:
  datasource:
    driverClassName: oracle.jdbc.OracleDriver
    tomcat:
      connectionProperties: defaultRowPrefetch=1000;

How would I set the defaultRowPrefetch when using Hikari? Can I do it only with Spring config files?

I’m pretty sure you can, but you’ll need to ask on stackoverflow or the Spring groups.

Linking to this SO question because Google brought me here ...
https://stackoverflow.com/questions/34856811/how-to-set-custom-connection-properties-on-datasource-in-spring-boot-1-3-x-with/55648779#55648779
(Despite the URL this is relevant for HikariCP and current versions of Spring Boot >= 2.x)


Provide instructions and tools to migrate from Ark to Velero

Current installs will need to have their existing CRDs as well as the deployments migrated to our new API groups, images, and namespace.
We need to provide instructions and tooling to help users accomplish this move.

We just started to use this and all of sudden it got migrated make us wonder where to start now. Anyways. If you can let us know will be able to create a helm chart as well to support your utility will be great as we were using the ark helm chart

@Sam123ben we'll be officially releasing v0.11 with the new name next week. the Helm chart for ark has historically been community-maintained, and gets updated following a new release of the product. I'd expect that to continue after we release v0.11.
Going forward, the core team also wants to be more proactive in assisting with getting the chart updated in a timely manner.
Closing this issue as the migration instructions have been merged into master.


Console.md

Command get start .html

Setting DHCP/Ports Apache Server

Not taking pull requests from folks not working at IBM
https://github.com/IBM/IBM-Z-zOS/blob/main/CONTRIBUTING.md#contributing-in-general


eks-distro-base: split presubmits into job for each image

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


Model gpt 4 don't work

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
The gpt 4 template does not work. The package says: Open API error model not found
Describe the Bug
.

The model: gpt-4 does not exist"

The model: gpt-4 does not exist"

same，with accesstoken ， did you solved

This project is undergoing a major revamp; closing out old issues as part of the prep process.
The chatgpt package is pretty outdated at this point. I recommend that you use the openai package or the openai-fetch package instead.


Add the ability to scroll with the mouse in terminal mode

Currently (Vim 8.0.1400), I can scroll a terminal window using the mouse only in Normal mode. I would really like the ability to scroll while in Terminal mode.

How would this work?  The job running in the terminal might want to do the scrolling.  E.g. when you run Vim in a terminal window, you scroll inside Vim, not the text in the terminal window.
If you are thinking of scrolling back to see the output of a "dumb" command,  we would another mode for that.  But perhaps we can check the terminal emulator if it's passing mouse events to the job.

If you are thinking of scrolling back to see the output of a "dumb" command, we would another mode for that. But perhaps we can check the terminal emulator if it's passing mouse events to the job.

If by “dumb” command you mean a possibly interactive tool that just prints to stdout/stderr, yes, that's what I mean. For example, if I open a shell in a terminal window and I type ls -l to see a long listing, to scroll back and forth through the listing I have to enter Normal mode. It is not a big deal, but feels a bit unnatural.
For TUIs like Vim, scrolling already works as expected when I run them inside a terminal window.

Do not know whether it is relevant or not.
:help terminal-typing
To change the keys you type use terminal mode mappings, see :tmap.
These are defined like any mapping, but apply only when typing keys that are
sent to the job running in the terminal.  For example, to make F1 switch
to Terminal-Normal mode:
tnoremap <F1> <C-W>N
You can use Esc, but you need to make sure it won't cause other keys to
break:
tnoremap <Esc> <C-W>N
set notimeout ttimeout timeoutlen=100

@lifepillar FWIW, the following is what I have for this.
<ScrollWheelUp> will enter terminal-normal mode & <RightMosue> will exit out of normal mode - they can be changed of course.  I'd actually tried to combine  and line("w0") but that was unreliable.
function! ExitNormalMode()
    unmap <buffer> <silent> <RightMouse>
    call feedkeys("a")
endfunction

function!
As @brammool said, the job running in the terminal might want to do the scrolling, and those mappings would prevent that (e.g., they prevent using the mouse to scroll a buffer in an instance of Vim run inside a Vim terminal window), but I think that, combined with a mapping to toggle mouse reporting, they can work fine for me.

It seems as though we could solve this by mimicking other terminal emulators do.  The ideal behavior would be that the scroll wheel scrolls through the terminal scrollback buffer by default (and any other keypress send the screen back to the bottom), but that applications inside the terminal can request mouse access the same way vim does when running in a terminal when configured with set mouse=a.
Is that reasonable/possible?
(and with that, we've exhausted my knowledge of how terminals and mouse reporting works)

This is the functionality that I get when using tmux with mouse bindings enabled. For applications like vim that capture the mouse themselves I just scroll within the application but when I'm at my shell prompt the mouse will scroll through my history buffer.  When I hit 'q' it exits the scrollback and returns me to my prompt.

Another consideration is that when a terminal window is in Normal mode, if the running application is continuing to output, the window scrolls with that output in a weird way. Ideally it would be like other terminal emulators in that when you're at the bottom it scrolls with output, but when scrolled up in the buffer it does not. In Vim's terminal mode, a simple implementation that would suit this might be to simply disable scrolling on output in Normal mode, and only scroll on output in Terminal mode. Mouse scroll in Terminal mode could simply initiate Normal mode, and then scrolling works as normal therein.

I do this:
tnoremap  <C->
And then  whenever I need to scroll back up, using gg to go all the way up, then G to come back down, (or  for page by page). I press 'i' or 'a' from any page to enter commands in the shell. Simplicity itself.

I do this:
tnoremap <c-b> <c-\><c-n>
And then <C-b> whenever I need to scroll back up, using gg to go all the way up, then G to come back down, (or <C-F> for page by page). I press 'i' or 'a' from any page to enter commands in the shell. Simplicity itself.

Really appreciate the snippet @damnskippy. This has made my terminal experience so much more comfortable.
I just wanted to add my riff on it that works with re-entry into terminal mode when you scroll past the end of the buffer. There's one variable defined within the function, overscroll which determines how far you have to scroll past the end of the buffer (as a fraction of the window height) before you'll re-enter the terminal.
    function!  ExitTerminalNormalModeIfBottom()
        call feedkeys("3\<c-e>") " retrigger default scroll
        let overscroll = 0 " fractional window overscroll to go terminal mode
        let winh = winheight(winnr())
        let is_terminal_normal = &buftype == 'terminal' && mode('') == 'n'
        let at_buf_end = (line('w0') + winh) >= (line('$') + winh * overscroll)
        if is_terminal_normal && at_buf_end | call feedkeys("i") | endif
    endfunction

    " scrolling up enters normal mod in terminal window, scrolling past end
    " of buffer triggers re-entry into terminal mode
    tmap <silent> <ScrollWheelUp> <c-w>:call EnterTerminalNormalMode()<CR>
    nmap <silent> <ScrollWheelDown> :call ExitTerminalNormalModeIfBottom()<CR>

@manjus3 you just saved me. this is just a brilliant. thanks mate

@damnskippy muchas gracias. thank you very much.

Simplification of dgkf's mappings in vim9. Comparing line('w$') == line('$') seems to be enough for me
def g:TermWheelUp()
	# Enter normal mode and send wheel up, unless the first line is visible
	feedkeys("\<c-w>N\<ScrollWheelUp>", 'x') # can't use line('w0') in terminal mode
	if line('w0') == 1
		feedkeys('i', 'x')
	endif
enddef
def g:TermNormalEnd()
	# Exit normal mode if last line is visible
	if &buftype == 'terminal' && mode(0) =~ '[nv]' && line('w$') == line('$')
		feedkeys('i', 'x')
	endif
enddef
def g:TermSelect()
	setlocal nonumber signcolumn=no foldcolumn=0
	feedkeys("\<c-w>N\<LeftMouse>", 'x')
enddef

tnoremap <silent> <ScrollWheelUp>   <c-w>:call TermWheelUp()<CR>
nnoremap <silent> <ScrollWheelDown> <ScrollWheelDown><c-w>:call TermNormalEnd()<CR>
# select text by dragging with left mouse
tnoremap <silent> <LeftMouse>       <c-w>:call TermSelect()<CR>


THRIFT-5564: setup java and kotlin lib building

setup java and kotlin lib building
this is a second step after:

#2592

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

I'm noticing the job name is quite long, which makes it hard to see the matrix params in the PR display. You can probably shorten it from "Build Thrift Compiler and Libraries" to just "Build" or similar.
You can also reduce the number of items in the matrix, and the output, by not making the Java distribution a matrix. It's probably sufficient to just use a single stable Java distribution. If temurin works, then just use that, but don't need to put it in the matrix.
You can also probably remove the Java 8 builds, since Java 11 should be using --release 8 when it builds anyway, if the intent is to support Java 8. It'd be more useful to include Java 17 (which is the current LTS), and Java 11 (former LTS, still popular).

It'd be more useful to include Java 17 (which is the current LTS)

for now we can't because we are on gradle 6.9; we'll have to upgrade to gradle 7 first but that in turn replies on migrating from maven to maven-publish plugin

Just want to drop the usual reminder of maintaining the ReleaseManagement.md because we will need this.

Just want to drop the usual reminder of maintaining the ReleaseManagement.md because we will need this.

thanks for the reminder @Jens-G, for now this and the currently open pull requests only adds CI part, not the CD part, also the travis CI is intact - so I have no changes to the ReleaseManagement.md just yet. When I do later make the final switch, I'll post to dev@thrift and update accordingly. For now, there's no change needed.


error "gatsby-transformer-bibtex" threw an error while running the onCreateNode lifecycle:

Hi - looks like a really useful package and I'm eager to give it a try. I get the following error when trying to set it up with the simple gatsby hello world starter though. Error is below. Any advice? Thanks.
error "gatsby-transformer-bibtex" threw an error while running the onCreateNode lifecycle:
invalid syntax at index 52
Unexpected {"type":"id","string":"%"}
158 |             content = _context.sent;
159 |             // Parse

160 |             bib = (0, _bibtex.parseBibFile)(content);
|                                            ^
161 |             keys = Object.keys(bib.entries$);
162 |             onlyFieldsWithKey = _lodash["default"].map(keys, function (key) {
163 |               var obj = jsonOfEntry(bib.getEntry(key));

Error: invalid syntax at index 52
Unexpected {"type":"id","string":"%"}
package.json
{
"name": "gatsby-starter-hello-world",
"private": true,
"description": "A simplified bare-bones starter for Gatsby",
"version": "0.1.0",
"license": "0BSD",
"scripts": {
"build": "gatsby build",
"develop": "gatsby develop",
"format": "prettier --write "**/*.{js,jsx,ts,tsx,json,md}"",
"start": "npm run develop",
"serve": "gatsby serve",
"clean": "gatsby clean",
"test": "echo "Write tests! -> https://gatsby.dev/unit-testing" && exit 1"
},
"dependencies": {
"gatsby": "^2.25.3",
"gatsby-source-filesystem": "^2.4.2",
"gatsby-transformer-bibtex": "^0.1.3",
"react": "^16.12.0",
"react-dom": "^16.12.0"
},
"devDependencies": {
"prettier": "2.1.2"
},
"repository": {
"type": "git",
"url": "https://github.com/gatsbyjs/gatsby-starter-hello-world"
},
"bugs": {
"url": "https://github.com/gatsbyjs/gatsby/issues"
}
}
gatsby-config.js
/**

Configure your Gatsby site with this file.

See: https://www.gatsbyjs.com/docs/gatsby-config/
*/

module.exports = {
/* Your site config here */
plugins: [
{
resolve: gatsby-source-filesystem,
options: {
name: publications,
path: ${__dirname}/publications,
},
},
'gatsby-transformer-bibtex',
],
}

Just guessing: Could it be that the ID of your 52nd BibTeX entry starts with a %? Can you try removing that comment?

🤦♂ that was the problem.
However, I think this is still kind of a bug that should be fixed to handle inline comments. Or at least the example in the readme should be changed. I used the same example as in the readme with an 'inline' comment, which was the problem:
@article{some_key,
 author = {Name, Your and Of-Yours, Coauthor},
 title = {Paper A},
 file = {paper_a.pdf} % path relative to where this .bib file is
}

Considering the comment was in the readme it wasn't immediately clear to me that it couldn't handle in-line comments. Removing the comment does indeed fix the error though.

I think I remember that BibTeX itself (or at least some styles) also do not like inline comments, but haven't checked lately. Can anyone verify this, so that we can either update the documentation or handle the %?

Hello,
Very sorry for not answering any of this earlier, this year is chaotic on all levels.
-> I'm editing the readme right away to remove that line comment, and add something about not putting comments in the bibtex.
It isn't something I can fix unfortunately, because that comes from the bibtex parser, which is this one and doesn't look maintain anymore.
The main pain with gatsby-transformer-bibtex is that there is no good library to handle bibtex in JS. So far my best hope would be to compile XdkBibTeX to WASM and use that... I don't remember why right now, but citation-js was a nightmare.
Anyway, thanks for pointing that out, and sorry again for not reacting earlier !

Edited in 7c26ae86 and f8292966

thanks


Feature/tests

Addresses #33

I updated the DatabaseSeeder before seeing this - note to self branch now we have other contributors. Can you correct the conflict, push and I'll merge it :+1:


Limited fields in cards

In dilps some fields of the cards are limited to about 256 characters. When they are exceeded, there is no warning message. At first sight, the card is recorded normally but everything beyond that is actually cut off.
This is problematic for Dilps, but it will especially pose a huge concern to Tiresias who publishes very long texts in his description field. It is essential to be aware of this before importing Tiresias data into dilps, as this could result in the loss of a significant amount of information.

Fixed in 2fb8fd7e2df63ad89c67ddf890efdf786af3bbfa and bcaf05d98592f41ae7cbea85fe405987be7c1aa3


W3C protocol session creation

Codecov Report

Merging #393 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #393      +/-   ##
==========================================
- Coverage   74.64%   74.57%   -0.07%
==========================================
  Files           9        9
  Lines        1065     1070       +5
==========================================
+ Hits          795      798       +3
- Misses        221      222       +1
- Partials       49       50       +1

Flag
Coverage Δ

#unittests
74.57% <100%> (-0.07%)
:arrow_down:

Impacted Files
Coverage Δ

selenoid.go
77.07% <100%> (+0.32%)
:arrow_up:

session/session.go
94.44% <100%> (-5.56%)
: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 fff0de2...9a11aeb. Read the comment docs.


Add support for changing status bar appearance

Summary
This PR adds support for setting the status bar appearance of the modal.
Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.

[x] I've read and agree to the Code of Conduct.

[ ] I've written tests to cover the new code and functionality included in this PR.

Thanks so much for the PR's you've contributed today. I've merged one and will test and review the others by EOD.


Test and Run Project on Google Colab, Report Bugs

Description:
This issue is dedicated to testing and running our project on Google Colab, with the primary objective of ensuring that the project runs smoothly in this environment. Additionally, we aim to identify and report any bugs or issues that may arise during this process.
Goals:

Google Colab Testing: Execute our project on Google Colab to assess its compatibility and performance.
Bug Identification: Identify and report any bugs or issues that hinder the successful execution of the project on Google Colab.

Tasks:

[x] Prepare the project for execution on Google Colab.
[x] Set up the necessary dependencies and configurations on Google Colab.
[x] Execute the project on Google Colab, following documented instructions.
[ ] Thoroughly test project functionality and record any issues or bugs encountered.
[ ] Document the steps taken to run the project on Google Colab for future reference.
[ ] Report identified bugs in detail, including error messages and stack traces.

I would like to contribute on this issue, Can you assign me this.


Password-protected links

Create password protected links, where hitting up the short URL would show an authentication page... either just password or with full authentication supporting OAuth with external providers...

https://github.com/wahyu9kdl/link-lock here is simple example


Error on Android platform

Hello, relying on the Android-Crypto branch, there is a new problem，
My project is currently developing Flow Android Wallet, hoping to solve this problem

https://github.com/onflow/flow-java-client-example/issues/5
This is a link to my problem in other projects and I hope it can be solved

Hello, could you arrange someone to fix the BUG of this library on Android platform? This BUG has been bothering me for more than three weeks. Thank you very much if you can fix it


Formatted a little bit

Hello, I would love to contribute to community classroom as Kunal is my mentor so I can get a giveback satisfaction

Fixes Issue

Changes proposed
just formatted a little bit

Screenshots

Note to reviewers

I am grateful that you have merged it. Thank you


Fix for dashboard does not start after PR#21

Fixes #28
Updated romi-js-core-interfaces to "0.0.2-alpha.7" and added dispenser_states on soss config

@Yadunund can you help test and approve this if it works, I am on a feature branch of rmf_core and would rather avoid recompiling stuffs.

Hi @baviera08 ,
I am able to open the dashboard and visualize the doors/lifts on canvas now. Two issues to highlight:

The main_door which is a "DoubleSwing" type is not drawn.

The yaw of the lift cabin is not considered when drawing on canvas. It should be rotated by 90deg as defined in the BuildingMap

Hi @Yadunund , Thanks for the feedback.

Fixed DobleHingeDoor draw and the lift rotation.

@Yadunund I don't have permission to merge it, you have to do it.


add myself as a maintainer

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@sternenseemann thanks! Done. I'm not sure why regenerate-hackage-packages.sh changed something webkit-related...

@ulysses4ever that is related to an issue that currently exists on master try rebasing your change on the haskell-updates branch and regenerating the package set again.

Done.


Installation problems with Laravel 5.1.11

I created a brand new Laravel project (Laravel 5.1.11). It is up and running with no built-in user authentication pages, etc.
I followed the Wiki to install "pingpong/admin" and update the Laravel configurations.
NOTE  Your documentation is not updated to use the new "::class" attributes in the configurations.
When I attempt to seed the database it throws an error that the base table or view is not found.
[Illuminate\Database\QueryException]
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'mydb.roles' doesn't exist (SQL: insert into roles (name, slug, description   , created_at, updated_at) values (Admin, admin, The one who manage the site, 2015-11-12 19:38:13, 2015-11-12 19:38:13))

I manually created the "roles" table following the structure from the RolesAndUserSeeder.php file and the output of the error message.
When I run the seed again a new error comes up for "role_user" table or view.
[Illuminate\Database\QueryException]
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'mydb.role_user' doesn't exist (SQL: insert into role_user (created_at, role_   id, updated_at, user_id) values (2015-11-12 20:01:44, 0, 2015-11-12 20:01:44, 1))

Well, I manually created the role_user table too and now the seed works.  I finally figured out what page to navigate to but there is an error about an "articles" table.
ErrorException in Connection.php line 651:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'mydb.articles' doesn't exist (SQL: select count(*) as aggregate from articles where type = post) (View: /var/www/html/customer/vendor/pingpong/admin/src/views/index.blade.php)
Obviously something did not work correctly during the install as I am sure all of the database tables should have been created by the package.


PlatformDependent.throwException() with null argument takes down the JVM

I'm not sure if it's a bug or feature but it's certainly not easy to debug.
Netty 4.1.0-Beta6-SNAPSHOT
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f80fbe956c9, pid=1196, tid=140191066928896
#
# JRE version: Java(TM) SE Runtime Environment (8.0_40-b25) (build 1.8.0_40-b25)
# Java VM: Java HotSpot(TM) 64-Bit Server VM (25.40-b25 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# V  [libjvm.so+0x5756c9]  Exceptions::_throw(Thread*, char const*, int, Handle, char const*)+0x1d9
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /<path>/hs_err_pid1196.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.sun.com/bugreport/
#

@rkapsi - Is PlatformDependent.throwException(null) called from Netty code, and if so do you know where from?

@Scottmitch no, not from Netty code. It was some code I wrote and one can argue it was my own fault (given the nature of that class) but I wish it'd have failed more gracefully.

@rkapsi - Humm I guess at least in the case of unsafe we should ensure the language specification is honored:
http://docs.oracle.com/javase/specs/jls/se8/html/jls-14.html#jls-14.18

If evaluation of the Expression completes normally, producing a null value, then an instance V' of class NullPointerException is created and thrown instead of null.

I have a PR pending for this.

+1. Thanks for taking care.


Reorder tasks via buttons

I moved the "Remove" button into a menu button and I also added "Move Up" and "Move Down" options:
Screencast from 2023-12-02 13-02-08.webm

It's probably better to put "Remove" in a new menu section.
Since "Remove" is a destructive action, it's better to separate it a bit more from the other options.
Maybe put it in a new menu section.
Or maybe add an option to undo the remotion.


ws:// urls don't work for web-repl bridge in chrome as the viper-ide site always redirects to https and therefore requires wss: links not ws:

Just discovered the Bridge mode / page at https://viper-ide.org/bridge.html and thought I'd attempt to connect to the serial port / repl of a circuitpython device, as it runs the web workflow and therefore has a websocket open at ws://<IP_ADDRESS>/cp/serial/
It fails due to "Connection to an unsecure WebSocket is blocked on a secure website".
I did try using http: instead of https: for viper-ide but the page forcefully redirects to the secure version.
I guess it could be made to work, but probably offers little benefit at the moment.
Thanks for the great tool!

Yes, this is a conceptual limitation, and it's documented.
With some efforts, you can run ViperIDE locally in an insecure context, and it should work with such devices

Thanks for your reply!
Yeah that makes sense, and nice one documenting it (can't deny I haven't
read all the docs yet).
I think I may revisit this once I want to run it from/on device, or have a
more pressing need for Circuitpython usage over the network
On Thu, 12 Sept 2024, 21:39 Volodymyr Shymanskyy, @.***>
wrote:

Yes, this is a conceptual limitation, and it's documented.
With some efforts, you can run ViperIDE locally in an insecure context,
and it should work with such devices
—
Reply to this email directly, view it on GitHub
https://github.com/vshymanskyy/ViperIDE/issues/32#issuecomment-2347198835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTBZ44T4EOPZUMYQAQJOALZWH3YPAVCNFSM6AAAAABOC5AQXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBXGE4TQOBTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

There's not much sense in running it on the device, it's ~10MB. Unless you want to sacrifice almost all of the goodness...
Well, it could be shrinked to ~100Kb (as it used to be in the early days).

Yikes, not small then! Only fit on a couple of my larger esp devices unless
I go for the lofi version.
On Thu, 12 Sept 2024 at 22:11, Volodymyr Shymanskyy <
@.***> wrote:

There's not much sense in running it on the device, it's ~10MB. Unless you
want to sacrifice almost all of the goodness...
Well, it could be shrinked to ~100Kb (as it used to be in the early days).
—
Reply to this email directly, view it on GitHub
https://github.com/vshymanskyy/ViperIDE/issues/32#issuecomment-2347250392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTBZ4ZGGGJU5D63CHZMZHTZWH7RBAVCNFSM6AAAAABOC5AQXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBXGI2TAMZZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Test review

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: xooiamd:x: mizhenYou have signed the CLA already but the status is still pending? Let us recheck it.


Query: Throw better exception message when composing after client projection

@smitpatel This change seems more appropriate for 3.1

@ajcvickers - retargeted.

Fixes #17504


How to change background Color? i need to set background color of pdf pages, I try from html css but when create the pages it only applied to length of conent

I need to set the background color of pdf pages, I try from HTML, CSS but when creating the pages it only applied to the length of content.

Same issue here i don't if you've find an anwer to it

@Azaphyr  sorry I did not find any solution

@A110SHT, @Azaphyr
Got mine to work by setting the color in the html body.

Then after your last content add a div that page breaks to a new page. This creates a new empty page so the height: -10px removes the new page.


Cache miss

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
3.0.9
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
3.6.0
Public or Confidential Client?
Public
Description
Method getTenantFromAuthorityString (used to read cache) returns different value than Authority.tenant (used to store cache) results in missing cache. Those methods are used when token doesn't have tif or tfp or acr claims.
Error Message
No response
MSAL Logs
No response
MSAL Configuration
{ not relevant }

Relevant Code Snippets
not relevant

Reproduction Steps

Token without tif, tfp, acr claims

Expected Behavior
Cache is hit when using fallback.
Identity Provider
Azure B2C Custom Policy
Browsers Affected (Select all that apply)
Chrome, Edge
Regression
No response
Source
External (Customer)

To fix this issue, you can either downgrade your MSAL.js library to version 3.0.9 or update your code to handle the new Authority.tenant property. Here's an example of how you can update your code to handle the new property:
import { PublicClientApplication, Configuration } from '@azure/msal-browser';
const msalConfig: Configuration = {
auth: {
clientId: 'your-client-id',
authority: 'https://your-authority-url',
knownAuthorities: ['your-authority-url'],
redirectUri: 'your-redirect-uri',
},
cache: {
cacheLocation: 'localStorage',
storeAuthStateInCookie: false,
},
};
const msalInstance = new PublicClientApplication(msalConfig);
// Check if the authority URL has a tenant ID
if (msalConfig.auth.authority.includes('your-tenant-id')) {
// Use the tenant ID from the authority URL
const tenantId = msalConfig.auth.authority.split('/')[3];
} else {
// Use the tenant ID from the Authority.tenant property
const tenantId = msalConfig.auth.authority.tenant;
}
By checking if the authority URL contains a tenant ID, you can use the appropriate tenant ID for your cache. This will ensure that your cache is hit when using the fallback method.

I am not sure what should we do with that tenantid. Cache is handled by library itself.

I tested overwriting Authority.tenant the [0] with .at(-1):
    public get tenant(): string {
        return this.canonicalAuthorityUrlComponents.PathSegments.at(-1)!;
    }

Works for me, however, I am not sure if there could be consequences.

Any updates on this?

Thanks for raising this, looks like a potential bug on our end. Marked to track internally. cc @hectormmg

Hi, same issue is also present in the react library.
Just want to chip in, after a full day of investigating why our access token was not cached I came to same conclusion. It seems that indeed the get tenant() method does not take into account non AAD scenarios.
Another less invasive way of dealing with this is to overwrite the account property when doing a acquireTokenSilent, for example:
import { useMsal } from '@azure/msal-react';

const { accounts, instance: msalInstance } = useMsal();

const { accessToken, account } = await msalInstance.acquireTokenSilent({
  scopes,
  prompt: 'login',
  account: { ...accounts[0], tenantId: 'yourTenantId' },
});

Could this be prioritised? For non AAD customers, by default msal frontend library does not work as expected and results in endless token calls.


Fail to provision and edge device under Azure .cn (China) subscription

Describe the bug
I am trying to provision a device into IoTHub via DPS. I am using a SymetricKey provision type and a scope Id from a azure.cn subscription.
The provisioning command returns success but NO IoTEdge device entity is created by DPS.
Provisioning command used: Provision-EflowVm -provisioningType DpsSymmetricKey -symmKey "" -scopeId  -registrationId
Accessing to the "/etc/iotedge/config.ymal" file I can see the property provisioning > global_endpoint is set to "https://global.azure-devices-provisioning.net".
Since we are using a .cn scope Id I think the global_endpoint should be set to "https://global.azure-devices-provisioning.cn".
Changing the global_endpoint in the config file manually and restarting the EFLOW VM. The Edge device entity is correctly created under the IoTHub portal (cn subscription).
Why is the service assuming the wrong global_endpoint if I am passing a .cn scope Id as a parameter?
Please let me know I can provide additional troubleshooting data.
Expected behavior
I expect that the IoTEntity war was created on IoTHub .cn portal without manually changing the global_endpoint property on the VM file.
Windows Host OS (please complete the following information):

Edition: [Pro]
EFLOW:  v1.1.2111.17111
Version: [19044.1348]
Physical Device located in Mainland China

Hello.
I originally open this issue under the EFLOW repo issues page. Then it has been transferred to the iotedge repo.
I have also used the official MS support channels to report this. I had a quite disappointing answer about this being a feature request (which obviously is not).
We are aware of the workaround but it is not suitable for large-scale automated deployments.
Please let me know how can we help from our side to help. We will be happy to bring our MS Account Manager to the loof if it helps somehow.
Thanks,
Nuno

I discussed it internally, EFLOW  Team will work on it.
I have not visibility in their schedule but it could be part of January release.

Hi @ngmartins,
This was fixed with our latest release.
Thanks,
Francisco

Hey,
Just confirmed it works as expected.
Thanks for implementing this :)
Just a quick note in case you are thinking to add this to the documentation. We should pass the endpoint with https:// prefix.


Design solution so that users don't need to create and push to their own DockerHub repo

Currently users need to create a public repository on their personal DockerHub accounts and then specify that repo to the publish command. They also need to be logged in to DockerHub when running publish. This is one of the bigger known kludges in the new publishing workflow.
It would be good
Assumptions:

This is a research spike ticket. The outcome should be a mini design doc and tickets for implementation.

Acceptance Criteria (for the end product)
Given I am a Garden publisher, when I publish a pipeline, then I don't need to think at all about my personal DockerHub account.

We've landed on pushing images to a Garden-hosted ECR repo. Sub-steps will be ...

infra: make dev and prod repos
backend: make a new endpoint that doles out temporary STS tokens to end users
SDK: after building a container, fetch an STS token and push the container to ECR. (Also clean up any DockerHub stuff)


Cannot target a context with capital letters

If a UAA instance were deployed into a context root containing capital letters, the uaac command lower-cases the target URL and therefore cannot use this deployment.
This can be validated by adding the following to spec/common_spec.rb.
it "maintains the capitalization of the target" do
Cli.run("target example.com/UAA --force")
Config.yaml.should include "https://example.com/UAA"
Config.yaml.should_not include "https://example.com/uaa"
end
On a real deployment, the 'uaac target' command asserts to succeed, and 'uaac contexts' shows me two contexts, but any other command (uaac users, uaac clients) complains that the target and context have not been set.

@lefty27513 We have stopped active development on cf-uaac, and are planning to build a replacement for it. I will close this issue now.


Setup a minimun zoom in config file

It would be nice if it will be possible to limit the minimum zoom at wich martin return tiles.
I know that Im able to do it from javascript in leaflet or mapbox but anyone is able to bypass this control and could perform like dddos attak against postgres
Thanks!

This would be very good to have. Though a workaround for now could be to prevent such requests from your reverse proxy? NGINX for example can use regex patterns.


feat(Responsiveness): changed the sizing of H1 tag for screen size up to 500px

Fixes Issue
My PR closes #issue_number_here
Closes #555
👨💻 Changes proposed(What did you do ?)
Changed the styling of the H1 text for a smaller screen size.
✔ Check List (Check all the applicable boxes)

[] My code follows the code style of this project.
[] This PR does not contain plagiarized content.
[] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Conventional Commit PR Title
In order to be considered for merging, the pull request title must match the specification in conventional commits. You can edit the title in order for this check to pass. Most often, our PR titles are something like one of these:

docs: correct typo in README
feat: implement dark mode"
fix: correct remove button behavior

Linting Errors

Found type "null", must be one of "feat","fix","docs","style","refactor","perf","test","build","ci","chore","revert"
No subject found

Please fix this error, if you don't know how, try this:

Read the error carefully or read the contributing file
Checkout past merged Pull requests to see how they were done for it to be merged

Note: you can edit your PR message, this PR won't be merged if the error isn't gone and you have a week max to fix it before the PR gets closed without being merged

now is it good? @Dun-sin

:tada: This PR is included in version 2.77.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


CNV-40059: Align MigrationPolicies, Checkups empty pages with new format

📝 Description
This PR is one of the PRs related to:
https://issues.redhat.com/browse/CNV-40059
Design doc:
https://docs.google.com/document/d/1q1J7QJPbklWNV02hqBIIi6uNwHKaipUFZLA3RJF5A9E
Align MigrationPolicies and Checkups (both Network latency and Storage) lists' empty states with the new format for more consistent look across the UI, edit appropriate text, remove unnecessary filtering component according to the new look (see the design doc). Also add missing links to related docs.
🎥 Screenshots
Before:
MigrationPolicies list page:

Network latency checkups:

Storage checkups:

After:
MigrationPolicies list page:

Network latency checkups:

Storage checkups:

@avivtur @pcbailey @upalatucci @metalice please review


/interface errors when called from rcon

[ERROR] command/interface :: ...ers/factorio-eu-01/temp/currently-playing/utils/game.lua:22: bad argument #-1 to '__index' (string expected, got nil)
Above error is caused when an rcon attempts to call /interface and appears to be because there is no player using the command and thus the error with game,get_player_by_index.

Fixed in: 5.1.1


MediаInfo throwing a fatal

2018-10-17T05:19:55.755Z 547088fc-235d-497e-9a1c-1cebb8f4ce9c TypeError: result.Mediainfo.File.track.forEach is not a function
at $xmlParserInstance.parseString (/var/task/lib/mediaInfoCommand.js:82:35)
at Parser.<anonymous> (/var/task/node_modules/xml2js/lib/parser.js:303:18)
at emitOne (events.js:96:13)
at Parser.emit (events.js:188:7)
at Object.onclosetag (/var/task/node_modules/xml2js/lib/parser.js:261:26)
at emit (/var/task/node_modules/xml2js/node_modules/sax/lib/sax.js:624:35)
at emitNode (/var/task/node_modules/xml2js/node_modules/sax/lib/sax.js:629:5)
at closeTag (/var/task/node_modules/xml2js/node_modules/sax/lib/sax.js:889:7)
at Object.write (/var/task/node_modules/xml2js/node_modules/sax/lib/sax.js:1436:13)
at Parser.exports.Parser.Parser.parseString (/var/task/node_modules/xml2js/lib/parser.js:322:31)

I'm still trying to understand what's going on here, but I think it's from a corrupted source video. When pulling the source video, it's not playable, and various desktop video players aren't able to inspect the video.
This error would make sense, but it's a misleading error, there should probably be a validation check that this collection exists before trying to iterate with forEach

@tomnight at first i thought it was something more complicated, but when i went back to investigate, it was just a bogus upload. I haven't had any issues with mediaInfo under normal circumstances, but i guess catching/validating would be helpful. That stack trace sound a lot more menacing than it is.


How I can close the event when use customViewer

I'm using customViewer, but I don't know how to close the event with onClick in IconButton. I try use Popover with a state open but I only get a white box instead of the content I need so I had to go back to using div like below.
        customViewer={(event: ProcessedEvent) => {
          return customAppointment(event);
        }}

  const customAppointment = (event: any) => {
    return (

        <Box
          sx={{
            bgcolor: event.color,
            color: "white",
          }}
        >

              {new Date(event?.date).toLocaleDateString("vi-VN", {
                weekday: "long",
                year: "numeric",
                month: "long",
                day: "numeric",
              })}

              <IconButton size="small" onClick={() => {}}>
                <EditRounded style={{ color: "white" }} />
              </IconButton>
              <IconButton size="small" onClick={() => {}}>
                <DeleteRounded style={{ color: "white" }} />
              </IconButton>

              <IconButton
                size="small"
                onClick={(el?: MouseEvent<Element>) => {
                  // How i can close the event?
                }}
              >
                <ClearRoundedIcon style={{ color: "white" }} />
              </IconButton>

        </Box>

    );
  };

Check #292
Feel free to re-open if you still have issue


About subnet proxy configuration

The virtual IP of machine A is <IP_ADDRESS>. I want to proxy the subnet of machine A. The subnet segment is <IP_ADDRESS>. How to configure the GUI software on machine A? (see picture)

参考配置


[Concurrency] Downgrade isolated_default_argument_context to a warning until Swift 6 for stored properties.

This code was accepted in compiler versions <=5.9:
@MainActor func requiresMainActor() -> Int {}

struct S {
  var x = requiresMainActor() // warning here
}

This code was made invalid by SE-0411. This change downgrades the error to a warning unless the var in question is a ParamDecl.
Resolves rdar://121060811

@swift-ci please test Windows


Refactor on logs and questions.

Move chalk, ora and inquirer to lib folder.
Create methods for abstractions.
Move logs to init method inside each module.
Re-design the flow: asking -> working -> finish; Soon we will be able to ask inside work, so we need to change the flow.

As I worked on this improvement I realized that it is not possible to change the flow to ask within the execution of each module. This happens due to the fact that the project is linked to the name that was defined for the folder where all modules will be installed and configured.
Despite this, it was possible to create the three modules: infinity-progress, print and prompt.
In the future it will be necessary to create the module for the service to run in the terminal.

After e2e tests, pull request will be created.


Not enough disk space error translation - Es

Spanish language translation for PR #9647
Added text: "Together with the commands used in Step 11, this adds additional parameters to the
mongod command to allow MongoDB to run in the C9 environment." & "Note: Use the command mongod --port 27017 in place of mongod --port 27017 --dbpath=./data to avoid a duplicate parameter error.--dbpath=./data is already included in the ./mongod command."

Added text: "Together with the commands used in Step 11, this adds additional parameters to the
mongod command to allow MongoDB to run in the C9 environment." & "Note: Use the command mongod --port 27017 in place of mongod --port 27017 --dbpath=./data to avoid a duplicate parameter error.--dbpath=./data is already included in the ./mongod command."
Translated text: "Conjuntamente con los comandos utilizados en el paso 11, esto agrega parámetros adicionales al comando mongod para habilitar a MongoDB correr en el entorno C9."
"Nota: Utiliza el comando mongod --port 27017 en lugar de mongod --port 27017 --dbpath=./data para evitar el error de parámetro duplicado. --dbpath=./data ya se encuentra incluido en el comando ./mongod"
In case you want to keep the error the same replace from "--dbpath=./data ya  se encuentra incluido en el comando ./mongod" to ".--dbpath=./data is already included in the ./mongod command."

Are the above translations good to go? If so I can add them to the project.
Unless you guys want a native Spanish speaker to work on this issue?

@jhuynh85 Hi there.
I'm a native Spanish speaker and fluent in English. My translation is accurate 😀

@josevill I wasn't calling into question your translations :smiley: I was just wondering if the project admins preferred that a Spanish speaker merge them to the project since my Spanish is minimal

Hey guys, I delved into this a little more and I noticed a couple things about Step 13:

The new text from PR #9647 makes a reference to Step 11, which in the English text corresponds to

"Set up your mongod alias by following the directions at https://community.c9.io/t/setting-up-mongodb/1717."
However, this step has been left out in the Spanish text (presumably because the directions in the link are in English), so adding the Spanish translation that we have would make no sense.

I also noticed the Spanish translation does not match the English. The English says to use ./mongod while the Spanish says to use mongod --nojournal:

ENGLISH: Whenever you run a command that includes mongod on c9.io, use ./mongod instead. Together with the commands used in Step 11, this adds additional parameters to the mongod command to allow MongoDB to run in the C9 environment.
SPANISH: Siempre que ejecutes un comando que incluya mongod en c9.io, asegúrate de usar la bandera --nojournal. Por ejemplo: mongod --nojournal.

Thanks to everyone for the attention to detail. We are switching away from Heroku/C9 and over to GoMix so these instructions will soon change.


Update amqp version to prepare for release

Need to release core-amqp again to address a customer reported bug.

/check-enforcer evaluate


delete the unnecessary develop branch

Everything should be added in its own appropriately named branch and merged/ rebased into the master/bleeding as needed.

Fixed. Issue #27 was created from the feature changed in develop.


Reorganize mempool modules

Description
Things have grown organically in the Mempool modules. This puts some order. Downstream clients should use Ouroboros.Consensus.Mempool.
There are deprecation warnings in place for downstream consumers.
The new structure of modules is the following (leaving aside the deprecated modules):

Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[x] The documentation has been properly updated
[x] New tests are added if needed and existing tests are updated
[x] Any changes affecting Consensus packages must have an entry in the appropriate changelog.d directory created using scriv. If in doubt, see the Consensus release process.
[x] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[x] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

Current dependencies on/for this PR:

master

PR #4332   👈

PR #4316

PR #4341

This comment was auto-generated by Graphite.

bors r+


Inputting an empty string returns a score of 4

this is a regression following v4 -- thanks for reporting!

No problem, thanks for fixing it! :)


Fail read resource from provider, trying import command panic: interface conversion: interface {} is nil, not string

It is strange but from 30-04-2021 we started to get this error using Terraformer for IAM AWS resources:
_terraformer import aws --resources=iam --profile=sandbox --connect=true
2021/05/03 10:46:59 aws importing default region
2021/05/03 10:47:02 aws importing... iam
2021/05/03 10:47:11 aws done importing iam
2021/05/03 10:47:11 Number of resources for service iam: 33
2021/05/03 10:47:11 Refreshing state... aws_iam_role_policy.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_role.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group.tfer--developers-002D-XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_user_group_membership.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 Refreshing state... aws_iam_group_policy_attachment.tfer--XXXX
2021/05/03 10:47:11 WARN: Fail read resource from provider, wait 300ms before retry
2021/05/03 10:47:11 WARN: Fail read resource from provider, wait 300ms before retry
...
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Refreshing state... aws_iam_role_policy_attachment.tfer--XXXX
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Refreshing state... aws_iam_user_policy_attachment.tfer--XXXX
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Refreshing state... aws_iam_user_group_membership.tfer--XXXX
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Fail read resource from provider, trying import command
2021/05/03 10:47:20 Fail read resource from provider, trying import command
...
2021/05/03 10:47:24 WARN: Fail read resource from provider, wait 300ms before retry
2021/05/03 10:47:24 Fail read resource from provider, trying import command
panic: interface conversion: interface {} is nil, not string
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*IamGenerator).PostConvertHook(0xc0000acb40, 0x52862db, 0xc0000acb40)
/home/runner/work/terraformer/terraformer/providers/aws/iam.go:376 +0x6a5
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).PostConvertHook(0xc0001f0bd0, 0x67857c8, 0xc0001f0bd0)
/home/runner/work/terraformer/terraformer/providers/aws/aws_facade.go:75 +0x3b
github.com/GoogleCloudPlatform/terraformer/terraformutils.(*ProvidersMapping).CleanupProviders(0xc000921530)
/home/runner/work/terraformer/terraformer/terraformutils/providers_mapping.go:175 +0x10a
github.com/GoogleCloudPlatform/terraformer/cmd.Import(0x6782d68, 0xc00062c000, 0xc00044c770, 0x1, 0x1, 0x806eb58, 0x0, 0x0, 0x5dc7005, 0x1e, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:104 +0x370
github.com/GoogleCloudPlatform/terraformer/cmd.importRegionResources(0xc00044c730, 0x1, 0x1, 0x806eb58, 0x0, 0x0, 0x5dc7005, 0x1e, 0x5d6acbe, 0x9, ...)
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:112 +0x238
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAwsImporter.func1(0xc000129080, 0xc0007c6300, 0x0, 0x3, 0x0, 0x0)
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:59 +0x4fb
github.com/spf13/cobra.(*Command).execute(0xc000129080, 0xc0007c6270, 0x3, 0x3, 0xc000129080, 0xc0007c6270)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850 +0x472
github.com/spf13/cobra.(*Command).ExecuteC(0xc000128000, 0xc00044d1a0, 0xc0000ae050, 0x5e4ab00)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute(0x661ac20, 0xc00044d1a0)
/home/runner/work/terraformer/terraformer/cmd/root.go:36 +0x2e
main.main()
/home/runner/work/terraformer/terraformer/main.go:39 +0x6a_
Terraformer version v0.8.13

Did you ever get this fixed?


How to write with xml tag at the first line of file?

I have written xml file from dataframe using write. But the xml file does not contain any xml tag like:
<?xml version="1.0"?>
How do I add this tag when writing dataframe into xml?

Does anyone know how to do this properly?

It's not supported yet. Probably we would need this option from a very quick look.

Dueplicate of #74


Problem with export in develop branch

This seems to be over and above the issue with tsugi includes breaking the current develop but offline export creates a zip which looks to be not complete on my machine - i.e. the offline/models are created in a folder but the offline includes aren't being written to the index file and the index file being written has the filename index.php instead of .htm/.html
Anyone else seeing this?

@JohnSmith-LT after your post I've just been testing and in my local copy of develop none of the export options work from projects based on the xot template. Decision and Site templates export but when I open the properties of an xot based project and try export none of the export buttons trigger an export. I don't see any browser console errors though. Not sure why you'rs is different to mine where you are getting a zip?

Oh sorry, i've reverted my develop branch export.php file to the one before all the xApi / Tsugi code was put in but it's only doing partial export, perhaps because it now interacts with that code in some way that i'm not seeing yet...

That particular issue should be fixed now.

I need to cleanup the export code (sorry I missed that) The TSUGI code is still in there, and in the end I don't want it in there. It should NOT be necessary to export a TSUGI package at all, because that is not the purpose of LTI2.

@torinfo @JohnSmith-LT Just testing export again after pulling down Tom's commits and the exports are still not working for me in the develop branch. Should this be fixed with those commits?
Does it work for you now John?

No I just tested and it doesn't work for me either @ronm123 but i've just pushed a fix

@JohnSmith-LT that's cracked it! :-)


Tagging the release to make it available

The release 1.3 is not accessible via Xcode SPM integration. Probably because the release is not tagged.

fixed in 1.3.1 please try it and let me know if it works.

Hi Rastislav.
Version 1.3.1 is not visible. Did you release it?

Am 09.10.2019 um 18:56 schrieb Rastislav Mirek<EMAIL_ADDRESS>fixed in 1.3.1 please try it and let me know if it works.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/RastislavMirek/FlexColorPicker/issues/17?email_source=notifications&email_token=AAOPYG3DVVMSSZBKAUYKRRDQNYEM7A5CNFSM4I2RKMLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAYSE7Y#issuecomment-540091007, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOPYG5UKFRTLPGYTJ4J3PLQNYEM7ANCNFSM4I2RKMLA.

Yes, sure, just a few minutes ago. Can you try again?

Wait, tag was not pushed, let me fix that

Still can’t see the tagged release 1.3.1 on github.com

Now you should see it, let me know whether it compiles


Added currently playing check to end game voting

The endvote code currently allows anyone in the gather chat to put forward an end vote, leading to the possible situation of non-playing users forcing a game to end without the participants consent. With this added playerIsPlaying check only players currently in game can vote to end the game.

It's Nand, btw.


Revert "fix(urllib3.py): urrlib3 event collection failed"

Reverts epsagon/epsagon-python#352

:tada: This PR is included in version 1.67.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


TCH003 false positive on __all__ combined with from __future__ import annotations

TCH003 seems to incorrectly trigger when re-exporting a module in __all__ if using from __future__ import annotations.
Minimal reproducible code:
# foo.py
from __future__ import annotations

from logging import getLogger

__all__ = ("getLogger",)

def foo() -> None:
    pass

$ pipx run ruff==0.0.281 foo.py
foo.py:3:21: TCH003 [*] Move standard library import `logging.getLogger` into a type-checking block
Found 1 error.
[*] 1 potentially fixable with the --fix option.

In the example above, we import getLogger from logging to re-export it in __all__, and TCH003 will get triggered, although it should not.
Note that this does not trigger when removing from __future__ import annotations. It also doesn't trigger in version 0.0.280, so this seems to be related to a change in 0.0.281.
What's even weirder is that the violation will also not trigger in the example if we remove the return type (-> None) from the function.
Command used:
pipx run ruff==0.0.281 foo.py

Minimal Ruff settings to reproduce:
[tool.ruff]
select = [
    # flake8-type-checking
    "TCH",
]

Ruff version:
ruff 0.0.281

Very weird, taking a look.

This is a subtle bug related to the order in which we visit things internally. I'll probably push a hotfix for this today.

This should be fixed in v0.0.282, which is out now.


Errors and warnings in "analyze_tests"

There is quite a number of failing tests, most probably because they become stale after library changes.
Now, with work on https://codereview.chromium.org/18174010/ they become exposed.

Attachment:
analyze_tests.txt (83.76 KB)

The linked issues are closed - I'll assume for now this is no longer a problem. Please reopen if there are still failing tests that make sense to track in this issue.


Add debug assertions to discourage violating dealloc/resize safety

I'm the author of another allocator for WASM, talc. I'm assuming this is still the default allocator in WASM?
In issues https://github.com/SFBdragon/talc/issues/17 and https://github.com/SFBdragon/talc/issues/24 I've had devs assume my allocator had a bug. Both times it turned out that the issue was with incorrectly sized layouts being passed to dealloc. Talc relies on the invariant stipulated by the safety contract of dealloc, but dlmalloc ignored this value entirely. The lol_alloc crate also relies on this, but will usually just "leak" memory instead.
I think this is a problem: it allows devs to write concealed bugs that only manifest if they override the global allocator (or a crate that depends on the buggy crate does so). This is completely avoidable, by having a debug assertion that the layout.size() equals the size stored internally by dlmalloc. Same goes for resize, I believe.
Side effects may include breaking some crates that violate the safety contract of dealloc/resize but have only been used with dlmalloc. I think this is necessary though. This implementation is setting an expectation as the default global allocator on WASM, and this expectation is a safety violation.
Happy to submit a PR, although I can't do it right now.

I think that this would be great to have! A PR would be most welcome here too!
This crate is indeed currently the default allocator for wasm and the main wrinkle in this is that the standard library is distributed in binary form with debug assertions disabled. This means that by default users would not get this assertion as a debug assert.
I'm not personally opposed to making it a runtime assertion, however, a single branch probably isn't gonna break the bank anywhere. While technically a "breaking" change in the sense that it could break things Rust has experience doing changes like this. I think it would be reasonable to ensure that there's a message in the release notes for this and otherwise users can recover the prior behavior with their own global allocator customization.

While technically a "breaking" change in the sense that it could break things

You can argue this is just a patch, anything that would trip the assert was already undefined behaviour.

I think that this would be great to have! A PR would be most welcome here too!

Great, I'll submit one soon.
Release-mode assertions sound good to me. Assertions are maximally runtime-predictable branches so it'll be negligible.
I agree with @alexcrichton and @khoover. I'd also posit that it's better to explicitly break code with UB instead of just leave it be. Better to cause a few headaches now, rather than even more later down the line? Idealistic, for sure, but let's give it a shot.

Here's a first stab at it. Unfortunately I'm only able to assert the size is within a certain range given certain conditions. It's not nearly as cheap as I feel comfortable with either.
I'm not 100% clear on whether I should be multiplying the chunk overhead by two, given malloc, to the best of my knowledge, puts a usize at the top and bottom of each allocation (but the overhead function by default only returns size_of::<usize>()), but the chunk's size might not be storing the full header-to-header size either.
In summary, the chunk might have to include extra space for:

alignment
metadata
space on either side of the allocation that's too small to be freed as its own chunk (if I'm not mistaken)

And so the upper limit check is a sum of the worst of the combined overhead.
I've ran both through Talc's random actions benchmark preliminarily but the results are a little too noisy to see a clear difference in between the current and patched version. I suppose that's a good thing, but I want to run both through the microbenchmarks as well, as that may make the runtime cost clearer.
Here's the diff:
diff --git a/src/dlmalloc.rs b/src/dlmalloc.rs
index a4b4b71..a889ef6 100644
--- a/src/dlmalloc.rs
+++ b/src/dlmalloc.rs
@@ -1163,6 +1163,18 @@ impl<A: Allocator> Dlmalloc<A> {
         }
     }

+    pub unsafe fn validate_size(&mut self, mem: *mut u8, size: usize, alignment: usize) {
+        let p = Chunk::from_mem(mem);
+        let psize = Chunk::size(p);
+
+        let min_overhead = self.overhead_for(p);
+        assert!(psize >= size + min_overhead);
+
+        if !Chunk::mmapped(p) {
+            assert!(psize <= size + self.min_chunk_size() * 2 + (mem::align_of::<usize>()).max(alignment) - 1 + min_overhead);
+        }
+    }
+
     pub unsafe fn free(&mut self, mem: *mut u8) {
         self.check_malloc_state();

diff --git a/src/lib.rs b/src/lib.rs
index b972ccb..2ec68b8 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -143,7 +143,7 @@ impl<A: Allocator> Dlmalloc<A> {
     /// method contracts.
     #[inline]
     pub unsafe fn free(&mut self, ptr: *mut u8, size: usize, align: usize) {
-        drop((size, align));
+        self.0.validate_size(ptr, size, align);
         self.0.free(ptr)
     }

@@ -164,6 +164,7 @@ impl<A: Allocator> Dlmalloc<A> {
         old_align: usize,
         new_size: usize,
     ) -> *mut u8 {
+        self.0.validate_size(ptr, old_size, old_align);
         if old_align <= self.0.malloc_alignment() {
             self.0.realloc(ptr, new_size)
         } else {

Regardless, this'll need to be tested on a few different platforms to guard against false positives. But even that might be jumping the gun: is this degree of overhead acceptable in the first place?

It's been a long time since I originally ported dlmalloc from C to Rust and I've long since lost the knowledge from that time, so I'm unfortunately not sure whether that check is right or not. As for benchmarks and/or the overhead, I don't have any in this repository but I do think it'd be good to measure just to be sure.

Fair enough.
Turns out psize can be equal to size + min_overhead, just 8 bytes. Reread the documentation in dlmalloc.c, it checks out, so we're good there. There's a benign error in the original though. I don't need to add the full alignment, as the wasted space below the allocation will become its own chunk (unless its too small, but that's covered by adding on the min_chunk_size for the bottom (*2 for the top)). This makes the check a bit cheaper too.
Finally, I've run the old and new versions side-by-side through talc's microbenchmarks and while there's clearly a small difference, which stays consistent with re-runs of the benchmark, so its probably not just chance/noise. It's a tiny difference though.

I'll open a PR with the changes in the meantime.


add test fer tryin ta get too many messages

Coverage increased (+1.6%) to 99.225% when pulling 824881baa8191f4d1c2219195cb9ba8eca9a7647 on add-test-messages-get-too-many into c6daa18c9aa8cf058c6a49b39b61e8d2d7818bf7 on master.


KeyError: 'total_interaction_count' with SNES

When doing a search as searcher = SNES(prob, stdev_init=0.1, num_interactions=100) and then run it, I get the error.

Hi @maulberto3!
Thank you very much for your feedback!
The keyword argument num_interactions was introduced to be used with reinforcement learning problems which provide an item with the key total_interaction_count in their status dictionaries. Could it be that your problem object (prob in your example code) is not a reinforcement learning problem, and therefore its status does not have the key total_interaction_count?
I looked at the docstrings of SNES and realized that the documentation for the keyword argument num_interactions says that it works with a problem class named GymProblem. I have to fix the documentation because the correct class name is GymNE (full name of the class: evotorch.neuroevolution.GymNE). Perhaps this mismatch in the documentation caused this confusion? Sorry about that. I will fix the docstrings.
Is it indeed the case that your problem object is not a reinforcement learning task?

Assuming that the issue is as described by @engintoklu , I'm closing this issue for now. @maulberto3 please reach out and re-open the issue if you believe there is a bug in the usage of num_interactions

This error occurs if I use the VecGymNE class because it's missing the hook and extra stats that GymNE class has:
# gymne.py
def __init__(...):
    ...
    self.after_eval_hook.append(self._extra_status)

# gymne.py
def _extra_status(self, batch: SolutionBatch):
    return dict(total_interaction_count=self.interaction_count, total_episode_count=self.episode_count)

Workaround is to manually add this hook after initializing VecGymNE object:
problem = VecGymNE(...)
problem.after_eval_hook.append(lambda batch: dict(total_interaction_count=problem.interaction_count, total_episode_count=problem.episode_count))