ContextFormSerializer misses presets fields

Missing presets fields
In ContextFormSerializer presets are not represented. But it's needed in order to validate form from the schema or for UI representation.
https://github.com/novafloss/django-formidable/blob/master/formidable/serializers/forms.py#L64
For Information, Formidable provides different access based on the role of the current user about accessing some fields. The "ContextFormSerializer" provides the final representation of the form based on specific role
For example, let's imagine a form , which actually provides two fields.
=> full name , required for a padawan
=> weapons , hidden for padawan, required for a jedi

>>> Formidable = Formidable.objects.get(pk{id})
>>> schema = ContextFormSerializer(instance=formidable, context={'role': 'padawan'})
>>> 'weapons' in [field['slug'] for field in schema['fields']]
False
>>> print filter(lambda x: x['slug'] == 'fullname', schema['fields'])[0].required
True
>>> schema = ContextFormSerializer(instance=formidable, context={'role': 'jedi'})
>>> 'weapons' in [field['slug'] for field in schema['fields']]
True
>>> print filter(lambda x: x['slug'] == 'fullname', schema['fields'])[0].required
False

The serializer actually does not serialize a presentation of its presets.
What are presets ?
Presets are bunch of action pre-programmed inside the formidable projects. For the moment, presets are purely global validations on the built forms. For example, in order to compare two fields, etc.
Presets have already their own representation with the
https://github.com/novafloss/django-formidable/blob/master/formidable/serializers/presets.py#L104
Ensure to retrieve the presets associated to the formidable objects (tests it !)
You can inspire yourself from the FormidableSerializer
https://github.com/novafloss/django-formidable/blob/master/formidable/serializers/forms.py#L15
Tips: The FormidaleSerializer is a write/read serializer ContextFormSerializer in just a read-only serializer.

fixed by PR #176


When a pkg_tar target is imported from an external dependency, strip_prefix='.' does not strip the 'external/repo/path' prefix.

For posterity, please provide more details as to the nature of the problem.  A brief example would be very helpful.

Seems to be related to the issue and workaround described in https://github.com/bazelbuild/rules_pkg/issues/251#issuecomment-499464418 -- possible duplicate.

As discussed in the resolution for #478, the recommended path forward is to use the pkg_filegroup framework for this, where this problem is already solved.


Dotnet SDK install on Arch broke startup

Version
Microsoft Windows [Version 10.0.22000.346]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Arch
Other Software
dotnet-sdk 6.0
Repro Steps
Install dotnet-sdk from pacman
Expected Behavior
WSL starts
Actual Behavior
WSL doesn't start and throws the following error
It was not possible to find any compatible framework version
The framework 'Microsoft.NETCore.App', version '5.0.0' (x64) was not found.

The following frameworks were found:
6.0.0 at [/usr/share/dotnet/shared/Microsoft.NETCore.App]

You can resolve the problem by installing the specified framework and/or SDK.
The specified framework can be found at:

https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=5.0.0&arch=x64&rid=arch-x64

[process exited with code 150 (0x00000096)]
Diagnostic Logs
No response

Just a dependency error with https://github.com/arkane-systems/genie. WSL not the issue - closing.


CHAOS-6544 Update first-chaos-experiment tutorial

Update first-chaos-experiment tutorial

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d4a9ed2d0c7b39218818eb--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d5c5de8b324c126cda8811--harness-developer.netlify.app


bugs with closers wrapping around comments

WIP with PR-58

This was a doozy of a problem to figure out. Hopefully PR-58 closes this out for good.

I'm following this project with great interest.

Thank you! Your interest is appreciated :nerd_face:

Why is the following considered beneficial?

The short answer is compatibility for Parinfer users. See parinfer issue #92 for some historical context.
We also have the gather trailing parens recommendation from the Clojure Style Guide, which would apply in the following example:
[aaa
 [bbb ccc]
 [ddd eee]
 ;; fff
 ]

I will give this problem a think, but I am currently leaning towards the way it is written now.
I think the core issue comes down to "should comment lines be considered elements of the form they are in?", and I think the answer should be "no, they are not". There is probably some friction between the process of structural editing (ie: paredit) and that approach.
Undoubtedly, how to treat comments will be one of the harder parts of this project. And it is unlikely that the solution will be optimal for all possible code examples.

Thanks for taking the time to respond!

compatibility for Parinfer users

I can't say I'm a fan of parinfer - it is truly an impressive project which can make it easier for some folks to work with a LISP but I'm on the side of being in full control of the source code for which I find paredit and its derivatives perfect. I do understand your goal of making this tool play nice with all ways of editing code though.

how to treat comments will be one of the harder parts of this project

I fully agree. Even paredit-likes don't always do what I expect them to do with comments. On the other hand, paredit doesn't prevent me from updating the code after to place comments where I think they should be.

the gather trailing parens recommendation from the Clojure Style Guide

There's another recommendation about margin comments there that is perhaps even more closely related and it also says "Avoid using those in situations that would result in hanging closing parentheses". I generally agree with this but I think there are exceptions.

See https://github.com/parinfer/parinfer.js/issues/92

I'd be interested to know why you think not doing this would break compatibility with Parinfer. I see there is some back-and-forth in the linked issue and even Shaun himself agrees that this is something controversial that Paredit does and created https://github.com/parinfer/parstager/issues/1 to perhaps mitigate the effect.
In my opinion - due to the reasons Shaun laid out - paredit is more or less forced to make a controversial source code change. Do you think this tool has to necessarily replicate that same controversial edit? My points are:

If a dev uses paredit and as a result makes this change, it won't be reverted by this tool, with or without #58 (if I understand correctly)

this means that if the dev triggering the change agrees with it, they can still have it
important to note that the code change here is triggered by an editing tool that the dev chose

A dev not using parinfer will be forced to suffer the change with #58

compared to point 1, the dev did not make the formatting change here

While I also prefer to gather trailing parens most of the time and in general that's a good advice I think being able to add comments this way is somewhat akin to whether you align map values or not - sometimes it's better to align and sometimes it's better not to. I think it's a wise choice from this tool to leave map value alignments up to the developer and my preference would be the same with regard to these trailing comments too

"should comment lines be considered elements of the form they are in?" ... "no, they are not"

I agree with this in general, but even if they aren't elements of forms, they are a very important part of the source code itself and often extremely helpful when done well.
Just food for thought. Thanks for reading.

I have given this much hammock time over the last several days and I think the best path forward is to wrap the closing parens "over" the comment. ie: how it is currently implemented via PR-58
both ways are fine
If I were doing a code review and I saw either way of formatting, I don't think I would ever suggest changing to "the other way". In other words, I do not think the difference between these two styles has a large effect on the readability of the program: both are acceptable.
Given that, I think it makes sense to pick the way that can always be enforced (wrapping), does not cause friction with Parinfer users, and aligns with the Clojure Style Guide recommendation.
it is better to be consistent
To the extent that Standard Clojure Style can always produce the same result, it benefits everyone. I have a strong bias toward "making a decision" instead of "let the user choose".
This will not always be possible (ie: this project is not a strict pretty printer), and there are cases where I think it is important to allow the user to nudge the formatter in a direction ("Rule 3" indentation, vertically aligning let forms and maps), but every instance of flexibility allowed is a potential additional diff when reviewing code, which lessens the overall value of adopting this tool.
once it is "one way", I don't think this will be important
Colin Fleming has a good quote from the Clojureverse thread that spawned this project:

"Formatting is something that devs tend to think is very important, until they can't control it and then in general as long as the output is decent they don't care."

The comment from user oskarth on this issue is instructive to consider: is the issue here the formatting? Or his expectation of what the formatting should be? I think it is the latter, and if tooling always made it work in the same way then that expectation can go away for everyone. This sort of comment on a Pull Request is exactly the kind of small friction that I hope for Standard Clojure Style to remove for teams who adopt it.

Thank you for the interest and thoughtful comments @imrekoszo! If everyone who engages with this project brings the same level of thought and consideration as you, we are sure to all find success :blush:

Just for clarification – it wasn't entirely clear what "wrapping over" meant, and I had to inspect the commits in the linked PR to figure out what it was implementing.
Was the final decision to follow Parinfer's current behaviour? i.e.

OriginalAfter formatting

[aaa
 [bbb ccc]
 [ddd eee]
 ;; fff
 ]

[aaa
 [bbb ccc]
 [ddd eee]]
;; fff

I'm surprised that no one has mentioned the #_ discard form, which is intended precisely for this sort of 'commenting out' of code.
It's compatible with non-dangling parens and sidesteps the issue entirely, for the case of commented-out code:

OriginalAfter formatting

[aaa
 [bbb ccc]
 [ddd eee]
 #_fff
 ]

[aaa
 [bbb ccc]
 [ddd eee]
 #_fff]

However this does not address the case of non-code comments, which are the main topic of discussion in the ongoing linked Parinfer thread.
In standard lisp jargon, the proposed rule could be seen as the "barfing" of the line comment out into the outermost structure containing the first subsequent non-commented line - possibly top-level. (I suggest using this term instead of the potentially ambiguous "wrapping")
Since the implied nesting of comments within the code might convey significant information, having a one-size-fits-all formatter violate this structure doesn't seem ideal.
Here is a contrived example to illustrate such a risk:

OriginalAfter formatting

[:attributes
 [:size #{:small
          :medium
          :large}]
 [:color #{:red
           :green
           ;; WARNING: do not extend this list!
           }]
 [:other :prompt/user]]

[:attributes
 [:size #{:small
          :medium
          :large}]
 [:color #{:red
           :green}]
 ;; WARNING: do not extend this list!
 [:other :prompt/user]]

Interpretation: no more colors allowed!
 => *no more attributes allowed!

From the reader's perspective, this is of course a semantically invalid transformation.
Even though there are obvious fixes like rephrasing the comment or placing it above the [:color .. line, these rely on spotting the "error" in the first place, which can be easy to miss in the context of an automatic formatter producing plenty of other semantically neglible diffs in the background.

Alternative proposal
One surprisingly robust workaround I've found over the years for this class of issues is to use a single comma character , as a sort of "anchor" for trailing delimiters - e.g. for the above examples:
[aaa
 [bbb ccc]
 [ddd eee]
 ;; fff
 ,]

(aaa
 [bbb
  [ccc
   ddd ;; need to use ddd here because of JIRA-1234
   ;; append more items to increase blergh threshold as needed
   ,]
  ggg])

[:attributes
 [:size #{:small
          :medium
          :large}]
 [:color #{:red
           :green
           ;; WARNING: do not extend this list futher!
           ,}]
 [:misc :prompt/user]]

(Note: commas are syntactic whitespace in Clojure, so this has zero effect on the expression being read)
This happens to play nicely with language-agnostic Parinfer, which treats the comma as a symbol constituent, leaving it and any following delimiters untouched.
I don't recall where I first learnt this 'trick', but it seems to be relatively rare / undocumented in the wild.
Perhaps this should be brought up on the Clojure style guide repo - if the guide ends up recommending such a usage, ideally the formatter could detect such a situation and automatically insert these anchors:

OriginalAfter formatting

[foo bar
 ]

[foo bar]

[foo ;; bar
 ]

[foo ;; bar
 ,]

[foo
 bar
 ;; TODO: implement quux
 ]

[foo
 bar
 ;; TODO: implement quux
 ,]

@yuhan0 thanks for the alternative suggestion. Sadly, the important point you raise in your previous comment still applies:

these rely on spotting the "error" in the first place

i.e. one needs to be aware that the formatter will make or has made a "semantically invalid transformation" and prepare for it/work around it
@shaunlebron's suggested resolution of the Parinfer issue appears very well thought out to me.
Allowing these still go against the general recommendation about margin comments in the Style Guide, but as @yuhan0 points out, there will be exceptions when you want to be able to add comments like this.

@yuhan0 just reacting to the examples you added in your earlier comment:
I think cases B, C, and D should be left alone by formatters as there might be a semantic importance of the comments being inside the vector.

Yeah, that was my line of thought originally, but then I find it hard to imagine any scenario where one assigns semantic importance to the nesting of inline comments.
Writing such comments 'in the margin' is nearly always an indication that they are directly referring to the code on that line, rather than belonging in any larger block structure.
In practice a far more common scenario is the following: an edit is made that leaves dangling parens following an inline comment: eg. deleting the forms z (quz y) in:
(let [x (foo) ;; Warning: side effects!
      y (bar x) ;; returns nil if invalid
      z (qux y)]
  (do-something x y))

Which results in this situation, where users may find difficulty getting their editor to un-dangle the ] without (temporarily) breaking the structure.
In other words, a great use-case for letting a formatter deal with it automatically :)

(let [x (foo) ;; Warning: side effects!
      y (bar x) ;; returns nil if invalid
      ]
  (do-something x y))

(let [x (foo) ;; Warning: side effects!
      y (bar x)] ;; returns nil if invalid
  (do-something x y))

@yuhan0 I do find your classify-squares example to be okay, but I still wouldn't enfore the barf. The let and comment examples I find much better with the dangling closer.

has to focus on being minimally invasive and respect one's intent

I think preserving intent is important no matter when the tool runs. It might even be more important for a check-in-time tool as you might not notice the tiny-yet-intent-hiding edit it makes when all you're thinking about is to finish the coding session.

simply the fact that decades-old norms around formatting Lisp code very strongly proscribe against them

Wait till you learn that this tool throws out the decades-old Lisp tradition of semantic indentation in favour of fixed indentation 😅

Yep, I'm aware of the indentation matter and I like the direction the tool is going there. My point was that such a deviation from tradition shouldn't be taken lightly – in that case the tradeoffs of REPL-driven vs static analysis were judged to be worth the cost of breaking tradition, a consideration that doesn't apply here.
In any case I'm not claiming to be the best judge of user intent / formatting choices etc, but I'm curious @oakmac what procedure you follow in such situations where there might not be a clear consensus on the Right Thing to do? (i.e. not covered by the existing Clojure Style guide either)
Maybe collate them and have some sort of community poll near the end? (It's unclear what extent this is intended to be a personal vs community oriented project) Thanks for working on it!

Wait till you learn that this tool throws out the decades-old Lisp tradition of semantic indentation in favour of fixed indentation 😅

@imrekoszo This isn’t true, right?  It compromises the two approaches by letting the user set the “semantic” indentation of the first line, then it “fixes” the rest of the lines to follow it.

Just a quick note to say that I am monitoring the discussion in this thread and think there are some great points being raised. I am confident that the current behavior (as of v0.3.0) will need to change.

From @yuhan0 comment:

After edit:Desired format:

(let [x (foo) ;; Warning: side effects!
      y (bar x) ;; returns nil if invalid
      ]
  (do-something x y))

(let [x (foo) ;; Warning: side effects!
      y (bar x)] ;; returns nil if invalid
  (do-something x y))

I agree with "Desired format" here.
If I saw the "After edit" format on a code review, I would probably suggest changing it to "Desired format", and that suggestion would be a nit, which is exactly the sort of PR comment that I hope Standard Clojure Style helps people avoid having to make.

One non-contrived situation where I suspect this rule will cause annoyance/friction is with "rich comment blocks"

I agree with this and think Standard Clojure Style needs a good answer / solution here.
I really like the ,)) trick because it 1) explicitly expresses developer intention to "hold" the parens and 2) plays nice with Parinfer.
I do not want Standard Clojure Style to add the comma automatically, but it may be necessary to promote "if you really do not want Standard Clojure Style to gather your trailing parens to one line, use a comma to hold them" advice.

I should clarify that Standard Clojure Style is not being designed as a "tool for Parinfer users" anymore than it is a tool for Paredit users or users who use no editor tooling to help manage their parens (yes - there are people who work in this way).
It needs to support as many Clojure developers as it can, work in as many environments as it can, and reduce friction and diffs between users who edit their code in different ways.
In this particular use case, there is friction between Parinfer users and Paredit users because Parinfer users never have the condition of trailing parens below a single comment line, whereas a Paredit user might actively choose the code to look that way. It is not "one group thinks it should look one way, and the other way is wrong". It is "one group doesn't think about this at all, and the other group sometimes does".
I think that the idea of a ;; comment being treated as "part of the structure" comes more from the editing process (ie: Paredit) than from reading the code. classify-squares is a great example of this IMO.

The rules for margin comments in the Clojure Style Guide seemed off to me, so I’m proposing a clarification here: https://github.com/bbatsov/clojure-style-guide/issues/258

I really like the ,)) trick because it 1) explicitly expresses developer intention to "hold" the parens and 2) plays nice with Parinfer.
I do not want Standard Clojure Style to add the comma automatically, but it may be necessary to promote "if you really do not want Standard Clojure Style to gather your trailing parens to one line, use a comma to hold them" advice.

Ha, I like this trick and completely forgot about it.
One thing to consider is that if the formatter requires the comma to be there, but doesn’t add the commas automatically, then it will create a lot of initial diffs in existing codebases have to be manually corrected per project.
Assuming I understand this correctly, if we want to avoid those diffs, this means either 1) Standard Clojure Style has to add them in automatically, or 2) Parinfer has to be updated to not require them.

Wait till you learn that this tool throws out the decades-old Lisp tradition of semantic indentation in favour of fixed indentation 😅

@imrekoszo This isn’t true, right? It compromises the two approaches by letting the user set the “semantic” indentation of the first line, then it “fixes” the rest of the lines to follow it.

@shaunlebron by fixed indentation I meant "Multi-line lists that start with a symbol are always indented with two spaces" per the Nikita's article, with the addition of your rule "3. multi-line lists are indented to "first-arg alignment" if and only if the 2nd line is indented as such". (It would perhaps be useful to give a name to this composite style, perhaps Soft Fixed or Hintable Fixed indentation 😅 )
By semantic indentation I meant what's in Bozhidar's article, with further details in the CIDER Indentation, and Indentation Specification articles. The bottom line of these is that formatting needs to understand the semantics of forms and apply different formatting rules based on those semantics (special args indented one space, body indented two spaces, etc.)
In the fixed vs semantic debate I agree with the Hintable Fixed position due to its general applicability (formatter doesn't need to know the semantics), but still letting the programmer (who does know) to align with the first arg if they consider that more expressive.
On the other hand, this allowing of Rule 3 carries the potential of PR "nit" comments, similar to what @oakmac points out above. Both the following are valid according to Standard Clojure Style, but only one each per either Semantic of Strict Fixed.
;; semantic ❌ strict fixed ✅ hintable fixed ✅
(map +
  (range 10)
  (range 10))

;; semantic ✅ strict fixed ❌ hintable fixed ✅
(map +
     (range 10)
     (range 10))

For the sake of completeness, here are the narrow formats too, of which each of the 3 formatting specifications only allow one:
;; semantic ✅ strict fixed ❌ hintable fixed ❌
(map
 +
 (range 10)
 (range 10))

;; semantic ❌ strict fixed ✅ hintable fixed ✅
(map
  +
  (range 10)
  (range 10))

I suppose there is a symmetry between "requiring a comma to prevent barfing the comment" and "requiring (manual/tool-assisted) lining up the second arg with the first to prevent the strict 2-space indent". Both of these require effort from the programmer in case they expect a benefit from the "alternative" formatting.
While aligning with the first arg is common practice, leaving commas around is less so, and Shaun is right to point out the possibility of additional initial diffs. Note that since the narrow indentation format is changing from semantic to fixed, that will probably bring even more initial diffs. However, those diffs in my opinion preserve programmer intent better than barfing a trailing margin comment, but again, it could be just me.

if the formatter requires the comma to be there, but doesn’t add the commas automatically, then it will create a lot of initial diffs in existing codebases that have to be manually corrected per project

It is not a goal of Standard Clojure Style to "avoid initial diffs when applied to a codebase for the first time". I suspect that for 99% of forms, gathering the trailing parens to one line generally improves the readability of the code. The "comment line holding trailing parens" is the exception, not the norm.

"Standard Clojure Style will always gather trailing parens to a single line (as recommended by the Clojure Style Guide and enforced by Parinfer). If you want a ; comment line to 'hold' some parens on the next line, place a comma before them to explicitly show that intent."

@oakmac I think it's still beneficial to allow indented comments to preserve their position in the AST, per https://github.com/parinfer/parinfer.js/issues/92#issuecomment-2334830694

"Standard Clojure Style will always gather trailing parens to a single line (as recommended by the Clojure Style Guide and enforced by Parinfer). If you want a ; comment line to 'hold' some parens on the next line, place a comma before them to explicitly show that intent."

I was thinking about changing the Parinfer enforcement, but maybe that ship has sailed.  I’m at least leaning toward an option for auto-inserting the commas.
I requested another clarification in the Clojure Style Guide to see if we can add a recommendation there about indented comments, or at least a mention.

Relevant to this thread, I learned a fun way to "hold" parens on the next line from @aiba at Clojure/conj 2024. Example rich comment block:
(comment
  (something)
  (something-else :foo :bar)
  (another-thing)
  :-)


Snake case API and OO API, fully typed

Making this pull request to keep track of remaining items to do.
API Structure, make modules of:

[x] Pickups
[x] Dialogs
[x] Menus
[x] Objects
[ ] Timers
[x] Player
[x] Textdraws
[x] PlayerTextDraws
[x] Textlabels
[x] Gangzones
[x] Actors
[x] Vehicles
[ ] Globals (server settings, etc)

Docs
At least a single line docstring in all methods of:

[x] Pickup
[x] Dialog
[x] Menu
[x] Objects
[ ] Timer
[x] Player
[x] Textdraw
[x] PlayerTextDraws
[x] Textlabel
[x] Gangzone
[x] Actor
[x] Vehicle
[ ] Globals (server settings, etc)
[ ] samp.pyi (Not prioritized)

Other tasks

[ ] Rewrite Grandlarc to OO API
[ ] Update readme to reflect the OO API
[ ] Add event methods to various classes

Closes these issues:

#30
#46

As discussed on Discord, it could be (and probably will) a good idea to put docstrings in samp.pyi ; this way, we could simply rename it to samp.py for documentation generation step! :-)

I would assume current progress is at 80-85% completion.
The biggest modules are documented and typed, like player.py, vehicle.py.

This PR is closed in favor of #85 .


add cross entropy and std_mean

Adds cross entropy and std_mean from #6146 with regression tests

tests are way too verbose


Docs.MS CI Updates for Azure.Template

CI Update for release of Azure.Template

Docs Build status updates of commit a7ef702:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


add dynamic route section to Next.js Pages guide

This PR adds a section of how to use Next.js dynamic routes on Cloudflare Pages. This is an information I myself was missing when reading the docs, and something that also came up in Discord just recently. So I think it is something a lot of people would like to know when reading the docs.
It is probably a good idea to have a technical writer look over the PR as English is not my native language. If this lands, it probably makes sense to add a similar section to the Gatsby docs (and maybe other frameworks, but Next.js and Gatsby the only ones where I've used dynamic routes in the past).

@rkusa looks great! Do you have an example repo we can test this with?

Do you have an example repo we can test this with?

@codewithkristian Yes, here is one: https://github.com/rkusa/cf-pages-next-test
I've added some examples:

main branch: uses the function from the code snippet of this PR; example: https://cf-pages-next-test.pages.dev/post/42
no-rewrite branch: same dynamic route, but no function to rewrite; example: https://cb0c5741.cf-pages-next-test.pages.dev/post/42 (you'll get a 404 without the rewrite)
trailing-slash branch: enabled trailingSlash Next.js option without any other change; example: https://c3629fae.cf-pages-next-test.pages.dev/post/42 (redirect loop)

trailing-slash-fixed branch: fixed the above by adding the trailing slash / to the assets request; example: https://5b584593.cf-pages-next-test.pages.dev/post/42/


Create patternprint

this program is used to print pattern

accept for hacktoberfest and this code is done by me.


When a snippet is selected, how can i cycle through the arguments with the tab key and make the necessary changes.

what i really want is like kite pro completions
https://help.kite.com/article/90-using-the-spyder-plugin
i am working on spyder 4.1.3, ubuntu 18.04

@Magicwangs thanks for your question.
Could you provide a more detailed description (with an example code) of what you want to achieve?

kite pro completions

That is a subscription service that must be paid for to kite AFAIK.

just like this following picture
the third completion is what i really want, but i have to use  “↑”，“↓” to switch,
can i use "tab" to do this switch

the third completion is what i really want, but i have to use “↑”，“↓” to switch,
can i use "tab" to do this switch

Nope, and we will move to use Tab as the key to select the snippet, so enter does not include completions when all you want to do is type an enter to move to the next line.
Thoughts @ccordoba12 ?

We could try ctrl+tab and shift+tab to move up and down. Then tab to select.

That is a great idea @bcolsen. I agree!


fix RAM access issues

renamed boot_data to do not overlap with global boot_data defined in loader.c;
modified a type cast to avoid incorrect value assignment (observed on Cypress PSoC6 - Cortex CM0p, GCC 7.2.1);

Yes, you're right.
cleaned it and verified on a device


Getting a reference to the resolved command when .throwErrors() is used

Hey there! I've run into a nontrivial structural error in my use of Cliffy. Because we might encounter exceptional conditions across a number of situations (a 401 error thrown as a DOMException, for example), I was hoping to be able to use .throwErrors() in Cliffy as a pathway to a global event handler. However, we'd like to show usage help when we catch an error from Cliffy in the same way that the default behavior does. We have subcommands, though, and so when we catch ValidationError, there's no reference to the command that had actually been called from upon which we can invoke .showHelp().
Could you point me towards the best way to be able to be responsive to CommandErrors without having to wrap every action() in its own error handler function?
Thanks!

Hey, sry for late answer, i thought i already gave an answer.
This is currently not possible but thanks for pointing this out. I will think about a solution for this.


Mask from atlas failing at some leaves

When getting masks from atlas regions. I am using:
from bg_atlasapi import BrainGlobeAtlas
import imio

bg_atlas = BrainGlobeAtlas("allen_mouse_25um")
stack = bg_atlas.get_structure_mask(672) # for CP (see image below)
stack = stack.astype(bool)
imio.to_tiff(stack, "/path/to/image.tiff")

This gives an empty image
However, if you use the hierarchy above (485), it does work

Ok, v1.0.2 is out, so just updating to the latest version should fix the issue!


React - jsx

Hi,
I'm a huge fan, but with React and jsx it doesn't work.
Thanks

Are they listed in the file type matrix? Do you mean the file types aren't supported yet or they aren't working properly?

Hi,
Maybe I'm misunderstanding how the vim filetype declaration works but I have my .js files set as javascript.jsx so I can get js and jsx formatting.  These files have a mix of plain js and blocks of jsx (typical react files) yet with this change all comments are of the form {/* to comment out */}. But this doesn't work on plain js. For example i see this
// This is correct
const render = () => (
  {/*
 *}
)

// This is incorrect
{/* const a = 'hello' */}

@chrisnojima Vim doesn't provide (at least not readily) context sensitive information about what type of language block you are it. As far as NerdCommenter is concerned this is pretty much a one shot deal. Either the filetype is plain JS (in which case you get  plain JS comments) or it is JSX, in which case you  get React style comments. It doesn't try to parse the surrounding lines to toggle back and forth between language modes.
There are several languages for which this would be desirable, but I'm not aware of a good way to implement this yet. I would suggest opening this as a separate issue (if we don't  have one already) as a potential improvement. I'd be happy to see a contribution to this effect.
In the mean time I don't thing you have an easy option to handle this other than  changing the file type or overriding the JSX comment type to generate plain JS comments.

Ok, IMO the fix that got merged in for this should be reverted.
JSX files are just Javascript really, with some specialized syntax every once in a while. This current change breaks commenting completely for JSX files. My vote would be to revert this change and just force people to comment manually within JSX scope.
Take this super simplified example:
import React, {PureComponent} from 'react';

class MyComponent extends PureComponent {
    handleClick() {
        console.log('You\'ve clicked the element!');
    }

    render() {
        return (
            // The only actual JSX witin the file

 this.handleClick()}>Example JSX syntax

        );
    }
}

export default MyComponent;

It's now impossible to use NERDCommenter anywhere within the file, EXCEPT within that single one line in the return statement.

totally agree with @amadeus, this change took me by surprise. I very rarely comment actual jsx code.
to bring back previous behaviour one can override delimiters for javascript.jsx filetype to be same as for javascript:
let g:NERDCustomDelimiters = { 'javascript.jsx': { 'left': '//', 'leftAlt': '/*', 'rightAlt': '*/' } }

ugly but you know, works

Thanks for the feedback @chrisnojima, @argshook and @amadeus. I've reverted the commit that added JXS "support" pending a more robust solution.

I use this in my .vimrc:
let g:NERDCustomDelimiters={
	\ 'javascript': { 'left': '//', 'right': '', 'leftAlt': '{/*', 'rightAlt': '*/}' },
\}

That way, I can use the alternate style for jsx comments when needed, while still using normal js comments as default comment style.

@Addisonbean Those alternate delimiters are already in the default javascript language definition. You shouldn't need to add this manual override, it should be ready for that usage out of the box.

@Addisonbean these alts are different - in this override they include {

@amadeus Oh my bad. Sorry I didn't inspect that closely enough.

For those using Vim to program in React please see PR #393 and comments on our support for these filetypes in 41d31b3.

@Addisonbean these alts are different - in this override they include {
Javascript in this plugin is just:
    \ 'javascript': { 'left': '//', 'leftAlt': '/*', 'rightAlt': '*/' },

How do you use it? It make anything if I trigger the comment command

@Addisonbean these alts are different - in this override they include {
Javascript in this plugin is just:
    \ 'javascript': { 'left': '//', 'leftAlt': '/*', 'rightAlt': '*/' },

How do you use it? It make anything if I trigger the comment command

Hey, @diegoulloao!
You can trigger alternative delimiters using <leader>ca. You can read more about it in the README.md description.


Update aws-java-sdk-kinesis, ... to 1.11.758

Updates

com.amazonaws:aws-java-sdk-kinesis
com.amazonaws:aws-java-sdk-s3
com.amazonaws:aws-java-sdk-sqs

from 1.11.745 to 1.11.758.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws" } ]

labels: library-update, semver-patch

Codecov Report

Merging #253 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #253   +/-   ##
=======================================
  Coverage   47.68%   47.68%
=======================================
  Files          24       24
  Lines         281      281
  Branches        1        1
=======================================
  Hits          134      134
  Misses        147      147

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 e3a5120...989d96f. Read the comment docs.


Remove HorizontalRuleNode from lexical types

Removes HorizontalRuleNode and associated functions from lexical as these exports are not present in lexical.

Mind updating Lexical.js.flow too please?

Done


[master] Workflow updates

Add labeler workflow
Add triage workflow
Add Ubuntu CI workflow with Codecov upload
Update badges
Port CODEOWNERS
This needs https://github.com/ignition-tooling/ubuntu-bionic-ci-action/pull/4 for the examples compilation to work.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@620527b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #118   +/-   ##
=========================================
  Coverage          ?   98.80%
=========================================
  Files             ?       59
  Lines             ?     5695
  Branches          ?        0
=========================================
  Hits              ?     5627
  Misses            ?       68
  Partials          ?        0

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 620527b...36d3601. Read the comment docs.


Problem: Acceptance tests on CI failing

Description
Acceptance tests on Travis are failing because the following command fails during travis-install.sh
/bin/sh -c pip install --no-cache-dir --process-dependency-links -e .[dev]

More details in the travis build.
Solution
Update the travis matrix to enable acceptance tests for python36

We should remove Python 3.5 from our Travis CI, instead of forcing Python 3.6 for a specific row of matrix.


Foreach to run multiple experiments

Dear dvc team,
in the discord I recently posted a question about using a foreach loop and realized that my intended way of using foreach might not be supported (yet). So I was wondering if the scenario is something which could be supported, or if you have Tipps for me on how to setup my workflow better:
We tried to build a ML pipeline which is easy to use and basically automatically configured so the users do not have to worry about setting up dvc workflows themselves.
We start with a csv file. This file has our ML input and multiple columns. Each of the columns contains a different "y" so a different, but usually related, machine learning task.
In order to easily train all the models without needing to run dvc exp run multiple times we use a foreach loop to train the models. Our stage looks like this:
trainBest:
    foreach: ${endpoints}  # Specified in data_config.yaml
    do:
      cmd: python ./scripts/02_train_best_model.py --endpoint ${item}
      deps:
      - ./config/data_config.yaml
      - ./modeldata/model_performances/${item}_performance.tsv
      - ./data/${data_path}    # Specified in data_config.yaml
      - ./scripts/02_train_best_model.py
      outs:
      - ./modeldata/final_models/${item}.joblib
      - ./modeldata/final_models/${item}_config.json

Now I thought that ultimately this would run an experiment such that I have n lines of experiments, one for each foreach loop. However, this is not the case.
My current solution is to use dvclive and track each loop in a different directory, so the end of my 02_train_best_model.py script contains the following lines now (as suggested in discord) :
metrics_dir = f"./modeldata/final_models/{endpoint}"
with Live(dir=metrics_dir, save_dvc_exp=True, exp_name=f"Performance_hyperparam_{now_str}") as live:
    for col, value in best_performance.items():
        live.log_metric(col, value)
    live.end()

However, since I run the script with dvc exp run the save_dvc_exp and exp_name parameters are ignored, I was hoping that via those I would ultimately end up with the n lines (one for each y) when using dvc exp show.
Running without the foreach loop is of course possible but leaving my users with just running "dvc exp run" to get all the desired models and evaluations just seems like such an easy thing to do instead of again compiling bash scripts to do an external loop.
I would be happy if you could guide me or let me know if this is a use-case you would consider for the foreach stages.

@JenniferHem Have you tried keeping them all "horizontally" in one line? How related is each experiment? How would you like to be grouping each "run" that combines the endpoints if you were to have a line per endpoint?

@dberenbaum thanks for the answer. We work in the bio field so the endpoints really are considering similar biology. Actually while experimenting a bit I realized, that with the setup I explained using dvclive I do get the horizontal tracking as you mentioned, I just had a displaying issue which did not show me the full horizontal table. Considering that actually all endpoints are trained in the same run it makes sense to save it like this.
One thing I am wondering about, which might be me not fully grasping the capacity is if there would be a way to track the same metrics files etc as I do above, keeping them horizontally but without needing to spread the metrics.json files manually via dvclive as an additional factor. Does that make sense?

It makes sense. You should be able to use Live(resume=True) to keep appending to the same dvclive folder for each model run. There was a bug that was fixed in https://github.com/iterative/dvclive/pull/740 that was preventing dvclive from preserving metrics when resuming, so it will not work as expected until the next release unless you use the upstream branch (pip install git+https://github.com/iterative/dvclive.git).


Issue with dummy.fromParens(expr) not putting circular brackets.

syntax m = ctx => {
  let dummy = #`dummy`.get(0);
  let expr = #`5 * 5`;
  return #`1 + ${dummy.fromParens(expr)}`;
}
m

when I run it it gives output
1 + 5 * 5;
Expeceted output is
1 + (5 * 5)
As mentioned in Reference documentation

Codgen (after all the macros have been expanded) removes redundant parens
so this is expected (and correct) behavior.
On Mon, Jan 9, 2017 at 8:26 AM Harsharanga R Patil<EMAIL_ADDRESS>wrote:

syntax m = ctx => { let dummy = #dummy.get(0); let expr = #5 * 5; return #1

${dummy.fromParens(expr)}; } m
when I run it it gives output
1 + 5 * 5;

Expeceted output is
1 + (5 * 5)
As mentioned on Reference documentation
http://sweetjs.org/doc/1.0/reference.html#_fromstring
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sweet-js/sweet.js/issues/625, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAJOqB6SoaTpi-aHIXD9vkTxaP19QO1ks5rQl-sgaJpZM4Lecrg
.

syntax cond = function (ctx) {
  let bodyCtx = ctx.next().value.inner();
  let result = #``;
  let dummy = #`dummy`.get(0);
  for (let stx of bodyCtx) {
    if (stx && stx.val() === "|") {
      var c = bodyCtx.expand('expr').value;
      bodyCtx.next();
      let r = bodyCtx.expand('expr').value;
      result = result.concat(#`if(${c}){ ${r}(state, global) }`);
    }
  }
  return result;
}

cond {
  | condition1  : f1,
	|	condition2  : f2,
	| condition3  : f3,
	| condition4 : f4,
}

This gives output
if (condition1) {
f1 state, global;
}
if (condition2) {
f2 state, global;
}
if (condition3) {
f3 state, global;
}
if (condition4) {
f4 state, global;
}
Here f1 f2 f3 are function calls. how do add those calls.

Humm, that does look like a bug

@disnet this is totally broken in master. Issues with RawSyntax and setLineNumber.

How should I fix it? Any pointers. I was not able to find RawSyntax and setLineNumber is not called in my code.

@iamnewspecies Those are references to the internal implementation of sweet.js. You've discovered a bug that will have to be fixed. Hopefully in the next release.

@gabejohnson what were you seeing broken on master? I just tried @iamnewspecies's cond example and after updating to the breaking changes we made the output looks fine.
syntax cond = function (ctx) {
  let bodyCtx = ctx.contextify(ctx.next().value);
  let result = #``;
  let dummy = #`dummy`.get(0);
  for (let stx of bodyCtx) {
    if (stx && stx.value.val() === "|") {
      var c = bodyCtx.expand('expr').value;
      bodyCtx.next();
      let r = bodyCtx.expand('expr').value;
      result = result.concat(#`if(${c}){ ${r}(state, global) }`);
    }
  }
  return result;
}

cond {
    | condition1  : f1,
    |	condition2  : f2,
    | condition3  : f3,
    | condition4 : f4,
}

produces:
;
if (condition1) {
  f1(state, global);
}
if (condition2) {
  f2(state, global);
}
if (condition3) {
  f3(state, global);
}
if (condition4) {
  f4(state, global);
}

@disnet I was referring to the first example. Sorry about the confusion.

I figured out the solution for this.
syntax ON_END = function (ctx) {
  let bodyCtx = ctx.next().value.inner();
  let result = #``;
  let dummy = #`dummy`.get(0);
  for (let stx of bodyCtx) {
    if (stx && stx.val() === "|") {
      var c = bodyCtx.expand('expr').value;
      bodyCtx.next();
      let r = bodyCtx.expand('expr').value;
      result = result.concat(#`if(${c}(state,global))${dummy.fromKeyword('return')} (${r}(state, global));`);
    }
  }
  return result;
}

ON_END {
  | condition1  : f1,
  | condition2  : f2,
  | condition3  : f3,
  | condition4 : f4,
};

I just had to add circular brackets outside it (${r}(state, global));)`
Now I have one more issue
How do I put whatever output this generate inside a function call
syntax ON_END = function (ctx) {
  let bodyCtx = ctx.next().value.inner();
  let result = #``;
  let dummy = #`dummy`.get(0);
  for (let stx of bodyCtx) {
    if (stx && stx.val() === "|") {
      var c = bodyCtx.expand('expr').value;
      bodyCtx.next();
      let r = bodyCtx.expand('expr').value;
      result = result.concat(#`if(${c}(state,global))${dummy.fromKeyword('return')} (${r}(state, global));`);
    }
  }
  return #`f(state, global)${dummy.fromBraces(result)}`;
}

ON_END {
  | condition1  : f1,
  | condition2  : f2,
  | condition3  : f3,
  | condition4 : f4,
};

I tried the above code but it gives assertion error.
Error: [assertion error]: Not a primary expression
    at assert (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/errors.js:26:11)
    at Enforester.enforestPrimaryExpression (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:1201:24)
    at Enforester.enforestLeftHandSideExpression (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:1235:26)
    at Enforester.enforestAssignmentExpression (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:1113:19)
    at Enforester.enforestExpressionLoop (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:1021:23)
    at Enforester.enforestExpression (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:996:21)
    at Enforester.enforestExpressionStatement (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:984:21)
    at Enforester.enforestStatement (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:412:17)
    at Enforester.enforestModuleItem (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:124:17)
    at Enforester.enforestBody (/Users/newspecies/Documents/juspay/flow_framework/node_modules/sweet.js/dist/enforester.js:110:17)

Sorry my mistake. I should have done something like this let construct = #function ${name} () {};


Can't install: '@geospoc/v-mapbox-geocoder@latest' is not in the npm registry.

npm i --save @mapbox/mapbox-gl-geocoder vue-mapbox @geospoc/v-mapbox-geocoder
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@geospoc%2fv-mapbox-geocoder - Not found
npm ERR! 404
npm ERR! 404  '@geospoc/v-mapbox-geocoder@latest' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

Can you try and login npm so that you can access GPR pkgs?
$ npm login --registry=https://npm.pkg.github.com
> Username: USERNAME
> Password: TOKEN
> Email: PUBLIC-EMAIL-ADDRESS

Read more: https://docs.github.com/en/packages/using-github-packages-with-your-projects-ecosystem/configuring-npm-for-use-with-github-packages

I see the README was updated not to use latest, so will stick to that. No other packages required npm login.


Cannot read property 'setNewState' of undefined

Hello -
Thanks for this.  I am surely missing something basic, but when I try to display an alert, the showCurrentAlert function of MessageBarManager barfs @ line 47 - "Cannot read property 'setNewState' of undefined.
My project is set up with a redux store and a router.  When I just drop your example code into my project, it works fine.
Kaveh

Hi,
Can you please paste the code that call MessageBarManager.showCurrentAlert({...}), notably explaining how you use this component in your architecture?
Which version do you use?
Can you also please check you implemented the following step:

Add a <MessageBar ref="alert" /> to your top parent view/component. (Children will use that one)
Register and unregister the alert in, respectively, componentDidMount and componentWillUnmount
Call MessageBarManager.showCurrentAlert({...}) with the right properties

Thanks,
Kevin

Hello -
Thanks for helping.
I figured the problem out, totally on my side.  My render function briefly branches and returns a view while the saved state is loading, and I did not have the MessageBar in that first returned view, so this.refs.alert was null when the component mounted.
Thanks,
Kaveh

Hi,
I'm glad you solved your problem.
However, I'm open to any suggestion to improve this component. Feel free to submit a PR :)
Thanks
Kevin


ARROW-1444: [JAVA] fix last byte copy in BitVector splitAndTransfer

cc @jacques-n

The logic seems correct. I left some comments about readability.

Addressed all review comments

Addressed review comments

LGTM +1


How to get provider instances in runtime?

Information

Version: 5.27.4
Type: Question

Description
I have some static methods that may use other providers which have to be retrieved in runtime.
If I use DI in class constructor, I have to change all the static methods to instance method.
Are there any static utility functions can retrieve component in runtime like ProviderContext.getProvider<T>(provide: TokenProvider) : T?
Also is there any method to check whether the application finish initialization or not? this can prevent retrieving any uninitialized component. Thank you :)

Hello,
Isn't possible. Injector is built when the server is loaded. The context is the Server instance. So by design isn't possible.
If you want to call static method in instance method, you have to give your service by parameters

Also is there any method to check whether the application finish initialization or not? this can prevent retrieving any uninitialized component

class MyService{
  $onReady() {
     // server is ready
  }
}

more info here: http://tsed.io/docs/hooks.html

Thanks for fasy reply. Finally I resolve it by using singleton server:
class MyServer extends ServerLoader {
    public static getInstance(): MyServer {
        //Singleton
    }
}

And then retrieve the provider from the singleton In the static method:
let provider: MyProvider | undefined = MyServer.getInstance()
                    .injector
                    .get<MyProvider>(MyProvider);

Thanks for reply. Finally I resolve it by using singleton server:
class MyServer extends ServerLoader {
    public static getInstance(): MyServer {
        //Singleton
    }
}

And then retrieve the provider from the singleton In the static method:
let provider: MyProvider | undefined = MyServer.getInstance()
                    .injector
                    .get<MyProvider>(MyProvider);

Yes, This is the only way to get injector outside the server context :)
See you
Note: Don't forget to close this issue ;)


Loan Eligibility Prediction(ML)

🚀 Feature
This project is about loan prediction. It's a classification problem. It predicts whether a person's loan is approved or not based on various parameters taken into consideration. I will compare the accuracy using various algorithms such as logistic regression, decision tree classifier, linear discriminant analysis, knearest neighbour and gaussiannb to check the best fit. etc.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
To make an efficient solution to classification problem with good accuracy
Pitch
Things are getting automated everywhere. This would help bank officials in giving a faster decision on loans getting approved or not through machine learning.

/assign


Docker build & publish workflow is broken

Describe the bug
The workflow is failing with the following error messages, in some case:
429 error
error: failed to solve: busybox:1.34.1-musl: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/busybox/manifests/sha256:0497d8cd241e7a2c44564a33dddd6b1a06fe000fb36ff2b3cdc12337a0ae45f2: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
Error: buildx failed with: error: failed to solve: busybox:1.34.1-musl: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/busybox/manifests/sha256:0497d8cd241e7a2c44564a33dddd6b1a06fe000fb36ff2b3cdc12337a0ae45f2: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

or 503 error:
error: failed to solve: failed to compute cache key: failed to copy: httpReadSeeker: failed open: unexpected status code https://ghcr.io/v2/strangelove-ventures/infra-toolkit/blobs/sha256:14bccb3a1cff3b5f09bf1f21ba9ecb17c96329fb82f96605d922cfa4c9a82032: 503 first byte timeout
Error: buildx failed with: error: failed to solve: failed to compute cache key: failed to copy: httpReadSeeker: failed open: unexpected status code https://ghcr.io/v2/strangelove-ventures/infra-toolkit/blobs/sha256:14bccb3a1cff3b5f09bf1f21ba9ecb17c96329fb82f96605d922cfa4c9a82032: 503 first byte timeout

and others:
Error: buildx failed with: error: failed to solve: process "/bin/sh -c if [ \"$TARGETARCH\" = \"arm64\" ] && [ \"$BUILDARCH\" != \"arm64\" ]; then       export CARGO_TARGET_AARCH64_UNKNOWN_LINUX_GNU_LINKER=aarch64-linux-gnu-gcc         CC_aarch64_unknown_linux_gnu=aarch64-linux-gnu-gcc         CXX_aarch64_unknown_linux_gnu=aarch64-linux-gnu-g++         PKG_CONFIG_SYSROOT_DIR=/usr/aarch64-linux-gnu;       cargo build --release --target aarch64-unknown-linux-gnu;     elif [ \"$TARGETARCH\" = \"amd64\" ] && [ \"$BUILDARCH\" != \"amd64\" ]; then       export CARGO_TARGET_x86_64_UNKNOWN_LINUX_GNU_LINKER=x86_64-linux-gnu-gcc         CC_x86_64_unknown_linux_gnu=x86_64-linux-gnu-gcc         CXX_x86_64_unknown_linux_gnu=x86_64-linux-gnu-g++         PKG_CONFIG_SYSROOT_DIR=/usr/x86_64-linux-gnu;       cargo build --release --target x86_64-unknown-linux-gnu;     else       cargo build --release --target $(uname -m)-unknown-linux-gnu;    fi;" did not complete successfully: exit code: 101

The tweak in #1649 should help, but unfortunately we're still likely to hit pull limits for tasks like the smoketest. I confess, I have been rebasing frequently too lately. :innocent:

Here's another example of a build failure, from the smoketest job.

relevant error output
Run timeout --preserve-status $TESTNET_RUNTIME docker-compose up --exit-code-from pd-node0
using --exit-code-from implies --abort-on-container-exit
Creating network "penumbra_localnet" with driver "bridge"
Creating network "penumbra_monitoring" with the default driver
Creating volume "penumbra_prometheus_data" with default driver
Pulling tendermint-node0 (tendermint/tendermint:v0.34.21)...
v0.34.21: Pulling from tendermint/tendermint
Digest: sha256:1bfeb39391b231f99a0068ed93777e648367ebd18831f521cf8c384768fd8562
Status: Downloaded newer image for tendermint/tendermint:v0.34.21
Pulling prometheus (prom/prometheus:latest)...
latest: Pulling from prom/prometheus
toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
Error: Process completed with exit code 1.

For context, we saw about double the usual build rate recently. Here's a graph from the buildjet dashboard:

Proceeding with adding authenticated pulls on the docker calls. Will perform that work in #1649.


Document how to add environment variables

Why
Once deployed, a project often needs to add environment variables.
The template already provides a file structure to do that, but it is not documented 🙃
Who Benefits?
Future developers won't need to ask veterans on slack 🪖
How?
We can add a section in the README.md file directly, after the starter-kit (#83 ):
## Add environment Variables

> TODO: Describe how to add env variables using the file `service.json.tftpl` in the ECS module.

@hoangmirs would you be interested in handling it? 😇  (tho it might worth to wait #83 to be reviewed so we avoid conflicts)


When use Masonry in ".mm" file, it will conflict with the keyword "and" of C++.

Try this:
#define and end
#import <Masonry/Masonry.h>
#undef and

:)

Unfortunately Masonry at this point has too many projects that would break if we were to remove this keyword and it's only an issue for Objective-C++. For this I recommend making a fork or using k06a's recommendation.


New Release

rust-lang/rust#104499 is in bors queue, will need a release some time today.
Not super urgent, as code written agains old version will still work (if it doesn't check format version), but should be done timely anyway.

Done in 1e078f7022fe6351209115c91717e9238af3fc99, released as v0.19.0


Tagging users

I would like to request an option to tag + broadcast users.

So we can tag a device as being for a certain sports game for example maybe
"game_28482"

And then we can push to that tag and it gets delivered.

This would mean that messages would be delivered quickly instead of grabbing
who's subscribed from a database and sending the requests separately.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 21 Dec 2012 at 9:30

We added topic messaging to the new GCM API, that is implementing the feature that you request.


select ntext and SET TEXTSIZE and SQLSRV_ATTR_DIRECT_QUERY php fatal error Invalid sql_display_size

PHP Driver version or file name
php-sqlsrv-5.8.0
SQL Server version
Microsoft SQL Server 2016 (SP2) (KB4052908) - 13.0.5026.0
Client operating system
CentOS Linux release 7.6.1810 (Core). Linux 3.10.0-957.12.1.el7.x86_64  x86_64 GNU/Linux
PHP version
PHP 7.4.3 (cli) (built: Feb 27 2020 15:30:56) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.3, Copyright (c), by Zend Technologies
with Xdebug v2.9.6, Copyright (c) 2002-2020, by Derick Rethans
with ddtrace v0.46.0, Copyright Datadog, by Datado
Microsoft ODBC Driver version
msodbcsql17-<IP_ADDRESS>-1.x86_64
Table schema
DECLARE @T1 TABLE (C1 NVARCHAR(10), C2 NTEXT)
Problem description
If you use the prepared statement option SQLSRV_ATTR_DIRECT_QUERY = true, in addition to using SET TEXTSIZE away from the default -1, then you get a php fatal error Invalid sql_display_size
Expected behavior and actual behavior
Output should be one row from table, C1 = TEST1, C2 = NULL
You can get this output in the default settings of the attached program.
But with settings  DB_OPTIONS=1 DB_SET_TEXTSIZE=1 you get
PHP Fatal error:  Invalid sql_display_size
Repro code or steps to reproduce
You need to pass the database connection credentials to the attached php program using environment variables. To reproduce the error
env DB_SERVER=localhost DB_NAME=master DB_USER=user1 DB_PASS=pass1 DB_OPTIONS=1 DB_SET_TEXTSIZE=1 php error.php
But either of these combinations will produce correct output
env DB_SERVER=localhost DB_NAME=master DB_USER=user1 DB_PASS=pass1 DB_OPTIONS=0 DB_SET_TEXTSIZE=1 php error.php
env DB_SERVER=localhost DB_NAME=master DB_USER=user1 DB_PASS=pass1 DB_OPTIONS=1 DB_SET_TEXTSIZE=0 php error.php
error-program.zip

Thanks @gjcarrette for the report. We will investigate and get back to you.

Hi @gjcarrette the bug has been fixed and merged. You can test it with the next preview release.

Hi @gjcarrette, 5.9.0-preview1 is just released, which contains a fix to your issue. Please check.

Closing this issue @gjcarrette. Please feel free to reopen if necessary.


Added support for named credentials

Salesforce can handle oauth authorization and refresh.
Opted for a static factory method because of constructor clashes. It would probably be a good thing do refactor the other constructors into  withAccessToken and 3xwithClientId

Hi @jrz, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

CLA signed

On 18 Oct 2016, at 14:36, Box CLA Bot<EMAIL_ADDRESS>wrote:
Hi @jrz https://github.com/jrz, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/box/box-salesforce-sdk/pull/23#issuecomment-254493798, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJIpEL-UbOFgI919WVOkDLgvRYKlqHFks5q1L05gaJpZM4KZvDC.

Verified that @jrz has just signed the CLA. Thanks, and we look forward to your contribution.

Can you please update the authentication doc: https://github.com/box/box-salesforce-sdk/blob/master/doc/authentication.md with sample code using Named Credentials with the SDK?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Would it be possible to merge this? Or are there other ways to use Named Credentials (or some other way to avoid requests automatically trying to auth)?


Component:  Audio Player Functional Specs

User Story

As a [user role below]:
DDS developer

I need to:
have the Audio Player functional specs

so that I can:
so that I can properly code the Audio Player component that meets the Design functions and intent

Additional information

Overview and scope of the engineering work should be included in the linked epic (#4083)
Design Issue
Development issue (#TBD )
Visual Designs & Functional Design Specs Box folder link(TBD)

Acceptance criteria

[ ] Documented Audio Player functional specifications
[ ] Sent and/or presented the specs to Design and Engineering for their review and comments
[ ] Additional stories are created, if needed, for any work to be done in future sprints

This story definitely needs to carry over into sprint 4. This week I worked with Olivia and the team to propose a scoped back version of this work for our first release of an audio player teams can adopt.

Without focusing on the visual design I took Olivia's visual explorations and looked at how we could stage some of this work starting with as simple as possible, and becoming more complex.

CTA link
CTA link that navigates the user the kaltura to listen to the audio file.

Open in Kaltura in new tab
play icon
Title
Length

Audio faux embed (video as-is mostly)

Video player
Optional thumbnail
Default audio thumbnail

CTA embed
CTA link that toggles play and stop from within the page.

Play/(Pause) icon toggle
Title
Length
Progress indicator
alternative consumption [Transcript /link to kaltura with transcript (adopter addition via link, or built in?)]

Audio embed

Play/(Pause) icon toggle
Title
Length
Progress indicator
alternative consumption [Transcript /link to kaltura with transcript (adopter addition via link, or built in?)]
---------- beyond depends on size
possible different sizes (sm, md, lg)
thumbnail
playback speed (2x etc.)
fast forward
jump forward (8s)
fast backward
jump backward (8s)
transcript download
closed captions
more information

CTA popup
CTA link that pops up a modal with audio embed.

Open in Kaltura in new modal
play icon
Title
Length

Sticky audio embed — (Kaltura builds for their own experience)

audio embed full (xlg)
sticks to bottom of page
communicates with sticky overlay grid influencers
persistent when navigating within kaltura

Persistent sticky audio embed — not practical at the moment

audio embed full (xlg)
included in dotcom shell across all IBM pages
persistent when navigating from one page to another

Data visualization
With IBM being big into the data and analytics business I think we should be looking at ways to emphasize data, and how we present it. The progress bar is one of those areas that might be one of those opportunities worth exploring more than the average bar we see everywhere.

Agreed not to build. Will close.


onBeforeRender() + pinia

Description
onBeforeRender() does not work with pinia, actually I want to use onBerforeRender() for both server and client side in .page.ts file, I got the following error while pinia is initialized.
there is a way which solve the problem, using onServerPrefetch() and onMounted() hook  according to pinia example, but I want to use onBeforeRender()
Error Stack
Error: [🍍]: getActivePinia was called with no active Pinia. Did you forget to install pinia?

Please respect https://brillout.github.io/rules/.

I am confused with the difference between onBeforeRender and Vue's onServerPrefetch. Can someone help me with when I should use which? I am using Client Routing, enableEagerStreaming, and pinia.


Setting to suppress confirmation prompt when opening an Extension URI

Extension URIHandlers currently prompt users to confirm opening, "Allow an extension to open this URL?" This is inconvenient for some workflows, having some method for users to "Always Allow" would be an improvement.
Example use case:

In a code browsing web interface, we want to provide an "Open in VSCode" button
We support remote editing via a FileSystemProvider

Due to the FileSystemProvider, built-in vscode://file/... URLs don't seem to solve this problem. I have been able to use registerURIHandler for this, but the extra click on every file open hurts usability.
Ideas:

"Always Allow"

Option: Extension level setting. (E.g. accept all URLs like vscode://my-extension/...)
Option: Path level setting. (E.g. accept all URLs like vscode://my-extension/open/...)
Add a checkbox or button to the confirmation dialog to enable the chosen setting.

/file/ URIs do not currently prompt people. Design a hook for extensions to integrate with these.

Due to the FileSystemProvider, built-in vscode://file/... URLs don't seem to solve this problem.

It doesn't work because file system providers have separate schemas?

@joaomoreno: Correct, the FileSystemProvider will expect a different scheme from vscode://.
I also plan to have other Web UI integrations in addition to 'Open File' where the confirmation may become cumbersome for people doing that do those actions frequently.
If the scheme issue is solved, there are a couple other challenges with the /file/ approach as well:

My current FileSystemProvider expects a hostname in the URI, but the web UI containing "Open in VSCode" wouldn't know which hosts are locally connected to construct that URI. This is solvable on my end by adding logic to FileSystemProvider.
In the above scenario, if there is not a matching remote connected, I'd want to launch a different workflow in my extension to connect.

I don't know offhand if launching a workflow (via VSCode command) from readFile() would work. Since readFile() returns a generic Thenable, I suppose it might.
If that is possible, it feels like FileSystemProvider can be abused to accomplish anything that URI handler can but avoiding the confirmation prompt.

@joaomoreno: I have been able to implement with two approaches, or can combine them. Let me know if you have a preference which PR I send.

"Don't ask again" checkbox on the confirmation dialog that saves via StorageService

Discoverable for extension users.
Cannot be undone to "un-remember".

Configuration entry "extensions.confirmedUriHandlerExtensionIds"  backed by ConfigurationService.

Not as discoverable for extension users.
Can be managed by workspace owner / enterprise.

Combine both to support Config and UI Checkbox.

UI + config sounds good.

Is the dialog supposed to have an extra button or checkbox?

Otherwise, works fine with the setting

@roblourens: Interesting, that dialog looks different from what I see. Screenshot from OSS running on Mac. Insiders looks the same but with a different icon:

My hunch is that SimpleDialogService supports the checkbox rendering, but it looks like DialogService ignores it. I'm not sure offhand how VS Code chooses which DialogService to use so I will take a closer look here.

Oh yeah, I have set a setting to opt into that. I filed https://github.com/microsoft/vscode/issues/78343 for this


Wrong Readme Steps

the commands in the readme arent clear and doesn't work

Since VERSION is an env var, this repo could be used to run v6.8 clusters (or any other version). There is no disclaimer about it.
For v6.8

The randomly generated password for Kibana will belong to the user name "kibana" instead of "kibana_system". This should be warned at the README steps, and also it must be modified in the kib01 definition (docker-compose.yml) at the ELASTICSEARCH_USERNAME env var.

In order to generate the random passwords the elasticsearch-setup-passwords binary requires to trust the ES certs. Step 6 must be modified to handle first the CA addition to the trust set.
6. Run the *elasticsearch-generate-passwords* tool on es01 to generate passwords for all built-in users and kibana_system. Make note of these passwords.
    ```
    docker exec es01 /bin/bash -c "ln -s /usr/share/elasticsearch/config/certificates/ca/ca.crt /etc/pki/ca-trust/source/anchors/es-cluster-ca.crt"
    docker exec es01 /bin/bash -c "update-ca-trust"
    docker exec es01 /bin/bash -c "bin/elasticsearch-setup-passwords auto --batch --url https://es01:9200"
    ```

Since VERSION is an env var, this repo could be used to run v6.8 clusters (or any other version). There is no disclaimer about it.
For v6.8
* The randomly generated password for Kibana will belong to the user name "kibana" instead of "kibana_system". This should be warned at the README steps, and also it must be modified in the _kib01_ definition (`docker-compose.yml`) at the ELASTICSEARCH_USERNAME env var.

Added disclaimer about the version in this commit: https://github.com/datallboy/elastic-kibana-docker-nginx-letsencrypt/commit/7254ff2c6b4eb917fb6786896a793895681027bd

In order to generate the random passwords the elasticsearch-setup-passwords binary requires to trust the ES certs. Step 6 must be modified to handle first the CA addition to the trust set.
6. Run the *elasticsearch-generate-passwords* tool on es01 to generate passwords for all built-in users and kibana_system. Make note of these passwords.
    ```
    docker exec es01 /bin/bash -c "ln -s /usr/share/elasticsearch/config/certificates/ca/ca.crt /etc/pki/ca-trust/source/anchors/es-cluster-ca.crt"
    docker exec es01 /bin/bash -c "update-ca-trust"
    docker exec es01 /bin/bash -c "bin/elasticsearch-setup-passwords auto --batch --url https://es01:9200"
    ```

Added a comment onto  #7 about CA trust store. Base image for Elasticsearch changed to Ubuntu 20.04.


EventTriggerBehavior not working on custom implementations of system event handlers in Windows Runtime

I ran into a strange error using a custom DataGrid control for UWP.  The control implemented it's own SelectionChanged event, basing it on the SelectionChangedEventHandler  from System.Windows.Controls.

The text associated with this error code could not be found.
Cannot add instance of type 'Microsoft.Xaml.Interactions.Core.EventTriggerBehavior' to a collection of type 'Microsoft.Xaml.Interactivity.BehaviorCollection'. [Line: 148 Position: 102]

This happened because the EventTriggerBehavior attempted to handle the event with WindowsRuntimeMarshal.AddEventHandler.
Xaml looks something like this:
<mytkctl:DataGrid ItemsSource="{Binding Entries}" Name="WeightGrid">
    <Interactivity:Interaction.Behaviors>
        <Core:EventTriggerBehavior EventName="SelectionChanged" >
            <Core:InvokeCommandAction
            Command="{Binding EntrySelectionChanged}"
            CommandParameter="{Binding Path=SelectedItem, ElementName=WeightGrid}" />
        </Core:EventTriggerBehavior>
    </Interactivity:Interaction.Behaviors>
    <mytkctl:DataGrid.Columns>
        <mytkctl:DataGridTextColumn Header="Date" Binding="{Binding Date}" />
        <mytkctl:DataGridTextColumn Header="Weight" Binding="{Binding StringWeight}" />
    </mytkctl:DataGrid.Columns>
</mytkctl:DataGrid>

After figuring out the issue, I was able to resolve it fairly easily in the MyToolkit library, by downloading the source and adding a local copy of the SelectionChangedEventHandler delegate...
public delegate void SelectionChangedEventHandler(System.Object sender, SelectionChangedEventArgs e);

The question is how common is this issue?  Is this something that all custom control libraries should watch out for?  Or is this something that can be fixed in XamlBehaviors?

I submitted a change request for the MyToolkit library to use EventHandler instead of SelectionChangedEventHandler in their reimplementations.
QUESTION:  Is this best practice guidance for control developers somewhere?

Fair question! Personally, I'd say one can (and should) be able to reuse any public type as he sees fit.
I think a quick change on the way EventTriggerBehavior.IsWindowsRuntimeType(type) method works would fix this, possibly by passing the sourceObjectType and checking that this is also a Windows Runtime object!

A pull request has now been submitted to fix this issue and improve checks for Windows Runtime events! :)

Thanks!


Recent Sql Script does not open with CTRL + Enter while focus on Database Navigator

System information:

Linux Ubuntu 18.04
DBeaver Version <IP_ADDRESS>102141704
No extra extensions

Connection specification:

SQLServer 2016, Postgres 10 and MariaDB 10
Driver name: Microsoft JDBC Driver for SQL Server, PostgreSQL JDBC Driver, MySQL/MariaDB
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
While focus is on Database Navigator, short cut CTRL+ Enter to open Recent Sql Script. Using mouse and menu works well.
Steps to reproduce, if exist:
Focus on Database Navigator, press shortcut CTRL + Enter
Include any warning/errors/backtraces from the logs
Nothing else

I'll try give more details about how reproduce.

Open DBeaver
Keep view Database Navigator focused
Navigate to a connection
Keep focus on a connection
Press CTRL + Enter

It should open a Recent Sql Editor, as context menu indicates, but opens tree in that connection instead. The behaviour is as I press only Enter
If focus is on a database inside a tree of opened connection, that shortcut opens properties, not Recent Sql Editor
If I use context menu over any of item of that tree, connection or database, Recent Sql Editor goes right.

Hi @moesio ,
cannot reproduce it. Can you check keybindings in preferences?

there is no update on the issue for a long time.
If it is still actual for you - feel free to reopen it with an additional info


PathDet Dataset Curation - ROADWork Dataset

Description
Parse the ROADWork dataset which can be downloaded from this link to create a dataset comprising input images in PNG format and a single drivable path as the ground truth, based on the trajectory annotations provided in the dataset.
Please create a script called process_roadwork.py in this folder which is responsible for creating the ground truth and saving the data.
The ground truth drivable path information should be stored as a list keypoints defining the path in a JSON format. The drivable path keypoints should be sampled with a vertical (y image axis) displacement of 10 pixels to match the ground truth format used in other datasets.
Please ensure that keypoints are stored in relative coordinates, where the top left-most corner of the image is 0,0 and the bottom right-most corner of the image is 1,1, and all other coordinates are floating point values in the range (0,1) for x,y directions.
Please ensure ground truth images are stored in PNG format
Please also save a drivable path mask which is drawn upon the input RGB image for data auditing purposes as well as a binary drivable path mask in PNG format, where drivable path pixels are assigned a value of 255, and non-drivable path pixels are assigned a value of 0.
Data Summary:

RGB image in PNG Format
Drivable path keypoints in JSON Format
Binary Drivable Path Mask in PNG format
Drivable Path Mask draw on top of RGB image in PNG format (not used during training, only for data auditing purposes)

@docjag - I have created a new branch for this development called 'pathdet-dataset-curation-roadwork-dataset' - please fork this repo to your github, push the changes on this new branch on your github and then submit the pull request once completed, for review. Thanks!


Statically linking llvm

I created PR #2425 "Support linking llvm statically" so I could test AUR ponyc-rpm project.
Is there already an easy way to build ponyc with llvm statically linked?

With PR #2425 already merged, this issue should be closed.


Allow importing into projects without Worlds SDK

-Replaced VRC.SDK3.Components.VRCSceneDescriptor with VRC.SDKBase.VRC_SceneDescriptor (VRC_SceneDescriptor is the abstract class which VRCSceneDescriptor directly inherits from, and is included in the avatar SDK) ( Fixes #3 )
-Replaced all VRC.Udon.Serialization.OdinSerializer.Utilities.IsNullOfWhitespace() with string.IsNullOrWhiteSpace() in editor assembly ( Fixes #3 )
-Removed obsolete references from runtime assembly
This should be a valid solution for allowing import into projects without Worlds SDK

Looks good. Strange how i never noticed before, i feel like i used this in avatar projects but apparently not.


is there any book to introduce kestrel?

is there any book to introduce kestrel?

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

ID: f22dcddb-0c25-be1a-9a02-9e71dce86bdb
Version Independent ID: 3e0e12c2-e544-5655-e001-d2ec180b11a8
Content: Kestrel web server implementation in ASP.NET Core
Content Source: aspnetcore/fundamentals/servers/kestrel.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: tdykstra

@tomkluas There's some discussion about Kestrel in the following book from @andrewlock: https://www.manning.com/books/asp-net-core-in-action. With that being said, is there something lacking or confusing in our doc that we could improve?

@tomkluas Tom, a good place to start looking if you want to understand the request pipeline would be Kestrel's "middleware". You can find more about that here:
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/middleware/?view=aspnetcore-2.2

Thanks for contacting us.
We don’t have the resources to invest in this area, so we are closing the issue. Should your request generate enough  👍 responses, we’ll reconsider.


Filter and sorting not working?

Hi
I have this example:
 $sorting =  [
                "Ascending" => false,
                "Field" => "price"
                ];

              $filter = [
                'PurposeStatusIds' => [3],
              ];

            $result = $api->estates()->list($filter, $sorting);

But i always get the same results.
Filter and sorting does not seem to work.
Am i doing something wrong?

If you pass the filter and sorting like this it works            $result = $api->estates()->list(['Filter' => $filter, 'Sort' => $sorting]);

The Sort parameter accepts an array of objects. Try this:
$sorting =  [
    [
        "Ascending" => false,
        "Field" => "price"
    ]
];

$filter = [
    'PurposeStatusIds' => [3],
];

$result = $api->estates()->list($filter, $sorting);

I've made a tweak in 1.5.3 to support passing a single sorting parameter as an associative array to estates()->list(), so updating the package should fix your issue as well.


rene: Watch amazingthaitaste.com

rene requests the watch of the watch_keyword amazingthaitaste\.com. See the Metasmoke search here and the Stack Exchange search here.

amazingthaitaste\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 65.091% when pulling 13eb8dbdcab801b8f834374afa3a99544b1e314b on auto-blacklist-1524559942.2831464 into 0e6394a30b07c4e18f61017c12abd3cbcf567eec on master.

!!/approve


input: don't emit idle activity when calling simulateMouseMovement

Describe your PR, what does it fix/add?
Fixes https://github.com/hyprwm/hyprlock/issues/430
Now simulateMouseMovement does not lead to idle activity any more.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Idk if there are cases where simulateMouseMovement should post idle activity but I don't think so.
Is it ready for merging, or does it need work?
Ready.

tho in the future it'd be better to move the idle resetting stuff directly to stuff like IMouse and IKeyboard


Could not enable GPU for Bert based model

I am training the Bert based RATSQL model in Google colab pro. It takes roughly around 55 minutes for 10 steps. After started training, a warning pops up in the colab notebook. "WARNING: you are connected to a GPU runtime but not utilizing the GPU". After debugging the code, it shows that GPU is not enabled for Bert training. What can be done to enable CUDA? And please tell me what is the good environmental setup for training Bert based model?

After updating the CUDA package, this issue has been resolved. We can train the BERT in P100 but it may take upto a week

Hi @senthurRam33
Can you tel me please how to run RATSQL in Google colab? Precisely how to set up the environment based on docker ?
Thank you very much.

I didn't implement docker on colab as it is nearly impossible to run docker in that. I installed all the packages by mounting google drive on colab and started training. If the training stops, then I started retraining from the previously saved model using checkpoints

Thank you @senthurRam33 four your replay.
you said 'I installed all the packages by mounting google drive on colab'
How did you do that?
I have the problem of setting up the environment !!

Can you precisely tell me what's the problem??

Since it is impossible to use docker in colab, I want to set up manually the environment (installing packages and dependencies) in order to run RATSQL.
How to do that in colab?

--> First clone the repo in the google drive
--> Then install all the packages required to run the repo
--> Then upload the datasets in the drive
--> Then run the commands given in the repo to start the training
I would recommend you to use colab pro for faster training

I have the problem just in the second step.
How to install all the packages required to run the repo ?
By simply running 'pip install -r requirements.txt' ?
sorry for the perturbation

Yes.
But there are some packages that needs to be installed separetely. In my case, jsonnet and attr packages

it requires only:
!pip install -r requirements.txt
!pip install jsonnet
!pip install attr
for setting up the environment?

For me those are the ones that required.
Just try to run the repo using the command. It will tell you what are the packages that you are missing

Thank you very much @senthurRam33
Just one more remark, I think it requires also running Stanford corenlp server, is it true ? if yes how?

Yes it requires that package too. Try installing with !pip install corenlp command

And I think also it requires python>=3.7 instead of the default python in colab which is 3.6.9
Thank you very much for all your help.
I will try to run it in colab.


[bug] filter validation bug

Hi,
after I updated the library to v2 my working filters are giving me error.
please let me know what is wrong with my filter below:
(&(userPrincipalName=sostad*)(&(objectClass=user))(|(objectClass=person))(!(objectClass=computer)(objectClass=group)))

it is giving me "unbalanced parens" error

Maybe has an issue with you anding only one filter (&(objectClass=user)) and notting multiple (!(objectClass=computer)(objectClass=group))?

sorry (!(objectClass=computer)(objectClass=group))  part was the problem, thanks!

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Added new files

🛠 Issue (Number)

Closes #1355
Description
I Added my own Project to the repo!
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x ] I have made this from my own
[ x] I have taken help from some online resourses
[ ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ x] My changes generate no new warnings
[ ] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

When this project is getting merged? @harshit-paneri @TusharKesarwani

When this project is getting merged? @harshit-paneri @TusharKesarwani

It will merge by project admin @TusharKesarwani


FWI-2988 - Include admission request to the OPA engine

This PR fixes #
Checklist

[X] I have signed the CLA
[X] I have updated/added any relevant documentation

Description
What's the goal of this PR?

Includes admissionRequest parameter as a valid input for insightsInfo function.
The returned value should be accessible on the OPA runtime engine for inspection.

What changes did you make?
What alternative solution should we consider, if any?

⚠  Add documentation here


Reduce the complexity of the GraphQL converter

Currently the complexity of getQuery has exceeded a comfortable level for readability and testability.
Areas identified for optimization:
Remove / Rename extraneous keys prior to actually converting incoming json to graphQL.

It was determined earlier that the way this would be tackled was by storing a full graphql query and then using the merge logic to pare down to what was asked for.


ValueError: Encountered text corresponding to disallowed special token '<|endoftext|>'.

Issue
running a fork of openpilot through aider and received the following error
2024-06-28 13:32:29.131 Uncaught app exception
Traceback (most recent call last):
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 589, in _run_script
exec(code, module.dict)
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/gui.py", line 543, in
status = gui_main()
^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/gui.py", line 539, in gui_main
GUI()
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/gui.py", line 374, in init
self.process_chat()
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/gui.py", line 417, in process_chat
res = st.write_stream(self.coder.run_stream(prompt))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/streamlit/runtime/metrics_util.py", line 408, in wrapped_func
result = non_optional_func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/streamlit/elements/write.py", line 167, in write_stream
for chunk in stream:  # type: ignore
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/coders/base_coder.py", line 588, in run_stream
yield from self.send_new_user_message(user_message)
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/coders/base_coder.py", line 806, in send_new_user_message
messages = self.format_messages()
^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/coders/base_coder.py", line 762, in format_messages
messages += self.get_files_messages()
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/coders/base_coder.py", line 530, in get_files_messages
repo_content = self.get_repo_map()
^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/coders/base_coder.py", line 520, in get_repo_map
repo_content = self.repo_map.get_repo_map(
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/repomap.py", line 84, in get_repo_map
files_listing = self.get_ranked_tags_map(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/repomap.py", line 407, in get_ranked_tags_map
num_tokens = self.token_count(tree)
^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/models.py", line 419, in token_count
return len(self.tokenizer(msgs))
^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/aider/models.py", line 408, in tokenizer
return litellm.encode(model=self.name, text=text)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/litellm/utils.py", line 1496, in encode
enc = tokenizer_json["tokenizer"].encode(text)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/tiktoken/core.py", line 117, in encode
raise_disallowed_special_token(match.group())
File "/home/user/.pyenv/versions/3.12.4/lib/python3.12/site-packages/tiktoken/core.py", line 400, in raise_disallowed_special_token
raise ValueError(
ValueError: Encountered text corresponding to disallowed special token '<|endoftext|>'.
If you want this text to be encoded as a special token, pass it to allowed_special, e.g. allowed_special={'<|endoftext|>', ...}.
If you want this text to be encoded as normal text, disable the check for this token by passing disallowed_special=(enc.special_tokens_set - {'<|endoftext|>'}).
To disable this check for all special tokens, pass disallowed_special=().
Version and model info
--browser

Thanks for trying aider and filing this issue.
It sounds like you are using a local API server and a local model? It appears they may not be compatible with litellm, which aider users to connect to LLMs.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

I have error, the problem is due to one of the source files used in chat has the special token in it, e.g. here's a piece of my Python file:
        # Did first attempt with direct apply_chat_template without add_generation_prompt=True and adjusting result, trained for a few epochs, bot turned up crazy
        # 'prompt': '<|user|>\nQ:Question: how ... Yes or no.\nA:<|endoftext|>\n',
...

As you can see I am experimenting with tokenizer and actually have the token mentioned in the comments.
Seems like setting disallowed_special to a few ChatML values (e.g. <|endoftext|>,  <|assistant|> etc.) might help solving the issues.
The case is relevant for those ones tinkering with HF Transformers and finetuning models.


Display Qwant map on a website

Hi,
I would like to display the result of my search with Qwant Maps. I use Algolia Places (https://community.algolia.com/places/examples.html#displaying-on-a-map). Is it possible ?
Thanks :)

Like this issue ?
https://github.com/QwantResearch/qwantmaps/issues/22

Not really like this issue because here it will be in JS and in the other issue it will be in iframe I think.

We are working a solution for this feature.

Is there any news about this feature?


Update go.mod to not reference mmacheerpuppy!

I ran into this issue on my local branch after some testing, simply change the reference here and the gomod should resolve again! Saw you merged in my PR and wanted to reach out quickly!

Sorry @taylorchu ^_^


Add a recreate coroutine function to ClientSession().

Long story short

ClienSession does not have a coroutine function that allows to recreate a client session after it has been closed unless the class is reinitialized which can generate a Resource or a Memory leak if done enough times.
Expected behaviour

A method that would abstract the whole needing to recreate it that actually calls the __init__ method without reinitializing the class to invoke that method.
Actual behaviour

__init__ method is only invoked when calling for a new instance which when done enough times makes a Resource / Memory leak (Spikes the Memory to python.exe considerably if done too much).
Steps to reproduce

have a class that creates a client session similar to how discord.py does it.
close and recreate the client session just by overwriting the session variable in the class with a new instance every time.
Now after that is done now you have a problem with spiking memory usage that cannot be explained and can spike till there is no more memory left.

No. It's  very bad idea.
If you have memory leaks -- fix them without aiohttp spoiling.

cant be helped when it is from an library that runs it in a loop if the session is closed that my code uses.

hmm actually I got a better idea to make us both happy.
The change would be in the close method where it would delete it's own instance of the class when someone forget tos delete the instance itself after calling close()
not sure if del self would do that or not but maybe it would help and then it would only be 1 line of changed code too.


debian setup

on my debian 12 this extension is fully working even if I don'enable it in .Xresources perl extension and without putting the script in .urxvt/ext/
How is it possible ?
thanks

It's a Magic 😉


Fix fake clientsets in metrics.k8s.io.

The generated fake clientsets were using the API group "metrics" instead
of "metrics.k8s.io". The +groupName annotation needed to be copied to
these packages to fix it.
As a result, tests using this fake clientset work as expected.
Sorry, we do not accept changes directly against this repository. Please see
CONTRIBUTING.md for information on where and how to contribute instead.

/assign @DirectXMan12

oops, this should have been done through k8s.io/kubernetes/staging.


Add RedGIFs to PlayableEmbeds

I have tested this and it works. But unfortunately the video plays automatically no matter what I tried. I think it's an issue with redgifs itself.

Nvm lol


LL-1693 Fixes send max broken button for erc20

The sendmax button was not working for tokens because the bridge was not being retrieved correctly.
Type
BugFix
Context
https://ledgerhq.atlassian.net/projects/LL/issues/LL-1693
Parts of the app affected / Test plan
Send flow, amount step, sending from token account

XRP shouldn't have send max at all, I'll check.
Please look at this only from the erc20 perspective @Arnaud97234

pretty straightforward to merge


bug in logarithmic calculator

WHen we type base = 0 the msg displayed is pls enter a positive number. But better msg can be base cannot be 0 as considering 0 as not a positive no. is  not apt.


feat(reactions): Update reaction UI (No Emoji Picker)

Task: Use the updated mockups to move the message reactions to the context menu.
The emoji bar in the context menu should only display five of the ten available. Once the plus button is pressed, it should reveal the additional five emoji.
The design can be adjusted as needed. Just message me if changes are needed.
References:

Figma Link:
https://www.figma.com/file/GrAhLmQWA2kfD1Nw9nd24m/Chats-%2F-Landing-Page?type=design&node-id=1152-39949&mode=design&t=YkyV5CfywPdsHkpX-4

This was partially merged in but now we are looking to get the "+" and other emoji options.

We need to do some small work here, the + button should not show up if the extension is not present.


Events API Issues

Issues apply to F7v1(master) and F7v2
GetEvents: group Null and title Null
GetEventsByColleagues: Errors on master and migrated branch - JSON title, no offset(?)
EventsByOwner: not by owner, just the all events with description and title the way they should be? No offset(?)

API changes merged, https://github.com/gctools-outilsgc/gcconnex/pull/1878