fix doc about vol

Fix doc about vol to point out that when use -v /hostpath:/containerpath, the host path can be relative and absolute, but the destination path must be absolute.
According to #15846.

ooo, now that'll be interesting for my toys

@thaJeztah @moxiegirl

Oh, so sorry about my careless!

The doc is updated, please review.
@SvenDowideit

Did this change recently? I always thought host path had to be (or was advised to be) absolute. Are there situations where relative paths don't work?

Yes, for now, if I use the following command
docker run -v relative_path:/container_path ubuntu /bin/bash

I will get a mountpoint as below:
{
        "Name": "relative_paht",
        "Source": "/var/lib/docker/volumes/relative_paht/_data",
        "Destination": "container_path",
        "Driver": "local",
        "Mode": "z",
        "RW": true
    }

The relative code is below:
https://github.com/docker/docker/blob/master/daemon/volumes_unix.go#L89
https://github.com/docker/docker/blob/master/daemon/volumes_unix.go#L400
@thaJeztah

Oh, but that's not a relative path; that's creating a new named volume.
With relative path, I mean;
docker run -v ./relative:/abs/path myimage

I tried this, and it looks like there's actually a bug, because using a relative path will create a named volume (named ./);
docker run -v ./:/foobar ubuntu

Inspecting that container;
"Mounts": [
    {
        "Name": "./",
        "Source": "/var/lib/docker/volumes/_data",
        "Destination": "/foobar",
        "Driver": "local",
        "Mode": "z",
        "RW": true
    }
],

Using an absolute path, correctly creates a bind-mounted volume;
docker run -v $(pwd):/foobar ubuntu

Inspecting that container:
"Mounts": [
    {
        "Source": "/root/dockerfiles",
        "Destination": "/foobar",
        "Mode": "",
        "RW": true
    }
],

Ok, relative paths are indeed not supported, for reference, see https://github.com/docker/docker/issues/15526, and https://github.com/docker/docker/issues/15526#issuecomment-135412496. So, the docs may need to be improved around this (to prevent confusion), but this PR currently is not correct.
@xlgao-zju do you want me to close this PR, or do you want to update the PR to clarify that the host path must be either an absolute path (to create a bind-mounted volume) or a "name" (to create a named volume)?

I created https://github.com/docker/docker/issues/16132 to track the ./ bug

Ok, I'll update this PR.

Thank you very much for your advice. I'll do it as soon as possible. @moxiegirl

The doc is updated, please review.
@thaJeztah @moxiegirl

Oh, but it's still not describing the behavior correctly.
The description of the flag looks to be incorrect (here, and possibly in other places);
-v=[]: Create a bind mount with: [host-dir:]container-dir[:rw|ro].

This only mentions creating a bind-mounted volume, however;
-v myvolumename:/container-dir

Will create a "named" volume, and not bind-mount a host-directory. It think the description of this flag needs to be updated to something like;
-v, --volume[] = [host-dir | volume-name :]container-dir[:rw|ro][,z|Z]

(the z/Z options were also missing)
So, the relative paths we're talking about here, are actually "named volumes". To list some examples:

# create an "unnamed" volume for `/container/path`. the volume is
# pre-populated with data from `/container/path` inside the container
# if `/container/path` exists.
docker run -v /container/path

# bind-mount directory `/host/path/` at the `/container/path` location inside the container.
# if `/host/path/` exists on the host, the contents of that directory is accessible in the
# container. If `/host/path` does *not* exist, docker will create a directory at that
# location on the host before bind-mounting it in the container. `/host/path/` *must*
# be an absolute path.
docker run -v /host/path/:/container/path/

# create or mount a "named" volume `foobar`. creates a new empty
# volume with the given name if it does not yet exist.
docker run -v foobar:/container/path/

@thaJeztah Since this appears to be a problem in general with this documentation, and extends beyond the simple adjustment to the parameter description --- should we merge this one and open a larger PR to fix the other issue?

@moxiegirl I agree that this needs to be fixed in general, and I'm good with adding a note to start that process.
My worry is about "The host-dir can be relative or absolute." which gives the impression that a relative path is allowed. Although it's explained later on ("If the host-dir value is foo, Docker will create a volume named foo"), the first sentence is still confusing.
Basically, host-dir must be either an absolute path, or a name. A name must start with an alphanumeric character, followed by a-z0-9, _, . or -.
(FYI: The "name" is currently too permissive (see https://github.com/docker/docker/issues/16132, worked on in https://github.com/docker/docker/issues/16132), the jury's still out on supporting uppercase A-Z for compatibility with Windows.)

@xlgao-zju Thanks for your patience while we work this out.... @thaJeztah how about:
The container-dir must always be an absolute path such as /src/docs. The host-dir can either be an absolute path or a name value. If you supply an absolute path for the host-dir, Docker bind-mounts to the path you specify. If you supply a name, Docker creates a named volume by that name.  A name value must start with start with an alphanumeric character, followed by a-z0-9, _ (underscore), . (period) or - (hyphen).
For example,  you can specify either /foo or foo for  a host-dir value. If you supply the /foo value, Docker creates a bind-mount. If you supply the foo specification a named volume.

That's Ok. And thank you so much for your advice, it is always detailed.

@moxiegirl perfect!
Sorry for being so stubborn on this one; trying to prevent confusion (and invalid "bug" reports :smile:)

The doc is updated, please review: )
@moxiegirl @thaJeztah

Thanks @xlgao-zju! LGTM @thaJeztah for the merge.

Thank you again for your advice! I've learned a lot from it.
@moxiegirl

@xlgao-zju You are welcome.  Thank you for contributing.  I (we) very much appreciate your time and work.

Great! Thanks so much @xlgao-zju!
LGTM


add base tag

Fixes #10

Thanks for the PR! However, this should be disabled by default and only added if the user passes a string (the href). Then budo can opt into this feature without it breaking all the other tools depending on simple-html-index.

Sure, that makes sense. Updated the PR. Do you think base is a good name for the option?

Is anything preventing this from being merged? Can I do anything to help?

Looks good to me. I'm on vacation right now so I won't be able to merge for a couple weeks. 😊

Fair enough. Will leave for hols end of the week as well :beers:

Co-maintainer to the rescue! :tada:

Released as v1.4.0 :sparkles:

Heh thanks Yoshua! 👌
Sent from my iPhone

On Aug 3, 2016, at 6:58 PM, Yoshua Wuyts<EMAIL_ADDRESS>wrote:
Co-maintainer to the rescue! 🎉
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Cool, Thanks! Will make a corresponding PR for budo then.


Crash on Safari using custom body font family within tooltip options

Expected Behavior

When hovering over a tooltip it is expected to show something like this

Current Behavior

When hovering over a tooltip the browser (Safari) crashes and reloads the page automatically.
Possible Solution

Steps to Reproduce (for bugs)
https://codepen.io/wriken/pen/bGNpEWz
Open on Safari and try to hover one of the bars on the chart to get the tooltip, this causes a crash and reload of the browser page.
Tested on:

Safari 13.0.3, Mojave 10.14.6
Safari 11.0.3, Sierra 10.12.6

Context

It's not possible to use a css variable on the bodyFontFamily/titleFontFamily attributes of the tooltip options (this may happen as well on other configs of chartjs that allow for custom font family).
This makes it so it is impossible to interact with the chart since it will crash and reload the page when hovering over a tooltip.
Environment

Chart.js version: codepen version (2.5.0), 2.9.3 (with Reactjs)
Browser name and version: Safari 13.0.3, Safari 11.0.3
Link to your project: https://codepen.io/wriken/pen/bGNpEWz

That is not how you use css variables in javascript.
Does that work in Safari: https://codepen.io/kurkle/pen/JjoXXXv ?

Yes that works on Safari, I have a question though, is there a reason bodyFontFamily: 'var(--body-font-family)' doesn't work, but bodyFontColor: 'rgb(0,255,10)' does? The examples on [(https://www.w3schools.com/css/css3_variables.asp)] show it is possible to use css variables this way, although I understand that using within a string it is different than calling the function.

Yes, that is how you use those in css. But when accessign the same variables from javascript, it needs to be done differently (like in https://codepen.io/kurkle/pen/JjoXXXv for example).
Closing as resolved.


Add folder/files without target but keeping tree structure

Hello all!
Just wondering if I can add files and folders to project without adding to target (like fileGroup) but keeping the tree structure.
Maybe something like this?
targets:
  MyApp:
    sources:
      - path: MyApp
        excludes:
          - "Resources/Environments"
    fileGroups:
      - Resources/Environments

It would generate the project structure as if I'm not excluding folders AND not adding to the MyApp target.
Thank you very much!

I'm writing this off the top of my head, but does this work?
targets:
  MyApp:
    sources:
      - path: MyApp/Resources/Environments
        buildPhase: none
      - path: MyApp

hey @yonaskolb ! unfortunately it doesn't 😕
it is adding to the root, to target and to build phase

I have the same question. I would like to keep MarkDown files visible within Xcode, but they shouldn't be included as a resource in my app.

I found a solution:
    sources:
     - path: myTarget
       excludes:
         - "**/*.plist"
     - path: myTarget
       includes:
         - "**/*.plist"
       buildPhase: none

This adds the files to the project, but not to the target.
Use the same lists of paths in includes and excludes, and add buildPhase: none in the includes.

works, but seems kinds h4xey :P


Fixed spec runner

Resolves #89

added setFixtures/clearFixtures
added testDiv to the DOM (jsDOM and specrunner)
modified component.spec and noconflic.spec to reflect new changes

:+1:


Possible memory leak

I get memory leak warning when I have more than 10 usages of ftp-deploy

It looks like the problem comes somewhere from bluebird, that is used by promise-ftp
Any idea how to fix that?

Fixed with basic-ftp upgrade, available in latest release!
https://github.com/SamKirkland/ftp-deploy/releases/tag/v1.2.4


Profile compilation times in tests

Profile the compilation times within tests to ensure that the majority of time spent is not from within JCT file management classes themselves.
If any issue is found, look into implementing optimisations.

This may be achievable alternatively by writing a custom test pack for this.

For now, I am closing this as it is not a hard requirement. Will open a new issue in the future if/when it is needed.


After upgrade to 029 US build, GPS no longer able to achieve fix reliably.

Stratux version: 1.6r1-eu029

Stratux config: Attached as startux_censored.conf.txt to get around github filters.  Removed wifi passwords and saved.
SDR

[ ] single
[X] dual

GPS

[X] yes
[ ] no
type: USB u-blox 8 GNSS (NMEA protocol)

AHRS

[X] yes
[ ] no

power source: 10Ah USB Battery Pack
usb cable: Elbow Cable with Power Button

EFB app and version: (e.g., WingX Pro7 8.6.2)
FltPlan Go 5.0.17
EFB platform: (e.g., iOS 9.2)
Android 14 AP1A.240305.019.A1
EFB hardware: (e.g., iPad Mini 2)
Pixel 5a

Description of your issue:
After updating to the newest build, my stratux no longer can achieve a GPS fix.  I have tried rerouting the antennas to be sure they were not near the GPS unit.  Reimaging the microSD card with the stratux-v1.6r1-eu029-94438ef6-us.img to see if the OTA update failed.  The GPS will see satellites but will not achieve a fix reliably.  I have flown three times since updating and two of those times I needed to switch over to my phone GPS.  The rest of the ADS-B features of the Stratux work as intended.

If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.
stratux_censored.conf.txt
stratux.log

I may have the same problem, but ordered a new GPS as it was cheaper than
the time needed to troubleshoot. I have not tested it yet. FWIW, it seemed
to work fine for a while, then would occasionally drop out, then stopped
working altogether.
On Tue, Mar 19, 2024 at 9:14 PM CharAznableLoNZ @.***>
wrote:

Stratux version: 1.6r1-eu029
2.
Stratux config: Attached as startux_censored.conf.txt to get around
github filters. Removed wifi passwords and saved.
SDR

single

dual

GPS

yes

no
type: USB u-blox 8 GNSS (NMEA protocol)

AHRS

yes

no

power source: 10Ah USB Battery Pack
usb cable: Elbow Cable with Power Button
3.
EFB app and version: (e.g., WingX Pro7 8.6.2)
FltPlan Go 5.0.17
EFB platform: (e.g., iOS 9.2)
Android 14 AP1A.240305.019.A1
EFB hardware: (e.g., iPad Mini 2)
Pixel 5a
4.
Description of your issue:
After updating to the newest build, my stratux no longer can achieve a
GPS fix. I have tried rerouting the antennas to be sure they were not near
the GPS unit. Reimaging the microSD card with the
stratux-v1.6r1-eu029-94438ef6-us.img to see if the OTA update failed. The
GPS will see satellites but will not achieve a fix reliably. I have flown
three times since updating and two of those times I needed to switch over
to my phone GPS. The rest of the ADS-B features of the Stratux work as
intended.
If possible, enable "Replay Logs", reproduce the problem, and provide a
copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and
http://<IP_ADDRESS>/logs/stratux.log.
stratux_censored.conf.txt
https://github.com/cyoung/stratux/files/14659068/stratux_censored.conf.txt
stratux.log https://github.com/cyoung/stratux/files/14659069/stratux.log
—
Reply to this email directly, view it on GitHub
https://github.com/cyoung/stratux/issues/884, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAT66CVBRADRYQGQITQXZ3YZDPI5AVCNFSM6AAAAABE6S54NOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4TMNBQG4ZDKMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

The only real change regarding UBlox 8 in v029 was, that it now uses GPS+Galileo+Beidou instead of GPS+Galileo+GLONASS.
If you are able, you might want to try to compile Stratux yourself, and undo this commit for testing?
https://github.com/b3nn0/stratux/commit/3c872082649e51f72b368782987da38f5911105c
Maybe GLONASS works much better in the US than Beidou?

Can someone who has this issue provide a screenshot of the GPS page so that we can see what is happening?

I'm not sure if this is the page you're talking about but here is a screenshot of it tracking five satellites with no fix.   It does seem to drop off to one satellite often.  The unit is attached to the window of my house that has a great view of the western sky with nothing in the way beyond a tree here and there.  It has the same behavior in the plane as well.
While typing this I saw it surge up to eight satellites tracked then settle back down to four tracked.

With GLONASS disabled and BeiDou enabled, I've never had a single resolved and fixed BeiDou satellite, and the best I've seen for the other satellites is maybe 5 or 6 in solution.  I disabled BeiDou and enabled GLONASS in gps.go and I get a minimum of 8 in solution.  Not sure if geographic location is pertinent, but I'm in central Texas

Mine has never achieved a fix inside.  This screenshot is the unit sitting outside on a table facing the same direction as the previous tests.
It has failed 2/3 times to get a fix from inside the plane, the only thing between it and the outside world is a thin piece of plastic window.  I'm in the Seattle area.

Mine is indoors and the ublox8 is in an upper USB receptacle above the 2 SDR's.  Im just curious, where are you (Virus Pilot) located?

I am located in the middle of Germany. Did you have significantly more satellites before migrating to v029? Would you never get more than ~10 satellites with v029? I did some measurements a few years ago and it turned out that with this particular setup of your GPS just above the SDRs you are loosing up to 10dB GPS signal. This is why I am recommending an external GPS in my shopping list: https://github.com/VirusPilot/stratux-pi4/wiki/Shopping-List-v3. Do you by chance have a USB extension cable so that you can move your GPYes away from the SDRs just for testing?

I do have an external GPS, I will plug that in and let you know.  I have been purposely running in a "adversely difficult environment" just to see what absolute minimums can be while still giving me a good position fix.

OK, this is with a NEO 8MU w/external antenna. It resolved 15 satellites almost immediately and signal strength is much better.  This is still inside my house, I expect it would be even better outside.  I'm going to recompile with BeiDou enabled and GLONASS disabled to see what that looks like...

And this is that exact same setup as mentioned, NEO M8U, nothing moved.  Recompiled with BeiDou enabled, GLONASS disabled.  A LOT different!! Going back to BeiDou disabled, GLONASS enabled.

And finally, returning to BeiDou disabled and GLONASS enabled... big difference.

VirusPilot - I've never really had any issues with any of B3NNO's versions - but only noticed after this thread started that I was only getting around 8 resolved satellites with V29.  I think the change of disabling GLONASS may be not give the best results for the USA, or at least where I am located.  It might make sense for BeiDou or GLONASS to be configuration setting...

VirusPilot - I've never had issues with any of B3NNO's versions - to me they have been nothing but positive for Stratux. I just happened to notice after this thread started that I was only getting around 8 resolved satellites with v29. I think the change of disabling GLONASS may be not give the best results for the USA, or at least where I am located. It might make sense for BeiDou or GLONASS to be a configuration option...

Alright, this is very helpful so we may revert back to GLONASS then.

If you by chance can do one more test, I would appreciate it. In gps.go the following change would be required:
GPS min 8, max 16 sats
GAL min 4, max 8 sats
BDS min 4, max 8 sats
It requires the following lines changed, starting with: https://github.com/VirusPilot/stratux/blob/de992daa057a1f7eb2793762493f066fb0bd5a14/main/gps.go#L517
to
galileo := []byte{0x02, 0x04, 0x08, 0x00, 0x01, 0x00, 0x01, 0x01}
beidou  := []byte{0x03, 0x04, 0x08, 0x00, 0x01, 0x00, 0x01, 0x01}

Actually, I went ahead and added a conf setting for using GLONASS... of course, this was just for my own education  I'll go ahead and run with those changes suggested by VirusPilot...

I went ahead and added an option for using GLONASS, just for my own education. I will do the changes suggested by VirusPilot re-test, and let you know...

Cool, feel free to submit a pull request accordingly.

Here you go.  bytes adjusted as requested, running for 25 minutes

aha, now we are talking. Btw, if your GPS is a USB ublox M8 then I wonder why you only arrive at 2.1Hz update rate - our setting for the M8 is 10Hz like in my screenshot above.

It's a ublox M8, however its a NEO M8U with on-board attitude sensors... Ive played with it a lot and it's possible I changed a setting in U-Center... Is that 10Hz assigned by a ConfigCommand, in Stratux, or is that supposed to be the M8 default?  If need be I can go into U-Center check.

I have now reverted to GLONASS instead of Beidou for the next version, as I was able to reproduce this more or less.
A web interface setting seems overkill to me. People tend to configure all kind of random stuff that they don't know about.. I have seen far too many completely miss-configured Stratuxes.
I'd rather find a default that works well for everybody, and I think in this case GLONASS is that default.

As a further test, this looks much like the originally posted issue.  This is using an older GPYes/ublox7 usb, running so far for 1/2 hour.

this is unrelated to any configuration, it is just poor GPS level

Yes, after plugging the ublox7 into a usb extension cable and putting it on a window sill I'm getting a minimum of 10 resolutions + SBAS.


bug/suggestion: saving ica objects

Hi
I ran into an issue when trying to save an ica object to disk. A PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: [...] was raised by the save method in mne/preprocessing/ica.py, specifically in the try/except block. However, the reason it entered the except block was that a TypeError: Object of type 'int32' is not JSON serializable was raised in _serialize (in _write_ica). Thus, it seems that (at least on Windows on which I am working) if an error occurs while writing the ica object then a PermissionError is raised (I guess because the file has not be closed) irrespective of the error causing _write_ica to fail. Furthermore, the reason for the TypeError (the 'real' problem in my case) was that I (incidentally) passed a numpy.int32 to ica.n_components.
Thus, I have two suggestions.
First, given that a PermissionError was raised instead of the 'actual' error suggest a (perhaps platform specific?) bug which might be fixed, e.g., by changing
try:
    _write_ica(fid, self)
except Exception:
    os.remove(fname)
    raise
end_file(fid)

to
try:
    _write_ica(fid, self)
    end_file(fid)
except Exception:
    end_file(fid)
    os.remove(fname)
    raise

in the ica save method (in ica.py) such that the file is closed before attempting to remove it (this would cause the 'correct' error to be raised by raise).
Second, I passed a numpy integer to n_components (particularly, it was a rank estimate as returned by raw.estimate_rank) which resulted in a TypeError from json. A possible fix might be to change
elif isinstance(value, int):
    value = int(value)

to
elif isinstance(value, (int, np.integer)):
    value = int(value)

in _serialize (in ica.py) [or perhaps casting to core int when instantiating the ica object?]. I am aware that the doc for mne.preprocessing.ICA actually do specify n_components : int | float | None but in many applications core and numpy ints behave similarly (correct me if I am wrong) and so one might not be aware whether one is passing one or the other unless explicitly testing it - at least in this case I wasn't :)
The issue can be reproduced using the MNE tutorial from http://martinos.org/mne/stable/auto_tutorials/plot_artifacts_correction_ica.html by replacing the line n_components = 25 with for example n_components = np.int32(25).
I am using python 3.6 on Windows 10 64-bit.

A possible fix might be to change

We usually use elif isinstance(value, numbers.Integral) to be general.
The other fix (closing before raising) seems reasonable to me.

yes, numbers.Integral is probably better

can you send a PR?

Yes, here it is #4270. I skipped the second part since that was actually fixed about a month ago in #4179 but which I did not have in my local copy. Thus, the PR concerns only the first part of this issue.


Taxonomy Menu Placement In AsideSecond Zone is not correct

I have tried to create a tag cloud for defined terms in a taxonomy and show it on the mentioned zone, while there are some widget on the AsideFirst the content on the page is pushed to the right and two aside zones are besides each other. This happened in 1.10 branch.
anyone happened to see this issue before?

Can you give some repro steps? It doesn't sound like an Orchard bug but a theme's one. Please try with the TheThemeMachine theme to reproduce the issue.

Please reopen when you provide the repro steps.


Set v7.0.0

What does this PR do?
Sets the version to 7.0.0.
How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 214

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

Totals

Change from base Build 213:
0.0%

Covered Lines:
590

Relevant Lines:
590

💛  - Coveralls


Facebook auth is no longer working

When trying to login with Facebook, I get this message which basically translates to:
The user is not allowed to see this application. The user is not allowed to see this application because of the config set by this developer.
I don't know if it can be related to #349 or #250 (which I had previously commented on).

If you need a sandbox to reproduce the bug, I would be happy to provide one to you in a private conversation.

I don't know if you fixed it but it seems to be working again.

We did fix the problem, thank you for reporting.


Updating environmental data load for new files

Data from https://github.com/SimonGreenhill/dplace-data/pull/12
Re-build the database and re-load the data.
Will give some errors, these can be ignored as not all SCCS and WNAI societies are stored in our database yet. There are also two environmental variables that are not floats (one is a boolean, the other is text, I believe) - these will also throw errors, but can be ignored for now as adding them to the database will require changing the environmental models, search, etc.
Also fix for #292

Current coverage is 92.59%

Merging #303 into master will decrease coverage by -0.03% as of b1d9a5c

@@            master    #303   diff @@
======================================
  Files           38      39     +1
  Stmts         1965    1972     +7
  Branches         0       0
  Methods          0       0
======================================
+ Hit           1820    1826     +6
  Partial          0       0
- Missed         145     146     +1

Review entire Coverage Diff as of b1d9a5c

Powered by Codecov. Updated on successful CI builds.

@xrotwang Can you check my latest commit and let me know if I've done that correctly? This isn't related to the environmental search, the society count is just needed for display in the language search (it tells the user how many societies there are for a given language).

Yes. This looks good. I think even in non-performance-critical code paths
it is good practice to avoid n+1 query issues.
Am 10.04.2016 01:28 schrieb "Stephanie Ng"<EMAIL_ADDRESS>
@xrotwang https://github.com/xrotwang Can you check my latest commit
and let me know if I've done that correctly? This isn't related to the
environmental search, the society count is just needed for display in the
language search (it tells the user how many societies there are for a given
language).
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/D-PLACE/dplace/pull/303#issuecomment-207878676

I think that this commit decreased the search speed enormously again.

Decreased or increased? I didn't change any of the search code, hmm.
-----Original Message-----
From: "Hans-Jörg Bibiko"<EMAIL_ADDRESS>Sent: 11/4/2016 1:37 AM
To: "D-PLACE/dplace"<EMAIL_ADDRESS>Cc: "Stephanie Ng"<EMAIL_ADDRESS>Subject: Re: [D-PLACE/dplace] Updating environmental data load for new files(#303)
I think that this commit decreased the search speed enormously again.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Ensure bundle gem default task matches framework

Previously running bundle gem with a global config containing
BUNDLE_GEM__TEST: 'minitest'

would incorrectly set the default task in the generated Rakefile to
task :default => :spec

whereas bundle gem --test=minitest worked as expected.  This happened because the test task was determined before checking the global config or asking for user input, making the test task and default task inconsistent.
This fix moves the decision to after the test framework has been properly determined.

👍🏻

Awesome, thanks!
@homu r+

:pushpin: Commit d70de62 has been approved by indirect

:zap: Test exempted - status


Rename ground_truths -> ground_truth

@tinomaxthayil are you still working on this?

This was completed with PR 302(https://github.com/explodinggradients/ragas/pull/302)


JWT-AuthAdded

Pull Request Template
Added authentication via JWT token(Backend part)
Fixes #41
How Has This Been Tested?
REGISTRATION: Token appended to the tokens array

LOGUT: Token removed from the tokens array

Checklist:

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules
I have checked my code and corrected any misspellings

Please review @agamjotsingh18

Please review @agamjotsingh18

Please review the PR @agamjotsingh18


OpenSSL: use Windows' system root certificate store

Resolves #12476. This makes things like HTTP::Client.get("https://example.com") work on Windows without any user configuration at all. (The problem technically still exists for non-system certificates, but then non-Windows systems have to face the same problem too.)

Awesome! 🙇
Wouldn't it be best to import the lib functions following the Crystal conventions? E.g.,
fun cert_enum_certificates_in_store = CertEnumCertificatesInStore(...)

Not the Win32 C bindings, they don't change case for some reason

I don't think we adapt the spelling of lib symbols to Crystal conventions anywhere. Except when necessary due to syntax restrictions (such as downcasing the first character).


Regarding transferring previous learned prompt params to the new prompt

Hi @KingSpencer @zizhaozhang , I have a doubt regarding prompt pool initialization in every new task, inside train_continual.py > train_and_evaluate_per_task(). Why do you transfer previously learned prompt keys to the new prompt parameters in prompt pool and prompt key (lines 550-584)?
Based on my understanding from the paper, the prompt pool and prompt key are shared across tasks and the method learns to select the relevant ones using query function-key matching. So if a subsequent incremental task is initializing prompts from the previous step, why are the prompts being shifted? Are new prompts being added at every step or am I missing something?
Any help will be appreciated.

Did you find reasons?


[Help needed] Minimal metrics setup to monitor node health

I'm very new to Cassandra and need to healthcheck a cluster of 5 Cassandra nodes with ~1000 of keyspaces created.
With the default setup, I'm getting a huge number of metrics in my Prometheus, which makes it impossible to query for a time range of more than 1 hour, then it gets killed by OOM. And all those metrics aren't informative to me.
So for the sake of future users, who faced the same problem, I wonder is there a minimal config for the metrics to scrape so average administrator could take a look at and see if there is (or gonna be) something wrong with Cassandra?
Thanks in advance for any kind of help and excuse me for my barbarian English.

Hi @Blasterdick ,
If you would like to check your node status (up/down) I would advise you to use the following exporter: https://github.com/criteo/casspoke
It exposes only few dozens metrics per node.

use the following metrics for grafana graph:
metric: count( count by (exported_instance)(cassandra_up > 0) )  by (exported_instance)
legend:  {{ exported_instance}}
metric: 1 - count( count by (exported_instance)(cassandra_up == 0) )  by (exported_instance)
legend:  {{ exported_instance}}
GL


Remove Apifox.Apifox 2.3.2 to Ruihu.Apifox 2.3.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: http://cdn.apifox.cn/download/2.3.2/Apifox-2.3.2.exe
Status Code: 200
(Automated message - build 869)


templateInstance.set() is not a function

I read in the README:
use template.set(propName, value) to set the value of the first property named propName on the current or an ancestor template instance.
but when trying to run the following code,
      Template.instance().set('someProp', 'someVal');

I get the following error
       TypeError: Template.instance(...).set is not a function

although the getter function is working well:
    Template.instance().get('someProp');

@MohammedEssehemy I'm not sure if you ever figured this out, but set was added in 4.1.0 I believe, so check in .meteor/versions to make sure you have that version installed. If another package you are using relies on an earlier version, that might have prevented Meteor from pulling in the latest package.


Change the date formatting

Is your feature request related to a problem? Please describe.
I don't know if the date shown in a page is using their own settings, or follow my locale. There is no way to know. Look at this:

Is that 12th August or 8th December?
Describe the solution you'd like
There are 2 possible solutions here:

Make it explicit. Show as 12th Aug. Whether you show it as month or day first, it does not matter.
Make it show the explicit date if I hover my mouse over to that date.

@DuncanmaMSFT as FYI.

As another person who faced this issue, I fixed it by scrolling all the way down and choosing my locale (bottom right):

...after setting it to English (United Kingdom) now all dates render the way I expect :+1:


How to install extension?

about:debugging -> this firefox -> load temporary addon -> select manifest.json
I have no idea if the addon actually still works though, it's a long time since I've used it.

Unfortunately it doesn't work. I think it can't get the scrapbook datas. When i open hall of fame it says NOPE always. Anyway thanks for response.


Add dry run for loaders to check for valid instances

Takes care of https://github.com/computational-cell-analytics/micro-sam/issues/703

Fyi @anwai98 I changed the target branch to dev, since this contained all recent dev commits.

Yayy, the windows CI works now! 🥳
cc: @constantinpape


Develop

#9 の対応がメイン、 #7 は作りかけ。。

Pull Request Test Coverage Report for Build 27

12 of 368 (3.26%) changed or added relevant lines in 18 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-13.0%) to 1.964%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/music/music.go
4
6
66.67%

lib/objects/animation.go
0
4
0.0%

lib/scenes/scenemanager.go
2
6
33.33%

lib/music/se_type.go
0
6
0.0%

lib/music/disc_type.go
0
8
0.0%

lib/objects/character_info.go
0
11
0.0%

lib/music/disc.go
0
11
0.0%

lib/scenes/gamescene.go
0
13
0.0%

lib/music/se_load.go
0
16
0.0%

lib/music/disc_load.go
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/game.go
2
0.0%

Totals

Change from base Build 21:
-13.0%

Covered Lines:
14

Relevant Lines:
713

💛  - Coveralls

テストカバレッジが下がってしまったが、一旦は目をつぶる。。。


Styling for suffix and affix actions

Package
filament/filament
Package Version
3.1.21
Laravel Version
10.10
Livewire Version
3.3
PHP Version
8.2
Problem description
When displaying affix or suffix actions for a field, the stye for the button is just rendered as a button.
Expected behavior
This isn't a bug per se - more of a request for styling - but it would be nice if these were flush with the field and rounded off at the appropriate sides
Steps to reproduce
The following code:
TextInput::make('quantity')
    ->hiddenLabel()
    ->numeric()
    ->required()
    ->minValue(1)
    ->step(1)
    ->default(1)
    ->suffixAction(
        Action::make('AddToCart')
            ->icon('heroicon-m-pencil-square')
            ->button()
            ->size(ActionSize::Large)
            ->action(function() {
                $this->addToCart();
            })
    ),

Renders the following:

Ideally this would render as:

Just head to site homepage in repo - Panels not needed.
Reproduction repository
https://github.com/titantwentyone/suffix-affix-actions
Relevant log output
No response

I don't know how well this would fit in our design system, potentially you could use link() instead of button()?

I agree with @danharrin's comment, and was about to suggest using link() as well.

I'm going to close this issue, but feel free to attempt a pull request that takes into account all edge cases. 🙂


Update iavl_v0.19.3 to v0.19.4

Related Github tickets

Related to #562 update

Background
_Fix to IAVL issues that blocked upgrade after Dragonberry.
Testing completed

[ ] Requires confirmation testing.

Breaking changes

[ ] No checks for breaking changes.

@tacoturtle: Any help with this?

@tac0turtle: Will you help us wit the failing test on the module? We are unable to proceed.

Pushed an update with the correct approach (I think?)

Would prefer the replace list were lexicographically sorted.

Golang determines the replace order AFAICT.


Changing channel ordering to NCHW

How should we specify 'param channel_first_inputs: A list of channel first input' parameters to change the channel ordering of all the layers so that there is no 'transpose layers' in the onnx model??
I tried to convert a tf.keras mode as follows
onnx_model7 = keras2onnx.convert_keras(slim_model,'testt', channel_first_inputs=['ip','conv2d_transpose_4'])
But the output still seems to have channel last format? I have previously tried tf2onnx and in a similar manner  i managed to remove all except one transpose layer at output node.
slim-net-157-0.02.zip
I'am using tf.15 with tf.keras and i need all layers with NCHW format with no transpose nodes?
How can we achieve this?

The current argument only supports the input, and outputs needs some extra work.

I got it working... We just need to add one premute layer at the end in keras model. After that the onnx conversion does not create transpose layer at output end. But in some networks internal transpose nodes(pairs) still get created.

Hi @anilsathyan7 . Could you pls show how you add the premute layer in tf.keras??
I have three Transpose layers after translate the model to onnx.
I need transfer the tf.keras trained  model to caffe(which is channel first format). I can't get the same results with keras .h5 model and caffemodel. I'm wondering if the format make any difference

Hi @DanielXu123  here is an example for converting keras nhwc to onnx nchw format: ipynb. This is a demo script for convertng a segmentation model trained in keras.
You need to specify channel first option for input during conversion and add permute layer at end of keras model for output(i.e before conversion).

I checed the scripts, it seems we need to load the trained model and then add the permute layer. After that, we transfer this new combined model to onnx and define the input as channel first format. Is it correct?

Yes, atleast that worked for me and there were no redundant transpose layers after conversion, inside the onnx model.


Unable to install package

Hi there,
I have issued the command:
devtools::install_github("rstudio/EDAWR")
from within RStudio's console and the following message was written:
Downloading GitHub repo rstudio/EDAWR@master
from URL https://api.github.com/repos/rstudio/EDAWR/zipball/master
Installing EDAWR
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
curl: (22) The requested URL returned error: 404
Warning in download.packages(x$name, destdir = dest_dir, repos = x$repos,  :
download of package ‘babynames’ failed
Error in download.packages(x$name, destdir = dest_dir, repos = x$repos,  :
subscript out of bounds
In addition: Warning message:
In download.file(url, destfile, method, mode = "wb", ...) :
download had nonzero exit status
Could you give me any clue to solve the problem?

@mocalvao The command is working for me. The best advice would be to check that you have the latest versions of R and devtools, then retry.


Rename hosts/users/ta Directory to Use a User-Agnostic Name?

Making an issue for this doesn't feel right, but making a PR seemed kinda ridiculous for what amounts to a cosmetic change
It just occurred to me as I was starting to look at refactoring my fork to use hostSpec that it might make more sense if https://github.com/EmergentMind/nix-config/tree/dev/hosts/common/users/ta is renamed to something like owner or something to make it more obvious that it is user-agnostic (and make configuring the root user there make more sense)?

I like it. Let's go with "primary" since that term is already used a little bit in core.
I'll push something this morning unless I run into trouble.

Completed with e198405
Good luck on the refactor btw. The infinite recursion stuff can be annoying to debug!

Good luck on the refactor btw. The infinite recursion stuff can be annoying to debug!

Yeah, I had looked into changing over configVars to use options on my own, and it took me a few days of staring at unintelligible recursion errors to give up and ask for help on the NixOS discord, where I was informed you couldn't use the values of options in imports. In hindsight, it makes sense, but it's annoying and the error messages really don't help.


Pragma directives in UIKit categories causing compiler errors

I'm not sure how I haven't seen anyone else run into this problem, but I'm getting compiler errors in Xcode 7 with some of the UIKit categories complaining about #pragma directives:
Example:
UIActivityIndicatorView+AFNetworking - line 138, 139, 141
"Embedding a #pragma directive within macro arguments is not supported"
Is this a known issue or is it just me? I couldn't find any other issue describing this problem.

Hello,
Did you found any solution?

Nope. I just commented them out for now.

What version of Xcode are you using?

I have the same issue.

Xcode 7.0.1
Targeting iOS 7.1 Builds, with iOS 9 SDK.

I have the same issue.
Xcode 7.0.1
Targeting iOS 7.1 Builds, with iOS 9 SDK.

pragma can not embeded in macro,so you have to move pragma outside the macro

Here's a quick update to my original post (9 days ago, 13th October, 2015);
I upgraded the project I was working on to use Cocoapods, and the errors are no longer.
So instead of manually moving the framework into my folder, I let pods do it for me.

Fwiw, I'm still seeing this in Xcode 7.3 (we don't use Cocoapods). I had to comment out all the #pragma directives manually.

It's worth noting though that just moving the #pragma directives out of the dispatch_async blocks also works.

I am having same kind of issue on my new xcode 8
can anybody help
refer attached screenshot


⚠ Bofrost.lu has degraded performance

In d761400, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1426 ms

Resolved: Bofrost.lu performance has improved in 700c74b after 6 minutes.


hasOne relation inconsistent return

"hasOne" relation inconsistent return
hasOne relationship is a one to one relation, when a related item exists the OBJECT is returned, but if the related item does not exists on the related collection and empty ARRAY is returned.
I think the query should return an empty object or better yet undefined.
js-data: 2.10
js-data-mongo: 0.7.0
Thanks!

stale


k8s 1.31 support for Istio

Add k8s 1.31 support for Istio

A new chart rancher-istio-105.5.0+up1.24.1 was released which supports k8s 1.31


Fix precedence for env over labels

Fixes https://github.com/hudl/registrator/issues/12 - env vars now take precedence over labels, and will override labels if one is set in both places.

lgtm 👍


how to set symbol to prevent rerender componets?

it rerender components while symbol as a params
<AdvancedRealTimeChart
theme="dark"
autosize
symbol={}
>

What components? How does it behave? Is it constantly rerendering? Elaborate please.


Fix runtime dependencies for shadow jars

Summary
This is a follow up of the the previous PR #157 as the PR did not address the issue. After gradle 7 upgrade, observed that the runtime scoped dependencies for openhouse-java-runtime fat jar are listed as compile time dependency in .pom file after the publish. It seems custom fatJarRuntimeDependencies does not seem to be working as expected. Also the .ivy file generated as part of ELR process contains only the implementation configuration dependencies as runtime scoped dependencies and does not contain any dependency with compile time scope. As hadoop-client and iceberg-core both are listed as compile time scope dependency on .pom file, they are not present in the ELRed .ivy file.  Hence, changed the to dependency to implementation configuration from custom  fatJarRuntimeDependencies. This seem to include all the implementation configuration dependencies as runtime scoped jars but not bundled in the fat jar, which is good.
Enabled back gradle module metadata publish as this is used in some other modules like storage. If the ELRed ivy file contains the required runtime dependencies gradle module metadata does not matter as this is internally not supported.
Changes

[ ] Client-facing API Changes
[ ] Internal API Changes
[ ] Bug Fixes
[ ] New Features
[ ] Performance Improvements
[ ] Code Style
[ ] Refactoring
[ ] Documentation
[ ] Tests

For all the boxes checked, please include additional details of the changes made in this pull request.
Testing Done

[ ] Manually Tested on local docker setup. Please include commands ran, and their output.
[ ] Added new tests for the changes made.
[ ] Updated existing tests to reflect the changes made.
[ ] No tests added or updated. Please explain why. If unsure, please feel free to ask for help.
[ ] Some other form of testing like staging or soak time in production. Please explain.

Gradle location publication works:
anath1@anath1-mn1 openhouse % ./gradlew -V publishToMavenLocal -Pversion=1.0.10-snapshot

Here is the locally published pom file.
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <!-- This module was also published with a richer model, Gradle metadata,  -->
  <!-- which should be used instead. Do not delete the following line which  -->
  <!-- is to indicate to Gradle or any Gradle module metadata file consumer  -->
  <!-- that they should prefer consuming it instead. -->
  <!-- do_not_remove: published-with-gradle-metadata -->
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.linkedin.openhouse</groupId>
  <artifactId>openhouse-java-runtime</artifactId>
  <version>1.0.10-snapshot</version>
  <name>OpenHouse</name>
  <description>Control Plane for Tables in Open Data Lakehouse</description>
  <url>https://github.com/linkedin/openhouse</url>
  <licenses>
    <license>
      <name>BSD 2-Clause License</name>
      <url>https://raw.githubusercontent.com/linkedin/openhouse/main/LICENSE</url>
    </license>
  </licenses>
  <scm>
    <connection>scm:git:git://github.com/linkedin/openhouse.git</connection>
    <developerConnection>scm:git:ssh://github.com/linkedin/openhouse.git</developerConnection>
    <url>https://github.com/linkedin/openhouse</url>
  </scm>
  <dependencies>
    <dependency>
      <groupId>org.mapstruct</groupId>
      <artifactId>mapstruct</artifactId>
      <version>1.5.0.Beta1</version>
      <scope>runtime</scope>
    </dependency>
    <dependency>
      <groupId>javax.servlet</groupId>
      <artifactId>javax.servlet-api</artifactId>
      <version>4.0.1</version>
      <scope>runtime</scope>
    </dependency>
    <dependency>
      <groupId>org.apache.commons</groupId>
      <artifactId>commons-lang3</artifactId>
      <version>3.12.0</version>
      <scope>runtime</scope>
    </dependency>
    <dependency>
      <groupId>org.apache.iceberg</groupId>
      <artifactId>iceberg-core</artifactId>
      <version>1.2.0</version>
      <scope>runtime</scope>
    </dependency>
    <dependency>
      <groupId>org.apache.hadoop</groupId>
      <artifactId>hadoop-client</artifactId>
      <version>2.10.0</version>
      <scope>runtime</scope>
      <exclusions>
        <exclusion>
          <artifactId>servlet-api</artifactId>
          <groupId>javax</groupId>
        </exclusion>
        <exclusion>
          <artifactId>commons-lang3</artifactId>
          <groupId>org.apache.commons</groupId>
        </exclusion>
        <exclusion>
          <artifactId>junit</artifactId>
          <groupId>junit</groupId>
        </exclusion>
        <exclusion>
          <artifactId>*</artifactId>
          <groupId>io.netty</groupId>
        </exclusion>
        <exclusion>
          <artifactId>HikariCP-java7</artifactId>
          <groupId>com.zaxxer</groupId>
        </exclusion>

Verified using spark shell and the table operation works.
anath1@anath1-mn1 openhouse % docker exec -it local.spark-master /bin/bash
openhouse@b57e76233ba7:/opt/spark$ bin/spark-shell --packages org.apache.iceberg:iceberg-spark-runtime-3.1_2.12:1.2.0   \
>   --jars openhouse-spark-runtime_2.12-*-all.jar  \
>   --conf spark.sql.extensions=org.apache.iceberg.spark.extensions.IcebergSparkSessionExtensions,com.linkedin.openhouse.spark.extensions.OpenhouseSparkSessionExtensions   \
>   --conf spark.sql.catalog.openhouse=org.apache.iceberg.spark.SparkCatalog   \
>   --conf spark.sql.catalog.openhouse.catalog-impl=com.linkedin.openhouse.spark.OpenHouseCatalog     \
>   --conf spark.sql.catalog.openhouse.metrics-reporter-impl=com.linkedin.openhouse.javaclient.OpenHouseMetricsReporter    \
>   --conf spark.sql.catalog.openhouse.uri=http://openhouse-tables:8080   \
>   --conf spark.sql.catalog.openhouse.auth-token=$(cat /var/config/$(whoami).token) \
>   --conf spark.sql.catalog.openhouse.cluster=LocalHadoopCluster
:: loading settings :: url = jar:file:/opt/spark/jars/ivy-2.4.0.jar!/org/apache/ivy/core/settings/ivysettings.xml
Ivy Default Cache set to: /home/openhouse/.ivy2/cache
The jars for the packages stored in: /home/openhouse/.ivy2/jars
org.apache.iceberg#iceberg-spark-runtime-3.1_2.12 added as a dependency
:: resolving dependencies :: org.apache.spark#spark-submit-parent-59ba3ffc-81dd-4a58-a773-77c90adc3609;1.0
	confs: [default]
	found org.apache.iceberg#iceberg-spark-runtime-3.1_2.12;1.2.0 in central
downloading https://repo1.maven.org/maven2/org/apache/iceberg/iceberg-spark-runtime-3.1_2.12/1.2.0/iceberg-spark-runtime-3.1_2.12-1.2.0.jar ...
	[SUCCESSFUL ] org.apache.iceberg#iceberg-spark-runtime-3.1_2.12;1.2.0!iceberg-spark-runtime-3.1_2.12.jar (3484ms)
:: resolution report :: resolve 770ms :: artifacts dl 3486ms
	:: modules in use:
	org.apache.iceberg#iceberg-spark-runtime-3.1_2.12;1.2.0 from central in [default]
	---------------------------------------------------------------------
	|                  |            modules            ||   artifacts   |
	|       conf       | number| search|dwnlded|evicted|| number|dwnlded|
	---------------------------------------------------------------------
	|      default     |   1   |   1   |   1   |   0   ||   1   |   1   |
	---------------------------------------------------------------------
:: retrieving :: org.apache.spark#spark-submit-parent-59ba3ffc-81dd-4a58-a773-77c90adc3609
	confs: [default]
	1 artifacts copied, 0 already retrieved (26650kB/71ms)
2024-08-07 16:57:25,930 WARN util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
Spark context Web UI available at http://b57e76233ba7:4040
Spark context available as 'sc' (master = local[*], app id = local-1723049851016).
Spark session available as 'spark'.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /___/ .__/\_,_/_/ /_/\_\   version 3.1.1
      /_/

Using Scala version 2.12.10 (OpenJDK 64-Bit Server VM, Java 1.8.0_232)
Type in expressions to have them evaluated.
Type :help for more information.

scala> spark.sql("CREATE TABLE openhouse.db.tb (ts timestamp, col1 string, col2 string) PARTITIONED BY (days(ts))").show()
++
||
++
++

scala> spark.sql("DESCRIBE TABLE openhouse.db.tb").show()
+--------------+---------+-------+
|      col_name|data_type|comment|
+--------------+---------+-------+
|            ts|timestamp|       |
|          col1|   string|       |
|          col2|   string|       |
|              |         |       |
|# Partitioning|         |       |
|        Part 0| days(ts)|       |
+--------------+---------+-------+

scala> spark.sql("INSERT INTO TABLE openhouse.db.tb VALUES (current_timestamp(), 'val1', 'val2')")
res2: org.apache.spark.sql.DataFrame = []

scala> spark.sql("SELECT * FROM openhouse.db.tb").show()
+--------------------+----+----+
|                  ts|col1|col2|
+--------------------+----+----+
|2024-08-07 16:58:...|val1|val2|
+--------------------+----+----+

For all the boxes checked, include a detailed description of the testing done for the changes made in this pull request.
Additional Information

[ ] Breaking Changes
[ ] Deprecations
[ ] Large PR broken into smaller PRs, and PR plan linked in the description.

For all the boxes checked, include additional details of the changes made in this pull request.

Please update the testing done section with test details for local docker.

Yes updated.


Do not omit hash on initial page load when using History API

Hello! With the browser plugin enabled, and when it's useHash option is set to false IMO router should not omit the hash part of the url on initial page load. Now it is impossible to reload a page with the hash in the url - hash gets removed on reload.
It happens here, I believe - https://github.com/router5/router5/blob/master/modules/plugins/browser/browser.js#L31
Thanks!

Yep, it is a request I've had before, will look at it.

Hello. Is there any progress on it? Thanks!

Unfortunately, I haven't had the time to look at it. Will do at some point though, unless someone beats me to it with a PR :)

I think I found the causing code, will try to send a PR today.

Should the hash be appended on replacing state as well? Now the hash being preserved only with start state, I think it makes sense to keep it on transitions from 'popstate' event too. What do you think?

That would make sense.


update: check-in test data, add new test options, reduce test flakyness for Falco

Adds support for setting a Context and environment variables for runners.
Checks in test data (conf files, rules files, outputs) that used to be downloaded and generated with go generate. This makes the codebase easier to navigate and allows customizing the test data, which was previously taken as-is from old commits of the Falco repo.
Reduces the flakyness of Falco tests regarding strict checks on the outputs (stdout, file, program), and improves the execution of the gRPC tests

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 7066bef3884a414d4ccabb53ef16086caa8f1c5f


BUG:limit cftime to <=1.1.0 for all but Python3.5

[x] Closes #944
[x] I am familiar with the contributing guidelines
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Limiting cftime<=1.1.0 . This is a few releases ahead of the suggested 1.0.4 but still passes all tests.

It's good that everything checked out on this run. I reran the failing tests from #937 and they still fail confirming that this is indeed a fix.
I'll add to what's new and this should be ready for approval thereafter.

This is ready for review.
This is by no means the solution and an issue to address needs to be drafted unless #920 encompasses it. I'll make the issue unless If someone who knows the inner workings of forecast.py and/or how this interacts with it can describe what needs to be done.

As proof of the error I pushed after reverting this branch to master to get the test results. Link to test results
The error is fixed in Python3.6 and above (thanks @kanderso-nrel!) but persists in Python3.5 even with the changes. Link to 3.5 error
cftime only supports >=3.6 and therefore doesn't have the kwarg only_use_python_datetimes for the downloaded version in the 3.5 test.
The Test_conda_linux environment for Python3.5 installs 1.0.0b1. Link to installed packages for 3.5 test
All other Python versions install the most current 1.1.1 Link to installed packages for 3.6 test
The most obvious options are:

Go back to pinning 1.1.0
Drop support for Python3.5

Are there any more options?

Pinning is ok with me, but Another option is to xfail or skip the test if run on 3.5. I’d rather wait until 0.8 to drop 3.5.

I think this, after much learning, is ready for review.
Thanks for your patience for the time it took and notification spam.
Please let me know if the codecov/patch failure needs to be addressed for test. I don't think so but there may be some historical context in which I'm unaware.

ok to ignore this codecov/patch failure. Ok with me to just let it break on python 3.5 if we're no longer supporting it in pvlib 0.8.

When you get a moment @wholmgren can you please merge this? Thanks!


Support 64bit HLSL atomic functions

Resolves #3951
This adds a few atomic functions for SM6.6.
The spec can be found from here:
https://microsoft.github.io/DirectX-Specs/d3d/HLSL_SM_6_6_Int64_and_Float_Atomics.html

seeing the PR was merged:
Shader capabilities will need to be filled, WIP PR https://github.com/shader-slang/slang/pull/3907 will handle this.
to add GLSL/SPIR-V/CUDA we will likely restructure the code to allow reuse with glsl.meta atomic code (supports int64).

@ArielG-NV Yeah we need to change glsl.meta.slang to call into the HLSL functions instead.

I just saw this issue. Did i read correctly that this does not implement 64 bit atomics for spirv out? I need 64 bit atomics for spirv out.

This implements for hlsl 64 bit atomics.

for example: void InterlockedAdd64(uint byteAddress, int64_t valueToAdd) supports glsl/spirv

glsl syntax (allow-glsl flag) supports glsl/spirv atomics fully for whatever glsl implemented.


Might & Magic IX (#134 continuation)

Describe the bug
The game runs now, but:

You have to change the native resolution (800x600) to 1920x1080 manually in autoexec.cfg, otherwise the game runs in a cropped window:

The game doesn't seem to change in terms of graphics, though the Alt + X menu works:

d3d9.log
lithtech_d3d9.log
NvRemixBridge.log
How do you reproduce the bug?
Just run the game.
What is the expected behavior?
At least some graphical changes. The game looks absolutely the same now (except for the cropped window).
Version
0.5.0

REMIX-3087 to track the resolution issue. Issue #2 may be a problem with setup; if you haven't already, we suggest checking with the Discord community to see if anyone there can help.

@an00biz , was the resolution issue resolved?


TypeError: linear(): argument 'input' (position 1) must be Tensor, not ReLU

I've made it to 12:54 in the 3rd part of his series, and I have come to a roadblock in the code. I cannot find a way to work around it. Any advice or potential fixes help. Thanks!
Output
[nltk_data] Downloading package punkt to
[nltk_data]     C:\Users\AlienUser\AppData\Roaming\nltk_data...
[nltk_data]   Package punkt is already up-to-date!
Traceback (most recent call last):
File "c:\Users\AlienUser\Documents\PyTorchBot\train.py", line 85, in
outputs = model(words)
File "C:\Users\AlienUser\Documents\PyTorchBot\venv\lib\site-packages\torch\nn\modules\module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "C:\Users\AlienUser\Documents\PyTorchBot\venv\lib\site-packages\torch\nn\modules\module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "c:\Users\AlienUser\Documents\PyTorchBot\model.py", line 15, in forward
out = self.l2(out)
File "C:\Users\AlienUser\Documents\PyTorchBot\venv\lib\site-packages\torch\nn\modules\module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "C:\Users\AlienUser\Documents\PyTorchBot\venv\lib\site-packages\torch\nn\modules\module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "C:\Users\AlienUser\Documents\PyTorchBot\venv\lib\site-packages\torch\nn\modules\linear.py", line 116, in forward
return F.linear(input, self.weight, self.bias)
TypeError: linear(): argument 'input' (position 1) must be Tensor, not ReLU
Link to Video: https://www.youtube.com/watch?v=Da-iHgrmHYg

Can you share your model.py code? Maybe there is some problem in it.

import torch
import torch.nn as nn
class YourModel(nn.Module):
def init(self, input_size, hidden_size, output_size):
super(YourModel, self).init()
self.l1 = nn.Linear(input_size, hidden_size)
self.relu = nn.ReLU()  # ReLU activation function
self.l2 = nn.Linear(hidden_size, output_size)
def forward(self, x):
    out = self.l1(x)
    out = self.relu(out)  # Apply ReLU activation
    out = self.l2(out)
    return out

Create an instance of your model
model = YourModel(input_size, hidden_size, output_size)
Assuming 'words' is your input tensor
outputs = model(words)


add missing code for mapping SignalR

Fixes #136

Nice catch! Thank you for taking the time to fix it.


Selection Doesnt get updated in newest VSCode

When you start your sequence and update the formatting string, the selection gets lost and new strings get appended to the old ones.

I think there was a change in vs code that causes that.

調査メモ: 選択が無い状態で builder.replace を読んだ後、置換後の文字列が選択状態にならなくなた事が原因。


Fetch- timestamp [2]

Part of fetch. Timestamp function is working for SB, but there is more to be done for timestamping done on local files, URLs, or other methods.
Rather than downloading the file each time, timestamp is inspected and service is checked to determine if there's a change. timestamp.sciencebase function does this for SB.
Needs to be tested and verified. There are three Fetch functions implemented, for each of those there's a timestamp function that needs to be checked.
Fetch: File, SB, and URL functions.
URL needs to be implemented, SB needs to be tested and verified. File needs to be tested.

timestamp for file is taken care of. URL timestamp is more complicated.
code to retrieve timestamp is in place, but nothing is using it.

[ ] override for timestamp
[ ] The logic needs to be put in place to check the timestamp.
[ ]  URL timestamp

The functions themselves should be good to go in #52, but some other issues:

For local files, there is some redundancy in checking the timestamp, since make already checks the modified tag itself when the data file is listed as a dependency.  Do we want to only use the timestamp as a dependency to avoid this?

What happens when fetchTimestamp is false in the viz.yaml?  It skips the fetch, but what if as in the example viz iris-data there is no processed file cached?  If you wanted to change the visualization step, the fetch would still have to happen.  Should there be 'skip' flags in each section, so that they can be turned off if you want?

An example with a local file that gets processed would make the things☝  easier to test

More examples in the Process script would help to show this works. Add another figure to the test viz that accesses something remote. Cuyahoga example.

#61 should finish this.  Also made a PR to the example viz repo to add a sciencebase block https://github.com/USGS-VIZLAB/example/pull/7

fixed in #61


Updated the README.md

Fixes Issue
This PR fixes the following issues:
closes #66
Made all the changes listed in #66 and added some extra features

This looks great! It is GTG. 🚀

Thanks!

@siddhigate Thanks for the reply
Is there any other way to contribute to WeMakeDevs ??
Like any other opportunities apart from GitHub?


How are HPE OneView event types structured?

How are events organized at HPE OneView resource model and how can they be mapped to Redfish schema?
Check python-hpOneView example code to connect to SCMB and receive events.

How should I respond to this? Here, in a TXT doc, in MD doc?

Well... at least a comment here should be enough. The goal here is to answer:

Does each OneView event type matches one Redfish event type?
Can we use a single class to map them or we need a hierarchy?
Which SCMB queues should we subscribe to?

From the last meeting's discussion:

for ResourceUpdated, ResourceAdded and ResourceRemoved: subscribing to scmb.*.Created.# in order to get all resource creation, for example, will result in receiving a lot of undesired events for Alert and Tasks resources among others. Our best idea is to subscribe to each OneView resource of interest independently, i.e.; subscribing to scmb.server-hardware.Created.#, scmb.enclosure.Created.#, scmb.rack.Created.#, and so on.
for Alert: subscribing to scmb.alert.# and then filtering by the associated resources of interest is the best plan to this kind of event.
for StatusChange: we have not yet identified which SCMB topic will lead to this events, see #226 for more info.

https://github.com/HewlettPackard/oneview-redfish-toolkit/blob/master/oneview_redfish_toolkit/api/scmb.py


feat: add has_training_resource to standards with training programs

Updated records in DataStandardOrTool.yaml with new has_training_resource slot to refer to associated records that are considered training programs (see category: B2AI_STANDARD:TrainingProgram)
Updated a record that was a DataStandardOrTool and made it a TrainingProgram (youtube tutorials for FHIR. See id: B2AI_STANDARD:845)
Updated modify_synapse_schema.py script with new column value + tidied up with ruff format (NOTE: THIS SCRIPT WILL NEED TO BE RUN MANUALLY BEFORE THIS PR IS MERGED OR THE AUTOMATED GITHUB ACTION TO UPDATE THE SYNAPSE TABLES WILL NOT WORK, AS IT DOES NOT SUPPORT SCHEMA CHANGES)
Updated readme for above since the og one was telling users to export auth tokens as environment variables (old me was in a rush :-) )
Pulled in updated schemas from standards-schemas

Thanks!
Have you already run the modify_synapse_schema.py? I tried running it locally but I'm missing the Synapse auth token.

Thanks! Have you already run the modify_synapse_schema.py? I tried running it locally but I'm missing the Synapse auth token.

Yep! I ran it right after I opened this PR :) so this should be all good to go


SSH-Key file path save fail

Version: 1.0.91
Platform: win32 10.0.18362
Plugins: none
SSH Key file path save no response

Possible duplicate of #1540


Expose fastly rt api 'recorded' timestamp in prometheus metrics

The fastly real-time api includes a recorded field in the response, which is a timestamp of when a metric was generated. This is currently represented in the APIResponse struct.
Prometheus can be asked to honour the metric timestamps seen when scraping a target.
And the prometheus golang client allows for the creation of metrics with timestamps.
This issue is to update the fastly-exporter to use the recorded field as the metric timestamp when converting fastly metrics to prometheus ones. Doing this could help keep fastly metrics from being offset from other scrape targets.

This is meaningful if the timestamp of a given metric is substantially different than the current time. But in the case of the Fastly real-time API, this should never be the case: there will naturally be some delay, but it will be on the order of a few seconds at most. So it's not obvious to me what this would accomplish. Are you experiencing a problem that this would solve?

We are looking at moving from another implementation of a fastly-exporter to this one, and noticed that metric timestamps could be skewed. I'll close the issue, and follow-up if we run into a problem that this would solve ^^.

OK! I'm happy to help if there does turn out to be an issue.


Don't require tags for inclusion in lockfile

Currently jpm can prevent some dependencies from being included in lockfile, e.g:
(declare-project
  :name "project1"
  :description "project1"
  :dependencies [
                 {:url "https://github.com/Jakski/http/archive/20c1787177243c213c0aafd37bf41eb3cfe30c83.tar.gz"
                  :type :tar}])

$ rm -rf jpm_tree/ lockfile.jdn
$ jpm -l deps
...
$ jpm -l make-lockfile
Cannot add local or malformed package /opt/jpm_tree/lib/.manifests/http.jdn to lockfile, skipping...
created lockfile.jdn

Lockfile resulting from above command won't actually install all dependencies with load-lockfile. It makes lockfiles useless while working with tar archives, because it's still required to run jpm deps in order to get project working.

LGTM


msg_send without explicit return types will break if/when ! is stabilized.

Code like this has an unconstrained return type.
msg_send![view, retain];

This presently will fall back to deducing () as the return type. Previous attempts to stabilize the ! type have involved unconstrained return types deducing to !. This has directly caused UB in winit https://github.com/rust-windowing/winit/pull/428 during both attempts to stabilize the never type.
https://github.com/rust-lang/rust/issues/48950#issuecomment-380603530
One recommended fix is to change the code to explicitly deduce the return type as () where appropriate.
let () = msg_send![layer, setBounds: bounds];

The first example is already UB because the return type of retain is *mut c_void but is currently being deduced as ().
So that should be changed to:
let _: *mut std::raw::c_void = msg_send![view, retain];

I recall us hitting this problem on Nightly at some point. Guess there is no way around it.
Also related to #1972


[#NOISSUE] cleanup permission code.

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

Codecov Report

Merging #5306 into master will increase coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #5306      +/-   ##
==========================================
+ Coverage   39.85%   39.88%   +0.03%
==========================================
  Files        2717     2717
  Lines       82273    82256      -17
  Branches    10992    10987       -5
==========================================
+ Hits        32786    32807      +21
+ Misses      46555    46517      -38
  Partials     2932     2932

Impacted Files
Coverage Δ

...p/pinpoint/web/controller/UserGroupController.java
0% <0%> (ø)
:arrow_up:

...corp/pinpoint/web/dao/mysql/MysqlUserGroupDao.java
0% <0%> (ø)
:arrow_up:

...rcorp/pinpoint/web/controller/AlarmController.java
0% <0%> (ø)
:arrow_up:

...rp/pinpoint/web/dao/memory/MemoryUserGroupDao.java
27.16% <0%> (-0.69%)
:arrow_down:

...orp/pinpoint/web/service/UserGroupServiceImpl.java
11.76% <0%> (+2.67%)
:arrow_up:

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75% <0%> (-3.13%)
:arrow_down:

...corp/pinpoint/rpc/stream/StreamChannelManager.java
72.41% <0%> (+1.72%)
:arrow_up:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0%> (+2.43%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Other linux distributions and support for ubuntu versions?

Hi I have a question about compatibility other than Linux 20.04. I have seen similar issues mentioning 22.04 will be tested soon soon, has this happened yet?
I'm also curious as to what is limiting BYOH to ubuntu? I see a DoUbuntu20_4K8s1_22 install script in the repo. Is this what limits support to ubuntu or are there any other dependencies? Would other debian based distributions be a safe bet? Was this an administrative decision to limit maintenance/testing overhead? I'm trying to understand  what the limiting factors are here before using it more widely.
I've tried it out and love it's simplicity compared to other infrastructure providers!

Hi @keshavcruise, the current installer interface is extensible and we could extend support for various OS. internal/algo could have implementations for other OS/Versions. We would be happy to accept any community contributions to enable support for other OSes.
Note that, if you have other means of installing the K8s components in your hosts, you could also use --skip-installation flag to skip the installer completely.

Can you describe how to do this ?
I would like to add support for Debian

@knfoo This can be done by passing --skip-installation flag while starting your host agent

@keshavcruise I guess I was unclear I already use --skip-installation to run on Debian.
I am more interested in some instructions on how to create the logic to support Debian via the https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/tree/main/installer/internal/algointernal/algo that @dharmjit mentioned.
When looking at the ubuntu20_4k8s.go file it looks like it expects that a bundle is stored somewhere outside this repo (github.com/vmware-tanzu/carvel-imgpkg/releases/download/$IMGPKG_VERSION/imgpkg-linux-$ARCH).
So I am asking about how to contribute to this flow and create bundle for a different OS.

Hi @knfoo, You are correct that we need to create a new carvel imgpkg bundle for supporting a new OS/version and this bundle needs to be published in OCI registry. The code for the same is in https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/tree/main/installer/bundle_builder. I think that the bundle for Ubuntu 20.04 and Ubuntu 22.04 will be same and we could just rename the same bundle.


適用対象no

適用対象の OS が異なるため以下の通り変更が必要です
【変更前】

適用対象
•	Windows 10
•	Windows 10 Mobile
•	Microsoft Edge

【変更後】

適用対象
•	Windows 10
•	Windows 10 Mobile
•	Microsoft Edge
•	Windows Server 8.1 以降
•	Windows Server 2012以降

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 4edaae97-455b-d25a-9228-5309e24b4aa3
Version Independent ID: d38e56a7-d553-e519-95a9-dc93e2cdc771
Content: Microsoft Defender SmartScreen の概要 (Windows 10) - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-smartscreen/microsoft-defender-smartscreen-overview.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Translated via online MT (Bing/Google):

Applies to no
Changes are required as follows because the applicable OS is different
【Before change】
Applies to
• Windows 10
• Windows 10 Mobile
• Microsoft Edge
【After change】
Applies to
• Windows 10
• Windows 10 Mobile
• Microsoft Edge
• Windows Server 8.1 or later
• Windows Server 2012 or later

Applicable target no
Since the applicable OS is different, the following changes are required.
【Change before】
Applicable target
• Windows 10
• Windows 10 Mobile
• Microsoft Edge
[After change]
Applicable target
• Windows 10
• Windows 10 Mobile
• Microsoft Edge
• Windows Server 8.1 or later
• Windows Server 2012 or later

As far as I know, "Windows Server 8.1" does not exist.
Microsoft Windows 8.1 is a version of Microsoft Windows for Desktop/Laptop/Notepad use, just like Windows 10.
Other than that, it looks like you are asking for the addition of Windows Server 2012 or later to be added to the page. This probably needs to be added in the EN-US version of the page first, because that page is the main source of automatic Machine Translation:

https://docs.microsoft.com/en-us/windows/security/threat-protection/microsoft-defender-smartscreen/microsoft-defender-smartscreen-overview
https://github.com/MicrosoftDocs/windows-itpro-docs/blob/public/windows/security/threat-protection/microsoft-defender-smartscreen/microsoft-defender-smartscreen-overview.md

Direct Link to open a New issue Ticket for the /en-us/ page (US English):

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/new?title=&body=

[Enter feedback here]

---
%23%23%23%23 Document Details

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

* ID%3A 4edaae97-455b-d25a-9228-5309e24b4aa3
* Version Independent ID%3A d38e56a7-d553-e519-95a9-dc93e2cdc771
* Content%3A [Microsoft Defender SmartScreen overview (Windows 10) - Windows security](https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fsecurity%2Fthreat-protection%2Fmicrosoft-defender-smartscreen%2Fmicrosoft-defender-smartscreen-overview)
* Content Source%3A [windows%2Fsecurity%2Fthreat-protection%2Fmicrosoft-defender-smartscreen%2Fmicrosoft-defender-smartscreen-overview.md](https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fblob%2Fpublic%2Fwindows%2Fsecurity%2Fthreat-protection%2Fmicrosoft-defender-smartscreen%2Fmicrosoft-defender-smartscreen-overview.md)
* Product%3A **m365-security**
* Technology%3A **mde**
* GitHub Login%3A %40mjcaparas
* Microsoft Alias%3A **macapara**

(This is to let the link & ticket creation insert most of the ticket content directly from the link itself.)

Never mind. The repositories are not different, the only difference is the source pages taking us users here to GitHub when the ticket is opened.

I guess the main point here is to check if the source page needs to be updated regarding which OS versions (Server/Desktop, 2012/8.1/10 and so on).

Closing via https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9199#issuecomment-786694081


mime/multipart: CreateFormFile does not allow setting the Content-Type for the file.

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

Go 1.6.2

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH=""
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

I used mime/multipart CreateFormFile but was unable to set the Content-Type for the part.

What did you expect to see?

Content-Disposition: form-data; name="photo"; filename="photo.jpg"
Content-Type: image/jpeg
xxxxxxxxxxxxxxxxxxxxxxx

What did you see instead?

Content-Disposition: form-data; name="photo"; filename="photo.jpg"
Content-Type: application/octet-stream
xxxxxxxxxxxxxxxxxxxxxxx

Use CreatePart.
CreateFormFile is a very thin wrapper around CreatePart. Click https://golang.org/pkg/mime/multipart/#Writer.CreateFormField and then click the CreateFormFile heading to see its source code.

I did that. It just seems to me like an absolutely necessary feature to make the method useful at all.

Where do you draw the line? Just content type? Content type and disposition? And Filename? All permutations thereof? Method for each?
Everybody's opinions on the most useful parts vary, I'm sure.
If you come back with some data suggesting that most users need a certain API which we don't provide and everybody is doing a tedious thing we could make easier, then I think we can make changes.
See https://medium.com/google-cloud/analyzing-go-code-with-bigquery-485c70c3b451#.9mr0dcybl for how to do some SQL queries against all code on Github.
/cc @campoy

will do.


Append to Pass Kernel Options

Sadly I wasn't able to find how to add something to Jorge's pull request (#3 )...
Sorry Jorge !
I modified mr_provisioner_machine_provision.py to support kernel opts and added default kernel option as an empty string.

@BaptisteGerondeau Please rebase your changes against a single commit , that way is easier to review.

@BaptisteGerondeau  since https://github.com/Linaro/ansible-role-mr-provisioner/pull/3 has been merged, please rebase your changes with the latest master.

This looks like a good change to me except for the one small nit on what looks to me to be an errant 'else'.

Deleted the else statement and made the kernel_opts default to "" instead, in the function declaration.
(Sorry I had completely missed that comment...)


fix github workflows

This is a work around the github limitation that a workflow can't be triggered by another one.
The dist-ci workflow was not triggered on PRs generated by the hints-and-dist workflow.
See https://github.com/peter-evans/create-pull-request/issues/48.

FYI: you can trigger workflows from other workflows. see https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_call


Amazon Echo cannot detect custom devices

I have Fibaro HC2 and successfully create HA Bridge with some devices.  Using HA bridge, I can turn on/off devices but I got stuck at Echo Discovering devices.  Echo simply cannot see the devices that I had added.  HC2 is running on cable and Echo is running on wireless.  HC2 can see alot of uPNP traffic in my home network but do not see anything coming from Echo.

When Echo discover devices, does it use uPNP protocol?
Does the discovery comes from Echo inside LAN or from Internet AWS cloud or something?  I have Sophos firewall at my gateway and had disabled everything from my Notebook including firewall and AV.
How can I get Echo see the devices I manually added?

I had been stuck in this for weeks.  Hopefully can have some expert here to advise.  Thank you.
Regards,
Javen

Windows and Mac version not working well for me.  Issue resolved after switching to Linux HA Bridge

I'm in the same position, Echo doesn't discover any of my devices listed in "Bridge Devices".  I can manually click on the buttons to control them from ha-bridge.
What did you do to fix? (I am already on Linux)

@Pete Please post your ip address of your bridge and then your configuration of what you are using for the upnp ip address. Also, post your ha-bridge version. Thanks


Look for metadata file in different versions of segment directory

V1 to v3 format converter deletes stale files after conversion. But parts
of code like segment refresh messages, look for segment metadata before
it can figure out the format to load. Looking for segment metadata
prevents refreshing converted segments even if stale metadata is deleted.
The order of loading metadata file is for backward compatibility.
The behavior to handle missing metadata file is same as before PR# 609

Updated existing UploadRefreshDelete integration test to test for v3 segment. This test did not fail with the older code - although we throw exception, server downloads the segment again. This caused scalability issue on restart if the server had thousands of segments.
This test needs more work to verify that server does not trigger segment download during refresh. I'll submit that in separate PR

Current coverage is 64.22% (diff: 88.88%)

No coverage report found for master at 78141ed.

Powered by Codecov. Last update 78141ed...911e1fc


Improve formatCurrency short form symbol formatting (with exceptions)

Description
Fixes #2304
Part of https://github.com/Shopify/web/issues/70196

Modifies i18n.formatCurrency short form to use narrowSymbol for currencyDisplay instead of the default symbol. The changes due to this modification are shown on this spreadsheet, which lists all the supported currencies and the "before" and "after" outputs.
Modifies i18n.getCurrencySymbol to allow users to pass in the Intl.NumberFormat options of their choice.
Creates exceptions for BRL (Brazilian Real) and HKD (Hong Kong Dollar), which should show the full-length symbol across Shopify (R$ and HK$, respectively).
Exposing i18n.getShortCurrencySymbol has already been handled here.

For auditing purposes, here's the Spreadsheet containing the "before" and "after" states of each supported currency given the modification to the formatCurrency short form.

Looks good! Before shipping, I recommend that you create a beta release (you can ask someone on Admin Web Platform to help with this).

Ohh that's easy. We have a new toy. Check this out:

/snapit

/snapit

Now this is awesome!


UI bug: receive address table goes blank

Reproduce case:

Go to "Addresses" tab to ascertain the list of receive adresses are shown. Remember the last one in the list. Scroll to the top of the list before step 2.
Go now to the "Receive" tab and press "Get New Address" enough times to be sure you exceed the addresses already shown in previous list.
Go to "Addresses" tab again. List is empty (until scroll).

Fixed in c067300.


Rerender with conda-smithy with explicit secrets

to test https://github.com/conda-forge/conda-smithy/pull/1343

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


stop checking for sublime-syntax files (v < 3092)

Usage numbers for Sublime Text 2 are really low and users cannot expect packages to still care if recent releases still work. Ie. nobody updates their package control entry to tag releases specifically for ST2. For ST3, 3092 is over 5 years old. Most syntax package developers fully switched to sublime-syntax files an I'd expect most will simply have dropped the ancient tmLanguage files without updating their package control entries. I don't want to bother package contributors with compatibility here: who still uses build 3083 but also downloads new packages from package control?
The build fails, but it doesn't look like something I can fix. I tried pulling in the crypto upgrade (#35) to no avail. error: can't find Rust compiler is among the errors 🤷🏻

I would prefer to move this check (and the one for .sublime-color-scheme?) to report the minimum required build in a separate field that other tooling, such as the PCC review bot, could process directly. The current handling of this is quite suboptimal imo.

Sure, we can always be more ambitious. Are there any real use cases that warrant the effort though?
I would like to just get rid of the useless noise in reviews that is always entirely ignored, because it is truly irrelevant if a new package works for ST builds < 3902. And would do the same for sublime-color-scheme.

My current plan is to determine the minimum version of the package as specified in the repository and pass that as a parameter to the reviewer tool.
Just as a heads-up: merging this PR won't have any effect on the review bot's output because the code that is used by the bot is vendored inside the packagecontrol.io repo. In order to get updates with the review, we first need to switch to running the checker tool using GitHub Actions entirely. Surely at some point I'll be able to work on this …

Right, moving the tests to actions would be a priority then. Maybe we can find a way to get someone who actually has time for these things on board? Then we can maybe do that first and think about additional ambitions later.
Anyway, I’ll close this then, because it’s not going anywhere this way and it seems like we need focus and time more than we need ideas 😉


Create xp-relic-sound plugin

Add Plugin that has a random chance of playing the leagues relic sound each time you gain xp.

how often the sound triggers is configurable as well as if maxed skills should be used

Don't use "example" as your plugin's config group

updated config group to xprelicsound


Unexpected expansion of parameter when working with lists of lists

We would like to use mpire to parallelize processing. We have lists of lists (batches, with n elements each) and use something like this
from mpire import WorkerPool

def print_for_me(file_list: List[str]):
    for file in file_list:
        print(f"Files: {file}")

batches = [["as", "bd"], ["bc", "dc"]]
# This does not work
with WorkerPool(n_jobs=2) as pool:
    pool.map(print_for_me, batches)

# This works, but it super weird to read
with WorkerPool(n_jobs=2) as pool:
    pool.map(print_for_me, zip(batches,))

While writing this issue, I had another idea and maybe this is the proper way how to map lists of lists with mpire:
from mpire import WorkerPool

def print_for_me2(*file_list):
    for file in file_list:
        print(f"Files: {file}")

batches = [["as", "bd"], ["bc", "dc"]]
with WorkerPool(n_jobs=2) as pool:
    pool.map(print_for_me2, batches)

This maps the list to tuples inside print_for_me2, but at least works as expected. Would it be possible with mpire to preserve the inner list, or is this by design impossible?
If this is the recommended way, would it be possible to add 1-2 examples like these to the documentation? That would be great.

Changing it is possible, but that would potentially break a lot of existing software that rely on this behavior.
There is documentation on this already available: https://slimmer-ai.github.io/mpire/usage/map/map.html#circumvent-argument-unpacking, which also shows how to avoid unpacking (well, it's more of a workaround).
Please let me know if this fixes your current issues.

Yes, that definitely helps. I didn't see that part of the documentation. Thanks a lot!


[WIP] Add a NIRCam notebook

This is for https://jira.stsci.edu/browse/JUSI-10.

@pllim - I might be misunderstanding here, but I think circleci runs on the last commit in a branch not the merge in master.  So I think that means this needs a rebase to actually run on circleci?

Yup, I did a rebase and now it is stuck in a failed DrizzlePac notebook that is unrelated to this PR.

It seems the conda environment is falling back on a previous cached one and not creating a new one as it should with a change to the environment.yml file.
    - restore_cache:
        keys:
        - v1-env-{{ checksum "environment.yml" }}
        - v1-env-

should be
    - restore_cache:
        keys:
        - v1-env-{{ checksum "environment.yml" }}

Looks like it also needs scikit-learn

Could not find conda environment: notebooks_env
@obviousrebel , any idea?

CondaHTTPError: HTTP 502 BAD GATEWA, it ran into a http error when getting a package for the environment. Trying again would (probably) fix it. But it saved the cache so any arbitrary change to the environment.yml file would be needed to create a new environment, maybe adding an extra line.

Sigh, now it is stuck at MAST/HSC/HSC_TAP. This is turning out to be quite painful.

Theoretically, I should squash but given CI is finally green, I don't feel like getting into the situation of waiting for it to run and pass again, so I'll just leave this as is for now until @eteq has a chance to review.

p.s. If you use the "squash and merge" button, that would save me a manual squash and also no need to re-run the CI (again).

@eteq , the rendered page is at https://46-138342879-gh.circle-artifacts.com/0/root/repo/notebooks/JWST/NIRCam/PSF_phot_01/GriddedPSFModel_simulation_and_photometry.html -- acceptable now?

@pllim It appears the STSci logo isn't loading on the rendered html page in the footer (url says the page/link expired). Maybe change the url to: https://raw.githubusercontent.com/spacetelescope/notebooks/master/assets/stsci_pri_combo_mark_horizonal_white_bkgd.png
and we should be all good to merge.

Thanks, @obviousrebel !


make the index.html more concept oriented #GSSoC24

@TeeWrath Requesting to improve the home page.  To improve visual appearance and reduce blurriness on a homepage i.e.  a little image & page optimization. When visitors come to the site for the intended purpose, they should have a pleasant experience.

Please assign this issue under GSSoc24.

hey i would like to contribute on this , plz assign me this issue


Trying to import a book in english results in "s.package is undefined" error

the_sky_crawlers.zip

ty for sharing a test file - it should be fixed on the current site for at least this file. FYI - I removed your comment with the file


Added ability to set api key using env variables for SendGrid adapter

It looks like this functionality was merged in in #342, so I'm going to close this. Thanks!


Supporting of alerts

I'm using this datasource with Grafana but when I'm trying to configure alerts I'm catching the following message:
error:"tsdb.HandleRequest() error Could not find executor for data source type: foursquare-clouderamanager-datasource"

Also Grafana gives me "the datasource doesn't supports alrerting queries"
Do you have the implementation of alerting executor in your Todo-list? It would be good to have this feature. Of course I can adjust alerting in Cloudera Manager but it is quite difficult if I have several Hadoop clusters and different Cloudera instances.

Giving this a try to be fixed

Alarm is still not supported at this time


Allow holding back spans in SpanFilter

As SpanFilter includes "filter" in its name, I expect that not only can spans be modified from it, but also completely undone, or suppressed. One could arguably do this some other way, but I think SpanFilter is the most sensible place, or an interface rather, for doing so.
As of the last commit, if null is returned, CompositeSpanExporter#export will not handle it in any special way, and a NullPointerException will be thrown as it gets converted to OtelFinishedSpan.
https://github.com/micrometer-metrics/tracing/blob/cc6bd1ba8a00eb391081ca7ab19caada54f45f56/micrometer-tracing-bridges/micrometer-tracing-bridge-otel/src/main/java/io/micrometer/tracing/otel/bridge/CompositeSpanExporter.java#L69-L73

If null is returned
java.lang.NullPointerException: Cannot read field "spanData" because "span" is null
	at io.micrometer.tracing.otel.bridge.OtelFinishedSpan.toOtel(OtelFinishedSpan.java:69) ~[micrometer-tracing-bridge-otel-1.2.2.jar:1.2.2]
	at io.micrometer.tracing.otel.bridge.CompositeSpanExporter.lambda$export$0(CompositeSpanExporter.java:71) ~[micrometer-tracing-bridge-otel-1.2.2.jar:1.2.2]
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[na:na]
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:179) ~[na:na]
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1708) ~[na:na]
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) ~[na:na]
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) ~[na:na]
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921) ~[na:na]
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[na:na]
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) ~[na:na]
	at io.micrometer.tracing.otel.bridge.CompositeSpanExporter.export(CompositeSpanExporter.java:72) ~[micrometer-tracing-bridge-otel-1.2.2.jar:1.2.2]
	at io.opentelemetry.sdk.trace.export.BatchSpanProcessor$Worker.exportCurrentBatch(BatchSpanProcessor.java:327) ~[opentelemetry-sdk-trace-1.31.0.jar:1.31.0]
	at io.opentelemetry.sdk.trace.export.BatchSpanProcessor$Worker.run(BatchSpanProcessor.java:245) ~[opentelemetry-sdk-trace-1.31.0.jar:1.31.0]
	at java.base/java.lang.Thread.run(Thread.java:1583) ~[na:na]

Just found SpanExportingPredicate after coming back to this. Maybe it should be mentioned in the SpanFilter documentation?

Glad you found it. Are you interested in opening a PR?
You can find the docs in https://github.com/micrometer-metrics/tracing/tree/main/docs, and build them using ./gradlew antora. I would base it off of the 1.2.x branch and I think adding a note about them somewhere would make sense (basically what their javadoc says).


Update middleware-opa.md

added body to struct and updated HTTPRequest docs to reflect.
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference
https://github.com/dapr/components-contrib/issues/1166
Associated PR: https://github.com/dapr/components-contrib/pull/1165

@richgo, I believe this functionality is targeting the next release v1.5. Please update the docs base branch to be 1.5.

@richgo - following up here, can you please update the docs base branch to be v1.5 for this PR? Once that's done we can review and merge given that the corresponding code PR dapr/components-contrib#1165 has been merged already.

Done 👍