Name clash with private methods

I solved a weird bug this week in an application. Draper was working correctly, but this application hit what could be a gotcha in delegate_all. I open this issue to explain it, and to ask if you'd like me to volunteer a doc patch. Let me explain.
The situation was:
class JobDecorator < Draper::Decorator
  delegate :final, to: :location, prefix: true
  delegate_all

  private
  def location
    (model.location || Location.new).decorate
  end
end

class LocationDecorator < Draper::Decorator
  def link_to_google_maps
    ...
  end
end

The symptom was that you deployed, everything was fine, and after some minutes
job.decorate.location_final.link_to_google_maps

crashed with a NoMethodError, saying Location objects do not respond to link_to_google_maps (I am making the names up). How on earth is job.decorate.location_final returning all of a sudden a Location instead of a LocationDecorator?
Problem is, Job instances have location in their public interface. When the app launches, JobDecorator has a private method called location, but as soon as someone invokes job.decorate.location, the method_missing is triggered because location is not a public method of the decorator, so it is (correctly) forwarded and redefined/cached in the decorator class. The private method is gone.
That was the bug, but once understood, the problem is in the implementation of the decorator, because delegate_all effectively (lazily) injects the public interface of Job into JobDecorator, so this is as if you were writing by hand both a public and a private method with the same name. And the last one wins.
My solution has been to adopt the convention of naming private methods with a leading underscore. That way you are forward-compatible with any public method in Job, since conventionally Ruby programmers do not put leading underscores in public methods.
Do you think it would be good to document this gotcha? Something like "Beware delegate_all injects the current and future public APIs of the model in here, make sure the name of any private methods in the decorator do not clash. A leading underscore is a forward-compatible convention to ensure that."

Alternatively, if delegating to a method that clashes with an existing one in the decorator, perhaps you could raise. But gauging whether that is a good idea is more difficult for me, would need to have more perspective about usage of Draper out there. You are in a better position to choose.

I think it'd make sense to raise an error.

Fixed by: https://github.com/drapergem/draper/pull/849


fix(gptimer): remove incorrect param from example (IDFGH-13420)

Fixes call to gptimer_start in one of the examples which includes extra second parameter &alarm_config, when the function takes only one.

This is a resubmit of accidentally closed PR #14310

FYI, the fix is already merged in https://github.com/espressif/esp-idf/commit/08fa1131ca13860fb4a3eb340fd37f76a1c651a6


Pull Request (Milestone 1) - Data Model and ERD

Data Modelling (Relational Entity Modelling)
Modelled core application internals using an Entity-relationship-diagram.
Interacting object models are:

[ ] Users,
[ ] Posts,
[ ] Comments,
[ ] Likes (activerecord polymorphic)
[ ] Friendships

Kindly review. Thanks in advance

Thank you @bellom  for that pointer. (I must have skipped that during the drawing)
I will update the changes accordingly. Thanks for your review


skip publish of confluence pages related on submodules declared in a maven multimodule project

I have a wiki site project under a maven multimodule project (packaging pom). So, when I publish documentation, also meta-inf pages related the maven submodules artifacts are deployed on confluence. Can I avoid to publish this pages? The version used of confluence plugin is the last (5.0). Thanks Bartolo :)

Let me understand, do you use site descriptor or not?

Yes, with a site.xml

so such meta-inf pages are published also if them are not included in site.xml ?

Yes, the page are not declared in site.xml. Now I explain you my case. I have a maven multimodule project (artifactId "custom-parent") with version 1.0-SNAPSHOT, and two submodules with artifactId "event" and "codegen". In my site I have two pages: "modelling" and "generation". Now when I publish with the confluence plugin, there are published: "modelling", "generation", "event-1.0-SNAPSHOT" and "codegen-1.0-SNAPSHOT" . The last two are paged related to jar meta inf of submodules (dependencies name, project name, pom properties etc) . This is the problem. I don't want the last two meta inf pages

Currently the only ways to do this are:

use -N,--non-recursive  Do not recurse into sub-projects maven options
create a specific pom (without modules) for generate documentation

I'll work on support of skip support


https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/permissions

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
there is a permission called search in manifest.json which is listed but does not appear in the browser compatibility table.
i don't have details on version support so no PR this time.

@chrisdavidmills Do you have any advice about how to track down the release number for this feature? I have checked the release notes and it's not mentioned there. I have looked at the source code, but can't immediately see where a definition of this parameter value might live.

@rebloor I'd probably just ask one of the engineers responsible for this stuff; it can be really tricky and be a massive rabbit hole to find such data. Someone like Rob Wu or Shane C, maybe?

Confirmed with Rob Wu that this was introduced in version 63.

See https://github.com/mdn/browser-compat-data/pull/4976


Configure model checkpointing

What I am changing

Configuring how model weights are saved to checkpoint (*.ckpt) files during training

How I did it

Using Lightning's ModelCheckpoint callback with the following parameters:

auto_insert_metric_name=False
filename="vit_epoch-{epoch:02d}_train_loss-{train/loss:.2f}"
monitor="train/loss"
mode="min"
save_last=True
save_top_k=3
save_weights_only=True

Added the AsyncCheckpointIO plugin to reduce the gap in time between epochs when a checkpoint is saved

How you can test it

Run python trainer.py fit --trainer.max_epochs=20 locally
Extra configuration options can be found using python trainer.py fit --trainer.callbacks.help=ModelCheckpoint

Related Issues

References:

https://github.com/Lightning-AI/lightning/issues/11561

Gonna merge this in directly since it's a relatively small change, but useful in terms of showing how callbacks and plugins can be added to the LightningCLI in trainer.py.


Core review issues

Problems found during code review:

I do not think that the "com.oxygenxml.docbook.checker.DocBookCheckerOxygen" should exist as it does not bring any extra logic to the base class.
com.oxygenxml.docbook.checker.OxygenInteractor.setButtonsEnable(boolean) The "state" param is not documented
OxygenSourceDescription - use "enum" instead of constant strings.
Spell check problems with field "com.oxygenxml.docbook.checker.OxygenSourceDescription.parrentFrame" and getter and setter methods. Maybe this particular field should not be in this "OxygenSourceDescription" class. Maybe the "OxygenInteractor" could instead contain a getter for the parent frame.
com.oxygenxml.docbook.checker.OxygenInteractor.setButtonsEnable(boolean) Maybe you could rename the method to "setOperationInProgress(....)" and you could enable/disable the actions on it's callback like you do now.
Maybe also rename OxygenInteractor to "ApplicationInteractor". Maybe same for "OxygenSourceDescription"
ValidationWorker - constructor is not documented, neither are all other methods
Maybe rename "com.oxygenxml.docbook.checker.ValidationWorkerInteractor.isSetIsCancelled()" to "isCancelled"
CheckBoxTree - variables with friendly access but used only in current class, you can make them private. Add more javadoc to variables, methods, document each parameter used in method. Same for CheckBoxTreeModel, CheckedNode. Also in CheckedNode you can make variables final to precisely avoid them being modified by any other means than the constructor. Also in ConditionValue no Javadoc.
ConditionValue, the equals(Object) should compare the internal values of both objects instead of relying on the hashcode(). Because it's possible that for two different strings the hashcode() returns the same integer.
Maybe rename MapConvertor->ConditionValueUtil because it has a very generic name. Not all params are documented in Javadoc

com.oxygenxml.docbook.checker.gui.ConfigureConditionsDialog.ConfiguteConditionPanel - Never start field name with upper case letter.

Code in "ConfigureConditionsDialog" like "for (TreePath tp : paths)" should probably be moved in a method in the checkbox tree as it is the one which knows the hierarchy that it displays.

DocBookCheckerDialog, make field "linkToGitHub" static final, rename it to all uppercase as it is a constant, pass the "Translator" interface to the DocBookCheckerDialog on the constructor instead of creating an implementation for it in the dialog . More Javadoc to methods.

find all places in code where "e.printStackTrace()" is used, decide if errors should be presented or not to end user. If not, use Apache log4j (should already be in classpath) to log the error on debug. Something like:
  private static final org.apache.log4j.Logger logger = Logger.getLogger(DocBookCheckerDialog.class);

and then use it like:
   		logger.debug(e.getMessage(), e);

You can read a little about the Apache Log4j logging facilities, it allows you to redirect the logging to an output file or to log only based on a certain category. We use it internally in Oxygen.

ProfilingPanel and similar places, a best practice try to instantiate the Swing components like buttons for example on the constructor and not when the field is declared.
For example instead of:
     private JRadioButton configProfilingCondSetRBtn = new JRadioButton();
    .......................
  configProfilingCondSetRBtn.setText(translator.getTranslation(Tags.CONFIG_CONDITIONS_SET));
  configProfilingCondSetRBtn.setSelected(true);

do this:
       private JRadioButton configProfilingCondSetRBtn;
        ........
	configProfilingCondSetRBtn = new JRadioButton(translator.getTranslation(Tags.CONFIG_CONDITIONS_SET));
	configProfilingCondSetRBtn.setSelected(true);

You also have various setter methods which do not seem to be used.
Also as a best practice, first configure the Swing component and then add it to the parent component.
So instead of this:
   		btnsPanel.add(remvBtn);
	remvBtn.setEnabled(false);
	remvBtn.setText(translator.getTranslation(Tags.REMOVE_TABLE));

do this:
            remvBtn = new JButton(translator.getTranslation(Tags.REMOVE_TABLE));
	remvBtn.setEnabled(false);
     	btnsPanel.add(remvBtn);

Avoid dependency on the "com.google.common.base" Guava library. Create an utility method for what you need. It started being shipped with Oxygen 18.1 I think so with this dependency the plugin would not work with an older Oxygen version.

Make sure every class, field and method has Javadoc, and the Javadoc also explains the params and return type.

I think the "com.oxygenxml.docbook.checker.parser.DocumentDetails" should not have methods which determine how the GUI progress will be advanced for each stage, methods like getImageProgress(). Maybe move those methods to an utility class which takes a DocumentDetails object as a parameter and returns such info about it.

Avoid code which makes computations on every iteration like:
for (int i = 0; i < modelTable.getRowCount(); i++) {

instead take the row count in a variable before the for and then use it.

Avoid instantiating boolean objects like "new java.lang.Boolean.Boolean(boolean)" in memory, instead use Boolean.TRUE and Boolean.FALSE.

The field com.oxygenxml.docbook.checker.parser.ElementLinkDetailsDetector.namespace is very generic by name, needs to be defined as a constant (static, final, all upper case letters like for example "XLINK_NAMESPACE"), needs to have Javadoc

Instead of calls like:
     			resultLinkDetails.getParaIds().add(newId);

I would prefer dedicated methods for adding an ID like:
         resultLinkDetails.addID(newId);

We usually avoid taking a list of items from an object and then changing it. So the getter which returns the list is mostly used to read and iterate the list, not to make changes to it.

The field "com.oxygenxml.docbook.checker.parser.Link.exception" should probably not be there. I would see the "Link" object only keep the info obtained from the XML document. Once you determine a cause for a broken link you could keep it maybe separately in a Map<Link, Exception> and not update the link object. By the way, if a document has two external links to the same web site, do you check the link once or twice?

Method "com.oxygenxml.docbook.checker.plugin.ProjectManagerEditor.getAllXmlUrlFiles(File, List)". To create an URL from a File instead of prepending the "file:" prefix you should use the:
ro.sync.util.URLUtil.correct(file)

as it handles more cases.  Also instead of looking only for files with the "xml" extension you could try to validate as Docbook any file which is not binary ("ro.sync.exml.workspace.api.util.UtilAccess.isUnhandledBinaryResourceURL(URL)").

The method "folder.listFiles();" can return null in case a folder does not have read access rights for the current user, this should be guarded in the code.

Fixed in: cb32c7bab4632539c6c593481e586f7383ea59d5


How to setup linter & linter-php on windows 10

I need help with lint configuration... I can't get it work.... I installed all lint packages and still atom doesn't check my phone files for errors... What am I missing?
Thanks

Probably an issue that belongs to the linter-php repo. by the way, You just have to install both packages and make sure the php executable is in your $PATH, otherwise specify an exact path to it


Remove rcng startupitem type

This was only used on non-darwin systems and has been broken for a
while; no port uses it.
see https://trac.macports.org/ticket/36770

No port using it isn't a factor as it's not something that a port would set; it's selected based on os.platform or set in macports.conf.
Are you sure it's actually broken? Sourcing ${prefix}/etc/rc.subr when it doesn't exist will fail harmlessly.

Ok, I agree the commit message needs changed.  However, it does not work as the rc.subr should contain commands (load_rc_config, run_rc_command); it errors out when trying to run the rc.d/ file.
Tested with  openntpd and linux.
https://trac.macports.org/ticket/36770 has more info on the subject - I believe the gist is to remove the startupitem_type

Sorry, totally missed the part about the commit message. It's still correct, unless a user manually installed a rc.subr to the prefix, the generated script was useless and would only produce an error, so that is kind of broken.


Integration Test - Release - 2019/06/12 19:11:06

Auto create for integration test. IssueCommandFlow

@PRMergerTest2 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Faker::Internet.domain_word == ""

This method return a void string, i.e "".

@polyglot0027 which faker version you are using?

@SagareGanesh faker (1.7.3)

@polyglot0027 I don't think there is such issue, I have tried to reproduce it locally, but didn't get anything.
Below is my testing report.

could you please add your screenshot here, so it ll help to reproduce it.

Sure! :)

@polyglot0027 This is weird issue. Because domain_name method internally uses domain_word method to create domain name, and add suffix(i.e .com, .org etc) to it.
Logically if domain_word method returns empty string, then the domain_name should return ".com" or ".org"

If your domain_name method is working, you can use
 Faker::Internet.domain_name.split('.')[0]
for domain word

@SagareGanesh The issue occurs because of localization settings:

@polyglot0027 You are right, got the issue. The reason behind the domain_name method working for any locale and domain_word not, is domain_name always returns domain name with en locale. I have resolved this issue and created PR.

@SagareGanesh Cool! Thanks

Closed by #957


Possibility for dynamic influencers to respect 3D particle effect's rotation

Currently, changing a 3D ParticleEffect's rotation (via setRotation or otherwise), only changes the rotation of the emitter - the dynamic influencers remain fixed.
While that may be fine for some cases, for a lot of effects - nozzles, thrusters, things like comets, basically anything that needs a changing directional displacement of the whole effect - this is not sufficient.
The only way to emulate the rotation of the influencers appears to be manually recalculating their theta and phi values, as described here - not really intuitive.
A good solution would probably be to introduce an additional attribute for the influencers (at least those subclassing Angular) that would toggle between respecting the entire effect's rotation.

Could you send a pull request?

@xoppa I'd love to, but that's pretty unlikely to be honest - 3D transforms are not my forte. I'll see what I can do, but no promises.


bootcards

how to do contacts drag from one group to another like icloud.com.

That would be something that you have to implement yourself: Bootcards is just there for the UI. You might want to a look at this for an example: http://www.bootply.com/dUQiGMggWO


Information add for Kocaeli University dev1

@karakayaonurr lib/domains/tr/edu/aof.anadolu.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@karakayaonurr Thank you for your request. Both universities are in the abuse list due to a history of massive abuses.
I have to close the request without merging.
It does not mean that something is wrong with the school. The issue is related to the fact that the schools' domains were heavily abused in the past. Please use an alternative way to get the license, as described below.
Please enroll to GitHub Student Developer Program at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).
Thank you for your understanding!


[📚] Pub site failed to generate dartdoc for this package

Hi guys, looks like the api reference for cloud_firestore over on pub.dev is broken.
https://pub.dev/documentation/cloud_firestore/latest/

Thanks for the report @ChariseWalraven. Looking at the dartdoc log, it is failing due to below error:
ERR : Because cloud_firestore depends on cloud_firestore_platform_interface ^5.7.7 which doesn't match any versions, version solving failed.
FINE: Exception type: SolveFailure
FINE: package:pub/src/solver/version_solver.dart 311:5               VersionSolver._resolveConflict
    | package:pub/src/solver/version_solver.dart 132:27              VersionSolver._propagate
    | package:pub/src/solver/version_solver.dart 96:11               VersionSolver.solve.<fn>
    | ===== asynchronous gap ===========================
    | dart:async                                                     Future.catchError
    | package:pub/src/utils.dart 109:52                              captureErrors.wrappedCallback
    | package:stack_trace                                            Chain.capture
    | package:pub/src/utils.dart 122:11                              captureErrors
    | package:pub/src/command.dart 189:13                            PubCommand.run
    | package:args/command_runner.dart 209:27                        CommandRunner.runCommand
    | package:pub/src/command_runner.dart 176:24                     PubCommandRunner.runCommand
    | package:pub/src/command_runner.dart 161:20                     PubCommandRunner.run
    | package:dartdev/dartdev.dart 45:56                             runDartdev
    | /b/s/w/ir/cache/builder/sdk/pkg/dartdev/bin/dartdev.dart 11:9  main
IO  : Writing 364388 characters to text file /tmp/tool-envBNZZPG/pub-cache-dirABDEAP/log/pub_log.txt.
[   +8 ms] pub finished with exit code 1

And a recent version of the plugin seems to be working: https://pub.dev/documentation/cloud_firestore/3.5.0/
So I think the issue could either be in https://github.com/dart-lang/pub/issues or https://github.com/dart-lang/dartdoc/issues

Nice, thank you! Do you need me to do anything further or will you guys
take it from here?
El vie, 7 oct 2022 10:50, darshankawar @.***> escribió:

Thanks for the report @ChariseWalraven
https://github.com/ChariseWalraven. Looking at the dartdoc log, it is
failing due to below error:
ERR : Because cloud_firestore depends on cloud_firestore_platform_interface ^5.7.7 which doesn't match any versions, version solving failed.
FINE: Exception type: SolveFailure
FINE: package:pub/src/solver/version_solver.dart 311:5               VersionSolver._resolveConflict
| package:pub/src/solver/version_solver.dart 132:27              VersionSolver._propagate
| package:pub/src/solver/version_solver.dart 96:11               VersionSolver.solve.
| ===== asynchronous gap ===========================
| dart:async                                                     Future.catchError
| package:pub/src/utils.dart 109:52                              captureErrors.wrappedCallback
| package:stack_trace                                            Chain.capture
| package:pub/src/utils.dart 122:11                              captureErrors
| package:pub/src/command.dart 189:13                            PubCommand.run
| package:args/command_runner.dart 209:27                        CommandRunner.runCommand
| package:pub/src/command_runner.dart 176:24                     PubCommandRunner.runCommand
| package:pub/src/command_runner.dart 161:20                     PubCommandRunner.run
| package:dartdev/dartdev.dart 45:56                             runDartdev
| /b/s/w/ir/cache/builder/sdk/pkg/dartdev/bin/dartdev.dart 11:9  main
IO  : Writing 364388 characters to text file /tmp/tool-envBNZZPG/pub-cache-dirABDEAP/log/pub_log.txt.
[   +8 ms] pub finished with exit code 1
And a recent version of the plugin seems to be working:
https://pub.dev/documentation/cloud_firestore/3.5.0/
So I think the issue could either be in
https://github.com/dart-lang/pub/issues or
https://github.com/dart-lang/dartdoc/issues
—
Reply to this email directly, view it on GitHub
https://github.com/firebase/flutterfire/issues/9688#issuecomment-1271308699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHN6QTUYOYULDYQNFU2GJFDWB7P7FANCNFSM6AAAAAAQ7K6BRU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Socket connection and authentication

Initial socket.io connections set up with basic chat room and client connections abstraction
Client socket authentication
Tests for the above

@kylelwm Can help review this?

All seems good to merge =)


fix(windows): react-native-config don't work with apps in RNW v0.64

Description
Fixes #606
Replace macro of msbuild in project file.
Compatibility

OS
Implemented

Windows
✅

Thank you!

You welcome!
This fix will be in v1.4.5?

Thank you!

You welcome!
@luancurti This fix will be in v1.4.5?

Thank you!

You welcome!
@luancurti This fix will be in v1.4.5?


Investigate why pre-commit hooks don't work in WebStorm

I've faced with an issue when I could commit code with linters/prettier warnings:

husky's hooks were not fired.

@softcat23 , which of WebStorm version?

@softcat23 , which of WebStorm version?

WebStorm 2020.3.2
Build #WS-203.7148.54, built on January 25, 2021
Licensed to ООО "Эпам Систэмз"
Runtime version: <IP_ADDRESS>+11-b1145.77 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 10.16
GC: ParNew, ConcurrentMarkSweep
Memory: 1981M
Cores: 8


double star (**) does not exponentiate, and doesn't give error

Describe the bug
As stated, double star (**) does not exponentiate, and doesn't give error. instead it returns the base.
The GUIDE.md file contains examples that uses **, however it doesn't behave as excpected.
To Reproduce
using an example from GUIDE.md:
def distance(a, b):
  (ax, ay) = a
  (bx, by) = b
  dx = bx - ax
  dy = by - ay
  return (dx * dx + dy * dy) ** 0.5

def main():
  return distance((10.0, 10.0), (20.0, 20.0))

Result: 200.000
Expected behavior
Either it should give correct Result: 14.142 or raise an error for not using ^
I prefer it working if it is supposed to be similar to python.
Also the guide should be updated.
Desktop (please complete the following information):

OS:   [Windows 10, WSL2]
CPU: [Intel i5-9600k]
GPU: [ RTX 1070]
Cuda Version [12.1.0]

Additional context
I just installed bend today to try it out through WSL.

similar setup here, also using WSL. It seems that most of the examples in GUIDE.md are not working.

This is actually a problem in an inet level transformation making the pair (10.0, 10.0) reduce to 10.0. We'll fix that.

#448 fixes this. The problem is that the order of the operators was getting inverted when the second argument of ** was a number


feat: Add CWC Tearsheet

Related Ticket(s)
Closes #https://github.com/carbon-design-system/carbon-for-ai/issues/17
Description
First pass at creating Tearsheet as a web component.
Changelog
New

Experimental tearsheet

Changed

Create ButtonSetBase so as to share between side panel and tearsheet

Removed

Added #11586 PR which builds on this. Happy to merge into this PR if desirable.


zero-3 dynamic tracing support

Previously ZeRO-3's prefetcher was essentially disabled if the sub-module access pattern changed after the first iteration. This PR adds support for detecting and adapting our sub-module trace to more dynamic access patterns.
In initial testing with a modified version of Megatron that adds a new sub-module into forward after a few steps we're seeing ~10% boost in performance.

Can one of the admins verify this patch?


Empty states for platforms portfolios and portfolio screens

Changes

added empty states for platforms portfolios and portfolio screens (not add/remove portfolio item screens)
Icons is mostly subject to change but changing that is just a matter of importing different icon from PF.

All of them look similar just different text

Codecov Report

Merging #153 into master will increase coverage by 0.11%.
The diff coverage is 88.23%.

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
+ Coverage    81.2%   81.31%   +0.11%
==========================================
  Files          78       79       +1
  Lines         729      744      +15
  Branches       59       59
==========================================
+ Hits          592      605      +13
- Misses        124      126       +2
  Partials       13       13

Impacted Files
Coverage Δ

src/smart-components/platform/platforms.js
88.88% <0%> (-11.12%)
:arrow_down:

src/smart-components/portfolio/portfolio.js
92.85% <100%> (+0.35%)
:arrow_up:

...l-components/shared/content-gallery-empty-state.js
100% <100%> (ø)

src/smart-components/portfolio/portfolios.js
100% <100%> (ø)
:arrow_up:

...mart-components/content-gallery/content-gallery.js
100% <100%> (ø)
: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 e38d2a7...e23491a. Read the comment docs.

cc @sbuenafe-rh

@Hyperkid123 if no products exist in a portfolio, we should disable the "Remove products" button in the toolbar.

@sbuenafe-rh:

@Hyperkid123 if no products exist in a portfolio, we should disable the "Remove products" button in the toolbar.

Well yes, he is, obviously:
https://github.com/ManageIQ/catalog-ui/pull/153/files#diff-da61114cab5c53d2b1183ac5c81b3825R7

@Hyperkid123 if no products exist in a portfolio, we should disable the "Remove products" button in the toolbar.

@Hyperkid123 : please, address that in a follow up PR


Add Support for Special Speeds when not separated correctly

Special Speeds currently need to be separated by semicolon, otherwise sbc breaks when it can't find the speed type in the actor sheet.

Fixed in 2.0.7

Just ran through an import with Speed 40 ft., climb 20 ft. and the import worked perfectly.


Fix links

It seems like the sources were removed.

If that is the case, we should archive the apps - not delete the entries.

We should archive the apps, not delete the entries

But if there is no source code?

@vpeschenkov I was thinking the important thing is to maintain the active list. Archiving is fine in this case as well. Let me know what you think
Sent with GitHawk

@vpeschenkov I was thinking the important thing is to maintain the active list. Archiving is fine in this case as well. Let me know what you think

@dkhamsing  Finally, I understood the meaning :-)

In this case, I think we have to remove all the links from the ARCHIVE.md list. Because that list has links that lead to source code, but there is no source code anymore :-). I see ARCHIVE.md as a place for old/unsupported projects, but with source code.

I don't understand for which reason we need to keep such kind of projects? If we keep projects without code, there is no sense. Such kind of projects doesn't make a value for a user because the most value is source code (I don't see any value in keeping a project name + description :-)).

So, if there is no source code, I think we have to remove such items from the list completely.

Hmm good point. The user @opensourceios usually archives code but if u can confirm it is missing there too, let’s delete
Sent with GitHawk


Adding solution of PageRank algorithm.

Fixing Issue #3773
Description
This pull request adds the implementation of the PageRank algorithm in C++ to the DSA folder. PageRank is an algorithm used by Google Search to rank web pages in their search engine results. It measures the importance of website pages by counting the number and quality of links to each page, which is then used to determine a relative score of the page's importance.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[x] I have ensured that my changes do not break any existing functionality.
[x] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[x] I have read the resources for guidance listed below.
[x] I have followed security best practices in my code changes.

Additional Context
The PageRank algorithm implementation includes:

Calculation of the PageRank of each page in a graph using an iterative approach.
Consideration of damping factors to simulate the probability of random web surfing.
Clear and well-commented code for easy understanding and maintenance.

Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[x] Markdown Guide

@Ajay-Dhangar sir, kindly review and let me know if any changes required.


Create submit button for work deposit form

Create a reusable button element for submitting the form and put at the last location in the form UI.  This is a UI ticket--does not need to kick off a job. Put at the end of the form, or on a Deposit tab if the tab set-up is available.  (See Figma design)

Resolved by #74.


Location LatLng problem

``
In this example, the coordinates are from Black Sea.
The map is not displaying. Why?
If I use other coordinates the map will show, but the marker is not in the right position.

I want to use the marker to get some coordinates (LatLng), then save them to database.
I want to use coordinates to display map and marker in exactly selected LatLng position.

My problem is similar to this https://github.com/ubilabs/geocomplete/issues/13, but I can't make it work.

Just stumbling across, first off it looks like you need more then just the geocomplete functionality and might be better off using the complete maps v3 api with something like gmaps.js - a jQuery wrapper for the api. Second your default location is much to precise for being in the middle of the ocean. You should not need more then 9 decimal places. Read this answer here: http://gis.stackexchange.com/a/8674.
It looks like your problem has nothing to do with geocomplete as it does with your logic. If you have a problem, bring it to stack overflow and maybe be more precise about what you are trying to accomplish.


c++ implementation of fddp solver

In GitLab by @cmastalli on Jul 15, 2019, 08:46
null

In GitLab by @cmastalli on Jul 15, 2019, 08:46
assigned to @cmastalli

In GitLab by @proyan on Sep 4, 2019, 13:26
assigned to @proyan and unassigned @cmastalli

In GitLab by @cmastalli on Sep 4, 2019, 13:29
I have improved a bit the DDP solver, please do the same with the FDDP.
Additionally, I have merged the missed part of the FDDP quite time ago. Everything is in the devel branch.

In GitLab by @cmastalli on Sep 7, 2019, 14:01
closed


Improve Logging / GroupBy Cluster and Operation + Show As Task List

Story

As operator (of the week) I want to access cluster operations logs (create, update/reconcile, delete) in a convenient way, so that I can easily see what the outcome of each operation was without trying to grep the information out of the overall/global Gardener logs.

Motivation
We should improve the way how we interact/access the Gardener/cluster logs. As an operator (of the week) I frequently have to check what the Gardener says/logs on a certain cluster and operation. Now with the reconciler, the logs grew and with all the other planned features, they will even grow more. Of course, we will have a logging stack eventually, but that's not what this story is about. Usually I need to know something specific about a particular cluster, for a particular operation at a particular date/time. A logging stack and UI will allow me to set filters, but it won't be semantic in this sense. Also, we can't give away the entire operator logs via a logging stack to the entirety of the people that will later use the Gardener dashboard as operators (there must be strong isolation, so that we can have only one Gardener instance running).
Acceptance Criteria

[ ] Describe ideas how to get to this semantic logging, ideally stored/associated with/accessible via a project or cluster

Implementation Proposal
What springs to mind is something similar to the Bosh tasks and logs. There, the user nicely sees what the Bosh director does and did: A creation of a certain deployment, a deletion of one, any scan & fix tasks that happened on it, etc. Information (collection of logs) on that granularity (cluster + operation) would be a great improvement over a large shared operator log where I have the access rights issue and can only filter by properties instead of seeing (in a list) which operations where carried out when on one or all of my clusters in my project.
Definition of Done

[ ] Knowledge is distributed: Have you spread your knowledge in pair programming/code review?
[ ] Unit Tests are provided: Have you written automated unit tests or added manual NGPTT tickets?
[ ] Integration Tests are provided: Have you written automated integration tests?
[ ] Minimum API exposure: If you have added public API, was it really necessary/is it minimal?
[ ] Operations guide: Have you updated the operations guide?

End of 2017 we implemented operation IDs that group all logs for an operation like create, reconcile or delete and report those as events of shoot resources, i.e. we have now the technical means to expose the logs in the dashboard (BLI).


BUG: URGENT - UnhandledPromiseRejectionWarning: DiscordAPIError: Missing Permissions

Describe the bug
We are not error handling for missing permissions. The bot craps out when there's a discrepancy in permissions somewhere: UnhandledPromiseRejectionWarning: DiscordAPIError: Missing Permissions
To Reproduce
Steps to reproduce the behavior:
I have no idea how this happens, but its been crashing the bot, needs to be taken care of ASAP

Hi there, is this issue still active ?

@jalalbmnf Apologies for the delay in my response. The bot hasn't been fully functional for a little while now due to my delay in implementing the required changes mandated by Discord. I'm still working on it, but as I'm working on it, the bot isn't fully functional and so there has been a significant decrease in requests the bot has been getting, so I haven't seen this error lately. However, I'm not sure if this error will continue to persist after it becomes fully functional again, so I'm keeping this issue open until I can verify whether or not it's no longer an issue.

@elenirotsides no problem, thank you for response, maybe then you can close this issue, if it will happen then someone will open issue about this. For now this just confusing us :)


GET /explorer/blocks/height response not consistent

BUG REPORT
Stack Trace or error message
GET /explorer/blocks/151739 the block fields are all empty, except transactions
Expected Behavior
GET /explorer/blocks/151738 is expected response
How to reproduce it (as minimally and precisely as possible)
Environment

Sia version: Git master
OS: Linux

The explorer says the block 151739 hash is<PHONE_NUMBER>000000780e884a62fc626afe5a16b7dc7baf5d2e6062bfa4b3b010  https://explorer.siahub.info/block/151739
Then the GET /explorer/hashes/0000000000000000780e884a62fc626afe5a16b7dc7baf5d2e6062bfa4b3b010 return {"message":"unrecognized hash used as input to /explorer/hash"}

explorer module is buggy and laggy, need total rework. Or something like.
So you can only delete explorer DB and build it again until it will work correctly.

Official made by @eddiewang
They did some fixes in explorer module: https://github.com/LuxorLabs/Sia/tree/luxor-explorer
I did my custom build too for SiaHub, but just as new endpoint in api, not via explorer fixes.
https://github.com/S-anasol/Sia/tree/siahub

without explorer module
use explorer api :D
https://siahub.readme.io/v1.0/reference#block

If there are fixes, why don't the team merge to the official sia repo?

We're not confident the fixes are an overall step in the right direction. Anything we merge, we'll be stuck with for a long time, and the fixes had issues and inconsistencies of their own.

better the devil you know than the one you don't, I guess.
Another contributor is taking a stab at making their own explorer module. I'm not sure how far they are or how committed, but they know the codebase well and have a good shot at making something that's better than everything else currently available.
fingers crossed

I have got a panic log
goroutine 1 [running]:
runtime/debug.Stack(0x446007, 0x0, 0xc42b8b0e00)
	/snap/go/1688/src/runtime/debug/stack.go:24 +0xa7
runtime/debug.PrintStack()
	/snap/go/1688/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Critical(0xc42b8b0ea8, 0x2, 0x2)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/build/critical.go:16 +0x93
github.com/NebulousLabs/Sia/modules/explorer.(*Explorer).ProcessConsensusChange(0xc420097c80, 0xc914d81931bbde74, 0x612a5aa14c8e1cb8, 0x4f3ad19124dc71a3, 0xc740bc5209ecd807, 0xc44c602d80, 0x1, 0x1, 0xc47e416a80, 0x2, ...)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/modules/explorer/update.go:282 +0xd4
github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedInitializeSubscribe.func2(0xc48529a0e0, 0xad6b38, 0xc48529a0e0)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/modules/consensus/subscribe.go:187 +0x236
github.com/NebulousLabs/Sia/vendor/github.com/coreos/bbolt.(*DB).View(0xc420156780, 0xc443479378, 0x0, 0x0)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/vendor/github.com/coreos/bbolt/db.go:704 +0x90
github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedInitializeSubscribe(0xc42018c780, 0xb3d420, 0xc420097c80, 0x0, 0x0, 0x0, 0x0, 0x0, 0xc420155490, 0x5d35f2)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/modules/consensus/subscribe.go:176 +0x264
github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).ConsensusSetSubscribe(0xc42018c780, 0xb3d420, 0xc420097c80, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/modules/consensus/subscribe.go:216 +0xee
github.com/NebulousLabs/Sia/modules/explorer.New(0xb47e80, 0xc42018c780, 0xc420022400, 0x18, 0x18, 0x1a, 0x0)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/modules/explorer/explorer.go:80 +0x197
main.(*Server).loadModules(0xc4200c20f0, 0xad6ec0, 0xc42008c300)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/cmd/siad/server.go:514 +0x194d
main.startDaemon(0x0, 0x0, 0xaa9599, 0xe, 0xa998e6, 0x5, 0xa998eb, 0x5, 0x0, 0x7ffd642457a4, ...)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/cmd/siad/daemon.go:188 +0x2b3
main.startDaemonCmd(0xc420101b80, 0xc420070780, 0x0, 0x4)
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/cmd/siad/daemon.go:236 +0x162
github.com/spf13/cobra.(*Command).execute(0xc420101b80, 0xc42001e150, 0x4, 0x4, 0xc420101b80, 0xc42001e150)
	/home/one/GOPATH/src/github.com/spf13/cobra/command.go:766 +0x2c1
github.com/spf13/cobra.(*Command).ExecuteC(0xc420101b80, 0xac72e3, 0x3b, 0xc4200bbb80)
	/home/one/GOPATH/src/github.com/spf13/cobra/command.go:852 +0x30a
github.com/spf13/cobra.(*Command).Execute(0xc420101b80, 0xefb640, 0xaae618)
	/home/one/GOPATH/src/github.com/spf13/cobra/command.go:800 +0x2b
main.main()
	/home/one/GOPATH/src/github.com/NebulousLabs/Sia/cmd/siad/main.go:174 +0x6fe
Critical error: explorer update failed: runtime error: invalid memory address or nil pointer dereference
Please submit a bug report here: https://github.com/NebulousLabs/Sia/issues


Semantic API signature of the error type symbol is invalid

Description:
Semantic API signature of the error type symbol is invalid. For instance return type symbol of the below is a union of error member types and it generates an invalid signature;
Function Invocation
import ballerina/io;
io:openReadableCsvFile("");

Expected
import ballerina/io;
io:ReadableCSVChannel|io:GenericError|io:ConnectionTimedOutError|io:AccessDeniedError|io:FileNotFoundError|io:TypeMismatchError|io:EofError|io:ConfigurationError openReadableCsvFile = io:openReadableCsvFile("");

Actual / Current
import ballerina/io;
io:ReadableCSVChannel|GenericError|ConnectionTimedOutError|AccessDeniedError|FileNotFoundError|TypeMismatchError|EofError|ConfigurationError openReadableCsvFile = io:openReadableCsvFile("");

Seems like we are not honoring package prefixes since default getModule() always return empty in https://github.com/ballerina-platform/ballerina-lang/blob/master/compiler/ballerina-lang/src/main/java/io/ballerina/compiler/api/impl/symbols/BallerinaErrorTypeSymbol.java#L63
Affected Versions:
2.0.0-alpha2

@daneshk I think we have a plan to introduce a single generic error instead of 10 different errors. Please correct me if I am wrong.

Yeah. We are planning to have one generic error for each package and then branch out from that to more granular errors.

@shafreenAnfar so the plan is to moving away from unions as in https://github.com/ballerina-platform/module-ballerina-io/blob/f7ef7f816aab8a53483a6a24117fbea765d1dc69/io-ballerina/io_error.bal#L39 ?

Yeah and build the error hierarchy as in https://ballerina.io/learn/by-example/user-defined-error.html.


Fix Javascript/Typescript errors

[ ] Resolve "tsconfig.json" errors

To be honest, using babel for non-browser node.js applications, is not needed anymore in 2021.
Checking the line history on git, the babel stuff is some dusty fragment from 4 years ago.
Would love to fix that one, replace it with default v10+ node, properly enable ESLint with .eslintrc file and some useful plugins for that. In combination with the VS code extension ErrorLens ~ you will see inline warnings, if you are writing some code with bad practices or old | deprecated syntax, that should be changed.

We uses a SvelteKit preconfigured setup. Do we still need this?

Yeah, on that side, its all good. But its about those root level package.json modules that are used to spin up the backend. These days, babel is deprecated, didn't see this in-use with node.js since ages. Frontend-wise, as you correctly said, svelte takes care of these things already.
https://github.com/open-marketplace-applications/dshop/blame/91a1e1be65cace6c57d96a39726af0bf5c9736a3/package.json#L8


Gameplay ends too soon

The traditional game of triominos ends when all players are unable to play a tile.
Currently, the gameplay ends when there are no more tiles in the pool.

This issue is being fixed in the endgame branch.


Separate HAL driver registration from threading

In the new HAL C world both VMLA and (legacy) dylib are just executable loaders that can be registered on the task-based CPU HAL driver. To remain compatible with the current infra which is assuming both exist as top-level drivers I've kept the registrations, however we should remove them as the new executable_library stuff comes online as part of #3580.

@benvanik Anything we should do to address https://github.com/google/iree/blob/f76a371/iree/hal/dylib/registration/driver_module.c#L22-L26 after the latest change? Or should we close the issue?


🤖🎩  semantic-release

I'll put a link to the last version of CHANGELOG.md into the next release notes

merged via e6a0950


Avoid race condition in maintain_leases by copying leased_messages

@jonparrott This change is causing unit test failures for me:
_____________________ test_maintain_leases_outdated_items ______________________
Traceback (most recent call last):
  File "/home/tseaver/projects/agendaless/Google/src/google-cloud-python/pubsub/tests/unit/pubsub_v1/subscriber/test_policy_base.py", line 327, in test_maintain_leases_outdated_items
    modify_deadline_seconds=[10],
  File "/home/tseaver/projects/agendaless/Google/src/google-cloud-python/pubsub/.nox/unit-3-6/lib/python3.6/site-packages/mock/mock.py", line 948, in assert_called_once_with
    return self.assert_called_with(*args, **kwargs)
  File "/home/tseaver/projects/agendaless/Google/src/google-cloud-python/pubsub/.nox/unit-3-6/lib/python3.6/site-packages/mock/mock.py", line 937, in assert_called_with
    six.raise_from(AssertionError(_error_message(cause)), cause)
  File "<string>", line 3, in raise_from
AssertionError: Expected call: send_request(modify_deadline_seconds: 10
modify_deadline_ack_ids: "ack2"
)
Actual call: send_request(modify_deadline_seconds: 10
modify_deadline_seconds: 10
modify_deadline_ack_ids: "ack1"
modify_deadline_ack_ids: "ack2"
)
------------------------------ Captured log call -------------------------------
base.py                    382 WARNING  Dropping 1 items because they were leased too long.

@tseaver if it's blocking you feel free to roll it back until I can take a closer look.,

I'm not blocked except from helping on pubsub issues.


Support creating proxies for artifacts from installed packages

Truffle supports loading an artifact from an installed package:
const ERC20 = artifacts.require("@openzeppelin/contracts-ethereum-package/ERC20PresetMinterPauserUpgradeSafe");

But trying to deployProxy it currently fails with the following error:
Error: The requested contract was not found. Make sure the source code is available for compilation
    at getContractName (/home/spalladino/Projects/sdk/truffle/foo/node_modules/@openzeppelin/upgrades-truffle/node_modules/@openzeppelin/upgrades-core/src/validate.ts:123:11)
    at Object.assertUpgradeSafe (/home/spalladino/Projects/sdk/truffle/foo/node_modules/@openzeppelin/upgrades-truffle/node_modules/@openzeppelin/upgrades-core/src/validate.ts:141:24)
    at deployProxy (/home/spalladino/Projects/sdk/truffle/foo/node_modules/@openzeppelin/upgrades-truffle/src/deploy-proxy.ts:30:3)
Truffle v5.1.37 (core: 5.1.37)
Node v12.7.0

Hello, any update or preferred approach for this? Thank you !

Hello @clemsos, there hasn't been any news, and I can't promise it will be supported anytime soon.
The best option, in my opinion, is to have an "importer" contract in your contracts folder
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.0;

import "path/to/the/source/file.sol";

This will cause truffle to recompile the contract, and make it available as a "local artifact" that you can deploy

thanks @Amxx . The problem here is that we don't have the .sol files - just ABIs. From what I understood, the upgrades lib require the .sol contract files to check storage layout etc. Is that correct?

The upgrade plugin compares the new artifact against the storage layout stored in the manifest that was produced when the previous version was deployed (assuming it was deployed using OZ's tolling).
You should have a .openzeppelin/network.json with this infos.


Coverflow slider with custom third slide

Hello. The project uses this slider, but I did not find a solution, because the task is specific. Is it possible to make the slider view below using this slider?

p.s. sorry for my english

Do you want to ask a question? Are you looking for support? Stack Overflow is the best places for getting support

Please, don't use GitHub issues for questions


Follow up history transaction when closing the app but the transaction has not been successful

Description the bug
Version test: v1.0.1(134)
Status of the NFT transaction displayed incorrectly
Steps:

Go to the Send NFT screen
Perform Send NFT
After transfered NFT successfully, go to the History sreen
Observe the transaction history displayed

Actual: NFT transfered successfully, however, the history still show status = Processing
https://user-images.githubusercontent.com/130966515/234806646-95f631f3-3fa4-48fc-a534-d32248191dc3.mp4
Expect: Show correct status

Version test: v1.0.3(161)
Devices test: IP 11
Validation passed.


Update robots.txt to disallow ChatGPT

Per the ChatGPT docs

To disallow GPTBot to access your site you can add the GPTBot to your site’s robots.txt:

User-agent: GPTBot
Disallow: /

HT to this toot

Fixed with 89215f2


Scene transforms unable to view.

See screenshot:

This issue is fixed, thus closing.

The height of this table is sufficiently big, but the width isn't, so you cannot see the values in the last cell. Moreover, there is actually quite sufficient real estate in the GUI in terms of width...


Unsupported file format exception when providing MICRONAUT_CONFIG_FILES

Expected Behavior
Should be able to supply MICRONAUT_CONFIG_FILES env var to tests without impacting behaviour of internalStartTestResourcesService or causing an error.
Actual Behaviour
When supplying MICRONAUT_CONFIG_FILES environment variable, I get the following exception in the internalStartTestResourcesService build job, this causes the test build to hang indefinetely:
12:02:38.343 [ForkJoinPool.commonPool-worker-6] INFO  i.m.t.e.TestResourcesResolverLoader - Loaded 2 test resources resolvers: io.micronaut.testresources.mysql.MySQLTestResourceProvider, io.micronaut.testresources.testcontainers.GenericTestContainerProvider
12:02:38.349 [main] ERROR io.micronaut.runtime.Micronaut - Error starting Micronaut server: Unsupported properties file format while reading local-secrets.yml from src/test/resources/local-secrets.yml
io.micronaut.context.exceptions.ConfigurationException: Unsupported properties file format while reading local-secrets.yml from src/test/resources/local-secrets.yml
	at io.micronaut.context.env.DefaultEnvironment.lambda$readPropertySourceListFromFiles$6(DefaultEnvironment.java:488)
	at java.base/java.util.Optional.ifPresent(Optional.java:178)
	at io.micronaut.context.env.DefaultEnvironment.readPropertySourceListFromFiles(DefaultEnvironment.java:470)
	at io.micronaut.context.env.DefaultEnvironment.readPropertySources(DefaultEnvironment.java:416)
	at io.micronaut.context.env.DefaultEnvironment.start(DefaultEnvironment.java:270)

Steps To Reproduce

Create tests using test-resources
Supply environment variable MICRONAUT_CONFIG_FILES
Start tests

Environment Information
No response
Example Application
No response
Version
3.6.2

Thanks for the bug report. It's an "interesting" side effect of the test resources service being a Micronaut application itself... Let me think about it.

I have a solution to this problem which would require a new test resources release, but only for Micronaut 3.7+.
Since you are using Gradle, there is an alternative to this. You have the problem because you are running something like:
MICRONAUT_CONFIG_FILES=my-config.yml ./gradlew test

which exports the MICRONAUT_CONFIG_FILES globally. Instead, as a workaroud, you could alter the build script so that only the test task sees this environment variable. Something like this:
tasks.withType(Test).configureEach {
    def overrideEnv = System.getenv("CI_MICRONAUT_CONFIG_FILES")
    if (overrideEnv != null) {
        inputs.property("MICRONAUT_CONFIG_FILES", overrideEnv)
        environment.put("MICRONAUT_CONFIG_FILES", overrideEnv)
    }
}

Then on CI, instead of setting MICRONAUT_CONFIG_FILES you would set CI_MICRONAUT_CONFIG_FILES.

Thanks, that workaround did the trick! Thanks for the help.
We now have a different issue, but not sure if I should create an issue for it or if it's just a config issue. It seems the mysql container that the plugin starts is always uses the database name 'test', however our Liquibase migrations sometimes use the expected db name 'verification' causing the migrations to fail due to that DB not existing. How can I change the default DB name from 'test' to something else for Mysql?

These docs suggest setting datasources.*.db-name https://micronaut-projects.github.io/micronaut-test-resources/latest/guide/
However, with this fix, test-resources does not see the src/test/resources/local.properties file where I assume this config should be applied.


IntelliJ plugin does not accept legal .bazelignore files

Starting in version 2.1.0, bazel has allowed comments in the .bazelignore file. commit.
However, when such a file is used with the IntelliJ plugin the project fails to sync. If your .bazelignore has a comment in it:
# a comment
some-package

The IntelliJ plugin will run something like this:
Querying targets in project directories...
Command: /usr/local/bin/bazelisk query [snip] - //# a comment/...:all - //some-package/...:all

Thus having a perfectly legal .bazelignore file will cause the project to be unusable in IntelliJ.

I already sent a (tested) PR fixing this issue in #1837. It's a one-liner exactly matching the original bazel one-line change. Obviously, I did not wait for discussion with maintainers, but it did not seem like much of a controversial change. Hopefully this is something we can discuss and decide to accept (or not) quickly.

Hey @brendandouglas - I was not going to ping for a few weeks, but I saw that you're active again on one of the issues I updated yesterday. I hope you had a good memorial day (if you're in US that is), as much as any of us had a "real" holiday in these times :).
I was wondering if there's anything I should do more to address this (and PR #1837). It does not seem like it ought to be controversial, and the functionality looks isolated from impacting Blaze.

Sorry I didn't reply for so long, just catching up on bazel issues now.
Definitely not controversial -- I'll look at applying it upstream now.

OK, should be landing here either today or next Monday -- thanks for the fix!


fix(worker): fix handler out tuple

it seems that it doesn't follow whatlambda.Start() Rules say. https://github.com/aws/aws-lambda-go/blob/3dedc3285b5533729a20927fbf0fea1f1a167747/lambda/handler.go#L145

Will test it in staging env.


Is data transmission safe with Gpiozreo?

The thing is at school we have to transfer data from one Pi to a nother Pi. Wher
by we need to type in Numbers on a keypad at one end and it needs to control a 7-segment on the other Pi connected via Ethernet cable (I have that all set up and running ;-). The data transmission needs to be secure. If for example if the code is corrupted on the way by a magnetic field for example it should recognise that and not show the wrong data. That would be ideal. Currently I am using the Gpiozero Libery to do the transmission. Wher as one Pi is the deamon. May be I could send back a signal and that would show the user if it was corrupted (Just a thought). My questionn is if it does have some feature that checks the data or makes sure the data is tranferd safe? Some Protocoll. It can never be a hundred procent but it still needs some check on it. I would be realy greatfull for your reply.

My questionn is if it does have some feature that checks the data or makes sure the data is tranferd safe? Some Protocol

Yes.  Transport Control Protocol (TCP) is used by pigpio socket interface (if you are connecting remotely to the pigpio daemon you are using sockets).  TCP is a reliable transport with guaranteed delivery.


ENH cache datasets

I think it makes sense to provide our own caching mechanism for datasets in benchopt. In many benchmarks we currently rely on libsvmdata to do the caching, but I think it makes more sense to do this in benchopt, especially because new contributors that want to add datasets from other sources need to then write their own caching functions, which in the end means there will be multiple cache locations that there is no control over through benchopt.
I can provide a PR if you think this sounds like a good idea.

Just to make sure: we already have a caching mechanism with BaseDataset._get_data that loads the data only once per benchmark. But you're talking about once and for all caching so that data is downloaded only once by the user, for example ?

Yes, sorry, that's right. I mean that downloaded datasets are cached.

where do we store it? how does the user specify a path if there is already a caching mechanism, eg sklearn data of libsvm data?

I don't think the user should specify the path. We should select a path in a platform-independent way, for instance using appdirs. The name could then be something like /var/cache/benchopt/benchmark_lasso/data/url for the url data set in the benchmark_lasso repo.

how do we handle datasets with the same name between benchmarks?

We would need to store duplicates. Anything else would just be too complicated and prone to fail. Yes, that's a definite downside. Maybe that one is large enough to make us stay away from doing this.

Btw, if the concern comes from the Breheny datasets, @theoguyard is adding them to libsvmdata this week : https://github.com/mathurinm/libsvmdata/pull/36

That's nice, but no, not really. I'm more concerned about making it easy for new collaborators to add datasets.
On Tue Nov 8, 2022 at 10:44 AM CET, mathurinm wrote:

Btw, if the concern comes from the Breheny datasets, @theoguyard is adding them to libsvmdata this week : https://github.com/mathurinm/libsvmdata/pull/36
--
Reply to this email directly or view it on GitHub:
https://github.com/benchopt/benchopt/issues/468#issuecomment-1306920377
You are receiving this because you authored the thread.
Message ID: @.***>


Automate Discord configuration

teams map to roles in Discord
all contributors are granted 'contributors' role

The aequasi/discord provider seems the best one (it has docs).
This will likely involve creating a Discord Application, Bot, and Team which should probably have the @concourse/infrastructure team as members (admins).

done!

cmd/harmonize
.github/workflows/discord.yml

ended up not using Terraform; it raised more questions than it answered. (primarily where does the state get stored, since it can't be imported, and without it the provider seems to just keep creating stuff rather than being idempotent.)

woops, this doesn't set the contributors role - reopening.

done via https://github.com/concourse/governance/commit/fab1b7625c3ac07e6fb5d671b7c406da7270bd04


Fix nightly CPU test failures

https://github.com/pfnet/pytorch-pfn-extras/actions/workflows/nightly-test-cpu.yml

Latest summary https://github.com/pfnet/pytorch-pfn-extras/actions/runs/6764593441/job/18383088613#step:5:366
=========================== short test summary info ============================
FAILED tests/pytorch_pfn_extras_tests/distributed_tests/test_distributed_validation_sampler.py::test_no_distributed_available - ValueError: Default process group has not been initialized, please make sure to call init_process_group.
FAILED tests/pytorch_pfn_extras_tests/training_tests/test_manager.py::test_ignite_extensions_manager_state_dict - UserWarning: PairwiseParallel is deprecated and will be removed soon. Use ColwiseParallel and RowwiseParallel instead.
FAILED tests/pytorch_pfn_extras_tests/training_tests/extensions_tests/test_evaluator.py::test_ignite_evaluator_reporting_metrics - UserWarning: PairwiseParallel is deprecated and will be removed soon. Use ColwiseParallel and RowwiseParallel instead.
FAILED tests/pytorch_pfn_extras_tests/training_tests/extensions_tests/test_progress_bar_notebook.py::test_ignite_extensions_manager_with_progressbar_notebook - UserWarning: PairwiseParallel is deprecated and will be removed soon. Use ColwiseParallel and RowwiseParallel instead.

@emcastillo
https://github.com/pfnet/pytorch-pfn-extras/blob/dd8ed077d17b27f5828a53cbf2156a408ecdcec8/tests/pytorch_pfn_extras_tests/distributed_tests/test_distributed_validation_sampler.py#L81-L85
The first error among the errors that are appearing is related to the content added in this PR.
https://github.com/pfnet/pytorch-pfn-extras/pull/551
The reason this unit test is failing is that despite the title of the test, torch.distributed.is_available() returns True, so the exception is not raised as expected. This test already does not behave as it is named.
torch.distributed.is_available() is a function to determine if the torch.distributed module is available, and is separate from torch.distributed.is_initialized(), which checks to see if the distributed communication has been initialized. torch.distributed.is_initialized(), which is supposed to be True in most cases.
Is it possible to remove this test in some cases?


[Feature]: Implement all interfaces of @tanstack/react-query@5 if we can

Package Scope
@suspensive/react-query
Description
Implement all interfaces of @tanstack/react-query@5 if we can
Possible Solution
No response
etc.
No response

[ ] HydrationBoundary
[ ] useMutationState
[x] infiniteQueryOptions
[x] queryOptions
[x] useInfiniteQuery
[x] useMutation
[x] usePrefetchInfiniteQuery
[x] usePrefetchQuery
[x] useSuspenseInfiniteQuery
[x] useSuspenseQueries
[x] useSuspenseQuery

The necessary APIs have been implemented. I will open an issue when needed.


AWS Authentication fails with InvalidSignatureException

I'm having trouble getting the AWS sigv4 authentication working as described in https://httpyac.github.io/guide/variables.html#aws-signature-v4. I keep getting a response like

The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.\n\nThe Canonical String for this request should have been ...

Repro
I have two files, local.env and test.http
local.env
accessKeyId=MY_KEY_HERE
secretAccessKey=MY_SECRET_HERE
sessionToken=MY_SESSION_TOKEN_HERE

test.http
@awsUrl = https://MY_APP_ID.execute-api.us-west-2.amazonaws.com/prod

PUT {{awsUrl}}/echo
Authorization: AWS {{accessKeyId}} {{secretAccessKey}} token:{{sessionToken}}
content-type: application/json
{"message": "hello"}

What i've tried

I get the same issue if I substitute the variables from my environment into the test.http file
Verified the credentials work when I use awscurl (https://github.com/okigan/awscurl)
tried adding region:us-west-2 and service:execute-api to the authorization line (didn't change the outcome)

More detail
VS Code: 1.68.1
httpYac - Rest Client extension: v5.5.1
From the "httpyac - request" window, the outgoing request looks like this:
PUT https://MY_APP_ID.execute-api.us-west-2.amazonaws.com/prod/echo
accept-encoding: gzip, deflate, br
accept: */*
authorization: AWS4-HMAC-SHA256 Credential=MY_KEY_HERE/20220623/us-west-2/execute-api/aws4_request, SignedHeaders=content-type;host;x-amz-date;x-amz-security-token, Signature=91cfc7b46d1466308423c2d2dfc62e5a48cd4d70db8faed7ab696167862f62dc
content-length: 20
content-type: application/json
host: MY_APP_ID.execute-api.us-west-2.amazonaws.com
user-agent: httpyac
x-amz-date: 20220623T075652Z
x-amz-security-token: MY_SESSION_TOKEN_HERE

{"message": "hello"}

response
HTTP/1.1 403  - Forbidden
connection: close
content-length: 1452
content-type: application/json
date: Thu, 23 Jun 2022 07:56:52 GMT
x-amzn-errortype: InvalidSignatureException
x-amzn-requestid: 77907a1c-6800-42dc-8887-6026a0d2e0da
x-amzn-trace-id: Root=1-62b41cc4-456a542c0def21f6652269f1

{"message":"The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.\n\nThe Canonical String for this request should have been\n'PUT\n/prod/echo\n\ncontent-type:application/json\nhost:MY_APP_ID.execute-api.us-west-2.amazonaws.com\nx-amz-date:20220623T075652Z\nx-amz-security-token:MY_SESSION_TOKEN_HERE\n\ncontent-type;host;x-amz-date;x-amz-security-token\ncf8f644b80ee8100db6f910fa33514be8e0ac3d3f54546de914b34c2206ad6a9'\n\nThe String-to-Sign should have been\n'AWS4-HMAC-SHA256\n20220623T075652Z\n20220623/us-west-2/execute-api/aws4_request\n670c4c7ba89357df2e024e3b5e0b2d917b3c111ded27e856d42638cecbf8da09'\n"}

By the way....I think the problem is that the request body isn't getting passed into the signer in https://github.com/AnWeber/httpyac/blob/bce65b50179a8ec25c2ef55dd5812b7fd6cce6f4/src/plugins/http/awsAuthVariableReplacer.ts#L19-L37.
Authorization works fine for GET requests.

Your proposed solution may be good. I'll have a quick read of the documentation, but unfortunately I don't have an AWS service I can address. I would just change it then and hope for a test from you. Is this ok?

Yep! I would be happy to test it.

Wow thanks for such a quick turn around. I am traveling without a laptop for a few days, but I will test it soon and let you know how it goes.

Just tested and it's working now. Thank you!


AIMA4e implement Fig 18.8 Cross-Validation-Wrapper from AIMA3e branch

I would like to work on this.


virtctl: update the install guide with the addition of installing via krew.

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind enhancement
What happened:
Issue #912 has been resolved. So now virtctl can be found in the krew index and installed as a kubectl plugin.
What you expected to happen:
A hint that the virtctl binary can be installed via kubectl krew should be found in the documentation. We should update the install documentation.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign

https://github.com/kubevirt/kubevirt.github.io/pull/248

https://github.com/kubevirt/user-guide/pull/255

/assign @fabiand
I created two PRs, I didn't think that updating the already published blog posts would be a good idea. Let me know if I have missed some places.

Added demo PR https://github.com/kubevirt/demo/pull/115

/close


Support metrics endpoint

https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/monitoring.md#metrics-endpoint
It would be great if metrics endpoint flag is enabled and etcd can be monitored by prometheus-bosh-release.

Hi @making
This idea definitely makes sense. Please send us the PR to allow us optionally enable this flag and we will merge it.

Sorry, I misunderstood. The endpoint has been already enabled
I was able to access /metrics
curl --cert /var/vcap/jobs/etcd/config/etcd.crt --key /var/vcap/jobs/etcd/config/etcd.key --cacert /var/vcap/jobs/etcd/config/etcd-ca.crt -k -v https://localhost:2379/metrics

Hi @making
I would like to mention that etcd 3.3 allows operator to specify custom property to disable authentication from /metrics endpoint https://github.com/etcd-io/etcd/issues/8060
This property is not exposed yet. I thought you meant exposing it. We will accept PR for exposing this optionally, so the operator can choose the best thing for their environment.

Ok, I'll make a pr to expose --listen-metrics-urls.
Is there any policy to test the change?


Deleting keys

Is there any way to safely delete keys?

Hi,
I am no exactly sure what you mean by safely and what you mean by delete :smile:
The simple answer is (based on my assumptions of what you mean), in order to safely delete, you should mark the key with a "tombstone" that lets the reader know that it is deleted.
So, for example, if you previously set a key to some value:
gossipmonger.update('foo', 'bar');

Later, if you want to delete it, you should update it to some "tombstone" value:
gossipmonger.update('foo', 'there-is-no-way-this-is-a-real-key-only-deleted-keys-will-have-this-value');

The less simple answer...
Gossipmonger implements scuttlebutt gossip protocol. It turns out that vector-clock scuttlebutt gossip is a special case of a delta-mutation CRDT. The paper where I first learned about delta-mutation CRDTs is here http://arxiv.org/abs/1410.2803 . What this means to me, is that any system using scuttlebutt gossip, is in effect, a CRDT. The "problem" with CRDTs is that they have to maintain the entire history of everything that has ever happened in order to maintain all the property guarantees of a CRDT. Logical deletes are possible, but they're more involved, with the tombstone example above being a.. hacky way of implementing an OR-Set. This means that in order to logically delete data, one must add historical data to the CRDT. Practically, you could chose to truncate the history that CRDT is maintaining, by using some sort of timeout, however, at that point, all of the gossip and CRDT guarantees of the distributed system are lost.

@tristanls Thanks for the detailed reply :)


Added Favicon as per issue #4

Fixes #4
I have added the following favicon to the website

@sk66641 please review and merge this pull request

@sk66641 can you please add the gssoc labels as the leaderboard will only accept the pull request if it contains the gssoc labels in it


Distortion problem of point cloud map

Hi, thanks for your great work!
I encountered some problems when converting the depth map to a point cloud. I used Depth anything to predict the depth of images from the KITTI360 dataset, and when I tried to convert it to a point cloud, I found that it confused me in some details.
The predicted image and its depth map are shown below:

The converted point cloud is shown below:

From the top view, we can see that the road is funnel-shaped, and the road signs also show that they are seriously distorted. The point cloud as a whole shows a distortion that converges towards the center of the camera.
I would like to ask what is the cause of these problems? Maybe the depth map is too dense? Or is it a problem with my camera model? Below is the conversion code I used (pinhole camera model).
image_path = './conver_100.png'

depth_image = Image.open(image_path).convert("I")
depth_np = np.asarray(depth_image, dtype=np.uint16)
depth_np = (depth_np - np.min(depth_np)) / (np.max(depth_np) - np.min(depth_np))

def depth_to_point_cloud(depth_map, intrinsic):
    h, w = depth_map.shape
    i, j = np.meshgrid(np.arange(w), np.arange(h), indexing='xy')

    x = (i - intrinsic[0, 2]) * depth_map / intrinsic[0, 0]
    y = (j - intrinsic[1, 2]) * depth_map / intrinsic[1, 1]
    x = x.reshape(-1)
    y = y.reshape(-1)
    z = depth_map.reshape(-1)

    points = np.stack((x, y, z), axis=-1)
    return points

def visualize_point_cloud(points):
    pcd = o3d.geometry.PointCloud()
    pcd.points = o3d.utility.Vector3dVector(points)
    o3d.visualization.draw_geometries([pcd])

fx = 552.554261
fy = 552.554261
cx = 682.049453
cy = 238.769549

intrinsic = np.array([
    [fx, 0, cx],
    [0, fy, cy],
    [0, 0, 1]
])

points = depth_to_point_cloud(depth_np, intrinsic)
visualize_point_cloud(points)

The point cloud below is what I got with the same code after getting the depth map from the LiDAR point cloud. It can be seen that its geometric performance is relatively good, without obvious distortion

I have the same question. Did you solve this distortion?


Add display_name modification in insights_register.py

Cleaned up #50 due to some conflicts with different histories on my workstations.
Original PR:
As of now, the variable is never uses, making it useless.
I added a check and edited the command accordingly.

Thanks for the update, and no worries for the old PR ;)
In general the change LGTM; a descriptive commit message was lost in the old PR, so would it be possible to amend it? From the old PR, it could be something like:
fix: use display_name variable in insights_register

As of now, the variable is never uses, making it useless.
I added a check and edited the command accordingly.

Thanks!

Done, thank you @ptoscano!

Ah, side note: I'm working on updating this collection for newer versions of Ansible, see PR #56. I needed to do a small change to the default of display_name to be None instead of ''; although I don't think it should affect your code (since it does if display_name:).
Since you use that feature, would it be possible to test that PR, and check it works fine in your playbooks/environments?
Thanks in advance!

tested, it doesn't conflict at all and it works!
btw I created a new PR to overcome the --force-reregister deprecation (#57) :)


Display items

In this pull request, I implemented the following:

When the page loads, the webapp retrieves data from the selected API and shows the list of items on screen with the basic data (e.g. title + image).
Displayed comment button.

Status: ✔ Project Approved ✔
Hi @SelmaNdapanda,
Everything is looking great! Please, merge this branch :shipit:
Congratulations! 🎉
Here are a few optional comment that can be executed at the end to make it look better:

[OPTIONAL] The images are linked as links to the pop up window.
[OPTIONAL] The heart symbol should be on the same line as the title of movie.
[OPTIONAL] The heat shape should be filled when pressed and remain so unless pressed again.


why preprocess_input of applications changes the input data?

for example
from keras.applications.inception_v3 import preprocess_input
y = preprocess_input(x)

but the x has been change too.
I think it is unreasonable.

There has been a debate over this at some point in an issue or a PR. I don't remember where. My personal advice would be to include the preprocessing in the model (much more portable and less error prone).

i think it required a inplace=True/False at least.


Release artifacts rework

Issue Number

N/A
Overview

d953498e2b4c1e16d493f7aaf948ad0bdb57963b
:round_pushpin: Alternate path: download byron binaries in CI

d71374ca87082b6e5b4f11314f99efa726d2efa6
:round_pushpin: adjust names of exported artifacts in Travis

Comments

Bash interpreted $TRAVIS_TAG_macos64 as one variable.

:man_facepalming:  ... Okay then.


No Elasticsearch Node Available

Can you please tell me why I am getting error. panic: no Elasticsearch node available. From java api this works.
client, err := elastic.NewClient(elastic.SetURL("http://localhost:9300"))
if err != nil {
	panic(err)
}

get1, err := client.Get().
Index("emdad").
Type("java").
Id("1").
Do()

if err != nil {
	panic(err)
}
if get1.Found {
	fmt.Printf("Got document %s in version %d from index %s, type %s\n", get1.Id, get1.Version, get1.Index, get1.Type)
}

I think it should be port 9200 for HTTP endpoints. 9300 is for the transport/TCP protocol.

I have found the problem. My Elasticsearch version was old. I upgrade it to 1.7.1 and its works. Thanks


GitHubIssueDetail Closed issues message color

GitHub's experience tiers have switched from displaying closed issues as red to purple (e.g. see our own closed issues https://github.com/badges/shields/issues?q=is%3Aissue+is%3Aclosed), ostensibly to avoid categorically connoting the failed/rejected/etc. theme of red.
I'm wondering if we should do the same for our issue status badge (PRs will remain the same since they already differentiate between merged/rejected) and curious what others think. I confess I'm still on the fence myself, but think I'm leaning towards changing to purple

I'd lean toward changing them too.

Sounds like sufficient consensus to me, will take a look at implementing later today. Do we think this will be one of those things that'll need a query param for toggling between red/purple in order to satiate those preferring to maintain the red?

I think we can just YOLO it


Aspnet core sip handler

#4
Starting the implementation of the SipTcpConnectionHandler
I had to modify the SipParser's Parse method to fit with the buffer stream paradigm of the ConnectionHandler class.
It now returns true/false based on whether it's been given a buffer containing a complete SIP message (complete here meaning the CRLF CRLF was received so we have the whole header and body length matches what was sent in content-length)
I'm not quite done yet but I wanted to open the PR so @jeoffman  can see where I'm going.
I have to sign off now but I'm hoping to finish this out tonight/tomorrow.

Just as a heads up @jeoffman , you probably don't want to merge this quite yet.  I'm going to try to finish out this thought tonight/tomorrow.


Compile error with 0.3.19

Hi, we are getting a compile error that we have not seen before within this library, the error is as follows.
Would you be aware what is causing this and how to fix it? It is causing the nightly Travis build to fail (eg this one https://travis-ci.org/rust-embedded/cortex-m-rt/jobs/510252119).
Thanks!
The errors:
error[E0432]: unresolved import `test::ColorConfig`
  --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/common.rs:20:5
   |
20 | use test::ColorConfig;
   |     ^^^^^^^^^^^^^^^^^ no `ColorConfig` in the root

error[E0433]: failed to resolve: could not find `OutputFormat` in `test`
   --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:102:41
    |
102 |         format: if config.quiet { test::OutputFormat::Terse } else { test::OutputFormat::Pretty },
    |                                         ^^^^^^^^^^^^ could not find `OutputFormat` in `test`

error[E0433]: failed to resolve: could not find `OutputFormat` in `test`
   --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:102:76
    |
102 |         format: if config.quiet { test::OutputFormat::Terse } else { test::OutputFormat::Pretty },
    |                                                                            ^^^^^^^^^^^^ could not find `OutputFormat` in `test`

error[E0425]: cannot find function `run_tests_console` in module `test`
  --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:87:21
   |
87 |     let res = test::run_tests_console(&opts, tests.into_iter().collect());
   |                     ^^^^^^^^^^^^^^^^^ not found in `test`

error[E0425]: cannot find value `AutoColor` in module `test`
   --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:110:22
    |
110 |         color: test::AutoColor,
    |                      ^^^^^^^^^ not found in `test`

error[E0412]: cannot find type `TestFn` in module `test`
   --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:272:75
    |
272 | pub fn make_test_closure(config: &Config, testpaths: &TestPaths) -> test::TestFn {
    |                                                                           ^^^^^^ not found in `test`

error[E0063]: missing field `exclude_should_panic` in initializer of `test::TestOpts`
  --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.3.18/src/lib.rs:98:5
   |
98 |     test::TestOpts {
   |     ^^^^^^^^^^^^^^ missing `exclude_should_panic`

error: aborting due to 7 previous errors

Some errors occurred: E0063, E0412, E0425, E0432, E0433.
For more information about an error, try `rustc --explain E0063`.
error: Could not compile `compiletest_rs`

Hi, that looks a little odd, if I understand your CI setup correctly, you only use compiletest with nightly?
Could it be that nightly has changed an we just haven't caught up yet?

The nightly build is broken right now due to the libtest changes, and we can't fix that without breaking the stable build.

Looks like it's related to #162 (see this comment).

@Manishearth Can't we just roll the libtest changes back? Why would that break the stable build?

No, rustc's libtest changed, we can't roll that back. It's part of a larger set of changes; @gnzlbg is working on it.

The fix is to use the libtest crate, however that crate doesn't work on stable right now, so we can't upgrade to use it.

Can you please explain why that is the fix? Was 0.3.19 broken prior to #162 being merged?

No, rustc's libtest changed, we can't roll that back.

Oh, I didn't read your comment closely enough, I think. How about a split down the middle, using libtest for nightly and the tester crate for stable?

#162 doesn't matter, we yanked that release
0.3.19 was broken on nightly prior to that being merged, I opened #162 to fix it, but the fix was broken.

Oh, I didn't read your comment closely enough, I think. How about a split down the middle, using libtest for nightly and the tester crate for stable?

We could except rust features don't let us turn off crates, only on. One suggestion is to add an unstable feature instead, but I was planning on just waiting for gnzlbg to make the changes

#162 doesn't matter, we yanked that release

Then I think we should just revert those commits. It's not even tagged, so it should be fine.

We could except rust features don't let us turn off crates, only on. One suggestion is to add an unstable feature instead, but I was planning on just waiting for gnzlbg to make the changes

Can we do both? Have both stable and nightly flags, and have default-features set to nightly. IIRC, you should be able to do:
[dev-dependencies]
compiletest_rs = { default-features = false, version = "*", features = ["stable"] }

(I'm probably missing some technicality here, but) it should allow us to continue the 0.3.x series for both nightly and stable.


Pass Django user when resource instantiated instead to every function call

I previously made a change to how resource.get_fields() is called by import_action() to take **kwargs, so that get_fields() would be able to change its behaviour depending on the Django user, like was being done by the call to resource.import_data().
I'm thinking now that it is cleaner to pass **kwargs to the constructor for the resource object instead, on the line resource = self.get_import_resource_class()(). It's clearer for the resource to get the user in its __init__ and store it, than to have the user passed as a parameter to every function call the resource has.
I'd be happy to make the change if you agree!

Hi @shaggyfrog. Yes, I agree. As we did not make a new release I think we can just revert changes in
https://github.com/django-import-export/django-import-export/pull/453/files


ERROR: Failed building wheel for mmcv-full

when using
mim install mmcv-full -f https://download.openmmlab.com/mmcv/dist/cpu/torch1.10.0/index.html
I am looking to setup a space on huggingface spaces(https://huggingface.co/spaces) for this model at https://huggingface.co/spaces/akhaliq/RealBasicVSR
I was able to get the model working in colab but the space does not support cuda, is there a way around this thanks?
space code: https://huggingface.co/spaces/akhaliq/RealBasicVSR/blob/main/app.py#L4

Hello, thank you for your efforts  to set up a space for RealBasicVSR.
mim will automatically determine the appropriate version based on your pytorch and cuda version. Therefore I think simply mim install mmcv-full should be fine. Could you have a try and let me know if it works?
Thanks again.

Oh my bad. Do you mean huggingface does not support cuda?

@ckkelvinchan Each Spaces environment is limited to 16GB RAM and 8 CPU cores
https://huggingface.co/docs/hub/spaces

@AK391 mmcv-full can only be installed with CUDA. If CUDA is not available, you can use mim install mmcv. It lacks CUDA ops but it shouldn't affect RealBasicVSR. Please have a try.

@ckkelvinchan thanks, I tried that now getting
Traceback (most recent call last):
File "inference_realbasicvsr.py", line 10, in
from mmedit.core import tensor2img
ModuleNotFoundError: No module named 'mmedit'

@ckkelvinchan thanks it is working now, opened a PR here https://github.com/ckkelvinchan/RealBasicVSR/pull/30


Check if we have shards where less followers than planned are in sync.

Just added analyze script to check if we have some shards where not all planned followers are in sync.

There is no repair script for this job, it is to collect information.
A healthy cluster will repair itself overtime.

please let's have this reviewed / merged asap so we can use this in support . thanks


Refactor views and schemas in backend

Just some refactoring...

[x] rename some files
[x] create a schema loader that can be used everywhere instead of implementing it in the validator.

Current dependencies on/for this PR:

main

PR #73

PR #74

PR #75   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #73

PR #74

PR #75   👈

This comment was auto-generated by Graphite.


How can external workflows be authored in a way that makes sure best practices are followed?

Release workflows will need to describe what actions should take place when its time to release the product.  These will be run only as often as releases occur.  How can workflows be authored in a way that makes them durable to changes in OpenSearch-Build repositories changes as well as getting notified about sensible workflow changes to make before the next release?
Acceptance Criteria:

[ ] Determine the smallest size a workflow can be to limit the number of practices that need to be enforced.

An example should be created in the documentation

[ ] Define when code should be authored externally vs at the central repository.

E.g. a new docker image creation workflow is created, should that be done in the client repo or in the shared repo?

[ ] Define ci-checks that teams should follow

On high level the Jenkins libraries to release the artifacts will be located in opensearch-build repo similar to other jenkins libraries https://github.com/opensearch-project/opensearch-build/tree/main/vars . In this way we can reuse the existing codebase (like for signing).
However in case of JenkinsFiles and its regression test files we have two options: (Need to choose one)
Option 1:
The jenkins files and its regression test files for each component will be situated in the respective component’s github repository.
Pros:

In this way the component owners have the control over their release workflow
Branching is supported out of the box, meaning components can incrementally improve their processes as per the version
No dependency on code owners for code approvals, etc.

Cons:

The code change across the repositories can cause breaking changes:

Jenkins Lib is changed in opensearch-build repo: Since the regressions files for all client’s workflows will be situated in their own repository, the changes won’t be detected until very late when some CI runs the workflow in client’s repo
JenkinsFile changes in component repo: The change needs to be conveyed to build repo code owner team in case of any variable, secrets addition, infrastructure change (like new agent requirement), etc.

Option 2:
Everything is located in opensearch-build repo.
Pros

One place for all jenkinsFiles, libraries and respective tests
The changes need not be propagated across the repositories

Cons:

Component team has less control over the release workflow
Chances of build repo code owner team being the bottle neck for code approvals to release workflows (which is okay since the infrastructure is owned by them so any breaking change can be detected in the review)
Component team would not be aware of any changes in the central jenkins libraries that affects component’s release workflow too.

Because of branching I would choose option 1. I think the Jenkins library should be split up in smaller parts (e.g. signing), each part versioned, and released as a standalone component, similar to GitHub Actions, to solve the cons.

I am not sure that is doable or not @dblock . Was researching regarding jenkins shared libraries and all the content I found is asking to use the git branch, tag or commit as version.

Researched a bit on pulling different versions for remote jenkins shared libraries, and found this plugin https://plugins.jenkins.io/workflow-cps-global-lib-http/
Discussed with @prudhvigodithi, we can publish the zips to maven or any other artifact repository (s3 works as well) for different versions and accordingly pull the required version.
Few things to keep in mind:

Jenkins Shared Library versions will be independent of opensearch and other product's version. (should we separate this out into new repository?)
Version release notes, change log needs to be maintained to let the consumer of this library know what changed and if the next version can cause any breaking changes.
Versioning should follow semver

Determine the smallest size a workflow can be to limit the number of practices that need to be enforced.
Will be added as a part of documentation issue

Define when code should be authored externally vs at the central repository.
See Option 1
The cons can be resolved by 1. versioning the shared libraries. 2. Adding OSEE team to approve PRs wherever required.

Define ci-checks that teams should follow
All jenkins libraries as well as jenkinsFile should have regression test.
More details on how to add them can be found here https://github.com/opensearch-project/opensearch-build/blob/main/DEVELOPER_GUIDE.md#regression-tests
More details will be added during documenting everything. See documentation meta issue https://github.com/opensearch-project/opensearch-build/issues/2493


[Question] Property for limiting items to display

Hi guys,
Is it possible to have a property to limiting the number items you want to display ?
I use that control to display several images and I want to have only two images display and the rest below and accessible by scrolling down.
Kinds Regards,
Tenjim
Describe your issue
A clear and concise description of what your issue is. If you think that it is a bug, please provide the steps to reproduce (e.g. a code snippet or screenshots).
Version Info
Package Version [e.g. 1.5.2]
.NET Version: [e.g. .NET Framework 4.7.2]
OS Version: [e.g. Windows 10 Build 19041.508]

Hi,
I don't understand what exactly you are trying to achieve. Could you give me more information on how the scenario you describe would look like.

Hi @sbaeumlisberger,
I use the VirtualizingWrapPanel with images as itemsource. I just want to know if there is a way to limit the number of object with a property (here images) to be displayed and not virtualized.
For example: I want to display only two images and scroll down to display the next two images from the itemsource.
Thank in advance,
Tenjim

Hi Tenjim,
the items (in your case images) that fit into the viewport (the area visible in the UI) are always realized. For items not yet visible there are different caching settings. See here and here for some documentation. You can also have a look into the sample application, there you can try it out. If you have further questions about caching feel free to ask.
If I misunderstand you and you want to achieve something else, let me know. In that case it would be helpful if you could provide a sample or mockup so that I can understand you scenario.

Closing this issue now because of inactivity. If you still have a problem please let me know.


Update dialects to use the folder adaptors

This is just an NFC change, and the folders should be cleaned up
afterwards.

@atondwal Looks like this can only be merged after an LLVM bump, so I'm handing this off to you since you're oncall this week.

Hi @tpopp! Thank you for the PR. May I ask you to rebase it on top of HEAD, so that it can successfully pass CI?


Slack/Discord + Flare Integration

Is your feature request related to a problem? Please describe.
I don't know if it is possible, but it would be very cool if you could connect a Flare bot to Slack so that you can do / commands in slack such as /hours which could show you how many hours you have or any other things that might work.
Describe the solution you'd like
A pilot being able to easily view certain things from Flare directly in Slack.
Describe alternatives you've considered
The pilot opening Flare and looking for themselves.
Additional context
Once again, this would be cool but it's not a top priority!

Could also be used for Discord through the Discord Slack Webhook format. See https://discord.com/developers/docs/resources/webhook#execute-slackcompatible-webhook.

Anyway to add Event notifications?

@Velocity23 , anyway to do the above and/or adding the pilot's callsign when Slack gets a notification when a pilot files a flight?


Vulnerability roundup 105: glances-3.1.7: 1 advisory [9.8]

search, files

[ ] CVE-2021-23418 CVSSv3=9.8 (nixos-21.05)

CVE details
CVE-2021-23418
The package glances before 3.2.1 are vulnerable to XML External Entity (XXE) Injection via the use of Fault to parse untrusted XML data, which is known to be vulnerable to XML attacks.

Scanned versions: nixos-21.05: ad6e733d633.
Cc @jonringer
Cc @k0ral
Cc @primeos

This was resolved by #134200


could use some help

@Acekorneya  I took your excellent worked and made my version of it, Removed most of the extra steps I could find, cleaned up the cli and sorted them out into functions.
I've tried to set it up so it can be more easily repurposed for other games.
This is working but I am not to familiar with ARK/Steam so I m struggling how to resolve the RCON setup, no matter what I do I can't seem to get the server (inside wine) to start listening on the RCON port, so the monitoring is suspect (but good enough for my purpose).
You can find my build here.
Any help or advice would be appreciated, I did this for my own personal use and I don't plan on maintaining this build so feel free to take anything from this that suit your purpose, after all the hard part was done by you.

sorry for the super late reply was working a new version of my script which i just release recently lol..  but did you ever get it working?

np I when through the whole code and clean allot of stuff up I may want to look through my setup and use some of it.


travis: remove rustfmt check

rustfmt just gets upgraded and its format changes again. Let's remove the check for now, we may need to figure out a long-term policy later.

@sre-bot try

LGTM

@sre-bot try

@sre-bot r+

@sre-bot retry

@sre-bot retry

@sre-bot retry

@sre-bot clean then retry


Fix MMTF writer for non-biological systems

The code made a few assumption on what data was present in the frame, assumptions only matched by files containing biomolecular systems.
In particular, atoms are not guaranteed by chemfiles to be ordered inside residues, and not all atoms have associated residues.
@frodofine I would appreciate if you could give this a look! If you don't have time, that's fine, just let me know 😃

And fix #287 !

I'll try today!

I've a few performance improvements that I should be posting soon and I need to reading to work as before.

This is nice to hear, I'll be waiting for them! I want to do a 0.9.2 release very soon, do you want your changes to be in or can they wait for the next one?

They can wait. I'm swamped until the 15th.

Ok, thanks for the review, and good luck until the 15th!