Adding single option to mathcomp-analysis

Description of changes
Add single option to mathcomp-analysis (similar to the one in matcomp)
Successfully tested on https://github.com/coq-community/coq-nix-toolbox/pull/122

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@CohenCyril this seems ready

Hi @vbgl please merge


Authorization page could not be loaded

Starting this past Friday (8/17/18) I've been getting this error when trying to log in using a social provider (Salesforce). The login window appears as if it's working, then it closes and I get this error in the console. I've attached a link to a video of this issue as well
Error: Authorization page could not be loaded.    at Object.callback (chrome-extension://eaonkakpkhppgedffnpohgknnhmclllk/node_modules/auth0-chrome/dist/auth0chrome.min.js:1:1850)    at safeCallbackApply (extensions::uncaught_exception_handler:27:15)    at handleResponse (extensions::sendRequest:67:7)

Hey, I was just having this issue, double check that you're using the correct extension ID in your Auth0 settings.

Hey @killa-kyle  have you had time to check what @mdbiundo  suggested?

@beardaway I did check. my configuration was correct at the time. the issue resolved itself days later, but not before I had written my own auth as a backup.

Thanks a lot for letting us know! I'll close this one as we gradually go through PRs and issues and will step by step implement fixes and improvements suggestions.

Also getting this error. Double-checked my extension ID in Auth0's application settings.
Does the extension need to be published beforehand (since it needs to access <extension-id>.chromiumapp.org)?

Turns out the callback URL had to be *.chromiumapp.org (no s)...

Thanks a lot for detecting that! I just merged your PR.


Example links broken

The example link https://github.com/rjackson/pivot.js provided in documentation returns 404.

I believe these were fixed in a commit I made a while ago? Either way, the new link should be https://github.com/rwjblue/pivot.js because the repo owner changed his username.

@sharjeel @augbog Yeah, GitHub does a great job of redirecting most things, but the links on the demo site are hardcoded to rjackson and he's since changed to rwjblue (to match his twitter).


[Newsletter] Support newsletter resubscription after unsubscribing using DOI

1. Why is this change necessary?
Right now, when using DOI for newsletter subscriptions, the user can't resubscribe.
2. What does this change do, exactly?
When DOI is confirmed, internally the confirmedAt field is set.
Now when subscribing a user, if the user already exists and has a confirmedAt set, then the request is aborted.
That is however incorrect in the current implementation: because we never reset the confirmedAt when unsubscribing the user, an unsubscribed user will also early exit here and can't resubscribe.
3. Describe each step to reproduce the issue or behaviour.
See the test case:

subscribe
confirm DOI
unsubscribe
try to resubscribe

4. Please link to the relevant issues (if any).
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

A different solution would be to reset the confirmedAt field when unsubscribing, however this field might be relevant if looking up old users ("when did I approve the confirmation? Oh, at 5th of May, ok")
A second solution might be to add a task that continuously removes opt-out newsletter subscriptions, but that might be unwanted for certain installations.

The test is pretty slow right now, as we need to hit 4 API endpoints consecutively.. Any tips on how to speed that up are highly appreciated.

Hey @apfelbox, you dont actually have to make 4 API calls, instead you could jut put an user who has been registered with double opt-in into the database at the beginnig of your test.
Also i'm currently waiting for an answer regarding the confirmedAt Field.
I will come back to you once i have an answer if we can just overwrite the field.

Ah, I wasn't sure on whether it is fine to directly insert these rows via query.
I will wait for your feedback regarding just resetting confirmedAt and then adapt the PR accordingly. 👍

I tried inserting the entry via an SQL statement, but I would need to fetch a lot of different IDs ( language_id, sales_channel_id), so it would be quite a lot of code. Is there some helper for that?

There is the IntegrationTestBehaviour which includes the BasicTestDataBehaviour so you can just use e.g. this->getValidSalutationId.

@J-Rahe thanks for the tip, I think the implementation is quite nice now.
Everything should be fixed 👍

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-15823
Please use this issue to track the state of your pull request.

thanks for your contribution @apfelbox   👍🎉💙


Preventing the multiple reddit background tasks bug

This PR closes #359
It prevents multiple background tasks for the new post poll and weekly top posts poll by saving a reference to task as a class attribute and checking if one already exists when creating one. The reason this needs to be done is because the on_ready event fires every time the bot reestablishes the connection, not just on the initial start of the bot.

An exception in a task shouldn't kill the loop, although it will complain if you never retrieve the exception from the Task after a while. I don't think that kills the loop, though.

If one were to explicitly write  raise Exception within the while loop, it would make the task "done" and the exception could be accessed with task.exception(). I am just not sure how exceptions from discord.py would be handled cause they have their own error handler system to which exceptions may get redirected.

It's just adding an extra if task.done() check so it wouldn't hurt to be safe and add it unless we can be sure of the behaviour to expect.

Ah, yes. Does this have a relevance to the current bug?
The bug is not related to an exception in a Task itself (they run fine), but that multiple tasks get created when the bot reestablishes a connection after it has been dropped. This causes the on_ready to fire again and, in the old version of the code, this would create duplicate tasks running concurrently.

No, it isn't directly related. However, as a consequence of this patch, if the task does prematurely finish for whatever reason, the task will never get re-created.

Actually, even with what I propose it would only cover cases of when the task is prematurely ended as a result of the bot disconnecting. If, for example, there is an error with the reddit API, that would not be covered, so there's some underlying issue with the implementation of the tasks. Perhaps it's beyond the scope of this PR after all.


Fix install compilation. Bump Cargo version.

Fixes strange error with cargo compilation in install command.

I merged a prior PR that bumped to Cargo 0.29 and had worked on an immediate follow on to bump to Cargo 0.37 which is now merged. See if that resolves your issue.

@cardoe Yes, it fixed my issue.


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1250 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 8bdca17 after 26 minutes.


Issue when loading samples derived from intermediate AmpliconSuite entry-points

When samples are run starting from an intermediate point (e.g. one stage for CN, one stage for AA), the copy number file will reflect only
a small subset of the genome. While the complete copy number file may exist in the collection of files, the one that is referenced in sample_data is the partial data used as a jumping-off point for the second-stage of a run.
This causes sample_plot.py to crash, as it expects complete copy number data.
This is an issue we should handle during sample aggregation, by searching in the expected location for the full copy number calls and linking those.
I am documenting this issue in the AmpliconRepository page so that others can be aware of what this issue is caused by if they encounter it with additional testing data that will arive, and will close the issue once we have addressed the root cause in the aggregation module.
We will need to also update sample_plot.py to handle the case where missing or incomplete CN data is present, as this same issue will occur. I will create a separate issue, or just issue a bugfix outright.

Added updates to sample_view_plotly branch. Can now load projects with missing or incomplete CN data. Tested partial and all-chromosome views.
Will update aggregator next.

A PR request is now open in the Aggregator module github that completely resolves this issue, and attempts to re-identify the original CNV bed file.


Add vagrant config

Addresses Issue #11
Creates two vagrant boxes—"deploy" and "target"

deploy - salt master and deployment_server
target - deploy target

example configuration is the same as in the README.md of the repo: it
checks out mediawiki-core in /srv/deployment/mediawiki/slot0
All dependencies for deploy should be setup via the states/deploy.sls

Apache git server
Redis returner server
TrebuchetTrigger from pip

Nice. Just a minor nitpick. Merging in.


Error: Type 'double' is not supported by blueprint

Hi, everyone!
I building Cesium for Unreal v 1.15.1 in Ubuntu 20.04 for Unreal Engine 4.27.
Wen i create package
with
"$UNREAL_ENGINE_DIR/Engine/Build/BatchFiles/RunUAT.sh" BuildPlugin -Plugin="/home/user/workspace/cesium-unreal/CesiumForUnreal.uplugin" -Package="/home/user/workspace/packages/CesiumForUnreal" -CreateSubFolder -TargetPlatforms=Linux
i have many errors as
CesiumForUnreal/packages/CesiumForUnreal/HostProject/Plugins/CesiumForUnreal/Source/CesiumRuntime/Public/GlobeAwareDefaultPawn.h(157) : LogCompile: Error: Type 'double' is not supported by blueprint. FlyToLocationLongitudeLatitudeHeight.YawAtDestination
etc.
How is possible to fix this problem?

Hello, those look like issues when using UE4 with the default ue5-main branch. If that's the case, then switch to the ue4-main branch.

Thank you!


Fixed range docstring to correspond to code

Docstring says it does from a to b, but code pops last, then first, then string. Fixed docstring to correspond to code.

lgtm


swap satori's uuid package to gofrs

This updates the sample bookshelf app to use the community recommended gofrs uuid package.
go.uuid is effectively unmaintained and contained at least one critical security flaw which took half a year to be merged as the owner isn't active. See https://github.com/satori/go.uuid/pull/75

I signed it!

Thanks, great catch.

Thank you! Would you mind updating appengine/gophers/gophers-6/main.go and appengine_flexible/analytics/analytics.go, too?

Definitely! Not sure I missed them initially. Must have grepped inside of getting-started.


Introduce data model change to add achievements

The goal of my bachelor thesis is to integrate gamification with achievements into Artemis (Proposal on Confluence page).
Achievements should be introduced as a new entity with properties as seen in the model down below: a title, a description, an icon, and a rank. The title and description are used to describe the achievement while the icon would be used to display an achievement in the browser. The rank should represent the degree of difficulty of attaining an achievement, gold could represent the highest level, silver and bronze the levels below. For example "Score a 100%  in an exercise" would be golden rank. At the beginning the few achievements should be predefined, but later there could be the possibility to add more new ones.
Now a user can receive several achievements while using artemis throughout the semester and an achievement may have been received by several users. For example user A can hold achievements "Score a 100% in an exercise" and "Hand in your solution within the first day of the exercise´s release" while user B holds achievements "Score a 100% in an exercise" and "Hand in 3 submissions in a row". So in this case it would make sense to link the achievement entity to the already existing user entity by a many-to-many mapping. Also the user should be able to access the list of the achievements he/she earned so far.
For the courses the link to achievements is quite similar: the courses should be able to give an overview of which achievements can be achieved within the course. Also in one course several achievements can be achieved by users and one achievement can possibly be achieved in several courses. For example course A could contain achievements "Score a 100% in an exercise" and "Hand in your solution within the first day of the exercise´s release" while course B contains achievements "Score a 100% in an exercise" and "Hand in 3 submissions in a row". So this link can also be modeled by a many-to-many mapping.

To realize this on database level, we need a new table achievement, which persists all achievements contained in Artemis with their attributes and a table user_achievement which links achievements to users by their respective IDs. Once again the same goes for courses only that the connecting table for that would be course_achievement with columns course_id and achievement_id.

Alternative Options

Omit the display of all accessible achievements in a course to make it more like a hidden easter egg hunt.
Link achievements to exercises to be able to show which achievements can be achieved on an exercise level.
Link achievements to participations, submissions and/or results to have further information about the submission/participation/result by which the achievement was obtained.

Sounds good. I like in this approach, that you also have the opportunity to create Artemis wide achievements, like "ask your first question" or "complete the artemis tutorial", which are not necessarily bound to a course. But I don't really get the rank property of achievements. I mean the achievement itself determines the difficulty. Why is the rank necessary, what is the concept there?
Another question: How was your decision making to go for achievements? Why are we not doing levels or a leaderboard?

While the ranks are not essential, they are a nice way to highlight the different difficulties visually. Also they add the possibility to implement leaderboards based on who has the most golden achievements and so on.
Since @sjagla27 and me both applied for the topic of integrating gamification, we split it up. He is responsible for the leaderboards while I take care of the achievements.

I agree with the achievement model proposed, it looks good to me. The user-achievement link is clear and I especially like the association with the course as this can leading to "reusing" achievements for different courses.

I like your proposed changes and the way you presented them. It doesn't change existing entities and can be extended by links to other entities as described in alternative options. Good job!
How are you planning to check, if an user has earned an achievements? For the example 'Score a XX% in an exercise' XX could be 80 for bronze, 90 for silver and 100 for gold. This mapping must be defined either in code or in the database. Do you think it would make sense store these kinds of conditions in the database or do you need more flexibility for more creative achievements?

The checks can be very different depending on what kind of achievement we are talking about, point based (reach 80% of the points), time based (submit your work within 3 days) or progress based (work on your first exercise, hand in three submissions in a row). My approach would be to check certain parameters of the submissions/results mainly after the assessment. Regarding your suggestion for checks on point based achievements that was quite my idea 😄 And I would define criteria like this in code for more flexibility, but it could also make sense to store them in the database.  So that´s actually a good point.

It is not clear to me why you need a many-to-many relationship between courses and achievements. No example in your text affects multiple courses.
Instead your examples reference exercises which are not linked at all to the achievements.
I suggest to use a many-to-one relationship between achievement and course and between achievement and exercise. Both of them could even be unidirectional because we will always navigate from a user to his/her achievements and are then interested in a course.
The navigation from a course (or exercise) to many achievements is typically not necessary (but would also be possible then).
Also I suggest to add more details about the different use cases / features related to achievements. When do you calculate that a user receives an achievement? where do you display it? These is important information to understand how this should be modeled in the database.
Your proposal above does not include enough information!

I suggested a many-to-many relationship between achievements and courses because I considered that it should be possible to get the same achievements in different courses. My concrete examples may not have been the best to illustrate this. Nevertheless it would probably make sense for a many-to-many relation to store the course in which the achievement was received in the achievement in order to be able to distinguish the achievements from each other if a user received the same ones in different courses.
So it might actually be easier if we simply assume that an achievement can only occur in one course and therefore use a many-to-one relation as @krusche suggested. In other words, in two different courses there could be achievements with the exact same properties but without them being the exact same instance (das Gleiche nicht dasselbe).
The navigation from a course to many achievements would make sense if we want to display all achievements within a course as a list so that the user can see which achievements he has earned already and which ones not yet. By displaying all achievements we could trigger the impression of autonomy in students by giving them the possibility to decide for themselves which achievements they want to attain and in which order (how should you be able to decide if you don't know the options?). And autonomy is one of the basic demands of intrinsic motivation. So while of course not being necessary this could still be a useful feature.
The calculation whether someone received an achievement depends on the criterion for the achievement. Most of it I would check if the result is calculated, because then we can check for example for scored points or the submission date. I'm not sure if it makes sense to check the submission date earlier, because I would not give an achievement for an empty or completely wrong submission, even if it would have earned one just by looking at the submission date. If there are achievements like "give a correct answer" (in the questions and answers feature) or "participate in your first task" I would award the achievement when the answer is marked as correct or when the participation is started.
As mentioned earlier I would display a list of all achievements in a course in its dashboard with the achievements not yet received grayed out and the earned ones in color. Otherwise it would be an option to implement an overview page for the user, where he can see all his achievements e.g. grouped by courses.

Here is the updated data model:


Working babel, react, and webpack configs.

Updated modules + webpack + gulp configs.
Converted Controller to use React components.

@bpan make sure to run npm install or yarn install after you checkout this branch; i've updated webpack and gulp (and related loaders/modules) to the latest versions.


CoTaskMemAlloc function

Link to article:
https://docs.microsoft.com/en-us/windows/desktop/api/combaseapi/nf-combaseapi-cotaskmemalloc
Problem:
DLL. It's in combase.dll, not in Ole32.dll. At least on my Windows 10.

@ginami Can you follow up?

Hi @Const-me -- thank you very much for your feedback; we're looking into the issue.


Add all users and last message to currentUser.rooms object

I maybe a bit unfamiliar with chatkit APIs, it seems like there is no way to see all users in currentUser.rooms and no information on last message.
This is commonly used in a chat feature like Facebook, when click on the chat icon, you see a list of rooms with users and last message.
Is this feature not currently supported? Then what do people typically display then?
Here is  a list of things I am running into:

cannot get users, and last message from room obj
limit to 1000 rooms, but no easy way to know which rooms are being used on the server side, if we are building a chat app, we need a way to purge rooms
naming a room A:B is really not a great solution for the long term, you end up searching for rooms both A:B and B:A
chatkit.getRoom only takes id, so there is no way to search for things by name.

It seems like the current solution is to subscribe to all rooms, and get those information from each room.
I personally think it makes more sense to get those information from currentUser, and use onRoomUpdated.

well, no point opening this anymore :(


Importing dates and timezone problems

I was working through some import problems and noticed that dates on the first of the month were being imported as the last day of the previous month.  My lucee app server is set to EST, the AWS server on which it is running is UTC, so off by -5 hours.  When I switched the lucee timezone to UTC, the dates were imported properly.  This threw the data off by quite a bit - not sure where the problem lies, but I suspect it's java date/time translation.  Lucee stores date objects as datetime, which isn't good.

Are you definitely running the latest version?
There was a similar issue recently caused by the Lucee timezone being different from the OS. There was a fix in version 3.1.0 to address that.
If you are on the latest then can you provide more detail on how you're doing the import?
This simple test seems to work OK even with a deliberate TZ change/mismatch.
//stash TZ
stashedTZ = GetTimeZone() //My system is GMT/UTC
SetTimeZone( "US/Eastern" )
result.tempTZ = GetTimeZone()

data = QueryNew( "Date", "VarChar", [ [  "2022-01-01" ] ] )
datatypes = { date: [ "Date" ] }

spreadsheet = New github.spreadsheet.Spreadsheet()

wb = spreadsheet.newChainable( "xlsx" ).addRows( data:data, datatypes:datatypes )
result.value = wb.getCellValue( 1, 1 )
result.excelInternalValue = wb.formatCell( { dataformat: "0.0" }, 1, 1 ).getCellValue( 1, 1 )

dump( result )
/*
tempTZ: US/Eastern
value: 2022-01-01
excelInternalValue: 44562.0
*/
//reset
SetTimeZone( stashedTZ )

Yes, I'm on 3.3.0 everywhere.
In my case, the system is GMT, the lucee instance is EST (-5) and I'm importing data from an xlsx file.  These dates, when inserted into the database, end up as 10/31 instead of 11/01:  https://www.screencast.com/t/6HAoklvq  I've seen Lucee import dates with timezone info (-5 in this case) so I tried inserting everything as ex: '2019-11-01 12:00', which, if the date had imported correctly should have inserted 12PM - 5, or 7AM, but still on 11/01.  It did not work.  Only changing Lucee to UTC temporarily did.  This tells me that the conversion happened on import rather than export.
In the end, this is a Lucee issue - they should not be using datetime objects for dates.  Something similar happened here:  https://luceeserver.atlassian.net/browse/LDEV-3751.

Sorry I should have realised "import" meant you were using read().
I can reproduce the issue with format="query" which as you say points to the way Lucee handles date objects. Just reading the file into a workbook object and using getCellValue() seems to give the correct date regardless of TZ setting.
Could you try adding queryColumnTypes="auto" to your read() call? i.e.
importedData = spreadsheet.read( src=yourfilepath, format="query", queryColumnTypes="auto" );

Warning: it might be slower, depending on how big the file is.

I could, but I don't use it like that.  I do query.each( (row)=>{} ) then row['columname']  Does that work the same way?  I used a workaround to get past this problem last night and it would take a lot of work to undo it in order to test.

Actually forget queryColumnTypes, it was a red herring, sorry. Thought I saw a result but then couldn't replicate it.
I think I've located the problem which is that the read() query handling extracts date values from cells in a different way to getCellValue(). They should do it the same way really but I'm a bit apprehensive in case it proves a breaking change. The test suite does break but that might just be the way I've set up the tests.
I'll need to give it some thought and get back to you.

Sounds great.  Thank you.

I'll find a way.  It would be good to get this fixed.

Rather than risk breaking things, I think I've found a simple and safe way of addressing this by applying the same fix as the previous issue I mentioned.
So when evaluating a date value, the library will now check for a Lucee/POI timezone mismatch (POI uses the system TZ), and temporarily making POI match Lucee - not the other way round as you've been doing.
Can you give the develop branch a try and report back?

@Daemach Did this fix work for you?

I haven't had a chance to test it, and I don't think I'm going to be able to get to it soon :/  Deep in the middle of sorting out some bad accounting system data.  I would say roll it, and if it comes up again we can deal with it then.

That's fine, I was just curious. I'll leave it on develop and won't cut another release specifically for it unless I hear from you.


Deprecate support of inline volumes

Is your feature request related to a problem? Please describe.
Need to deprecate inline volumes support. Generic ephemeral volumes must be used instead.
Describe the solution you'd like
No support for inline volumes will be provided.
Describe alternatives you've considered
Generic ephemeral volumes.
Additional context
n/a

/ci


local function-worker use localhost broker service url

Motivation
If pulsar service starts broker and worker in the same process then worker should not use broker-advertised address but use localhost address to connect.

rerun java8 tests


Improve the new-version detection code so that it correctly checks values.

Right now it just checks for a diff which wrongly highlights a new version if we're running a beta that's ahead of the official releases:

In addition, if it's previously checked, it appears to always indicate the older version; I think it's picking up the requests cache.


python3Packages.automx2: init at 2022.01

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build python310Packages.automx2 python311Packages.automx2

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

4 packages built:

python310Packages.automx2
python310Packages.automx2.dist
python311Packages.automx2
python311Packages.automx2.dist


Plugin doesn't utilize yapf correctly

I'm attempting to format some Python using yapf and the output does not appear correct.

I installed yapf via pip; all successful, no errors.
I'm using vim 8.

I'm using a test Python file that looks like this:
x = {  'a':37,'b':42,

'c':927}

y = 'hello ''world'
z = 'hello '+'world'
a = 'hello {}'.format('world')
class foo  (     object  ):
  def f    (self   ):
    return       37*-+2
  def g(self, x,y=42):
      return y
def f  (   a ) :
  return      37+-+a[42-x :  y**3]

When I run yapf via the command line:
yapf test.py

...I get PEP8-compliant Python that looks correct:
x = {'a': 37, 'b': 42, 'c': 927}

y = 'hello ' 'world'
z = 'hello ' + 'world'
a = 'hello {}'.format('world')

class foo(object):
    def f(self):
        return 37 * -+2

    def g(self, x, y=42):
        return y

def f(a):
    return 37 + -+a[42 - x:y**3]

However, when I utilize the plugin (:Autoformat), I get Python that looks like this:
x = {
    'a':
    37,
    'b':
    42,
    'c':
    927
}

y = 'hello ' 'world'
z = 'hello ' + 'world'
a = 'hello {}'.format(
    'world'
)

class foo(
        object
):
    def f(
            self
    ):
        return 37 * -+2

    def g(self,
          x,
          y=42
          ):
        return y

def f(
        a
):
    return 37 + -+a[
        42
        -
        x:
        y
        **
        3]

For what it's worth, vim's messages report this:
Formatter autopep8 has errors: b'/bin/sh: autopep8: command not found\n' Skipping.
Failing config: 'autopep8 - '
Definition in 'g:formatdef_autopep8' was unsuccessful.
Definition in 'g:formatdef_yapf' was successful.

I pushed a fix for this. Can you check whether this solves it for you?

@Chiel92: Looks fixed! With the new update, I see the format that I should:
x = {'a': 37, 'b': 42, 'c': 927}

y = 'hello ' 'world'
z = 'hello ' + 'world'
a = 'hello {}'.format('world')

class foo(object):
    def f(self):
        return 37 * -+2

    def g(self, x, y=42):
        return y

def f(a):
    return 37 + -+a[42 - x:y**3]


VMD error: Uncaught TypeError: $ is not a function

Tried loading it via bookmark and in the console, for both ways I get this message.
In this particular site jQuery is running under the jQuery and not $. Any way to get around this?

Not entirely sure - having a quick look at the code doesn't show anything obviously using it untested - if it still happens on the latest (unminified) v1.3.0 can you give a line number for me to check out :-)

I went with a different route, but thanks for checking it out.


Create Test Plan for Translation Template Updates

User Story

As ReportStream,
I want to have a test plan I can refer to that specifies what/how/where to test transform template changes
so that I can ensure transform template changes:

do not break existing senders/receivers
behave as expected, especially around edge cases

Description/Use Case

Presently, RS engineers maintain sample input/output files for each onboarding initiative on the UP that we work on. This may be fine as part of a test plan, but it is not a great way to test individual mappings as specified in the mapping inventory. The present tests could be thought of as "functional" tests (I'd call them sender verification tests and receiver verification tests), where as we probably ALSO need unit tests for each mapping that we create. If we say that we do, then we need to define how they are structured and organized.
There is a design note for managing translation templates that indicates how sender and receiver validation tests can be run and managed, we should look to adopt that as part of this work.
Risks/Impacts/Considerations

Dev Notes

Acceptance Criteria

[ ] Document titled "Translation Template Test Plan" created and completed

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @brick-green @jack-h-wang @JessicaWNava @JFU-NAVA-PBC @jimmyfagan @luis-pabon-tf @mkalish @thetaurean


[BUG] Change default action after submit Create request ?

Hi,
I just update refine and seen default action after submit Create request was changed.
Old:

Create Post
Save
UI redirect to List Post

New:

Create Post
Submit
UI redirect to Edit Post

All examples in refine.dev has New behavior. https://refine.dev/docs/examples/form/antd/useForm/
How can I change this behavior ?

Hey @vnStrawHat 👋,
Thank you for contacting us. You can change it with the "redirect" property of useForm.
By the way, sorry for this situation, We didn't want that to change, we'll look into the matter.

Hey @vnStrawHat ,
We've released<EMAIL_ADDRESS>Could you approve this fix?

Hi @omeraplak,
Sorry for not getting back to you sooner. This fix look good to me.
Thanks.

The List view has already active syncWithLocation, but after clicking save in Edit view, the page still jumps to the default state in List view


upd(prismlauncher-git):7.2->8.0

solve https://github.com/pacstall/pacstall-programs/issues/4679

libqt6core5compat6-dev is for ubuntu 22.04
qt6-5compat-dev for all others
@Elsie19 : Can I set incompatible=(ubuntu:22.04) ?
I have tested with | seems not working

Can you conditionally check $DISTRO and use the correct names from there?

go reading the documentation =) and installing ubuntu

I had trouble understanding all.
The documentation https://github.com/pacstall/pacstall/wiki/Pacscript-101 say :
prepare() {
  ./autogen.sh
  case "${CARCH}" in
    amd64) ;;
    arm64) ;;
    *) ;;
  esac
  case "${DISTRO}" in
    ubuntu:*) ;;
    debian:sid) ;;
    *) ;;
  esac
}

https://github.com/pacstall/pacstall-programs/blob/master/packages%2Fhyprland%2Fhyprland.pacscript#L7-L11

Also run pre-commit. You should've done that already, it's listed in our setup instructions...

Also run pre-commit. You should've done that already, it's listed in our setup instructions...

I use github with the web interface... git from the terminal is not my friend.
I have to do some extra efforts to learn it.
Well, this time it's done I hope.

It's required that you run pre-commit.

It's required that you run pre-commit.

Ok. Understand what you are speaking about.
It's the tool that check the code when I do a PR (and use shellcheck and other things)
It has passed successfully.
So I don't understand why the label had changed to invalid here...

Ok run shfmt -bn -ci -sr -s -i 2 -w packages/prismlauncher-git/prismlauncher-git.pacscript and push it then.

Ok run shfmt -bn -ci -sr -s -i 2 -w packages/prismlauncher-git/prismlauncher-git.pacscript and push it then.

davius@davius-gladius:~/pacstall-programs$ git commit
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check for added large files..............................................Passed
mixed line ending........................................................Passed
shellcheck...............................................................Passed
shfmt....................................................................Passed
update packagelist file..................................................Passed
[master f315fb48] upd(prismlauncher-git): `7.2`->`8.0`
 1 file changed, 11 insertions(+), 2 deletions(-)
davius@davius-gladius:~/pacstall-programs$ git push
remote: Permission to pacstall/pacstall-programs.git denied to Xdavius.
fatal : impossible d'accéder à 'https://github.com/pacstall/pacstall-programs.git/' : The requested URL returned error: 403

git push origin HEAD.

git push origin HEAD.

davius@davius-gladius:~/pacstall-programs$ git push origin HEAD
remote: Permission to pacstall/pacstall-programs.git denied to Xdavius.
fatal : impossible d'accéder à 'https://github.com/pacstall/pacstall-programs.git/' : The requested URL returned error: 403

Create a fork (click on "Fork" on this repo's main page)
Clone it
If you haven't set up Git with your GitHub credentials, follow these instructions:
https://docs.github.com/en/get-started/getting-started-with-git/caching-your-github-credentials-in-git#github-cli
Create a branch and switch to it

git checkout -b <branchname>

Make your changes and commit them
Push

 git push -u origin <branchname>

A branch will be created in your fork, open a PR from there

I don't understand this command  git push -u origin . If I first create a branch, why I need to create an other ?

Because if you created the branch locally, then you need to push it to GitHub now.
-u is the short form of --set-upstream, so you're linking your local branch to a remote branch.

Is it a way to PR directly from the terminal, or I need to turn to webUI ?
Yes, using gh : https://cli.github.com/manual/gh_pr_create

Why'd you close this?
I haven't

Wait there are two PRs..? #5680 and this one.


Visual sort tour guide

Description

had added a complete tour guide for beginners to get how the visualizer works
Issue number
#74

Issue fixes: #74
Screenshots (if any)

@Ayushswirlon loved the work !!
can you please add skip button in red color besides next button in guide so that people who revisited can skip it!

Done bro review it again

Okk will check and submit till tomorrow


Error on Visual Studio 2019 - Wextra

Hey there!
I make some tests in DicomToMesh on different OS. All OK on Linux and Mac OS X, but in Windows the result is not good, twice as slow.
I wrote about it here: http://ortogonline.com/doc/pt_br/OrtogOnLineMag/2/Slicer.html
And here: http://ortogonline.com/doc/pt_br/OrtogOnLineMag/3/VTK_ITK_multi.html
To solve it I tryied to build DicomToMesh again, years after the last compilation. All right with Cmake, but when I run on Visual Studio I have the Wextra error:
Build iniciado... 1>------ Compilação iniciada: Projeto: ZERO_CHECK, Configuração: Release x64 ------ 1>Checking Build System 1>CMake is re-running because C:/Users/cogit/Downloads/DicomToMesh-vtk-9-novisual/build/CMakeFiles/generate.stamp is out-of-date. 1>  the file 'C:/Users/cogit/Downloads/DicomToMesh-vtk-9-novisual/CMakeLists.txt' 1>  is newer than 'C:/Users/cogit/Downloads/DicomToMesh-vtk-9-novisual/build/CMakeFiles/generate.stamp.depend' 1>  result='-1' 1>-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19042. 1>-- Configuring done 1>-- Generating done 1>-- Build files have been written to: C:/Users/cogit/Downloads/DicomToMesh-vtk-9-novisual/build 2>------ Compilação iniciada: Projeto: dicom2meshlib, Configuração: Release x64 ------ 2>Building Custom Rule C:/Users/cogit/Downloads/DicomToMesh-vtk-9-novisual/lib/CMakeLists.txt 2>cl : linha de comando error D8021: argumento numérico inválido '/Wextra' 2>Projeto de compilação pronto "dicom2meshlib.vcxproj" -- FALHA. 3>------ Compilação iniciada: Projeto: dicom2mesh, Configuração: Release x64 ------ 3>cl : linha de comando error D8021: argumento numérico inválido '/Wextra' 3>Projeto de compilação pronto "dicom2mesh.vcxproj" -- FALHA. 4>------ Compilação ignorada: Projeto: ALL_BUILD, Configuração: Release x64 ------ 4>Não há projeto selecionado para compilar nesta configuração de solução  ========== Compilar: 1 com êxito, 2 com falha, 0 atualizados, 1 ignorados ==========
Do you know a way to solve it?
A big hug!

Hi Cicero, Your are not the first one with this issue. It seems to be a new problem on Windows. I have to come up with a solution. In the meantime, just delete those two lines in from your cmake files:
target_compile_options( dicom2meshlib PRIVATE -Wall -pedantic -Wextra -Wno-extra-semi )
target_compile_options( dicom2mesh PRIVATE -Wall -pedantic -Wextra -Wno-extra-semi )

Then redo the cmake configuration and it should work. Sorry for the hassle.
Adrian

Hi @eidelen
Thank you, I use and run. But I have some problems, Ill share here.
Only to documentate porpouse, the two files are:
DicomToMesh-vtk-9-novisual\dicom2mesh\CMakeLists
DicomToMesh-vtk-9-novisual\lib\CMakeLists
About the problems, the time with old version is 103 second, and the sime with new version is 148 seconds!
More than this, the reconstruction present some mesh erros:

Look at knee, that triangulated mesh is not present on CT-Scan.
Do you think I made some mistake?
A big hug!

All this processing is done in VTK. If you have the same VTK version:

the time difference could be caused by the fact that one is a release build and the other one a debug build.
the triangulation error can be caused by running dicom2mesh with different parameters. Is it happening during reduction, smoothing or surface creation?


MSAAHelper file downloads download wrong version

The documentation for the DirectXTK12 MSAAHelper has links to download the DirectX12 versions of the files.
When copy and pasting this does indeed link to the file, but when saving with ctrl-s, it instead downloads the DX11 versions.
Obviously this is a very small issue, and it's understandable it fell under the radar! But I thought I would at least let you know.

Thank you for taking the time to report an issue. I don't actually understand what you mean by "Ctrl+S saves the DX11 versions". If you mean while in the linked page Ctrl+S in the browser saves the wrong version, then I'm not sure what is causing that. I don't see that behavior in my browser.


Spying Input results in error

The used GdUnit4 version:
gdUnit4 4.1.0
The used Godot version:
v4.0.2.stable.official [7a0977ce2]
OS including version:
Windows 10
Describe the bug
Trying to spy Input results in: error: Parser Error: The function signature doesn't match the parent. Parent signature is "is_anything_pressed() -> bool".
Steps to Reproduce

Generate test suite & function where:

func test_input() -> void:
	print("Before")
	var input_spy = spy(Input)
	print("After")
	pass

See error encountered on line var input_spy = spy(Input)
GdUnitSpyBuilder.gd:067 is suggesting to report the bug 🐛

<!--

Thanks for reporting this problem, I will look into it tomorrow.


pipes | tanisha | ada trader

Ada Trader
Congratulations!
Comprehension Questions

Question
Answer

How do Backbone Views help structure your code?
backbone views removes code that deals with user interface from the app.js to allow for cleaner code

Did you use jQuery directly in your Views? How, and why?
no, anytime I wanted to grab a DOM element, I used this. before it.Using this puts the jQuery selector in the context of that particular view. Without that context, the selector would be look through the entire project and so you would need to use ids to be more specific.

What was an example of an event you triggered? Why did you need to trigger it?
in my quote view, I trigger a tradeUpdate in my buy and sell functions. the quote list view is listening for the trade update which is triggered in the buy or sell functions.

In what was is unit testing in JavaScript similar to unit testing in Ruby? In what ways is it different?

Ada Trader
What We're Looking For

Feature
Feedback

Core Requirements

Git hygiene
yes

Comprehension questions
yes

Organization

Models and collections are defined in separate files
yes

Code that relies on the DOM is located in or called by $(document).ready
yes

Functionality

Quote prices change when clicking Buy and Sell
yes

The Trade History updates when buying and selling a quote
yes

A user can create an open order using the Order Entry Form
yes

An open order removes itself from the open orders and updates the Trade History when fulfilled
no

General

Has separate views for different parts of the app
yes

Uses events (listening/handling and triggering) to manage different behavior in views
yes

Practices good standards for views (ES6 syntax, doesn't directly use jQuery, returns this in render)
yes

Error handling for the Order Entry Form
no

Testing

Has unit tests for models
no :(

Overall
This is a good start. Your code for handling quotes and market orders looks good, and you're very close to having limit orders working. There's just a little more plumbing that needs to be done to ensure data is available where you need it. Keep up the hard work!


Restore query in case search

Restoring query string in search bar for case search screens.

Looks good other than the missing import


Working offline!

Tried to initiate the r3f-perf offline, but it didn't work! Is there a way I can work around that error?
The UI appears, but the stats are not displayed!

I have a similar issue (am unable to load assets not on our whitelisted domains).
The root of the problem is that troika-text-3d (which this tool depends on through its dependency on drei) loads fonts via a CDN, if a unicodeFontsUrl is not provided and that is not exposed in the Perf interface.
You can see where you would need to add it here (and below): https://github.com/utsuboco/r3f-perf/blob/292adb112ce57e0780e57454452a6b450412035f/src/components/TextsHighHZ.tsx#L108

I have a similar issue (am unable to load assets not on our whitelisted domains).
The root of the problem is that troika-text-3d (which this tool depends on through its dependency on drei) loads fonts via a CDN, if a unicodeFontsUrl is not provided and that is not exposed in the Perf interface.
You can see where you would need to add it here (and below):
https://github.com/utsuboco/r3f-perf/blob/292adb112ce57e0780e57454452a6b450412035f/src/components/TextsHighHZ.tsx#L108

Knowing the root of the problem is amazing. Seen that once I was using drei, but I was always able to refer a local font. Will try to do that here on Perf and see if that would make sense to be a useful PR!
Otherwise, I will just use it for myself 😅
Thanks, that was really useful 👍


Append extra component after the content or under the Navlinks

Hi Gentlemen,
I'd like to know if there are proper ways to append content inside the main content or Navlinks without tweaking the core files of /nextra/ and /nextra-theme-doc/ ? Planning on adding a Survey feature in each page below them.
Thank you in advance.

Use this theme option https://github.com/shuding/nextra/blob/core/packages/nextra-theme-docs/src/types.ts#L24

That was it! Thank you!


[BEAM-6161] Introduce PCollectionConsumerRegistry and add ElementCoun…

…t counters to the java SDK
Please add a meaningful description for your change here

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

@swegner @Ardagan @robertwb @echauchot
This PR is long, most of the changes are refactoring the bundle processing startup code's setup of Pcollection consumers to go through a common class, where wrapper code could be inserted to add an element count counter.

Please, add unittests for newly added classes.

@boyuanzz @robertwb


Don't upscale smaller images

The above comment is true, so this is a non-issue. Just pass withoutEnlargement: true in the blunt.config.js under sharpOptions


Add support for Sorting files downloaded into Series name and Season folders

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ X] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[X ] At least skimmed through the README, most notably the FAQ and BUGS sections
[X ] Searched the bugtracker for similar issues including closed ones
[ X] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ X] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
Would it be possible to add functionality to automatically put downloaded content into Series name and season folders?

https://github.com/rg3/youtube-dl#output-template-examples


feat: opt-in TTL on trivy-operator managed jobs

We are trying out trivy-operator in our multi-tenant (and air-gapped) clusters. Our clusters are quite big, and constantly growing. Because of the nature of our clusters, we probably have to adjust the trivy-operator configuration to support scanning across the cluster.
Currently our only feedback source seems to be the operator logs, and that is a bit overwhelming in a big cluster.....
So I would like to see some improvements to this over this. Some ideas:

Introduce status on more trivy-operator CRDs. Personally I prefer a kstatus compatible status structure, which seems like an adopting standard. I.e. FluxCD understands kstatus.
Metrics indicating failing operator reconciles. This can easily get problematic because of Prometheus label cardinality, so status on CRDs is better IMO.

But as a workaround (and maybe even a minor improvement) to this, I suggest to introduce an option to keep jobs around some time after they are completed/failed (TTL). That will allow us to more easily track scanning issues per workload. WDYT?

I think we should  start with workaround (to improve the immediate problem ), and log issue for long term solution

I've had a look at the relevant code, and it seems like the jobs are deleted as soon as the job completes, and the report(s) are updated with the result:
The by far easiest solution would be to replace explicit Job deletions with setting .spec.ttlSecondsAfterFinished on the Job on creation.. The potential issue with this change, is that the TTLAfterFinished feature gate was only enabled by default in K8s 1.21. For a core functionality like cleaning up completed jobs, we should not rely on users enabling this feature gate. So this will effectively force us to drop support for K8s version < 1.21.
Related issue #221.

currently I see two issues (might be more) with suggested approach which needed to be addressed :

After scan job has been completed trivy-operator is processing the logs and building the reports , if it to use the TTL approach then it need to be deterministic that job will be evict only after reports has been produced.

deletion of the job is done on a several condition , not only on success , need to make sure that TTL approach will satisfied all conditions

The TTL-after-finished controller is only supported for Jobs. A cluster operator can use this feature to clean up finished Jobs (either Complete or Failed) automatically by specifying the .spec.ttlSecondsAfterFinished field of a Job, as in this example. The TTL-after-finished controller will assume that a job is eligible to be cleaned up TTL seconds after the job has finished, in other words, when the TTL has expired.

So the TTL will only start ticking once a job is finished (completed or failed). A TTL set to low will be a bad idea, at least if setting the value when trivy-operator creates the job, but I suggest setting a default value that allows the jobs completed reconciler in trivy-operator enough time to pull out the logs. Another option can be to set the TTL on jobs in the trivy-operator jobs completed reconciler, after processing the logs, which should address your concern for sure. The latter might need some special handling, since setting the TTL will trigger another reconcile.

I'm in favor of the later one as it will remove the concern of post processing timing.
how about other cases where we delete the job ? there are 3 cases we delete the job before completion

I'm in favor of the later one as it will remove the concern of post processing timing. how about other cases where we delete the job ? there are three cases where we delete the job , which could be before completion/ fail

Could you reference the code in question? I think all explicit delete of jobs should be replaced by the TTL mechanism.

Could you reference the code in question? I think all explicit delete of jobs should be replaced by the TTL mechanism.
yes , probably all should be cleared by TTL


[TRIVIAL] Make fee command public (RIPD-1113)

Really not much to it.

And yet...the commit disturbs the formatting of the table...

We care about the formatting of the table?
Ok, I amended to fix that, and added the Jira to the commit message.

Current coverage is 68.33%

Merging #1662 into develop will decrease coverage by -0.08% as of 6a54f44

@@            develop   #1662   diff @@
=======================================
  Files           939     939
  Stmts         69491   69486     -5
  Branches          0       0
  Methods           0       0
=======================================
- Hit           47540   47481    -59
  Partial           0       0
- Missed        21951   22005    +54

Review entire Coverage Diff as of 6a54f44

Powered by Codecov. Updated on successful CI builds.

Squashed and merged as b509263ef54408dc971781e8e6d39dc79620ded1


refactor: move static metadata from setup.py to pyproject.toml

Description
This PR moves the project metadata from setup.py to pyproject.toml, as described by PEP 621 and supported by setuptools. It is a close 1:1 conversion, with a few changes:

The description is shortened, since this should be a one-line summary of what the distribution does
The previous "description" content is set as the "readme" content that you'd see rendered on PyPI here
The "Home" URL is replaced with a "Documentation" URL
Dependencies and extra dependencies are alphasorted
Expand Python version classifiers
Packages are discovered automatically via "find", rather than explicitly listed
Before publishing a release, run twine check --strict dist/*

Supersedes #2020
Closes #1990

Hi @johanneskoester the only conflicting edits with setup.py is from #2020, but this PR already expands the Python version classifiers. The only other trivial change from master in this PR is:
-     "Programming Language :: Python :: Only",
+     "Programming Language :: Python :: 3 :: Only",


Fix wrong option keys in docs.

The option keys in the docs are not consistent with code.

Fixed! I actually renamed the options to offset and limit but I kept backwards compatibility with start and num


remove clerk for testing

not going to merge this, but the branch is here for anyone to use to debug


Z-Bonemerang does twice the damage

I know it's a very unlikely calc to do, but it seems it still considers the move as 2 hits since it triggers the consumption of berries and does more damage than Z-Earthquake

Fixed by b441405


calibre: Pin autoconf dependency to 0.31

Motivation for this change
Presently, Calibre fails to build (stacktrace) on nixos-unstable since autoconf was updated to 0.32 because of a regression in calibre (https://bugs.launchpad.net/calibre/+bug/1936889) which has been patched, but not yet released (https://github.com/kovidgoyal/calibre/commit/4f9e83e6426483b6cc0929c61f9207c33c573fec).
This pins Calibre's autoconf to 0.31 until that gets merged and we can update.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

ZEROCONF! I meant zeroconf, not autoconf. Let us never speak of this again.
Recreated as https://github.com/NixOS/nixpkgs/pull/131477 :sweat_smile:


Checkbox for "This will delete all cookies for the domain you're recording for."

In some circumstances, this is not what you want. This should be a checkbox, though turned on by default.

This will be useful for me too. Is there a starting point in the code I can look at?

This is the line that does the deed. If you add a checkbox to here/here to make it toggle-able that should do it.

Implemented, will be in the next release.

@Zarkonnen Awesome!


Tweak Qiskit release notes structure for future release notes

Follow up to https://github.com/Qiskit/documentation/issues/294 and https://github.com/Qiskit/documentation/issues/281.
@frankharkins found that it works well using the HTML generation script. A key change from Qiskit itself is we have one page per minor release, like release-notes/0.45. We need to make two improvements:

Restructure Qiskit/qiskit to have one release note file per release for future releases. Whereas right now they have the file release_notes for all notes from Qiskit 0.45 forward
Update our script updateApiDocs.ts to handle new release notes automatically, including the wonkiness of the prior release note files (legacy_release_notes.html).

I'm thinking that the simplest solution will be for Qiskit to continue having only a single file release_notes.rst. Note that we removed legacy_release_notes.rst already.
Then we have the Qiskit devs configure Reno so that release_notes.rst only has the release notes for the current version of that branch, such as 1.0 vs 1.1. That will simplify updateApiDocs.ts because it only needs to convert release_notes.rst into a single file under the release-notes/ folder for the current API version. That is, it doesn't need logic to parse the file and extract out the relevant version from the file; the file already only has what is relevant.
We need Qiskit devs to be okay with the plan, since they'd need to tweak the versions listed in release_notes.rst with releases. I'll present this plan to them at the weekly meeting next week.
cc @arnaucasau and @francabrera

We decided in the Qiskit dev meeting after talking this over with @ElePT, @jakelishman, and @mtreinish that the changes should happen in this repo, not in Qiskit/qiskit. They value building the historical release notes in their local builds more than I realized.
They also do sometimes change prior release notes, such as fixing typos. It's important that Qiskit/documentation pulls in those changes, even if the release notes changed don't correspond to the current version of the docs we're building. For example, if we're building 1.1 docs, we should still see if there were any updates to 1.0 release notes.
So, this issue now tracks us improving updateApiDocs.ts to properly handle release notes for Qiskit. It will still only look for the file release_notes.html, but it needs to properly split out the file into its distinct minor releases, such as by using the logic from the PR description in https://github.com/Qiskit/documentation/pull/346.


Allow third-party backends to add submodules to triton.language.extra

Add an optional language directory to backends. The contents of the directory is added to triton.language.extra when the wheel is built.
Update the existing triton.language.extra.cuda and triton.language.extra.hip modules to use the new mechanism.
The core Triton is a small number of people, and we receive many PRs (thank
you!).  To help us review your code more quickly, if you are a new
contributor (less than 3 PRs merged) we ask that you complete the following
tasks and include the filled-out checklist in your PR description.
Complete the following tasks before sending your PR, and replace [ ] with
[x] to indicate you have done them.

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[x] This PR does not need a test because It is already tested by python/test/unit/language/test_core.py::test_math_extern.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added follow these best practices,
including the "tests should be minimal" section. (Usually running Python code
and using the instructions it generates is not minimal.)

The tests should pass now.

For some reason there were more pre-commit changes after the running it a second time. Everything looks to be passing now.

Rebased

Rebased

Would https://github.com/triton-lang/triton/pull/4539 solve your problem?

Would #4539 solve your problem?

Yes, that solves the issue of dispatching to third-party libdevice functions 👍.
Maybe the rest of this change is still nice to have though to improve parity between the in-tree backends and third-party backends?
With the current third-party backend mechanism, it's possible for a backend to be entirely out of tree. The only caveat is that without in-tree changes you can only get python files into triton.backends.x, and even that feels slightly hacky since the compiler.py and driver.py in that namespace are generally loaded via a separate mechanism.

Sorry for the delay. Do you still want to push this? The changes seem reasonable. Could you resolve conflicts if you still need this PR

Sorry for the delay. Do you still want to push this? The changes seem reasonable. Could you resolve conflicts if you still need this PR

Hi, no worries about the delay. Yes I'm still keen to push this. I've rebased the changes onto main.


Consider turning some of the numeric helpers into constants

https://github.com/microsoft/windows-rs/pull/469#discussion_r566847851

All but the most trivial of these are not expressible in Rust today due to restrictions with const functions or const expressions in Rust involving floating point values.


NIFI-4424 org.apache.nifi.NiFi does not allow programmatic access to …

…the NiFi engine
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

Please note that the mvn -Pcontrib-check clean install failed with the following test case errororing out, which I believe to be a timing issue, that cannot be related to the change.
testStateTooLargeExceptionThrownOnReplace(org.apache.nifi.controller.state.providers.zookeeper.TestZooKeeperStateProvider)  Time elapsed: 90.345 sec  <<< ERROR! org.junit.runners.model.TestTimedOutException: test timed out after 20000 milliseconds.
For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Hi @patricker, please let me know if there is anything that requires clarification or if there is any way I could help the review. I would love to see this pull request being merged.


Hermes not working with react native 0.71 on Android

Command appcenter codepush ... working but not show message

Converting JS bundle to byte code via Hermes ...

appcenter-cli: 2.31.1
react-native: 0.71
react-native-code-push: 7.1.0 with patch https://github.com/microsoft/react-native-code-push/issues/2418#issuecomment-1426484628

@sontruongna Thank you for reporting this issue. Could you provide more information about this error?
Is it blocking you?

@sontruongna Thank you for reporting this issue. Could you provide more information about this error? Is it blocking you?

For previous react native versions, when I run "appcenter codepush ..." command, on my log screen appears the line "Converting JS bundle to byte code via Hermes, running command:". That means hermes worked. When I upgraded to react native version 0.71.2 two problems occurred. First I cannot run android with<EMAIL_ADDRESS>I have to use patch-package like in the following link https://github.com/microsoft/react-native-code-push/issues/2418#issuecomment-1426484628. Then I can run on Android. But I found out now that when I run appcenter codepush in command line, on my log screen there is no longer the line "Converting JS bundle to byte code via Hermes". That means the bundle just contains the regular js code. And I've seen performance drop on app startup because this package didn't come with hermes.

Hi @sontruongna , do you have the property enableHermes set to true in your android/app/build.gradle file? If not, does it exist in any other build.gradle files?

If your build.gradle file is located in different place, try to specify it using gradleFile argument for the codepush command.
If it doesn't exist in any of your build.gradle files, then try to add the following lines to your android/app/build.gradle file:

project.ext.react = [
      enableHermes  :   true
]

If the suggestions above don't work for you, try to bypass the hermes check by providing the argument useHermes for the codepush command.
Please let me know if it solves your issue.

Hi @sontruongna , do you have the property enableHermes set to true in your android/app/build.gradle file? If not, does it exist in any other build.gradle files?

If your build.gradle file is located in different place, try to specify it using gradleFile argument for the codepush command.
If it doesn't exist in any of your build.gradle files, then try to add the following lines to your android/app/build.gradle file:

project.ext.react = [
      enableHermes  :   true
]

If the suggestions above don't work for you, try to bypass the hermes check by providing the argument useHermes for the codepush command.
Please let me know if it solves your issue.

Thanks you. Hermes worked when I added these lines to android/app/build.gradle
project.ext.react = [
      enableHermes  :   true
]

So I think the problem is in React Native version 0.71.2. This version default has removed the above lines. The way to check on off hermess has changed. There is only one configuration hermesEnabled=true in gradle.properties. I think appcenter cli will need another way to check on off hermes on android.

@sontruongna, Thank you for the information! I'll keep this issue open as a feature request.

We have decided not to implement this feature in the foreseeable future. However, we truly appreciate and welcome your contribution!


Isn’t Working

I bought this issue up in the DAPI server, and I believe someone mentioned about it just not working when so I was having issues.
https://hastebin.com/sezihudude.yaml
This is the code I tried, simply doesn't edit through the two pages, other than that the other features work fine

Realised it was due to it not being updated npm side so used it from the GitHub directly


Make -u and -p parameters optional at cf login

It may not be necessary on certain platforms to use the -u and -p parameters when using a one-time-passcode. Using an explicit option (or maybe when the username is not provided in the service), the -u and -p flags should not be included in the cf login command line.

@acervantes  is this still an issue?

@mozts2005, greetings. I haven't had the chance to work in this anymore since 2017, so I'm not up to date on the current state of the art to evaluate it. The issue may be closed if you don't find it justified anymore.


After Installation via HACS there are only 3 entities available

After configuration via HACS and entering my login credentials, there are only three entities available. The Most crucial ones like mileage, fuel level and Trip Data are missing.
Expected behavior
Having Access to all other sensor data like mileage, fuel level and single trip data.
Your Vehicle Details
Model: Audi A4 Avant 1.4 TFSI
Year: 2017
Type (ICE/PHEV/BEV): B9
I am using an Audi Data Plug to collect the data from the vehicle

I am not sure this supports those plugs. Does the connect app show the data?

Yes it does.

Can you provide a log file?

audiconnect_log_ha_28-05-2024.pdf
I removed the FIN (vehicle ident) for safety reasons. As I am quite new to home assistant, I am not sure whether such kind of log file is okay. Please let me know in case more information is needed.

Hi @mamillion12 . Thanks for providing more info. Specifically, we would need to see the vehicle data that is returned to understand more about the issues you're seeing. Can you enable debug logging (from integration > audiconnect screen), run the service "Refresh Cloud Data" and then disable debug logging. You will then be given a log file.
I'm thinking this is a duplicate of #263.

Thank you!
Haven't seen this type of data response before. My understanding is bad gateway errors typically have to do with network configuration or server issues. Has this integration worked for you in the past and this just started occurring? Have you recently made any changes to your network configuration?

No, it has never worked at any time. All other integrations are working as expected in my Home Assistant instance. Thank you for your support!

Is there anyone who might be able to support me?

We would need to capture & analyse data traffic from your app to understand your interfaces.

To confirm you don't use the Audi connect app for the data but rather another Audi branded one?

Yes, I am using the Audi connect plug and play app.

Yes, I am using the Audi connect plug and play app.

That's good. Now we need data :)
@t0bias-r explained here how he does it with Android: https://github.com/audiconnect/audi_connect_ha/discussions/282#discussioncomment-9060228 - can you provide us something like that?
Do you also see data in https://my.audi.com/ ? Maybe this universal API (with limited data) can also be integrated as fallback.


add Proline input - parsing

I have:

replaced the "replicate_mapper" with "condition_mapper" everywhere
removed Wombat from the input options of the DDA quant module since it is precursor ions, and there is not output for this from Wombat. I kept the associated files though. So that we have them when we copy the script for the peptidoform module
tried to parse the Proline input: before, we used as input a txt file that contained only the ion quantification. Now we use the excel file as input. This way, we also have the tabs with the parameters.
BUT: I did not manage to parse the modifications. I don't really understand how the Modified sequence is generated and how the toml [modifications_parser] should look like.
Maybe @RobbinBouwmeester could give me pointers?

Does wombat still follow the same formatting as before (e.g.; "EFHQAGKPIGLC[Carbamidomethyl]C[Carbamidomethyl]") that should be good without parsing. If you want no parsing you can completely leave out the modifications_parser section in the .toml.

I was not clear, sorry.

For WOMBAT: it cannot be implemented in the DDA-quant module for ions because the outputs do not report the ion quantification. So we just ignore this tool for the ion-level module, it will be implemented for the peptidoform module.
the modification parsing is for Proline output. In Proline, we have the stripped sequence and the modifications in two different fields:

sequence
modifications

LMSAISK
Oxidation (M2)

CLATHGR
Gln->pyro-Glu (Any N-term); Carbamidomethyl (C1)

And I don't really understand how to make the correct toml file for generating the modified sequence.
Is this clearer?

Do you have an example of the proline file? We will need to add it to the test routine anyway.

Do you have an example of the proline file? We will need to add it to the test routine anyway.

Yep, let me find it and put it with the other test files.

Done, the file is here: https://github.com/Proteobench/ProteoBench/tree/main/test/data

I see what you mean right now, this is not yet supported, should be easily fixed, will look at this ASAP.


Add Single Member Fragment

What this PR does / why we need it: Showcase a single member on its own fragment.
Which issue this PR fixes (optional - uncomment and add issue):
fixes #27
Special notes for your reviewer: I added bootstrap card style for the display i'd like to get your opinion about it.

I like the general design. Not sure yet about the specific ordering concerning the readability and prioritization. The name is a bit "hidden" on the left side.
Also we can do it as separate fragment, but I think it would make sense to reuse the member fragment and add single member mode there. Happy to iterate with a separate fragment for now to keep the code simple and then later integrate the finished design into the member fragment.
Happy to discuss this. That's just my thinking, that a single member fragment doesn't add completely new features and could be folded into the already available member fragment.

Yeah, I understand adding the single member mode within the existing member fragment sounds okay but a bit of concern will be the styling since they'll both share the same template.

@stp-ip I updated the fragment to reflect the current changes please review

We don't have an example within the PR. Might be interesting to have an example of a personal page.
/personal or /bio or so could showcase this.
Later we could use the various subpages and link to the from the documentation or so.
For example we have:

resource showcase

page: https://syna-demo.netlify.com/resources/page/
fragment: https://syna-demo.netlify.com/resources/fragment/
global: https://syna-demo.netlify.com/resources/global/

nesting: https://syna-demo.okkur.io/nesting/
split: https://syna-demo.netlify.com/about-split/
single: https://syna-demo.netlify.com/about/

Oh alright. I'll add that. Also does Netlify generate deploy preview on PRs?

Yes. The small green arrow next to commits are links to the specific deploy for each commit.
https://deploy-preview-130--syna-demo.netlify.com/

Thanks. How did I miss that all this while.

Should have mentioned that more broadly I assume. Sorry :P

@mpourismaiel The last commit shocases my worry around having special cases for the whitespace.

/LGTM besides the whitespace. So either we go full whitespace reduction, add docmentation for how to do whitespace reduction or ping @erbesharat to code the rules into the makefile as a "make format" or something.
Sidenote: Awesome that we finally have a single member mode. Looking forward to the first personal sites from the team and getting feedback on Syna and the single member mode. From a structure perspective I think we should iterate in follow on PRs, but that needs actual user feedback.

Yeah. the whitespace might be an issue if we have to keep checking and removing it manually.

Merging this now and opening a separate whitespace issue. I did some cleanup of files and they might give a merge conflict. So better I clean that up.


feat: implement connection pool

[x] Implement connection pool
[x] Add tests for connection pool

Hey, please rebase


Fix light client call_executor execution_proof_is_generated_and_checked test

Migrated from: https://github.com/cennznet/plug-blockchain/issues/62#issue-510369916

On master branch:
failures:

---- tests::voter_catches_up_to_latest_round_when_behind stdout ----
thread 'tests::voter_catches_up_to_latest_round_when_behind' panicked at 'assertion failed: set_state.read().last_completed_round().number >= 4', core/finality-grandpa/src/tests.rs:1501:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

failures:
    tests::voter_catches_up_to_latest_round_when_behind

test result: FAILED. 58 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

error: test failed, to rerun pass '-p substrate-finality-grandpa --lib'

This is the line to cause the error: https://github.com/plugblockchain/plug-blockchain/blob/develop/core/client/src/light/call_executor.rs#L497
Not sure what the purpose of it is yet. Commenting it out passes the test.

This test also fails on substrate/master with the same error log:
cargo test -p substrate-client light::call_executor::tests::execution_proof_is_generated_and_checked

Error:
Thread 'light::call_executor::tests::execution_proof_is_generated_and_checked' panicked at 'Externalities not allowed to fail within runtime: "Trie lookup error: Database missing expected key: 0xb3fa3eff4d4d5d287ebc1be63f1d0236b2c21aaea7d0c20fe1e4ca597a0705b3"', src/libcore/result.rs:1165

Our error just has a different key: 0xd8a1f856f5faa67db595950a0b5a965e03b3499aa48c3a05d1f8e948f81a1854
Upon closer examination, the issue is more related to not being able to write to the trie database. Can track this issue https://github.com/paritytech/substrate/issues/1952

Seems to be resolved on latest develop


Broken link

Link ListBlobsSegmentedAsync, DownloadToFileAsync and more are broken.
(The latter is/was leading to https://docs.microsoft.com/en-us/dotnet/api/microsoft.windowsazure.storage.blob.cloudblob.downloadtofileasync)

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

ID: 60e56e69-1c44-2056-ad5f-319535e4b525
Version Independent ID: 49fa6098-0b8e-ebf5-efe7-f50f3604e325
Content: Quickstart: Use .NET to create a blob in object storage - Azure Storage
Content Source: articles/storage/blobs/storage-quickstart-blobs-dotnet.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@LosManos Thanks for your question. We are checking on this and will respond to you soon.

@LosManos  Thanks for bring this to our attention, The PR request has been initiated
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


prediction convertion?

Hello,
I try to run the code of testing, and use the model "mode_best.pth" in hugging face for s3dis segmentation: https://huggingface.co/Pointcept/PointTransformerV3/tree/main/s3dis-semseg-pt-v3m1-0-rpe/model
However, when I apply the predicted label on the coordinates of the point clouds, for example, on area5_hallway9, the obtained hallway form does not correspond to the one obtained when I apply the groud truth label on the same coordinates of the point clouds.
For example, in the point clouds with the predicted labels, the roof has holes, and the hallway has some obstacles, which do not appear in the point clouds with the ground truth labels.
However, when I do not apply the labels to the point clouds, and only visualize the point clouds, the form is different from both the point clouds with predicted labels and those with groud truth labels.
Do you know what the problem is?
https://github.com/user-attachments/assets/769f35f9-e010-4ecf-acf8-9cc7730f2fd7
https://github.com/user-attachments/assets/1e7af0e2-af9b-4b88-b5fe-69a92549fd53
https://github.com/user-attachments/assets/f0d427b5-cbac-4f89-b387-b6b7c4bfbcdf

Nice visualization, how about first check the number metric of testing is matched? The released weight is for PTv3 is with Pointcept v1.5.1. I remember there exist a small gap between current version.

Hello, I would like to ask, how do you visualize it? I am using the same pre trained model as you. I am unable to visualize it, and I am not sure if the result I ran is incorrect or for some reason. I hope you can help me answer this, thank you very much!

Nice visualization, how about first check the number metric of testing is matched? The released weight is for PTv3 is with Pointcept v1.5.1. I remember there exist a small gap between current version.

Thank you for the suggestion. I will check it out about the metric value. In addition, do you have the latest version of the model of ptv3 for dataset s3dis?

Hello, I would like to ask, how do you visualize it? I am using the same pre trained model as you. I am unable to visualize it, and I am not sure if the result I ran is incorrect or for some reason. I hope you can help me answer this, thank you very much!

the output is the label of each point. You should create a file in which you define a color for each label. In this way, each point has its corresponding color, and you can visualize it. I visualize it based on open3d library

Thank you very much for your reply. I have tried to visualize the data I output, but it cannot be visualized correctly.
I visualized the output. npy file using the open3d library, but it was not successful. Then I output the data format of the file and found that it is a one-dimensional array, not in N * 3 format.
If possible, could you send me an output file after you run it? For example: Area_5_office_8_pred.npy
I can clarify where my problem lies. Thank you.

Nice visualization, how about first check the number metric of testing is matched? The released weight is for PTv3 is with Pointcept v1.5.1. I remember there exist a small gap between current version.

Thank you for the suggestion. I will check it out about the metric value. In addition, do you have the latest version of the model of ptv3 for dataset s3dis?

Hi, sorry for the late response, I just recovered from the busy schedule for the CVPR project. I will have a big update a few weeks later. A stronger self-supervised PTv3 and turned weight for S3DIS sem seg will be available in Jan of next year after necessary review process.


Since 2.0.0, predictions are different when using custom and built-in losses in regressions with L2 (reg:squarederror) losses

# train xgboost with toy regression data
# use custom objective function for l2 loss and original l2 loss
# compare the results
import numpy as np
import xgboost as xgb
from sklearn.datasets import load_diabetes
from sklearn.metrics import mean_squared_error
from sklearn.model_selection import train_test_split

X, y = load_diabetes(return_X_y=True)
X_train, X_test, y_train, y_test = train_test_split(X, y)
r1 = xgb.XGBRegressor(objective="reg:squarederror")

def squarederror(x, y):
    return -(x - y), np.ones_like(x)

def wrongerror(x, y):
    return (x - y), -np.ones_like(x)

r2 = xgb.XGBRegressor(objective=squarederror)
r3 = xgb.XGBRegressor(objective=wrongerror)
r1.fit(X_train, y_train)
r2.fit(X_train, y_train)
r3.fit(X_train, y_train)
y_pred1 = r1.predict(X_test)
y_pred2 = r2.predict(X_test)
y_pred3 = r3.predict(X_test)
print(mean_squared_error(y_test, y_pred1))
print(mean_squared_error(y_test, y_pred2))
print(mean_squared_error(y_test, y_pred3))

> python run.py
3656.0092943260324
3656.0092943260324
29423.763513513513
> pip install -U xgboost
Requirement already satisfied: xgboost in  (1.7.6)
Collecting xgboost
  Using cached xgboost-2.0.0-py3-none-win_amd64.whl (99.7 MB)
Requirement already satisfied: numpy in  (from xgboost) (1.24.3)
Requirement already satisfied: scipy in  (from xgboost) (1.9.3)
Installing collected packages: xgboost
  Attempting uninstall: xgboost
    Found existing installation: xgboost 1.7.6
    Uninstalling xgboost-1.7.6:
      Successfully uninstalled xgboost-1.7.6
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
boost-loss 0.0.0 requires xgboost<2.0.0,>=1.7.5, but you have xgboost 2.0.0 which is incompatible.
Successfully installed xgboost-2.0.0

[notice] A new release of pip is available: 23.1.2 -> 23.2.1
[notice] To update, run: python.exe -m pip install --upgrade pip
> python run.py
4721.520754922534
4154.287296383936
29991.078828828828

As you can see, the results are better with custom loss.

We added automatic estimation of base_score in 2.0, you can obtain the same result of the custom objective by setting it to 0.5:
r1 = xgb.XGBRegressor(objective="reg:squarederror", base_score=0.5)

The estimation is based on the training data, in some cases it can lead to overfitting.


Test IN Query Modifier with empty array input.

Some databases fail if your IN clause is empty. This test checks that
those sorts of queries are well behaved.
This test currently fails on sails-mysql - hence why I wrote the test :-)

More than a year later... @leedm777 it's no longer possible to automagically merge this PR. Would you mind mind merging master to it and solve any conflicts?
BTW, your test seems related with issue balderdashy/waterline#412.

@dmarcelino I've rebased against master. Thanks!

Thanks @leedm777! The build has failed (in all adapters) but not for the predicted reason:
Queryable Interface IN Query Modifier with a record when given an empty array should return an empty array:
     ReferenceError: User is not defined

There seems to be a problem with your test.

@dmarcelino I think that's got it. Thanks!

Your test actually passes on all official adapters:
| adapter          | version | result | failed | total | wl-sequel |
|------------------|---------|--------|--------|-------|-----------|
| sails-postgresql | 0ad94e5 | passed |      0 |   235 |     0.3.1 |
| sails-memory     | 7cf3600 | passed |      0 |   211 |           |
| sails-disk       | 5210c18 | passed |      0 |   208 |           |
| sails-mongo      | ae251dc | passed |      0 |   209 |           |
| sails-mysql      | d289f11 | passed |      0 |   234 |     0.3.1 |
| sails-redis      | 6eb8f92 | failed |     41 |   209 |           |

Someone must have fixed sails-mysql meanwhile. :) Looks good! :white_check_mark:
I'll give others the opportunity to check your PR and if no one opposes I'll merge it soon. Thanks @leedm777!


Caddy model fails when increase step size to 0.02 in gz

Bug report
Description of the bug
Caddy fails when increases the step size to 0.02 in gazebo. Notice that the model will spawn back to the origin
https://user-images.githubusercontent.com/32189404/118593540-75ada580-b7da-11eb-8daf-1d024f8b149f.mp4
(not sure why the video is not viewable on firefox :disappointed: , try chrome)
Steps to reproduce the bug

ros2 launch rmf_demos airport_terminal.launch.xml
Run this when launch is completed: gz physics -s 0.02

Observe the caddy robot.
Occasionally will get this error when repeat the above steps:
[gzclient-14] gzclient: /build/ogre-1.9-kiU5_5/ogre-1.9-1.9.0+dfsg1/OgreMain/include/OgreAxisAlignedBox.h:251: void Ogre::AxisAlignedBox::setExtents(const Ogre::Vector3&, const Ogre::Vector3&): Assertion `(min.x <= max.x && min.y <= max.y && min.z <= max.z) && "The minimum corner of the box must be less than or equal to maximum corner"' failed.
[ERROR] [gzclient-14]: process has died [pid 5

Note that this will not happen when using stepsize 0.01 in gazebo. On ignition, it works perfectly fine.
I tried to remove the readonly plugin in the rmf_demos_assets/models/Caddy's model.sdf and spawn caddy at other location. But this issue still happens. Will require a further look into the model to debug on this.

This is most likely an internal Gazebo/physics issue rather than anything we're doing with RMF. If I understand correctly, when model states start to get populated with NaNs, Gazebo might fall back to just placing the model at the origin.
NaNs happen when the physics can't manage to solve the constraints in a numerically stable way. Doubling the step size can easily make the physics constraints numerically unstable. I think we'll have to accept that we can't use the caddy model when we want the step size to be 0.02. Personally I never fiddle with the step size because of precisely this kind of issue.


NFT: create NFT Validity Check (Closes #160)

[x] Make sure the validity of the NFT is checked with each transaction.

Other improvements:

[x] Separate API logic into its own module.

@cstml is there a point in merging this if the proof-based way is more like the current way?

The current way is flawed because the validator will validate the buying/setting price of NFTs that are minted with a different Minting Policy than ours, as long as the Datum of these NFTs is correct. This is what this PR was fixing.
#170 is an improvement that guarantees that NFTs are only minted once for any given Data/Artwork/Content. We would still need to constrain the ability to buy/set-price to NFTs minted with the correct MintingPolicy. I can have a think if there is a different way to do so stemming from the changes of #170, but it might require a similar solution to this.


Merge with branch 1.4

Description
Merge with branch 1.4

LGTM

@jjlopez and @darroyocazorla review this PR again, please

+1


ESLint error: Cannot read property 'externals' of undefined

I've come across an issue with ESLint when using eslint-import-resolver-webpack and webpack config is an array of functions.
ESLint starts to fail almost on all rules, saying:
[eslint] Resolve error: Cannot read property 'externals' of undefined
I've managed to found the part of sources which responsible for that. The cause is that it first checks whether config is a function (and then executes it) and after that check whether it's an array.
https://github.com/benmosher/eslint-plugin-import/blob/b4a2f11fcacc6b2f048da4b29cfc896e682f17d1/resolvers/webpack/index.js#L88-L109
When it tries to find proper config with resolve field in the array, it finds nothing, because all elements are functions.
And, in my case, it turns that config is undefined which leads to the error I see.
My proposition is to check if config is an array, try to convert every element from function to object.
In case I understood everything correctly and didn't miss anything, I can make a PR for that, just going to add some tests as Contribution Guide claims

Closing the issue because the PR has been merged.


[GSSOC'21] Square sub matrix problem

Finding a solution for square sub matrix problem.
Problem: Given a binary matrix.find number of square submatrices in it using DP
This is one of the best applications of dynamic programming.
@anubhavitis  @dheerajkotwani  @tarun26091999  @avijit1999  @todi-2000  Kindly assign this task to me as part of gssoc'21.

I have proposed my solution in PR #811 ..Kindly have a look

Hi,
I am GSSOC'21 participant
My code will be in JAVA with proper comments and test cases.
@rudrakshi99 @nakul-19 @plazzy99 please assign me this issue

Hello! I am a participant in GSSoC'21. I am really intrigued by this issue. Would be highly obliged if you could assign this to me. I will be using Java. Thanks!

Hey. I am unable to provide an efficient solution, hence unassigning myself.


Gallery extension

Partially fixes #1129
Example here https://mss-test.readthedocs.io/en/latest/gallery/index.html
Includes the ability to specify one or more levels, init times and valid times.
In case of multiple the user can go through them via a combobox on the website.
Does not yet include multilayering.

This is a very good start.

I have some troubles with the UI. I can switch between the 18th and the 19th, but not select the bolded 17th. (Firefox on Linux).
The unified levels drop down box will not be usable on a "real" server. I think one needs to either have a more complicated interface with different leveltypes and stuff, or have a reduction to a (much) smaller set of layers as in the example.
Switch pressure levels seems to work, selecting, e.g., 2PVU does not seem to do anything. This is confusing. I assume that the PV plots change, but that is not visible. The disconnect between action and appearance need be adressed. E.g. either only display plots of a type or have multiple boxes for different level types...?
We could also restrict a specific gallery to one type of plot, i.e. offer multiple galleries on one server for different products to circumvent the problem.
I see that this is contradictory to the purpose of documentation, where we want all plots on one page.
It would also be nice to have a similar interface on the individual plot page, because the plots on the main page are very small and I do not see how I could enlarge them (and if they were enlarged the UI would be far away).

The 17th is the init-time, the 18th and 19th are the valid times for this init-time. It looks odd mostly because there aren't multiple init times. One click in the time selection directly chooses init and valid times.
I am also very undecided on how to do the level selection, as layer have different units for it. Right now, changing a level only changes the layers whose unit is in the selected level, the others stay as is.

Integrating the init time as such is very efficient, but with the given times it was a bit confusing. With "real" data, this will likely be obvious.

Level selection now hides all layers which don't have the specified unit. There is now also a time and level select inside the individual plot pages.


Fix no error output in quickstart

Show the captured stderr in case of a error exit code
Makes fixing problems easier

Codecov Report

Merging #443 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #443   +/-   ##
=======================================
  Coverage   92.13%   92.13%
=======================================
  Files          16       16
  Lines         865      865
=======================================
  Hits          797      797
  Misses         68       68

Flag
Coverage Δ

#bridge_end2end
84.97% <ø> (-0.47%)
:arrow_down:

#bridge_unit
75.95% <ø> (ø)
: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 92e6bb3...17a369d. Read the comment docs.


about joint detector

Why not use openpose, alphapose joint detector？The current joint detector hrnet is slower than alphapose and openpose?

Hi there, OpenPose and AlphaPose are faster, but we care more about accuracy and less about speed during this CVPR challenge, especially the accuracy for hands keypoints. That's why we chose the HRNet trained on whole-body pose dataset.
As a bottom-up method, OpenPose is less accurate than most top-down methods. I haven't checked out the new version of AlphaPose yet (which includes the hands and feet keypoints). It might also be a reasonable choice if it is accurate enough.

Thank you for your reply, and thank you for the open source code.


Implement first POC

Simple implementation:
class Dataset:
    def __init__(self, name: str):
        ## 1. Check if available locally
        ## 2. Check if available on GS and download it
        if not is_available_in_cache(name):
            self._fetch_from_gs()
        self.path = cache_path
    def _fetch_from_gs(self):
        """Fetch the dataset from GS and store it in cache for later use."""
        pass
    def to_npy_arrays(self, inputs, targets):
        # If 1 input: X is an numpy array
        # If several inputs: X list of numpy arrays
        # This format is usable by tf.keras.Model.fit(...)
        # cf https://www.tensorflow.org/api_docs/python/tf/keras/Model#fit
        return X, y
    def get_embeddings(self, column):
        """Return a 2D numpy array with the CLS ProtBert embeddings of each sequence."""
        return X
    def to_torch_data_loader(self):
        return Data

[x] Possibility to cache the dataset
[x] load_dataset() function which calls this dataset
[ ] Progress bar when download from GCP

We first only support .csv dataset with .npy aligned files for the embeddings of a sequence column.

Progress bar done in https://github.com/DeepChainBio/bio-datasets/issues/6


Add monthly_hrs to aws_db_instance

We shutdown EC2 and RDS SQL server instances when not-in-use. Can the ability to calculate costs based on number of uptime hours per month be added please for EC2 and RDS instances using the usage-based-resources concept. Thank you!

Hi @saucecon-aanand - great question! For EC2, that's already supported in the usage-file, see this reference file for the monthly_hrs option.
The same can be added to the aws_rds_cluster and the aws_rds_cluster_instance resource, https://github.com/infracost/infracost/pull/1812 is the pull request that shows how in case you or other community members would like to try it.

Thanks, I will follow the reference file for EC2.
My request regarding RDS was more towards aws_db_instance resource, if this resource can also support monthly_hrs option.

Thanks @saucecon-aanand good point, I mixed-up that resource with the _cluster resources, I updated my previous comment as the monthly_hrs can be added to that too.

Howdy, any movement on this?