Calendar representation for value in CalendarDate appears very wrong

On your example page (and even in my local exploration) the CalendarMonth widget is not being correctly represented in a couple of ways:

The highlighted day specified by the value property is a month behind. In you example, value is specified as "2024-01-10" (January 10th 2024), but the highlighted day is December 10th, 2023. I see the same inconsistency locally.
Perhaps worse, is that the days do not begin on the proper day for the displayed month and year. On your example page, the month of December in 2023 is starting on Sunday, when it should be Friday. I'll note that the month of 12/24 does start on a Sunday.

Thanks. Can you give some further information to help me out? What browser, version, and OS? What is your system locale?
Seems like there's an off by one error from your description, which is a classic error for JS Dates since months are zero based.
However, everything looks fine to me. I've tested on Firefox and chrome on my phone.

Perhaps I've misunderstood.

I see this in both Firefox and Chrome:

I'll check with others on my team to see if they see the same issue.

fwiw, I'm seeing this issue, as well, on your demo page at https://wicky.nillia.ms/cally/guides/theming/ (using a Chromium-flavored browser on MacOS). Here is a screenshot

Thanks, good to have a second confirmation. I'm not able to reproduce in either case. It looks like the issue is the headings are incorrect rather than the month itself.
What do you get if you log in the console:
console.log(navigator.languages[0])

But hmm it explicitly sets the the locale there, so that shouldn't matter. Interested to hear anyway

I get en-US. Not sure where you're located geographically, but it could have something to do with timezone offsets. For example, if I log the result of const date = new Date(Date.UTC(2024, 1 - 1, 1)) (from https://github.com/WickyNilliams/cally/blob/main/src/utils/temporal.ts#L21), I get "Sun Dec 31 2023 19:00:00 GMT-0500 (Eastern Standard Time)".
If I then log new Intl.DateTimeFormat('en-GB').format(date), I get "31/12/2023"

That's got to be it! Thanks for digging in. I wonder if I explicitly set the timeZone option of the DateTimeFormat to UTC it'll fix it.

Yes, that should do it: new Intl.DateTimeFormat('en-GB', {timeZone: 'UTC'}).format(date) === '01/01/2024'

OK, i just merged a PR which uses UTC timeZone across the board. The docs actually use a local build of the lib (not the npm version). So before I publish to npm, can you confirm whether this has fixed the issue? https://wicky.nillia.ms/cally/components/calendar-date/

Yes, LGTM!

Released as 0.4.2. Thanks so much for your help

So, not to rain on the parade, but when I go to https://wicky.nillia.ms/cally/components/calendar-date/, I see:

and while it's now correctly selecting the expected day of the displayed year, the days do not begin on the proper day of the week. January 1st 2024 is on a Monday.

😂 Damn it
I think I know the cause. I'll fix tomorrow and reopen in the meanwhile

OK, i have merged another PR. Now all instances of Intl.DateTimeFormat use the "UTC" timezone. Can you let me know if this has resolved the issue?

It was the same bug as before, just a different manifestation.
I use DateTimeFormat to get day names based on locale. But the user's timezone could introduce drift in the output. For the same reason it output new Date(2024, 0, 1) as December due to timezone differences, it treated a Monday as a Sunday and so on.

Published as 0.4.3. Will close this now. Hoping that's the last of the issues here :)


remove yaml loading from runtime

too slow

This was not actually done yet. This will be done when we blacklist wowapi.yaml from loading from the runtime.


add ACL paper to readme

Add ACL paper to the readme file and also fixed a broken link for the ICLR paper.

Papers sorted in time order.


Disable menu animations

Is it possible to disable the menu animations in rmwc?

@beadle19 did you find a solution to your problem?

Should be setting quickOpen=true on the mdcRef of the menu.


Fix dask flaky tests

Fix #2341
Previously, running this test 10 times resulted in some >1 test failures:

This seemed to be an issue related to the parallelization of the search. By removing the threads_per_worker parameter, we default the threads to 16, which is more than enough to handle AutoMLSearch.
50 runs of this test:

100 runs of this test:

Interestingly, 8 threads fails, but 12 threads passes.

@jeremyliweishih @chukarsten Unfortunately, I wasn't sure why 8 fails. I initially thought that having 4 threads would be enough (since there were 3 pipelnes), but it seems like dask's distributed might not work like that or there's additional stuff happening. I got some performance results here:

But I'm not sure how to further evaluate what is happening. Would appreciate any advice or tips if yall have any!


model_.trees.size() < model_.trees_to_update.size()

Hi, I have the following issue, training incremental XGBoost model, For some reason it works OK with with small amount of data, but when I make training set bigger it fails.
xgboost.core.XGBoostError: [13:08:33] src/gbm/gbtree.cc:278: Check failed: model_.trees.size() < model_.trees_to_update.size() (159 vs. 159) :  Stack trace:   [bt] (0) 1   libxgboost.dylib                    0x0000000125d4f579 dmlc::LogMessageFatal::~LogMessageFatal() + 57   [bt] (1) 2   libxgboost.dylib                    0x0000000125daf83c xgboost::gbm::GBTree::BoostNewTrees(xgboost::HostDeviceVector<xgboost::detail::GradientPairInternal<float> >*, xgboost::DMatrix*, int, std::__1::vector<std::__1::unique_ptr<xgboost::RegTree, std::__1::default_delete<xgboost::RegTree> >, std::__1::allocator<std::__1::unique_ptr<xgboost::RegTree, std::__1::default_delete<xgboost::RegTree> > > >*) + 620   [bt] (2) 3   libxgboost.dylib                    0x0000000125dabfd0 xgboost::gbm::GBTree::DoBoost(xgboost::DMatrix*, xgboost::HostDeviceVector<xgboost::detail::GradientPairInternal<float> >*, xgboost::ObjFunction*) + 320   [bt] (3) 4   libxgboost.dylib                    0x0000000125d4b8be xgboost::LearnerImpl::UpdateOneIter(int, xgboost::DMatrix*) + 1518   [bt] (4) 5   libxgboost.dylib                    0x0000000125d6a314 XGBoosterUpdateOneIter + 180   [bt] (5) 6   _ctypes.cpython-37m-darwin.so       0x000000011237536f ffi_call_unix64 + 79   [bt] (6) 7   ???                                 0x00007ffee6dd0b20 0x0 +<PHONE_NUMBER>66720
the code for model
params = { 'verbose': False,  'eval_metric': 'rmse', 'n_jobs':24, 'max_depth':params_xgb['all']['max_depth'],
                                 'min_child_weight':params_xgb['all']['min_child_weight'],
                  # 'learning_rate': 0.007,
                  'updater': 'refresh',
                  'process_type': 'update',
                  'refresh_leaf': True}
 model = xgb.train(params, dtrain=xgb.DMatrix(X_train, y_train), evals = watchlist,num_boost_round =300, early_stopping_rounds=20,
                          xgb_model=model,evals_result=progress, verbose_eval = 10)

xib model in the code is defined like this
params = { 'verbose': False,  'eval_metric': 'rmse', 'n_jobs':24, 'max_depth':params_xgb['all']['max_depth'],
                                 'min_child_weight':params_xgb['all']['min_child_weight']}
model = xgb.train(params, dtrain=xgb.DMatrix(X_train,y_train), evals = watchlist, num_boost_round =300,early_stopping_rounds=20,
                          evals_result=progress,verbose_eval = 10)

I have Mac.os Mojave 10.14.3
python 3.7.4
xgboost 0.90

If you don't silent XGBoost it should give you a warning about using updater parameter.  The refresh updater is used to refresh built tree, not building new trees.  I suspect that's the problem.  :-)

@trivialfis it does warn but in incremental learning it is the point, isn't it? I want to update my trees not to increase amount of them according to the new data...
I do not understand why for some amount of data it works and for some - does not
Thanks in advance

I don't understand what you meant.  Model in boosting is fixed once learned.  You can't modified the built tree, only adding new ones is valid.

@trivialfis , here it is shown how incremental learning works https://gist.github.com/ylogx/53fef94cc61d6a3e9b3eb900482f41e0. Likewise my case here updater is "refresh"

@Mary-Kul Thanks.  Reopening so that we can take a closer look into it.

@Mary-Kul Could you provide a reproducible script?  I can't reproduce it on my end with 1M rows of data.

I ran the gist on master branch and it works fine. Should be fixed with new model IO routines.

I ran the gist on master branch and it works fine. Should be fixed with new model IO routines.

It is not fixed for me, I keep having the same error even though the tree sizes are exactly the same (42), as you can see in the log:
/workspace/src/gbm/gbtree.cc:275: Check failed: model_.trees.size() < model_.trees_to_update.size() (42 vs. 42) :
Stack trace:
  [bt] (0) /home/kubote/.local/lib/python3.8/site-packages/xgboost/./lib/libxgboost.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x54) [0x7fdef0686614]
  [bt] (1) /home/kubote/.local/lib/python3.8/site-packages/xgboost/./lib/libxgboost.so(xgboost::gbm::GBTree::BoostNewTrees(xgboost::HostDeviceVector<xgboost::detail::GradientPairInternal<float> >*, xgboost::DMatrix*, int, std::vector<std::unique_ptr<xgboost::RegTree, std::default_delete<xgboost::RegTree> >, std::allocator<std::unique_ptr<xgboost::RegTree, std::default_delete<xgboost::RegTree> > > >*)+0xe7f) [0x7fdef073592f]
  [bt] (2) /home/kubote/.local/lib/python3.8/site-packages/xgboost/./lib/libxgboost.so(xgboost::gbm::GBTree::DoBoost(xgboost::DMatrix*, xgboost::HostDeviceVector<xgboost::detail::GradientPairInternal<float> >*, xgboost::ObjFunction*)+0x121) [0x7fdef0737b91]
  [bt] (3) /home/kubote/.local/lib/python3.8/site-packages/xgboost/./lib/libxgboost.so(xgboost::LearnerImpl::UpdateOneIter(int, xgboost::DMatrix*)+0x3e1) [0x7fdef0772a11]
  [bt] (4) /home/kubote/.local/lib/python3.8/site-packages/xgboost/./lib/libxgboost.so(XGBoosterUpdateOneIter+0x29) [0x7fdef0675639]
  [bt] (5) /usr/lib/libffi.so.6(ffi_call_unix64+0x4c) [0x7fdf36a9b69a]
  [bt] (6) /usr/lib/libffi.so.6(ffi_call+0x196) [0x7fdf36a9afb6]
  [bt] (7) /usr/lib/python3.8/lib-dynload/_ctypes.cpython-38-x86_64-linux-gnu.so(+0xf11d) [0x7fdf36b0111d]
  [bt] (8) /usr/lib/python3.8/lib-dynload/_ctypes.cpython-38-x86_64-linux-gnu.so(+0x1383d) [0x7fdf36b0583d]

Let me take another look.

@kevinkvothe If you are running the latest XGBoost release without silent, there should be a warning saying parameter update is not used. The correct parameter name should be updater. To modify that notebook to run it correctly, first you need to train a model with default process_type, so that you can have some trees to update. Then use pruner or refresh for updating existing trees.

I'm going to share a notebook I made to test this error. It should be completely reproducible.
Btw, same error appears if I use prune as the updater parameter together with update as process_type.
test.zip

So if I understand correctly, setting a fixed num_boost_round equal to the number of trees trained in the first model (the one we update over it) should solve the issue?

Yup.  The gist is don't update non-existing tree.

I also get teh same error, but with 0.
heck failed: model_.trees.size() < model_.trees_to_update.size() (0 vs. 0)
I tried this after the first iteration but I get another error saying Get_booster is not a class of Boost.
dump_list = walmart_model.get_booster().get_dump()
num_trees = len(dump_list)

model=None
params={'learning_rate': 0.007,
'tree_method': 'exact',
'refresh_leaf': True,
#'reg_lambda': 3,  # L2
'reg_alpha': 3,  # L1
'silent': False}
for (i,j) in zip(X_train_list,y_train_list):
dtrain=xgb.DMatrix(i,j)
model=xgb.train(params, dtrain, num_boost_round=10, evals=(), obj=None, feval=None, maximize=False, early_stopping_rounds=None, evals_result=None, verbose_eval=True, xgb_model=model, callbacks=None)
params={'learning_rate': 0.007,
'tree_method': 'exact',
'updater':'refresh',
'process_type': 'update',
'refresh_leaf': True,
#'reg_lambda': 3,  # L2
'reg_alpha': 3,  # L1
'silent': False}
This worked for me. I hope its correct


Fixed the issues you were talking about.

Sorry for the delay...

Will give it a try tomorrow

Justed tested the current version. Here is a Screenshot of how the changes look like ATM:

Feature wise:

atm occurs a js syntax error, please fix that

SyntaxError: missing ) after argument list php-console.js:201:9
TypeError: $.console is not a function

after fixing this error the feature itself works for me in FF41 & Chrome44

UI wise:

To separate the new feature from the current "reset" feature, I propose to align the new inputs to the right.
After running the script the UI gets even more crowded with the memory informatioin, see

Summary: the feature itself is nice, but the UI needs some polish before this can be merged.
Thanks for your work so far.

@lowrykun any updates on this ?


classList support

From your website it shows that you don't support classList for IE 10 and 11 but do support it in earlier browser? This is pretty confusing. Is that the case or is the website wrong? It seems like an easy thing to polyfill.
From my own testing classList .contains did not work. in IE9.

I think you are describing two issues.
Polyfilling classList for IE10/11:
classList exists in IE10 and IE11, it however does not support multiple parameters to add and remove methods. The polyfill we use also does not support multiple add or remove. If we add support for mulitple parameters to add and remove methods, we would include IE10/11 in the polyfill targeting.
classList polyfill not working in IE9:
You haven't given us much information to go on. Could you please provide the minimum amount of code required to show the issue you are seeing, preferably in a jsbin or codepen page. With a reproducible test, we should be able to help diagnose if there is an issue and whether it can be resolved.

Hi Jake. Here's a bin.
http://jsbin.com/puvikinotu/2/edit?html,js,output
Console error message: error Unable to get property 'contains' of undefined or null reference (line 3)

This should be fixed by https://github.com/Financial-Times/polyfill-service/pull/1347


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade" workflow

Codecov Report

Merging #68 (1de1fe3) into main (4958138) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #68   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           27        26    -1
=========================================
- Hits            27        26    -1

Impacted Files
Coverage Δ

src/backend-stack.ts
100.00% <100.00%> (ø)

src/lambda/process-metrics-function.ts
100.00% <100.00%> (ø)

src/stateful-stack.ts
100.00% <100.00%> (ø)

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 27e0f2e...1de1fe3. Read the comment docs.


Fill button does not indicate status (toggle)

Bug Description
When making an image full-bleed (fill), the button is not toggled
Expected Behaviour
The button should show the current status
Steps to Reproduce

Insert image
Mark as full-bleed
Notice that fullbleed button does not change status to indicate that the image is fullbleed.

Screenshots
Status quo (see button on the right)

It should be marked "active" just like the alignment button here:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

@swissspidy What about flip controls? Should they also reflect the status when selected?

@obetomuniz IIRC no. But I'll let @samitron7 confirm.

Pascal is correct. It is not a toggle.


Typescript defined config file

I've been using webpack with a typescript config file. Unfortunately, the webpack_bundle rule is not allowing this:
(14:32:23) ERROR: BUILD.bazel:130:15: in webpack_config attribute of webpack_bundle rule //:bundle_dev: source file '//:webpack-dev-build.config.ts' is misplaced here (expected .js)

Webpack supports typescript itself:
https://webpack.js.org/configuration/configuration-languages/#typescript
Just looks like an incorrect restriction from the Bazel rule

Have you tried compiling that ts first and passing the compiled js? Normally any typescript would be compiled to js with ts_project before being passed to webpack, and I think the same should be done for config files.

I'm not talking about the code being packed, I'm referring to the actual webpack configuration file being a Typescript file. The Bazel rule is only allowing .js, but Webpack supports multiple formats, at least .js and .ts files. So I'm just requesting the attribute of the Bazel rule to permit .ts as a file extension for the configuration file

I'm talking about the webpack config file.
Outside bazel typescript is normally compiled with a webpack plugin, I assume the webpack config as typescript is the same. In bazel (or the aspect rules_* at least) the typescript is compiled by ts_project and webpack is passed the resulting .js files, webpack itself doesn't need to know anything about typescript.
Can you compile the webpack config into .js before passing it into webpack?

Webpack supports typescript config files, for the reason that you don't HAVE to transpile it to JavaScript first. All the bazel rule needs to do is add .ts as a supported file extension. So someone could transpile it first, but that is a whole lot more work than just allowing a ts file to be passed as a config file. As for:

webpack itself doesn't need to know anything about typescript.

It already has code to support typescript config files directly without any other help. It has supported Typescript types since 2020
Why is there a pushback for such a simple change.
Just change:
    "webpack_config": attr.label(
        doc = """Webpack configuration file.

See https://webpack.js.org/configuration/""",
        allow_single_file = [".js"],
        mandatory = False,
    ),

to:
    "webpack_config": attr.label(
        doc = """Webpack configuration file.

See https://webpack.js.org/configuration/""",
        allow_single_file = [".js", ".ts"],
        mandatory = False,
    ),

If that's all that's needed it is probably fine, as long as we don't need to configure typescript within webpack. I'd still think it's better to let bazel compile it and cache it but one file is pretty minor 🤷


Support @RequiresStableInput on Dataflow runner in Java SDK

During pipeline surgery, DataflowRunner will replace a transform T with (Reshuffle + T), if T is a ParDo.SingleOutput or ParDo.MultiOutput transform with a DoFn whose ProcessElement method is annotated with @RequiresStableInput.
r: @kennknowles

retest this please

retest this please

retest this please


Technical Documentaion for all Platform Wikis

https://github.com/ASKnetCommunity/ASKnet.Community/wiki/asknet-community/
For Technical documentation all Wikis/Platfoms need a section (Specs, References, HowTos for Maintainers that are not specific to a section etc.)
This is also possible to link to the right README Section

done  thank you @bmen
https://github.com/ASKnetCommunity/ASKnet.Community/wiki/asknet-community#technical-documentation
https://github.com/ASKnetCommunity/LEAD/wiki#technical-documentation


Post-Op bailout with unconditional branches

When searching for new bytecode instr for a post-op bailout, follow unconditional branches.
OS#17472973

This change is

What does the test/IR look like?

The test looked something like
function foo(a) {
  switch (a) {
    case '': return this;
    case 1:
  }
}
foo(1);

Because of Switch case int/string optimisation we could end up with the following IR after IRBuilder
BrSrEq_A $L2 s1, s2
// case 1:
BrSrEq_A $L3 s1, s3  #0004
Br $L1                #0004
// EndSwitch
Br $L1                #0006
// return this <== body of first case
$L2: ... Br $L1
$L3: ...
$L1: // end of switch

FG Peeps would then remove Br $L1 #0006 because it is unreachable.
Later, when trying to add a bailout on implicit calls to BrSrEq_A, just looking at next instr, we would end up tyring to bailout to the body of the first case instead of the end of the switch case.
By Following unconditional branches, we at least bailout after the switch case.
This sounds pretty bad as it is, however, it turns out that bailout is actually impossible, because BrSrEq_A cannot have implicit calls when one of the opnd is guarantied to be an int or string.
I will follow up in another PR to prevent emitting the bailout in the first place.

ping

Would be good to add the unit test. Otherwise LGTM.


Add facility to plug-in custom DB connection pools

Dropwizard by default is shipped with the Tomcat JDBC Connection pool. It's mostly fine, but has some disadvantages (performance, limited metrics, handling network failures). It would be great if we had facility to plug-in other DB pools into Dropwizard. For example, Brett Wooldridge's
wonderful HikariCP or BoneCP.
Of cource, nothing stops us from implementing a factory, that creates a ManagedDataSource. The problem is that dropwizard-jdbi, dropwizard-hibernate and dropwizard-migrations modules require DataSourceFactory for the configuration. DataSourceFactory is closely tied with the Tomcat pool. So if we use an another pool, we are forced to write own factories for JDBI, Hibernate, etc...
A solution is to provide an interface that abstracts a configuration of a connection pool. Database manipulation modules will be linked with the interface, but not with DataSourceFactory. A custom pool factory just need to implement this interface, and it could be plug-in as a Tomcat pool
replacement.
If we writed Dropwizard from the scratch we would move the Tomcat pool to a separate module from dropwizard-db. But now we have many users and it would be a nightmare for them to migrate to a new implementation.
So we are forced to retain the Tomcat JDBC pool in dropwizard-db. New pool users should be advised to exclude it manually in the Maven configuration.

Ping. Does any additional work needed in this PR?

One small nitpick, 'HelloWorldConfigurationin the example should declare the variables asPooledDataSourceFactory`.
Otherwise this PR looks good to me. Thanks for the hard work!

One small nitpick, 'HelloWorldConfigurationin the example should declare the variables
asPooledDataSourceFactory`.

I don't think it's farsightedly, because in this case we will need to enable polymorphic-configuration for PooledDataSourceFactory. And this looks like overkill, because nobody probably will want to change pool implementations in the configuration.

Coverage increased (+0.72%) to 72.99% when pulling 1033cc60dc04e1879c28f8b28678ce0e637da65f on arteam:abstract_ds into b63feeb184200818f34b7ac693c32bdc03e5c0a0 on dropwizard:master.

@arteam is there documentation on how to use this? I don't quite see how:
Of cource, nothing stops us from implementing a factory, that creates a ManagedDataSource. The problem is that dropwizard-jdbi, dropwizard-hibernate anddropwizard-migrations modules require DataSourceFactory for the configuration. DataSourceFactory is closely tied with the Tomcat pool. So if we use an another pool, we are forced to write own factories for JDBI, Hibernate, etc...

has been sidestepped.

Seems like you're still left needing something like https://github.com/DeloitteDigitalUK/dropwizard-hikaricp, which as far as I can tell, still misses out on the nice features that DBIFactory provides.


You are not authorized to read this file or the file does not exist

I have 22 songs in a shared folder and when I log into a different account and select the same drive folder I get this message at the bottom many times. I have no other non-music files in the shared folder.

Hey @eaesch,
The way the permissions work for Google Drive is that you have to "open" these files with your new account via Google Drive (right Click > Open-with) or via the "Add audio files from Google Drive" Widget inside the app (which is provided by Google). As you've noticed sharing the files with a new account and opening the same URL won't work unfortunately...


Add nlpsandbox.io badge

Fixes #88

The badge renders correctly in VS Code but not on GitHub:

@thomasyu888 This is the page of the leaderboard of the Date Annotator (staging; one table for now, maybe more in the future). It would be awesome if we could add a query parameter to the URL to filter the content of the table(s). Is it possible? Can the Synapse tables use a query parameter to filter their content?

You unfortunately can't add to query parameters to the URL but you can definitely add to the query statement in the widget. Or... You can link out to a specific query from the submission view ( it's a pretty ugly and long link per query )

I opened a ticket on Jira to register the Synapse logo on https://simpleicons.org/:
https://sagebionetworks.jira.com/browse/SWC-5415

You unfortunately can't add to query parameters to the URL but you can definitely add to the query statement in the widget. Or... You can link out to a specific query from the submission view ( it's a pretty ugly and long link per query )

This idea - which you probably understood - is to customize the URL of the badge so that when clicked, we navigate to the leaderboard page and see only the submission for the tool name "awesome-team" or made by the submitterId "xyz". A button on the page would allow to clear this filter.
If I understand correctly, there is no way on Synapse to customize the content of a table embeded in a page using query parameters, is that correct? But are you saying that it's possible if we redirect to a Synapse Table directly (i.e. not embeded in a wiki page)?

So there are already dashboard pages that allow submitters to see their own submissions.  This is done by adding to the SQL query and displaying it on the leaderboard.
You are correct when you say that there is no way of customizing the contents of table embedded in a wiki page with URL OR the table itself.  To clarify, you cannot do this:
wiki page: Synapse.org/...syn123/wiki/1234?query=submitter=1111
Table: synapse.org/...syn1233?query=submitter=1111
You must do the query first in the table, then you will get a link. You can provide this link to people but it will always be that specific query.  Does this make sense? (Sorry am on my phone)


Address performance regression introduced in 63d17b0

Address performance regression introduced in 63d17b0
(Please see https://github.com/flutter/flutter/issues/73794 for more details)
This issue has been created to track the Dart side changes to address the regression.

https://dart-review.googlesource.com/c/sdk/+/181542 is the CL to revert the changes (63d17b0 and 0bf8d38).

https://dart-review.googlesource.com/c/sdk/+/181542 is the CL to revert the changes (63d17b0 and 0bf8d38).

https://dart-review.googlesource.com/c/sdk/+/181542 has landed

https://dart-review.googlesource.com/c/sdk/+/181542 has landed

Is this still a P0 now that the revert has landed?

Is this still a P0 now that the revert has landed?

I wanted it to be a P0 until I got the cherry pick request into the beta branch completed.

I wanted it to be a P0 until I got the cherry pick request into the beta branch completed.

A cherry pick request for this change into the beta branch has been created https://github.com/dart-lang/sdk/issues/44807,
lowering the priority and changing the milestone to reflect the forward fix that @rmacnak-google is working on.

A cherry pick request for this change into the beta branch has been created https://github.com/dart-lang/sdk/issues/44807,
lowering the priority and changing the milestone to reflect the forward fix that @rmacnak-google is working on.

@rmacnak-google Any update?  Are we on track for Feb. milestone?

The revert addressed this performance issue and other changes will be tracked in https://github.com/dart-lang/sdk/issues/41974


URL not working

Error running app. If this keeps happening

Error running app. If this keeps happening

Can you share more details of the error?


Test job verification steps missing/unclear

Re the guidance at netarbiter/sds/ceph-docker/examples/helm/README.md:
Attach the pvc to a job and check if the job is successful (i.e., 1)
kubectl create -f tests/ceph/job.yaml
kubectl get jobs ceph-secret-generator -n ceph
The verification if the job is successful involves more than testing the ceph-secret-generator job. The test also creates job "ceph-test-job" in namespace default, which has to run to completion. The guide should clarify that additional verification step needed.

"kubectl get jobs ceph-secret-generator -n ceph" was wrong, so I fixed it. For verification, you can run the followings:
kubectl create -f tests/ceph/job.yaml
kubectl get jobs ceph-secret-generator -n ceph

Please refresh your browser.
Hee Won Lee<EMAIL_ADDRESS>


Refactor(#905): Implementing Class, Linting

Partial in #905

Implementing Classes
Code Refactor of wire.js
Fixed Linting errors.

Coverage remained the same at 83.902% when pulling f6cd43f044ed55a3c3359601d2e4fde82833c7bf on YashKumarVerma:refactor into 097930075bb42cc181a9742cfbd6d3e83431f826 on CircuitVerse:master.

The errors of codeClimate should not be resolved. The reason being, it is asking to dissolve the functions into more dry methods, which can be good for the language, but not at all for the developers.
Consider this:

If the above code is refactored and put in blocks of if conditions to not repeat the conditions, then it will be very difficult to let alone edit, even understand the conditions.
All the issues raised by code Climate are listed here.


Flowline Points Calibration

Using 2 or more years of data with calibration  flowline points in use the flowline points tool will return no features.

Eagle Creek Oregon Reach 1, 2008 data Flowline Points creates no features when tool completes successfully. Both 2008 and 2018 data create flowline points when tool 06 is run without calibration. When tool 06 is run to calibrate the 2008 data with the 2018 flowline points it completes but does not create points for the 2008 data.


[v1.6] ci: update k8s to 1.13.12, 1.14.8, 1.15.5 and 1.16.2

also update vendor libraries
Add support for k8s 1.16.2

This change is

never-tell-me-the-odds

test-me-please

test-me-please


(hot)fix ANR: Add runOnJS prop

Hot fixes #259

@hoangnm it fixes the issue in RNGH, as I reported in this bug in their repo.
Setting gesture.runOnJS(true) makes the animations run in the JS context instead of the UI thread. In other words, the callback provided inside onEnd(() => {...}) is run in the JS context instead of the UI thread. Apparently, the bug only occurs when running certain code on the UI thread.
In theory, animations are smoother when run in the UI thread. But still they can be run in JS and works fine. (Read more details in rn-reanimated worklets docs)

thanks, it makes sense, let's merge this to see if it works for other users.

wait, I can create the alpha version first.

@hoangnm do you have an ETA on merging this? This issue has been waiting for long :sweat_smile:

ok, merging.


Add multi-container recording support

What type of PR is this?
/kind feature
What this PR does / why we need it:
This adds support for multi container recording via the OCI seccomp BPF
hook. To not confuse users, we now require the (unreleased) version of
CRI-O v1.21.0. We always suffix the recording by the container name,
which allows the user to distinguish the profiles in a sane way.
Which issue(s) this PR fixes:

None
Does this PR have test?
Yes

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm

This is awesome!


.github: decrease the OSS fuzzing time

10 minutes looks like a too big number, especially given that other CI
actions finish much faster.

Did this rule ever fail?
Let's listen to dvyukov@ here.

Did this rule ever fail? Let's listen to dvyukov@ here.

https://github.com/google/syzkaller/actions/workflows/oss-fuzz.yml?query=is%3Afailure
It seems that it's only failing with some internal OSS-Fuzz errors.
I don't mind either way. This step does not block merging.
Btw, there is now something called ClusterFuzzLite:
https://google.github.io/clusterfuzzlite/
ClusterFuzzLite ([repo](https://github.com/google/clusterfuzzlite/)) is a continuous [fuzzing](https://google.github.io/clusterfuzzlite/overview/#fuzzing) solution that runs as part of [Continuous Integration (CI)](https://en.wikipedia.org/wiki/Continuous_integration) workflows to find vulnerabilities faster than ever before. With just a few lines of code, GitHub users can integrate ClusterFuzzLite into their workflow and fuzz pull requests to catch bugs before they are committed.

But not sure how it's different from what we do now...


Unable to compile

You are using TSC -g installed at 1.8.10 but local build of tsc.gulp is at <IP_ADDRESS>. I would suggest installing typescript <IP_ADDRESS> with -g flag.
npm install -g<EMAIL_ADDRESS>

Thx. It worked.


circular link to itself

This page has a circular link to itself on the last bullet for "Next Steps" :)

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

ID: 5f7a4070-1be8-5f1f-0092-68fb6b9e2305
Version Independent ID: cd34312f-b507-affe-555e-1e0c63e7d494
Content: Dynamic automatic group membership rules - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/groups-dynamic-membership.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@netwovenIT Thank you for pointing that out . We will investigate and link it correctly .

@netwovenIT We have initiated a change request for the same and it should be updated in the next 24-48 hours as per the next release cycle. We will now close this issue. Should you have any further query, feel free to open a new issue.
Thank you.


simplify package structure

In order of commits:

Files were moved so that we are not working with so many __init__.py files.

sequence.py moved into top level
eitdata.py, sparsedata.py, continuousdata.py now all live inside datacollection (formerly data_collection) folder.
eitdata.loading is now datacollection.loading
binreader.py moved into datacollection.loading

Phase and Events moved into sparsedata.py and are now subclasses of SparseData
Vendor class moved into datacollection's init.
use mmap to speed up draeger data loading (this got lost somehow, probably in #160 )

supplanted by #166


[now-next] Add support for SPR in now-next

Adds support for new SPR handling in Next.js
relies on https://github.com/zeit/next.js/pull/8840
x-ref: #3021

Codecov Report

Merging #3067 into canary will increase coverage by 1.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #3067      +/-   ##
==========================================
+ Coverage    11.6%   13.47%   +1.86%
==========================================
  Files         267      267
  Lines       10069    10301     +232
  Branches     1299     1212      -87
==========================================
+ Hits         1169     1388     +219
- Misses       8809     8851      +42
+ Partials       91       62      -29

Impacted Files
Coverage Δ

src/util/metrics.ts
68.18% <0%> (-2.41%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
28.57% <0%> (+3.57%)
:arrow_up:

src/util/get-files.ts
91.66% <0%> (+3.88%)
:arrow_up:

src/util/dev/router.ts
82.69% <0%> (+4.12%)
:arrow_up:

src/util/dev/builder.ts
76.92% <0%> (+4.4%)
:arrow_up:

src/util/dev/yarn-installer.ts
90.74% <0%> (+5.02%)
:arrow_up:

src/util/dev/server.ts
66.43% <0%> (+5.41%)
:arrow_up:

src/util/prefer-v2-deployment.ts
61.9% <0%> (+5.65%)
:arrow_up:

src/util/output/create-output.ts
43.58% <0%> (+12.33%)
:arrow_up:

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 9ae747a...a899ab8. Read the comment docs.


http request proxy to https backend server

how can i use this to proxy http request from frontend to https backend server?I aways get this message:
Error occurred while trying to proxy request /api/login from localhost:8888 to https://myserverip:22 (EPROTO) (https://nodejs.org/api/errors.html#errors_common_system_errors)

This has been asked and answered before: https://github.com/chimurai/http-proxy-middleware/issues/24#issuecomment-139020987
Please follow the guidelines: CONTRIBUTING.md#got-questions-or-problems


OpenJ9 reproducibility

openjdk-build seems overly complicated, but it looks like openj9 builds from a branch, not a tag. for example, https://github.com/ibmruntimes/openj9-openjdk-jdk14/commits/openj9-0.21.0 has had several updates since initial branch. how can users determine which version of openj9-openjdk-jdk14 is used? furthermore, the openj9 and omr hashes are not locked, so do they just float with whenever the build happens to be triggered?

@pshipton Could you please answer?

@Hello71 see https://github.com/eclipse/openj9/releases/tag/openj9-0.21.0
OpenJ9 and OMR are built from tags, in this case openj9-0.21.0
OpenJDK is built from the branch https://github.com/ibmruntimes/openj9-openjdk-jdk14/tree/openj9-0.21.0, but the contents won't change. If you think it's necessary, we can ask the maintainers to create a tag to correspond to the release.

This is where the tags for OpenJ9/OMR are set. https://github.com/ibmruntimes/openj9-openjdk-jdk14/blob/openj9-0.21.0/closed/get_j9_source.sh#L60

Thanks, I saw that already. I was concerned because 14.0.2 was released on July 8, but there were commits at https://github.com/ibmruntimes/openj9-openjdk-jdk14/commits/openj9-0.21.0 until July 15. Checking closer though, https://github.com/AdoptOpenJDK/openjdk14-binaries/releases/tag/jdk-14.0.2%2B12_openj9-0.21.0 wasn't built until July 16, so I now see that there is no problem there.
For Linux distro integration, we strongly prefer tags, as git branches connote potential change, even if they are technically identical. Even if there is a statement that branches are considered fixed, it is not always clear from outside when exactly that is the case. For example, https://github.com/ibmruntimes/openj9-openjdk-jdk14/commits/openj9-0.21.0 had commits between July 8 (?) and July 15. I guess it was discussed internally and "released" on July 16, but as far as I (and other developers) can see, there doesn't seem to be any clear external indicator of when that is the case. I am currently working on Arch and Gentoo, but these points are common across all Linux distros. https://github.com/eclipse/openj9 and https://github.com/eclipse/openj9-omr are fine here, because they have both branches (vx.y.z-release) and tags (openj9-x.y.z).
We would also slightly prefer a single source tarball as opposed to downloading separate files. Most Linux distros do not permit network access during builds for security, reproducibility, and upstream load reasons. From what I can see now, that would need to include openj9-openjdk-jdk14, openj9, openj9-omr, and freemarker. I haven't finished testing that though, because  It is of course possible to download multiple files, but it would be more convenient for both us and any other users building their own JDK if for example https://github.com/ibmruntimes/openj9-openjdk-jdk14/releases had unified tarballs so you can just download, extract, ./configure; make; run. In general, bundled libraries are frowned upon on Linux, but it seems to me that the JVM is currently closely tied to the JDK, so that it is not possible to use a single system OpenJ9 with multiple JDKs linked to it.
I think it would also be nice if AdoptOpenJDK was more clear about which sources it's building from. For example, on https://adoptopenjdk.net/releases.html it could say something like "Built from https://hg.openjdk.java.net/jdk-updates/jdk14u/rev/jdk-14.0.2-ga using https://github.com/AdoptOpenJDK/openjdk-build/commit/2259450b995e6ca0ea46d442db2064d5d0392c30" or something like that. For OpenJ9, it would need to be a little bit longer, but hopefully you get the idea.
This is getting a bit long for a bug report though, and we can move it to the mailing lists if you'd prefer.

AdoptOpenJDK does not build from Mercurial forests but from our mirrored Git repositories, e.g. https://github.com/AdoptOpenJDK/openjdk-jdk14. You can find metadata next to the binaries on GitHub, for example https://github.com/AdoptOpenJDK/openjdk14-binaries/releases/tag/jdk-14.0.2%2B12. Look for the JSON files. If you want more metadata, please open an issue on https://github.com/AdoptOpenJDK/openjdk-build/issues.

OpenJDK didn't release the final code to open source until July 14, even if it was finalized internally by July 8, which accounts for most of the changes to the openj9-0.21.0 branch. The release process started asap after that, with mirroring, merging, build, test, assessment of the test results, creation of the release tags taking the time before the July 16 release. The branch was initially created in June, and used to produce some milestone builds, with additional fixes added prior to the release. There was a last minute fix added (https://github.com/ibmruntimes/openj9-openjdk-jdk14/pull/67), which we knew was coming but couldn't make until we had the final OpenJDK code. The other change sets the default OpenJ9/OMR release tags to build from.

JVM is currently closely tied to the JDK, so that it is not possible to use a single system OpenJ9 with multiple JDKs linked to it.

Correct. Although the same OpenJ9/OMR source supports Java 8, 11, 14, etc., after compilation some of the shared libraries are specific to the Java version, and the others are closely tied to those versions. In the future there will be fewer OpenJ9 related shared libraries.
@andrew-m-leonard please see this request for tagging the release commits https://github.com/AdoptOpenJDK/openjdk-support/issues/148#issuecomment-663687754. Also the request for creation of a single tarball containing all the sources.

It looks like all questions have been answered. If you think otherwise, please reopen.


⚠ Jenkins Plugins Artifactory has degraded performance

In 283012a, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2575 ms

Resolved: Jenkins Plugins Artifactory performance has improved in f0187d7 after 1 hour, 27 minutes.


aur srcver didn't return anything

https://github.com/AladW/aurutils/blob/3bd2fcbcacdb939cbc782a99702580db35cdc24b/lib/aur-srcver#L68
I think while IFS=$'\0' read -r; isn't working properly. Maybe use while IFS= read -r -d ''; instead.

Thanks & fixed.


Fixes public API by exporting Extends

Hi @garronej,
I forgot to update the public API defined in index.ts.
This PR adds the missing export of the new Extends type.
Could you please release 1.1.1 containing this patch?
Many thanks!
(Note: The Gitbook update is also on its way soon...)

Hi @danieldietrich,
Thanks for the patch!

@deusz Equals<X, Y> would satisfy your requirements.

Not exactly, bacause in Y type I don't need x: string prop.

@deusz ok, I understand your usecase.
I assume that this isn't an option:
import { assert } from "tsafe/assert";
import type { Equals } from "tsafe";

type Y = {
  y: { a: 'x' | 'c' | 'r' }
}
type X = {
  x: string
  y: { a: 'x' | 'c' }
}

assert<Equals<Y, Omit<X, "x">>();

I'd call this util IsPick<A,B> and I would define it as follow:
Be it two types Y and X
IsPick<Y,X> =>  There is a K that extends keyof X such that Equals<Pick<X,K>, Y>
In the above example type K = "y"
With another example:
type Y= {
    b: string;
    c: string[];
};

type X= {
    a: number;
    b: string;
    c: string[];
    d: number[];
};

K would be "b" | "c"
Are we on the same page?

Yes, IsPick coresponds with Pick util, so that is what I mean. Would be nice if that util works also with nested props.

Then maybe IsPartial would be a beter name for it...
Do you feel like trying to implement it yourself?

Well, I already tried. I have problem in part 'x' | 'c' | 'r' like above. And now I don't believe it is possible.

Feel free to open a PR even if you don't have the complete solution. If you write the test file and the doc we are 80% there. Then writing the implementation is the fun part and we should be able to mak it work.


Pre trained weights dataset and input size

💡 Your Question
Hi,
I read it mentioned somewhere that Yolo-NAS's "multi-phase training process involves pre-training on Object365, COCO Pseudo-Labeled data, Knowledge Distillation (KD), and Distribution Focal Loss (DFL)". Can I check if the pretrained weights in the model zoo here: https://github.com/Deci-AI/super-gradients/blob/master/documentation/source/model_zoo.md are prerained on ALL the above?
Also, can I clarify if I can use these pretrained weights (like YOLO-NAS L) as a backbone for a model of a different input size? Like 1024 x 576. And is there a specification for what input sizes are allowed? (e.g. multiple of 32)
Lastly, can I clarify if these pretrained weights are already trained with quantization awareness, and if there is much value in adding quantization awareness in fine tuning to my tasks?
Versions
No response

The model was pretrained using the setting you mentioned, yes
The model supports different input sizes. I do believe that it should be a multiple of 32, as the maximum downscale is 32
The quantization awareness comes in the form of architectural structure (quantization friendly layers), so performing QAT on your dataset should help

Let me know if there are further questions

The model was pretrained using the setting you mentioned, yes
The model supports different input sizes. I do believe that it should be a multiple of 32, as the maximum downscale is 32
The quantization awareness comes in the form of architectural structure (quantization friendly layers), so performing QAT on your dataset should help

Let me know if there are further questions

Was it trained with those datasets though? Here (https://github.com/Deci-AI/super-gradients/issues/1143) it states that the Object365 weights were not released. So the pretrained weights are just purely trained on COCO?


[BUG] Variable namespace pollution

Are you using the latest version?
4.0.0
The issue is with the following feature(s):

[ ] Traversal to root of a git repository.
[ ] Search & traverse w.r.t. to a base directory.
[ ] Search & traverse recently visited directories.
[ ] Sub-directory search & traversal.
[ ] Parent-directory search & traversal.
[ ] Piping.
[ ] Cleaning of log file.
[ ] Installation.
[ ] Dependencies/Requirements.
[ ] Configurations.
[ ] Documentation.
[x] Other(s)

Issue observed in the following shell(s):

[x] Bash
[x] Zsh

Describe the bug
I get unwanted and unneeded extra variables after sourcing this.
Eg _feats.  (which may be better named singular anyway).
To Reproduce
Steps to reproduce the behaviour:

( set -o posix ; set ) > before
Source plugin
( set -o posix ; set ) > after
diff before after

Expected behaviour
Minimum set of required variables are added, and only exported if needed

Nice catch! Thanks!


Would you mind sharing your code?

Dear Sir
I am very interested in your project. Would you mind sharing your code?

Dear Sir I am very interested in your project. Would you mind sharing your code?
Thank you very much for your interest in this work. I'm very sorry, I've been busy with reporting, next work and rebuttal. So, I haven't found the time to post the code yet. However, I promise to release the code at the end of the month.

Dear Sir I am very interested in your project. Would you mind sharing your code?

Thank you very much for your interest in this work. I'm very sorry, I've been busy with reporting, next work and rebuttal. So, I haven't found the time to post the code yet. However, I promise to release the code at the end of the month.

Thank you very much. Looking forward to your code and wishing you all the best.

亲爱的先生，我对您的项目非常感兴趣。您介意分享您的代码吗？

非常感谢您对这项工作的兴趣。非常抱歉，最近忙于报道、下一步工作和反驳。所以，我还没有找到时间发布代码。不过，我保证在月底发布代码。

Thank you very much. Looking forward to your code and wishing you all the best!


luigi is a color channel?

It should have been called green instead of luigi.

Hey @bjorn3 this was supposed to be a joke. 😜  since Mario is red then Luigi might be the green channel.

:)


Better way to load grunt tasks

From https://saucelabs.com/javascript/jasmine-js:
// Loading dependencies
for (var key in grunt.file.readJSON("package.json").devDependencies) {
    if (key !== "grunt" && key.indexOf("grunt") === 0) grunt.loadNpmTasks(key);
}

A possibly even better solution: https://www.npmjs.com/package/load-grunt-tasks


Inbound queue benchmarks

Resolves: #SNO-515
Cumulus companion: https://github.com/Snowfork/cumulus/pull/39

The inbound-queue-benchmark is functional now. I'm busy generating a test fixture with many transactions for a real-life representative receipts trie.

By defining it here in the pallet, you're making it have a hard dependency on the beacon light client pallet.

@vgeddes thanks mentioning the dependency issue, I will have a look at that.

Though you still need to update the bridge-hub-rococo runtime to be aware of the benchmarks:

I updated the bridge-hub-rococo runtime in the Cumulus companion (https://github.com/Snowfork/cumulus/pull/39). Is something else needed there?

I updated the bridge-hub-rococo runtime in the Cumulus companion (Snowfork/cumulus#39). Is something else needed there?

Oops I overlooked the companion PR.

@vgeddes added weights in the inbound queue pallet. 😄

@claravanstaden  Would suggest merge cumulus change together with changes here. Or other prs based on latest cumulus is likely to fail as following:
https://github.com/Snowfork/snowbridge/actions/runs/5586563561/jobs/10210752797?pr=883

@yrong apologies! It looked like I had to merge the cumulus companion to get my check-cumulus-bridgehub GA to pass. Is it OK if it fails until the companion is merged?


Doesnt generate .zip file

Hello I'm using
serious_python 0.7
python 3.11
Windows 11
Trying to run python script for my flutter Windows app
After running command
dart run serious_python:main package --asset assets/myapp.zip app/
Extracting Python distributive from C:\Users\user\AppData\Local\Temp\cpython-3.11.6+20231002-x86_64-pc-windows-msvc-shared-install_only.tar.gz to C:\Users\user\AppData\Local\Temp\hostpython3.11_b621a440
Compiling Python sources at C:\Users\user\AppData\Local\Temp\serious_python_temp1991b3a3

and command stops there I don't get any log even when I run --verbose, When I check temp fold everything seems to be there and requirements are downloaded successfully. Since there is no error log I'm lost and dont know how to proceed. Any help is appreciated

Found the cause .conda and .pycache my app folder was causing this


adobepass.py: add Bright House (Spectrum)

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Bright House Networks was the third company in the creation of Spectrum, but while Charter and Time Warner both had entries added to adobepass.py, Bright House hadn't.  Trying to use the Bright House credentials grandfathered into Charter fail to authenticate.
I've tested with FOX to verify that episodes requiring cable provider login to view work with this change.  I couldn't verify on ABC Go or Discovery Go, as it looks like they're either not set up to handle Adobe Pass or require more than that to access locked videos.

Where should I send the testing account details?  I'm assuming via email?

By mail to any core developer. You should also provide example URLs for testing.

Email sent.

Do you want to be credited as one of the authors of youtube-dl? If so under what name?

Sorry about not responding earlier.  It probably went without saying already, but no, I don't need/want credit.


Using initNoWalletConnection

using an init with no wallet connection needed where ever was possible
adding a check for the load safe test that waits for the "add existing safe" button
giving the initNoWalletConnection a parameter to start from the safe balances or the welcome page

I know, the / are changed again in the coverage file. I thought with something we did with Diogo that would not happen anymore. Well for this time I'll have to ask somebody with linux to change them

There are still tests failing

Thank you Franco!


Add release notes about Groovy script compilation using build cache

We should probably mention that Groovy build scripts are using build cache now. See https://github.com/gradle/gradle/pull/27271#issuecomment-1909857040
Relates to:
https://github.com/gradle/gradle/pull/27271

Added to release notes doc.


Use Lookup.defineClass instead of ClassLoader.defineClass

Context
This is follow-up of #4976 .
Some history:

Previously, we use reflection to invoke protected ClassLoader.defineClass to inject byte[] into a ClassLoader.
After Java9, reflection is forbidden in this case, so we change to Unsafe.defineClass.
Java11 removed the API Unsafe, so we change to Lookup to invoke protected ClassLoader.defineClass.

However as pointed out here:

Using the legacy or new reflection APIs to call the protected ClassLoader.defineClass methods from the wrong context will break once --illegal-access=deny is the default. Better to work out how to use Lookup.defineClass.

The reason why we didn't use Lookup.defineClass is, Lookup.defineClass can only inject class bytes into same package and same class loader of Lookup object. This is a huge limitation for us because in Gradle code base we need to inject class bytes into any class loader without limitation.

Fixed by https://github.com/gradle/gradle/pull/5811

The comment states that Lookup.defineClass injecting a class in the same runtime package as the lookup class is a huge limitation to Gradle.  What classes is Gradle spinning and what package are they intended to place in?


Issue: Removal of IRC hooks

-Removed the IRC source files
-Updated Qt Creator project to not include irc source.
-Removed command line test for -irc

Did you also remove all dependencies from other sources?
example: net.cpp, line 1678 -
threadGroup.create_thread(boost::bind(&TraceThread<void (*)()>, "irc", &ThreadIRCSeed));

Forgot to add the file from the stage. Either way I had forgotten to comment out the #include "irc.h" so was good I had to go back and look. Thank you.

Awesome. Did you compile to check there are no other dependencies included?

Hi. I've added the code changes required for the IRC deprecation at pull request #25
Please close this pull request as it is a duplicate


master: Add changelogs and statistics for zeus-v2021.04 and warrior-v2021.04

I haven't pushed the tags yet, I'll do it after the review-


Support _netrc files for windows

Uv does need a .netrc file on Windows. Unfortunately, pip needs a _netfile on Windows.
Uv should also use a _netfile on Windows.
This Issue is similar to #1405, but it's different.
Source: https://stackoverflow.com/questions/6031214/git-how-to-use-netrc-file-on-windows-to-save-user-and-password

Can you share some more about this? We're just using the rust-netrc crate which should look in your home directory as described?
https://docs.rs/rust-netrc/latest/src/netrc/lib.rs.html#60-71

Ah the title vs the issue confused me, you're saying we need to support _netfile? You can use the NETRC variable to point to an arbitrary file path.

Well, I created an issue at https://github.com/gribouille/netrc/issues/1 as well.


fix pose issue in vision sim

Currently in vision sim, we are using the robot's pose instead of the camera's pose. This is only working because they are currently the same. Once we specify an non-zero difference, it will no longer work.

fixed


updates to address swiftlint lint --strict errors

This is fine, but we should also ignore the directories in the monorepo that contain submodules, so that we don't lint code that isn't in the same repo.

This is fine, but we should also ignore the directories in the monorepo that contain submodules, so that we don't lint code that isn't in the same repo.

I thought it was intentional to take advantage the swiftlint step in the monorepo :D


S105: Exclude variable names containing token but also suffix or prefix

https://github.com/astral-sh/ruff/blob/main/crates/ruff_linter/src/rules/flake8_bandit/helpers.rs
For example, it currently triggers on these 3 definitions:
# The prefix for OAuth tokens
TOKEN_PREFIX_OAUTH = 'indo_'
# The prefix for personal tokens
TOKEN_PREFIX_PERSONAL = 'indp_'
# The prefix for service tokens (not handled by this module)
TOKEN_PREFIX_SERVICE = 'inds_'

However, clearly those are not hardcoded tokens and having suffixes/prefixes for tokens (especially since github's secret scanning becamea thing) is quite common, so maybe it'd be nice to resemble this in the regex that checks for token/password-like variable names instead of just noqa'ing it.

For security-oriented checks, I would personally accept some false positives over risking false negatives. It seems reasonable to me to require users to explicitly # noqa: S105 these, but I'd like to hear others' opinions.

I agree I prefer false positives over negatives in security related rules. I wouldn't be opposed to an exception forPREFIX and SUFFIX unless there's a case someone has where it'd cause a false negative.

A regex setting to whitelist variable names (and dict keys!) for "possible plaintext token/password" would be excellent :)

We are also experiencing some issues like that but from another angle.
We are  writing parsing code for several kind of files and we have functions that are looking or handling some behavior of specific tokens in the file.
In those functions we get FP alerts for each such constant so if we could suppress those by having some prefix to the token it would save us many #noqa.

@IdoPort have you considered disabling the rule for those files? It seems very unlikely that they'll ever contain any real authentication token.

@MichaReiser I didn't know I can disable lints per file, that awesome! Thank!


fix(router): define title property

Attempt to resolve #573 .
Added a property in router. It is used in NavigationInstruction and RouterConfiguration

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Alexander Taran seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Why does promisifyAll modify the actual API instead of returning a new version of it?

Just curious. I love this library because you guys seem to think a lot about the details. I'm sure you have a reason for doing it the way you're doing it, but it feels like a tiny bit of an anti-pattern to modify the original API rather than just return a new promisified version of it. Kind of like how plain old promisify works, by returning a new function you can use. Does it have something to do with non-singleton APIs?
Thanks and keep up the great work!

If you actually have a singleton API, it's trivial to create a promisified copy e.g. Promise.promisifyAll(Object.create(require("fs")))
However, most of the time you have classes and creating your own copies of those doesn't work because the code uses the old classes. And you can't just promisify every instance of a class separately at runtime because it's prohibitively expensive (it takes 50ms to promisify fs module for instance).

That makes sense. Thank you for explaining. I didn't realize you could make easy copies of objects using Object.create. I've always used "extend" libraries to do that, but now I see that I only need those if I actually want to copy and extend the object. Thanks!

The reason Object.create works here to make a copy is because promisifyAll walks prototype chains instead of own properties only.

:+1:


bind K and gO to vscode actions

@asvetliakov This plugin is awesome 🚀
I found out that i could call any vscode command from my vimrc.
Even with arguments!
I think the above should be mentioned in the readme.
nnoremap K  :<C-u>call VSCodeCall('editor.action.showHover')<CR>
nnoremap ? :<C-u>call VSCodeCall('workbench.action.findInFiles', { 'query': expand('<cword>')})<CR>

@aminroosta
From readme 😄

It's possible to call vscode commands from neovim. See VSCodeCall/VSCodeNotify vim functions in vscode-neovim.vim file. VSCodeCall is blocking request, while VSCodeNotify is not

Probably need to move it to own section for better visibility.
Also there are also VSCodeCallRange/VSCodeNotifyRange in case if you need to specify some visual range for vscode command
Generally use VSCodeNotify unless you're really need blocking call


New version: PurpleI2P.i2pd version 2.43.0

Result: Installation Successful

Manifest
Add/Remove Programs

Name
i2pd
i2pd

Version
2.43.0
2.43.0

Publisher
PurpleI2P
PurpleI2P

ProductCode

{621A23E0-3CF4-4BD6-97BC-4835EA5206A2}_is1

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2808
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Fix #10965: wrap access to sheet.cssRules in try/catch for Firefox SecurityError exception

r? @busykai

This is ready for review, and I'd love to get it landed so I can take it downstream in our fork.

1 week overdue on my list. :( will get to it ASAP.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [brackets] Fix #10965: wrap access to sheet.cssRules in try/catch for Firefox SecurityError exception (#10990)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

@humphd, this looks good. gave it some testing, works as expected. wanted to add an automated test, but I am not sure if connectivity can actually be assumed, so it's OK.
please correct these formatting comments though. generally, it's a good idea to use Brackets to hack on Brackets. it would run JSLint properly configured for you.

I'll fix the JSLint issues, but why does Travis not flag them?  Shouldn't your grunt jshint task reveal this stuff if it's important?

@humphd, yes it is a little bit confusing. Travis actually runs jshint on brackets sources, but JSLint is still a

Updated, thanks for the review.  Hopefully this one can land.

Thanks @humphd! And sorry for delay. Merging.


feat: add spain autonomous comunities

Removed old duplicates
Added some spain region

cf issue : #143

Thanks @samuelbelolo!


Fix: test_auto_remount_with_subpath test case

I think we don't need data locality in this test since it doesn't make any impact.
Fixed some timing issues.

longhorn/longhorn#1719

cc @c3y1huang


Database of lighting effects

What are you experiencing an issue with?

[ ] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[x] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[x] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[ ] Visual bug
[ ] Localization issue
[x] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

Would like to see some kind of online database for sharing of lighting effects. Maybe you could add a save to github like feature that Aurora could hook in to and update on startup. Could have a the effects broken up in the category's IE away effects, Game profiles, Animation effects. This way everyone can contribute and make your program even more awesome then it is :)

This sounds a lot like the plugin system that I'm going to be implementing(fully) for v0.7. Does having plugins that you can drop in to add extra layer types, device support and application support sound like what you're looking for? If it is, I'll close this issue as plugins have come up plenty in other issues.

Kind of. Thinking more of a way for others to upload there keyboard profiles to an online storage system so other can download and share profiles. Not really sure how to explain it. But if this could be done from within the program then that would be great.

@Wolfieeewolf In theory that would require a server that can automatically store profiles and retrieve them. Sounds very similar to what the Wootility has with sharing profiles using codes. This isn't really on the table for v0.6.*, but I'll consider it for v0.7.0 as I might be able to use the ModWorkshop server for this.

That is pretty much what I would like added. That is a very awesome keyboard by the way. I'm not sure if there is a way to make it update the GitHub repository when people want to upload a profile. It would only be a very small file and once you get a few profiles uploaded I'm sure there would be more downloads then uploads.  I'd say that goes against GitHub's rules somewhere I'm sure. Anyways, I know you probably don't want to charge people for using your program but perhaps a one off fee to help you maintain a server for uploads could be an idea too.

There wouldn't be any way to do it with a GitHub repo, I wouldn't be charging people for a service like that, as I can't see why anyone would pay for a slight improvement of convenience. Like I said, I can use the ModWorkshop server, so maybe I'll use it for this


Examples: Use new three.js background blur and intensity in examples

Three.js still does not support environment intensity on the scene object. And the background intensity must be conditionally set to the environment intensity only if it's displaying the environment map to match the path tracer.

Setting per material environment intensity should work the same way right ?
scene.traverse((node) => {
			if (node.isMesh && node.material.isMeshStandardMaterial) {
				node.material.envMapIntensity = params.environmentIntensity
				// node.material.needsUpdate=true
			}
		})


Unable to initiate Device authentication as described in the documentation-- Even after Incorporating the recent changes in the documentation

Hi Team,
I'm trying to follow the documentation provided here: https://github.com/Azure/go-autorest/tree/master/autorest/adal
for programming device authorization. I'm unable to instantiate deviceCode as described in the documentation.
The following is my code:
oauthClient := &http.Client{}

oauthConfig, err := adal.NewOAuthConfig(LoginURL, Tenant)
deviceCode, err := adal.InitiateDeviceAuth(
	oauthClient,
	*oauthConfig,
	ClientID,
	Resource)
if err != nil {
	fmt.Errorf("Failed to start device auth flow: %s", err)
}

I get the following runtime error:
panic: runtime error: invalid memory address or nil pointer dereference

The error returned from the InitiateDeviceAuth() call is as follows:
err
<error>
data:<*errors.errorString>(0xc0002e6030)
<errors.errorString>
s:"autorest/adal/devicetoken: Error occurred while handling respons...+52 more"

I can't repro this.  Can you please provide the following.

Values for LoginURL and Resource
Full stack where the panic happens
Complete error message returned from InitiateDeviceAuth()

This was an error in configuration. Thank you for helping with this and the earlier issue as well.


Remove 1.8 and 1.9 specific code

This is what I found with `git grep -n '1.8':

[ ] https://github.com/opscode/chef/blob/0760983e7aff5bafe5b2c8c43891fbb3d0fd3ed3/lib/chef/chef_fs/parallelizer/parallel_enumerable.rb#L121-L124
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/mixin/securable.rb#L115-L119
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/mixin/template.rb#L26-L36
[x] https://github.com/opscode/chef/blob/master/lib/chef/monkey_patches/numeric.rb#L1-L15
[x] https://github.com/opscode/chef/blob/master/lib/chef/monkey_patches/regexp.rb#L22-L34
[x] https://github.com/opscode/chef/blob/master/lib/chef/monkey_patches/string.rb#L19-L49
[x] https://github.com/opscode/chef/blob/master/lib/chef/monkey_patches/tempfile.rb
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/node/immutable_collections.rb#L78-L83
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/resource/lwrp_base.rb#L76-L87
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/resource/template.rb#L105-L107
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/run_context.rb#L197-L208
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/run_lock.rb#L97-L102
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/search/query.rb#L135-L136
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/lib/chef/util/path_helper.rb#L104-L106
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/support/platform_helpers.rb#L26-L28
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/chef_fs/parallelizer.rb#L223-L225
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/node/attribute_spec.rb#L1028-L1036
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/policy_builder/policyfile_spec.rb#L344-L348
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/provider/mount/solaris_spec.rb#L97
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/provider/remote_file/cache_control_data_spec.rb#L161-L162
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/provider/remote_file/cache_control_data_spec.rb#L161-L162
[ ] https://github.com/opscode/chef/blob/45167319996c4fc7e65351a459ae2ee8a5960b31/spec/unit/provider/remote_file/cache_control_data_spec.rb#L161-L162

a lot of this got done.  there's probably more.  wherever it is found it should be killed.  closing this issue since nobody is working on it directly.


Update enable compat

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:14: invalid_url: ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz : No connection adapters were found for 'ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz'
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:14: missing_hash: The recipe is missing a sha256 checksum for a source file
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:24: patch_must_be_in_build: patch must be in build when source/patches is set.
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:43: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: APL-1.0
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:48: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:48: missing_description: The recipe is missing a description
WARNING: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:48: missing_license_url: The recipe is missing a license_url
ERROR: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:48: missing_doc_url: The recipe is missing a doc_url
WARNING: /tmp/abs_464yoh8pz0/clone/recipe/meta.yaml:48: missing_doc_source_url: The recipe is missing a doc_source_url
Errors were found

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_a8xfyygcwl/clone/recipe/meta.yaml:14: invalid_url: ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz : No connection adapters were found for 'ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz'
ERROR: /tmp/abs_a8xfyygcwl/clone/recipe/meta.yaml:14: missing_hash: The recipe is missing a sha256 checksum for a source file
WARNING: /tmp/abs_a8xfyygcwl/clone/recipe/meta.yaml:51: missing_description: The recipe is missing a description
WARNING: /tmp/abs_a8xfyygcwl/clone/recipe/meta.yaml:51: missing_doc_source_url: The recipe is missing a doc_source_url
WARNING: /tmp/abs_a8xfyygcwl/clone/recipe/meta.yaml:51: missing_license_url: The recipe is missing a license_url
Errors were found

Linter check failed, with the following output:
The following problems have been found:
ERROR: /tmp/abs_e1l73c2_y5/clone/recipe/meta.yaml:14: invalid_url: ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz : No connection adapters were found for 'ftp://ftp.gnu.org/gnu/gdbm/gdbm-1.18.tar.gz'
ERROR: /tmp/abs_e1l73c2_y5/clone/recipe/meta.yaml:14: missing_hash: The recipe is missing a sha256 checksum for a source file
WARNING: /tmp/abs_e1l73c2_y5/clone/recipe/meta.yaml:51: missing_doc_source_url: The recipe is missing a doc_source_url
WARNING: /tmp/abs_e1l73c2_y5/clone/recipe/meta.yaml:51: missing_license_url: The recipe is missing a license_url
WARNING: /tmp/abs_e1l73c2_y5/clone/recipe/meta.yaml:51: missing_description: The recipe is missing a description
Errors were found

Yes, it would be nice to add these suggestions from Maxwell to the about. Aside from that it seems good, and I can approve it when they are in.

added suggests by Maxwell, thanks


trying to compile but got an error

Describe your problem here.
got this while compiling
Called from ? line 1
Called from CommandLineTools.hx line 1895
Called from CommandLineTools.hx line 22
Called from a C function
Called from CommandLineTools.hx line 125
Called from CommandLineTools.hx line 1703
Called from lime/tools/ProjectXMLParser.hx line 21
Called from a C function
Called from lime/tools/ProjectXMLParser.hx line 50
Called from lime/tools/ProjectXMLParser.hx line 2043
Called from lime/tools/ProjectXMLParser.hx line 1528
Called from lime/tools/ProjectXMLParser.hx line 608
Called from a C function
Called from lime/tools/ProjectXMLParser.hx line 719
Called from a C function
Called from lime/tools/ProjectXMLParser.hx line 719
Called from a C function
Called from lime/tools/ProjectXMLParser.hx line 709
Called from /Users/runner/hostedtoolcache/haxe/4.2.1/x64/std/neko/_std/sys/FileSystem.hx line 69
Called from /Users/runner/hostedtoolcache/haxe/4.2.1/x64/std/neko/_std/sys/FileSystem.hx line 60
Uncaught exception - std@sys_file_type
any help?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I edited a lot of stuff for stages and notes also added a new file called Shaders.hx which is taken from dave and bambi mod.

Can't help with Shader stuff specially considering it is taken from another mod.


[OWL-ViT] Bounding box all around the image

Hi,
I was testing out the one shot feature of the model in Google Colab and i've noticed that there's always an extra bounding box all around the image:

The same bounding box appears even in the minimal example in Google Colab:

Is there a way to filter it out?
Thank you

How about simply removing boxes based on area? If the area relative to the image is, e.g., > 0.95, remove the box?


Please make it explicit that 32bits are not supported

How I wasted 2h of my life:
I started tutorial by going to the page: https://github.com/Gallopsled/pwntools-tutorial/blob/master/installing.md
I had a VM with 32-bit Ubuntu 16 LTS, I used it to solve challenges from OverTheWire (they are mostly 32 bit).
I started with:
pip install --upgrade git+https://github.com/Gallopsled/pwntools.git

and it failed, looks like I need some other libraries, ok no problem:
sudo apt install libffi-dev
sudo apt install libssl-dev

But then pip cannot install cryptography module, it was failing with strange compile time errors.
OK probably old OpenSSL version (and BTW installing Python packages is getting more terrible than compiling C code). I compiled a new version from the sources: https://cloudwafer.com/blog/installing-openssl-on-ubuntu-16-04-18-04/ Yay!
Finally managed to compile the rest:
sudo apt install libsodium-dev # one more cr**p to install

pip install --global-option=build_ext --global-option="-L/usr/local/ssl/lib" --upgrade git+https://github.com/Gallopsled/pwntools.git

And of course one more thing was missing:
pip install python-dateutil

And what I get for all this effort:
$ python -c 'from pwn import *'
[!] Pwntools does not support 32-bit Python.  Use a 64-bit release.

So please, please add a big huge bolded text saying that 32-bits are not supported....

We tried to put it as bold as we could!  Maybe we can move the warning into setup.py so that it's shown when attempting to install Pwntools rather than after it's installed (which, in hindsight, seems obvious).

We tried to put it as bold as we could!  Maybe we can move the warning into setup.py so that it's shown when attempting to install Pwntools rather than after it's installed (which, in hindsight, seems obvious).

I think too that putting it in setup.py is a great idea!

I think too that putting it in setup.py is a great idea!

@marcin-chwedczuk You should submit a Pull Request ;)

@marcin-chwedczuk You should submit a Pull Request ;)


MM-17612 Fix fetch periodic status updates when re-connecting

Summary
The periodic status updates was first initialize when connecting the WS for the first time but was not being triggered on reconnect. Also the stopPeriodicStatusUpdates was being called on every reconnect attempt as well as closeWebSocket. Both issues are addressed.
Ticket Link
https://mattermost.atlassian.net/browse/MM-17612

@enahum WHy do we bother stopping the periodic update?. Other than logging out i don't see a reason we should be doing it.

@sudheerDev there is no point of having an interval running if it does not accomplish anything

@sudheerDev there is no point of having an interval running if it does not accomplish anything

We need to periodically poll to get the new statuses.
I was asking about why we stop making them on websocket disconnect etc. Do you know why we cannot keep them running?

@enahum The reason why i was asking is because this has a lot of complexity in terms of when we call and when we stop calling. Which to me seems something we don't need to do if there are hardly any downsides.
Never mind with the PR as of now. We can create a tech debt ticket if needed

@sudheerDev

Do you know why we cannot keep them running?
cause when the app is send to the background the JS context stops execution and will queue them all to get executed when the context is regained

Do you know why we cannot keep them running?
cause when the app is send to the background the JS context stops execution and will queue them all to get executed when the context is regained

@enahum How does one verify this? When I test this out with this in Redux:
diff --git a/src/actions/users.js b/src/actions/users.js
index eb9161e..ead10f9 100644
--- a/src/actions/users.js
+++ b/src/actions/users.js
@@ -905,6 +905,7 @@ export function startPeriodicStatusUpdates(): ActionFunc {

         statusIntervalId = setInterval(
             () => {
+                console.count('statusInterval');
                 const {statuses} = getState().entities.users;

                 if (!statuses) {
diff --git a/src/constants/general.js b/src/constants/general.js
index 7397709..86b3a4b 100644
--- a/src/constants/general.js
+++ b/src/constants/general.js
@@ -15,7 +15,7 @@ export default {
     TEAMS_CHUNK_SIZE: 50,
     JOBS_CHUNK_SIZE: 50,
     SEARCH_TIMEOUT_MILLISECONDS: 100,
-    STATUS_INTERVAL: 60000,
+    STATUS_INTERVAL: 10000,

     AUTOCOMPLETE_LIMIT_DEFAULT: 25,
     AUTOCOMPLETE_SPLIT_CHARACTERS: ['.', '-', '_'],

... it doesn't seem to queue any extra invocations.


feat: remove key prefix and key range types

These types bloat the API surface without adding much value. Off you go!

Codecov Report

Merging #19 (a2bf39a) into master (4fa5e17) will increase coverage by 1.60%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   58.00%   59.61%   +1.60%
==========================================
  Files          20       18       -2
  Lines        2129     1862     -267
==========================================
- Hits         1235     1110     -125
+ Misses        844      706     -138
+ Partials       50       46       -4

Impacted Files
Coverage Δ

internal/codegen/databasecodegen/database.go
100.00% <ø> (ø)

internal/examples/musicdb/database_gen.go
24.76% <25.00%> (+3.80%)
:arrow_up:

...egen/databasecodegen/interleavedreadtransaction.go
97.24% <100.00%> (ø)

internal/codegen/databasecodegen/interleavedrow.go
98.00% <100.00%> (+1.80%)
:arrow_up:

Codecov Report

Merging #19 (a2bf39a) into master (4fa5e17) will increase coverage by 1.60%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   58.00%   59.61%   +1.60%
==========================================
  Files          20       18       -2
  Lines        2129     1862     -267
==========================================
- Hits         1235     1110     -125
+ Misses        844      706     -138
+ Partials       50       46       -4

Impacted Files
Coverage Δ

internal/codegen/databasecodegen/database.go
100.00% <ø> (ø)

internal/examples/musicdb/database_gen.go
24.76% <25.00%> (+3.80%)
:arrow_up:

...egen/databasecodegen/interleavedreadtransaction.go
97.24% <100.00%> (ø)

internal/codegen/databasecodegen/interleavedrow.go
98.00% <100.00%> (+1.80%)
:arrow_up:


Unable to define PascalCase column names for relations

Not sure if I've missed something in the docs but I'm having trouble setting the to and from tables for relationships to use PascalCase column names e.g. Game.CardID, it just refuses to enter the column into the insert statement. I've managed to get around this by using camelCase snake_case but it took hours to figure out. You might want to make it more obvious in the documentation that this format is required.

If you don't have any case conversion set up for your models you need to give every column in the database format. If your columns are pascal cased in the db, then use pascal case in relation mappings. If they are snake cased, then use snake case. Objection doesn't require you to use any specific casing. You have probably messed something else up. If you post your migrations and models here I may be able to help.
The next version will have a better snake case conversion system. It's already in the master, but not yet released.

I don't have case conversion setup.
My models couldn't really simpler.
export default class CardModel extends Model {
  static get tableName() {
    return 'Card'
  }
  static get relationMappings() {
    return {
      game: {
        relation: Model.BelongsToOneRelation,
        modelClass: GameModel,
        join: {
          from: 'Card.GameID',
          to: 'Game.ID'
        }
      }
    }
  }
}
export default class GameModel extends Model {
  static get tableName() {
    return 'Game'
  }
  static get relationMappings() {
    return {
      cards: {
        relation: Model.HasManyRelation,
        modelClass: CardModel,
        join: {
          from: 'Game.ID',
          to: 'Card.GameID'
        }
      }
    }
  }
}

The database columns are in pascal case.
If I keep the relations in pascal case and attempt to perform a graph insert
GameModel.query().graphInsert({
  cards: [{ Text: 'card a' }, { Text: 'card b' }]
})

The ID columns of the cards are skipped. If I change the case of the relation to camel case then they do insert but performing a select with eager('cards') fails (it was performing two queries with a 'where 1 = ?' on the cards, very strange). The only remedy was to change both database and relation names to snake case.

The default id column is lowercase id. You probably need to add this to your models:
static get idColumn() {
  return 'ID';
}

Was that it?

Looks like that fixed it. Thanks.


MA-pilot-Hockman

Project short name:
MA-pilot-Hockman
Primary Wrangler:
@arschat
Secondary Wrangler:

Associated files:
document

Google Drive:

Key Events

[ ] Send initial instructions about contributing, including questionnaire and T&Cs
[ ] Receive project questionnaire
[ ] Assign the Primary Wrangler
[ ] Receive T&C form filled out by at least 1 representative of group/project
[ ] Send custom metadata template spreadsheet, how-to guide and directions for submitting data, matrix and cell annotation files.
[ ] Receive spreadsheet (to a sufficient standard to submit dataset)
[ ] Receive data, matrix and cell annotation files (to a sufficient standard to submit dataset)
[ ] Curate metadata spreadsheet with ontologies
[ ] Upload sheet to validate metadata
[ ] Run graph validator & Check linking
[ ] Validate data files
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Get final approval of submission from the contributor including release date, metadata spreadsheet and data files
[ ] Submit to production and inform ingest-devs the project is ready for archiving
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Please track the below as well as the key events:

Track date first spreadsheet received and final spreadsheet sent by editing ticket to include
date next to event.
Track spreadsheet iterations by placing asterisks next to receive spreadsheet event.
Track any metadata issues/tickets made for dataset with a bulleted list of links under
received spreadsheet event. Links should be to the ticket in the metadata repo.

Had a call with Dorit Hockman today, I will share with the contributor:

[ ] template for spatial transcriptomics (pre-filled for 10x 3' v3.1 and 10x Visium using this)
[ ] hca-util link and guidelines
[ ] hca site metadata dictionary

Project title has/will change since DCA was signed. Dorit asked if she can use new title in submission. John R said that

I think that's fine since the Project ID will remain the same.

Have to send to Jonathan to request DUOS ID. Need to decide if we will wait for new project title for DUOS entry as well, or use the existing one (the one on DCA).

[X] request DUOS-ID

[ ] received DUOS-ID

[x] contributor uploads files and spreadsheet
[x] create submission
[x] spreadsheet reviewed
[ ] metadata valid

2 files were not validating, and asked for re-upload
after re-upload, those 2 files are valid but 4 other files are stuck in validating

[ ] graph valid
[ ] export

We will have to decide if we will request it to be public in the data library or wait until data is on portal.
Two points I wanted to write down after my first MA dataset:

How do we want contributor to check our review? Contributor's account can't download data with hca-util. For now I did a zoom call and shared the workspaces screen.
Before submitting, we should remove the metadata spreadsheet files from the s3://org-hca-data-archive-upload-prod/ area (not hca-util-upload-area!) and from ingest-ui (unknown files are in draft state)

verified in browser

Per contributor's request we will update the analysis file included.

Submission used the previous ontology version and after update all files have wrong description ontologies.

[ ] Removed rds file: s3://org-hca-data-archive-upload-prod/f6bf854d-6ce9-4cbf-bb39-b6f1c5d5dd0c/Steyn_etal_seurat_object.rds
[ ] Re-sync'd the hca-util area to upload the updated file
[ ] Update all file's ontology term from data: to EDAM:
[ ] Re-trigger validation for updated Steyn_etal_seurat_object.rds

As per contributor's request, we will update also the project metadata to include the updated publication, remove previous biorxiv, and update project description with the publication abstract.
Since the export was already done, we will export only metadata now.


Fixed #XXX: Added support for RSS feed stylesheets (contrib.syndication)

@moan0son do you have an opinion on this before I open an official ticket and PR?


Question: Equivalent BirdNET Analyzer Model Version to App

Thank you for developing and maintaining an excellent application! I'm using the latest BirdNet App iOS in the field to quickly check bird calls and then analyze our entire recordings after returning to the lab using BirdNet-Analyzer.  So, I'm interested in which version of BirdNet-Analyzer is equivalent to the App model.  Thank you in advance.

The app actually uses two different models depending on wether you are in north-america/europe or anywhere else.
When not in NA/Europe the model used is the 3K-species App-model from the model-history is used (https://github.com/kahst/BirdNET-Analyzer/blob/main/checkpoints/README.md).
For NA/Europe an older implementation is used based on the original birdnet-repo (https://github.com/kahst/BirdNET), but this is not compatible with the analyzer.


[ACI-1491] Debug print when cache download retry happens

For better debugging/observability.

No need for this - fixing how the retriable client is passed to got Downloader in a separate PR.


ESP32 ERROR

Hi
When connecting the esp32 module internally in a ps2 fat it turns on but does not link the controls.
Try externally and there if you link them
try the following diagram:

What can be?

You might have the same issue that I had, my ESP thought the PS2 was a PC-FX.  I was able to solve it by using the 3V3 pin vs the 5V. Apparently some PS2 have issues with either the 3V3 or the 5V/8V.
You can see my discussion here:
https://github.com/darthcloud/BlueRetro/discussions/234
Basically disconnect the 5V on the ESP/PS2 USB port and connect Pin 5 on the controller port to the 3V3 pin on the ESP32

.


Add download data for May 2019

1,856,601 downloads for the organization (from 44,086 unique IPs), up 49.8%  from April. Still only a few hundred 2.13 downloads.

Codecov Report

Merging #1144 into master will increase coverage by 0.46%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1144      +/-   ##
==========================================
+ Coverage    85.6%   86.06%   +0.46%
==========================================
  Files          87       87
  Lines        2821     2821
  Branches      118      118
==========================================
+ Hits         2415     2428      +13
+ Misses        406      393      -13

Impacted Files
Coverage Δ

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
90.36% <0%> (+0.6%)
:arrow_up:

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
100% <0%> (+8.33%)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
97.5% <0%> (+8.75%)
:arrow_up:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.28%)
:arrow_up:

...e/generic/extras/decoding/EnumerationDecoder.scala
100% <0%> (+14.28%)
:arrow_up:

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 68fca69...b331d83. Read the comment docs.


cleanup and expand architecture page

These changes include

Adding Components breakdown to the EKS Anywhere Architecture page
Cleaning up the descriptions of standalone and management cluster deployment architectures

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.19


Create the secrets bucket if left blank

This creates a secrets bucket if none is provided. This bucket will be left in place when the stack is deleted.

I'm not 100% sure this is useful. Thoughts @toolmantim?

I think it is! I think the "Getting Started" guide will be improved, because it's just "Add a key to this bucket and run your GitHub project's build" instead of "Create a bucket, edit the stack, etc etc"

With the caveat that you need to explain how to figure out the bucket name
On Mon, 16 Jan 2017 at 11:47 am, Tim Lucas<EMAIL_ADDRESS>wrote:

@toolmantim approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/buildkite/elastic-ci-stack-for-aws/pull/218#pullrequestreview-16728573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA9jhrOgfo_hPk-Vv41Yz15_6hJJyHMks5rSr4kgaJpZM4Le6ax
.

Yeah… it's still better I think. "Find your stack’s secrets bucket, add your key, run a build" vs "Create a secrets bucket, add the key, edit your stack, update your agents, run a build"

Yup, true.
On Mon, 16 Jan 2017 at 11:58 am, Tim Lucas<EMAIL_ADDRESS>wrote:

Yeah… it's still better I think. "Find your stack’s secrets bucket, add
your key, run a build" vs "Create a secrets bucket, add the key, edit your
stack, update your agents, run a build"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/buildkite/elastic-ci-stack-for-aws/pull/218#issuecomment-272752122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA9jikPhQ_zRt7h3mKYlGlAwfYeShzfks5rSsCVgaJpZM4Le6ax
.

Would it be going to far to have a policy on the bucket that prevented non-encrypted contents?
{
   "Version":"2012-10-17",
   "Id":"PutObjPolicy",
   "Statement":[{
         "Sid":"DenyUnEncryptedObjectUploads",
         "Effect":"Deny",
         "Principal":"*",
         "Action":"s3:PutObject",
         "Resource":"arn:aws:s3:::YourBucket/*",
         "Condition":{
            "StringNotEquals":{
               "s3:x-amz-server-side-encryption":"aws:kms"
            }
         }
      }
   ]
}

Oh, if it's not standard I think that'd be a very nice option!
If it is standard, can you upload it as encrypted via the AWS web console? Or cli only?

Thoughts on getting this in as 2.1.0 @toolmantim? It should be non-breaking.


feat: add move description

add move description on pokemons page, just like abilities

Hey I am interested in working on this.
Can you clarify for me where to find

Hey I am interested in working on this. Can you clarify for me what is meant by abilities description that should be the blueprint for the moves description?

Hey, sorry, I just did that last night, if you want you can check the last commit to see how I did it. It's very nice of you to come here and try to help though. I'll mark this issue as closed now.


Ability to clone variable groups

It would be cool to be able to clone variable groups. I want to create a group for my dev environment, clone it for my production environment so it has the exact same variables, and then update the variable values.

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

ID: 4827aa99-ae62-bd63-6f2f-a87a4065ed27
Version Independent ID: c9e461ee-7f42-3503-9460-18fa3a807bbb
Content: Variable groups for VSTS and TFS
Content Source: docs/build-release/concepts/library/variable-groups.md
Service: unspecified
GitHub Login: @erickson-doug
Microsoft Alias: ahomer

I echo jeffhebrank, even i am looking for this change.

Thank you for your feedback. I have passed the comment on to the product development team and will advise you of their response.

The product team have asked that you post the request on UserVoice (https://visualstudio.uservoice.com/forums/330519-team-services?category_id=145269). It seems a reasonable suggestion, and can be implemented if there are sufficient votes. Thank you!

Exactly. It's not professional to define from the blank list Variable Groups on each projects. Is non sense.

Import and Export Variable Groups is must-be feature. I think that Variable Groups should be defined outside the particular project.

Thank you. I will pass on your comments.

Octopus Deploy has this feature since a long time. As @TheBeardedLlama said, this feature should be standard feature of this system and we should not have to request for it. Think from the user's perspective. This feature, exporting and  ability to specify multiple environment scopes to each variable - should have been included by now.

Badly needed this feature as well. So went ahead and created a small utility to copy variable groups. Please feel free to use: https://github.com/AssureCare/vststools

Thanks. I'll try this weekend.
2018-05-18 18:39 GMT+02:00 Kiryl Bokach<EMAIL_ADDRESS>
Badly needed this feature as well. So went ahead and created a small
utility to copy variable groups. Please feel free to use:
https://github.com/AssureCare/vststools
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/vsts-docs/issues/447#issuecomment-390264114,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aff0DGITy33rXaRt17TlIjLqE5K8MMa8ks5tzvlcgaJpZM4TCEn0
.

cheers @kirylbokach
I'll give it a go at some point and give you feedback

Clone feature should be there for variable group.

The product team have asked that you post the request on UserVoice (https://visualstudio.uservoice.com/forums/330519-team-services?category_id=145269). It seems a reasonable suggestion, and can be implemented if there are sufficient votes. Thank you!

Really! keep moving the issue from one place to another, then to another, rather than solving the issue!!