Safari / Firefox : S3 hosted audio file and CORS

We have files hosted on S3, loading them into wavesurfer.
This works fine in Chrome OSX, but Safari and FireFox report
Error decoding audiobuffer
Any ideas?
You can easily reproduce using the wavesurfer website:
http://www.wavesurfer.fm/
In chrome or safari run this on the console
wavesurfer.load('https://s3-eu-west-1.amazonaws.com/soundmites/f5/4779e0c3a111e3b368f97e5bff4d34/coincidence.mp3');

I have this exact same issue; can you speak more directly as to how the file needed to be encoded?

The problem for me was that the audio file was not an MP3 even though I thought it was. The (correct for me) way I encode my files to MP3 is:
cmd = "ffmpeg -y -i #{source_file} -vn -ab #{@mp3_bitrate}k -f mp3 #{mp3_file}   >/dev/null 2>&1"
system(cmd)

before (the wrong way) was:
cmd = "ffmpeg -y -i #{source_file} -f mp2 -b #{@mp3_bitrate}k #{mp3_file}   >/dev/null 2>&1"

Thanks. Spoke to the engineer handling the uploading of the file sound now, but he is adamant that there is nothing he can do beyond what he has done (not 100% clear on what that is just yet). So weird that whatever is happening causes no issues with Firefox or our iOS app but is creating mayhem in Chrome.

What's the output of ffprobe?
ffprobe example/media/demo.wav

Output:
ffprobe version N-69339-gb04cbbe Copyright (c) 2007-2015 the FFmpeg developers
  built on Jan 27 2015 20:40:04 with llvm-gcc 4.2.1 (LLVM build 2336.11.00)
  configuration: --enable-gpl --enable-version3 --enable-nonfree --enable-postproc --enable-libcelt --enable-libfaac --enable-libfdk-aac --enable-libmp3lame --enable-openssl --enable-libspeex --enable-libtheora --enable-libvo-aacenc --enable-libvorbis --enable-libvpx --enable-libx264 --enable-libxvid --prefix=/usr/local
  libavutil      54. 18.100 / 54. 18.100
  libavcodec     56. 21.100 / 56. 21.100
  libavformat    56. 19.100 / 56. 19.100
  libavdevice    56.  4.100 / 56.  4.100
  libavfilter     5.  9.101 /  5.  9.101
  libswscale      3.  1.101 /  3.  1.101
  libswresample   1.  1.100 /  1.  1.100
  libpostproc    53.  3.100 / 53.  3.100
Input #0, wav, from 'example/media/demo.wav':
  Duration: 00:00:21.77, bitrate: 176 kb/s
    Stream #0:0: Audio: pcm_u8 ([1][0][0][0] / 0x0001), 22050 Hz, 1 channels, u8, 176 kb/s

ffprobe version 2.2.2 Copyright (c) 2007-2014 the FFmpeg developers
  built on May  7 2014 16:24:57 with Apple LLVM version 5.1 (clang-503.0.40) (based on LLVM 3.4svn)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/2.2.2 --enable-shared --enable-pthreads --enable-gpl --enable-version3 --enable-nonfree --enable-hardcoded-tables --enable-avresample --enable-vda --cc=clang --host-cflags= --host-ldflags= --enable-libx264 --enable-libfaac --enable-libmp3lame --enable-libxvid
  libavutil      52. 66.100 / 52. 66.100
  libavcodec     55. 52.102 / 55. 52.102
  libavformat    55. 33.100 / 55. 33.100
  libavdevice    55. 10.100 / 55. 10.100
  libavfilter     4.  2.100 /  4.  2.100
  libavresample   1.  2.  0 /  1.  2.  0
  libswscale      2.  5.102 /  2.  5.102
  libswresample   0. 18.100 /  0. 18.100
  libpostproc    52.  3.100 / 52.  3.100
Input #0, wav, from '164_02_25_2015-12_00_39-PST_73393.wav':
Duration: 00:00:03.50, bitrate: 16 kb/s
Stream #0:0: Audio: pcm_s16le ([1][0][0][0] / 0x0001), 1000 Hz, 1 channels, s16, 16 kb/s


Operation not permitted

When i run 'script/setup', I got the message:
script/setup
The Brewfile's dependencies are satisfied.
ln: /Users/zmm/.config//karabiner: Operation not permitted

My soft info:
macOS Sierra 10.12.5 (16F73)
Hammerspoon 0.9.55 (0.9.55)
Karabiner-Elements 0.91.12

@peng051410 Ran into the same probably today. Was able to solve it by removing the existing ~/.config/karabiner folder.

Yup. That's what it was. I think we can close this.

Yes，it works for me!Thanks!


[otp_ctrl/lc_ctrl] Open RTL alignments / security bugfixes

This issue is intended to keep track of the comments on #7150 and the decision whether to move forward with this and related changes.

@cindychip, please pull this PR locally and run full otp ctrl nightly to assess V2 readiness. If our coverage drops below V2 level, we will need to revert the D/V stage and do a version bump. @msfschaffner, i suppose you need to check the same for D2 readiness.
Also, please assess the amount of work to account for this change and lets have the smartsheet updated to reflect that.
@msfschaffner, are there any more changes anticipated in otp ctrl? We really ought to start freezing blocks so that we can make progress on other things in our (huge) list of pending items. Though i do understand it may be hard to foresee it.
@dsandrs, do you agree?
Originally posted by @sriyerg in https://github.com/lowRISC/opentitan/issues/7150#issuecomment-868948337

Thanks Sri for tagging me. I do need to add more checkings for this new valid output ports.
As for DV impact: I ran the DV nightly with this new change. The V1 and V2 tests are all passing. Only failing test is otp_ctrl_tl_intg_err.
Not much coverage impact for V2 either:

SCORE
LINE
COND
TOGGLE
FSM
BRANCH
ASSERT
GROUP

96.11 %
99.24 %
91.16 %
97.73 %
100.00 %
95.40 %
92.81 %
96.44 %

Originally posted by @cindychip in https://github.com/lowRISC/opentitan/issues/7150#issuecomment-869999600

Sorry I should have communicated this better.
This is fixes a security bug.
For context:
The life cycle controller did not check the provisioning status of SECRET2 upon life cycle transitions from PROD*/DEV into RMA, which opens up an undesirable loop hole. In particular:

The HW currently does not  It is not enforced that SECRET2 becomes unreadable by SW when moving through PROD*/DEV into RMA.
Further, a life cycle transition to RMA can be initiated with an all-zero token in certain cases when the SECRET2 partition is not locked right away, e.g.:

when only the root key is provisioned, but not the RMA_UNLOCK_TOKEN token.
when the RMA_UNLOCK_TOKEN is provisioned, but the chip is has not yet been reset.

There are provisioning flow workarounds for this, but it would be good to enforce this check directly in hardware. The SECRET2 partition holds the root key which should not remain readable by SW when transitioning into RMA. Also, there should not exist another way to move from PROD*/DEV into RMA without providing a valid, non-zero RMA token.
Although the change does not look trivial in the diff, it is actually rather localized.
I.e., this change updates the OTP->LC_CTRL connection such that the (readily available) "digest locked status" is output alongside each life cycle token as a redundantly encoded "token valid" signal.
This "token valid" signal is then used as an additional condition when the token comparison is made in the life cycle controller. If a token is not valid / has not been provisioned, this check would fail and abort the life cycle transition.
That way it becomes impossible to move from PROD*/DEV -> RMA without provisioning and locking the SECRET2 partition.
This change does not really add a new feature, but rather fixes existing mechanisms.
As @cindychip confirmed, this change does not impact coverage adversely and can be tested with existing test sequences.

There are currently three other pending changes to OTP in addition to this PR:

Bus integrity alert hookup https://github.com/lowRISC/opentitan/pull/7151
Width alignment of OTBN scrambling nonce https://github.com/lowRISC/opentitan/pull/7144
Increase of background check timer reseed period https://github.com/lowRISC/opentitan/pull/7152

I'll comment more on the corresponding PRs, but 1. is a known enhancement that we have to add to all IPs, and 2./3. are minor adjustments that should not have an adverse impact on DV (these are not new features).
If nothing changes in the provisioning and production test flows, I do not expect additional changes at the moment.
Originally posted by @msfschaffner in https://github.com/lowRISC/opentitan/issues/7150#issuecomment-870106859

The minor alignments #7151 and #7152 have been merged in by now.
The two that are still open are:

#7150
#7152

From a discussion that @cindychip and I had, these changes should not affect V2 status (backed by the coverage statistics posted above), nor introduce significant DV overhead as they are not really new features and can be covered with existing vectors.
Can you confirm this, @cindychip?
Everybody else: are we ok moving forward with these two changes?

The minor alignments #7151 and #7152 have been merged in by now.
The two that are still open are:

[lc_ctrl/otp_ctrl] Check whether a token is valid before performing a transition #7150
[otp_ctrl] Increase LFSR reseeding period #7152

From a discussion that @cindychip and I had, these changes should not affect V2 status (backed by the coverage statistics posted above), nor introduce significant DV overhead as they are not really new features and can be covered with existing vectors.
Can you confirm this, @cindychip?
Everybody else: are we ok moving forward with these two changes?

okay from my side.

Hi Michael. Thanks for documenting it. yes I can confirm both should not affect DV V2 status.

Thanks for the feedback. Can someone approve the two PRs?

OK, the pending items have been merged - closing this one down.


Use a dedicated #define to adjust the CPU clock

As discussed here.
I moved the speed adjustment code from boot() to the constructor. The sooner it gets done, the better. Ideally, it should be done immediately after the bootloader exits.
I added the capability of setting 4MHz, just because. It doesn't hurt anything.

This is broken.  These tests needs to be based on the REAL F_CPU value, and that needs to change.  There are timing related assembly scattered all throughout the Arduino libs that depends on knowing the CPU value the CPU will be running at.  So any board file that runs at 8Mhz (or 4Mhz) would need to pass in the correct F_CPU value and be compiled for that speed or a lot of routines would not work properly.
So what the define should look like is:
#if F_CPU == 8000000L && defined(ARDUBOY_HARDWARE)

Then the board file needs to set a specific flag to indicate that Arduboy board is being used vs say Leonardo. I suggest ARDUBOY_HARDWARE, but open to suggestions.  Perhaps one is already set based on the device name?  I'd have to test compiling with the boards file again.
I don't think anyone needs 4Mhz, so lets drop that... it's too slow for most games and doesn't have many real advantages over sleeping at 8Mhz.  I only added 8Mhz so those who were paranoid could run the hardware within the rated hardware specs (which specify 8Mhz for the low voltage we are using).
This wouldn't be accepted until ARDUBOY_HARDWARE was also added to the boards file (or if there is already another define we can use that the custom boards file is setting).

Your prior point about dropping 8Mhz is probably valid, but I'm attached to it and I think it could serve a purpose for some sketches, so since it's already in there I vote it stays. That doesn't mean we need to go and add all the other possible slow clock speeds that COULD be supported.
We support overclocked 16 and with-in spec 8.  I don't see the need to add others unless a specific need for them becomes apparent.

This is broken.

Yes, you're right. My mistake :flushed: .  As you say, we don't want a new #define to indicate the desired clock speed; we want to know that the hardware is actually physically running at 16MHz, regardless of what F_CPU is set for, so we can compensate.

I suggest ARDUBOY_HARDWARE, but open to suggestions. Perhaps one is already set based on the device name?

Well, there will be ARDUBOY_10 or AB_DEVKIT but we'd have to test for both, and possibly update for any new variants added in the future. I'd just use a #define dedicated to the purpose. Instead of ARDUBOY_HARDWARE, I suggest we make it a bit clearer and future proof. I'd call it ARDUBOY_HARDWARE_16MHZ or if too long, ARDUBOY_16MHZ.

I don't think anyone needs 4Mhz, so lets drop that.

That's fine with me. Like I said, it doesn't hurt anything, but if you feel it's of no value, it can go.
Let me know if ARDUBOY_HARDWARE_16MHZ is OK or if you want something else. If everything else is fine, including other names I've used, and doing the work in the constructor, then I'll fix this PR or make a new one.
Note: In boards.txt you should change all Mhz used as text or info, to MHz. The H should be capitalized.

ARDUBOY_HARDWARE_16MHZ or if too long, ARDUBOY_16MHZ.

I had the same though, that that would be clearer. But what about being explicit.  ARDUBOY_16MHZ_HW_CLOCK.

ARDUBOY_10 or AB_DEVKIT

These can't be used as  ARDUBOY_10 will be defaulted even if nothing else is passed, so it would still be denied on a Leonardo.

and doing the work in the constructor

I don't think it matters that much, I prefer init functions vs the constructor (gives the user more power).  Someone would have to make a point of doing a lot of timing specific work first before calling begin() for this to even matter.

Note: In boards.txt you should change all Mhz used as text or info, to MHz. The H should be capitalized.

No objection if they aren't required to be that way for some funny reason, probably just typos.  Make a PR.

and doing the work in the constructor

I don't think it matters that much, I prefer init functions vs the constructor (gives the user more power).

But I think in this case we don't want to empower the user. As you've said, it must be done and then not touched after that, if we want all the internal timing to work properly. And as I said, it would be even better if it could be done before the sketch even enters setup(), which would give the user no control of it whatsoever.
I'm wondering if it could be done before sketch execution by compiling and linking in some code using the capabilities of platform.txt and/or boards.txt, with the source code residing somewhere under hardware along the same path as those .txt files. This would make the speed change work even if the Arduboy library wasn't used. I haven't the time to research this right now.

Closed. Replaced by PR #112.


NuGet Package Source Mapping Problem

https://github.com/MicrosoftDocs/azure-devops-yaml-schema/issues/100

Moved the Important Notice on top of article
Added the section "Why am I getting NU1507 warnings ..."

@microsoft-github-policy-service agree

Is there any action needed from my site for the PR to proceed?

@chkoella Thank you for your PR. I made a few small changes (moved the note slightly and added it to the remarks for that task input) and it should be live soon. Thank you.


PR for Issue #36 and Issue #25

C#6 upgrades as well as some style cleanup. If they style leaves something to be desired, I can correct it in another PR.

@mgmccarthy Thank you very much for the contribution! The new C# features certainly enhance the readability of the code. Let's discuss the points I brought up and get this merged!

@ledbutter, just making sure you got my latest pull request. This is my first fork/pull request work I've done on GitHub, so I want to make sure I'm doing it right.

@mgmccarthy Yep, I got it. I'll go through it today.

@mgmccarthy Looks great now, thank you again for your contribution!


Cannot find module 'rxjs/observable' error

I got this when using<EMAIL_ADDRESS>with<EMAIL_ADDRESS>ERROR in node_modules/angular-idle-preload/dist/index.d.ts(2,28): error TS2307: Cannot find module 'rxjs/observable'.
Please advise. What should I do?

@teeSee can you try<EMAIL_ADDRESS>

Very Cool. Trying it now...

kool, thanks for creating the issue


Locations of populated images are not recognized by the CMS

I've started using the "Default assets". I love the concept, though it seems images that are placed in a directory are not properly picked up by the CMS.
It takes for me to sync assets via the cms for this to happen. See screenshots of the issue:
Image appears in the root of assets - with no title:

Even though it should have appeared in this directory (that's also where it's placed):

Initially I thought I could fix this by adding the following to PopulateFactory.php before returning the object, as to my knowledge that's what needs to be triggered, it's not working though, so I'm a little lost:
//making sure to update file system after adding/editing files and images
if(isset($data['PopulateFileFrom'])) {
	$obj->updateFilesystem();
}

This is my yml:
Image:
  demo3a:
    Filename: assets/immodb/demos/demo3a.jpg
    PopulateFileFrom: immodb/images/demos/demo3/3761870901_953b1c54c9_b.jpg
    PopulateMergeMatch:
      - Filename

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8466846-locations-of-populated-images-are-not-recognized-by-the-cms?utm_campaign=plugin&utm_content=tracker%2F668165&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F668165&utm_medium=issues&utm_source=github).

Is this still an issue? I haven't tried this module yet. If so, is it fully fixed using your technique (Filesystem::sync())?

I havent worked on that project for while, but I believe it solved my issue then.

Ok no problem, just curious. Either way, thanks for not only posting about the issue but documenting your fix (for the rest of us)!

Looks like it's just not populating the file name? @patricknelson I can take another look at this over this week but if you come across a fix let me know. Running filesystem::sync() is one, but would be interested to work out why it's out of sync in the first place.

I actually posted my comment above when researching tools to assist in provisioning content on the site I'm working on but I end up just writing a bash script that synchronizing our development environments with a central server to keep our dev's working with the same files/database. So -- I haven't taken time to look into this myself since I've never even tried this module!

Ha yes we had a similar situation and the reason this exists. 8 devs on the team, 2 testers, 1 code base. Populate was super useful, git pull and do a dev/build and you'd all be on the same page. Easy to share state around.

5 devs, 1 tester and 0 (to -1) time.
Ok, I decided to test this. Got the same result.
Reading through this code though my hunch is that it's probably not setting the title, which is why the title isn't set. It sounds facetious but I'm not seeing any mechanism here throughout which would do anything to set that title. The Name field, sure. The Filename, sure. Not the title, hence null.
So, I was looking through Filesystem::sync() to try and figure out why the hell you get inconsistent results here. Like, you create a File instance and the data model should intuit a title if you don't set one. It does not. Also, somehow, you still get Title's when you run this ::sync(), but apparently:
It does this without actually using the object model, as this starts messing with all the data.  Rather, it does a direct database insert.

Wat? Ok... why not just work with the DataObject model? Is it really that messy or that big of a deal? If you've got good separation of concerns then doing simple tasks wouldn't be so damned complicated and things just fall into place :) Maybe I'm just being a bitter old man.
Anyway -- https://github.com/dnadesign/silverstripe-populate/blob/master/code/PopulateFactory.php#L113
$obj = parent::createObject($class, $identifier, $data);

That raw $data array is all I see going in there so maybe calling setters there would help a bit. That and calling ->setName(...) with the filename (to initialize a default) would do, for now. Or you can just set $data["Title"] = $data["Filename"].

By the way I get that comment above from Folder's ->constructChild() method here: https://github.com/silverstripe/silverstripe-framework/blob/3.1/filesystem/Folder.php#L218


Documentation/release: sign *.aci files

Thanks to https://github.com/coreos/etcd/issues/8085#issuecomment-308232300.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #8142   +/-   ##
=========================================
  Coverage          ?   76.37%
=========================================
  Files             ?      342
  Lines             ?    26653
  Branches          ?        0
=========================================
  Hits              ?    20355
  Misses            ?     4832
  Partials          ?     1466

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 1df8b90...ca6d7bd. Read the comment docs.


Symbol($PPL$XPIXEL) doesn't work in -gif mode

Reported by hankin on 16 Aug 2000 22:39 UTC
/export/home/ja3/hankin/tmp> ferret -gif
NOAA/PMEL TMAP
Program FERRET
Version 5.22 - 07/27/00
16-Aug-00 15:39
yes? go ptest
! Description: create a simple test line plot
! spirograph picture ...
PLOT/VS/LINE/I=1:314 icos(i/20),isin(i/20)
yes? say ($ppl$xpixel)
ginqpixelarraydim()   39 Specified workstation is neither of category OUTPUT nor
of category OUTIN
!-> MESSAGE/CONTINUE 1
1
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/114

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:16 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 3 Nov 2003 23:14 UTC
reassign to Ansley

Comment by<EMAIL_ADDRESS>on 12 Nov 2004 23:17 UTC
fixed and code checked in 27-Aug-2004, including changes in xgks/src/fortran to
get the pixel values.
fer/plt/get_window_pixels.F
ppl/tmap_inc/gkscm2.cmn
ppl/tmapadds/get_pix.F  (new routine)
ppl/tmapadds/SOURCE_FILES
xgks/src/fortran/finqpixel.fc


Raspberry Pi documentation links need updating

This repo has several links to the "old" Raspberry Pi documentation URLs. Note that as a result of both https://www.raspberrypi.com/news/bring-on-the-documentation/ and https://www.raspberrypi.com/news/new-raspberry-pi-website/ that all the documentation URLs have changed. There are redirects in place, but these are only at a "broad level" and so you may want to update your links to more specific parts of the pages.
For example, you have a link to https://www.raspberrypi.org/documentation/hardware/computemodule/cm-emmc-flashing.md - this currently redirects to https://www.raspberrypi.com/documentation/computers/compute-module.html but the part of the page that you'll actually want to link to is https://www.raspberrypi.com/documentation/computers/compute-module.html#flashing-the-compute-module-emmc
Similarly, you have several links to https://www.raspberrypi.org/documentation/configuration/config-txt/README.md which currently redirects to https://www.raspberrypi.com/documentation/computers/configuration.html but https://www.raspberrypi.com/documentation/computers/config_txt.html would be a more accurate link.
Of course this is only an issue if you want to make your links more targeted - if you're happy with the "broad level" redirects then you obviously don't need to do anything.

Of course this is only an issue if you want to make your links more targeted - if you're happy with the "broad level" redirects then you obviously don't need to do anything.

I don't see us killing off those redirects any time soon. But there is the risk that bitrot will eventually make them less relevant, or kill them off for us entirely. Once you're two or three levels of redirect in things can get weird.

Thanks for bringing this to our notice @lurch @aallan Created a PR to get this resolved: https://github.com/balena-io/docs/pull/2143


Specify current animal being analysed in log and at runtime for run_cellpose_threads

Easy fix, duplicate issue because added from vs code


Missing part on the technical details of external transfer

This link https://learn.microsoft.com/en-us/microsoftteams/create-a-phone-system-auto-attendant?tabs=general-info#external-phone-number-transfers---technical-details
There is a bullet point "External phone number - any phone number. See external transfer technical details." but this ink does not work

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

ID: a42f9e2c-aeee-76c4-620a-668cce21ba10
Version Independent ID: 6ff06eab-9c80-9c30-95ef-c17e157fbea1
Content: Set up a Microsoft Teams auto attendant - Microsoft Teams
Content Source: Teams/create-a-phone-system-auto-attendant.md
Service: msteams
GitHub Login: @DaniEASmith
Microsoft Alias: danismith

@Larryngolog It seems like a Markdown issue where there was a missing space between the "---" line and the previous text line that wasn't properly closing out the tabbed conceptual section. Adding the line break seemed to add the missing section back into the article.

This will be updated today, so I'm closing this issue.


Lens sessions are jammed when redis pod is re-created

When master node where redis pod is running is rebooted for some reason old Lens sessions are jammed although new redis pod is up and running. What is interesting, new sessions are working fine.

Fixed in 1.7.0 which does not require redis at all.


Mario Kart 7 Error

I have followed all the steps correctly on the 3ds hacks guide menu. Until I get to the part where I open up the street pass list and have to press (A) is where I'm stuck. Whenever I open up the street pass list it freezes and I get kicked out with the prompt "An error has occurred, forcing the software to close" and then my system will restart. I have a USA console and copy of Mario kart and everything is in the right place. I'm not huge on tech stuff but I need help on how to fix this. I have also updated my MK7 to 1.2

I have reset the moding process to where my 3ds doesn't have the extra files in it. Booting up the street pass list still crashes my 3ds so it might not be the fault of the kartminer. Ay idea how to fix it?

im having the same issues as well but on a japanese console

Hi i found a fix to this. if you have another modded 3ds lying around, I used the "kartdlphax" section of the 3ds hacks guide website and it ended up working <3

im having the same issues. Is ther any fix yet?

https://www.youtube.com/watch?v=bIPHP6dRJkk
As this video mentioned, change the language may fix this.
But my console is japanese version, I am also facing the same problem.

Changing the language to Italian fixes it for most people but i don't know about Japanese consoles... Sorry,


Updating Profile Fields

ensure updating profile fields is seamless for any combination of updated fields.

Updating profile fields logic is handled in user_service.py and user_schema.py, and has been configured so any field can be updated with a request, and that no fields are required in the body of an update request. The only required field is the user_id parameter. In user_schema.py in the update function we set all available fields to optional, ensuring no field is required during an update. In user_service.py, validation is done on email, nickname, and password, only if they were sent as part of the request. Only the data that was sent with the request body is updated in the users table.
#user_schema.py
class UserUpdate(UserBase):
    email: Optional[EmailStr] = Field(None<EMAIL_ADDRESS>    nickname: Optional[str] = Field(None, min_length=3, max_length= 20, pattern=r'^[\w-]+$', example="john_doe_456")
    first_name: Optional[str] = Field(None, example="John")
    last_name: Optional[str] = Field(None, example="Doe")
    bio: Optional[str] = Field(None, example="Experienced software developer specializing in web applications.")
    profile_picture_url: Optional[str] = Field(None, example="https://example.com/profiles/john.jpg")
    linkedin_profile_url: Optional[str] =Field(None, example="https://linkedin.com/in/johndoe")
    github_profile_url: Optional[str] = Field(None, example="https://github.com/johndoe")
    password: Optional[constr(min_length=8)] = None
    role: Optional[UserRole] = Field(UserRole.AUTHENTICATED, example="AUTHENTICATED")

#user_service.py
 @classmethod
    async def update(cls, session: AsyncSession, user_id: UUID, update_data: Dict[str, str]) -> Optional[User]:
        try:
            validated_data = UserUpdate(**update_data).dict(exclude_unset=True)

            if 'email' in validated_data:
                existing_user = await cls.get_by_email(session, validated_data['email'])
                if existing_user and existing_user.id != user_id:
                    logger.error("User with given email already exists.")
                    raise HTTPException(status_code=422, detail="User with given email already exists.")

            if 'nickname' in validated_data:
                validate_nickname(validated_data['nickname'])
                existing_user = await cls.get_by_nickname(session, validated_data['nickname'])
                if existing_user and existing_user.id != user_id:
                    logger.error("User with given nickname already exists.")
                    raise HTTPException(status_code=422, detail="User with given nickname already exists.")
           ...
        existing_user = await cls.get_by_id(session, user_id)
            if not existing_user:
                logger.error(f"User with ID {user_id} not found.")
                return None

            query = (
                update(User)
                .where(User.id == user_id)
                .values(**validated_data)
                .execution_options(synchronize_session="fetch")
            )
            await session.execute(query)
            await session.commit()

            updated_user = await cls.get_by_id(session, user_id)
            return updated_user


[JOSS REVIEW] Contribution and authorship

The list of authors needs clarification. Three authors are listed in the paper, but only one (or maybe two) author(s) seems to have committed to repository. The command git shortlog --summary yields
107  Amedeo
1  Pariterre
19  aceglia
72  ceglia

Can you please explain the contribution of the other authors?
This is related to https://github.com/openjournals/joss-reviews/issues/5091

Of course. In the list you gave in the issue, the other contributor is Pariterre but he didn't contribute to this project; he is named here because he is the maintainer of the github group "pyomeca" and he had to accept the project and some first PR before giving me full access.
There are two other authors in my article, one (Felipe Verdugo) who developed the code with me, not in terms of coding but more in developing the methods and testing the code in a real application. The other one (Mickael Begon) is my supervisor and supervised me giving advice about the code, the application or which interfaces to implement, in the whole project and still now.


 It's not perfect, but maybe let's get it in?

I'll have to look in more detail in the morning.

I'll add OS version and Go version too

LGTM, minor details mentioned above.


What the coordinate system of point cloud is?

Hello,
when I'm tried to  acces the center point of a point cloud, which made by:
k4a_transformation_depth_image_to_color_camera(transform,img_capture_depth,img_depth2color);
k4a_transformation_depth_image_to_point_cloud(transform, img_depth2color,K4A_CALIBRATION_TYPE_COLOR,img_point_cloud);

it shows the xyz as  (-1,-7,535).
Which in my opinion should be (0,0,535).
Is the coordinate system of the point cloud different from the rgb camera?
Thank you.

The point cloud origin matches the color camera physical origin due to your parameter of  K4A_CALIBRATION_TYPE_COLOR
What do you mean when you write "center point of a point cloud"? There are too many possible definitions of that.
The variance you wrote is a difference of 1 millimeter in the X, and 7 millimeters in the Y, and exact in Z.
It might be possible that the implementation of k4a_transformation_depth_image_to_color_camera causes small variations in the upscaling as it creates the mesh. https://docs.microsoft.com/en-us/azure/kinect-dk/use-image-transformation#implementation
This also might be due to the 6 degree down tilt of the depth sensor https://docs.microsoft.com/en-us/azure/kinect-dk/coordinate-systems#depth-and-color-camera. If somehow you had a perfectly centered depth pixel, when that perfect pixel is transformed into the color camera coordinate system, its Y coordinate would be negative due to the tilt, and its X coordinate could also be changed because nothing is perfectly aligned -- the camera calibration matrix used in k4a_transformation_depth_image_to_color_camera would adjust these.

My question: function k4a_transformation_depth_image_to_color_camera() takes care of the camera/depth coordination system transform. So, we do not need to specifically write a transform function with the down tilt of Y axis, correct?
For IMU data, the value is returned in the IMU coordinate system, which is usually aligned with world coordinate system. We don't need to transform IMU reading to camera.

The documentation I linked above about coordinate systems has good information about both your inquiries:

Correct. The depth camera is physically tilted down when compared to the color camera. Your example of a depth coordinate that is perfectly centered on the depth camera and 535mm away from the depth camera will be visually lower to the color camera. Therefore the pointcloud coordinates in the color camera space for this point will be lower in value than they would have been in the depth camera space. You can check this by doing some trigonometry or matrix math. Or keeping everything in depth-space and see if the point cloud for that example point is (0,0,535). Remember that the perfect center of all the cameras is not the middle row and middle column of the image. All cameras have distortion and focal centers that are usually not the middle row/column.

You are errant in your understanding of IMU coordinates. The link I provided above defines the IMU coordinate systems and the API provides transformations to move data to/from these coordinate systems. :-)


Edit tutorial02 to explain where PollsConfig comes from

Resolves #27174
This change needs to be made back to Django 1.9. Can anyone help me in how to backport documentation changes? Do I need to merge this change with 1.9.x and 1.10.x branches?

Do I need to merge this change with 1.9.x and 1.10.x branches?

Backporting documentation changes and bug fixes are usually handled by Django core developers.

merged in 40d501147176352b694ec8b67b09127b20193ad2, thanks!


Offer overloaded API for provision of thread executor

In order to specify the thread that a task should be executed on before conversion to an Observable, a thread executor must be provided.
Tasks API Documentation
https://developers.google.com/android/guides/tasks#threading
[Ref: #16]

Available with version 2.2.0


How to feed back the output.txt files to specific modules

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Apollo installed from (source ):
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0, 5.0):

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
*Is there any way or proper  guide  to feed back the changed inputs.txt files to the specific module to check the original output and new output .
*I got all the inputs.txt for prediction modules using Cyber_channel echo  but i am unable to find the way to to feed these inputs back .
*please provide proper documentation regarding to test the specific module not the whole system specially on prediction module
Regards

Supporting materials (screenshots, command lines, code/script snippets):

@natashadsouza @HongyiSun @kechxu  please provide supporting sugestions as there is no proper documentations from apollo ,its seems we are wrong with apollo


Add Infra-as-Code scans

Consider using tfsec or checkov to do automated terraform scans

https://github.com/aquasecurity/tfsec
https://www.checkov.io/

Both tools have pre-built actions:

https://github.com/aquasecurity/tfsec-pr-commenter-action
https://github.com/bridgecrewio/checkov-action

@rocketnova thoughts on this? i saw from this slack thread that CMS uses tfsec

I know we used checkov when I worked with Sean Fern. I believe checkov is more mature, we could try both and then compare which catches more? In this comparison checkov caught more, but that could have changed in the last year. Also, from reading through this it looks like they both have their pros / cons w/o a strictly clear winner.

I've only tried tfsec before, but based on the slack thread it doesn't seem like there's an issue running both for full coverage.


[4.0] WIP: Add support for search panes extension.

Fix https://github.com/yajra/laravel-datatables/issues/2463
Fix https://github.com/yajra/laravel-datatables/issues/2471
Fix #130
USAGE
// enable search panes
->searchPanes(true)

// enable search panes with array options
->searchPanes([
    'viewTotal' => true,
    'hideCount' => false,
])

// enable search pane with SearchPane builder class
->searchPanes(SearchPane::make()->cascadePanes(false)->hideCount())

Note to self:
It appears that custom panes are not yet fully supported for server-side but the builder added on this PR is already generating a working a scripts.
https://datatables.net/extensions/searchpanes/serverside

It is also worth noting that custom panes will not be supported by server-side processing in SearchPanes 1.1 but this is something that we are aiming to provide in the future with SearchPanes 1.2.

Also, search panes can be hooked directly to the column builder which triggers a correct ajax request.
Column::make('carrier.name')
    ->searchPanes(SearchPane::make()->modelOptions(Carrier::class, 'name', 'id'))
    ->title('Carrier'),

Server-side implementation notes
As of writing, it seems like search panes on server-side only works via column definition. To implement, the following builder options can be used:
Implementation 1
// HTML Builder
->addColumnDef([
  'searchPanes' => ['show' => true],
  'targets' => [3],
])

Implementation 2
use Yajra\DataTables\Html\ColumnDefinition;

// HTML Builder
->addColumnDef(fn(ColumnDefinition $definition) => $definition->searchPanes()->targets([3]))

Implementation 3
use Yajra\DataTables\Html\ColumnDefinition;

// HTML Builder
->searchPanes(['hideCount' => true]) // hideCount can be removed if your search pane options supports it
->addColumnDef(fn(ColumnDefinition $definition) => $definition->searchPanes()->targets([3]))

Finally, thanks @yajra

Hi @yajra ,
i'm testing searchpane with serverside.
I see values in my single simple pane. (using modelOptions Column method : ->searchPanes(SearchPane::make()->modelOptions(Carrier::class,'name','name'))  )
i'm using editor to edit name.
when i click a value in my pane i see the processing but filter not applied in resultset.
Whats appen?
this column definition
{"data":"carrier.name","name":"carrier_name","title":"Carrier Name","orderable":false,"searchable":false,"searchPanes":{"options":[{"value":"MBE","label":"MBE"},{"value":"UPS","label":"UPS"},{"value":"Corriere Espresso TNT","label":"Corriere Espresso TNT"}]},"visible":true,"editField":"id_carrier"}


refactor: logics

Right now there are too much coupling with how start and build interact
We need to resolve:

If the given model_id is a pretrained path?
If the given model_id is a custom file path?

Do we create a new model entry to the ModelStore?

This should be the behaviour since ModelStore changed a bit in BentoML from 1.0.23

Do we only include this entry into the Bento during the build?

Right now service starts relies on environment, so it is working for now, but very fragile
I will work on this during the weekend

addressed on 0.1.20


[FR] Add option to generate both dev and dist IPAs in a single build like in Android

It would be great if the moe gradle task of creating an ipa will have an option to generate both dev signed and dist signed ipa's in one build, like in Android.

+1

+1


Is there the plans to support << merge key?

As default values are not possible at the moment, and merge doesn't work well, the document could have too many copypastes.
here is more info on aliases override:

Yup. I would like to support it if I could.
But it is not a very high priority for complexity.


Can it support custom action?

I want to use this tool to bulk block some spam users, however, it does not have a "block" action so i have to pull cmdline.py down, modify and replace it to add this action.
Can cmdline add such ability: when invoke as twitter blocks create screen_name=xxxxx, it will perform blocks/create api call with param screen_name=xxxx ?

Hello,
The command line tools of this project are not really maintained, only the python library part is.
But what you want to do would be quite easy to do with a simple script such as something like this:
from twitter import Twitter, OAuth
t = Twitter(auth=OAuth(token, token_secret, consumer_key, consumer_secret))
t.blocks.create(screen_name="xxxxxx")


Added pkg-config support

This pull request simply add the pkg-config functionability

I incorporated #71; closing this pull request.


feat: add some ring theory lemmas

From flt-regular.
Co-authored-by: Andrew Yang<EMAIL_ADDRESS>

Thanks!
bors merge

I see Eric has some comments.
bors r-

bors d+

bors r+


Flash 404 error in ssr since v5.0.0

Hi all,
first and foremost, thanks very much for Next.js. It's great.

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

Expected Behavior
In versions prior to 5, our ssr was functioning as expected for every page.

Current Behavior
Since we upgraded to 5, the page mapped as in
https://github.com/em-casa/frontend/blob/master/server.js#L39
started flashing 404 and displaying if rendered server-side (losing the context).
I ended up making a bare-bones page just to make sure, see
https://github.com/em-casa/frontend/blob/fix-filtered-calls-to-url/pages/listings/index.js
and it still happens.

Steps to Reproduce (for bugs)

In production: https://emcasa.com/imoveis
A branch that if cloned will reproduce the issue without needing the backend:
https://github.com/em-casa/frontend/tree/fix-filtered-calls-to-url
If testing locally, the page in question is localhost:3000/imoveis

Context
I tried to make sure this isn't an Express issue or anything of the sort. Wonder if anyone has been experiencing something similar. Any help will be greatly appreciated.

Your Environment

Tech
Version

next
5.0.0

node
9.5.0

OS
macOS highSierra

browser
Chrome

I just upgraded to 5.0 today, and came on to search for this very issue. (Except for me, receiving the 404 response routes to a 404.html page. Upon refresh, the correct page is load.)

Tech
Version

next
5.0.0

node
7.9.0

OS
macOS Sierra

browser
Chrome

Tech
Version

next
5.0.0

node
9.5.0

OS
macOS highSierra

browser
Chrome

Same, nested routes like /pages/folder1/page1.js stops working and showing 404 after upgrading to 5, currently on the latest version as well (canary-5).
I wonder what's going on with 5 that prevents this from working or if there is a different way to make this work.

Try upgrading to latest next@canary


Problem with HomeKit when NOT in the WLAN

Hi Sam,
CONGRATULATIONS, GREAT job on this piece of software. During install everything worked fine for me. And everything is working when sitting in my house using my WiFi. But when leaving the status of the items changes to OFFLINE. I USE an AppleTV (3rd gen) and my Elgato Eve power plug works fine with HomeKit even when I am on the road using 3G or LTE. It still says ONLINE. Only the Energi items are OFFLINE!?
Do you have an idea what is wrong here? Is there maybe a problem or even a bug in your software?
THANKS so much for your help
Manfred

Mandy2000,
It took an hour or two and a sign out/in to iCloud for it to fully work on the Apple TV. This is why I say that a Raspberry PI implementation is the best. I'll be doing a video and Software on that very soon.
Sam

Hi Sam,
thanks for your fast reply. Well so I hope to see the video soon. As I am not really good in programming and computing I please need an explanation “for Dummys” ;)
The video for the setup on the Mac was by the way GREAT. Except for the part with the AppleTV everything was clearly explained and easy understandable. Thanks for this perfect tutorial.
all the best
Manfred

Am 09.05.2016 um 17:50 schrieb SamGabbay<EMAIL_ADDRESS>Mandy2000,
It took an hour or two and a sign out/in to iCloud for it to fully work on the Apple TV. This is why I say that a Raspberry PI implementation is the best. I'll be doing a video and Software on that very soon.
Sam
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/SamGabbay/tesla-siri-homekit/issues/1#issuecomment-217904533

Hi Sam,
just one short information. As I own an AppleTV 3rd gen, I just tested with the AppleTV 4th gen of my neighbor. It IMMEDIATELY worked ... So there really might be a problem with the 3rd gen, which many people confirmed on different websites on the internet ...

Mandy,
You are 100% Correct. It would not work on my Father-in-laws 3rd Generation Apple TV but works on my 4th Generation!


docker-compose file for quick boostrap

Hi,
Hope you are all well !
Would be awesome to have a docker-compose file to bootstrap the front-end,
https://github.com/bobinette/papernet-front, and the back-end. And if they are Alpine based to have small containers it would be really awesome. I can do that but I would need a more simple auth mode for test/demo purpose.
Ps. Have you seen these references ?

https://github.com/feedlabs/elasticfeed
https://github.com/feedlabs/feedify
https://github.com/urandom/readeef

they could help you to aggregate more sources and have a better semantic search mode.
Have a great day !
Cheers,
Richard

Hi!
I agree, a docker-compose would be really cool! If you have some time for that I would very much appreciate the help :+1: I'll do the new login system ASAP
I had not seen those links, I'll definitely have a look at them!
Cheers!

Hi!
The front should be good now, so if you are still willing to help with this, I'd greatly appreciate that. If you have any question/run into any trouble, do not hesitate.
One more thing, I don't know if you saw it but I also have a repo https://github.com/bobinette/papernet-ops for the ops side of the project, so we could move this issue over to that repo. For now I only have ansible deployment in there, and a docker folder would look gorgeous 😍
Cheers

tomorrow, it will be done.

Work in progress... Sorry for the delay...
https://github.com/roscopecoltran/papernet

Awesome! No worries, no pressure here 😉


checkpoints error

The keys in the provided checkpoints may be wrong. I download the r50 of ref-youtube and the keys contain vita. Please update the right checkpoint.

Thank you so much for your feedback. The weight address of the model has been updated. If you have any other questions, please point them out in time.


Adding OAuth support, adding Cattask/variable retrieval

First time pull request.
Edited Set-ServiceNowAuth and Get-ServiceNowTable, created Set-ServiceNowAuthToken. These proposed changes allow OAuth to be used. Token refreshing behavior inconsistent with my account. Access token works fine. Defaults to credentials if token access fails.
Added Get-ServiceNowCatalogTask to pull Cat Tasks from Service-Now. There is an optional -ReturnVariables that uses the last changed file Get-ServiceNowVariables. This one pulls the variables associated with the Cat task down and adds them to the variable field in the $result object.
Final Notes:

Tokens don't seem to work with the links returned to get the question/answer variables. Thus it uses a web request and the supplied credentials instead.
The client ID and client secret are plain text. I'm not fond of that, but anything else seemed hacky.

@Rick-2CA WHat's the status of this PR?
I see that you are still waiting for @robison-scott to replace Write-Host. Probably that is just a small change and can be done in a separate commit. The PR has some good enhancements that I would like to use.
Let me know if I can help.

@DexterPOSH Anyone could pick this up, to be fair.

@DexterPOSH The delay has strictly been scheduling issues on my side.

@Rick-2CA Let me know if you want me to pick this up. I have some time at hand.

@GilmondRoss & @DexterPOSH - I don't have the ability (or perhaps just the know how) to test the OAuth stuff.  Have either of you played with the code to ensure it's functional?
I spent some time with the catalog task and variable functions.  I figured out how to use the API to get the variables.  Get-ServiceNowCatalogTask will get a small change and Get-ServiceNowVariables will be replaced.

@Rick-2CA Let me try this in our test environment and come back. I think it did work last time, but want to re-confirm that.

Has anyone gotten OAuth working? I'm starting to dig into this at work and hoping that I won't have to reinvent the wheel. Thanks!

Going to close out this PR.  The changes here were considered and worked further upon by #83.  Interested parties should review where that PR is and provide feedback to the author, please.


Backport "HBASE-28716 Users of QuotaRetriever should pass an existing connection (#6065)" to branch-3

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ branch-3 Compile Tests _

+0 :ok:
mvndep
0m  7s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  0s

branch-3 passed

+1 :green_heart:
compile
3m 44s

branch-3 passed

+1 :green_heart:
checkstyle
0m 57s

branch-3 passed

+1 :green_heart:
spotbugs
2m 17s

branch-3 passed

+1 :green_heart:
spotless
0m 46s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m  5s

the patch passed

+1 :green_heart:
compile
3m 43s

the patch passed

+1 :green_heart:
javac
3m 43s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 55s

the patch passed

+1 :green_heart:
spotbugs
2m 32s

the patch passed

+1 :green_heart:
hadoopcheck
10m 41s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 44s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 21s

The patch does not generate ASF License warnings.

40m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6096/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6096

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux 27cd348f4052 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / 02d76d206d44556c44e3bfe2074e5dd1db1315ae

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
84 (vs. ulimit of 30000)

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6096/1/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 36s

Docker mode activated.

-0 :warning:
yetus
0m  4s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+0 :ok:
mvndep
0m 10s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m  2s

branch-3 passed

+1 :green_heart:
compile
1m 21s

branch-3 passed

+1 :green_heart:
javadoc
0m 50s

branch-3 passed

+1 :green_heart:
shadedjars
5m 22s

branch has no errors when building our shaded downstream artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 57s

the patch passed

+1 :green_heart:
compile
1m 21s

the patch passed

+1 :green_heart:
javac
1m 21s

the patch passed

+1 :green_heart:
javadoc
0m 48s

the patch passed

+1 :green_heart:
shadedjars
5m 19s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 36s

hbase-client in the patch passed.

+1 :green_heart:
unit
285m 56s

hbase-server in the patch passed.

315m 16s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6096/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6096

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 4d05d0516350 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / 02d76d206d44556c44e3bfe2074e5dd1db1315ae

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6096/1/testReport/

Max. process+thread count
4870 (vs. ulimit of 30000)

modules
C: hbase-client hbase-server U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6096/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


Remove use of resolverproxy in non-dnssd-directory

Problem
ResolverProxy adds lifetime complexity to code and makes it non-obvious when resolvers are or are not multiplexed.
Lifetime complexity stems from ResolverProxy requiring a separate init from the main dnssd resolution and circular referencing in platform dnssd code (resolver proxy initializes dnssdimpl and dnsssdimpl contains a resolverproxy).
Change overview
Replace ResolverProxy with direct calls to the global resolver instance or passed in resolver instance.
NOTE: I believe resolverproxy on bonjour/avahi allows some internal multiplexing even though the implementation for minmdns does not. I am unsure if this is an issue or not or if we need workarounds here (i.e. should "FindCommissionable" or similar take in a callback argument instead of using the global callback argument).
Testing
CI will validate that commissionable node discovery still works.

As in, the "multiplexing" is the sole reason ResolverProxy was introduced, and is the key feature it provides.


Use new logo

Copied the svg from here
Adding Drew as a reviewer as I'm not sure how to release this :)

@amy-byrum can you give the updated deploy preview another look?

Beautiful! 👍

Does this get merged into master? Not sure what the gitflow is for this repo

@clrcrl I can merge and release this...

Go for it!


Avoid deduplication of objects when forking

Feature Description
At fork time, Gitea creates a Git repository for the new project. This repository is a copy of the original project. When forking large projects, it consumes a lot of disk space.
Other git servers make use of the Git alternates feature :

Github: Fork would not duplicate (on the server side) the full repository, as explained in Counting Objects
GitLab: How Git object deduplication works in GitLab

It would be nice to have such an optimization in Gitea.
Screenshots
No response

GitHub's deduplication method also has the downside that pull requests are actually part of the original repo, so these can be technically be made to appear as part of the original repo on the UI as has happend on GitHub's DMCA repo in the past.

Yes, so we need to find object's origin repository if we use this method. I have sent #18459 to try to have a warning once view a commit which actually don't belongs to the main repository.


hlo_helpers.dtype_to_ir_type is not complete comparing with jax.interpreters.mlir.dtype_to_ir_type

Please:

[ x ] Check for duplicate requests.
[ x ] Describe your goal, and if possible provide a code snippet with a motivating example.

I noticed that the dtype look up table defined in https://github.com/jax-ml/jax/blob/main/jaxlib/hlo_helpers.py#L28 is missing dtypes comparing with the one in jax.interpreters.mlir: https://github.com/jax-ml/jax/blob/main/jax/_src/interpreters/mlir.py#L158. A few popular dtypes such as bfloat16 is missing and it impacts the usability of.
Is there any reason why we couldn't directly use jax.interpreters.mlir.dtype_to_ir_dtype in hlo_helpers?

Good questions!

Is there any reason why we couldn't directly use jax.interpreters.mlir.dtype_to_ir_dtype in hlo_helpers?

There is! since hlo_helpers is in jaxlib which is a separate package that jax depends on, it can't import from the jax namespace.
Can you expand a little bit more on the issue that you're seeing? You said:

A few popular dtypes such as bfloat16 is missing and it impacts the usability of.

but I don't think you finished the sentence. Where is the usability problem? What operation are you trying to use and what error are you seeing?
The jaxlib.hlo_helpers and jax.interpreters.mlir APIs are both private, so they should typically be transparent to users. Perhaps you could provide more details about the problem and I can give some alternative suggestions!

Thank you for the response. I found this problem when trying to call hlo_helpers.hlo_const with a numpy array of type dtypes.bfloat16, and it complains that it does not recognize the datatype. It hurts the usability of hlo_const at the very least.

Sure, but the question remains: why are you using hlo_helpers at all? It's a private module.

(We're trying to encourage everyone to migrate to the newer JAX FFI, fwiw: https://jax.readthedocs.io/en/latest/ffi.html )

we are building extensions of JAX by introducing our own jax primitives. we call functions from jaxlib.hlo_helpers and jax.interpreters.mlir in the lowering rule for our primitives. We found that jaxlib.hlo_helpers and jax.interpreters.mlir has overlapped functionality.
I wonder what are the high-level guidance on when to use  jaxlib.hlo_helpers vs jax.interpreters.mlir.
also any plan to unify these two?

The high level recommendation would be to avoid using either of these modules if possible. It's best to combine the FFI (for external library calls) with public APIs like jax.custom_jvp or jax.custom_vjp for customizing behaviors under transformations. If there is a use case that isn't covered by these public APIs, please let us know and I'm always happy to provide guidance. It's a high priority for me to support these kinds of extensions.

I wonder what are the high-level guidance on when to use jaxlib.hlo_helpers vs jax.interpreters.mlir.

If you can't use public APIs for your needs, I'd probably recommend using jax.interpreters.mlir with the understanding that it can change at any time without compatibility guarantees, and we don't formally support such use cases.

also any plan to unify these two?

I don't think that's likely! I think that jaxlib.hlo_helpers will probably disappear at some point, but it'll still be a while before that happens.

We are lowering our jax primitives to a custom call (by doing mlir.register_lowering(xxx, xxx_lowering_rule, platform="neuron")). We are calling things like jaxlib.hlo_helpers.custom_call.

I'd probably recommend using jax.interpreters.mlir with the understanding that it can change at any time without compatibility guarantees, and we don't formally support such use cases.

This sounds good, thanks
`

Makes sense. I will say that jax.extend.ffi.ffi_call should support everything that a custom primitive does (with some small caveats around customization under function transformations for now) with far less boilerplate. I'd recommend checking it out instead of writing your own primitives!
I'm going to close this issue as solved for now, but please open other issues if you run into problems.


WIP: Cost for soft quadratic limit on full state - [closed]

In GitLab by @proyan on Jan 24, 2019, 18:32
Merges jointlimits -> master
Do not merge yet.
Criticism on implementation are welcome.
Deals with #57

In GitLab by @proyan on Jan 24, 2019, 18:36
closed


Show warning messages when removing bound pvc

Proposed changes
If a user deletes a PVC in active use by a pod, the PVC is not removed immediately. PVC removal is postponed until the PVC is no longer actively used by any pods.
We should show a warning message when user deletes it as the state of the PVC will be always Removing

Types of changes

New feature (non-breaking change which adds functionality)

Linked Issues
https://github.com/rancher/rancher/issues/16268

Further comments

@loganhz @vincent99 I could be off base here but I think this should be a bug for the API. If the PVC is in a state where is can not be removed shouldn't the delete action fail? Or possibly the state could go to pending-removal? Possibly with a message from the api as to why it can not be removed? Throwing a warning at the user solves the immediate issue of why its not removed but that's just a bandaid. If i'm wrong let me know and I will merge this.

Yes I agree.  It may be a "feature" of k8s that you can ask to delete a PVC while the PV is in use and it will someday succeed once it's not bound anymore, but I don't think we need to expose that to the user.


Multiple publication through prototype

Fixes #299
Overview
Adds more comprehensive metadata support to functions decorated with the garden_pipeline decorator and uses this addition to employ multiple publication/execution. (i.e. multiple uniquely executable publications in one call to garden-ai prototype register [planted-image])
Discussion
This implementation is very subjective, and open to input on all fronts.
Testing
Completely untested. Due to time limitations, I don't know when I'll be able to start going through it locally and debugging the almost certainly numerous problems.
Documentation
This whole thingamajig deserves some rather serious documentation overhaul, and this bit in particular will make the structure rather different compared to without.

:books: Documentation preview :books:: https://garden-ai--302.org.readthedocs.build/en/302/

Codecov Report

Merging #302 (57dfb3e) into main (acb4f4a) will decrease coverage by 0.07%.
The diff coverage is 20.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
- Coverage   73.35%   73.29%   -0.07%
==========================================
  Files          52       52
  Lines        3765     3771       +6
==========================================
+ Hits         2762     2764       +2
- Misses       1003     1007       +4

Files
Coverage Δ

garden_ai/container/containerize.py
23.72% <50.00%> (+1.31%)
:arrow_up:

garden_ai/app/prototype.py
25.17% <17.85%> (-0.20%)
:arrow_down:


AndroidMode='spinner' works on date but not time mode

I have 2 separate datepicker and timepicker. On datepicker, I have successfully set the androidMode to spinner and it works but when i try to set on the timepicker, it doesn't.


System representation or compute graph should be insensitive to atom pairs with distances greater than cutoff

As discussed with @jchodera, the assumption that the neighbor list returns atom pairs with distances below the cutoff distance might be violated under specific scenarios. The features produced by the neural networks should be insensitive to these atom pairs.

Can you expand further on what this means?  I'm not quite understanding.

A given neighbor list implementation might not always return atom pairs with distances strictly below the cutoff distance. Therefore, the system representation of the neural network potential should ignore such atom pairs.
This is already the case for models representing their environment using atom-centered radial symmetry functions: the radial basis function goes to zero as the cutoff is approached. For other representations we need to make sure that a similar cutoff to the representation is applied.

Ah, now I understand.  The way I implemented the Chiron neighbor list, it returns a mask for the neighbor list (allowing for fixed size of the array) and I use this mask as a pre-factor of the potential energy (i.e., it will multiply anything beyond the cutoff with 0 so it does not contribute to the energy, anything within the cutoff is by 1).  I was thinking about how this would translate/be implemented for the NNPs.

I think this issue has become more relevant since we realized that some of the neighborlist implementations are not able to compute distances using the minimum image convention.

This has been resolved for all architectures. Edge features that contain signals for distances greater than the cutoff are set to zero by using cutoff (annealing) functions for each edge representation.


Add 'modal' and 'fade' classes to configuration.

Is it possible to add the names of these classes (or maybe all template html) in component definition below to configuration?
@Component({
  selector: 'simple-modal-wrapper',
  template: '
    <div #wrapper class="modal fade" [ngStyle]="{display:'block'}" role="dialog">
        <ng-template #viewContainer></ng-template>
    </div>'
})

So, I have two different problems:

I want to use BEM notation and\or prefixes in my code.
So, for example I want to have o-modal instead of modal, o-modal--fade instead of fade .

Sometimes our modals contains two parts: the firt part is same for all, another is different. With ability to set template it is will be easier to maintaing such modals.

I see, so you're looking for overrides to the classes that are within the wrapper itself, I follow you now. No harm in.  I honestly thought i'd done this once, but maybe we dropped it for simplicity.

@Aracturat give the PR a review and let me know if i've understood your requirement.

@KevCJones Thank you! It is ok for me =)


Remove kube-rbac-proxy from EKS-A controller deployment

We don't need kube-rbac-proxy in the EKS-A cluster controller Deployment manifest. We should have done this way back during v1alpha4 migration.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test eks-anywhere-e2e-presubmit

/test eks-anywhere-presubmit

/test eks-anywhere-mocks-presubmit

/test eks-anywhere-mocks-presubmits

Force-merging to fix E2E tests.


Cannot open saved BF description

I created and saved a description using the Sinopia user feedback template. I was able to open and edit it. While editing it, I clicked on a field and it took me to a blank screen. Since then I haven't been able to open the description (although I can search for it). The title is: Cooperación transfronteriza en el marco de la globalización.

This is the Trellis URI https://trellis.stage.sinopia.io/repository/stanford/5ee35475-f6f9-4c28-acd4-94890d16585c

That's the description for it. The problem I'm having is that I cannot open it for editing in the Linked Data Editor.

I'm able to open this description on stage using both through the Search Tab and from the Load RDF Tab with Chrome on a Mac:

Clicking the Preview RDF button will generate an error and a blank screen.
Fails and shows the blank screen if I use Firefox.

I wasn't able to open it up last evening, but this morning I cleared the cache in Chrome and was able to open the description through the Search tab. I'm also getting a blank screen when I click Preview RDF.

I can open from Search tab on both Firefox and Safari but get a blank screen on both upon Preview. @jermnelson is this related to the problem you fixed this week on DEV where preview gave blank screen?

Stage has been cleared out, so cannot reproduce this problem. If in the future you are able to reproduce, please re-open or re-ticket.


Replace BrowserDriver map with RegisterMap

Create a smarter reflection based RegisterMap for handling registered BrowserDrivers. Should have the ability to create new or replace current BrowserDrivers at runtime.

Replaced BrowserDriver mapping with an customization runtime loosely based off of ScenarioServer's RegisterMap.


Dynamic Firmware via AIO

This is a draft for review, but must not be merged until other things are deployed. I want to get the review underway so I can deploy a bunch of things at once when all are ready.
@makermelissa take a look!
This code is technically live on my GH:P at https://lorennorman.github.io/WipperSnapper_Firmware_Uploader/, but it interfaces with the AIO staging server which is locked down hard, and so won't work unless you have your IP address allow-listed by Justin.
Code Highlights:

nothing in the bin/ directory anymore, all bins come dynamically from GH:R via AIO
added ~80kb zip file decompressor lib, the bins come packaged into zip files for littlefs boards now
getFirmware() method now rifles through the zip contents to find the files, instead of making web request
added a select box for users to select which firmware to install. it auto-populates with all available littlefs-related firmwares, and it comes from an API request to AIO, which proxies to what's available on GH:R
the structure collection now:

...comes from the board definitions repo, example
...holds the hex values as strings that are converted to numbers by js, since json does not support hex numbers
...has a magic VERSION token in the filenames that is used to regex match filenames, allowing the firmware version to change without updating the structure

Looks good to me.

@makermelissa this is ready for merge now, please.


Android variant release not working

I have a project with multiple flavors the build.gradle file looks like this
project.ext.envConfigFiles = [
        flavorx: ".env.flavorx",
        flavory: ".env.flavory",
]
apply from: project(':react-native-config').projectDir.getPath() + "/dotenv.gradle"

android {
    ...
    productFlavors {
        flavorx {
            applicationId 'flavor.x'
            resValue "string", "build_config_package", "default.package.name"
        }
        flavory {
        }
    }
}

in the debug mode everything works fine, and each flavor get its specific config.
but in release mode the .env file is not loaded and I get empty values, even though this line appears in the terminal

Reading env from: .env.flavorx

I also tried this starting the build using this command
ENVFILE=.env.flavorx ./gradlew assembleFlavorxRelease
and the result is similar
any suggestions ?

It was just a proguard issue. Resolved by adding this line to proguard-rules :
-keep class default.package.name.BuildConfig { *; }

@sodaoud does default.package.name get somehow magically substituted?

@YarekTyshchenko mypackage should match the package value in your app/src/main/AndroidManifest.xml file.

I am having the same issue, but even after configuring everything as in the docs, it just don't work here :(
I see that even generates the resource xml file with the correct values in the output folder, but on react native code the Config object is completely empty

@frankiewiczkamil I couldn't get it working, so in the end I used the library: https://github.com/zetachang/react-native-dotenv

I've had the same problem, setting proguard-rules didn't help, because I had proguard already disabled.
what helped was setting a fixed build_config_package in android/app/build.gradle
defaultConfig {
    ...
    resValue "string", "build_config_package", "YOUR_PACKAGE_NAME_IN_ANDROIDMANIFEST.XML"
}

hope this helps

I really don't get the build_config_package string there... is this exactly word or should I use the env string I set?
env:
APP_ID=com.appname.here
resValue "string", "build_config_package", project.env.get("APP_ID")
or
resValue "string", "APP_ID", project.env.get("APP_ID")
??

I've had the same problem. Solved by @krystofbe solution. Remember to remove resValue of your productFlavors config, and set it only in defaultConfig. Otherwise, it will get overwritten.


update + google-services.json instructions

Adapted codinguser's README.md to GnuCash-Pocket's one. Added google-services.json instructions based on ccomeaux/boardgamegeek4android/CONTRIBUTING.md

It is solution / instruction for Issue #54.

Thanks, @djbrown, for a review 👍


[#41] Handling non-existent sub-array elements of source object

For further details please refer to the issue: [#41] Handling non-existent sub-array elements of source object

Hi @nagarajanchinnasamy, thank you for putting this together. Could you please add a couple more tests to test your proposed functionality? At least the same example as from issue #41.
Thanks!

You are welcome. Thanks for the small but powerful library 👍
I will add the tests and resubmit.


Printer.text() method, no errors, no print

Bug description
Printer.text() method gives no errors but the printer does not print either
Steps taken

echo hi > /dev/usb/lp0
execute the script

#!/bin/python

from escpos import printer as Printer

printer = Printer.Usb(0x0483,0x5743) # lsusb

printer.text("hi")

check usb devices ls /dev/usb

Observed

In step 1 everything goes as expected and the printer prints the text.
In step 2 nothing is printed by the printer, the printer seems to ignore the text.
In step 3 observe that the printer is no longer available as /dev/usb/lp0 is this normal? Because when I try printer.cut() instead of printer.text() the printer does cut the paper but it's also removed from /dev/usb

Device info
Archlinux latest packages,
printer by STMicroelectronics, some generic Chinese 80mm POS printer (I don't know the exact number but I do have access to the programmer manual)

Have you tried sending a newline to the printer? Typically, a printer will only print full lines.
Apart from that, if you know a certain sequence of symbols that should definitely work, you can try to send them with printer._raw().

I didn't think of that but that's a very good point and after trying it with a newline it seems to work fine so thank you.


React-router-dom not working with redux, clicking links changes url but does not execute component

I've seen solutions for this issue before and I've tried all of them but nothing seems to resolve the issue. I am running. I get no error, what happens is that clicking on a link changes the url but my component associated to that route does not render. I'm running react-router-dom 4.3.1 and redux 4.0.0. Sorry for poor formatting, for some reason code shows up like this. Please let me know if you need more information.
// index.js

// Create browser history to use in the Redux store
const baseUrl = document.getElementsByTagName('base')[0].getAttribute('href');
const history = createBrowserHistory({ basename: baseUrl });

// Get the application-wide store instance, prepopulating with state from the server where available.
const initialState = window.initialReduxState;
const store = configureStore(history, initialState);

const rootElement = document.getElementById('root');

// also tried ConnectedRouter
ReactDOM.render(
  <Provider store={store}>
    <ConnectedRouter history={history}>
      <App />
    </ConnectedRouter>
  </Provider>,
  rootElement);

// app.js
import React from 'react';
import { Route, Switch, withRouter } from 'react-router-dom';
import { connect } from 'react-redux';
import ReportQuery from '../../silos/reporting/components/ReportQuery';
import Reporting from '../../silos/reporting/components/Reporting';
import Nav from '../components/Nav';
import { userAction } from '../../store/actions';
import { NotificationContainer, NotificationManager } from 'react-notifications';

import 'react-notifications/lib/notifications.css';
import '../style/App.scss';

const showNotification = ({ type, title, message, callback, timeOut = 3200 }) => {
    switch (type) {
        case 'success':
            NotificationManager.success(message, title, timeOut, callback || (() => { }));
            break;
        case 'warning':
            NotificationManager.warning(message, title, timeOut, callback || (() => { }));
            break;
        case 'error':
            NotificationManager.error(message, title, timeOut, callback || (() => { }));
            break;
        default:
            NotificationManager.info(message, title, timeOut, callback || (() => { }));
            break;
    }
}

class App extends React.Component {
    componentDidMount = () => {
        this.props.getUser();
    }

    render = () => {
        return (
            <div id="container" className="container">
                <Nav user={this.props.user} />
                <Switch>
                    <Route path="/test" render={(props) => <Reporting user={this.props.user} />} />
                    <Route path="/ReportQuery" render={(props) => <ReportQuery />} />
                </Switch>
                <NotificationContainer enterTimeout={500} leaveTimeout={500} />
            </div>
        );
    }
};

const mapStateToProps = state => ({
    user: state.user
})

const mapDispatchToProps = dispatch => ({
    getUser: () => {
        dispatch(userAction(showNotification));
    }
})
// tried wrapping with withRouter, but does nothing
export default connect(
    mapStateToProps,
    mapDispatchToProps
)(App);

I edited your post. You need to use triple back ticks for code blocks:

```jsx
your code
```

As for your issue, you are most likely having issues with blocked updates caused by your App being connect()-ed to your Redux store.
Additionally, react-router-redux is no longer supported. I'm not sure if you're using that because you didn't include your imports, but if you really need routing information in your Redux store, you should checkout out connected-react-router.
Finally, the issues section here is for bug reports, not support. If you are still having issues, please check out StackOverflow or the Reactiflux Discord; both of those venues are setup for helping with support problems. Thanks!


Version conflict check doest not support complex semver

Description of the bug / バグの説明
I have a package that uses a semver involving the || operator in its vpmDependencies as follows:
"vpmDependencies": {
    "sh.orels.shaders": "^6.3.0 || ^7.0.0",
  }

In the VCC this is interpreted correctly and the package is allowed to install against ^7.0.0. However in ALCOM the || symbol is seemingly not understood and it interprets it as if only the left operand ^6.3.0 exists.
Steps to Reproduce / 再現手順
We will be using the following two repos as an example:

https://orels1.github.io/orels-Unity-Shaders/index.json
https://lackofbindings.github.io/ORLModules/index.json

Starting with any blank project.
Install<EMAIL_ADDRESS>Attempt to install<EMAIL_ADDRESS>
ALCOM will attempt to downgrade sh.orels.shaders to 6.4.0-dev.12.
Expected Behavior / 想定される挙動
ALCOM should recognize that 7.0.0-dev.3 also satisfies ^6.3.0 || ^7.0.0 and allow the install without attempting to downgrade sh.orels.shaders.
Screenshots / スクリーンショット
No response
Software Version / バージョン情報
0.1.16
OS
Windows 11 Home
Webview Version / Webviewバージョン
131.0.2903.99
Additional context / 補足
No response

There is code to support || pattern so it might e some other issue

This behavior is expected in vrc-get since accepting this is an bug in SemanticVersioning nuget package, which is used by VCC.
I'm considering if this should be supported for now, but I don't like implement bug

I forgot to describe what's the bug.
^7.0.0 means 7.0.0 or later but not 8.x.x series so it should mean v >= 7.0.0 and v < 8.0.0-0.
However, old SemanticVersioning package was think as v >= 7.0.0-0 and v < 8.0.0-0, which falsely accepts beta releases of 7.0.0, which is older than 7.0.0.

Ah ok so it’s mostly just a mistake on my part then. What should the config look like to get my desired behavior in both the VCC and ALCOM?
Should it be ^6.3.0 || ^7.0.0-0 ?

Should it be ^6.3.0 || ^7.0.0-0 ?

Yes or possibly ^6.3.0 || ^7.0.0-dev.1 for cleaner description.
(to understand -0 may require knowlage that 0 is the smallest prerelease version, but -dev.1 is does not require I think.)

Thanks for your help, I'll close this as mostly being my fault for not fully understanding semver & the VCC being slightly buggy.


Clean way to report line numbers for asserts

As of e2a09aec8ae706313ddc28607b6779f1d121f51e we no longer use asserts. The new system supports line and column numbers.


Timespan EF Queries

Hello,
We are wondering what the querying would look like for querying timespans using the TimeSpan Converters. We are using Postgresql using npgsql and are unable to identify the right way to query this data. Can you please amend this document with any possible examples?
Thank you.

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

ID: 03dfe9f6-e831-ac3f-0b90-acb9b4c0f828
Version Independent ID: a26156d1-a4b3-7df0-e63e-b8e9f804abbf
Content: Value Conversions - EF Core
Content Source: entity-framework/core/modeling/value-conversions.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @ajcvickers
Microsoft Alias: avickers

Duplicate of #10434

EF doesn't currently support translations when value converters are configured - see https://github.com/dotnet/efcore/issues/10434.
However, note that EFCore.PG translates some basic operations for TimeSpan, when it is mapped without a value converter (i.e. to a PG interval). If you open an issue on the EFCore.PG repo I can try to assist more.


How can I see the cloud_result and edge_result in the joint inference experiment?

How can I see the cloud_result and edge_result in the joint inference experiment?

I ran this experimenton my k8s cluster, cloud_result  was found in /data/hard_example_cloud_inference_output and edge_result saved in /data/hard_example_edge_inference_output.
Note: both  /data/hard_example_cloud_inference_output and /data/hard_example_edge_inference_output are  located on the docker container of edge node.
#198


Receive HTTP 404 with ASP.NET Core Test Server when controllers are added to a separate assembly

Is this a Bug or Feature request?:
I am experiencing the same exact issue referred here:
https://github.com/aspnet/Mvc/issues/5992
Steps to reproduce:
I have a basic test controller within my ASP.NET Core application:
using Microsoft.AspNetCore.Mvc;
namespace Web.Controllers
{
[Route("api/[controller]")]
public class TestController : Controller
{
[HttpGet]
public IActionResult Test()
{
return Ok();
}
}
}
This is stored within my VS solution as follows:
With this configuration I receive a HTTP 200 when attempting to access TestController via localhost, and my integration test passes using the following scenario:
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.TestHost;
using System.Net.Http;
using System.Threading.Tasks;
using Web;
using Xunit;
using System.Net;
namespace IntegrationTests.Controllers
{
public class CustomerControllerTests
{
private readonly TestServer _server;
private readonly HttpClient _client;
    public CustomerControllerTests()
    {
        _server = new TestServer(new WebHostBuilder().UseStartup<Startup>());
        _client = _server.CreateClient();
    }

    [Fact]
    public async Task HttpPost_TestController_ReturnsSuccess()
    {
        var response = await _client.GetAsync("/api/test");

        Assert.Equal(HttpStatusCode.OK, response.StatusCode);
    }
}

}
However if I move the same controller to a separate .NET Standard based library in the following structure:

I have added the library project as a project reference to my api host project containing the Startup class.
Then the same integration test referenced above fails, however the localhost test returns a HTTP 200 as expected.
I would expect the separation of the Controller into a separate binary to work against the TestServer too - have I missed a manual step?
Version of Aspnet libraries including TestHost are:

To save time I have copied info from the previous bug but my issue is exact same issue.
To workaround I used the same solution of registering the Assembly of the controller class in my API project's start up class like this:
 //Added this so that TestServer can find the Controller assembly services.AddMvc().AddApplicationPart(typeof(Web.Controllers.TestController).Assembly);
After I did this, now my TestServer can be invoked and works as expected.
If having a link to reproducible code is required I will post it later when I get chance.
But if you see anything obvious missing please let me know.
Thanks a bunch!

Thanks for contacting us, @vikasillumina.
@javiercn, can you please look into this? Thanks!

@vikasillumina Thanks for contacting us.
I think you might be experiencing issues with the assemblies not being picked up appropiately. Can you  check that you meet the prerequisites described in the link below?
https://docs.microsoft.com/en-us/aspnet/core/test/integration-tests?view=aspnetcore-2.1#test-app-prerequisites
Can you call
services.AddMvc()
   .ConfigureApplicationParts(apm => ...)

Put a breakpoint on the lambda apm and check in apm.Assemblies that your assemblies are being loaded?

@mkArtakMSFT @javiercn thanks for looking into this so quickly. I checked the referred page and I see I am not using either of those packages. I actually just looked for package that contained the TestServer class through Nuget package inside VS 2017 and it only added following package references:
 <PackageReference Include="Microsoft.AspNet.WebApi.Client" Version="5.2.6" />     <PackageReference Include="Microsoft.AspNetCore.Hosting" Version="2.1.1" />     <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="2.1.1" />     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.7.0" />
I can try to add those references mentioned in the link and see if it makes difference.
However its not very intuitive to know this until we read the wiki page you shared. :)
I expected when I added the packages for using TestServer, it should automatically download other packages that it would need, does it make sense?
I also see the label added to this issue '2.2.0-preview2' is there a fix that your team is working on? Or you expect users to refer the link and add the references to resolve this issue?
Thanks once again for your prompt response.


Jazzy docs generation workflow

@maratal did you see my question above about it not working locally?

@maratal did you see my question above about it not working locally?

I thought the issue with Gemfile.lock will do the trick. I guess you need to run bundle install first?

Nope, I'm still getting the same issue. Is it happening for you locally? If not, I can investigate a bit on my machine.

I've also noticed that in your CI output there's a /Users/runner/work/ably-cocoa/ably-cocoa/Source/include/Ably/ARTDeltaCodec.h:2:9: fatal error: 'AblyDeltaCodec/AblyDeltaCodec.h' file not found, did you see that?

I've spent a bit of time looking into the error I've been getting, but I think that to investigate it in depth would take more time than I can afford to put into it right now. Just to double check - you're not seeing it locally? Might be worth asking a few other people in the team to try it out too and see if it works for them.

I've spent a bit of time looking into the error I've been getting, but I think that to investigate it in depth would take more time than I can afford to put into it right now. Just to double check - you're not seeing it locally? Might be worth asking a few other people in the team to try it out too and see if it works for them.

Works locally, but it gives me some Carthage build artifacts:

But since the docs were generated regardless this output, and Carthage is not in favor now I decided not to dig into that. I'll investigate /Users/runner/work/ably-cocoa/ably-cocoa/Source/include/Ably/ARTDeltaCodec.h:2:9: fatal error: 'AblyDeltaCodec/AblyDeltaCodec.h' file not found issue, looks like was fixed already, but here it is again!

/Users/runner/work/ably-cocoa/ably-cocoa/Source/include/Ably/ARTDeltaCodec.h:2:9: fatal error: 'AblyDeltaCodec/AblyDeltaCodec.h' file not found error was due to absence of dependencies. I've added this step here https://github.com/ably/ably-cocoa/pull/1428/commits/94bfab9cbb6961ccfc79b2901d220bff8f4380b2
Also there were multiple warnings like warning: no 'assign', 'retain', or 'copy' attribute is specified - 'assign' is assumed [-Wobjc-property-no-attribute]. I've created a separate issue for that: https://github.com/ably/ably-cocoa/issues/1433
Regarding local jazzy command. Strangely enough I began experiencing the same error /Users/lawrence/code/ably/ably-cocoa/Source/include/Ably/Ably.h:1:9: fatal error: could not build module 'Foundation', but only after I've installed sourcekitten and used it directly like that:
sourcekitten doc --objc $(pwd)/Source/Ably.h \
      -- -x objective-c  -isysroot $(xcrun --show-sdk-path --sdk iphoneos) \
      -I $(pwd) -fmodules > objcDoc.json

jazzy command works without issues. I still have Big Sur and ruby 2.6.3 though, will update to Monterey and check it again. But CI build is now clean. @lawrence-forooghian

@maratal great news about the CI build, thanks!
As for my local issues, I just tried with Ruby 2.6.3, didn't make a difference, still get the error. As for the rest of my environment, I'm on macOS 12.4, Xcode 13.4.1.

@maratal great news about the CI build, thanks!
As for my local issues, I just tried with Ruby 2.6.3, didn't make a difference, still get the error. As for the rest of my environment, I'm on macOS 12.4, Xcode 13.4.1.

Yes, after fresh install of macOS 12.4 and Xcode 13.4.1. it's now reproducible. @lawrence-forooghian

Jazzy stopped crashing after I've replaced --framework-root . with  --framework-root Source. But it still struggled with error 'AblyDeltaCodec/AblyDeltaCodec.h' file not found. Then there were some manipulations with the make command (make update_carthage_dependencies_<platform>). After that it never appeared again no matter what to cleanup and what --framework-root value was. So I can't say for sure what "fixed" this crash.
But like on previous macOS installation, when I installed sourcekitten separately, it was there again:
sourcekitten doc --objc $(pwd)/Source/Ably.h
  -- -x objective-c
  -isysroot $(xcrun --show-sdk-path --sdk iphoneos)
  -I $(pwd) -fmodules > objcDoc.json

/Users/admin/Work/ably-cocoa/Source/Ably.h:1:9: fatal error: could not build module 'Foundation'
After iphoneos was replaced to iphonesimulator (as suggested in this thread https://github.com/realm/jazzy/issues/840):
sourcekitten doc --objc $(pwd)/Source/Ably.h
  -- -x objective-c
  -isysroot $(xcrun --show-sdk-path --sdk iphonesimulator)
  -I $(pwd) -fmodules > objcDoc.json

/Users/admin/Work/ably-cocoa/Source/Ably.h:9:9: fatal error: 'Ably/ARTLog.h' file not found
And after adding symlink to Source directory with the module name (as suggested in this article https://nicedoc.io/realm/jazzy/ObjectiveC.md):
/Users/admin/Work/ably-cocoa/Ably/ARTDeltaCodec.h:2:9: fatal error: 'AblyDeltaCodec/AblyDeltaCodec.h' file not found.
Also, as that article suggests, playing with clang command might be helpful, as jazzy relies on that, but the thing is that it doesn't work even in our main branch:
clang -c -x objective-c -isysroot $(xcrun --show-sdk-path) -I $(pwd) Source/Ably.h -fmodules
In file included from Source/Ably.h:60:
/Users/admin/Work/ably-cocoa/Ably/ARTDeltaCodec.h:2:9: fatal error: 'AblyDeltaCodec/AblyDeltaCodec.h' file not found
#import <AblyDeltaCodec/AblyDeltaCodec.h>
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Adding -F to pass Carthage frameworks doesn't help (from that article: "If you are happy to build the framework project before generating docs and all the problematic paths have the form ModuleName/PublicHeader.h then have clang resolve those includes to the built framework by passing -F "):
clang -c -x objective-c -isysroot $(xcrun --show-sdk-path) \
  -I $(pwd) Source/Ably.h \
  -F Carthage/Build \
  -fmodules

So, at this point I kinda ran out of ideas. Let me know if the last version of jazzy.sh works for you @lawrence-forooghian (it works for me locally and on CI without errors).

Hey @maratal - I haven't read your above comment in detail, but unfortunately on commit 3f7a94dde182cfaf775b5eb9eedf4a9bcde4f288 I'm still getting the exact same error as before on my machine.


How to have Policy XML for method level of an api

Is there a way to add policy at the method level of an api in this arm template generator for creator tool. Can see api level policy but not on method or Operation level.

currently it's not supported

@SSanjeevi did you find a workaround?

@outifaout  Yes I used api level policy with when condition with operation name filter.

@SSanjeevi could you provide an example?

@outifaout - example is below:
<choose>

            <when condition="@(context.Operation.Name.Equals("myOperationName"))">

                -----Your policy here-------

            </when>

</choose>

Above policy i used at api level and i use the 'myOperationName' value with the operation for which i need to apply a policy.

@SSanjeevi Thank you very much !


Update docker building tool

Generating appliation.yml according to environment variables
Adding Makefile to build binary and docker image

Coverage remained the same at 16.498% when pulling a317558cb43ea9dcdd3eaf67ac34481a08274649 on docker/build into 91ffae2526920e9f7c62a14d210f250608bf030d on master.

Coverage remained the same at 16.498% when pulling a317558cb43ea9dcdd3eaf67ac34481a08274649 on docker/build into 91ffae2526920e9f7c62a14d210f250608bf030d on master.

@hanahmily There is a conflict you need to solve.


enable alternate replacement keys to control case of replaced strings

@CVarisco this branch  addresses my request made in issue #73.  I know you closed the issue saying it wasn't something you wanted to do, and I respect that.  But I still needed this functionality so I built it.
You expressed a desire to leave the existing API unchanged, this commit achieves that.  It ONLY adds new functionality and is completely backwards compatible.  Added functionality is as follows:
--
Templates can now contain 3 additional replacement keys (if desired)

component_name: applies a .toLowerCase() to the specified component name and uses that for replacement
COMPONENT_CAP_NAME: applies a .toUpperCase() to the specified component name and uses that for replacement
cOMPONENT_NAME: applies a .toLowerCase() to the first character of the specified component name and uses that for replacement
--

Just like the existing COMPONENT_NAME replacement, these new keys can be applied to files or to content within a file.
I was unaware of a test suite to write tests for, so there's no update to automated tests.  I'd be happy to add tests if you show me how/where to do that.

Hi @damiansilbergleithcunniff !
Cool! I really appreciate your support and I promise that I'll review your PR as soon as possible.
Probably I'll tell you if need to change your code to make the code more reusable 👍
Thanks again 🏆

Thanks! Take your time, no rush. Your project is so helpful to me. Happy to be able to contribute back.

Bump! :)

@CVarisco good suggestions.  I've updated the code to reflect them.  Again, without automated tests it's a little tough to prove that nothing broke, but I did my best to test the functionality that I'm aware of and it all looked good.
The only part that was troublesome was testing the changes to files.getFileNames.  I couldn't seem to pass the proper inputs in to exercise the code.  You may want to give that a once over, but it should work considering that it's using the same functionality which was tested elsewhere.

Perfect! Thanks a lot for your work 👍
Last thing, can you create the documentation about the feature please?

@CVarisco please review the docs.  I put them in the "custom template" documentation as I figured that was where people would be most likely to leverage them.  Please make sure this is acceptable to you.

Cool man!
Thank you so much for your contribution. I'm publishing a new version of the tool 👍

Awesome!  I've said it before, this project is a HUGE help to me in what I'm doing.  It's let me reach a level of code coverage (and storybooking) that wouldn't be possible without the custom templating.  Thank YOU for building such a great tool and I'm super excited that I get a chance to add to it.

I'm very happy to know that my idea can help someone else 🙂
Thank you a lot for your contribution, I appreciate.