Problem with numeric

Hey, I have a table with one of the properties being numeric type. When I try to insert values there is a error thrown:
sql: Scan error on column index 6, name "bet_amount": bun: can't scan "42.12" into float64Result []. The problem was solved when I changed the sql type for the property to be float. Could u dig into this problem, please?

Please show the code that reproduces the problem.

My mistake- the operations is a select one-
var result []CasinoRoundBet

		db.
			NewSelect().
			Model(&result).
			Where("player_id = ? AND round_id = ? AND bet_status = ?", playerID, roundID, _placedStatus).
			Scan(ctx, &result)

This returns - sql: Scan error on column index 6, name "bet_amount": bun: can't scan "42.12" into float64 in case where bet_amount is numeric. But if I change the type of bet_amount to float this works out fine.

If you use numeric in database, you should NOT use float64 datatype to store their values. Float datatype has loss of precision. There are some third party libraries available to store numeric or currency datatypes. You should use those

Yes, I'm using float64 and numeric. Can you give me link for such library ?


Previously closed: Modify "new action" files

Hi,
My previous "issue" has been closed and I "don't have permission to reopen it".
So, I explain my "issue".
I want to create a class, which extends "Frapi_Action". And my final generated files will extends my class.
I'm using DoxyGen, and I also want to modify the @link and @author which have a wrong syntax.
So, I want to modify the generated class (when creating a new action) to make them extands the right class, and to rewrite the comments.
Can you tell me how to do that please ?
Thanks.

@Max13 Just edit the Action files by hand; FRAPI will not overwrite your changes.

I'm currently doing it, but it's heavy to do that for all my frapi projects files and new files.
That's why i'm asking.
Isn't it possible ?

It's possible but will not be forward compatible when you update/upgrade FRAPI. You'll have to modify the file https://github.com/frapi/frapi/blob/master/src/frapi/admin/application/modules/default/models/Action.php#L442 and change the line linked to have the name of your class. Make sure that your base class extends Frapi_Action otherwise you'll end up with weird behavioural issues.
Disclaimer: I do not recommend doing it, but it's possible :-)

Ok okay...
I won't do it.
But thank you anyway.
PS: Which doc generator can generate api doc with frapi ?

From the administration panel you access the URL:

/docs/generate/format/mdown
/docs/generate/format/html

It'll generate the docs from your defined actions :)

Keep in mind that the goal of frapi is to generate end-user documentation and not API documentation. Should you wish to generate API docs, I'd recommend using Doxygen as you do or phpdox or phpDocumentor and run it on the custom/ folder.

End-user doc, in this case, isn't it the same as API doc?
(I'm asking because I don't find the folder /docs/generate, I'm checking the repo.)


Demo FHIR Auth to Recuro Health

Problem statement
Recuro Health is a sender interested in connecting with us. O&O had the kickoff for RS on 10/14, and sender expressed interested in learning more about FHIR based authentication.
What you need to know
Sender does several test for employers, and is interested in learning about FHIR based auth.
Acceptance criteria
Hold a demo/kickoff with sender and determined next steps for partnership.
To do
Schedule conversation with sender and @whytheplatypus
FYSA @jimduff-usds @jbiskie

Meeting with Recuro scheduled for 2:30 PM ET on 11/8. A flow chart or some sort of visual would be helpful for the "demo".


Logic reversed?

Looking into why this isn't working for me I realised you have a blackList but then you check to make sure the property is on the blacklist before trying to unsub?

You are right, my mistake. Thanks, fixed in version 1.0.2.


Block user from drawing and getting rid of all the tools

I want the user to only watch the drawing but he won't have any access to select anything or draw anything.
designer.setTools({ 					    pencil: false, 					    text: false, 					    ...............                                            ................ 					});
After I set everything to false plus not calling designer.setSelected('pencil'); for a particular user, two things are happening.

3 tools are still showing: Line, Width and Extras
Pencil tool though not showing is selected by default and I can draw easily.

So first of all, how can I call Line, Width and Extras so that I can set them to false? And how can I unselect pencil tool so that the user can't draw?

I fixed the problem.Worked around a hack.

The Line, Width and Extras can be gotten rid by removing them from html on widget.html plus by removing their specific functions defined on the javascript

2.The pencil tool or any tool is getting activated by default on widget.html page by window.setItem.
Removed that to not autoselect any tool.


DOC: Fix package name transition year date

Fix package name transition year date.

If you give me the mecessary rights I can unarchive, apply the change to eddymotion and archive it again so that the notice does not become misleading.


unshare failure when building on ubuntu:jammy container

I'm trying to build a few packages using this action on an ubuntu:jammy Docker container, rather than the Github 22.04 runner, and I'm running into some issues with unshare:
sbuild (Debian sbuild) 0.84.2~ubuntu2204+3 (28 December 2022) on 75794b96caf5

  +===================================================================================+
  | ros-humble-octomap 1.9.8-2<IP_ADDRESS>.51 (amd64) Wed, 08 Nov 2023 00:51:04 +0000 |
  +===================================================================================+

  Package: ros-humble-octomap
  Version: 1.9.8-2<IP_ADDRESS>.51
  Source Version: 1.9.8-2<IP_ADDRESS>.51
  Distribution: jammy
  Machine Architecture: amd64
  Host Architecture: amd64
  Build Architecture: amd64
  Build Type: binary

  E: unshare failed:
  E: unable to to unshare
  E: Error creating chroot session: skipping ros-humble-octomap

  +------------------------------------------------------------------------------+
  | Summary                                                                      |
  +------------------------------------------------------------------------------+

  Build Architecture: amd64
  Build Type: binary
  Build-Space: 0
  Build-Time: 0
  Distribution: jammy
  Fail-Stage: create-session
  Host Architecture: amd64
  Install-Time: 0
  Job: /__w/tesseract_packages/tesseract_packages/src/octomap/ros-humble-octomap_1.9.8-2<IP_ADDRESS>.51.dsc
  Machine Architecture: amd64
  Package: ros-humble-octomap
  Package-Time: 0
  Source-Version: 1.9.8-2<IP_ADDRESS>.51
  Space: 0
  Status: failed
  Version: 1.9.8-2<IP_ADDRESS>.51
  --------------------------------------------------------------------------------
  Finished at 2023-11-08T00:51:04Z
  Build needed 00:00:00, 0k disk space
  E: Error creating chroot session: skipping ros-humble-octomap
  Error: Process completed with exit code 1.

Here is the link to the full action log. I'm fairly new to sbuild, so I'm not entirely sure what is going wrong here.

Afaik Docker does not support user namespaces (unshare syscall), is there any reason you want to use Docker?

Not particularly. I don't exactly know how the Github runners are configured, so I thought it would be "cleaner" to try to run this action out of a container. If it's not easily supported, then I don't think I have a problem running it on the Github runners.
Does this action only work with 22.04, or can it also support 20.04?

Not particularly. I don't exactly know how the Github runners are configured, so I thought it would be "cleaner" to try to run this action out of a container. If it's not easily supported, then I don't think I have a problem running it on the Github runners.

Note that user namespaces are cleaner then Docker.

Does this action only work with 22.04, or can it also support 20.04?

Only 22.04.


Add pyopenssl dependency

Hello bot, I signed it!
2017-06-06 12:44 GMT-04:00 googlebot<EMAIL_ADDRESS>
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-python-client/pull/402#issuecomment-306546451,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHJLDfKkc30EIPjU7C-anZhtHo-I4aYKks5sBYIFgaJpZM4NxnWe
.

Can we get some context here? What's the purpose of this change?
On Tue, Jun 6, 2017, 9:49 AM googlebot<EMAIL_ADDRESS>wrote:

CLAs look good, thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-python-client/pull/402#issuecomment-306547963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc9V-2bDCp0JOb_vGiHC9ZJsu_jQ_ks5sBYMxgaJpZM4NxnWe
.

When using the python sample from the Reporting API v4 : https://developers.google.com/analytics/devguides/reporting/core/v4/quickstart/service-py, that library is required.
It is already marked as required for developement, however if used in sample it makes sense to add it as a requirement or to edit the reporting doc to formalize that requirement.

Hmm.. this shouldn't require pyopenssl. oauth2client (while deprecated) has
pure-python support for cryptography. Which version of oauth2client do you
have?
On Tue, Jun 6, 2017 at 10:00 AM Guillem Lefait<EMAIL_ADDRESS>wrote:

When using the python sample from the Reporting API v4 :
https://developers.google.com/analytics/devguides/reporting/core/v4/quickstart/service-py,
that library is required.
It is already marked as required for developement, however if used in
sample it makes sense to add it as a requirement or to edit the reporting
doc to formalize that requirement.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-python-client/pull/402#issuecomment-306550904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc4F_QNGQw-5ZuFhuS1PgBAQcEli2ks5sBYWSgaJpZM4NxnWe
.

Here's my python version
$ python --version
Python 3.5.3

And the dependencies after I installed pyopenssl (which installed cryptography)
$ pip freeze
asn1crypto==0.22.0
cffi==1.10.0
cryptography==1.9
google-api-python-client==1.6.2
httplib2==0.10.3
idna==2.5
oauth2client==4.1.0
pyasn1==0.2.3
pyasn1-modules==0.0.9
pycparser==2.17
pyOpenSSL==17.0.0
rsa==3.4.2
six==1.10.0
uritemplate==3.0.0

@glefait can you give me the full stacktrace of the error if you don't have pyopenssl installed?

@jonparrott yup, here's a new env:
virtualenv env
source env/bin/activate
pip install --upgrade google-api-python-client
# I edit the sample to replace print with print() and default key/email with mine

python sample.py
Traceback (most recent call last):
  File "sample.py", line 85, in <module>
    main()
  File "sample.py", line 80, in main
    analytics = initialize_analyticsreporting()
  File "sample.py", line 32, in initialize_analyticsreporting
    SERVICE_ACCOUNT_EMAIL, KEY_FILE_LOCATION, scopes=SCOPES)
  File "/home/glefait/workspace/test-ga/env/lib/python3.5/site-packages/oauth2client/service_account.py", line 334, in from_p12_keyfile
    token_uri=token_uri, revoke_uri=revoke_uri)
  File "/home/glefait/workspace/test-ga/env/lib/python3.5/site-packages/oauth2client/service_account.py", line 289, in _from_p12_keyfile_contents
    raise NotImplementedError(_PKCS12_ERROR)
NotImplementedError:
This library only implements PKCS#12 support via the pyOpenSSL library.
Either install pyOpenSSL, or please convert the .p12 file
to .pem format:
    $ cat key.p12 | \
    >   openssl pkcs12 -nodes -nocerts -passin pass:notasecret | \
    >   openssl rsa > key.pem

By the way, there is also a discrepency between the doc (https://developers.google.com/analytics/devguides/reporting/core/v4/quickstart/service-py#4_run_the_sample)

Move the previously downloaded client_secrets.p12 to the same directory as the sample code.

and the replace_me info in the python file:

KEY_FILE_LOCATION = '<REPLACE_WITH_JSON_FILE>'

I see, the issue here is that we no longer recommend .p12 keys, instead, we recommend .json keyfiles. The tutorial itself should be updated to use .json keys which do not require pyopenssl.


Confused on the AMASS SMPL format

Hi @EricGuo5513 !
Thanks for the great dataset. I am struggling to read how you convert SMPL to the 263 dimensional vector that is used with the AMASS dataset. I am confused on how the pose processing and raw motion representation iPython notebooks are doing.
Say, I have the SMPL params (global orientation, body pose, beta). Can you please walk me through how these get converted and in what files?
Thank you so much in advance.

+1


Permissions overlay

Closes #36

Did a quick test, and at first it showed me the overlay because location was disabled for the app. Then I enabled it, went back to the app and the overlay was gone. I then closed the app, and disabled location in the settings again. Going back to the app: no overlay.

I suppose it did not ask me again because of the 24 hour limit?

See comment in #36 for more requirements.


Updating the go get path for vet and cover

Signed-off-by: Sergio Schvezov<EMAIL_ADDRESS>

@jessevdk: LGMT

@jessevdk LGMT


Add theme: Compact tabs title

Add theme: Compact tabs title
This PR adds a new theme to the theme library.
Theme Details

Name: Compact tabs title
Description: Show the first few title characters even in compact mode
Homepage:
Author: @Tc-001

Great idea!


Lint/UnusedMethodArgument overriding reassignment and zsuper

Per https://github.com/bbatsov/rubocop/issues/4823#issuecomment-338120387 @mvz discovered an interesting case when overriding reassignment appears with zero-arity super call.
In this case, the value of a variable is unused, so cop suggests to rename the variable. But since super call is zsuper, it captures the reassigned variable:
def foo(a)
  a = 43

  super
end

The problem has been resolved by using non-zero arity super:
def foo(_a)
  super 43
end

That being said, reassignment is capured by zsuper, but original variable's value is not used.
What should we do in such situation? Suggest using super instead of zsuper? Add a configuration option to ignore this case? Both?
Update: binding call should have a similar problem.

I think we could augment the "unused argument" logic to tell if there's a call to super. In that case we consider it used. 🙂

@Drenmi, i thought of that solution. My concern is that argument's value is unused. Is it a big deal?

@akhramov: Perhaps that is a new cop entirely? Lint/ShadowedArgument?
Something like:
Argument `a` was shadowed by a local variable before it was used.

WDYT?

@Drenmi, agreed.
Should the cop have config options for disabling zsuper / binding cases? Like IgnoreBinding and IgnoreZeroAritySuper?

I solved my case by changing the zsuper to super with arguments. If that's the preferred syntax (and I tend to agree that it is), then the cops basically work and the only issue is documentation.


Add calc.random(seed, min, max) function

Adds a random() function to the calc category. This function takes 3 optional arguments:

seed: The RNG seed (default: 0x4d595df4d0f33173)
min: The minimum number in range (default: 0)
max: The maximum number in range (default: 100)

Note that this RNG is deterministic, as we considered that, the same input should always return the same output.
Fixes #984

Marked as draft until consensus about API is reached in #984.

As there hasn't really been any progress on this and the design isn't clear yet, I'm going to close this. Still, thanks for your work!


Old wheels culling.

Another action that allows to upload to anaconda a is https://github.com/OpenAstronomy/publish-wheels-anaconda, in particular it allows to remove older wheels.
I don't this we need to implement that as part of our action – too keep it  simple though as we are publishing everything on the same repo we could have a cron that cull all the old wheels from here.

I don't this we need to implement that as part of our action – too keep it simple though as we are publishing everything on the same repo we could have a cron that cull all the old wheels from here.

@Carreau What's the scope of the action? Is it taking a page from https://github.com/pypa/gh-action-pypi-publish (in terms of scope) and only doing the upload? This is (old wheel removal) is part of my https://github.com/scientific-python/upload-nightly-action/issues/5#issuecomment-1564723037.
Removal is pretty straightforward though and you can avoid a cron job that might remove more then you care about by having someone do something similar to what I did for matplotlib in https://github.com/matplotlib/matplotlib/pull/23349 (now expanded to the following by others):
      - name: Remove old uploads to save space
        run: |
          N_LATEST_UPLOADS=5

          # Remove all _but_ the last "${N_LATEST_UPLOADS}" package versions
          # N.B.: `anaconda show` places the newest packages at the bottom of
          # the output of the 'Versions' section and package versions are
          # preceded with a '   + '.
          anaconda show scipy-wheels-nightly/matplotlib &> >(grep '+') | \
              sed 's/.* + //' | \
              head --lines "-${N_LATEST_UPLOADS}" > remove-package-versions.txt

          while LANG=C IFS= read -r package_version ; do
              echo "Removing scipy-wheels-nightly/matplotlib/${package_version}"
              anaconda --token ${{ secrets.ANACONDA_ORG_UPLOAD_TOKEN }} remove \
                --force \
                "scipy-wheels-nightly/matplotlib/${package_version}"
          done <remove-package-versions.txt

Thanks for the comment and sorry for the confusion.
Basically I don't want to cull each time a new nightly is published, I wan to keep the action to just publish new nightly and as simple as possible. Then we can centrally decide how we cull older wheels separately.
We had some of those discussion in person – like should the action both built the wheel and upload ? Should it also remove, and should the number of wheel kept is configurable. But the more things you try to do, and the more option you had, the more disagreement there is.
https://github.com/OpenAstronomy/publish-wheels-anaconda is another action that takes care of removal

I think doing the clean up centrally will be easier than asking each project that uploads wheels to also clean up (and pick sensible parameters).

+1 to centralise this.
If we do the cleanup ourselves, we have to consider the case were projects don't upload nightlies but weeklies or have another timeline. i.e. maybe keep at the last n versions.

i.e. maybe keep at the last n versions.

we also need a time cutoff, maybe a month or 2 months? (see the case of scikit-image in the old location where it was built once, but never again, and therefore was extremely outdated, tripping the CI of a few projects)

Agreed, I would give 3 to 6 months to be more generous with "slower" projects. But we could start with a lower bar and revisit if the use case ever comes up.

I would argue that those "slower" projects may not be a good fit for these nightlies, and would not provide many benefits for nightly testers?

You could still want to have a way to propose a development version which is easy to pull vs making a release.
e.g. I am working on SALib and we have a very very low volume of commit which does not require frequent updates. Still, some of our users sometimes ask us to test our things before a release. Sure in this case the package is easily built enough, but you might have a more complex project or just add the mechanism as a way to show that your project is serious.
But as I said, we can also wait for folks to ask that and start as you proposed with something a bit more aggressive.

Fair enough, thanks for clarifying with an example.

Of course 😃 But now sleep, I should not play with the jet lag haha

I like the idea of keeping the last N wheels.
Situations like scikit-image sound like something we should notice and somehow address "human to human". For example, do they upload so rarely because the project rarely changes, because something broke, etc.

Given https://github.com/scientific-python/upload-nightly-action/pull/12#issuecomment-1570414245 is there agreement on 1-action-per-repository? If so, then if I've understood @tupui correctly this means we need a new remove-nightly-wheels repo (or something of that sort).

Why not having a single action which go through all our packages and apply a fixed policy? I think that's what we were saying here no?
The only concern I have is that we need to be careful with this action and its token as it would be an admin token in the wild. So it's going to be a trade-off between security and conveniency for all packages.
Here I think it would be an acceptable risk to use an admin token for that purpose as it's to apply a global policy.
Should we do a small call to discuss these?

+1 on keeping it here


Problems with parameters: a bytes-like object is required, not 'str'

Hello,
when starting adb-sync I always get the following error message. I'm using Windows 10, Python 3.7.2 and the latest version of the script.
C:\Users\Marco>adb-sync --reverse /sdcard/DCIM/Camera ~/Downloads INFO:root:Sync: local b'~/Downloads/Camera', remote b'/sdcard/DCIM/Camera' Traceback (most recent call last):   File "C:\Users\Marco\Desktop\adb-sync-master\adb-sync.py", line 883, in <module>     main()   File "C:\Users\Marco\Desktop\adb-sync-master\adb-sync.py", line 870, in main     if not syncer.IsWorking():   File "C:\Users\Marco\Desktop\adb-sync-master\adb-sync.py", line 507, in IsWorking     return self.adb.IsWorking()   File "C:\Users\Marco\Desktop\adb-sync-master\adb-sync.py", line 221, in IsWorking     b'date +%s' % (self.QuoteArgument(test_string),)]) as stdout:   File "C:\Users\Marco\Desktop\adb-sync-master\adb-sync.py", line 76, in __init__     self.popen = subprocess.Popen(args, stdout=subprocess.PIPE)   File "C:\Users\Marco\AppData\Local\Programs\Python\Python37\lib\subprocess.py", line 775, in __init__     restore_signals, start_new_session)   File "C:\Users\Marco\AppData\Local\Programs\Python\Python37\lib\subprocess.py", line 1119, in _execute_child     args = list2cmdline(args)   File "C:\Users\Marco\AppData\Local\Programs\Python\Python37\lib\subprocess.py", line 530, in list2cmdline     needquote = (" " in arg) or ("\t" in arg) or not arg TypeError: a bytes-like object is required, not 'str'
I saw under Issue #10  that chaning the script could solve this problem but I'm not familiar with Python at all. I also tried different Python versions but no success. Are there any other solutions or will this problem be fixed in the future? I love this tool and I would be happy to use it again.
Thanks in advance!

Hi.
One line above this file C:\Users\Marco\AppData\Local\Programs\Python\Python37\lib\subprocess.py, line 529, add this line "arg = arg.decode() if type(arg)==bytes else arg" without quotes and in file adb-sync line 214 change and add this lines:
s = '(;  #ls$PATH'"(\\\\){};!\xc0\xaf\xff\xc2\xbf'
test_strings = [
b'(', s.encode('utf-8')
]
After that, it works like a charm.
Test it.

I followed the instructions above, and eventually got it working. However, the instructions are a little unclear, so I thought I'd help clarify.
For the first step, the line above which you insert the new line is indeed line 529 at the time of this writing (v3.7.4). To prevent a misalignment from an update however, this is the actual content of line 529 above which to insert: needquote = (" " in arg) or ("\t" in arg) or not arg (technically this is line 530, but line 529 is an empty line, so just put the new line above one of them)
For the second step, the new code was not formatted properly as a code block, meaning when I copy and paste it in it's invalid. Instead of trying to figure out how Github mangled the string, I just added a "return true" line at the start of that "IsWorking()" function.

I think it is a bad idea to directly edit the files in the python standard library. It might break other python scripts/programmes using that function. Also, you will need to edit it every time when python is updated.
In fact, python allows you to replace functions on the fly (also called monkey-patching). So you can change the behaviour of the relevant function in the subprocess module within adb-sync itself without modifying the file in standard library.
Just add the following after all the from ... import ... lines after line 28 of the adb-sync file like the following (code copied from standard library file with adjustments made):
... # code before
from typing import Callable, cast, Dict, List, IO, Iterable, Optional, Tuple, Type

def list2cmdline_patch(seq):
    """
    Translate a sequence of arguments into a command line
    string, using the same rules as the MS C runtime:

    1) Arguments are delimited by white space, which is either a
       space or a tab.

    2) A string surrounded by double quotation marks is
       interpreted as a single argument, regardless of white space
       contained within.  A quoted string can be embedded in an
       argument.

    3) A double quotation mark preceded by a backslash is
       interpreted as a literal double quotation mark.

    4) Backslashes are interpreted literally, unless they
       immediately precede a double quotation mark.

    5) If backslashes immediately precede a double quotation mark,
       every pair of backslashes is interpreted as a literal
       backslash.  If the number of backslashes is odd, the last
       backslash escapes the next double quotation mark as
       described in rule 3.
    """

    # See
    # http://msdn.microsoft.com/en-us/library/17w5ykft.aspx
    # or search http://msdn.microsoft.com for
    # "Parsing C++ Command-Line Arguments"
    result = []
    needquote = False
    for arg in seq:
        bs_buf = []

        # Add a space to separate this argument from the others
        if result:
            result.append(' ')
        arg = arg.decode() if type(arg)==bytes else arg
        needquote = (" " in arg) or ("\t" in arg) or not arg
        if needquote:
            result.append('"')

        for c in arg:
            if c == '\\':
                # Don't know if we need to double yet.
                bs_buf.append(c)
            elif c == '"':
                # Double backslashes.
                result.append('\\' * len(bs_buf)*2)
                bs_buf = []
                result.append('\\"')
            else:
                # Normal char
                if bs_buf:
                    result.extend(bs_buf)
                    bs_buf = []
                result.append(c)

        # Add remaining backslashes, if any.
        if bs_buf:
            result.extend(bs_buf)

        if needquote:
            result.extend(bs_buf)
            result.append('"')

    return ''.join(result)

subprocess.list2cmdline = list2cmdline_patch

class OSLike(object):
... # code after

And change the isWorking() function as below:
  def IsWorking(self) -> bool:
    """Tests the adb connection."""
    # This string should contain all possible evil, but no percent signs.
    # Note this code uses 'date' and not 'echo', as date just calls strftime
    # while echo does its own backslash escape handling additionally to the
    # shell's. Too bad printf "%s\n" is not available.
    s = '(;  #`ls`$PATH\'"(\\\\\\\\){};!\xc0\xaf\xff\xc2\xbf'
    test_strings = [
        b'(', s.encode('utf-8')
    ]
    for test_string in test_strings:
      good = False
...


Add more screenshot options (format, quality and base64)

Add possibility to specify the quality of the screenshots and filetype of the screenshots and whether to return a base64 string or store it.

Should make #14 Visual Regression testing with jest-image-snapshot https://github.com/americanexpress/jest-image-snapshot possible.

Adds options to change format and quality as requested in #256


Add blog post about SIG Storage new features for v1.17

These 2 blogs, from SIG Storage, should be published today (9th): pending approval from the authors.
This is a precursor to the 1.17 release blog itself, which will be published at 1600 Eastern on the 10th

/hold

/assign @kbarnard10

/retitle Add blog post about SIG Storage new features for v1.17
Hope this is OK @rawkode ?

/test all

I don't know how I messed up that rebase, but I think I got most of the changes manually again
@xing-yang @sftim

volume snapshot blog looks good.
/lgtm

Talked to @rawkode on slack. LGTM for CSI Migration blog post with the understanding that the minor comments I had are resolved first thing tomorrow (December 10)
Thanks for porting this over.
/lgtm

/hold cancel

/approve

/assign @kbarnard10

/approve


Optional chaining not working with void type

Demo
playground demo
TypeScript source with nightly version 3.8.0-dev.20191224
type Address = {
    city: string
}

type User = {
    home: Address | undefined
    work: Address | void
}

declare let u: User

u.home?.city
u.work?.city

JavaScript output the same for both "work" and "home" properties
"use strict";
var _a, _b;
(_a = u.work) === null || _a === void 0 ? void 0 : _a.city;
(_b = u.home) === null || _b === void 0 ? void 0 : _b.city;

Bug
If you run the code above, you will see that for u.home?.city there are no errors.
But for u.work?.city there is an error:
Error: Property 'city' does not exist on type 'void | Address'
Expectation
Optional chaining works for both 'undefined' and 'void' types the same way.

Please follow the issue template for bugs. Here you would have to fill out terms you searched for to find existing issues.
Duplicate of #35236.

void is meant to be treated as an opaque return type for functions, since the type system allows you to assign () => number to a () => void.
In other words, it signals not just "this function doesn't return anything", but also "this functions values are not meant to be witnessed".

I believe my issue not about void itself. We could leave void definition the same as now.
It's about optional chaining doesn't skip empty types in union type.
Not working, but expected to work:
type Address = {
    city: string
}

declare let value: Address | void | null | undefined | never

value?.city

Working good:
declare let value: Address | null | undefined | never

value?.city

Why void breaks optional chaining - not clear for me still.
What stops us from adding support for optional chaining with void in union type - not clear.

I believe my issue not about void itself.

Your issue is about your type being potentially void. Please check the other issue I linked you, the one you duplicated. Ryan provided reasoning there.

ok, I will try to find out my root issue.
Originally, my type was Address | undefined.
For some unknown yet reason undefined type getting lost through layers of generics and type inference.
Therefore, in output I've got just Address instead of Address | undefined.
Once I replaced undefined with void my output got much stronger and void not lost.
So, output became expected Address | void.
And then, I found that I cannot use optional chaining with void as undefined.

Actually, this is a big issue for functions with optional contexts.
Currently, the only way to make a function that accepts an optional context is to define it as
this: SomeType | void, since this?: SomeType doesn't work.
Due to this issue, it's currently impossible to use optional chaining inside the function to access properties of such context:
type Foo = { bar?: { baz: string; }; };

function example(this: Foo | void, value: string) {
  return this?.bar?.baz ?? value; // Currently an error here
}

example('default string');

One could argue that this is solvable by using undefined instead of void, but this forces all invocations to provide the context of undefined:
type Foo = { bar?: { baz: string; }; };

function example(this: Foo | undefined, value: string) {
  return this?.bar?.baz ?? value; // No error, cool
}

example('default string'); // "The 'this' context of type 'void' is not assignable to method's 'this' of type 'Foo | undefined'." 😕

So, currently, optional chaining is broken completely for functions with optional contexts. See more examples in this playground.

I, personally, have many uses for functions like this in my libraries and projects. And I believe other people have use-cases too.

I think this is a simple example showing that unions with void are actually helping prevent runtime problems:
function simpleCallback() { }

async function advancedCallback() {
  await Promise.resolve();
}

// registerCallback can be called using simpleCallback or advancedCallback for example
function registerCallback(callback: () => Promise<any> | void) {
  // Does not compile unfortunately:
  // callback()?.catch(console.error);

  // workaround without optional chaining
  let result = callback();
  if (result) {
    result.catch(console.error)
  }
}

@johan-gorter I think you should rather show why void should be part of the union type, and not undefined.
void in TypeScript means the absence of a type, not the absence of a value. There might be a value, but it shouldn't be used. If a value is present your check might even produce false-positive.
http://www.typescriptlang.org/docs/handbook/basic-types.html#void

void is a little like the opposite of any: the absence of having any type at all.

https://github.com/microsoft/TypeScript/wiki/FAQ#why-are-functions-returning-non-void-assignable-to-function-returning-void

Another way to think of this is that a void-returning callback type says "I'm not going to look at your return value, if one exists".

See also this comment by @RyanCavanaugh:

the problem with assuming a void-returning function is going to return a falsy value is that this code is legal:
var x: () => void = () => true;
if (x() || somethingImportant()) {
    // ...
}

@MartinJohns, there's no arguing that void shouldn't be a part of a union type that defines a value.
But there is a case I mentioned earlier in which there's no way for void not to be a part of such a type, as it just breaks typescript altogether.
In @johan-gorter's example, the void type can be easily replaced with undefined - voilà, the problem is solved, optional chaining works:
function simpleCallback(): undefined { return; }

async function advancedCallback() {
  await Promise.resolve();
}

// registerCallback can be called using simpleCallback or advancedCallback for example
function registerCallback(callback: () => Promise<any> | undefined) {
  // EZ
  callback()?.catch(console.error);
}

However, in the case of void being the part of a union type for a function context (this) - it's impossible to just remove the void or replace it with undefined.

@Raiondesu You definitely brought up a valid case for void as part of the union type, but I think optional chaining support for void is the wrong approach to solve this (for the issues mentioned already).
void doesn't mean there is no context, it means there's no type information for the context - but it might very well be present and truthy and screw over optional chaining. So even if they'd allow ?. on union types involving void, they couldn't narrow the type and would have to fall back on any (or add another unsoundness to the type system).
To slightly adjust your example:
// `bar` is not optional anymore.
type Foo = { bar: { baz: string; }; };

function example(this: Foo | void, value: string) {
  // Only use optional chaining for `this`:
  return this?.bar.baz ?? value;
}

this might be truthy, but not match the Foo type, and accessing bar would cause a runtime error.

To solve your issue it would be better to allow to omit the context argument when the type is undefined.
type Foo = { bar?: { baz: string; }; };
function example(this: Foo | undefined, value: string) { return this?.bar?.baz ?? value; }

// Make this legal!
example('default string');

Or alternatively accept that the caller has to pass along undefined.

I now clearly see the problem with this. I do agree.
But I still don't see how the context could just "suddenly" be different from the expected one (assuming we run code in 'use strict' mode).
I mean, unless the user explicitly sets the context using call, bind or apply, the context is undefined by-default. This means that TypeScript would intervene whenever the user changes context to the one that doesn't match the type.
And, also, if (this) still works, so this doesn't really add up with the "this might be truthy" argument. Why is optional chaining outlawed in this case, but a simple if type-guard is not?
type Foo = { bar?: { baz: string; }; };

function example(this: Foo | void, value: string) {
  if (!this) {
    return value;
  }

  // TS doesn't complain here!
  return this.bar?.baz;
}

example('default string');

If what you're implying is indeed true, and void represents that "no type information" is given - then we should've also outlaw if (this) here along with optional chaining on this (which would break the backward compatibility). Or... we could just allow optional chaining, as it is just a shortcut for if (this), is it not?

To solve your issue it would be better to allow to omit the context argument when the type is undefined.

Should I create a separate issue then?

This is not an intended use of void and you should never, ever on purpose write something like Foo | void in an input position. Any time you're tempted to write void like that, use unknown, never, or undefined depending on what you mean.

Not sure if this is the right issue to leave this, but I think inference of void is a problem, as a function that does not return anything is inferred as () => void, and cannot be assigned to () => undefined, even though in this case TypeScript should know that my function actually returns undefined. See this playground for an example.
I'm also not sure why || works in place of ?? when void is involved.

@bschlenk see https://stackoverflow.com/questions/58885485/why-does-typescript-have-both-void-and-undefined

I understand why both exist. What I don't get is why typescript would ever infer a function to return void. If void means that a function could return anything, but we won't use the value, then typescript should never infer void - how does typescript know we won't be using the return value? It seems much more sensible for a function like () => {} to infer as () => undefined, since typescript knows in this case that the return value is undefined, not "anything".

So let's say you write something like this
class Base {
  func() {  }
}
class Derived extends Base {
  func() {
    return 0;
  }
}

Is this an illegal class hierarchy because callers of Base#func are guaranteed to get back undefined ?
It doesn't seem like it is. It seems like Base didn't say anything about the return type of func, because it didn't, and Derived is acting in a legally covariant fashion by returning "more" than its base type.
Moreover, it seems nearly pointless to talk about a function whose return value is contractually obligated to be a particular primitive value. How many functions do you call that are specified to return 5 where you consume that value? Or false ? Or anything else? I've never read docs where they say "Accepts a function; this function must return true". Why would anyone do that? The idea of defaulting to that mode when describing a function this way seems counter to usefulness.

I have faced this in such situation:
const p = await new Promise<{ id: number; }>((resolve) => resolve({ id: 1 })).catch(console.warn);
console.log(p?.id);

Property 'id' does not exist on type 'void | { id: number; }'.

Playground
I'm not using void anywhere in my code, but Promise.catch itself returns void | T which is very annoying.

@HanabishiRecca The problem isn't catch but console.warn, which returns void.


⚠ ClamAV has degraded performance

In b27531d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 0b7c90f.


feat: Add an animation to hide the card from the carousel

Hi everyone!
When we remove a card from the carousel, it isn't easy to understand that it worked, as there is no animation.
Furthermore, if we have several cards, it seems that nothing happens at all, as the card will be switched immediately.
By adding a slight animation, the user will better understand what happened.
Here is what I've done (reduce opacity + translation):
https://user-images.githubusercontent.com/246838/200371952-c1dc0f1e-a1ed-4e11-b6e8-754a08fd9cc4.mp4

Codecov Report

Merging #3285 (04334d2) into develop (ea9904f) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #3285      +/-   ##
===========================================
- Coverage    10.53%   10.48%   -0.05%
===========================================
  Files          249      249
  Lines        12248    12302      +54
===========================================
  Hits          1290     1290
- Misses       10958    11012      +54

Impacted Files
Coverage Δ

...pp/lib/cards/product_cards/product_title_card.dart
0.00% <0.00%> (ø)

...mooth_app/lib/pages/product/edit_product_page.dart
0.55% <ø> (ø)

...ges/smooth_app/lib/pages/product/summary_card.dart
0.00% <0.00%> (ø)

...s/smooth_app/lib/pages/scan/scan_product_card.dart
0.00% <0.00%> (ø)

...ib/pages/preferences/user_preferences_account.dart
19.10% <0.00%> (-1.88%)
:arrow_down:

...oth_app/lib/pages/product/simple_input_widget.dart
2.02% <0.00%> (-0.23%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


StackOverflow shift-clicking items in any GUI from CM

Same as https://github.com/thraaawn/CompactMachines/issues/31, re-opening because it still present.
Steps to reproduce:

Place down any CM block with GUI
Open the block
Shift-click any item in your inventory. Boom, crash.

Crash-report: http://hastebin.com/itavimujev.coffeescript

Currently working on a 1.10.2 version, the 1.7.10 version is not being maintained anymore, sorry.
Closing all issues not relevant for the new version.


[Rules] Specific hardware targeting (rust)

Does that rule "No: Specific hardware targeting" means the code should not be written to target a specific hardware, or does that also includes compilations options such as the rust:
File .cargo/config.toml
[build]
rustflags = ["-C", "target-cpu=native"]

From https://nnethercote.github.io/perf-book/build-configuration.html#cpu-specific-instructions
Which doesn't target a specific hardware, but compile for the hardware it runs on. Which is different:

The code works for any hardware
Only the compiled binary will be restricted to the hardware it was compiled on (but without it, in Rust, the binary is still restricted to the CPU architecture class)

With that configuration, I observe ~25% speedup (on GitHub codespace) for rust (but it's slower with rust_con)

The rule was originally made to prevent simd optimization on certain CPUs, so this would fall into that category.

Thank you for the clarification


Update README.md

his real name is aryan, we both are doing this project together, but he don't know he can update his name in GitHub.

Thank you very much for the update haha
Can I also include your name and github then?

He did the main work, but thank you for asking. Don't need to add my name.
Though if you want to add custom dataset training of YOLOv3 then let me
know, I have tried it. most people have done it on COCO. It is not
implemented fully, but I know the flow now.
Also, thank you for your GitBook, it helped us a lot.
On Sun, 10 Nov 2024 at 02:23, André Araújo @.***> wrote:

Merged #17 https://github.com/andre1araujo/YOLO-on-PYNQ-Z2/pull/17 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/andre1araujo/YOLO-on-PYNQ-Z2/pull/17#event-15239458602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKS5VG737OHEBAIMUEDIXBDZ7ZY25AVCNFSM6AAAAABROXVWTCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGIZTSNBVHA3DAMQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

We didn't actually implemented it, but we guess hypothetically that it is
possible.
We went for 2 classes, fighter jets B2 and Rafal and we also started
training but the laptop didn't have GPU so it was taking too long for our
patience, so we dropped it then. It is also possible to train on Google
colab. After that we only need to change some parameters in the script
where we need to replace classes and number of classes with coco's data and
that should work if everything goes well.
On Mon, 11 Nov, 2024, 02:20 André Araújo, @.***> wrote:

Thank you and also I am glad the work was helpfull.
Can you elaborate on the custom dataset part? Did you use a different one
than COCO? What classes did it detect?
—
Reply to this email directly, view it on GitHub
https://github.com/andre1araujo/YOLO-on-PYNQ-Z2/pull/17#issuecomment-2466912703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKS5VG5ZXR7YZ6K5BRUZI63Z77BIBAVCNFSM6AAAAABROXVWTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWHEYTENZQGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah nice.
I guess it would be a one class detector because it isn't suposed to see the B2 because it is a stealth bomber, isn't it? (that was a joke)
The GPU is always a problem, especially for more complex models. You could try something lighter like YOLOv3-Tiny whish should take less time and trials to train.
Any question you or Aryan may have, just feel free to ask!

We have changed number of classes and class names in yolo_image.cpp, but it
still detects the older labels of COCO which we removed from the code to
keep our own labels, still.
On Mon, 11 Nov, 2024, 15:05 André Araújo, @.***> wrote:

Ah nice.
I guess it would be a one class detector because it isn't suposed to see
the B2 because it is a stealth bomber, isn't it? (that was a joke)
The GPU is always a problem, especially for more complex models. You could
try something lighter like YOLOv3-Tiny whish should take less time and
trials to train.
Any question you or Aryan may have, just feel free to ask!
—
Reply to this email directly, view it on GitHub
https://github.com/andre1araujo/YOLO-on-PYNQ-Z2/pull/17#issuecomment-2467662834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKS5VG4DHUVMOTC5RIEKMGT2AB26PAVCNFSM6AAAAABROXVWTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRXGY3DEOBTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

We implemented tiny yolo and it worked in first time at morning, now it is
not just working. We don't know why, we are getting the same gcc error in
all different datasets. We don't know why. Do you have any idea?
On Mon, 18 Nov, 2024, 10:10 Mantra Solanki, @.***>
wrote:

We have changed number of classes and class names in yolo_image.cpp, but
it still detects the older labels of COCO which we removed from the code to
keep our own labels, still.
On Mon, 11 Nov, 2024, 15:05 André Araújo, @.***>
wrote:

Ah nice.
I guess it would be a one class detector because it isn't suposed to see
the B2 because it is a stealth bomber, isn't it? (that was a joke)
The GPU is always a problem, especially for more complex models. You
could try something lighter like YOLOv3-Tiny whish should take less time
and trials to train.
Any question you or Aryan may have, just feel free to ask!
—
Reply to this email directly, view it on GitHub
https://github.com/andre1araujo/YOLO-on-PYNQ-Z2/pull/17#issuecomment-2467662834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKS5VG4DHUVMOTC5RIEKMGT2AB26PAVCNFSM6AAAAABROXVWTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRXGY3DEOBTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi!
I don't know why the gcc error might be happening. Maybe it was due to the last modification you made on the c++ script. Before checking out what it says, try to delete the executable files (for example, one named "yolo" that is created after the gcc compilation) and delete the object files as well (the files in the objects folder). Then try to compile again with the Makefile.
Good luck!


CRITICAL: Placing Call on Hold

When you place a call on hold and you come back to pick the call back up, the callers voice doesn't get transmitted over but the hold music does cut off on the receivers side and the receiver can talk and the caller can hear the receiver, but the caller can talk but the receiver can not hear the caller.

Hi @AndyHaas I will look at it.

Hi @AndyHaas  Would you please provide more information, such as browser version, network situation?  Actually, I can't reproduce this problem. Would you try it again?

@embbnux I was not able to reproduce this morning. I will continue to test.
Chrome 63.0.3239.87

Thanks, @AndyHaas I think it was caused by unstable network. Web phone audio is based on WebRTC which dependent on stable network.

Closing, until user can reduplicate, will troubleshoot further.


[otbn] Clarify error behavior in specification

Use consistent terminology: an error is when OTBN detects something
went wrong. An alert is one of multiple actions as result of an error.
Explicitly spell out what happens as result of a recoverable or fatal
error.
Give a bit more general context on errors, including a "over the
thumb" guidance on when an error is recoverable, and when it is fatal.
To avoid error nesting, specify that a secure wipe operation does not
trigger any errors on its own.
Specify that secure wipe operations triggered as part of the error
handling scheme, or as part of a normal program termination, do not
raise the done interrupt. Otherwise, we get two done interrupts for a
normal program termination (one from the program, and one from the
internal wipe), or a spurious done interrupt if a fatal alert was
detected, but host software never started an operation (e.g. if a
integrity violation is detected when reading the DMEM).
Remove the big TODO note in the specification. Most of it is already
covered implicitly in the text (e.g. program termination is following
ECALL semantics, and we do an internal wipe, which clears the reg
files). For the bus blocking topic we have an issue on file.
Use "signaled" instead of "signalled" consistently, following American
spelling.

Thanks @rswarbrick, updated to address your comments.

Thanks for having another close look @rswarbrick, your eyes are clearly better than mine.


Add contract id to data source http call

what
provides a provision extension to enrich the HttpDataAddress with an header containing the contract id
why
to being able to forward that information to the backend data source
how
registers a resource definition and a provisioner that on the provider side will modify the content data address adding the contract id header: Edc-Contract-Id

side note: on consumer side the EDR contains a parameter with the name cid which contains the agreement id (?)

side note: on consumer side the EDR contains a parameter with the name cid which contains the agreement id (?)

yes, but that can't be usable in the data-plane-http because it get lost before

side note: on consumer side the EDR contains a parameter with the name cid which contains the agreement id (?)

yes, but that can't be usable in the data-plane-http because it get lost before. I mean, there's no injection point where we can make use of that field to be put in the request

Yes, that was my guess too, of course. My intention last evening was just to document that we now have different names for the same field for somehow the same purpose. We should be as consistent as possible to avoid as much confusion as possible for end users and operators.


Definition sources?

In response to  issue #93 pertaining to sloppy naming conventions and descriptions, is there any source in particular you would like us to be drawing definitions from?

There is no one place, unfortunately.  We'll have to do it example-by-example.

So just implement definitions as we find them and if there is a problem we will be told?

That's probably the most efficient.

Alright, thank you.


bpf: Updated CoverBee for improved eBPF test coverage reporting

This PR updates the CoverBee version. This new version uses source
code in addition to the compiled binaries to generate a more accurate
code coverage report. In the previous version, the calculated
percentages were inaccurate since code that was optimized out was not
accounted for. This has been fixed and thus, coverage reports are now
actually usable as indicators.
This PR also embeds the coverage report merging code to reduce
dependencies and added the ability to generate go-cover style reports
which can be interpreted by other tooling or converted into other
coverage formats.
Lastly, this PR moves BPF tests into its own CI action which is
triggered by both changes in the bpf/ dir as well as the tests/bpf_tests
dir. The CI now runs with coverage reporting enabled and will submit the
HTML report as an artifact so we can see our coverage via github.

FYI, coverbee dependency was automatically updated to 0.2.1 by dependabot in #20663.

@tklauser That dependabot update is fine, those are the bugfixes for https://github.com/cilium/cilium/pull/20647. The new version adds additional functions which need to be invoked, hence this PR.
I don't know if it is desirable it merge the dependabot PR and then rebase this PR or to close the dependabot PR and merge this one?

I'm going to approve since this is just a bump, but I'd also like to know the answer to your above question.

@tklauser That dependabot update is fine, those are the bugfixes for #20647. The new version adds additional functions which need to be invoked, hence this PR.

I see, thanks for the explanation.

I don't know if it is desirable it merge the dependabot PR and then rebase this PR or to close the dependabot PR and merge this one?

In general, I'd say we should rely on dependabot PRs bumping dependencies. Though there might be cases where a manual bump is needed, e.g. due to API incompatibilities or for dependencies explicitly excluded from dependabot updates. For these we should ideally bump the version in a separate commit to simplify bisectability.
For this particular PR, it looks like #20663 was merged and you've already rebased.

For this particular PR, it looks like #20663 was merged and you've already rebased.

Yep. Thx for the reviews.


Merged into transformers?

Does this mean the package is going to be archived?

The wording may be a little confusing. I meant that it is integrated into the core of transformers. Transformers rely on AutoAWQ for inference only. It is not going to be archived as a result of this.


Feature requests for Test Result panel

As python moves to use the Test Result panel, there are a few feature requests I thought might make the experience better. Open to your thoughts / feedback as I don't understand all the use cases for this panel! Thanks

When you click on a test that has succeeded there is no change in the display, this felt confusing to me because it doesn't signal to the user that there was no output from this test. It also may make people may get mixed up because in the list the successful test is highlighted but the output shown is from a failed test.

https://github.com/microsoft/vscode/assets/26030610/c085a796-9a32-437c-83fc-c6bd8653bfaf

The output in the test result panel is not searchable when you have the test run dropdown selected. I think our users  would value being able to search through this output

when I press cmd+F with the different test header selected I get a search box from one but not the other

I found myself frequently wanting to click on a single test then go back to the whole test run output. I kept getting confused when clicking on the Test Run item in the list collapsed the block vs open the output. This is just different from the individual tests which clicking on them loads their output. I know you can right-click on the TestRun to get back there but it is not very discoverable.

I found myself frequently wanting to click on a single test then go back to the whole test run output. I kept getting confused when clicking on the Test Run item in the list collapsed the block vs open the output. This is just different from the individual tests which clicking on them loads their output. I know you can right-click on the TestRun to get back there but it is not very discoverable.

You can also click the "terminal" icon in the inline actions. There's not a great way to handle this without making it hard to figure out how to collapse/expand the item.

Ah yes, that makes sense I missed that icon initially. Thanks

The find widget (2) is now available in the Test Results terminal

@eleanorjboyd the find widget always worked in the former case where we show a text editor. Can you confirm that you're on the latest version of VS Code Insiders?

hm yeah I am seeing it now- sorry about that I am not sure what was wrong. Thanks

This is only currently on VS Code Insiders.

We should release to stable in 6-7 days https://github.com/microsoft/vscode/issues/198893

1 is what I fixed in my last PR. We now show the message/output associated with a successful test, if any, otherwise we show that there was no output

Thanks again for these fixes it looks great!


Making sign up design more consistent with Figma

Description
Updated the styling (and some text) to be more aligned with @r13serra11's Figma designs. No underlying logic was touched, just a lot of CSS. Gets a bit cramped in smaller screens (4" iPhone 5) but it should work fine everywhere else.
Motivation
Part of #832
Examples
Desktop:

Mobile:

Testing Guidelines
Release Checklist

[ ] This code has unit tests
[ ] I have a plan to verify that these changes are working as expected after deploy
[ ] I have a plan for how to revert, rollback, or disable these changes if things are not working as expected

Security Checklist
This PR creates, modifies, or deletes:

[ ] API routes: API routes, parameters, or user authorization
[ ] Authentication: Authentication mechanism
[ ] Credentials: Server side credentials, or secrets in configuration / source code
[ ] Cryptography: Encryption, hashing, certificates, signatures, random numbers, etc.
[ ] User data: personal information handling, logs, error messages, etc.

Additional Notes

@schowdhury8 hey can you check this again? I am having issues like this:

I can't scroll and the extra information here is getting hidden.


Bugs on Mobile Version

Bugs reported by @ericboucher:

When you click on a reef on the map and open a new reef page, it very often does not load properly and shows “No data” and stays this way. I investigated a bit and I think that it’s because “loading” is set to false and the mobile processing is too slow to catch the change. If you reload the page it works fine. This is obviously not the expected behavior….

If you play around on the map, especially around sites that have overlapping pin and click in the overlapping zone a few times, the app crashes on mobile. I was not able to reproduce on desktop. Might have something to do with the touch on mobile trying to send update requests for two competing markers. I was able to reproduce the bug using the mobile dev test mode on chrome desktop

Stacktrace
index.js:1 The above error occurred in the <Popup> component:
  in Popup (created by Context.Consumer)
  in ForwardRef(Leaflet(Popup)) (at Popup/index.tsx:32)
  in Popup (created by WithStyles(Popup))
  in WithStyles(Popup) (at Markers/index.tsx:113)
  in Marker (created by Context.Consumer)
  in ForwardRef(Leaflet(Marker)) (at Markers/index.tsx:100)
  in MarkerClusterGroup (created by Context.Consumer)
  in ForwardRef(Leaflet(MarkerClusterGroup)) (at Markers/index.tsx:90)
  in LayerGroup (created by Context.Consumer)
  in ForwardRef(Leaflet(LayerGroup)) (at Markers/index.tsx:89)
  in ReefMarkers (at Map/index.tsx:62)
  in div (created by Map)
  in Map (at Map/index.tsx:50)
  in HomepageMap (created by WithStyles(HomepageMap))
  in WithStyles(HomepageMap) (at HomeMap/index.tsx:39)
  in div (created by ForwardRef(Grid))
  in ForwardRef(Grid) (created by WithStyles(ForwardRef(Grid)))
  in WithStyles(ForwardRef(Grid)) (at HomeMap/index.tsx:38)
  in div (created by ForwardRef(Grid))
  in ForwardRef(Grid) (created by WithStyles(ForwardRef(Grid)))
  in WithStyles(ForwardRef(Grid)) (at HomeMap/index.tsx:37)
  in div (at HomeMap/index.tsx:36)
  in Homepage (created by WithStyles(Homepage))
  in WithStyles(Homepage) (created by Context.Consumer)
  in Route (at App/index.tsx:49)
  in Switch (at App/index.tsx:47)
  in div (at App/index.tsx:45)
  in Router (created by BrowserRouter)
  in BrowserRouter (at App/index.tsx:44)
  in ThemeProvider (at App/index.tsx:43)
  in App (at src/index.tsx:37)
  in Provider (at src/index.tsx:36)
Consider adding an error boundary to your tree to customize error handling behavior.
Visit https://fb.me/react-error-boundaries to learn more about error boundaries.

Screenshots:

1
2
3

This appears to be a leaflet issue. I was able to reproduce it by clicking on a marker and immediately zoom out and select another marker multiple times. There is also an open issue on github about this: https://github.com/Leaflet/Leaflet/issues/5035. I opened https://github.com/aqualinkorg/aqualink-app/pull/351 in order to add an error boundary to the app. Should I link it to this issue?

1. This appears to be a `leaflet` issue. I was able to reproduce it by clicking on a marker and immediately zoom out and select another marker multiple times. There is also an open issue on github about this: [Leaflet/Leaflet#5035](https://github.com/Leaflet/Leaflet/issues/5035). I opened #351 in order to add an error boundary to the app. Should I link it to this issue?

Thanks for the insight! Your welcome to link it, if this issue closes I'll just open it again and link the pr to solve 1.

I thought of the following workaround for 1.:

I thought of the following workaround for 1.:
Our Map component accepts a style prop. We could set
<Map
  ...
  style={hasPointerEvents ? {} : { pointerEvents: "none" }}
  ...
>
  ...
</Map>

We could use a callback updateMap that is fired whenever a user clicks on a marker, defined like this:
const updateMap = useCallback(() => {
  setHasPointerEvents(false);
  setTimeout(() => setHasPointerEvents(true), 2 * 1000);
}, []);

In other words: When a user clicks on a marker, we disable all the pointer events on the map for 2 seconds (or any other timeout) so that he cannot do nothing else than wait for the map to center around that reef. It seems to work fine for desktop, but not for mobile. Just wanted to share my thought :)


[Bug Report] no tr tag in thead generated by calendar component

Element UI version
2.13.0
OS/Browsers version
any/any
Vue version
3.0.0-alpha.12
Reproduction Link
https://jsfiddle.net/eq12ovnu
Steps to reproduce
use el-calendar in nuxt project.

What is Expected?
thead's children should be tr tags not th tags.
What is actually happening?
no tr tag in thead generated by calendar component

Should notice that the fix may be incompatible. It may affect people who use css like .el-calendar-table thead > th.


SD-230 Change toast style

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 7 committers have signed the CLA.:x: dzungmmp:x: lcvinny:x: Vinh Hoi Le Chau:x: johnathan93:x: annguyenthanh126:x: duongntd99:x: dzungtranVinh Hoi Le Chau seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Create Dharma

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 1896a72567f654f140bfb82d0ef8aa6e3f655836 at: https://66fe9c8390c7d99d5652f80e--meshery-docs-preview.netlify.app


Improve the Corporate Benefits page

What's the improvement:

Improve the Corporate benefit page design to simplify it and make it cleaner.
Change "Employees Benefit" to "Corporate Benefit" all around the page and menu.
Change all references to "Deals" to "Benefits"

How should it look like (screenshot or sketch):

How should it work (write use case):

When users click on the contact us button it will take them to the contact form. We will create a new purpose for this case if one has not been already created.

Priority (tag accordingly): urgent / regular (next column) or non-urgent / wontfix (backlog column)
Domain (tag accordingly): frontend / backend
Tested on: (platform, OS, browser, version)
Branch: (name of the branch derived from staging or another development another branch)
Work on it

Derive the branch from staging.
Tag with ongoing and move the card to ongoing.
Develop the feature
Commit your work
Create a PR and ask for validation. Move the card to staged.
Ask for validation.
Close the issue if all the work have been done

** Validate **

Check on mobile and on desktop if the feature have been correctly implemented.
If everything is working fine, move the card toaccepted column.
If the feature isn't completely implemented, re-open the issue with a comment clearly what the problem is and what is the desired result. Move the card to have issues column.

Please follow https://github.com/Whichost/frontend/wiki/Git-workflow

Works ongoing. Here's how the coded page looks now. Waiting for #161 and #158 to be finalized, before I can finish this page.

Works ongoing. Here's how the coded page looks now. Waiting for #161 and #158 to be finalized, before I can finish this page.

Looking good @buckwierd
The text for the terms and conditions, can we change it to normal text in a normal colour with normal sizing?
Airbnb on their side they don't highlight them. I believe this is healthier.

This issue is a duplicate of #157.
@buckwierd let's try to keep thing organised: one issue per feature, one branch per issue, one Pull request per branch.
Here, you work on both page on the same branch, inside the same commit.


Comments in proto files misleading

Describe the bug
All the proto files that use a type call uint256 contains a comment (uint256, serialized little endian) . which indicated the byte order it is expecting , it will be copied to all the files it generated.
So when user import WalletCore and trying to use it , will also see the comments . For example , I am using it in swift to develop an IOS application,  Refer to the following screenshot

This comment is wrong , and mislead user , is says (uint256, serialized little endian) , but in fact it is actually expecting big endian also called network order
To Reproduce
Just search the code base
Expected behavior
the comment should be (uint256, serialized big endian)
Screenshots
Additional context

Hi @johnnyluo, that's right, thank you for the notice! We'll definitely fix the comments


[Bug] Registration page

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description?
All the entered details must not disappear when the entered credentials do not meet the valid format after clicking on submit, and it must show a proper error message instead of reloading.
Add Screenshots
https://github.com/user-attachments/assets/b1cb1615-9ef4-4d53-9880-f14d1c395cea
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue (Optional)
[X] I'm willing to provide further clarification or assistance if needed.

ohh, you can work on it

Hi @Luson045  issue is fixed in #385

okay 👍 actually i was checking from top


Inherit the build arguments from the base PHP image

It would be great if the Drupal-PHP image allows to pass the same build arguments as the base PHP image: https://github.com/wodby/php#build-arguments
We had several write issues with running composer inside the Drupal-PHP container which had to be solved by passing the UID and GID as build arguments. We had to combine the Dockerfiles from both wodby/php and wodby/drupal-php to get this working.

@csandanov could you please give some feedback on this issue?

This would require finding all files/directories with old uid/gids and fixing it, sounds like a lot of excessive work and it has to be in the constant sync. So, for now, my answer is no


HSEARCH-2503 Execute ExpectedLog4jLog properly even if log expectations are added during the test

https://hibernate.atlassian.net/browse/HSEARCH-2503
This reverts commit b1867179159fa2784344d922cf142524c180ac61. That
commit added an optimization that turned out bypassing assertions
completely when expectations are defined during the test (and not inline
just after calling ExpectedLog4jLog.create()).

interesting one! merged


Why there is a server?

A decentralized im system should have only clients, and all data should be stored on the block chain.
How to explain that your system is decentralized?

This service does not use any centralized middleware (such as Redis), nodes communicate with each other using gossip protocol, nodes can be scattered all over the place, and everyone can contribute their own nodes, which I summarize as decentralization, maybe a little different from the decentralization in blockchain.


Add new page 'Code of Conduct'

As a visitor of our site
I want to have a page with a code of conduct
So that I can access and read it easily.

[ ] build layout and fill in content [de]
[ ] build layout and fill in content [en]
[ ] add link to footer
[ ] add link to navigation
[ ] add link to contact form
[ ] add link to profile new

like /app/views/pages/about.de.html.erb and /app/views/pages/about.en.html.erb

German text:
Speakerinnen_CodeofConduct_de-final.docx

@athomschke could you try that?

English text:
Speakerinnen_CodeofConduct_en-Carlas-edit.docx

done :)


azureProfile.json got corrupt and shown empty

Describe the bug
azureProfile.json recently in the test environment got corrupted and contains nothing inside. This will lead none of the az command work and the following exception will be thrown
Traceback (most recent call last):
  File 'runpy.py', line 193, in _run_module_as_main
  File 'runpy.py', line 85, in _run_code
  File 'C:\\Users\\trdai\\AppData\\Local\\Temp\\pip-install-8jgnm5o1\\azure-cli\\azure\\cli\\__main__.py', line 32, in <module>
  File 'C:\\Users\\trdai\\AppData\\Local\\Temp\\pip-install-8jgnm5o1\\azure-cli-core\\azure\\cli\\core\\__init__.py', line 509, in get_default_cli
  File 'C:\\Users\\trdai\\AppData\\Local\\Temp\\pip-install-8jgnm5o1\\azure-cli-core\\azure\\cli\\core\\__init__.py', line 52, in __init__
  File 'C:\\Users\\trdai\\AppData\\Local\\Temp\\pip-install-8jgnm5o1\\azure-cli-core\\azure\\cli\\core\\_session.py', line 40, in load
  File 'json\\__init__.py', line 299, in load
  File 'json\\__init__.py', line 354, in loads
  File 'json\\decoder.py', line 339, in decode
  File 'json\\decoder.py', line 357, in raw_decode
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

The way to fix it is to delete the .azure folder in the user folder so the .azure folder will be re-created during az command execution but it would be better if RCA of why the azureProfile.json is corrupted can be found

Interesting, I see the exact same error:
Interesting. I see the exact same error:

Traceback (most recent call last):
File "runpy.py", line 193, in run_module_as_main
File "runpy.py", line 85, in run_code
File "C:\Users\trdai\AppData\Local\Temp\pip-install-8jgnm5o1\azure-cli\azure\cli_main.py", line 32, in
File "C:\Users\trdai\AppData\Local\Temp\pip-install-8jgnm5o1\azure-cli-core\azure\cli\core_init.py", line 509, in get_default_cli
File "C:\Users\trdai\AppData\Local\Temp\pip-install-8jgnm5o1\azure-cli-core\azure\cli\core_init_.py", line 52, in init
File "C:\Users\trdai\AppData\Local\Temp\pip-install-8jgnm5o1\azure-cli-core\azure\cli\core_session.py", line 40, in load
File "json_init_.py", line 299, in load
File "json_init_.py", line 354, in loads
File "json\decoder.py", line 339, in decode
File "json\decoder.py", line 357, in raw_decode
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

As in, why is it showing that same path when we are on two different machines.... `C:\Users\trdai\AppData\Local\Temp\pip-install-8jgnm5o1`.

@jpoon this issue will be fixed in the incoming release. And the stack path is a static snapshot of the paths of the environment where we built the CLI package. It is irrelevant to your local environment. It is unfortunately misleading but harmless.


Support specifying additional mounts in up command

In some contexts, it would be useful to be able to specify additional mounts that should be combined with those specified in the devcontainer.json.
An example is in devcontainers/ci to allow additional mounts to support GitHub actions behaviours such as the change to use echo myOutput=123 >> $GITHUB_OUTPUT instead of `echo "::set-output name=version::123" (link)

I wouldn't mind giving this a go if that is okay

Ignore me... it has been pointed out that this is already suppported!


Laravel 5.5 support

Hello.
Is there any hope for the Laravel 5.5 compliant package?

@rluders I just tested it in L5.5 and it works fine. I did fork it (and change testing to use sqlite instead of mysql) here, if you're interested.  Everything passes:
https://github.com/nateritter/laravel-friendships
https://travis-ci.org/nateritter/laravel-friendships

@nateritter good job! Could you submit a pr with the L5.5 tests?

@hootlex actually, the tests work fine as-is with L5.5.  I just changed composer requirements.  I made some other changes, like using sqlite instead of mysql. Reduces a dependency on Travis, and tests are slightly faster.  If you want all that, I can just submit the PR with what I've got now?

@nateritter great. Do not forget about the package autodiscovery.

@rluders Yup, all part of it.

PR added: https://github.com/hootlex/laravel-friendships/pull/92

Hey @nateritter, thanks for clarifying that it works fine as-is with 5.5. Since #92 is failing I'd rather stick with the current testing set up. Feel free to open a new pr if you think you can make the tests run faster. 🙂


Cognito - Google signon does not work

Describe the bug
Successfully created the stack and I can access the cloudfront Url without any issues.
Followed the instructions (listed below) to integrate the cloudfront url with google authentication via AWS cognito. After creating the user pool within cognito and setting up the google project within the developer account. The hosted UI within AWS cognito redirected me to the google page and the URL redirects with the cloudfront url + the token but prompts me for username and password.
https://aws.amazon.com/premiumsupport/knowledge-center/cognito-google-social-identity-provider/
To Reproduce

Click on create stack - https://github.com/aws-solutions/content-analysis-on-aws
Once the stack gets created successfully then output tab within the stack provides cloudfront URL and also creates a cognito user pool.
Follow the instructions listed in https://aws.amazon.com/premiumsupport/knowledge-center/cognito-google-social-identity-provider/ URL to configure google OAuth.
After cognito google auth is complete the hosted UI will be available within cognito - app client settings.
Click on hosted UI and then you'll be redirect to google auth but after selecting your gmail the page gets redirected to cloudfront landing page instead of creating the user account and logging in directly to the website.

FYI - Tried deploying this stack in different regions numerous times and I still see the same issue. I believe the application is not storing the access and token ID or something which is not letting the website to authentication.
Expected behavior
After google authentication AWS content analysis page redirects to the login page instead of using google / gmail credentials to log directly into the AWS content analysis page
Please complete the following information about the solution:

Version 1.0.0 (deployed)
Tried Version 2.0.0 deployment and the build failed.

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SO0021) - Video On Demand workflow with AWS Step Functions, MediaConvert, MediaPackage, S3, CloudFront and DynamoDB. Version v5.0.0". If the description does not contain the version information, you can look at the mappings section of the template:
Mappings:
  SourceCode:
    General:
      S3Bucket: "solutions"
      KeyPrefix: "video-on-demand-on-aws/v5.0.0"

[ ] Region: us-east-1
[ ] Was the solution modified from the version published on this repository? No
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the sevices this solution uses? Yes
[ ] Were there any errors in the CloudWatch Logs? No

Starting on August 30, 2023 Content Analysis on AWS will no longer be supported and the GitHub repository will be archived. Existing deployments will continue to run. If you have deployed Content Analysis on AWS via cloning GitHub open source code, you may continue to use the solution.
The functionality provided by Content Analysis on AWS will be superseded with functionality in Media2Cloud on AWS and Content Localization on AWS. We encourage you to explore these solutions.
https://github.com/aws-solutions/media2cloud-on-aws
https://github.com/aws-solutions/content-localization-on-aws


[Feature request] Suggest existing vault tags

It would be amazing if auto-tag could query existing tags and serve those up too. Here's what I imagine it would look like
Vault tags to insert
#SomeOftenUsedTag
New tags to insert
#autogeneratedTag
...
I know I personally have a lot of tags but never know off the top of my head which ones to apply. It would be really cool if the AI could remind me of relevant tags I'm already using that should be applied to this note/text selection.

It would be amazing indeed! Keeping adding new tags would get out of hand quickly enough.
For this, it's text classification with an existing set of tags. This could be done with OpenAI too of course but I'd rather look into options for handling it locally and offline if possible (free, fast, better for privacy).
Will start to look into it :-)

Started looking into it, but as the plugin is newly launched some bug reports are coming in and these get priority. 🙏
Will update here once there's progress.


Ensuring IPAddresses and IPNetworks are correctly serialized.

Closes #816

Coverage decreased (-0.03%) to 60.671% when pulling 2b260884072c5744077e197c6cf743fb0239d993 on kevgliss:816-ip-network into 249ab23df4ea78eda1abfefd8a2bdb065c5f7a88 on Netflix:master.


Bump k8s to v1.26.0

This PR also modifies MACAddressPolicy in preKubeAdmCommands for ubuntu tempaltes which should be none on ubuntu 22.04. This change was added by https://github.com/Nordix/metal3-dev-tools/pull/592 but for some this is not getting persisted in the node image. Until that is solved we will exclusively set it here.

/test-ubuntu-e2e-main
/test-centos-e2e-main

/test-upgrade-e2e-main

/test-ubuntu-integration-main

/hold
I will remove test commit once review is done.

/test-ubuntu-integration-main
/hold cancel
Passed e2e tests
https://jenkins.nordix.org/view/Metal3/job/metal3_capm3_main_e2e_test_ubuntu/124/consoleFull
https://jenkins.nordix.org/view/Metal3/job/metal3_capm3_main_e2e_test_centos/108/consoleFull
https://jenkins.nordix.org/view/Metal3/job/metal3_capm3_main_e2e_upgrade_test_ubuntu/168/consoleFull

/lgtm


Readded drop shadow on spinner

Based off of discussion on #581
Here's the spinner on a few different background colors

Closes #581

Oh, so by moving the shadow to the <svg> (instead of the <div>) that made the glitch go away?

Yup.

Done and done.


Fix typo

Learn Build status updates of commit 228dafd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-desktop/disaster-recovery-concepts.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Update the UnitMass.pounds coefficient to be more precise

The current coefficient for UnitMass.pounds should be more precise.
For example:

@swift-ci test

@swift-ci Please test


linuxPackages.bcc: use llvm 7.x

This fixes a number of the tools on aarch64.

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc maintainers @mic92 @thoughtpolice @ragnard

@GrahamcOfBorg build linuxPackages.bpftrace

Isn't llvmPackages_7 already the default?
a@link> nix-build --no-link -QA llvmPackages.llvm
/nix/store/0ylrvcslbf95qj9j889i84rkr6xc4sza-llvm-7.1.0

Or is it not the default on aarch64 yet?

Oops, you're right. I originally had the problem (and made this change) on 19.03, where the default is still 5, but it should be fine on unstable actually. I'll rebase and retarget to 19.03.

I think you need to rebase this further.

I don't know what's going on here, git log origin/release-19.03..bcc-llvm7 only reports the one commit for me.


Fixed Clone Issue in ContentProviderUtils

Fixed clone found in the class - ContentProviderUtils
Link to the issue
https://github.com/anurag444/opentracksFall2024/issues/14
Refactored the clone found in the class, created one separate function for the code, and replaced the code with a function call.

The code is reviewed. Merged.

Closing th Issue


[Program]: Write a Ruby program to convert binary to hexadecimal

Description
Write a Ruby program to convert binary to hexadecimal
Binary is a base-2 number system. It uses only two digits: 0 and 1. The digits are called bits. A bit is the smallest unit of information in a computer. A bit can be either 0 or 1. A bit can be used to represent a single value, such as 0 or 1, or it can be used to represent a single state, such as on or off.
Hexadecimal is a base-16 number system. It uses sixteen digits: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, A, B, C, D, E, and F. The digits are called hexits. A hexit is the smallest unit of information in a computer. A hexit can be used to represent a single value, such as 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, A, B, C, D, E, or F, or it can be used to represent a single state, such as on or off.
To convert a binary number to hexadecimal, we can use the following algorithm:

Convert the binary number to decimal.
Convert the decimal number to hexadecimal.

Input  : 1010
Output : A

How to contribute

Save the solution in program/program/convert-binary-to-hexadecimal/ConvertBinaryToHexadecimal.rb file

!assign


RocketJob not compatible with ActiveJob

I am running rocketjob with activejob 4.2.8. If I specify :rocket_job as the queue_adapter, I get an undefined enqueue method error.
config.active_job.queue_adapter = :rocket_job
To work around it, I explicitly initialized an instance of RocketJobAdapter like:
config.active_job.queue_adapter = ActiveJob::QueueAdapters::RocketJobAdapter.new
However, with this, I get the error:
Mongoid::Errors::UnknownAttribute:
message:
Attempted to set a value for 'job_class' which is not allowed on the model ActiveJob::QueueAdapters::RocketJobAdapter::JobWrapper.

Upgraded the ActiveJob adapter to work with Rocket Job v3, can you try the latest code in the master branch?
gem 'rocketjob', git: 'https://github.com/rocketjob/rocketjob.git'


Change DocumentTest to test from binary (pdf) file instead of text

Let's change DocumentTest to test from binary (pdf) file instead of text.

Closed by ec2e527.


[WIP] Feature/dashboard fixes 4

Purpose
Trello fixes:
https://trello.com/c/tl0zBta4
https://trello.com/c/Amuq2yTM

Looks there's a merge conflict left in the code. Can you please resolve it?

Alright, on it.


Installation Error: Stable_baselines

Hey guys! Basically I was just trying to install stable_baselines on Mac m1
Here is the code:
pip install stable-baselines

stable_baselines_error.txt
I got nothing. Please help me!!!!!

Looks like either Numpy or OpenCV could not be installed. I would recommend checking their communities for help on this.
PS: stable-baselines3 is more actively maintained, so you might want to try if it works :)

Looks like either Numpy or OpenCV could not be installed. I would recommend checking their communities for help on this.
PS: stable-baselines3 is more actively maintained, so you might want to try if it works :)

Thank You for your advice. I did try stable-baselines3 then I got:
stable_baselines3_error.txt
Then back to stable_baselines, I thought I should fix the opencv problem first, so I try to install OpenCV, then I got:
opencv_error.txt
I think there is a link.

Yeah indeed these are numpy and OpenCV errors. Looks like the older versions of these libraries do not work.
Try installing stable-baselines3 with only pip install stable-baselines3 (without the [extra]). That skips OpenCV installation but it is only used for some rendering purposes (visualizing envs etc).

pip_install_stable-baselines3.txt
I used pip install stable-baselines3 still got error. This sucks :(

I just realized we had same problem here: https://github.com/DLR-RM/stable-baselines3/issues/360
Try using conda to install libraries instead of pip.

Last login: Mon Jun 21 10:24:42 on ttys001
(base) zhangjing@zhangjingdeMacBook-Pro ~ % conda activate tf_m1
(tf_m1) zhangjing@zhangjingdeMacBook-Pro ~ % conda install -c conda-forge stable-baselines3 -y
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: |
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions

(tf_m1) zhangjing@zhangjingdeMacBook-Pro ~ % conda install stable-baselines
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.

PackagesNotFoundError: The following packages are not available from current channels:

  - stable-baselines

Current channels:

  - https://repo.anaconda.com/pkgs/main/osx-arm64
  - https://repo.anaconda.com/pkgs/main/noarch
  - https://repo.anaconda.com/pkgs/r/osx-arm64
  - https://repo.anaconda.com/pkgs/r/noarch
  - https://conda.anaconda.org/conda-forge/osx-arm64
  - https://conda.anaconda.org/conda-forge/noarch

To search for alternate channels that may provide the conda package you're
looking for, navigate to

    https://anaconda.org

and use the search bar at the top of the page.

(tf_m1) zhangjing@zhangjingdeMacBook-Pro ~ % conda install stable-baselines3
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: -
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions

Still nothing 0.0

stable-baselines is not in conda repositories... Please look at the link I gave you.
I am not able to give further technical support on this as the issue lies in the dependencies of stable-baselines, not the stable-baselines itself, and close this issue as "no tech support".


Send Identity Verification Email [#1010]

Purpose
If subject identity verification is turned on, have the user verify their identity before queuing the request for execution.
Changes

Update existing create_privacy_request endpoint to check if identity verification is on

If yes, set privacy request status to identity_unverified
Caches a 6 digit code in Redis that lasts for ten minutes
Emails that code to the user
Suspends further processing of that request.

To resume where we left off, send a POST to /privacy-request/{{privacy_request_id}}/verify added here https://github.com/ethyca/fidesops/pull/1115

Checklist

[x] Update CHANGELOG.md file

[x] Merge in main so the most recent CHANGELOG.md file is being appended to
[x] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[x] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[x] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #1010

Thank you for the review @seanpreston!

Verified I received new email with:
Your privacy request verification code is 442512. Please return to the Privacy Center and enter the code to continue. This code will expire in 10 minutes

^ Note this code is just for my local environment, nothing secure is pasted here.


Provide cell-pixel geometry in TIOCGWINSZ

Abstract
Contour's implementation of the TIOCGWINSZ ioctl(2) doesn't fill in the following two optional fields:
               unsigned short ws_xpixel;   /* unused */
               unsigned short ws_ypixel;

Contour supports Sixel. Unfortunately, without this knowledge, it's difficult to use Sixel correctly for anything but trivial work.
Motivation
These fields ought be filled in, as they are by most terminals supporting bitmap graphics (all of which I am aware).
Specification
When TIOCGWINSZ is answered by Contour, the ws_xpixel field ought be filled in with the number of native pixels per cell width, and ws_ypixel ought be filled in with the number of native pixels per cell height.
When the cell-pixel geometry changes (for instance in response to a font change), the information ought be updated in subsequent ioctl(2)s, and ideally a SIGWINCH ought be delivered as it would upon a cell geometry change.
see https://github.com/dankamongmen/notcurses/issues/1805

Oh.

Thanks @dankamongmen, I really appreciate your reports. I owe you a beer or two ;-)

Thanks @dankamongmen, I really appreciate your reports. I owe you a beer or two ;-)

i'm strictly a krokodil / bathsalts / adrenochrome man, though i have been known to quaff some Sterno in my time

btw i gave the wrong semantics here. i said they should have the cell-pixel geometry; it ought be the window geometry in pixels.

confirmed, thanks much holmes!
 notcurses 2.3.4 by nick black et al on Contour 0.2.0-unreleased-master-96b8f89
  30 rows (22px) 130 cols (11px) (60.94KiB) 48B crend 256 colors+RGB
  compiled with gcc-10.2.1 20210110, 16B little-endian cells
  terminfo from ncurses 6.2.20201114
  avformat 58.45.100 avutil 56.51.100 swscale 5.7.100

 colors: 256 rgb: y ccc: y setaf: y setab: y
 sgr: y sgr0: y op: y fgop: y bgop: y
 rows: 30 cols: 130 rpx: 22 cpx: 11 (660x1430)
 sixel colorregs: 256

  Width: 1446
  Height: 660

checks out (i've got a scrollbar which could easily be 16 pixels wide)!

Is this a problem? Just so i understand :)
On Tue, Jun 22, 2021, 06:58 Nick Black @.***> wrote:

Width: 1446
Height: 660
checks out (i've got a scrollbar which could easily be 16 pixels wide)!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/christianparpart/contour/issues/241#issuecomment-865558039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN3O5UWLRGFHVOEI7HALDTUAJ7HANCNFSM47CA72TQ
.

Is this a problem? Just so i understand :)

nope, just confirming via xwininfo that the rematerialized window pixel geometry was reasonable. it was! =]

rows: 30 cols: 130 rpx: 22 cpx: 11 (660x1430)     <----------- notcurses-info

Width: 1446   <--------------------- xwininfo
Height: 660

they're not an exact match, but i think the 16 extra pixels are probably due to the scrollbar.

Then I shall fix this

no it's working! i must not be clear


pyMARs documentation for newbies

Hi! So I am working on developing a pymars documentation for beginners who are unfamiliar with this platform and are new to model reduction in general. I currently have the documentation developed for this and was wondering where I can go from here. For instance, what platform would be best to publish this on?

I've transitioned documentation for software that I am working on over to a Sphinx/readthedocs style and it's great. That would probably be one of the best ways to integrate documentation into the project/ Plus, it is free if the repository is public.

Hi folks, thanks for your interest. pyMARS does have documentation available: https://niemeyer-research-group.github.io/pyMARS/ (this is linked to from the main repo page).

Hello friends, I'm interested in pyMARS and want to learn the simplification of chemical reaction mechanisms, but I don't know how to use it, can you guys help me? Is it possible to communicate and learn by email?


Feature request; CPU topology, sockets, cores, threads

Hi there
On a true SMP system with hardware sockets it would be beneficial to be able to specifiy a manual CPU topology for domains. Eg. sockets, cores and threads.
At the moment vcpu maps to sockets and might cause drastic performance decrease on real SMP systems due to expensive memory access when crossing over the other socket(s).
Thank you and kind regards
Sascha

thx @sspreitzer  for issue!

Related to #1020