fix: fix error when estimate fee without connecting wallet

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #2530
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[x] I've tested PR on mobile and everything seems works
[x] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

@vikiival  Thank a lot for your review. I have modified the code. ❤

😍 Perfect, I’ve sent the payout
💵 $100 @ 124.24 USD/KSM ~ 0.805 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x28eaef6507ec5a6664c51914c49f24b0354915c57b60eed894f9a7276cda743d
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


traceback: left arrow hotkey

at Object.l.fatal (https://recurse.zulipchat.com/static/min/common.4584f908a1d2.js:254:105)
= static/js/blueslip.js line 400 column 27
at MessageList.select_id (https://recurse.zulipchat.com/static/min/app.737d5314c023.js:1244:437)
   = static/js/message_list.js line 162 column 13

at Object.e.edit_last_sent_message (https://recurse.zulipchat.com/static/min/app.737d5314c023.js:1438:362)
   = static/js/message_edit.js line 373 column 9

at Object.a.process_hotkey (https://recurse.zulipchat.com/static/min/app.737d5314c023.js:1587:287)
   = static/js/hotkey.js line 524 column 9

at Object.a.process_keydown (https://recurse.zulipchat.com/static/min/app.737d5314c023.js:1591:121)
   = static/js/hotkey.js line 653 column 12

at HTMLDocument.<anonymous> (https://recurse.zulipchat.com/static/min/app.737d5314c023.js:1591:176)
   = static/js/hotkey.js line 657 column 9

at HTMLDocument.g (https://recurse.zulipchat.com/static/min/common.4584f908a1d2.js:251:25)
   = static/js/blueslip.js line 221 column 20

at HTMLDocument.dispatch (https://recurse.zulipchat.com/static/min/common.4584f908a1d2.js:125:73)
   = static/node_modules/jquery/dist/jquery.js line 5225 column 16

at HTMLDocument.h.handle (https://recurse.zulipchat.com/static/min/common.4584f908a1d2.js:118:327)
   = static/node_modules/jquery/dist/jquery.js line 4878 column 6

Potentially should fix https://github.com/zulip/zulip/issues/4259 together with this.

I am closing this.  I wasn't able to reproduce the above traceback, even with trying to force local echo delays, but as part of fixing the other issue (only edit content, not topics), I am cleaning up the error handling, so hopefully we'll get a more clear symptom if the problem doesn't go away on its own.


Cliparts size is 40x11 even though badge size is 44x11

First up, thanks for your amazing work on this app. Really love the easy way to change badges!
Describe the bug
I noticed that cliparts can only be created in a 40x11 px format (stored png also has this format). But the badge has a 44x11 px width. Any reason for that?
To Reproduce
Create a new clipart and fill it with pixels. Now go to "create badge" and insert it. There will be an area of 2px left and right of the clipart which will not have any red pixels.
Expected behavior
I would expect the canvas for new cliparts to be 44px wide.
Screenshots
Photo of the clipart:

Photo of the badge with padding.

Smartphone Info:
|Device         | Samsung Galaxy S5 (klte) |
|Android Version| 9 (Lineageos)    |
App version: 1.5.0
Would you like to work on the issue?
My android skills are low, so rather not.

@johannesschweig glad you're liking our application. I'll take up this issue and get it fixed in the upcoming version asap. Feel free to track the progress of this issue here :). Regards

as labeled bug for width(40 to 44) , i'm trying for height 16px (led) instead of default 11px(led) to generate binary matrix data.
i simply tried with variable
badgeHeiight = 11
to
badgeHeiight = 16
i got perfect view in app but when i fill any single char in text box , app just crashed & closed
LOG CAT
08-01 12:42:57.192 32128-32128/org.fossasia.badgemagic E/DrawerActivity: OnCreate, false 08-01 12:43:00.642 32128-32128/org.fossasia.badgemagic E/AndroidRuntime: FATAL EXCEPTION: main     Process: org.fossasia.badgemagic, PID: 32128     java.lang.StringIndexOutOfBoundsException: length=22; regionStart=22; regionLength=2         at java.lang.String.startEndAndLength(String.java:298)         at java.lang.String.substring(String.java:1087)         at org.fossasia.badgemagic.ui.custom.PreviewBadge.setValue(PreviewBadge.kt:499)         at org.fossasia.badgemagic.ui.fragments.TextArtFragment.setPreview(TextArtFragment.kt:429)         at org.fossasia.badgemagic.ui.fragments.TextArtFragment.access$setPreview(TextArtFragment.kt:63)         at org.fossasia.badgemagic.ui.fragments.TextArtFragment$textChangedListener$1.afterTextChanged(TextArtFragment.kt:176)         at android.widget.TextView.sendAfterTextChanged(TextView.java:8041)         at android.widget.TextView$ChangeWatcher.afterTextChanged(TextView.java:10354)         at android.text.SpannableStringBuilder.sendAfterTextChanged(SpannableStringBuilder.java:1043)         at android.text.SpannableStringBuilder.replace(SpannableStringBuilder.java:560)         at android.text.SpannableStringBuilder.replace(SpannableStringBuilder.java:492)         at android.text.SpannableStringBuilder.replace(SpannableStringBuilder.java:34)         at android.view.inputmethod.BaseInputConnection.replaceText(BaseInputConnection.java:704)         at android.view.inputmethod.BaseInputConnection.commitText(BaseInputConnection.java:207)         at com.android.internal.widget.EditableInputConnection.commitText(EditableInputConnection.java:188)         at com.android.internal.view.IInputConnectionWrapper.executeMessage(IInputConnectionWrapper.java:286)         at com.android.internal.view.IInputConnectionWrapper$MyHandler.handleMessage(IInputConnectionWrapper.java:78)         at android.os.Handler.dispatchMessage(Handler.java:102)         at android.os.Looper.loop(Looper.java:148)         at android.app.ActivityThread.main(ActivityThread.java:5628)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:853)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:737)
Errors
    **at org.fossasia.badgemagic.ui.custom.PreviewBadge.setValue(PreviewBadge.kt:499)
    at org.fossasia.badgemagic.ui.fragments.TextArtFragment.setPreview(TextArtFragment.kt:429)
    at org.fossasia.badgemagic.ui.fragments.TextArtFragment.access$setPreview(TextArtFragment.kt:63)
    at org.fossasia.badgemagic.ui.fragments.TextArtFragment$textChangedListener$1.afterTextChanged(TextArtFragment.kt:176)**

@Kamal-Sonani We might have to do a lot of changes to the PreviewBadge, I wish I could help out but I have to spend time working rn. I could help but someone has to understand the math and the algorithms used to make that work.

Last hour i debug and finally i got succes.
In bitmap creation their is fixed value 11 ,
I changed it to badge.height & it works 👍
Thanks for sharing .

Last hour i debug and finally i got succes.
In bitmap creation their is fixed value 11 ,
I changed it to badge.height & it works 👍
Thanks for sharing .

Good stuff mate ☺

Is this still up?

Is this still up?

Yep, it is. Feel free to take it

Yep, it is. Feel free to take it


Bokeh Application Handler Error

Hi there,
I am trying to get this to work on a Windows 10 machine. The error below persists using both Python 3.5 and Python 3.7. The Bokeh window opened in my browser is entirely blank. I'd be happy about any suggestions. Thanks!
===
(py37env) D:\Tools\Bioinformatics>bokeh serve --show bulkvis
2020-05-05 13:42:40,750 Starting Bokeh server version 0.12.15 (running on Tornado 5.0.1)
2020-05-05 13:42:40,750 Bokeh app running at: http://localhost:5006/bulkvis
2020-05-05 13:42:40,750 Starting Bokeh server with process id: 11364
2020-05-05 13:42:41,137 Error running application handler <bokeh.application.handlers.directory.DirectoryHandler object at 0x00000263371E0EF0>: 'plot_opts'
File "configparser.py", line 958, in getitem:
raise KeyError(key) Traceback (most recent call last):
File "d:\tools\bioinformatics\py37env\lib\site-packages\bokeh\application\handlers\code_runner.py", line 173, in run
exec(self._code, module.dict)
File "D:\Tools\Bioinformatics\bulkvis\main.py", line 19, in
cfg_po = config['plot_opts']
File "c:\python37\lib\configparser.py", line 958, in getitem
raise KeyError(key)
KeyError: 'plot_opts'
2020-05-05 13:42:41,153 200 GET /bulkvis (::1) 124.97ms
2020-05-05 13:42:41,388 101 GET /bulkvis/ws?bokeh-protocol-version=1.0&bokeh-session-id=vy5wnHDyUkL8mwnRHRzsDJXbl9HUHX9wwrHROmOe7DOK (::1) 0.00ms
2020-05-05 13:42:41,388 WebSocket connection opened
2020-05-05 13:42:41,388 ServerConnection created
===============================
Here's my list of packages installed.
(py37env) D:\Tools\Bioinformatics>pip freeze
bokeh==0.12.15
h5py==2.8.0
Jinja2==2.10
MarkupSafe==1.0
numpy==1.14.6
packaging==20.3
pandas==0.23.0
Pillow==5.1.0
pyparsing==2.4.7
python-dateutil==2.7.2
pytz==2018.3
PyYAML==3.12
selenium==3.11.0
six==1.11.0
tornado==5.0.1
tqdm==4.20.0

Have you set up a config file? See conf.md

Thanks for the quick response, I did in fact forget that. Has Oxford Nanopore changed its data structure? I have fast5_pass and fast5_fail folders as well as fastq_pass and fastq_fail folders, both containing several *.fast5 and *.fastq files. I don't seem to have an overall bulkfile. So when I try to setup the config file, I get the error:
KeyError: "Unable to open object (object 'IntermediateData' doesn't exist)"

The bulk FAST5 file is different from the ones contained in the pass/fail folders. It is a complete record of the sequencing experiment and usually gigabytes in size. The option to collect a bulk FAST5 file is in the "Advanced user options" section of the "Output" tab:

This must be turned on when you setup the experiment.

Got it, thanks a lot!


 I get an Invalid Argument whenever I do anything besides /wild
How To Reproduce:
Check out Arclight. I got a sneaky feeling.
Screenshots/Error Log:

Additional Context:
The Error happens in the console on bootup.

https://gyazo.com/2516bf48854b2df9d7dab31b00d1d25b

Unsupported platform. Feel free to make a pull request if there are simple changes but this is not claimed as supported or planned to as far as I know.


add backuper service

Looks like it's ready for testing
@lambotik
curl -H "Authorization: toke" \
-d '{"db_uuid":"xxx-xxx-xxx"}' \
 https://dbend.areso.pro/db_backup_create -v

to check:
non existing database: code 400
without auth: code 401
existing database of other user: code 400

I think it's done


UICircularProgressRing inside backround color?

:warning: This issue will be closed if the required information is not filled :warning:
If it's a question about something that is happening within your code base, either post a small amount of code, or post an example project where I can try to assist.
Make sure to communic

ate back and keep issue updated or it will be closed
UICircularProgressRing inside backround color?
Version

Version = ANSWER HERE

Details

Has the question been answered here before?

ANSWER HERE

Please provide as much detail as possible below

ANSWER HERE

Please follow issue guidelines, I cant help off of a single image and no description of the problem...


apps update

Reviewed /docs/apps pages.

I made a bunch more formatting fixes. One thing that I think could be still addressed in this PR is the Trinity page. It is mentioned that the program can create a lot of files (up to 100 000) and that user should apply for more quota because of this. Is this something we want to recommend as a first resort? Or perhaps NVMe and packaging the data would be better? @amsaren

NVMe is not a solution in the case of trinity as the job execution is typically distributed over several nodes.

NVMe is not a solution in the case of trinity as the job execution is typically distributed over several nodes.

Ok I see, thanks for clarifying!


Ticket token update

@wrandall22

Looks good to me. As Daniel said, we could put the Key client id in build.gradle and use BuildConfig.THEKEY_CLIENTID as they do in the gma app.

the key client id has moved to gradle. Ready for me to merge Bill?

looks good to me


Reverting multiple valuefile change

Task name: HelmDeployV0
Description: Reverting multiple valueFile changes done in https://github.com/microsoft/azure-pipelines-tasks/pull/13123/files
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Thanks!
I guess this revert was triggered by the order of the value files getting flipped. https://developercommunity.visualstudio.com/content/problem/1109248/helm-deploy-task-renders-the-helm-command-with-wro.html

@zentale Yes, this change was done to fix the order of valurFiles getting changed in HelmDeploy task.

Any plan reinstate the multiple values files functionality without the reported bug? There are still asks for this functionality.


Override for holdfor not working

This is not working

Hi,
sorry for responding this late, but I'm out of business for a while.
It's strange... Can you give me some other detail?


E   pandas._libs.tslibs.np_datetime.OutOfBoundsDatetime: Out of bounds nanosecond timestamp: 9999-12-31 00:00:00

GEOS, GDAL, PROJ INFO
---------------------
GEOS       : 3.10.3
GEOS lib   : /usr/share/miniconda3/envs/test/lib/libgeos_c.so
GDAL       : None
GDAL data dir: None
PROJ       : 9.0.1
PROJ data dir: /usr/share/miniconda3/envs/test/share/proj
PYTHON DEPENDENCIES
-------------------
geopandas  : 0+untagged.1.gd0911e5 # branch of 0.11 editing fiona datetime reading
numpy      : 1.23.1
pandas     : 1.4.3
pyproj     : 3.3.1
shapely    : 1.8.2
fiona      : None
geoalchemy2: None
geopy      : 2.2.0
matplotlib : 3.5.2
mapclassify: 2.4.3
pygeos     : 0.12.0
pyogrio    : v0.4.0
psycopg2   : None
pyarrow    : 8.0.0
rtree      : 1.0.0

(for the full context see https://github.com/geopandas/geopandas/runs/7480800393?check_suite_focus=true)
Given that pyogrio datetime support is only partial at this point, I'm going to skip the failing geopandas test case for the sake of that PR.

@m-richards @jorisvandenbossche I'm unclear on what the fix for pyogrio should be in this case; the datetime written to the file is invalid, and it is failing when trying to construct a pandas DataFrame due to validation logic there.  Even though these are written as strings to GeoJSON, they are detected by GDAL as datetime fields, presumably becasue they look like datetimes:
>>> from pyogrio import read_info
>>> read_info('/tmp/test.geojson')['dtypes']
array(['datetime64[ms]'], dtype=object)

Reading these as numpy arrays shows them as datetime objects:
>>> from pyogrio import read
>>> read('/tmp/test.geojson')[3]
[array(['2014-08-26T10:01:23.000', '2014-08-26T10:01:23.000',
       '9999-12-31T00:00:00.000'], dtype='datetime64[ms]')]

If GDAL showed these as string rather than datetime fields, we'd read them in just like Fiona and not run into issues.

@brendan-ward In the fiona -> geopandas case in https://github.com/geopandas/geopandas/pull/2505 we return series of object dtype with the user to then decided how to coerce to a datetime manually - this seemed like the least worst option (perhaps this could have also issued a warning, but for now it does not).
I'm not super familar with pyogrio (hoping to look at it a bit more soon) but this kind of object downcasting does seem awkward here given there is a numpy interface which will read as datetime correctly - I haven't tested explicitly but I expect the reason numpy works and pandas doesn't is because of the fixed [ns] resolution in pandas datetimes, while pyogrio and numpy is reading in correctly at the [ms] precision, which covers a larger date range in a 64 bit representation.

With latest pandas (this will appear in pandas 2.0), when a datetime64 array with non-nanosecond resolution is passed to the DataFrame constructor, pandas will no longer try to cast to nanoseconds.
So using the geojson file from above, I now see:
In [38]: df = pyogrio.read_dataframe("temp.geojson")

In [39]: df
Out[39]:
                 date                 geometry
0 2014-08-26 10:01:23  POINT (1.00000 1.00000)
1 2014-08-26 10:01:23  POINT (1.00000 1.00000)
2 9999-12-31 00:00:00  POINT (1.00000 1.00000)

In [40]: df.dtypes
Out[40]:
date        datetime64[ms]
geometry          geometry
dtype: object

So maybe that is sufficient, and we can point people running into this to upgrading pandas.
It would be nice if we can avoid the error for older pandas versions, though. We could catch the out of bounds error specifically, and in that case convert the datetime64 columns to object dtype (datetime.datetime objects). But not sure that is worth the extra code.
Is there a use case for automatically converting such values to NaT? I suppose that would require the user to specify the exact value (because it could be valid dates), but at that point we can just leave that to user to clean-up after reading, I suppose.

I'd be fine with pointing to pandas 2.0.

Is there a use case for automatically converting such values to NaT?

I guess that in my case in #230 where the datetime is 9999-09-09 00:00:00 that would be better to cast to NaT but as you say, a user would need to specify which value represents NaT, so I guess it is fine as a post-processing step.

In the cases where I've seen this in the wild, I've had manually populated fields where the year was entered as 1018 instead of 2018, so if pandas 2.0 would now parse that at a lower resolution, then it can be inspected and fixed which would be good enough for me (the alternative was to read the raw numpy arrays, and then convert that to a dataframe).


Adopted NumPy's new random generator

Addressing #1886 . This PR adopts NumPy's random generator to enable reproducible results, as well as better performance in some applications. See #1886 for a detailed explanation of this issue. See also #946 for a previous discussion among users and developers.
In order to adopt this new feature, support for NumPy <1.17 is being dropped.
Theoretically, it is possible to cross-support both old and new NumPy, as follows:
def get_rng(seed=None):

    # new random generator (numpy >= 1.17)
    if hasattr(np.random, 'default_rng'):
        if seed is None or isinstance(seed, int):
            return np.random.default_rng(seed)
        if isinstance(seed, np.random.Generator):
            return seed
        raise ValueError('Invalid seed.')

    # legacy random generator (numpy < 1.17)
    else:
        if seed is None or isinstance(seed, int):
            return np.random.RandomState(seed)
        if isinstance(seed, np.random.RandomState):
            return seed
        raise ValueError('Invalid seed.')

However, this workaround has limitations:

Some methods are not cross-supported, such as integers (new) vs randint (old).

Old and new generators produce different results given the same seed. For example,

>>> rs = np.random.RandomState(42)
>>> rs.randint(100)
51
>>> rng = np.random.default_rng(42)
>>> rng.integers(100)
8

Therefore, I think keeping backward compatibility is disadvantagous.
Please complete the following checklist:

[x] I have read the guidelines in CONTRIBUTING.md.

[x] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[ ] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

@mortonjt I added a test to see if random integers generated using 10 different seeds on my machine are the same as in the GitHub CI. Btw, this PR is not related to #1888 .


Support for prefers-reduced-motion

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
The transition components Collapse, Fade, Grow, Slide, and Zoom (and other MUI components with animations) should have reduced motion in their animations, by either disabling the animations altogether or using a simple fade animation, when the prefers-reduced-motion: reduce media query is true.
Current Behavior 😯
None of the components change behaviour and show the same animation when the media query is true.
Examples 🌈
Here’s the MDN doc for this media query. It’s supported on Chrome, Firefox, and Safari on Windows, Mac, and iOS.
This WebKit blog has several examples of how to support this media query.
Context 🔦
I’m currently adding support for this media query in my web app by disabling page transitions and other custom transitions by using the media query in JSS. But I’m also using MUI’s built-in transition components and there is currently no easy way to disable those animations.
I could use the useMediaQuery hook to modify what my component returns to remove the transition components, but this is quite cumbersome. Supporting this media query directly would make it a lot easier to build more accessible web apps.

Would the following be an acceptable solution?
import React from 'react';
import useMediaQuery from '@material-ui/core/useMediaQuery';
import { ThemeProvider } from '@material-ui/core/styles';

function App() {
  const prefersReducedMotion = useMediaQuery('(prefers-reduced-motion: reduce)');

  const theme = React.useMemo(
    () =>
      createMuiTheme({
        ...(prefersReducedMotion
          ? {
              // So we have `transition: none;` everywhere
              transitions: { create: () => 'none' },
            }
          : {}),
      }),
    [prefersReducedMotion],
  );

  return (
    <ThemeProvider theme={theme}>
      <Routes />
    </ThemeProvider>
  );
}

@oliviertassinari I'm guessing that doesn't affect the transition components? Fade, Grow, etc.

@joshwooding It does disable these components too.

An alternative with global CSS:
@media (prefers-reduced-motion: reduce) {
  * {
    animation-play-state: paused !important;
    transition: none !important;
    scroll-behavior: auto !important;
  }
}

I'm wondering 🤔 I have seen it in https://hankchizljaw.com/wrote/a-modern-css-reset/ (BTW, there are potential good resets we can deploy in v5).

Hey @oliviertassinari, just to confirm: would your solution be something I’d have to add to each of my React projects using MUI?

@notseenee Regarding your question. Should we have built-in support for the preferred reduced motion preference? Yes, I think that we should explore this path, it sounds better.

@oliviertassinari Yep, I think it’s in the interest of users and developers to have built-in support for prefers-reduced-motion as opposed to making devs implement it themselves

Suggested here (https://css-tricks.com/revisiting-prefers-reduced-motion-the-reduced-motion-media-query/) to include something along the lines of:
@media screen and
  (prefers-reduced-motion: reduce),
  (update: slow) {
  * {
    animation-duration: 0.001ms !important;
    animation-iteration-count: 1 !important;
    transition-duration: 0.001ms !important;
  }
}

in code.
Easy to include at the root of your app.
makeStyles(theme => ({
  root: {
    "@media (prefers-reduced-motion: reduce)": {
      "& *": {
        animationDuration: "0.001ms !important",
        animationIterationCount: "1 !important",
        transitionDuration: "0.001ms !important"
      }
    }
  }
))

Wanted to comment here to resurface this issue.
Also +1 for built in support! :)

Commenting here to resurface this issue.
Also, +1 for built in support :)

Can we please have this built-in?
Being someone who suffers from motion sensitivity, it imeediately stood out to me. And since MUI is almost screaming off the rooftops its support for accessibility features, you can imagine my dampened enthousiasm about a11y when I saw the first transition ever so smoothly giving me my next headache.
I sure hope MUI isn't giving us motion sensitive people an unearned finger, but it looks like that, frankly, seeing how this issue is from 2019. It doesn't take four years to get a fairly simple a11y into your framework, does it.

Would this apply to the Skeleton component as well? I don't want the loading animation to run when reduced-motion is enabled.

@IDrumsey Yeah, this would make sense to me. Maybe to make the animation run more slowly rather that entierely disabled?

@oliviertassinari I was thinking more along the lines of completely disabling the animation in case people have motion sensitivity. I'm currently just using a hook right now to check if reduced-motion is enabled and the manually setting the animation= based on that. Would be good if this was just built into the component.

@IDrumsey Completely disabling animations is the correct way to respond to this setting. I too don't understand why W3C calls it "reduce", because there's no way of know by how much to reduce. Depends on the user's level of sensitivity. For me persoanlly, only "big" animations are problematic, so a small loader spinner is fine. But a screen-sized menu sliding into the page, or a full page dim for a dialog, those are not okay. There are, however, many people much more sensitive than me, who can't even handle a small spinning loader icon.
HOWEVER, please bare in ind that by removing animations, you potentially introduce another problem: flickering. So a fading animation removed turns into a flicker. That's great for me, but now you've created a problem for people sensitive to flashing lights/images (epilepsy). This can happen (at least by default) for animationless dialogs that have a dimmed backdrop, for example.

Completely disabling animations is the correct way to respond to this setting.

@thany macOS behaves differently, see my recording:
https://github.com/mui/material-ui/assets/3165635/b3f60b09-9eed-4067-9535-99274dd48f76

Switch animation is retained
the window expander animation is replaced with an opacity animation

@oliviertassinari It depends on the form of sensitivity one is suffering from. I for one, cannot handle motion, but crossfades are less of a problem. Other people might still have a problem with crossfades.
The safest course of action is to disable all forms of animation, because we've only got the one switch to flip. We haven't got separate switches for motion, flashing lights, and crossfades, have we? 🙂


ASP.NET vNext -> ASP.NET 5

/cc @Eilon

Hi @hishamco, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Is it somehow possible to fix sharp, wall-like draw distance cutoff due to less dense distant fog?

Hello, as you probably know, this is one of the most jarring issues of the  mobile version – this sharp, wall-like draw distance cutoff due to less dense distant fog (it's even more noticeable during the foggy weather since it greatly decreases draw distance), where as original PS2/PC versions masked that draw distance boundary much more effective with thicker fog
Would be really great if it would be possible to somehow fix this issue and bring original behavior back through JPatch in the future
Unfortunately, as workaround, when trying to increase draw distance/streaming distance a bit through libdrawdistance.so, higher values often cause flickering in a game world, especially when looking down and when falling from some high place (even at the values like 2.00)
Thank you for your work and good luck
Example screenshots:

Even more jarring example during the foggy weather without the libdrawdistance.so:
screen-20230421.mp4.zip

As i said, everything is possible in this world. I will try few fixes i have in my mind but not now since it is not that boring. Atleast for me

Version v<IP_ADDRESS> fixes this. Closing as completed.


theme variable values with numbers and operands can't be interpreted as strings

With a theme like this:
vars:
    date: "01-04-2016"
footer:
    right: $vars_date

I get -2019 instead of 01-04-2016

This is where our custom expression language is bleeding through, which itself is an attempt to cover the limitations of YAML. The date is being interpreted as a math expression (1 - 4 - 2016).
The quickest way to prevent dates from being confused with a math expression is to require a space on either side of the operator. The theming guide already states this as a requirement, so we'd simply being enforcing it. I'll send a PR with that change.
However, there's still a chance that other patterns inside content get mixed up. Therefore, another change we could make is to not run any key with a content segment through the evaluate_math function. That seems to make sense. I can't see why you'd want content to be a math expression.
The problem that remains are variable values. The value of an arbitrary variable is still going to get run through the evaluate_math function (since we don't know what it's used for). We'd need to come up with a way to tell the theme loader to treat a value as a string literal (no math). We could use the convention of spreadsheets and prefix the value with a single quote.

I implemented both the enforcement of whitespace on either side of a math operator and skipped evaluating math in content-related keys. I have not yet implemented a way to indicate that a value should be treated as a string literal (variable expansion only). Let me know if you think we should proceed with that idea.

Thanks! For me with these changes it's good enough.

Super! I'll file a separate issue to discuss the idea of having a string literal (variable expansion only) for the value.
Thanks for testing!


Make fields within mutex volatile, just in case it matters.

Make fields within mutex structure volatile. This should prevent caching in register or stack. SDK currently manages barriers and has no cache to worry about. This is not likely required under most cases. but for completeness.

If the mutex is cached on two threads on the same core undefined behavior may occur. I will just sort this out on my side.

Re-opening as i was unaware that GCC would cache memory values across a compiler memory barrier, which is surprising

Yup; things work as expected...
The compiler will not cache a variable across a compiler memory barrier, so variables are reloaded after entering a spin lock whether volatile or not, and equally a context switch for the correct core cannot occur while IRQs are disabled (as they are during the spin locks used here)
... so i'm not sure what issue you are worried about

not sure what you mean by "destroy the SDK function"... if you mean inline it (which would only happen with LTO) it still doesn't matter, as the compiler memory barriers are in place (in the spin lock functions) to make sure things are reloaded where necessary

My concern is for decomposing to register in two places. The stacks are not shared address space and the compiler does not know to commit into shared address space.

Due to #1453, you may want to reconsider this.


NAT traversal info

Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>

I will continue with changes on my local editor

hopefully ready to review again.

I feel we are closer :rocket:

lift off!! :rocket:


Performance improvement when clearing *Map classes

When clearing a Map instance the keyTable array is traversed in reverse order when setting the array elements to EMPTY, thus clearing the array.
When iterating in 'normal' order the JVM seems to be able to optimize the loop and can see great improvements.
Current code (excerpt from IntIntMap.clear()):
for (int i = capacity + stashSize; i-- > 0;)
	keyTable[i] = EMPTY;

Changing it to
for (int i = 0, n = capacity + stashSize; i < n; i++)
	keyTable[i] = EMPTY;

or even using Arrays.fill(keyTable, 0, capacity + stashSize, EMPTY); brings noticable performance improvements.
Stumbled upon it when doing tests with ModelCache and changing the IntIntMap as mentioned brought an improvement from ~200 to ~250 fps (just for reference, ignoring what and how it was being tested) clearly showing this change can be useful.
Other Map classes should be changed in the same way.
Can contribute with a push request, just can't do it sometime soon, but leaving this suggestion in case someone has any interest in doing it first.

It was fixed!


Foreign Currency bug in AR Payment Class

When payments are applied to an invoice that was committed with a non-default currency, the invoice amount and base currency invoice amount are the same leading to overrepresentation in Cash Applied.

bug was introduced in refactoring from version 5.x to 6.

Patch 6.1.3 has been uploaded.


Opening the SquidNet2.py script just opens the logo and then closes/crashes.

I am running Python 3.11 and using Windows 10 Pro on a VM.

Hey! It's been quite a while since I've been on Github. Would you please be a bit more specific about the problem? Did you provide the arguments specifying the host IP and port? Or does it just completely crash without displaying a message?

It just crashes without displaying a msg. It flashes a windows command prompt and then just crashes. Cant seem to find it in task manager either.

What specifically are you entering for your arguments?
In case you're wondering what I mean, it's the stuff you enter in terminal.
An example:
python3 squidnet2.py --ip localhost --p 8080

Im not really good at explaining, could i add you on discord and screenshare there?

Sorry I'm not available. Are there screenshots you can provide?


BUGFIX: engine initialization prevented overrides in Solidus v4.4

While testing with solidus v4.4, I was getting an exception trying to override configuration:
uninitialized constant Spree::DigitalConfiguration (NameError)

  Spree::DigitalConfiguration[:authorized_clicks] = nil
       ^^^^^^^^^^^^^^^^^^^^^^
Did you mean?  Spree::SpreeDigitalConfiguration

There were changes made in PR #26 that I believe broke this.  While the engine "ran".....it couldn't be "configured"
This PR is a hybrid between what was there before that PR and what is required to work in v4.4

@kennyadsl, this PR is now green and ready to go. Please take a look when you have a chance.


chore(build): Remove deprecated build tags

Description
Remove deprecated build tags
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI

Looks like !ignore_autogenerated is not handled properly

/retest


FIx issues with ConfigMap for Trust Bundles

There are two issues with the Trust Bundles configMap today:

[ ] Today the configMap name is hardcoded, making multiple installations of ambient to clash due the configMap name.
Use the Helm Release name to customize the name of the configmap. Here: https://github.com/dapr-sandbox/dapr-ambient/blob/main/chart/dapr-ambient/templates/_daemondeployshared.yaml#L23 . Use the args option to set the release name. Inside the init-container read the argument and use for the configmap name
[ ] The configMap is not deleted if the helm chart is deleted

@mcruzdev as part of this issue you can rename these things:

the init-container name should be ambient-init-container instead of setup-trust-bundles
the name of the property: ambient.channelAddress should be ambient.remoteURL
the name of the property: ambient.daprd.app.port should be ambient.remotePort

Make sure these changes are part of the tutorial and main readme

Hi @salaboy,
We need to decide what we will do...
To use a helm hook, we need to call the ambient-init cmd to delete the configMap.
I suggest this approach:
on initContainers: ambient-init run --config-map=my-config-map
on hook delet: ambient-init delete --config-map=my-config-map

Let’s try that approach yes
On Sat, 3 Jun 2023 at 01:11, Matheus Cruz @.***> wrote:

Hi @salaboy https://github.com/salaboy,
We need to decide what we will do...
To use a helm hook, we need to call the ambient-init cmd to delete the
configMap.
I suggest this approach:
on initContainers: ambient-init run --config-map=my-config-map
on hook delet: ambient-init delete --config-map=my-config-map
—
Reply to this email directly, view it on GitHub
https://github.com/dapr-sandbox/dapr-ambient/issues/19#issuecomment-1574462561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACCMXSORUMQHVVK5TOTTRDXJJ6M5ANCNFSM6AAAAAAYYFEWSY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--

Blog: http://salaboy.com http://salaboy.wordpress.com

Github user: http://github.com/salaboy

Twitter: http://twitter.com/salaboy

Mauricio "Salaboy" Salatino -

Just waiting on some small configMap changes.. but the PR is almost ready to merge

Hi @salaboy, I think that we can close this issue!


Issue using library on text containing colon (:)

Encrypting a string already containing a colon : causes an issue when decrypting it.
It always throws Maize\Encryptable\Exceptions\UnserializationException
<?php
use Maize\Encryptable\Encryption;

$e = Encryption::php()->encrypt("test:string");
$p = Encryption::php()->decrypt($e);

+1 Same problem

Dear all, thank you for reporting the issue.
I just merged a PR that fixes it, will make a release right now!


Build a Literature Survey for Transfer Learning + Popular Pre-Trained Models for Binary Classification

Type
Advancement
Description
Build a Literature Survey for Transfer Learning including some Popular Pre-Trained Models for Binary Classification. Add the following there:

What is Transfer Learning?
Use-Cases of Transfer Learning to solve Complex Problems
Popular Pre-Trained Models for Transfer Learning
Comparative Case-Study of various Models

Tools

Markdown
Python

Have you read the Contributing Guidelines on Pull Requests?
Yes

Should I work on this @HarshCasper

Working as a mentor for the project in HakinCodes, can i make some additions here? @HarshCasper

Should I work on this @HarshCasper

@ashishsahu1 are yo working on any other issue currently ?

Not working on any of the issue, just landing page UI is yet to be reviewed @BALaka-18

Working as a mentor for the project in HakinCodes, can i make some additions here? @HarshCasper

Let's shift this conversation on Slack for better accessibility

Not working on any of the issue, just landing page UI is yet to be reviewed @BALaka-18

Assigned to you

Are you working on this @ashishsahu1 ?
It is way MUCH time now for this Issue.

I would like to work on this issue!

I would like to work on this

Assigned this to you @amoghatsunil

I am willing to work on this issue.

I want to work on this..

I am assigning to you @v-sonawane

Mentors do let me know if any changes required :-)

Mentors could you review this PR please.


Conflict with requireJs

The requireJs global variable "require" is overwritten, my website is not working.

Add config for preview..which does the jquery injection...so switch if off and you should have the problem.but if you needed preview the you need allow some code. check the latest commit.

I'm sorry I don't understand what you are talking about

Go to the settings  for the browsersplus package  and switch off preview

it's not working either
Uncaught TypeError: undefined is not a function  require.config()

is there a website..that I can look at to replicate the issue..

grouping all the module issues..https://github.com/skandasoft/browser-plus/issues/18


Autodetect language

Would it be possible to use deepl's auto-detect language api for translation?

We are now using a library of Golang to automatically judge the language. As far as I know, DeepL does not provide an open language detection API.

Thanks for the quick answer. How is that library called?

"github.com/abadojack/whatlanggo"


Provide test files for conformance testing

It would be much easier to write an importer if there would be some test files available with different important scenarios (like transformations or reusing a mesh with different colors).

Thank you for that comment, I uploaded right now few test files from different sources here: https://github.com/paireks/dotbim/tree/master/test/ExampleFiles
Each test file has it's own .png to check visually if it's ok. All of them are quite simple, but checks various things. I think later it would be great to add some big one e.g. for performance check.

I've tried this one: https://github.com/paireks/dotbim/blob/master/test/ExampleFiles/TestFilesFromC%23/Cubes.bim
The png looks like this:

But in the official viewer it looks really strange:

I think that the rotation value is not correct in that file.

Yeah, you're right, rotation values are probably wrong, I will change this file

I've changed this example already. For rotation checks you can use this example: https://github.com/paireks/dotbim/blob/master/test/ExampleFiles/TestFilesFromGh/BricksRotated.bim


Update data.js

Added FrenchPress MVC/Template engine.

https://github.com/DylanPiercey/FrenchPress gives me a 404 now

If the repo link gives a 404, then I vote to close this PR.

Sorry, the project has been re-branded and isn't re-released yet. Thanks

No worries, I look forward to the future PR after that process is finished =)


Change the "edit label action" to rename all spots of a branch at once (Branch&Hierarchy TrackScheme)

Mastodon users usually would like to assign the same name to all the spots in a (BranchGraph) branch.
Both the Branch TrackScheme and Hierarchy TrackScheme could be very useful in this context.
The normal TrackScheme has a EditFocusVertexLabelAction for renaming individual spots.
This PR extend this class as a new BranchTrackSchemeEditLabelAction.
This BranchTrackSchemeEditLabelAction is used in the Branch- and Hierarchy TrackScheme to rename all the spots that belong to a branch at once.

Hello @maarzt
I made some small changes. Could you check them?


Add JScience library definition

I hope this is ok.
I didn't ask the author to include the definition directly in his library because it's not an android specific one.

Thank you so much!


Failed to initialize CEncoder

Failed to initialize CEncoder. All VideoEncoder are not available. VCE: AMF Error 1. g_AMFFactory.Init(), NVENC: NvEnc CreateEncoder failed. Code=22 NvEncoder::RegisterResources : m_nvenc.nvEncRegisterResource(m_hEncoder, &registerResource) returned error 22 at c:\src\alvr\alvr_server\nvencoder.cpp:542
I don't know why. It display these.
I use GTX 1060 and 5G router connect to Quest.
Thank you!

Update to the latest nvidia driver. If you have a laptop with a nvidia and an intel gpu, you need to disable the intel gpu...

I had been using ALVR successfully for a couple of weeks after seeing this and updating to the latest NVIDIA drivers (at this time 436.30 running on a GTX 1080).  After using ALVR all day yesterday, I open it up today and it starts throwing the following error when launching SteamVR.
Failed to initialize CEncoder. All VideoEncoder are not available. VCE: AMF Error 1. g_AMFFactory.Init(), NVENC: NvEnc NvEncoderD3D11 failed. Code=10 NvEncoder::NvEncoder : m_nvenc.nvEncOpenEncodeSessionEx(&encodeSessionExParams, &hEncoder) returned error 10 at c:\src\alvr\alvr_server\nvencoder.cpp:51
I don't know of anything that changed overnight while the laptop was sleeping.

After a PC restart, uninstalling/installing SteamVR, getting the issue with SteamVR constantly starting in Safe Mode (without ALVR running), and trying to "Create a System Report" from SteamVR, SteamVR started working outside of Safe Mode again.

If you are using a mobile chip (like i do), say 960M, then make sure you are using drivers of version 417.71 (or perhaps higher, but this version is 100% ok). You don't have to disable integrated Intel graphics.

One more note: as you know, GTX 960M does not support H.265 (HEVC), therefore, an attempt to start ALVR with this option selected in the "Video" tab will cause the application to stop with an error. Make sure you've selected H.264 and restarted the application before starting SteamVR.

yo im a brainlet when it comes to computers and stuff, just like... what do I DO to fix this issue lol


Bump version 4.0.0-beta

From the following propose.
https://github.com/el-hoshino/NotAutoLayout/pull/143#pullrequestreview-735064242

          3 Warnings

:warning:
This PR contains release modification. Please remember to check if the version in Info.plist has been updated.

:warning:
This PR contains release modification. Please remember to check if CHANGELOG.md has been updated.

:warning:
Sources/NotAutoLayout/NotAutoLayoutCompatible/NotAutoLayoutCompatible.swift#L11 - Sources/NotAutoLayout/NotAutoLayoutCompatible/NotAutoLayoutCompatible.swift#L11: using 'class' keyword to define a class-constrained protocol is deprecated; use 'AnyObject' instead
public protocol NotAutoLayoutCompatible: class {

          2 Messages

:book:
Executed 26 tests, with 0 failures (0 unexpected) in 7.039 (7.099) seconds

:book:
What a good PR :white_flower:

Develop PR Check Result

Item
Result

Head Branch check
:o:

Base Branch check
:o:

Info.plist modification check
:heavy_exclamation_mark:

CHANGELOG.md modification check
:heavy_exclamation_mark:

Merge Commits check
:o:

Modification Volumn check
:o:

  Generated by :no_entry_sign: Danger


swagger-parser 1.0.57 adding responsesObject to the definition

We recently updated the swagger-parser from 1.0.51 to 1.0.57. After this upgrade, when we add this[1] swagger to our application, the parsed definition has an additional responsesObject field. The swagger editor shows this as a Structural error

The output of [1] after being parsed gives the following.
/pets/{petId}:
    get:
      summary: Info for a specific pet
      operationId: showPetById
      tags:
        - pets
      parameters:
        - name: petId
          in: path
          required: true
          description: The id of the pet to retrieve
          type: string
      responses:
        "200":
          description: Expected response to a valid request
          schema:
            $ref: '#/definitions/Pets'
        default:
          description: unexpected error
          schema:
            $ref: '#/definitions/Error'
      responsesObject:
        "200":
          description: Expected response to a valid request
          schema:
            $ref: '#/definitions/Pets'
        default:
          description: unexpected error
          schema:
            $ref: '#/definitions/Error'

We are doing the following in our code
SwaggerParser parser = new SwaggerParser();
SwaggerDeserializationResult parseAttemptForV2 = parser.readWithInfo(oasDefinition);

Is there an option to disable this? Is this a bug in the parser?
[1] - https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/yaml/petstore.yaml

I'm also getting this issue. Any update on this?

I think This happens from swagger.models.version 1.6.3.

Hi, @RakhithaRR @chashikajw I have tested locally the definition provided and I haven't been able to reproduce the issue, note that I am using latest parser release version: 1.0.58 and latest core version: 1.6.5
Please try test this again with the current version.

I think we need to add this to the README then (cc @gracekarina )


CLOUDP-225724 Use Github Actions Job Summary instead of Gist

Update setup step to use setup/au_setup.ps1
Use Github Actions Job Summary instead of Gist

LGTM. I wasn't aware of this fork 😅 . Where can I see more info on why this fork is needed and how is used for? Thanks!

this is new, basically, we are moving chocolatey out of mongodb/mongodb-atlas-cli into here. After the next version is out and this gets kicked in I can give more context.


Vue Truthiness/Falsiness with v-bind:checked

Version
2.6.10
Reproduction link
https://jsfiddle.net/starkovsky/27h8wg5n/2/
Steps to reproduce

Run example in jsfiddle

What is expected?
<input type="checkbox" v-bind:checked="''" /> it should be true
What is actually happening?
<input type="checkbox" v-bind:checked="''" /> rendered false

<input type="checkbox" v-bind:disabled="''" /> rendered right, as true

This is just how the attribute checked works, try it by yourself in the console by setting the checked property of the element to an empty string, it will uncheck the checkbox

When used with v-bind, the only falsy values are null, undefined, and false. This means 0 and empty strings will be rendered as truthy. For example, v-bind:draggable="''" will become draggable="true".
From Migration guide https://vuejs.org/v2/guide/migration.html#Truthiness-Falsiness-with-v-bind-changed


Onboard Renovate to repo.

Note: I named it json5 instead of json because the former allows comments.  But I can rename to json if that's preferred.

Note: I named it json5 instead of json because the former allows comments. But I can rename to json if that's preferred.

The json5 is actually preferred. The location might be a problem if the repo is synced to gitlab at some point.
Would it make sense to get the Renovate team to agree on the format and location?

The json5 is actually preferred. The location might be a problem if the repo is synced to gitlab at some point.
Would it make sense to get the Renovate team to agree on the format and location?

I'm assuming "Renovate team" means upstream?
Docs: https://docs.renovatebot.com/config-overview/#repository-config
If the repos are sync'd I think we specifically don't want renovate operating on them twice.  We'd end up with PRs/MRs on the same topic in different systems.  Unless I'm misunderstanding the question.

The json5 is actually preferred. The location might be a problem if the repo is synced to gitlab at some point.
Would it make sense to get the Renovate team to agree on the format and location?

I'm assuming "Renovate team" means upstream?

No. I referred to our internal Renovate team

Docs: https://docs.renovatebot.com/config-overview/#repository-config
If the repos are sync'd I think we specifically don't want renovate operating on them twice. We'd end up with PRs/MRs on the same topic in different systems. Unless I'm misunderstanding the question.

That is a valid argument. Let me try with a different use case. What if the repo migrated to gitlab?

Please rebase

force-push: rebased

LGTM

That is a valid argument. Let me try with a different use case. What if the repo migrated to gitlab?

Just to follow up on this.  I think we're still okay in case of migration.  It seems to me reasonable to expect some items like adding a .gitlab-ci.yml and reviewing repo settings is "normal" for a migration.
In fact I could be wrong about renovate + gitlab ignoring .github/ files.  Searching there IIUC, is part of the bot's logic and it may not care what's actually hosting the repo.

That is a valid argument. Let me try with a different use case. What if the repo migrated to gitlab?

Just to follow up on this. I think we're still okay in case of migration. It seems to me reasonable to expect some items like adding a .gitlab-ci.yml and reviewing repo settings is "normal" for a migration.
In fact I could be wrong about renovate + gitlab ignoring .github/ files. Searching there IIUC, is part of the bot's logic and it may not care what's actually hosting the repo.

I don't want to drill into it. We can always wait until such a use case is actually happen and then take action.
Just a food for thought. What is the advantage of having the renovate config file "hidden" inside the .github folder?

What is the advantage of having the renovate config file "hidden" inside the .github folder?

Purely organizational, as it serves no direct function in the project, i.e. I see it as a kind of meta-config.  It also keeps the repo-root ("Prime real-estate") clutter-free.  But I don't have very a strong opinion, if it needs to live at the root for consistency with other projects, I'm fine with that.


Multimeter displaying NaNs after running overnight

The screenshot tells it all:

Thanks for creating an issue, but maybe it doesn't quite tell all ;)  I have a few questions:

Are the remaining fields still updating, or are they "frozen" at the values shown?
What is your OS and version (looks like Mac OS)?
Do you know if your computer went into sleep mode overnight?

Would you mind attaching the Joulescope file log for this run (the filename is UTC time for the application start)?  You can find it under:
~/.joulescope/log/

Thanks!

Are the remaining fields still updating, or are they "frozen" at the values shown?

They are still updating, including plots.

What is your OS and version (looks like Mac OS)?

macOS 10.14.5

Do you know if your computer went into sleep mode overnight?

shouldn't have, but I can't guarantee it.
Logs
joulescope_20190626_180736_3378.log
joulescope_20190627_150054_1471.log
joulescope_20190627_183238_2584.log

Thanks @JayKickliter!  The last log file joulescope_20190627_183238_2584.log is the correct one for the overnight run.  I am not sure what happened as the log does not show anything abnormal other than a few missing samples at 11:58, 12:25 and 8:16.   I will try to duplicate this.

Thanks for looking into it. It's barely an inconvenience for me, but other customers might rely on long runs.
As an aside, everyone in my office is impressed with the Joulescope after just one day, and we'll probably be ordering more.

Great to hear that you are finding Joulescope useful!
The NaNs on the multimeter view should not happen, and I will definitely investigate.

Hi, I can confirm this issue

All I did was let it run from around 12 hours over night.

Hi @heroichornet and thank you for the posting your confirmation of this issue.  Could you check what version you are running (click Help → About)?  I fixed two related defects (here and here) that were the likely contributors to this issue.  Release 0.4.5 contains these fixes, and I ran 0.4.5 for 2.5 days on Win 10, Ubuntu 18.04 and macOS 10.14.5.  I no longer saw this issue, so I think it is fixed.  However, I have not yet created a good regression test, and I did not try Win 7.  If you are not running 0.4.5, could you upgrade to 0.4.5 and see if that fixes the issue?
Under Win 7, the Joulescope log files are located in:
{your_appdata_path}\Local\joulescope\log

For me with username mliberty, the full path is:
C:\Users\mliberty\AppData\Local\joulescope\log

@mliberty1 Thank you for the quick reply
I am running 0.4.5

@heroichornet If you can post the log for the NaN run, I would appreciate it.  The logs are named based upon the UTC timestamp.  I will also try to duplicated using my Win 7 VM.

joulescope_20190708_081558_8556.log

I just tried with the small test script in the "Developer" section that calculates the mean of 0.25 seconds of measurements and the output was also similar with "nan" average
$ python test.py
[[0.01208843 3.3185446 ]
[0.01210787 3.31968   ]
[0.0121348  3.31968   ]
...
[0.00943616 3.31968   ]
[0.00942718 3.31968   ]
[0.00940175 3.3174095 ]]
nan A, nan V
(joulescope)
Maybe that helps

@heroichornet Thanks for taking a closer look.  The example code you found in usb_inrush.py is essentially doing a nanmean by filtering to include only finite (no NaN, +INF, -INF):
valid = np.isfinite(i_mean)
voltage = np.mean(v_mean[valid])

I should take another look at that usb_inrush.py code as I doubt the USBET tool handles NaNs nicely.
NaNs are injected into the Joulescope sample data whenever samples are missed.  This happens frequently at the start of data streaming.  Dropped samples should happen very infrequently once streaming is ongoing.  However, they do happen regardless of USB transfer mode.  In USB BULK mode (which the Joulescope software uses today), data integrity is guaranteed, but throughput is not guaranteed.  The host should be able to keep up.  I could switch to using USB ISOCHRONOUS mode, which guarantees throughput but not data integrity, so we could still get some corrupted USB frames = missing samples.  I now have a setup under a Win 7 VirtualBox VM that drops data all the time (the USB VM adapter just doesn't seem to keep up), so I can better observe this case.  I'll be doing more testing today & tomorrow.
The software is designed to handle missing samples as best as possible, but I found & fixed another issue that could be contributing (pyjoulescope bc50129).
If you just want a capture containing contiguous samples, the driver has a contiguous_duration option.  Also see the "--contiguous" option for the command-line capture command:
python -m capture --help

I am adding an indicator to the UI that will more clearly show if your system is dropping samples.  Right now, you have to look at the log or open the developer View.  You can look at continguous_count which is the number of contiguous samples in the current block.  You can also compute the dropped fraction:
dropped_fraction = sample_missing_count / sample_id

The 0.5.0 release should fix this issue.  Could you try upgrading and let me know if you still observe this issue?  Thank you!

I will upgrade and report back. I actually have not seen the issue since upgrading to 0.4.6

I let 0.5.0 run over night and was unable to reproduce this issue. I think you can close this issue.

I will close the issue since we audited the code and NaN contributors.  Although we think we got them all, it's hard to be 100% confident with NaNs.  If you do happen to see NaNs again, please reopen this issue or start a new issue.  Thank you @heroichornet and @JayKickliter.

Hi @pawelm87 - thank you for reporting this issue.  What likely happens is that your mac goes off and does something that causes Joulescope sample drops.  The UI should handle this better than displaying NaNs.  It looks like you have pressed the "Accumulate" button.  I inspected the code, and I found that this accumulate function does not handle NaNs well.  We will fix this.
If you want to confirm that your Mac dropped Joulescope samples, you can either inspect the log yourself or post it here, and I'll take a look.  On Mac, you should find Joulescope logs under ~/.joulescope/log, and the file is the starting time in UTC.

joulescope_20190923_113000_94106.log
joulescope_20190924_061401_57851.log
joulescope_20190923_112946_94099.log

Hi @pawelm87 and thank you posting the log files.  The overnight log files are:
joulescope_20190923_113000_94106.log
joulescope_20190924_061401_57851.log
Both show that your computer went through a few time periods where it did not keep up with the
USB data stream.  All of those log entries:
joulescope.stream_buffer:Fill missing samples
indicate that your mac is no longer keeping up with the Joulescope USB data rates.  These sample drops means that the Joulescope software never receives this data resulting in inaccurate values being displayed.  I suspect that your computer is entering a low power mode, but I'm not exactly sure.
Here are some things to try, if you haven't already:

Ensure that your mac is powered from the wall (not just battery)
Select System Preferences → Energy Saver → Power Adapter.
a. Check "Prevent computer from sleeping automatically when the display is off"
b. Uncheck "Enable Power Nap while plugged into power adapter"
c. Click **Schedule..." and ensure that no schedules are set.

The upcoming 0.6.4 software release will fix the NaN display, but samples will still be dropped on your computer without any changes.

So, if the Joulescope miss samples that means the power consumption which is display is also inaccurate values
I will check setting in Mac

I uncheck  "Enable Power Nap"
No Schedule is set
When I left for the night, I used "screen lock" on the Mac

I changed all the settings on the Mac you wrote about, and again NaN. Mac wasn't powered from battery.
I need to test device for long time and meassure power consumption

Hi @pawelm87 - How long do you plan on performing testing?  For very long-term testing, I recommend using the downsample logging script.  For very long-term testing, you should also disable network and all other software to minimize the chance of things going wrong.
I also thought of something things to check:

Please uncheck "Put hard disks to sleep when possible".
For highest reliability, please ensure that your Joulescope is plugged directly into your computer's USB port (no hubs or docks).  Note that many USB-C adapters are hubs, so you just want to use a USB C to USB A cable.
How much total and free RAM does your system have?  Open finder → Applications → Utilities → Activity Monitor.  Click Memory.  You should have at least 1 GB free for the Joulescope UI.  I measure that it uses 786.5 MB on my 2018 13" MacBook Pro with macOS Mojave 10.14.6.  Also check on system CPU utilization.  I noticed that 0.6.3 does use quite a bit of CPU (about 60% from my) on mac, even when just in multimeter view.  I will investigate.

Hi @mliberty1  I tryed run this script but I have got error :
Installing collected packages: pymonocypher, pywin32, pypiwin32, joulescope, pyparsing, kiwisolver, cycler, matplotlib   Running setup.py install for pymonocypher ... error     ERROR: Command errored out with exit status 1:      command: 'c:\users\p\appdata\local\programs\python\python37-32\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\PAWEL~1.MAR\\AppData\\Local\\Temp\\pip-install-5fuvcsct\\pymonocypher\\setup.py'"'"'; __file__='"'"'C:\\Users\\PAWEL~1.MAR\\AppData\\Local\\Temp\\pip-install-5fuvcsct\\pymonocypher\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\P\AppData\Local\Temp\pip-record-2gkf8n3s\install-record.txt' --single-version-externally-managed --compile          cwd: C:\Users\P\AppData\Local\Temp\pip-install-5fuvcsct\pymonocypher\     Complete output (5 lines):     running install     running build     running build_ext     building 'monocypher' extension     error: Microsoft Visual C++ 14.0 is required. Get it with "Microsoft Visual C++ Build Tools": https://visualstudio.microsoft.com/downloads/     ---------------------------------------- ERROR: Command errored out with exit status 1: 'c:\users\p\appdata\local\programs\python\python37-32\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\P\\AppData\\Local\\Temp\\pip-install-5fuvcsct\\pymonocypher\\setup.py'"'"'; __file__='"'"'C:\\Users\\P\\AppData\\Local\\Temp\\pip-install-5fuvcsct\\pymonocypher\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\P\AppData\Local\Temp\pip-record-2gkf8n3s\install-record.txt' --single-version-externally-managed --compile Check the logs for full command output.
I have installed Microsoft Visual C++ Build Tools

Ok I install all dependencies and run this script:
PS C:\Users\p\Desktop\workspace\Joulescope\pyjoulescope_examples\bin> py.exe .\downsample_logging.py
Starting logging - press CTRL-C to stop
Filename: C:\Users\p\Documents\joulescope\jslog_20191002_124932_20452.csv
2019-10-02T14:49:32 WARNING  No Joulescope connected
2019-10-02T14:49:43 WARNING  No Joulescope connected
PS C:\Users\p\Desktop\workspace\Joulescope\pyjoulescope_examples\bin>

Joulescope is connect and all app which will be use it is closed

Hi @pawelm87 - Can you use 64-bit Python?  We do not support 32-bit python.  You can follow the Quick Start Guide.  Once the command
python3 -m joulescope capture --contiguous 1.0 mycapture.jls

works, then you can try downsample_logging.

ok work now i will test it

Following up on this issue, I ran for nearly Joulescope UI 0.6.3 in multimeter view for 3 days on my 2018 13" MacBook Pro with all power management settings disabled.  No NaNs.  With the upcoming Joulescope UI 0.6.4 (available today or tomorrow), you will no longer see NaNs, but you may see sample drops.  Sample drops are currently only visible in the Developer view.

I am going to close this issue, but feel free to reopen if you still see NaNs in 0.6.4.


Is it possible to set document ID using pdf-lib?

Hi,
Is it possible to set unique document id using pdf-lib? Document ID should show up like this in the PDF file
trailer
<<
/Root 1 0 R
/Info 3 0 R
/ID [<B6014D890345884B0EDF8D5AA0C95B37> <B6014D890345884B0EDF8D5AA0C95B37>]
/Size 47
>>

Where https://www.oreilly.com/library/view/pdf-explained/9781449321581/ch04.html

ID | Array of two Strings | Uniquely identifies the file within a work flow. The first string is decided when the file is first created, the second modified by workflow systems when they modify the file.

Here’s an example trailer dictionary:

<<
/Size 421
/Root 377 0 R
/Info 375 0 R
/ID [<75ff22189ceac848dfa2afec93deee03> <057928614d9711db835e000d937095a2>]

I have tried following by looking how pdf-lib is setting author etc. inside trailer Info section.
const ID = PDFName.of('ID')
const existingID = pdfDocument.context.lookup(
  pdfDocument.context.trailerInfo.ID
)
if (existingID instanceof PDFDict) {
  existingID.set(ID, PDFString.of('1234567890'))
} else {
  const newID = pdfDocument.context.obj({})
  pdfDocument.context.trailerInfo.ID = pdfDocument.context.register(newID)
  newID.set(ID, PDFString.of('1234567890'))
}

If I use console.log to print "pdfDocument.context.trailerInfo.ID" it has been changed from "undefined" to "PDFRef" (or similar) but when document is saved into file, I cannot see ID to show up properly, like /ID [<1234567890>]
Br,
Markku

This should be possible to do. I don't have time to provide an example right now though. I'd suggest checking out the following classes:

https://github.com/Hopding/pdf-lib/blob/master/src/core/writers/PDFWriter.ts
https://github.com/Hopding/pdf-lib/blob/master/src/core/writers/PDFStreamWriter.ts

That's where the serialization is implemented. Serializing a trailer is slightly different from the main body of the document, so you'll want to review that code in particular to make sure the ID isn't somehow omitted.
Also, you'll want to save the document with pdfDoc.save({ useObjectStreams: false }) when you're testing. It's much easier to debug things that way 🙂.


[breaking] Sentient instead of sapient

Closes #3

Hurrah..! :tada:
I used to be quite competent at toki pona (I have Sonja's book, used to inhabit the channel on Freenode and almost helped start an educational project called Sonapona back in 2005). I was delighted to see the toki pona translation (and also saddened that my toki pona is so rusty - hence my not attempting any here). :-)
However... looking at the toki pona translation, I believe (and I'm just putting it "out there" for the sake of sharing and learning) that jan is still an accurate translation: being sentient is a fundamental aspect of person-hood (i.e. jan) and so the term still works.
Thoughts..?


Appearant divergence from the specification in samplerz.py

In the samplerz.py file, at line 111 in the function BerExp, we have:
    z = (approxexp(r, ccs) - 1) >> s
But according to the spec (https://falcon-sign.info/falcon.pdf page 43), it should be:
    z = (2 * approxexp(r, ccs) - 1) >> s
or
    z = ((approxexp(r, ccs) << 1) - 1) >> s
The corresponding line in the C implementation:
z = ((fpr_expm_p63(r, ccs) << 1) - 1) >> s;
Thank you.

Hello, this times 2 actually happens inside approxexp at line 97:
z = int(ccs * (1 << 63)) << 1
Sorry for the misunderstanding.


Set pod affinity for calico-node deployed using tigera operator

When installing the calico using tigera operator, the calico-node daemonset only have nodeSelector ( kubernetes.io/os=linux ), there is no option to add affinity, for example when we need to exclude fargate nodes when running on AWS EKS.
https://docs.projectcalico.org/reference/installation/api#operator.tigera.io/v1.InstallationSpec
Expected Behavior

There is option to add affinity when deploying calico using tigera-operator
Your Environment
Tigera-operator: v3.19

Hi @alfianabdi , if I'm understanding correctly, you're looking for a way to prevent the calico-node daemonset from running on specific nodes (fargate nodes in your use case)? We generally provide the controlPlaneNodeSelector option in the Installation which should allow you to select the proper nodes for the other pods except for the daemonsets (which sounds like it doesn't quite fit your use case.
I'm not super familiar with running clusters with fargate nodes but the calico-node daemonset generally needs to be running on each node to enforce policy so if it is not running on the fargate nodes, then you'll have to be careful about how you craft network policy to reach those pods. What happens if the calico-node pods attempt to run on a fargate node? Do they cause any issues or is there another reason you would like to keep them off of those nodes?

Hi @mgleung
It is impossible to run the calico-node in fargate, it is a special microvm node, where only one pods can run inside the node. Without the affinity, I will see many pending calico-node pods, as many as the number of fargate nodes.
https://docs.aws.amazon.com/eks/latest/userguide/calico.html
Without tigera-operator, I can set the affinity, but I would like if tigera-operator support it as well.

Looks like we already have built-in logic for a very similar scenario on AKS: https://github.com/tigera/operator/blob/caac83c79646a11ea94a0f2efff388f0b22bf989/pkg/render/node.go#L663-L678
We could automatically add this when running on EKS pretty easily I think.
It seems inevitable as well that we'll need to expose general controls for node affinity for calico/node, although we originally tried to avoid this because it introduces a potential avenue to break your cluster via misconfiguration. CC @tmjd

I also look for this feature but my use case is different. We use custom installation of calico and CNI plugins. calico-node runs as systemd service, CNI plugins are installed as debian package, and kubelet is directed to load them from non-default path.
We'd like to migrate this setup to tigera operator. We're looking for a way how to configure the new setup and than switch nodes one by one. I think that, when we'll be able to configure the daemonset's affinity to some node label, we could proceed in following steps (for each node):

drain the node
reconfigure kubelet and stop calico-node systemd service
relabel the node and let the deamonset's pod inside
uncordon the node

I believe having this feature would enable us to convert to tigera operator without cluster outage.

@mgleung I have raised a PR to resolve this in the operator by adding proper affinity - https://github.com/tigera/operator/pull/1725


add findKeys method for CouchDB

Hi,
I'v added the findKeys methods for being able to use the listAllPads methods of Etherpad.
Also the felix-couchdb was missing in the package.json.
And about felix-couchdb, it was also broken for me… so I made a pull request in order to make it work. Without this patch, this module will sadly throw errors :disappointed:

Afaik this should not be how it's done.

:S
I'm not a couchDB expert at all ¬_¬'
But I haven't seen any way to filter by a dynamic regexp in a design document…
I might look it further, but being in a hurry I can't do it right now.

Adding findAllPads to the module is wrong.  findAllPads is a method exclusive to Etherpad.
That logic needs to be in Etherpad and not ueberDB :)  Make sense?  Look at the other modules for ueberDB, you will see they don't contain findAllPads

You're right :+1:
Trying to make Etherpad working, I haven't seen this huge flaw.
I really have to figure out to put a regexp in the design doc…


CDC ChangedSince is unpredictable based on local time

When setting the ChangedSince time for the $dataService->CDC() call. The time sent to the server is expected in ISO 8601 format it would best be wise to send the correct timestamp.
On line 891 of DataService we should change it from $formattedChangedSince = date("Y-m-d\TH:m:s", $this->verifyChangedSince($changedSince)); to $formattedChangedSince = date("Y-m-d\TH:m:sP", $this->verifyChangedSince($changedSince)); or $formattedChangedSince = date("c", $this->verifyChangedSince($changedSince));

@challgren
I didn't realize it. Let me check and I will change it accordingly.

Fixed.

Ok then the docs need to clearly state send a PST timestamp. I messed around with this for almost 3 hours trying to get the timestamps figured out as one would assume a timestamp without a timezone is UTC time and not PST time. But this is not the case because UTC sent timestamps would fail.


RFE: please provide support for markdown-it-py 3.x

https://github.com/Textualize/rich/blob/b9c9eb57b4fc3b371298205f4ec51320ae804dd0/pyproject.toml#L33

Duplicate https://github.com/Textualize/rich/issues/2995


Support for Attachments

This functionality would be similar to the image support released in v3.4.0 but would allow users to upload any file type after which a download URL would be added to the editor.

Oh, I sign the petition for the implementation as well.

Plus one from me.
I have worked out a hacky work around though :)
PDF Link

I add the pdf as image then change the link to above and it open a new tab :)
[PDF LINK](attachments/test.pdf){:target="_blank"}

The above works if I right click and open in new tab. I include the the reference {} as it does not work. But some markdown parsers do support this

Plus one from me.
I have worked out a hacky work around though :)
PDF Link

I add the pdf as image then change the link to above and it open a new tab :)
[PDF LINK](attachments/test.pdf){:target="_blank"}

The above works if I right click and open in new tab. I include the the reference {} as it does not work. But some markdown parsers do support this

Yeah, I know. I went through code and the workflow is basically supporting any type of files already, with minor changes, and with renaming of "uploadImage()" to "uploadAttachment()". There till will be some questions like: setting up the maximum file size, adding some basic progress bar and some mechanism to delete attachment together with note(it's not that crucial with pics, but will bloat the hdd much faster with bigger files).
Right now I am thinking betwen choosing other app like Memos(which has cons as well), or proceed with Flatnotes and some kind of file system manager, where all files will be stored, like FileGator, etc.

Oops, that was me on that duplicate in #262! I'd done some Google and Perplexity homework, but I did not browse the open issues. Sorry.
So if anyone is in doubt, +1 for libre uploads!


Path must be a string or buffer type error

I get this error when I try to send my photo using a photo on my computer system.

The example seems to be wrong. It should be:
const photo = './myphoto.jpg';
const { media } = await client.uploadPhoto({ photo });


Allow splitting a config command into multiple lines

I created a very long filter macro in the config file like this:
macro F set-filter " content !~ \"keyword1\" and title !~ \"keyword1\" and content !~ \"keyword2\" and title !~ \"keyword2\" and ...........................and title !~ \"keywordN\" "
I would like to split it into multiple lines that I can easily edit and manage the list of filter.
But those are inside a pair of double quotes, I don't know how to break into new lines with \.
Any way to split the filter languages to multiple lines??? Thanks!

Hi, sorry for the delayed response!
This isn't possible yet, but seems like a good feature to have. I think the most common way to support this is to allow backslash at the end of the line, so one could write:
macro F set-filter " \
  content !~ \"keyword1\" \
  and title !~ \"keyword1\" \
  and content !~ \"keyword2\" \
  and title !~ \"keyword2\" \
  and ........................... \
  and title !~ \"keywordN\" "

Looks like this is fairly simple to implement. The relevant files are include/configparser.h, src/configparser.cpp, and test/configparser.cpp.

Thanks for your explanation!
I agree with your suggestion, that's also what I think of!


Add virtctl to tekton task

What this PR does / why we need it:
This PR adds functionality to create-vm-from-manifest task to allow user to create vm with virtctl command.
Release note:

Good start! I added some comments and one further question: Can the task return the name of the VM? This leaves it open to create a VM with a random name.

The task is just creating a VM and it does not return its name. It depends on a user if he specifies a name of a VM, if he does not for virtctl command, then the created VM will have a random name.

@opokornyy Tasks can have return values, so why not return the name/namespace of the created VM? Other tasks (e.g. creating DataVolumes) do the same.

@opokornyy Tasks can have return values, so why not return the name/namespace of the created VM? Other tasks (e.g. creating DataVolumes) do the same.

Yeah, I will take a look on that.
@ksimon1 What do you thing about adding name/namespace of the VM as a return value of this task?

@opokornyy Tasks can have return values, so why not return the name/namespace of the created VM? Other tasks (e.g. creating DataVolumes) do the same.

I have to correct myself here, the task already have a return values with name/namespace of the VM that was created.
https://github.com/kubevirt/kubevirt-tekton-tasks/blob/9cfaf988cde7cfc2a549d94414ea8834020224ba/templates/create-vm-from-manifest/manifests/create-vm.yaml#L85

/retest

/retest

/retest

/retest

/lgtm


Peft fsdp

In the refactor in #1467, we accidentally moved the prepare for fsdp call before the super init. This results in peft models not being wrapped correctly, because the peft model change happens in the super init. This PR moves the prepare for fsdp call after the super init (same behavior as before #1467), and adds a unit test for the correct behavior.
Manual test (this oomd before this PR due to incorrect wrapping): updated-image-6-9W2TKE

@mvpatel2000 yup, test fails before this PR.


Unable to get a shell

I'm trying to do some trouble shooting by getting a shell on the docker container.  I've tried everything I can think of, but it seems that /bin/bash or /bin/sh don't exists on the image.
sudo docker run -t -i --entrypoint /bin/bash wrouesnel/postgres_exporter
docker: Error response from daemon: Container command '/bin/bash' not found or does not exist..
Is there some other way I can get a shell, so that I can test if it's able to connect to the db?  The actual error that I'm trying to solve is this, which I'm assuming is a connectivity issue:
time="2016-05-19T21:21:31Z" level=info msg="Error running query on database:  pg_stat_database_conflicts missing "=" after "ubuntu:password@localhost:5432/db" in connection info string"" file="postgres_exporter.go" line=277
time="2016-05-19T21:21:31Z" level=info msg="Error running query on database:  pg_stat_bgwriter missing "=" after "ubuntu:password@localhost:5432/db" in connection info string"" file="postgres_exporter.go" line=277
time="2016-05-19T21:21:31Z" level=info msg="Error running query on database:  pg_stat_database missing "=" after "ubuntu:password@localhost:5432/db" in connection info string"" file="postgres_exporter.go" line=277

The docker container is just the compiled executable and nothing else. If you need a shell you'd have to derive an image which included gnu tools.
Can you show your actual docker command line?

Sure, the actual docker command I was using was something like this:
sudo docker run --net="host" -e DATA_SOURCE_NAME="ubuntu:password@localhost:5432/db" -p 9113:9113 -d wrouesnel/postgres_exporter

Oh wow, my README.md documentation is atrocious - so the top line I give for a connection string is totally wrong. The correct command would be
sudo docker run --net="host" -e DATA_SOURCE_NAME="postgresql://ubuntu:password@localhost:5432/db" -p 9113:9113 -d wrouesnel/postgres_exporter
You might also need to add ?sslmode=disable to the end of it if you don't have SSL turned on.

Well, that did help.  Now I'm getting these types of errors with or without the ?sslmode=disable at the end of the DATA_SOURCE_NAME.  Any ideas?
time="2016-05-24T16:50:08Z" level=error msg="Unexpected error parsing column:  pg_stat_database_conflicts confl_snapshot " file="postgres_exporter.go" line=334
time="2016-05-24T16:50:08Z" level=error msg="Unexpected error parsing column:  pg_stat_database_conflicts confl_bufferpin " file="postgres_exporter.go" line=334
time="2016-05-24T16:50:08Z" level=error msg="Unexpected error parsing column:  pg_stat_database_conflicts confl_deadlock " file="postgres_exporter.go" line=334

Any suggestion for the above errors?  Or a work around.  I would rather not install go, but it looks like I may need to.

That's a separate bug with something which has changed with newer versions of postgres. I need to look into why the column types are different.

Related to #8

This should be resolved in the newest docker image on docker hub. docker pull wrouesnel/postgres_exporter:latest will get it for you.


i can't install the tool

sudo go install github.com/tomnomnom/anew@latest
internal/unsafeheader
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/abi
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/cpu
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/goexperiment
compile: version "go1.17.2" does not match go tool version "go1.17.1"
runtime/internal/atomic
compile: version "go1.17.2" does not match go tool version "go1.17.1"
runtime/internal/sys
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/race
compile: version "go1.17.2" does not match go tool version "go1.17.1"
sync/atomic
compile: version "go1.17.2" does not match go tool version "go1.17.1"
unicode
compile: version "go1.17.2" does not match go tool version "go1.17.1"
unicode/utf8
compile: version "go1.17.2" does not match go tool version "go1.17.1"
math/bits
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/itoa
compile: version "go1.17.2" does not match go tool version "go1.17.1"
container/list
compile: version "go1.17.2" does not match go tool version "go1.17.1"
crypto/internal/subtle
compile: version "go1.17.2" does not match go tool version "go1.17.1"
crypto/subtle
compile: version "go1.17.2" does not match go tool version "go1.17.1"
unicode/utf16
compile: version "go1.17.2" does not match go tool version "go1.17.1"
vendor/golang.org/x/crypto/cryptobyte/asn1
compile: version "go1.17.2" does not match go tool version "go1.17.1"
internal/nettrace
compile: version "go1.17.2" does not match go tool version "go1.17.1"
vendor/golang.org/x/crypto/internal/subtle
compile: version "go1.17.2" does not match go tool version "go1.17.1"
med@med:~$ go version
go version go1.17.1 linux/amd64

I have a problem after installing the performance
I used the command:
go install -v github.com/tomnomnom/anew@latest
It was installed successfully but not working!
┌──(root💀kali)-[/home/kali/Desktop]
└─# for url in $(cat test-url.txt);do http -b $url | grep 'Disallow' | awk -F ' ' '{print $2}' | cut -c 2- | anew test-words.txt;done
Command 'anew' not found, did you mean:
command 'afew' from deb afew
command 'new' from deb mmh
command 'new' from deb nmh
command 'znew' from deb gzip
Try: apt install

┌──(root💀kali)-[/home/kali/Desktop]
└─# go version                                                                                2 ⨯
go version go1.17.2 linux/amd64


Appstudio update hacbs-test-project-t05f

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-t05f-on-pull-request-
creating pipelinerun hacbs-test-project-t05f-on-pull-request- in namespace rhtap-demo-tdkw-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-t05f-on-pull-request-
creating pipelinerun hacbs-test-project-t05f-on-pull-request- in namespace rhtap-demo-cyxb-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Option to disable downloading?

For our app, we don't wish to expose the ability to download the remote mp4 video; we want the user to only be able to play in-browser.

See https://github.com/CookPete/react-player/issues/323#issuecomment-364357541


Usage of default AET timezone

This is related to issue #481. Opening a new issue to track this separately.
Issue: The current timezone addition provides the ability to specify timezone associated with a file during upload and hence fixes the issues in upload step. However, apart from upload, the default AET timezone was assumed in the following instances as well, which may lead to job failures.

Methods byYear, byMonth, byDay, byHour, byDate etc. in Partition.scala will parse the timestamp in partition field based on a given format. So viewHive step will fail if the timestamp is invalid in default timezone.
isDate method in Validator.scala will fail if timestamp in validator field is invalid in default timezone.
now method in TimeSource.scala also use the default timezone to generate TimeSource. It doesn't cause a job failure, but it restricts from creating TimeSource in any specific timezone.

To fix these issues, i think we can add the capability to specify desired timezone for above scenarios.

For byYear, byMonth and byDate, the set of valid dates is the same for every time zone, so it can stay the way it is, or always use UTC via DateTimeFormat.forPattern(dateFormat).withZoneUTC() or we can have a timeZone that defaults to UTC (similar to the previous timezone fix #483).
For byHour we could do the same, but then there's the potential for a column that is Sydney-time to be treated as UTC by not being updated.  This may be okay here though, since this isn't intended to be where we do validation of the data, and the parsed date-time is just used to extract the fields, which always exactly match the data.  I.e., we never do time zone adjustment here, this isn't the right place to do that.
isDate is the right place to do validation, so we should have isDateUTC and isDateTZ and require these to be used if an hour is included in the format.  Alas, I don't see a nice way to check for hours in patterns via http://joda-time.sourceforge.net/apidocs/org/joda/time/format/DateTimeFormatter.html
For the TimeSource now method, the date will be different, so we should probably always specify the timezone here - so I'm thinking there will be a new timezone argument with no default, and nowUTC and perhaps nowSydney for convenience.
Feedback welcome.

cc @AnnTomie @sibisasidharan @josephshobin

After discussing with @SajeevAr for the first part, it's not true that "it can stay the way it is" because the  format may include hours even for byYear, byMonth and byDate.

For isDate it could be better to have the timezone in the metadata in the thrift (and RDV metadata).  See: https://github.com/CommBank/datahub/issues/301


Exceeding quota after ocis_max_size_quota set gives 500 status code

Description
Configuring and running ocis with env OCIS_SPACES_MAX_QUOTA, when a space is created exceeding the quota. The status code is 500.
Steps to reproduce:

Run ocis with env OCIS_SPACES_MAX_QUOTA=50
And try to create a space project with quota size 51.

curl command to create space with quota with admin user:
curl -XPOST -k -uadmin:admin https://localhost:9200/graph/v1.0/drives -H 'Content-Type: application/json' --data-raw '{"Name": "testQuota", "description": "This is a test quota", "quota": {"total": 51}}' -v

Expected Behaviour:
The status code should be 507 insufficient storage.
Actual Behaviour
< HTTP/1.1 500 Internal Server Error
< Content-Length: 157
< Content-Security-Policy: frame-ancestors 'none'
< Content-Type: application/json; charset=utf-8
< Date: Wed, 31 May 2023 08:42:19 GMT
< Vary: Origin
< X-Content-Type-Options: nosniff
< X-Frame-Options: DENY
< X-Graph-Version: 3.0.0-rc.4+3bbed2fc7
<
{"error":{"code":"generalException","innererror":{"date":"2023-05-31T08:42:19Z","request-id":"desktop/gWCjfgA4bL-000516"},"message":"error creating space"}}

CC @micbar @ScharfViktor

You cannot create space with certain quota in the real life (first we create space with default quota)
and if you have correct configuration GRAPH_SPACES_DEFAULT_QUOTA < OCIS_SPACES_MAX_QUOTA You will never have this problem.
my settings: OCIS_SPACES_MAX_QUOTA=2000000000 GRAPH_SPACES_DEFAULT_QUOTA=1000000000
you cannot change quota more than MAX_QUOTA via web

You cannot create space with certain quota in the real life (first we create space with default quota) and if you have correct configuration GRAPH_SPACES_DEFAULT_QUOTA < OCIS_SPACES_MAX_QUOTA You will never have this problem.
my settings: OCIS_SPACES_MAX_QUOTA=2000000000 GRAPH_SPACES_DEFAULT_QUOTA=1000000000
you cannot change quota more than MAX_QUOTA via web

From UI it might not be possible but giving 500 status code with API seems not be feasible IMO.

From UI it might not be possible but giving 500 status code with API seems not be feasible IMO.

I don't understand the sentence.
IMO it is possible to send an API request to create a space, specifying a quota that is too large, and a 500 status is returned.
Even if the web UI does not make a request like that, some other client might one day. Any API request that can result in a 500 response is a bug, by definition, and should be fixed.

I would suggest another expected result, like when we set a quota higher than the max quota
400
{
    "error": {
        "code": "notAllowed",
        "innererror": {
            "date": "2023-06-06T07:51:08Z",
            "request-id": "Viktors-Air.local/S5s3y8xnwl-015573"
        },
        "message": "decompsedFS: requested quota is higher than allowed"
    }
}


feat(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1559

Questions ?
Please refer to the Backport tool documentation


Allow users to override the user-agent

Is your feature request related to a problem? Please describe.
We need to differentiate requests from an instance of Data Prepper that our solution is using and from the rest of a cluster's clients.
To migrate data, our solution uses does a bulk move of data from a source cluster to a target cluster.  Independently, individual requests are recorded from the source cluster and replayed to the target to both keep the target cluster in sync and also to compare the behavior of the two clusters.
When we capture traffic, depending on the order that a customer chooses to perform each step, there may be overlap with the Data Prepper requests to the source.  We'd like to be able to mask out those requests from our replay.  Those would at the very least, create more noisy data for users and could cause confusion as they would see updates replayed on already existing data that was migrated with Data Prepper.  Allowing the customer/us to set a unique value that we can easily filter on the capture side would eliminate this problem and be more more efficient (much lower costs).
Describe the solution you'd like
I'd like to have a command line flag to set the user-agent HTTP header for all requests that Data Prepper sends.  A default value of something different than the ES/OS user-agent may be beneficial too.
Describe alternatives you've considered (Optional)
Other HTTP header values could work too, but user-agent seems like it could be the most natural and easy to explain one.  For our greater solution, dealing with the duplicate data better is possible, but it is 1) considerable effort to mitigate, 2) still will be expensive as we aren't able to remove the data passively.
Additional context
N/A

@gregschohn , Thanks for this request. Do you want this configurable for both the opensearch sink and the opensearch source?
Do you have a proposal on how the user would configure this?

A command line argument or a setting in the pipeline file would work (so would an environment variable, but that seems like it wouldn't be the best experience for users in general).  We'll want the same user-agent for all requests, so a static value loaded once is fine.
Our needs at this time are just for the source - so you can use one user-agent configuration for both or separate ones.  We don't have an opinion on that detail.


i am getting exception when using this code.

[Enter feedback here]
Getting the below 2 exceptions when I use the original code
Error CS1503 Argument 1: cannot convert from 'int[]' to 'Microsoft.TeamFoundation.WorkItemTracking.WebApi.Models.WorkItemBatchGetRequest
Error CS1503 Argument 3: cannot convert from 'System.DateTime' to 'System.Threading.CancellationToken'
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e978322f-15ae-3a89-ae89-497cdfa5070a
Version Independent ID: f8654902-e266-c12b-8e25-a2fa7a2ac626
Content: Get work items programmatically from Azure DevOps Services - Azure DevOps
Content Source: docs/integrate/quickstarts/work-item-quickstart.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @chcomley
Microsoft Alias: chcomley

@Umapathymicrosoft - Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

If there are documentation issues for us to address, we can reopen this issue.

@Tim Sherer - The samples in the document throwing exceptions. Do you want to check what is causing the issue ?
From: Tim Sherer @.>
Sent: Tuesday, November 9, 2021 12:51 PM
To: MicrosoftDocs/azure-devops-docs @.>
Cc: Umapathy Venkatachalam @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-devops-docs] i am getting exception when using this code. (Issue #11586)
Closed #11586https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-devops-docs%2Fissues%2F11586&data=04|01|umaven%40microsoft.com|a28ac55d29234583eb3808d9a3c2ad94|72f988bf86f141af91ab2d7cd011db47|1|0|637720878816698868|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2FPidw8qHiZThjQ0ATpnEFKotMmbAVvlSPfgioVvIuXk%3D&reserved=0.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-devops-docs%2Fissues%2F11586%23event-5592713909&data=04|01|umaven%40microsoft.com|a28ac55d29234583eb3808d9a3c2ad94|72f988bf86f141af91ab2d7cd011db47|1|0|637720878816698868|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ZpqoF3nqjULJC0nJgt4IVN%2FHiIbGNx%2FqlYCm0gvu9KE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALWVG2GZWJEE62IQE67FW53ULGCULANCNFSM5HWGDRBQ&data=04|01|umaven%40microsoft.com|a28ac55d29234583eb3808d9a3c2ad94|72f988bf86f141af91ab2d7cd011db47|1|0|637720878816708823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=wm0Mfz2Z9HmE3H%2BhwDaHgbzGYNhdCf31S4p1Xp60f2I%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|umaven%40microsoft.com|a28ac55d29234583eb3808d9a3c2ad94|72f988bf86f141af91ab2d7cd011db47|1|0|637720878816708823|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0RaJQVP1yThy7rEclEMXjrC7oBcyYSRtA18GZAhw1Rc%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|umaven%40microsoft.com|a28ac55d29234583eb3808d9a3c2ad94|72f988bf86f141af91ab2d7cd011db47|1|0|637720878816718773|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ojujM%2FrOEX3zX%2F7dGoLqfG5J6qUSNhbfIAslY8y54Jc%3D&reserved=0.

Were you able to find a solution for this problem ?

@arshiamaner We're reaching out to a subject matter expert. We want to track down potential issues with the docs, but if you're encountering issues with your configuration, you need to reach out to your support provider. Also, consider the dev team board.

@TimShererWithAquent https://githubmemory.com/@chcomley?page=2 Reverting back to 'GetWorkItemsAsync' helped!

Hi all, Is this still an issue? Do we need to update to the article?