[Alien RPG] v4.01 - Minor bugfixes

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Fixed counters in Roll template CSS that broke in previous version, they should now work again
Fixed roll named and roll commands to make Character and Hostile tab behave the same, for macro consistency
Previously, some rolls still use regular rolls and Roll template even when API Dice Roller is used, and those were not properly whispered to GM. This is now fixed, and selecting 'Make roll secret' now properly honors it for all rolls when API Script is configured for rolls.

Character Sheet Info Roll20 Internal Use only.


Add custom data (e.g. TLCC MRENCLAVE) to genesis block

The genesis block defines the root of trust of a blockchain. Therefore, it seems to be the ideal place to put security sensitive information such as the MRENCLAVE of TLCC. This information should then be accessible during transaction validation phase when enclave-created transactions are processed.
The purpose of this task is to identify ways how to extend or add custom data to a channel genesis block and consume them.
For instance, one possible solution is to create FPC specific configuration transactions. But what does it need to create those config transactions?
Also could MRENCLAVE be added by using the configtxgen tool?
There may exist multiple ways how to implement this. Is there a short term quick and dirty solution and a long-term elegant and clean solution?
Last, implement this.

obsolete now. new design handles that via encoding in version number or baked into ercc


Update README.md

I change the readme file i added "." to your files
3

every contribution worth a lot. Thank you.


How to train with my downloaded data ?

hello, I'm wondering if I can train with my downloaded dataset rather than to download the dataset at the folder again.
I'm hoping for your help.
BTW, can u tell me the number of parameters of your Unflow-CSS model ?

If I remember correctly, if the data is in place in the correct folder structure, it shouldn't be downloaded again. Sorry, don't know the parameters by heart.


Clean up of app.h

app.h, generated from app.h_in, has unnecessary code duplication
and isn't a header file (it defines globals, static functions
and doesn't have a header guard, moreover, it has a 'using namespace
std;'). Because of this, a real headerfile that declares the stuff
defined in apps.h was missing leading to even more code duplication:
scattered forward declarations in .cpp files and an often repeated
type of std::map<std::string, px4_main_t>.
This patch moves cmake/qurt/apps.h_in to src/platforms/apps.cpp.in
(with some changes) and removes cmake/posix/apps.h_in.
Then src/platforms/apps.cpp.in is split into src/platforms/apps.cpp.in
and src/platforms/apps.h.in, splitting declarations from definitions.
A typedef is defined for the map (apps_map_type).
The main difference between cmake/posix/apps.h_in and
cmake/qurt/apps.h_in was that the first defined a global 'apps',
while qurt stores the apps in QShell. I opted to get rid of
the global variable (which are in general evil for various reasons)
and used the API of cmake/qurt/apps.h_in where a provided 'apps'
map is initialized with a call to init_app_map. Thus removing
the existing code duplication.

Unfortunately, this is going to conflict big time with #5162.

@julianoes After having looked quickly at #5162, I believe you mean in terms of merge collisions due to proximate changes, not because they overlap technically.
Since my commit is the smallest, it seems most logical if I'd do the merging; and that wouldn't be a problem if we actually USED merging... but since we use git rebase, the only way to do that is when I'd wait until your PR is in master... I'm not sure if I want to wait that long, cause I need some of the changes in here for fix_headers and am stalled again until it is accepted.
Perhaps this is a solution: 1) This pull request is merged into master. 2) I actually merge that into your branch and do a pull request for that. 3) you won't do a rebase anymore, but your branch will just be merged this time. Aka, using git as it is supposed to be used :p.
If that is not an acceptable option, then we can do this: 1) again, merge this into master (as I need it quick), 2) I rebase your whole branch on top of this, fixing the collisions... 3) you wait until that is done and then hard reset to the rebased branch that I'll push to my github (I expect to need 1 hour).
Thoughts?

I would say you get this into master, and I understand that you don't want to wait foe my huge branch. I'll then rebase my branch on master again and resolve the stuff. I should get to it over the weekend.

I wanted to do a rebase myself, to see how hard it would be, but it turns out your pull request hasn't been rebased onto master for a long time; so when I try it I get many conflicts that have nothing to do with my commit :/. Could you first do a rebase onto the current master?

I just did a rebase to melt the three patches into one and force pushed this branch.

I reviewed this, looks good. Thanks @CarloWood.

@CarloWood I've already rebased #5162, it wasn't too bad :smile:.

Thanks!!


one-line log output

Hello,
devd looks really great, so far, but I was wondering if it was possible to add an option to enable one-line log output. My use-cases:
What if I want to grep devd command line output to look after:

the 404 or the 500 errors
which .css files are loaded and see their different HTTP status

by just adding a "--one-line" option, one could filter out unwanted lines.

What's wrong with grep? It does exactly that.
For spotting 404 errors:
devd | grep 404

For CSS files:
devd | grep .css

well, not really...
1 - Spotting 404
devd . 2>&1  | grep 404
<- 404 Not Found

You sureley know that there's a 404, but you don't know which resource is not found.
2 - Css (or whichever resource you want)
devd .  | grep css
14:08:05: GET /src/main/html/css/typography.css
14:08:05: GET /src/main/html/css/reset.css
14:08:05: GET /src/main/html/css/print.css
14:08:05: GET /src/main/html/css/screen.css

but there you don't have their status (200 or 304, or...)
The fact that the log is splitted in two distinct lines makes the grepping slightly more difficult than if it was just on one line.

Oh, I see what you mean. Agree with you, makes sense to have a --one-line option then, or even by default.

You can use context control:
Context control:
  -B, --before-context=NUM  print NUM lines of leading context
  -A, --after-context=NUM   print NUM lines of trailing context
  -C, --context=NUM         print NUM lines of output context

devd ... | grep -B 1 404

@marksteve I don't you're getting it. I think that a one-line log entry is a several-decade-old standard for servers and is the most convenient way to find information when it comes to debugging or compiling log data into another format (doing stats, or whatever)

@brunobord I've intentionally broken away from one-line log entries with devd. The primary use case here is interactive - i.e. a developer looking at live terminal output. Given this, I made the decision to move to a less traditional log format with syntax colouring, and more information spread legibly over multiple lines. Yes, there's a tradeoff versus using a standard log format, but I think it's worth it.
That doesn't mean I don't want to make it easer to pick information out of verbose logs. The -I option is a step in that direction - it lets you ignore logs selectively to filter out things like static assets you might not be interested in. I would contemplate expanding that sort of filtering in a measured and modest way - maybe a grep-like positive filter that turns off logging for everything except for matches.
I'm closing this ticket, because I don't think a single-line log option is the way to tackle this, but if you have ideas for how to proceed, please keep the discussion going.


Filter data of vector same as matrix column direction

See : https://github.com/Caleydo/taggle/issues/332

Looks good


Cross-platform testing is not truly cross-platform

See original issue on GitLab
In GitLab by [Gitlab user @tlater] on Jan 18, 2018, 09:49
Our integration tests currently use a super-secret environment variable to run tests as if they were run on different platforms - this works to test the general code paths, but is far from ideal.
Now that interest for different platforms is growing (see #190) we should add runners on machines that run the actual platforms we are testing.
After that is done, we should remove the super-secret environment variable.

In GitLab by [Gitlab user @tlater] on Nov 29, 2019, 17:57
The concept of "platforms" has since been removed from the codebase.
Instead, we test specific "sandbox" implementations, and while their testing is not as cross-platform as I'd hoped back then, the platforms that contributors are interested in do have dedicated test runners.

In GitLab by [Gitlab user @tlater] on Nov 29, 2019, 17:57
closed


Expected performance

Thanks for the repo! Do you have expected algorithm outputs for when we use or don't use the pretrained weights? Also how did you pretrain the model considering a large portion of the 2021 and 2023 dataset overlap (as far as I know)?

I didn't get your question? Could you please explain?


Issues on windows

I am having a bit of a problem getting the script to run on Windows. This is the error I am getting: "'python3' is not recognized as an internal or external command, operable program, or batch file.". I have set my environment variables; I also copy and paste the python.exe, then rename it python3.exe, just to see if it would work, and I keep getting the same error.
These are my environment variables: (A) C:\Program Files\Python312\Scripts
(B) C:\Program Files\Python312\
I would appreciate any suggestions.

Hi, sorry for the late reply
Did you set the python path when creating the object?
Like this:
$deepface = new \Astrotomic\DeepFace\DeepFace(
    python: '/usr/bin/python3',
);

Also, do you have the proper PYTHONPATH set on Windows?
Consider using my branch until the PR is merged, it has support for the latest version of deepface and it has some issues fixed that were added by mistake


Add raw-key-value migration type

Add unit test later

@caipengbo Great! Overall is quite clear and concise. Just a few questions inline.

Just a few questions inline.

I have modified the question you mentioned before. Are there any other questions? @git-hulk

@caipengbo Sorry for didn't send out the comments. 😭


Is there way to avoid 'Prefer java.util.Optional.orElseThrow' if 'isPresent()'  is right before

detected at: https://github.com/checkstyle/checkstyle/pull/16014#discussion_r1874927411
if (inlineTag.isPresent()) {
            final DetailNode inlineTagNode = inlineTag.get();
....

from modernizer: Prefer java.util.Optional.orElseThrow
I mam not sure why we should do it here as we did inlineTag.isPresent().
I agree that without isPresent() right before is it better to use orElseThrow as we do not know context, but if there is context .... can we somehow to avoid violation of such code ?

is there a way to disable specific validator ?

Modernizer has only simple bytecode-level checks with no flow control so it cannot understand if isPresent { get } sequences.  But I would encourage you to call orElseThrow instead of get for reasons that Brian Goetz explains here: https://stackoverflow.com/a/49159955/2800111.  It is possible that a future Java version will deprecate get which seems like something Modernizer should warn about today.
exclusionPattern is a good way to address this if you disagree with the suggestion and Modernizer supports custom violation files if you only want to use a small subset or custom checks.
I do appreciate any suggestions you have from the Checkstyle project as this project may benefit from better configuration.

Following if statements from the bytecode may be near the same level of complexity as it is for us to track variable usage by the name. Even we can't do that correctly all the time for all our checks and there has been issues opened to build such a utility in CS.
It doesn't help that each compiler will write the byte code in different ways for the same Java code. I've seen code block off logic in the if statements to the end of method's code, or inline it with the rest of the other logic by jumping around it. That doesn't even account for determining if it is a simple if or really a loop.
pitest also works on the byte code only, and it used to write weird things in the report (from the source perspective) which made more sense looking at the bytecode.

I understand general benefits of new methods, It is first time we run into this, and when I did suggested code it looked weird, we over care to readability of code in checkstyle project, and we try to find equilibrium between all best practices, and  readability for regular humans(not very professional engineers).
For now I will probably disable this valuation, and we might come back to it later on, maybe new jdk will not give us option anymore or something else come up as good pattern to write such code

From the executing code perspective, there is no difference between get and orElseThrow. It is really just the name difference between the 2 in my Java 11 JDK.


Rationalize AvailabilityProfile default

What this PR does / why we need it: We were setting AvailabilityProfile to VMSS by default. However, VMSS is not supported for K8s < 1.10. This PR ensures that we set AvailabilityProfile to AvailabilitySet if using K8s < 1.10.
Backwards compatibility should not be affected because we were validating before to make sure that VMSS was not set with k8s < 1.10. This change just makes it easier for the user by setting a default that works with validation instead of requiring an override.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[x] documentation
[x] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

@sozercan could you please take a look at this PR?

/approve
/lgtm


Will NOT Exit

I have a Sony A6300 and I understand the "Menu" button should exit the app but it DOES NOT.  I can't get out of the BetterManual application.  Need some help!
Thanks!

Exit the app using the help/trash button.

NICE WORK!  Thanks so much.  F’n awesome ☺
From: obs1dium<EMAIL_ADDRESS>Sent: Monday, October 2, 2017 12:56 PM
To: obs1dium/BetterManual<EMAIL_ADDRESS>Cc: Mike Shonk<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [obs1dium/BetterManual] Will NOT Exit (#9)
Exit the app using the help/trash button.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/obs1dium/BetterManual/issues/9#issuecomment-333595654, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ae9XGGMREfOsT-8KiAemYbRZP1W55Vmmks5soRW0gaJpZM4Pq7rf.

Exit the app using the help/trash button.

this has no effect on a6500, not even powering off the camera! not even removing the battery. please help!

update: exited via trash but only after removing the battery


fix for SwiftSyntaxParser changes

This PR updates the imports to handle the separation of SwiftSyntax and SwiftSyntaxParser:
https://github.com/apple/swift-syntax/pull/309

@allevato not sure who is the maintainer, could you add a reviewer if you know? thanks

Why is import SwiftSyntaxParser necessary to add in so many files? /cc @ahoppen

The Diagnostic types were moved to SwiftSyntaxParser, and these are used extensively.

The Diagnostic types were moved to SwiftSyntaxParser, and these are used extensively.

Interesting, @ahoppen should we keep Diagnostic in SwiftSyntax module? It has more general uses than just for parsing.

This should be merged ASAP, because the main branch cannot even be build currently with below compilation error messages.

Cannot find type 'DiagnosticEngine' in scope
Cannot find type 'Diagnostic' in scope

(Tested on 2021.10.03 07:55 GMT-4, M1 Macbook Pro)

Interesting, @ahoppen should we keep Diagnostic in SwiftSyntax module? It has more general uses than just for parsing.

I think we should not move Diagnostic back to the SwiftSyntax module. SwiftSyntax should just represent the syntax tree (+ operations on the tree). Since it doesn’t support parsing, there’s no need for diagnostics handling in the SwiftSyntax module. I would be open to discussions moving Diagnostics to a new module that’s neither SwiftSyntax nor SwiftSyntaxParser though, but I’m not sure if that’s gaining us anything.

Since it doesn’t support parsing, there’s no need for diagnostics handling in the SwiftSyntax module

But swift-format is using it as a general diagnostic mechanism for its own diagnostics; within this context it's not just about parser diagnostics.
For example one could create a tool that forms a tree via the APIs, based on some non-swift inputs, and still want to emit diagnostics.
Or maybe you suggest that Diagnostic should only be used for the parser diagnostics and not for anything else?

SwiftSyntax should just represent the syntax tree (+ operations on the tree). Since it doesn’t support parsing, there’s no need for diagnostics handling in the SwiftSyntax module.

At first I felt that SwiftSyntax was the more appropriate place for the diagnostics functionality, but I agree with this as well; it doesn't necessarily feel right to put it there just because other code that uses the functionality in that module might also want to use diagnostics.

I would be open to discussions moving Diagnostics to a new module that’s neither SwiftSyntax nor SwiftSyntaxParser though, but I’m not sure if that’s gaining us anything.

I think there would be value to this but it doesn't specifically help SwiftSyntax in isolation, and that SwiftSyntax probably isn't the right place for it. For example:

Or maybe you suggest that Diagnostic should only be used for the parser diagnostics and not for anything else?

If I adopted this approach, I'd be creating a third (at least?) diagnostics engine among Swift projects—that new one, plus the one in SwiftSyntax, plus the one in tools-support-core that's used by swift-driver and SPM. That makes me think this is useful functionality that should be shared by all three, somewhere. But it's also important to keep SwiftSyntax and SwiftSyntaxParser free of heavy dependencies.
So... here's a bolder idea: maybe the whole concept of a diagnostic engine/consumers shouldn't live in SwiftSyntax/SwiftSyntaxParser at all? If the parser is the only user of it, it could just take a callback function instead of a diagnostic engine reference, and pass its own lightweight Diagnostic value containing information relevant to a parse issue. Then, clients of SwiftSyntaxParser would translate those into whatever diagnostic engine they wanted to use (for example, swift-format could start using TSC instead).

So... here's a bolder idea: maybe the whole concept of a diagnostic engine/consumers shouldn't live in SwiftSyntax/SwiftSyntaxParser at all?

Seems like a good idea that has better separation of concerns 👍

Filed https://bugs.swift.org/browse/SR-15280 to keep track of moving DiagnosticEngine to another module.
I would suggest to merge this PR to unblock swift-format until that change has been made.

Coming back around to this, I'm going to close this in favor of https://github.com/apple/swift-format/pull/270. Instead of staying coupled to the SwiftSyntaxParser.Diagnostic type (or any other diagnostic engine's type), that PR creates a swift-format-specific type to communicate those back through the API layer, and then limits the use of a specific diagnostic engine (in this case, the one from swift-tools-support-core) to the frontend executable.
This sets us up to make some nice improvements later, like:

Splitting out the lint/format methods that parse source code in the SwiftFormat API module into their own module, which would make the core API completely decoupled from SwiftSyntaxParser. This would be useful for code-generation use cases, where someone can build up a tree in-memory and then format it, without needing to link to the parser library.
Applying different severity levels to diagnostics based on their category/rule, which was discussed a bit in https://github.com/apple/swift-format/pull/252.


Supporting iterable collections

My understanding is that Solid.js uses this library to convert JSX to JavaScript that manipulates the DOM.
In experimenting with the Squint library, I've noticed that if a JSX value contains a custom collection of JSX nodes that are not in an array, these nodes are not inserted in the DOM. Instead, a warning message is printed to the console stating Unrecognized value. Skipped inserting. I assume this message is generated by the following code:
https://github.com/ryansolid/dom-expressions/blob/82c377d19c7e096fe92e04411e1d332536cddec5/packages/dom-expressions/src/client.js#L460-L470
If the collection is iterable (i.e. it has Symbol.iterator set), is there a reason not to iterate over it and extract its elements that way? A number of Squint's functions return a LazyIterable object rather than an array, and it would be helpful if those were supported.

Note that React supports anything that has [Symbol.iterator]().

React has only recently started supporting iterables, but yes, there is some things to be discussed whether or not this is worth adding.

I've been using it in React for over a year, FWIW

I wonder would we need to expand support for things like For. Because it isn't only the JSX insertion but anything that maps it I imagine.


ModuleNotFoundError: No module named 'triton'

@danielhanchen  Ran into ModuleNotFoundError: No module named 'triton' while fine-tuning google/gemma-7b-it.  I installed xformers successfully through a documentation that I found by Unsloth but while running From unsloth import FastLanguageModel I was thrown this error. I've been trying to solve this error quite sometime and can't seem to find any reddit or git solutions to this triton issue. Also I am working on windows 11.

Oh if it's Windows, it means Triton needs to be installed - I forgot which guides provide a Windows installation process for Triton though - maybe best to ask on Discord

Hi @pratmegs, this is the issue with the guide on how to setup unsloth on windows. https://github.com/unslothai/unsloth/issues/210


A11y_Azure Tools Storage Explorer_FileShare_CreateNewDirectory_StatusMessage: After we create a new directory/Folder in fileshare, no status message is announced by the screenreader

GitHub Tags:
#A11yMAS;#A11yTCS;#Mac;#DesktopApp;#A11ySev3;#BM_AzureToolsStorageExplorer_Mac_Dec2022;#Azure Tools Storage Explorer;#WCAG4.1.3;#Voiceover;#FTP;#DesktopApp;#AINotInScope;
Actual results:
Screen reader is not announcing the status message when we create any folder or directory.
Expected results:
Screen reader should announce the "New folder successfully created"
##Application details
Microsoft Azure Storage Explore
version: 1.27.0
OS Details
Mac details: macOS Ventura 13.1
Screenreader: Voiceover
Repro Steps:

Launch Storage Explorer
Tab to "Open Connect dialog" pane and Sign in to Azure.
Go to "Manage Accounts" pane and Select the subscriptions you will use. Once selected, click on "Apply" button.
Tab to "File Shares" of the selected subscription and expand it.
Tab to tester and click on it.
Turn on voiceover.
Tab to New Folder button and click on it.
Observe if any status message is announced by screenreader or not.

User Impact:
Screen reader user will not know about if new folder is created.
Attachments:
6373.zip

i have verified in the Environment, Issue is no repro. Hence, closing the bug.

GitHubTags:#Closed;


Leverage lightgaussian for 15 times smaller PLY files

lightgaussian has a very small effect on quality (PSNR) and reduce size by 15 times, which btw improve FPS
https://github.com/VITA-Group/LightGaussian
it  is made of 3 components

gaussian pruning
SH distilation
vecquantization informed on gaussian importance weights from 1) + octree losless compression on position data and FP16

gaussian pruning is the only component with code released for now
both 1 and 2 are unique in that they selectively remove gaussians and SH with almost no loss of accuracy
most compression alternatives online act on step 3, as mostly non selective compression on attributes
the process though can be further optimized on the 40% unquantized gaussians if we follow aras-p blog https://aras-p.info/blog/2023/09/13/Making-Gaussian-Splats-smaller/
via BC7 partitioning and k-means on SH for truly SOTA compression but this is a digression as no one has combined both solutions ATM

I don't see how SuperSplat can call/execute LightGaussian to compress PLY files. It's runs in a Python environment. Or are you suggesting that we write an importer for the LightGaussian quantized format?

I've tried with https://github.com/yangchris11/samurai and it really worked!

You can check the result here: https://3d.explorerglobal.cn/model/6l7vdmlj


Tags within in some objects (e.g. data blocks, headers ) are not shown in queries for tag

Hi,
I just encountered some behavior, which I think could be a bug - or I misunderstand the feature :-)
According to the documentation, I should be able to add tags to objects (data block) like this:
´´´#person
name: Pete
age: 55
tags: one, two
´´´

I was hoping to get all tags listed, if I query for tag, like it works for tags in items for example.
But I get no result:
´´´query
tag where name= "one"
´´´

On the other hand, I do find the "person" that has the tags assigned:
´´´query
person where tags =~ /one/
´´´

However, that has strange behaviour, too. Look at the itags column. There are single letters listed as itags? Makes no sense to me...

ref
tag
name
age
tags
pos
page
itags

test@933
person
Pete
55
one, two
933
test
person, o, n, e, ,,  , t, w

Playing around a bit, I realized, that the first issue applies to other objects, too:
Example paragraph #two
I can query fine for paragraph where tag =~ /two/ , but querying for tag where name =~ /two/ gives no result.
Not sure, if that is by accident or by design :-)

Hello, we tried to solve the issue.
This is what we did:
This change addresses the issue with tag indexing and querying for objects and paragraphs. It modifies the indexTags function to include tags from data blocks and updates the tagComplete function to query all tags, including those from data blocks and paragraphs.
You can review changes in this commit: https://github.com/mnmnk43434/silverbulletmd-silverbullet-1178/commit/25fad667a888e885ad93a28c2bdcabcf353363ac.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Ensure that empty elements don't break the parser

Empty elements like  need special handling,
because the previous parsing code assumed that each
start element had a corresponding end element.

Thanks Martin, looks great and I appreciate the PR.


runtime: remove unnecessary allocations in convT2E

This was at lengths discussed in:
https://golang.org/issue/8405
and previously on golang-dev:
https://groups.google.com/d/msg/golang-dev/pwUh0BVFpY0/zqJInvU3NkQJ

Namely, we should allocate heap block for scalars iff the scalar look like a pointer
into heap (otherwise GC will ignore it anyway).

This will allow to have 1-bit/word GC pointer type info *and* don't allocate additional
memory for scalars in interfaces in most cases.

@dvyukov, does this issue look relevant to you?

@ysmolsky I don't understand the question. What do you mean?

@dvyukov I think he is asking whether you think this issue should still be open.
Really this is a question for @aclements and @RLH.  Is this a feasible optimization with the current GC?

It should still be a feasible optimization because we still support Go pointers to point to C heap, right?
Another potential implementation would work for value types that are smaller than pointer size (e.g. uint16, uint32 on 64-bits, bool, small structs, etc). Namely: let's say we have amd64, pointer size is 64 bits, let's say we are storing uint32 into an interface{}, when we are storing the value we put 0xffffffff into the high 32-bits. This makes this value invalid/outside-of-heap if treated as pointer. Extraction won't require any special code, we just take the low 32 bits.
This would require to calculate if this optimization is applicable for a type in the compiler (based on type size, and arch), and adding some high bits in runtime/compiler.
Here is a quick proof-of-concept:
--- a/src/cmd/compile/internal/gc/subr.go
+++ b/src/cmd/compile/internal/gc/subr.go
@@ -1857,6 +1857,10 @@ func isdirectiface(t *types.Type) bool {
                return t.NumFields() == 1 && isdirectiface(t.Field(0).Type)
        }

+       if t.Size() == 2 && t.Align == 2 {
+               return true
+       }
+
        return false
 }

--- a/src/runtime/iface.go
+++ b/src/runtime/iface.go
 func convT16(val uint16) (x unsafe.Pointer) {
-       if val < uint16(len(staticuint64s)) {
-               x = unsafe.Pointer(&staticuint64s[val])
-               if sys.BigEndian {
-                       x = add(x, 6)
-               }
-       } else {
-               x = mallocgc(2, uint16Type, false)
-               *(*uint16)(x) = val
-       }
-       return
+       return unsafe.Pointer(uintptr(val) | 1<<63)
 }

This passed bootstrap and almost passed go tool dist test. I guess it also needs some changes in the reflect package.
And at this point we obviously want to inline this conversion wholesale because there is no point in the function call now.
Besides avoiding mallocgc call and not generating garbage, this also makes accesses faster (no indirection).
Thoughts?
@dr2chase  @mknyszek

This is inspired by a real use case. I need to store lots of integers in interfaces, but these won't fit into staticuint64s (up to thousands/tens of thousands). But they perfectly fit into uint32 and that's what I use as the type.

I’m on my phone, but you probably also need to adjust ifaceData in the compiler. And ifaceeq and efaceeq in the runtime. (Also, it’s be more impressive to get uint8/uint32 working—uint16s are not heavily used.)
Another option, which is less likely to cause rare breakage (e.g. in assembly) is to keep an atomic counter in the slow path of convT32 for smallish values and, when used enough, persistent alloc and populate a staticuint64s-like array with a larger range.
Or keep a per-P cache of recently allocated values so that we can re-use them. (I tried this for string interning, and it was pretty straightforward.)

There's an endianness issue in that code, that's a minor problem, also the
description of the pointer smash (0xffffffff) in upper 32 bits doesn't
match what the code does (uintptr(val) | 1<<63).
But otherwise, interesting, though I am worried about assembly language.
On Sat, Apr 25, 2020 at 12:21 PM Josh Bleecher Snyder <
<EMAIL_ADDRESS>wrote:

I’m on my phone, but you probably also need to adjust ifaceData in the
compiler. And ifaceeq and efaceeq in the runtime. (Also, it’s be more
impressive to get uint8/uint32 working—uint16s are not heavily used.)
Another option, which is less likely to cause rare breakage (e.g. in
assembly) is to keep an atomic counter in the slow path of convT32 for
smallish values and, when used enough, persistent alloc and populate a
staticuint64s-like array with a larger range.
Or keep a per-P cache of recently allocated values so that we can re-use
them. (I tried this for string interning, and it was pretty
straightforward.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/8892#issuecomment-619404170, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOW6JZM77AJJBGOQP7NRXLROMEXTANCNFSM4GCAPQ3Q
.


[Issue]: EL9.4 ROCM 6.3 repo installation results in amd-smi-lib RPM and python errors

Problem Description
Following the published Quick start installation guide instructs results in rpm and python errors.
Operating System
RHEL9.4
CPU
n/a
GPU
n/a
ROCm Version
ROCm 6.2.3
ROCm Component
amdsmi
Steps to Reproduce

Follow the instruction for RHEL 9.4 here: https://rocm.docs.amd.com/projects/install-on-linux/en/latest/install/quick-start.html
To take a shortcut, do 'dnf -y install amd-smi-lib' ... bug will reproduce either way but easier to just install the offending package and its two dependencies.

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
PLEASE NOTE THAT THIS BUG IS AGAINST ROCM 6.3 -- the ROCm Version drop down in the issue reporting tool does not have a 6.3 available.
[root@vm-10-0-186-164 ~]# dnf -y install amd-smi-lib
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered with an entitlement server. You can use "rhc" or "subscription-manager" to register.
Last metadata expiration check: 0:08:08 ago on Fri 06 Dec 2024 01:07:51 PM EST.
Dependencies resolved.
Package              Architecture    Version                         Repository               Size
Installing:
amd-smi-lib          x86_64          <IP_ADDRESS>300-39.el9             amdgpu-6.3              1.3 M
Installing dependencies:
python3-pip          noarch          21.2.3-8.el9                    rhel-AppStream          2.0 M
rocm-core            x86_64          <IP_ADDRESS>300-39.el9              amdgpu-6.3               25 k
Transaction Summary
Install  3 Packages
Total download size: 3.3 M
Installed size: 13 M
Downloading Packages:
(1/3): python3-pip-21.2.3-8.el9.noarch.rpm                          5.3 MB/s | 2.0 MB     00:00
(2/3): rocm-core-<IP_ADDRESS>300-39.el9.x86_64.rpm                       41 kB/s |  25 kB     00:00
(3/3): amd-smi-lib-<IP_ADDRESS>300-39.el9.x86_64.rpm                   1.8 MB/s | 1.3 MB     00:00
Total                                                               4.7 MB/s | 3.3 MB     00:00
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.
Running transaction
Preparing        :                                                                            1/1
Installing       : rocm-core-<IP_ADDRESS>300-39.el9.x86_64                                        1/3
Running scriptlet: rocm-core-<IP_ADDRESS>300-39.el9.x86_64                                        1/3
Installing       : python3-pip-21.2.3-8.el9.noarch                                            2/3
Installing       : amd-smi-lib-<IP_ADDRESS>300-39.el9.x86_64                                     3/3
Running scriptlet: amd-smi-lib-<IP_ADDRESS>300-39.el9.x86_64                                     3/3
Using pyproject.toml for installation due to setuptools version 53.0.0
DEPRECATION: A future pip version will change local packages to be built in-place without first copying to a temporary directory. We recommend you use --use-feature=in-tree-build to test your packages with this new behavior before it becomes the default.
pip 21.3 will remove support for this functionality. You can find discussion regarding this at https://github.com/pypa/pip/issues/7555.
ERROR: Command errored out with exit status 1:
command: /bin/python3 /usr/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py prepare_metadata_for_build_wheel /tmp/tmpfq576nu4
cwd: /tmp/pip-req-build-x_y8503g
Complete output (11 lines):
running dist_info
creating /tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info
writing /tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/PKG-INFO
writing dependency_links to /tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/dependency_links.txt
writing requirements to /tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/requires.txt
writing top-level names to /tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/top_level.txt
writing manifest file '/tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/SOURCES.txt'
reading manifest file '/tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/SOURCES.txt'
writing manifest file '/tmp/pip-modern-metadata-fcs9_3es/amdsmi.egg-info/SOURCES.txt'
creating '/tmp/pip-modern-metadata-fcs9_3es/amdsmi.dist-info'
error: invalid command 'bdist_wheel'
----------------------------------------
WARNING: Discarding file:///opt/rocm-6.3.0/share/amd_smi. Command errored out with exit status 1: /bin/python3 /usr/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py prepare_metadata_for_build_wheel /tmp/tmpfq576nu4 Check the logs for full command output.
ERROR: Command errored out with exit status 1: /bin/python3 /usr/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py prepare_metadata_for_build_wheel /tmp/tmpfq576nu4 Check the logs for full command output.
[WARNING] Could not find argcomplete or argcomplete3.  Argument completion will not work...
Removed "/etc/systemd/system/timers.target.wants/logrotate.timer".
Created symlink /etc/systemd/system/timers.target.wants/logrotate.timer → /usr/lib/systemd/system/logrotate.timer.
Verifying        : python3-pip-21.2.3-8.el9.noarch                                            1/3
Verifying        : amd-smi-lib-<IP_ADDRESS>300-39.el9.x86_64                                     2/3
Verifying        : rocm-core-<IP_ADDRESS>300-39.el9.x86_64                                        3/3
Installed products updated.
Installed:
amd-smi-lib-<IP_ADDRESS>300-39.el9.x86_64               python3-pip-21.2.3-8.el9.noarch
rocm-core-<IP_ADDRESS>300-39.el9.x86_64
Complete!
[root@vm-10-0-186-164 ~]#

Hi @prarit, thanks for the report. Was able to reproduce this issue am looking into this further.

@prarit, the error during amd-smi installation is caused by missing python3-setuptools and python3-wheel packages. As a temporary workaround, you can install these packages prior to ROCm/amd-smi installation as highlighted here. We will be adding these packages as dependencies for amd-smi in an upcoming patch and also temporarily updating our ROCm installation documentation to notify users of this requirement.


Training Custom model - Issue saving model

Hello,
I have been trying to train the model with custom dataset using train_ssd.py.
Training goes well but after each epoch the model gets saved with "inf" instead of the loss value in the file name.
And therefore while converting to .onnx file , the program fails to find the model with the minimum loss.
Why is it not able to save the model with the loss value itself (as the file name) ?

Hi @Lyns01, you can manually specify the pytorch checkpoint with the --input flag to onnx_export.py
But even if you did, the model is unlikely to work well since it has INF loss.   This typically occurs because of some malformed/invalid bounding box(s) in your dataset.  Maybe some bounding box has very large or negative coordinates somehow.
What I recommend doing is to uncomment this line of code:
https://github.com/dusty-nv/pytorch-ssd/blob/e7b5af50a157c50d3bab8f55089ce57c2c812f37/vision/datasets/voc_dataset.py#L76
And then run train_ssd.py with --batch_size=1 --num_workers=1 --debug_steps=1
And then when you see the INF/NaN loss first appear, look at the annotations that were loaded just above.  Check the XML file to make sure they are ok.
You can also post the console log here when you run it as described above.

Hello,
I did uncomment the print() statement and trained once again using train_ssd.py with batch size, workers and debug steps as mentioned above.
I can see every image being trained and therefore could see that one image gives "inf" in the place of loss. But couldn't find which image is causing the trouble as it doesn't print the image name.
I have attached the console image

Hi,
I did everything as described above.
Was able to notice that one image gives "inf" in the place of loss value. But couldn't find which image becoz it doesn't print the image name even after uncommenting the print statment as u said.
Have shared a screenshot of the console window

But couldn't find which image becoz it doesn't print the image name even after uncommenting the print statment as u said.

Are you running this in the container by chance?
If so, you need to edit the version of the file inside the container.  You can do this by starting the container, and then editing the file inside the container (with a text editor like nano).  If text editor isn't installed, you can install it into container by running apt-get update && apt-get install nano

Hi yes I was running in docker. And yes there was a problem with the image annotations in the dataset. I tried again with a proper data set and everything works fine. Thank you for helping me sort it out.


Move timeseries fixtures to their own directory

A few of the tests were a bit odd. They had one more data point than label. That data point wasn't plotted, but stretched out the range of the y-axis. I went ahead and updated these tests because I think they included more data than labels on accident. If there's some behavior around what happens in an edge case we want to test we should test that specifically with a test whose name and comments explain what is being tested

At least some of those test tick source (in the name) an it makes sense to have different number.


p. 278 typo

Just after equation 8.16 change "The gradient at the current iterate," to "The gradient at the current interation,".

fixed


[Bug] 启动失败  java.lang.NoSuchFieldError: IS_SECURITY_ENABLED

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Ubuntu 20.04,  按https://datavane.github.io/datavines-website/docs/get-started/deploy-from-source 步骤，在启动时报错
java.lang.NoSuchFieldError: IS_SECURITY_ENABLED
删除了libs包中的jsp-2.1-6.1.14.jar 后，启动正常。
DataVines Version
最新版一直有
DataVines Config
与配置无关

Running Command
正常启动

Error Exception
java.lang.NoSuchFieldError: IS_SECURITY_ENABLED

Engine Type
No response
Java Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

The problem has been fixed.


core(fix): error on modules include / exclude patterns conflict

Current implementation of module inclusion / exclusion can easily lead to silent conflicts between glob patterns, during a dagger install: https://github.com/dagger/dagger/pull/8042#issuecomment-2252289788.

For example, one submodule can exclude: **/sdk, whilst we try to install a module located at depth1/sdk/*

For BuildKit, this would lead to a priority for the exclusion, leading to a weird failure on dagger install
This commit ensures that Dagger fails with a readable explanation in case such occurence of conflict.
TODO:

[ ] add an integration test

I'm not sure we should prevent an exclude from overriding an include or vice versa. It can be used by the user to override a default pattern.

This is a potential stopgap for https://github.com/dagger/dagger/issues/6902

I'm not sure we should prevent an exclude from overriding an include or vice versa. It can be used by the user to override a default pattern.

I am not sure either, thanks for that example🙏
Maybe shall we just provide intelligible warning to the user, showing that some of them might conflict (with the extension of the provenance: which module have which inclusion / exclusion ?) cc @sipsma @jedevc WDYT ?

This is not just about:

https://github.com/dagger/dagger/issues/6902

The problem you saw in:

https://github.com/dagger/dagger/pull/8042#issuecomment-2252289788

Actually comes from a bug:

https://github.com/dagger/dagger/issues/8136

So that **/sdk pattern in sdk/python/dev/dagger.json should be excluding sdk/python/dev/**/sdk and not **/sdk. Without that bug, there would be no conflict here.


support for client certificates

Have you ever considered implementing support for client certificates for authenticating on capsules like Astrobotany or Martin Rue's Station?
I did a quick test and i't should possible without too much hassle.
My proposal:

allow a single cert by (sub)domain, automatically selected by astro
store certs in .config/astro/certs in PEM format (crt and key file separate) named by (sub)domain, e.g. example.com.crt and example.com.key
if a request requires a cert

check if the cert for this (sub)domain is already there
if so repeat the request with the cert: openssl s_client ... -cert <crt> -key <key>
otherwise print instructions on how to create a cert with openssl, e.g.: openssl req -x509 -newkey rsa:4096 -keyout ~/.config/astro/certs/example.com.key -out ~/.config/astro/certs/example.com.crt -days 36500 -nodes

We could even create the cert ourselves if need be.
This is essentially how client certs are currently handled in gmni/cgmnlm and it works quite well.

Hello again!
I thought about client certificates but never found a capsule that requested certs, so I did not implement it.
I like your proposal, you can go ahead and submit a PR if you want, surely this will be of great value for the project.
And thanks again.


Sanitize column names when reading data

In this file the RI is not found for some bizarre reason - this needs to be investigated.
fake_ri_ms_w4e.csv

Ideally using a package like this: https://towardsdatascience.com/how-to-clean-messy-pandas-column-names-20dc7400cea7 or

Another package that would work well is dataprep. This should be used in the pandas data class.


new min version - doesn't scroll - non touch

Hi Robert,
This time through github :)
Roto isn't scrolling anymore on my macbook, on my iPhone it still works flawless.
I am getting an error in Safari:
TypeError: 'undefined' is not an object (evaluating 'fn.guid')
Safari on mac says the problem originates from line 3487 in jquery-1.6.4.js
Update:
On Google Chrome the issue is also present
Firefox though works flawless (and you fixed the second click issue, great!)
Hope you can fix it, so it will be usable on Safari and Chrome too.
(this is all on Mac, with the latest Safari and Chrome, Firefox is also the latest version)
Thanks again! :D
Robert

Mysterious. This only happened in Safari, and seems to be fixed now. But I only added and then removed a trace statement ...
Can you confirm it's working for you now?

Yes! You've fixed it!
I've checked the new roto min version in Safari, Chrome, Firefox on Mac and on mobile Safari on my iPhone and it is 100% working.
Thanks!
Another "problem" solved!
Have a nice evening.

Oh well, I'll mark that one down to gremlins ...

In dutch we say: de kaboutertjes hebben het gedaan :-) Kabouters are goblins, so it translates something like: The goblins did it :D
I really want to thank you for fixing the "double click feature" !


[MM-36481] Fix thumbsup emoji parsing to support skin tones

Summary
Thumbs up wasn't being skin toned properly, updated the regex to support that.
Ticket Link
MM-36481
Related PRs

Server https://github.com/mattermost/mattermost-server/pull/17846

Screenshots

Release Note

NONE

approved but we need to fix the tests.

approved but we need to fix the tests.

yeah, I was on it, but my laptop is slow running the tests, so it took more running them than actually fixing 😓
anyway that's done now

@Willyfrog Leaving this to you to merge since I'm not 100% sure if it requires the server changes as well.

no they don't I related them as the todo in the other PR especified changing the strings that document available formats

/cherry-pick cloud

Cherry pick is scheduled.


Druid 0.12.3 - Middle Managers - NullPointerException Jackson

Hi, all
We are running Druid 0.12.3 on production. Every hour we run approximately 2000 tasks on middle managers.
Every few hours one or two real time index tasks are failing with the same error:
2019-05-23T16:23:52,414 ERROR [main] io.druid.cli.CliPeon - Error when starting up.  Failing.
java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_77]
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_77]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_77]
at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_77]
at io.druid.java.util.common.lifecycle.Lifecycle$AnnotationBasedHandler.start(Lifecycle.java:421) ~[java-util-0.12.3-111.jar:0.12.3-111]
at io.druid.java.util.common.lifecycle.Lifecycle.start(Lifecycle.java:319) ~[java-util-0.12.3-111.jar:0.12.3-111]
at io.druid.guice.LifecycleModule$2.start(LifecycleModule.java:134) ~[druid-api-0.12.3-111.jar:0.12.3-111]
at io.druid.cli.GuiceRunnable.initLifecycle(GuiceRunnable.java:101) [druid-services-0.12.3-111.jar:0.12.3-111]
at io.druid.cli.CliPeon.run(CliPeon.java:301) [druid-services-0.12.3-111.jar:0.12.3-111]
at io.druid.cli.Main.main(Main.java:117) [druid-services-0.12.3-111.jar:0.12.3-111]
Caused by: java.lang.NullPointerException
at org.codehaus.jackson.JsonFactory.createJsonParser(JsonFactory.java:604) ~[jackson-core-asl-1.9.13.jar:1.9.13]
at org.codehaus.jackson.map.ObjectMapper.readValue(ObjectMapper.java:1973) ~[jackson-mapper-asl-1.9.13.jar:1.9.13]
at org.apache.curator.x.discovery.details.JsonInstanceSerializer.deserialize(JsonInstanceSerializer.java:86) ~[curator-x-discovery-4.0.0.jar:?]
at org.apache.curator.x.discovery.details.ServiceCacheImpl.addInstance(ServiceCacheImpl.java:193) ~[curator-x-discovery-4.0.0.jar:?]
at org.apache.curator.x.discovery.details.ServiceCacheImpl.start(ServiceCacheImpl.java:96) ~[curator-x-discovery-4.0.0.jar:?]
at org.apache.curator.x.discovery.details.ServiceProviderImpl.start(ServiceProviderImpl.java:75) ~[curator-x-discovery-4.0.0.jar:?]
at io.druid.curator.discovery.ServerDiscoverySelector.start(ServerDiscoverySelector.java:138) ~[druid-server-0.12.3-111.jar:0.12.3-111]
... 10 more
I see someone also meet this problem and He thought this is is a issue of curator : https://github.com/apache/curator/commit/cc30b67c8dbe24babd31ed654e9536fe776b8a18.
how about merge this patch or replace our curator version?

That commit is marked as https://issues.apache.org/jira/browse/CURATOR-452, which says it was fixed in 4.1.0. We updated to this version in 0.14: https://github.com/apache/incubator-druid/pull/6862, so we should have this update already. If you still see this issue in the latest release (0.14.2) please let us know.

That commit is marked as https://issues.apache.org/jira/browse/CURATOR-452, which says it was fixed in 4.1.0. We updated to this version in 0.14: #6862, so we should have this update already. If you still see this issue in the latest release (0.14.2) please let us know.

ok, thank you!


端口被占用依然启动成功

address  already use 。 starting should be fail in spring cloud project

I don't think this has anything to do with spring cloud alibaba

i think  dubbo don't need port , so dubbo can started。but service in spring cloud is restservice ，so ,that should be fail

this issue in  spring-cloud-dubbo

Do not worry, it's a dubbo issue : https://github.com/apache/incubator-dubbo/issues/3958


Error when trying to run pine.py

C:>python C:\PineBOT\Pine-master\pine.py
Traceback (most recent call last):
File "C:\PineBOT\Pine-master\pine.py", line 1, in
from lib import pine
File "C:\PineBOT\Pine-master\lib\pine.py", line 2, in
from termcolor import colored
ModuleNotFoundError: No module named 'termcolor'

try

pip install termcolor

In console pip install termcolor

It didnt resolve

In console pip install termcolor

Always the same error

I uninstalled it and reinstalled, ir worked for me.


Why is to_lowercase called?

Before sending to postgres and after parsing from postgres, the text content is always passed through the Rust String to_lowercase method.

Why is this done? Isn't the point of the CITEXT type that the DB backend handles this?

CiString ,which is what CiText converts to, calls to_lowercase() when instantiated to not have to call to_lowercase when doing an equality check are when creating a hash key. I'm open to suggestions and/or PRs if something could be improved upon.

But why is CiString needed?

It's needed so that you can do case insensitive comparisons in rust. It easily converts to string if that's what you need,

Ok gotcha. It might be cool if there was a way to just use String as the Rust type instead of the wrapper if you don't need that functionality.

What is the point to lower the string in CiText? The point to have CiText in Postgres is to have strings that looks like a regular text, but internally compares insensitively. You effectively destroyed the mission of CiText.


error: the server doesn't have a resource type "machinedeployment"

What happened:
I tried to delete a cluster with the following config:
apiVersion: kubeone.io/v1alpha1
kind: KubeOneCluster
name: demo
versions:
  kubernetes: "1.13.2"
cloudProvider:
  name: "none"
hosts:
- publicAddress: '<IP_ADDRESS>'
  privateAddress: '<IP_ADDRESS>'
  sshPort: 22 # can be left out if using the default (22)
  sshUsername: ubuntu
machineController:
  deploy: false
  provider: "none"

This failed with the following issue:
~/go/bin/kubeone reset config.yaml -v
INFO[08:22:07 CEST] Resetting kubeadm…
INFO[08:22:07 CEST] Destroying worker nodes…                      node=<IP_ADDRESS>
+ kubectl cluster-info
+ 1> /dev/null
+ kubectl annotate --all --overwrite node 'kubermatic.io/skip-eviction=true'
node/master0 annotated
+ kubectl delete machinedeployment -n kube-system --all
error: the server doesn't have a resource type "machinedeployment"
Error: failed to exec command: export "PATH=$PATH:/sbin:/usr/local/bin:/opt/bin"

set -xeu pipefail

if kubectl cluster-info > /dev/null; then
  kubectl annotate --all --overwrite node kubermatic.io/skip-eviction=true
  kubectl delete machinedeployment -n "kube-system" --all
  kubectl delete machineset -n "kube-system" --all
  kubectl delete machine -n "kube-system" --all

  for try in {1..30}; do
    if kubectl get machine -n "kube-system" 2>&1 | grep -q  'No resources found.'; then
      exit 0
    fi
    sleep 10s
  done

  echo "Error: Couldn't delete all machines!"
  exit 1
fi
: Process exited with status 1

What is the expected behavior:
Deletion proceeds.
How to reproduce the issue:
Try to delete a cluster when the following is set in config.yaml
machineController:
  deploy: false
  provider: "none"

Anything else we need to know?
Information about the environment:
KubeOne version (kubeone version): v0.6.0-6-g6407139
Operating system: CentOS Linux release 7.6.1810 (Core)
Provider you're deploying cluster on: Bare-metal
Operating system you're deploying on: CentOS Linux release 7.6.1810 (Core)

Hey, thanks for the report!
We already aware of this issue.
Closing as duplicate of #398

@Kube-ASY Thanks for reporting the issue!
This is a known issue and we have a ticket open for that one - #398. Considering this issue has more details than #398, I'll leave it open and close another one. We'll get this fixed for v0.7.0 release, which will be released in the upcoming weeks. As a workaround, you can skip destroying workers with the --destroy-workers=false flag.

changing milestone to 0.8 as original issue was in it

Considering this issue should not take too long to fix, I think we should get it done for 0.7, so I'll move it back and assign it to myself.
/assign


Clean up Contacts throughout the site

Decide which pages should have contacts (individuals, slack, emails). No twitter, discord.
Airtable audit where everything shows up.
Decide on a uniform way of presenting contacts (i.e. button, mini cards, or links)
Delete lingering contacts (many are throughout remote MDX pages)
Related: maintain people's roles and where they show up

Related:
https://github.com/carbon-design-system/carbon-platform/issues/1044
https://github.com/carbon-design-system/carbon-platform/issues/295

Where contacts show up:

Footer:

IBM Design Medium ( https://medium.com/design-ibm)
Twitter (https://twitter.com/ibmdesign)

Case studies

Email<EMAIL_ADDRESS>

Meetups: All IBMer only

Carbon and IDL playbacks: Subscribe on Your Learning
Data viz playbacks: email
Design pin-ups: email

Contact Page:

Github issues
Discord (https://discord.gg/SYjvP8epkM)
Slack channels: #carbon-design-system #carbon-components, #carbon-react,
Email<EMAIL_ADDRESS>Twitter: @carbondesign

Designing>Figma: #figma-guild
Contributing

Overview and Bugs pages should redirect to Contact, but need updated link (current: https://next.carbondesignsystem.com/help/contact-us)

Data viz get started

Links to Carbon Charts core team in GitHub (also shows contributors)

Aubrey’s suggestions (let’s start the conversation here)

Decrease sprawl of contacts throughout the site by directing everyone to the Contact page for any case involving email, social media, opening bug/suggestion issues, or general slack/discord channels. For example, replace the Medium and Twitter links in our footer with the Contact page. Same goes for various contacts on the Case studies and Contributing pages.
Other pages can still have contact links, but we will unify how they look. I.e. We can retain the #figma-guild slack channel on the Figma page and channels on the Library pages (issue #1044). I’m uncertain about the Carbon Charts Github link, which goes to a directory of current and past members as well as contributors.
Re-work the Contact page a little. I.e. Only link to the Carbon Medium and Twitter (not IBM Design)
Defer adding contributors on library page until post v1 (issue #295)

@jeannie ok. i'll add the blue to the file and redlines.

@aubrey-oneal
I think that funneling people to the contacts page is great. I think things get a little tricky with opening bug/suggestion ideas. With so many libraries, we'll have to think about which repos to surface and where people can open general bugs. We should probably stop pointing people to our twitter. No one really maintains it. I do wonder if there is any benefit in directing people to our medium (@mattrosno do we have any data if people click the medium link in our footer?)
Agreed that we should unify how we are presenting contacts—links verses cards. Are you seeing any patterns or is it sort of random how they are presented? I don't think that we need cards for everything and that links, like you mentioned in the Figma page, do work well in some scenarios.
Agreed on the contact page needing some work. Again, it get a little tricky on which contacts show up here—do we want to include carbon core things as well as other local systems? only carbon core? do we want to keep it even more general?

@aubrey-oneal we spoke with our open source manager this week, and for now, we aren't listing individual names on the site for contributors/maintainers. That's something that we can explore post-v1, but that's why https://github.com/carbon-design-system/carbon-platform/issues/295 is deferred for later.
Like @kingtraceyj said, some Slack channels are library-specific, like how #carbon-charts should be linked to from the Carbon Charts libraries. We determined that this issue is also post-v1, as it's an enhancement to the library and asset overview pages, to list the best contact channels for that library (e.g. Slack, Discord, etc.) https://github.com/carbon-design-system/carbon-platform/issues/1044

If we have to choose between showing two equivalent links like https://medium.com/carbondesign or https://medium.com/design-ibm (it could be an "and", but if we have to choose...), I'd choose the option that's most relevant to our users, which in this example would be Carbon's Medium, Carbon's Twitter, etc. I'd choose the higher-signal option.

Design decisions are finished. After the code freeze there will be dev work (already started by my me) to:

Redo Contacts page based on Figma design. This is done on my local branch save for icons for discord, slack, and twitter.https://www.figma.com/file/ZOILKMMO2LCrSeUYIBHi7C/Contacts?node-id=6%3A2474
Delete medium/twitter in footer and insert contacts page link
Add medium link to 'Articles' page

Link case study contact to contact page: https://github.com/carbon-design-system/carbon-platform/discussions/1270#discussioncomment-3684307


Support @this tags

Currently this is for type checking only -- I haven't added any support in the language service yet, so quickinfo probably doesn't display anything useful.
Fixes #13308

CI is failing because of public API baselines that I have now updated.
After some discussion with @mhegazy we decided to support the brace-free specification of types, to align with usejsdoc.org, even though this makes comments ambiguous in the Typescript scenario, where the type is not normally specified:

/** @this A - the type is 'A', the comment starts with '-' */
function f(this: number) {
}

However, @param already works for this scenario, so we expect @this to be used almost exclusively in Javascript.

CI is failing because of public API baselines that I have now updated.
After some discussion with @mhegazy we decided to support the brace-free specification of types, to align with usejsdoc.org, even though this makes comments ambiguous in the Typescript scenario, where the type is not normally specified:

/** @this A - the type is 'A', the comment starts with '-' */
function f(this: number) {
}

However, @param already works for this scenario, so we expect @this to be used almost exclusively in Javascript.


listBranches() only returns the last component of the branch path

After looking at the code, it looks like this was done intentionally, but I'm not sure why.  I feel I should be able to pass the values returned by listBranches() to getRef() (appended to "heads/") without issue, but if I have a branch 'feature/foo', listBranches() will return 'foo', and getRef() will fail.

Fixed here

Fixed by #231. Thanks, @olivierbeaulieu!


Progress Percentage inside loading bar

Hello,
I would like to know if its possible to show the progress of the loading bar.
For example:
http://www.tennodrops.com/relics/
I recreated the page above with the Nanobar (the one above conflicts allot(html css build))
But i can't figure out a way to add the percentage numbers inside the bar.
Thankyou

Hello, I didn't design the progress bar to do that, but you can try changing the innerHTML of the element:
const progress = nanobar({
  id: 'my-id'
})

const elem = document.getElementById('my-id')

function go (x) {
  progress.go(x)
  elem.lastChild.innerHTML(x)
}

go( 30 )


Qa

Coverage increased (+0.03%) to 83.21% when pulling 60f1284b658a7abeda6272146dbe4895d1a01968 on qa into ae9ea838b4437b281fb0db79576e237d5e39a826 on master.


Asu 1226 add sap invoice sender

Codecov Report

Merging #176 (823d7c8) into develop (2ce845f) will increase coverage by 0.07%.
The diff coverage is 95.12%.

@@             Coverage Diff             @@
##           develop     #176      +/-   ##
===========================================
+ Coverage    94.72%   94.80%   +0.07%
===========================================
  Files           97      100       +3
  Lines         2846     3081     +235
  Branches       235      255      +20
===========================================
+ Hits          2696     2921     +225
- Misses         109      115       +6
- Partials        41       45       +4

Impacted Files
Coverage Δ

invoicing/models.py
92.85% <92.30%> (-0.59%)
:arrow_down:

apartment_application_service/settings.py
85.29% <100.00%> (+0.14%)
:arrow_up:

invoicing/enums.py
100.00% <100.00%> (ø)

application_form/models/reservation.py
95.34% <0.00%> (-4.66%)
:arrow_down:

application_form/api/sales/views.py
97.22% <0.00%> (-2.78%)
:arrow_down:

invoicing/api/serializers.py
96.92% <0.00%> (-0.10%)
:arrow_down:

application_form/services/application.py
97.95% <0.00%> (-0.05%)
:arrow_down:

invoicing/utils.py
100.00% <0.00%> (ø)

application_form/enums.py
100.00% <0.00%> (ø)

application_form/services/queue.py
100.00% <0.00%> (ø)

... and 8 more

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 2ce845f...823d7c8. Read the comment docs.

Superseded by https://github.com/City-of-Helsinki/apartment-application-service/pull/195


Created counting sort algorithm.

Let me know if there are any changes to be made.

Also, one more thing.. I think the filename is incorrect.
In the README file, I think the number 3 was assigned to Counting Sort.
This does not really matter, but p1_ is already taken by bubble sort, so yea.

Also, one more thing.. I think the filename is incorrect.
In the README file, I think the number 3 was assigned to Counting Sort.
This does not really matter, but p1_ is already taken by bubble sort, so yea.

Yeah, I'll fix it and create a PR.

Yeah, I'll fix it and create a PR.


Key stage text derived from list of available key stages

Description
The text "Key stage 1 and 2" and  "Key stage 3 and 4" will now be generated from the key stages available.

Although we haven't got any it also now supports single key stages, such as "Key stage 1" or "Key stage 3"
Issue(s)
Fixes CUR-857
How to test
Test for regressions in the above UI. There should be no functional changes until we enable cycle 2.

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


[BUILD] Bump jetty 9.4.41.v20210516

Why are the changes needed?

close #710
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #711 (95bd9a3) into master (2a05326) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #711      +/-   ##
==========================================
- Coverage   80.52%   80.51%   -0.02%
==========================================
  Files         122      122
  Lines        4730     4737       +7
  Branches      572      574       +2
==========================================
+ Hits         3809     3814       +5
- Misses        600      601       +1
- Partials      321      322       +1

Impacted Files
Coverage Δ

...g/apache/kyuubi/engine/spark/KyuubiSparkUtil.scala
90.90% <0.00%> (-3.21%)
:arrow_down:

...rg/apache/kyuubi/engine/spark/SparkSQLEngine.scala
68.51% <0.00%> (-0.72%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a05326...95bd9a3. Read the comment docs.


add "iHostART" to the list of Seedbox Hosting Providers

add this in Seedbox Hosting Providers
No DMCA torrenting allowed and seedboxes
iHostART based in romania

Send a PR, please. Don't forget to remove index.php.


chore(react-ui): refactor code to improve DX

~Also automatically formatted code with prettier for consistency (yarn run format).~ (I'll make another PR for this)

Basically this code here
https://github.com/atlasmap/atlasmap/blob/3f9b80366ffb0310f3df2b0bd8bc741e3a6e7547/ui-react/packages/atlasmap-standalone/src/App.tsx#L117-L138
is a shortcut to this
https://github.com/atlasmap/atlasmap/blob/3f9b80366ffb0310f3df2b0bd8bc741e3a6e7547/ui-react/packages/atlasmap-ui/src/AtlasmapUI/AtlasmapUI.tsx#L32-L50
right now the differences are not many, but when the MappingDetail components will be fleshed out and the TableView component added, the advantage will be more evident.

Okay - so is this the final commit for review or is there another coming?

Final one. The one I removed is the automatic formatting done with prettier, that doesn't need a review really but since it will touch a lot of files it's better kept separated.

(actually let me change a little thing I just noticed 😅)

Done, now the comment I made before makes more sense.


xmonad recompile fails

Issue description
Trying to set up dzen on xmonad, xmonad --recompile fails.
Steps to reproduce
I have an xmonad.hs like
import XMonad
import XMonad.Hooks.DynamicLog

main = xmonad =<< dzen defaultConfig
              { modMask = mod4Mask  -- super instead of alt (usually Windows key)
              , terminal = "urxvt"
              }

(I think this is right? It's basically verbatim from http://xmonad.org/xmonad-docs/xmonad-contrib/XMonad-Hooks-DynamicLog.html)
When I xmonad --recompile, though, I get an xmessage with
xmonad.hs:2:1: error:
    Failed to load interface for âXMonad.Hooks.DynamicLogâ
    Use -v to see a list of the files searched for.

I'm a novice with xmonad, but searches for the errors give suggests like "you're missing certain Haskell packages", which makes me wonder if the xmonad-with-packages doesn't include the DynamicLog stuff?
Technical details

System: 16.09.1803.40de598 (Flounder)
Nix version: nix-env (Nix) 1.11.6
Nixpkgs version: "16.09.1803.40de598"

@NeQuissimus uses xmonad, they might know

xmonad-with-packages is only a wrapper which you pass packages to that you want included with your ghc environment xmonad is running in (see the xmonad module) for how it should be used). If you are on nixos you should install xmonad with the module:
services.xserver.windowManager.xmonad.enable services.xserver.windowManager.xmonad.enableContribAndExtras
Otherwise just override the xmonad-with-packages just as in the nixos module.

You may just need contrib and extras. Try adding the following:
      windowManager.xmonad = {
        enable = true;
        enableContribAndExtras = true;
      };

That was exactly what I needed. Thanks!


Duplicate workspace reruns all 'folderOpen' task in the new one

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.78.2
OS Version: Win10
Multiroot workspace

Don't know if this is a feature request or a bug, but I want the ability to duplicate the workspace, without rerunning all the 'auto-run' tasks.
If it's not a bug, maybe a possibility of a second command Duplicate Workspace without tasks? or a new setting don't run 'folderOpen' tasks when duplicate: true

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Tap to focus is not working

Steps to Reproduce

Execute the example https://pub.dev/packages/camerawesome/example
Tap different locations on screen
-> Focus is not at the place where you tapped

Expected results
Focus should be at the place where you tapped
Actual results
Focus is not at the place where you tapped
Video
camera awesome example
https://github.com/Apparence-io/CamerAwesome/assets/13718034/da61e3fc-7fb5-4598-bce7-2aa0f2beba8d
iOS photo app
https://github.com/Apparence-io/CamerAwesome/assets/13718034/b3d35cf6-b295-437a-bc5f-371d97778b40
About your device

Brand
Model
OS

Apple
iPhone 12 mini
16.4.1 (a)

Your flutter version
Framework • revision 4d9e56e694 (4 weeks ago) • 2023-04-17 21:47:46 -0400
Engine • revision 1a65d409c7
Tools • Dart 2.19.6 • DevTools 2.20.1

camerawesome version: 1.4.0
I have the same issue with my Samsung Galaxy smartphone. It looks like the tap to focus behaves more like a slider. If you tap to the left it will focus the far, when tapping the right it wil focus the near.
I also tried it with camerawesome version 2.0.0-dev.1. Same issue.

Same here.

Same on Android I think the problem is the tap is not switching between the lenses. So it is not able to focus correctly.
Is the plugin still manteined??


feat: create new playground layout

Description
This is a draft PR for building a new layout of Modelina Playground. The new layout will be similar to the layout of Async API Studio. The first step will be to create a new layout and then I will be working on the logic. I have created a temporary component file NewPlayground.tsx to build the new layout. We will replace the old components with the new ones once the layout is ready.
Related issue(s)

Resolves #1346

Hi @jonaslagoni @codingmickey, I was trying to use React Split Pane for the section which will show input, output, and the output navigation. But it seems that it haven't been updated since last three years and doesn't support React v18.
I found this new react-split-pane implementation by Jeremy Grieshop  which supports React v17 and above. It's also been used in the AsyncAPI Studio. Can I use this instead? The problem is that it isn't available as package so I will have to copy paste whole repo as a component as also done in the studio.

@devilkiller-ag I would say no, dont use a tool that has no users 🙂 Then its better to have no split pane functionality.

Hi @jonaslagoni, I agree with you. For now I have used the code for the split pane from Async API Studio Codebase. But if you want I can try to implement it from scratch for our playground. What's your opinion?

Hi @jonaslagoni, I agree with you. For now I have used the code for the split pane from Async API Studio Codebase. But if you want I can try to implement it from scratch for our playground. What's your opinion?

You can select the direction you want to take, there is no requirement here 🙂 We don't have split panel right now anyway, so it can be added later if desired 👌 Then all you need to focus on is the basic layout, making it easier to achieve.

Okay Sure, I will go with the basic layout for now 🙂

I have added the mobile responsive layout without split-pane. Now I will add the Editor, Options, and Output UI.

@devilkiller-ag this is starting to look great!
Only point I have is that it's impossible to see the models + options that are generated. Would it be possible to have the sidebar + input + models + output side by side somehow? 🤔

Hi @jonaslagoni, I did exactly that. Strangely, it's not visible on your side. Here is the recording of the new UI.

I have also checked the PR deploy preview, and it's working exactly the same as the local deployment. Are you talking about the sidebar, input, output, and models same as shown in the above gif or have I misunderstood something? I'm sorry for the confusion.

Do you see that when you have the different models visible and what to change the options, the models view disappears?
Thats the part I don't like, i.e. I think the models view should always be visible and maybe right next to the output view 🙂 Similar to what we have in the current modelina playground. We can still have the button in the sidebar to toggle it to be visible and hidden, but it should not be closed just because you want the options 🙂

Okay Got it. So you want something like this?

Yes 👍

Do we need to make the layout mobile responsive? Or Should we ask the user visiting from mobile to visit from desktop? The current layout is not mobile-responsive. Do you have any layout for mobile devices in mind?

I don't think there exists a layout that will work well with a mobile view 🤔 Even in it's current form it's almost impossible to see anything. You could just display all the views on mobile devices like it currently works. I.e. so there is no regression between the current and new version of playground 🤔

Did you mean, we should display the old playground layout for mobile viewers? Can't we just display something like this: "Revisit from desktop to use Modelina Playground"?

@jonaslagoni The work is complete! Kindly check it out.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.516%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5600

Relevant Lines:
5901

💛  - Coveralls

Hi @jonaslagoni,
I have incorporated all the suggestions. Kindly check it out.

Once we merge this I would love to offer you code ownership of the modelina-website code, let me know if you want to stick around for longer than a single PR 😉
Let me know if you have any questions about it 👍

Hi @jonaslagoni, I have made all the changes you suggested, additionally, I have changed the Tooltip color and padding to make it visible. Thanks for offering me the code ownership of the modelina-website. I would love to be able to contribute forward. I had an issue in mind that I was going to open just after this PR closes: Currently, the website code uses class components, we should update the code base to use functional components and hooks at all places. This will result in cleaner, easier-to-understand components compared to class components. What's your view on this?

Hi @jonaslagoni, I have made all the changes you suggested, additionally, I have changed the Tooltip color and padding to make it visible

Perfect 👍

I had an issue in mind that I was going to open just after this PR closes: Currently, the website code uses class components, we should update the code base to use functional components and hooks at all places. This will result in cleaner, easier-to-understand components compared to class components. For the Playground, I have updated most of the class components to functional components on which I worked. What's your view on this?

Sounds good, feel free to create an issue for it to track it 🙂

/rtm

@all-contributors please add @devilkiller-ag for code, design


AFHTTPClient DefaultHeaders not propagating to HTTPRequestOperationWithRequest

I have created an AFHTTPClient and setDefaultHeaders.
When I use the AFHTTPClient to HTTPRequestOperationWithRequest:success:failure: the resulting AFHTTPRequestOperation does not have the client's Default Headers added to the request.
Is this expected behavior?
I would expect the client's headers would be appended to the URLRequest's headers when AFHTTPRequestOperation is created.

Right, expected behavior would be for any requests generated either directly or indirectly would have those default headers applied. See the Gowalla API clients in either one of the sample projects for example usage.
Could you show exactly me how you are using it?


Update tutorial to show use of the task parameters

Once #15 is complete, we should also update the tutorial notebook https://github.com/rly/ndx-beadl/blob/main/docs/tutorial_nwb_userdays_2022/beadl_example_nwb_userdays_2022.ipynb to show use the new task parameters

@mavaylon1 do we have updated data files so that we can update the tutorial?


Disable unused function warning in MacOS

Fixes #87

Codecov Report

Merging #106 into master will increase coverage by 0.2%.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##           master     #106     +/-   ##
=========================================
+ Coverage    91.4%   91.61%   +0.2%
=========================================
  Files          11       11
  Lines        2152     2217     +65
=========================================
+ Hits         1967     2031     +64
- Misses        185      186      +1

Impacted Files
Coverage Δ

titan-compiler/driver.lua
70.65% <85.71%> (+0.53%)
:arrow_up:

titan-compiler/coder.lua
95.53% <0%> (+0.31%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f797af...83d995e. Read the comment docs.


adds support for fn as a modifier key

The Fn key (at least on macOS) is actually a modifier key, so this commit adds support for it to be used as such...the only thing is...is that it doesn't actually seem to operate the same way as on the physical keyboard. For example, "Fn Backspace" should send "Delete", but it ends up just sending "Backspace" again. Based on my testing, "Fn" is being seen as a modifier, but it doesn't actually seem to be modifying. I'm not sure if this is the expected behavior since there are, indeed, separate keycodes for Backspace and Delete.
I initially came down this path hoping to get around the missing "Enter" keycode I implemented in #168. I was hoping I could just get around it by sending Fn + Return, but it didn't work. Based on how the other modifier keys are implemented, I believe I have implemented Function correctly, but have not seen the expected results.
Looking for some guidance... thanks!

Hi @smithkyle 👋
I'm quite busy at the moment, will try to get back to you as soon as things calmed down.
Sorry for the delay

No problem at all! :)

It looks like some keys implicitly set the Fn modifier per this:
https://github.com/koekeishiya/skhd/issues/1
But that doesn't explain why Fn+Backspace doesn't work...


Feature/os xcompile

The work  on this branch should allow compilation on OSX by fixing a rather byzantine regex issue; one that gave working code on linux but failed on OSX.

Addressed with PR #61


Added Dulwich College Suzhou

Dulwich College Suzhou Added

emails are in the form of<EMAIL_ADDRESS>Thank you very much :)

@cgodbold Pull request merged. Thank you!

Thank you Philipto :)


[WEB] Error: The string supplied did not seem to be a phone number

Hi @natintosh
I use the below api with number = '4' and isoCode='en'
I know that 4 is not a valid number but the user is typing in to a search box and I have to call parse at every key stroke.
final parsedNumber = await PhoneNumberUtil.normalizePhoneNumber(number, isoCode);
and this throws an exception. Is this expected behavior? Shouldn't we just get a null String returned instead?

The error text right before the exception is from https://github.com/natintosh/plugin_libphonenumber/blob/main/libphonenumber_web/web/js/libphonenumber.js
NOT_A_NUMBER:"The string supplied did not seem to be a phone number"

Yes... Just wrap it in try catch. When it's unable to parse a number it's throws an exception

@natintosh thanks for the answer. In libphonenumber I guess they handled this in the plugin code. When I migrated to your plugin these exceptions appeared. I still want to suggest we have the same behavior for the sake of consistency. What do you think? best regards.


Transfer ipyigv

Target

The code repository: https://github.com/QuantStack/ipyigv
The npm package on npmjs.com (if available): ipyigv
The python package on pypi.org (if available): ipyigv
The conda-forge recipe on https://github.com/conda-forge (if available): ipyigv
The documentation website (if available):

Reason

Checklist

[x] You are owner of the repository you want to transfer
Which role do you want to have in the jupyter-widgets-contrib organization (check one of):

[x] Help with all jupyter-widgets-contrib repositories
[ ] Help with the transferred repository only
[ ] I don't have time anymore to keep helping the project

The transferred repository (check one of):

[x] Must keep the extension name
[ ] Cannot use the original name
[ ] I don't care

[x] The code is available under a Open Source license
[x] A license file is present in the code repository
[x] The repository contains a README file describing:

[x] The provided feature(s)
[x] A picture illustrating the feature(s)
[ ] The available options/settings (if applicable)
[x] How to install the extension
[x] optional a Binder link to test the feature online

[ ] optional The frontend code (Typescript/Javascript) is tested
[ ] optional The backend code (Python) is tested
[x] I certify the extension is not using exclusively paid-plan of any commercial service
[x] I will help the transfer process on the code repository AND the built packages

You must enable two-factor authentication on GitHub

https://github.com/jupyter-widgets-contrib/ipyigv


[Feature] add outdated packages list output

As with nu -i, add an option for listing outdated packages.
This would be an alias for npm outdated, pnpm outdated and yarn outdated.
Example: nu -o
Thank you for this very helpful tool! ❤

@DrJume First off, sorry for digging up an old issue! I'm experimenting with extending the existing commands in a fork I've created at https://github.com/nberlette/n (published as @brlt/n on npm). One of several new commands is no, which lists outdated dependencies via <npm|pnpm|yarn> outdated.
If you're able to check it out by any chance, your opinion will help quite a bit with moving towards a PR for it here. Thanks!

We recently introduced a new na command for aliasing agents purely. So you could run na outdated for that. And if you prefer, you can set the alias to rename it to no or anything you want in your local shell.
alias no="na outdated"


有人安装成功吗？？？

死活安装不成，在多机环境下安装成功吗

Currently install package is not stable. If use docker, you can get image from https://hub.docker.com/r/paddledev/paddle/tags/

close due to no details, reopen it if @ylqfp still has issue


python check_services failing for STR (TA3) and langston

I realized this is a python issue that may involve CLI more than anything on the docker end, so I added Eric and removed John for now.

Notes from Tyler at steresearch

update
We’re referring to the programmatic endpoint exposed in the Python semtk3 library. It’s imported like this:
from semtk3 import check_services
There’s an illustrative example of how this is intended to be used – as a lambda function that tells docker when the service is ready – in the RACK codebase at:
RACK-Ontology/cli/tests/conftest.py
But the code that invokes it is actually commented out there, which seems to suggest this has been down for a while.
We actually learned of the GUI’s ability to check services after we sent this issue report out. It’s promising that there’s a working example of service checking!
original
SemTK exposes an endpoint that checks if associated services are ready. This feature would be very convenient for us to use to avoid errors when spinning up our containerized services. Unfortunately it is broken, unilaterally returning a value of False even once all associated services are up.

I'm not familiar with the services status API. Is this something I can have the CLI invoke before it does anything else?

My experience when writing those tests was that check_services never reported that services were up, even when the GUI would, which is why that's commented out. It's probably worth checking whether that's still true, and actually investigating it if so.

In my dev environment, when I runsemtk-python3's demo.py, check_services reports correctly.   I need more info on how you are running this.   This might be some kind of docker port issue?
Can I see your semtk3.set_host("http://whatever") code?

@tuxji , is there something obvious that we're missing in order to get python to call each service's serviceInfo/ping
It is on the same port, obviously, but the path starts with serviceInfo instead of the normal service name.

I don't see anything obviously missing from the Python code.  Sure, it doesn't verify that the "available" field both exists and has the value "yes" but the Java service code always returns "available": "yes" as far as I can see.  No one has reported what they actually saw logged via logger.ERROR(error) so I don't know what the problem might be.  My guess would be something incorrect was passed as the service's address or port.
 103   │     def ping(self):
 104   │         '''
 105   │             logger.INFO(success)  or  logger.ERROR(error)
 106   │             returns True (success) or False (failure)
 107   │         '''
 108   │         try:
 109   │             res = self.post_to_simple("serviceInfo/ping")
 110   │             semtk3_logger.info(self.baseURL + self.get_simple_field_str(res, "available"))
 111   │             return True
 112   │         except Exception as e:
 113   │             semtk3_logger.error(str(e).replace("\n", ""))
 114   │             return False

Steps to reproduce with the CLI test suite:

Apply this diff:

diff --git a/cli/tests/conftest.py b/cli/tests/conftest.py
index 7070f80..8768a16 100644
--- a/cli/tests/conftest.py
+++ b/cli/tests/conftest.py
@@ -13,8 +13,8 @@ def rack_in_a_box(docker_ip, docker_services) -> str:
     url = "http://{}".format(docker_ip)
     set_host(url)
     # TODO(lb): For some reason, check_services always returns false.
-    # docker_services.wait_until_responsive(
-    #     timeout=240.0, pause=0.1, check=lambda: check_services()
-    # )
-    sleep(120)
+    docker_services.wait_until_responsive(
+        timeout=240.0, pause=0.1, check=lambda: check_services()
+    )
+    # sleep(120)
     return url

Setup the CLI as you normally would for testing:

virtualenv venv
source venv/bin/activate
pip install -r requirements.txt
pip install -r dev/requirements.txt
python3 setup.py install

Run pytest and wait for the timeout

This totally might be an issue with the Docker configuation, see cli/tests/docker-compose.yml for how it's configured.

@langston-barrett :  is this one solved?

@langston-barrett :  is this one solved?

@cuddihyge I haven't tried it since I posted my last comment, so I'm not sure but I'd assume not.

@cuddihyge I haven't tried it since I posted my last comment, so I'm not sure but I'd assume not.

@langston-barrett : could you check if this is resolved. Thanks

@langston-barrett : could you check if this is resolved. Thanks

I'll try running the reproduction instructions above to see if this still happens.

I tried restoring this check_services code and the tests once again stall. While the semtk3 instance was running in docker I tried manually opening python3 and calling check_services manually:
>>> import semtk3
>>> semtk3.set_host('http://<IP_ADDRESS>')
>>> semtk3.check_services()
http://<IP_ADDRESS>:12068/fdcCache/serviceInfo/ping: failed with reason: {"timestamp":"2021-02-17T17:39:25.389+0000","status":404,"error":"Not Found","message":"No message available","path":"/fdcCache/serviceInfo/ping"}Details:

Am I supposed to have a fdcCache service on the current development version of rack?

➜  ~ curl -d "" "http://<IP_ADDRESS>:12068/serviceInfo/ping"
{"message":"operations succeeded.","status":"success","simpleresults":{"available":"yes"}}%

It looks like the python3-semtk library is just polling the wrong address. I'll try fixing this in my semtk3-python branch.

There is such a service, but we don't use it in RACK right now.  There may be a disconnect between semtk-python3 library and the list of services we start in RACK-in-a-box.    So the check-services() results are correct.
Do you think that any fix/change is worth doing?

@cuddihyge I fixed this in the branch of semtk-python3 that has the fixes/enhancements we need.
https://github.com/glguy/semtk-python3/commit/2ceee9a255fd0da92baf1b7946c4cc17c1fb4415
(This includes the two patches from PR#9 and then the new change for fdcCache)

@glguy very nice catch.
But the problem is on the Java side, not the python side.    Leave the python as-is.
I'll push a fix into the next -dev container.

@glguy very nice catch.
But the problem is on the Java side, not the python side. Leave the python as-is.
I'll push a fix into the next -dev container.

OK, cool, I'll drop that incorrect fix commit then.

This is fixed and working on our master branch using the latest rack and semtk-python3


Sort recurring jobs by name in web view

When you have a lot of recurring jobs it makes it much easier to find the one you are looking for.

Coverage remained the same at 97.971% when pulling 4fb65270c70468f0a813d1d4230f28a4626136d9 on c-sonnier:master into 41990aafa80a0f458e988733189fbf4b07e0894a on moove-it:master.

@c-sonnier I think that is what happens already, this has been done by this PR https://github.com/moove-it/sidekiq-scheduler/pull/259 and released on v.3.0.1 https://github.com/moove-it/sidekiq-scheduler/releases/tag/v3.0.1


Install digits problem

"/home/ubuntu/a/caffe/" from CAFFE_ROOT does not point to a valid installation of Caffe.
Use the envvar CAFFE_ROOT to indicate a valid installation.
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/home/ubuntu/digits/digits/main.py", line 70, in
main()
File "/home/ubuntu/digits/digits/main.py", line 53, in main
import digits.config
File "digits/config/init.py", line 7, in
from . import (  # noqa
File "digits/config/caffe.py", line 226, in
executable, version, flavor = load_from_envvar('CAFFE_ROOT')
File "digits/config/caffe.py", line 33, in load_from_envvar
% executable_dir)
ValueError: Caffe executable not found at "/home/ubuntu/a/caffe/build/t
my caffe_root:/home/ubuntu/caffe/

did you try to do export CAFFE_ROOT=/home/ubuntu/caffe/ before starting DIGITS?

Yes,I have tried,therefore show this error
this is a caffe install problem?or other?

Check the fist line you posted: "/home/ubuntu/a/caffe" whereas you state in the end your caffe_root is "home/ubuntu/caffe".

oh no ,i have modify ,but also error
ubuntu@tegra-ubuntu:~/digits$ ./digits-devserver

|   _ / | | / |
| |) | | ( || |  | | _
|/_|| || |__/ 5.1-dev
"/home/ubuntu/caffe/" from CAFFE_ROOT does not point to a valid installation of Caffe.
Use the envvar CAFFE_ROOT to indicate a valid installation.
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/home/ubuntu/digits/digits/main.py", line 70, in
main()
File "/home/ubuntu/digits/digits/main.py", line 53, in main
import digits.config
File "digits/config/init.py", line 7, in
from . import (  # noqa
File "digits/config/caffe.py", line 226, in
executable, version, flavor = load_from_envvar('CAFFE_ROOT')
File "digits/config/caffe.py", line 33, in load_from_envvar
% executable_dir)
ValueError: Caffe executable not found at "/home/ubuntu/caffe/build/tools"

It seems you are not comfortable with command-line instructions and/or manual builds. Why not try one of the other installation options to get everything set up for you? The deb packages and the docker images both come with Caffe built and DIGITS configured to use it.


UnicodeString>>reverse returns an unreversed copy

'abc' asUnicodeString => 'abc'

Fixed in 7.1, and will correctly handle reverse any UTF-8/16 string.