Live Playready content is not playing in Shaka Player demo app.

Mpd file is attached below:
TG_TEST_SD.txt

What version of Shaka Player are you using? - Shaka Player v2.0.5-debug

Can you reproduce the issue with our latest release version? - Yes

Can you reproduce the issue with the latest code from master?

Are you using the demo app or your own custom app? - Demo App

If custom app, can you reproduce the issue using our demo app?

What browser and OS are you using?
Browser - IE11
OS - Windows10

What did you do? - I created a live Dash content with cenc:pssh and mspr:pro data included in mpd. There is no pssh information stored in init.mp4. When I tried to play my content in shaka player (IE11), one frame is being displayed but content is not playing. I did not find any license call in debugger tool nor any console error. I did not understand how it is able to decrypt one frame and why there is no license call. I am attaching my mpd file and screen shot for shaka player & it's network log. I am not getting any console error why playback is not happenning.

What content did you load? - Custom

If standard demo asset, which one?

If custom asset, what manifest and license server URIs?
Manifest url - http://<IP_ADDRESS>/mm/dash/live/7015/LIVESERVICE_1234/TG_TEST_SD.mpd
License Server url - http://<IP_ADDRESS>/drmclientproxy/playready/get_license

How did you interact with the content, if at all?
This content i created locally and using local server for getting license.

What did you expect to happen?
I expected playback to happen

What actually happened?
When I am trying to play this content, it is displaying first frame but not playing the content. It is continuously downloading live segments but there is not license call. When i have pssh data in my init segment and not in mpd then my content is playing fine. But i am trying to use mpd to play my content as pssh data may be different for segments.

Please help me with this issue.
Thanks,
Raj Laxmi

TG_TEST_SD.txt

What packager are you using to encrypt content but not include a pssh?  Most packagers will add a pssh box when they encrypt the content.  Are you sure the content is actually encrypted?
You can include a <cenc:pssh> element in the mpd even if it is in the segments.  The value given in the manifest will override the one given in the content.  The player will ignore any pssh boxes in the content if there are pssh boxes in the manifest.
First I notice the cenc:default_KID attribute in the manifest doesn't match the key id specified in the pssh box.  This will cause the player to throw an error once the license request comes back since the license won't have that key id.  The attribute must either be missing or match the key id the content is encrypted with.
I don't see anything wrong with your content.  It is possible this is a bug in IE.  But without a live URL to test with, we can't help.  Can you please provide a public URL so I can test (you can send it privately if you want, my email is in CONTRIBUTORS).  What you gave is your internal IP, which I can't access.

Sorry for delay in response to your comment.
I have corrected the value of cenc:default_KID attribute in the manifest. It is now equivalent to kid specified in pssh box.
Public url:
Manifest : http://<IP_ADDRESS>/mm/dash/live/7015/LIVESERVICE_1234/TG_TEST_SD.mpd
License Server - http://<IP_ADDRESS>/drmclientproxy/playready/get_license
Thank you for your reply.

That URL works, but I am getting 404 errors for the segments.  Is the stream still running?  Please keep the stream running until this is fixed.

Now the stream works.  I am able to reproduce your error.  This seems to be a bug in Edge.
If you remove the audio stream from your manifest, I can play the video fine.  I am also able to reproduce with our test content by removing the pssh box from the audio init segments (video seems fine with it).
I will create a self-contained repro and submit a bug to Edge.  In the meantime, you'll have to put the pssh in the init segment.  There doesn't seem to be any workaround we can do.

FYI bug filed: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/11016936/

Thank you very much for you support.

I have occasionally seen Edge not send license requests, though not that often.  It is possible that it is storing the licenses on disk and when we request the license, it loads it from disk rather than asking the server.  But I'm not sure.
I don't know how key rotation works in PlayReady.  In Widevine, it works as you describe: media segments contain pssh boxes that describe newer licenses.  As we play the media, the browser gives us those boxes and we request new licences.
Try putting the first pssh box in the init segment and then newer pssh boxes in the media.  Also try putting a breakpoint in DrmEngine.prototype.onEncrypted_ to see if we are getting the new events correctly.  You'll need to use the uncompiled library, but the demo app does by default.
As for testing, Shaka Player doesn't require https.  The demo site redirects because Chrome requires it.  If you are running a local copy of the player on your own web server, it will not redirect.  Additionally, localhost is a secure domain, so Chrome will still allow playing protected content.

Thank you very much for your support. It helped me a lot.
I will try to run a local copy of shaka player to check key rotation playback for playready.

Reported fixed by Microsoft in Edge, but we are unsure what version and have not yet verified the fix.

As the issue [link to https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/11016936/]  seemed fixed, I tried to play a video only live content protected by Playready and has key rotation enabled. Content has pssh box in media content and ContentProtection Descriptor element in MPD; no pssh box in init segment.
When playing the content in Microsoft Edge, Shaka-Player is giving error "Shaka Error DRM.FAILED_TO_GENERATE_LICENSE_REQUEST (InvalidAccessError) -- The CDM was unable to generate a license request for the init data it was given. The init data may be malformed or in an unsupported format.
error.data[0] is an error message string from the browser."
content url: http://<IP_ADDRESS>/mm/dash/live/7015/LIVESERVICE_1234/TG_TEST_SD.mpd
license server url: http://<IP_ADDRESS>/drmclientproxy/playready/get_license

@rajlaxmi28, what version of Edge are you running?

@joeyparrish I am using latest version of Edge (Microsoft Edge 41.16299.248.0).

Hi Joey,
Is there any update on this issue. For your reference i have provided public url to content.
content url: http://<IP_ADDRESS>/mm/dash/live/7015/LIVESERVICE_1234/TG_TEST_SD.mpd
license server url: http://<IP_ADDRESS>/drmclientproxy/playready/get_license
Please provide your feedback on this issue.

@rajlaxmi28, this is an issue with Microsoft Edge, so you will need to ask Microsoft for update: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/11016936/
Although they have marked the bug fixed, they have never stated what version of Edge will contain the fix.

Thank you for your response.

I'm happy to help, and I'm sorry I can't give you an authoritative answer on this.  Hopefully Microsoft will get back to us, and if not, I can try to escalate the issue with them.

That would be very helpful. Much appreciated.

I just confirmed that this is fixed in Edge/16.16299.


Vendor Publishing: Views in Base Path not in Resource-Path

Hi @gravitano,
when I execute vendor:publish all files published into the base path with subdir "views".
That's okay, but when I need to copy vendor views the must be place within resource/vendor.
So can I publish the views from modules within resource/views?

You can edit the view path as you want in your module's service provider file. Please see this stub https://github.com/pingpong-labs/modules/blob/master/Commands/stubs/scaffold/provider.stub#L56-L66.

Thanks!


How do you redirect to a route name in v1?

Hy!
I'm using the beta version and I've noticed that you don't have names in the routes anymore.
Given this routing config:
<Route name='myRoute' path='aaa/:xxx/:yyy/some/:a'/>

How can I redirect properly to this route?
In the previous version I could just do this:
transition.redirect('myRoute', {xxx: 0, yyy: 0, a: 0});

But now it seems that there are no names in the route, so this would not work:
transition.to('myRoute', {xxx: 0, yyy: 0, a: 0});

So the only way to make it work would be to:
transition.to('/aaa/0/0/some/0');

Which is pretty bad...

transition.to('/aaa/0/0/some/0'); is the right way to do it in 1.0.  There's a long discussion about it here: https://github.com/rackt/react-router/issues/1514


K-means stopping condition

Not sure if the condition on the summary is equivalent, but in the slides the k-means stopping condition is given by:

Whereas in the summary it states:

As the differences are always 0 or 1, it is actually equal. But I think it is confusing too. I will change it. Thanks!


Added sshd-PermitEmptyPasswords-no v0.0.1

I believe the failure here is because the dependent module lib-sshd-config is not yet in the index, i expect if the job is re-run after https://github.com/cfengine/build-index/pull/107 is merged that it will green up

I believe the failure here is because the dependent module lib-sshd-config is not yet in the index, i expect if the job is re-run after #107 is merged that it will green up

Or if I use the right commit sha it will work :exploding_head: :rage4:


Publish this project to LuaRocks

This is great! Would you like to publish this to LuaRocks?

@bungle
We have published this repo to LuaRocks now. See https://github.com/tokers/lua-resty-http2/pull/3 for the details.


Supervisor chains/Supervisors returned from backend are using previous runs

Supervisor chains don't seem to be filtered correctly on the backend. When i run the the same code multiple times, it looks like the backend creates the same supervisors again and attach it to the same tool_id. Resulting problematic behaviour that I have different supervisor_ids saved on the client side and server side is returning the old supervisor_ids (that have been registered in the previous run). The supervisors should be associated with a run.

fixed


Update merge editor diff and word diff colors

Uses defaultInsertColor and diffInserted for diffs and word diffs in the merge editor. Fixes #158797.

Eventually, no. I'm doing a separate pass on the hc colors. We're kind of using dark and light colors for their hc counterparts across the board until then.

dark is different from the other colours though because it ends in 33 instead of 66. Is that intentional?

dark is different from the other colours though because it ends in 33 instead of 66. Is that intentional?

See other thread—fixed it.


CON-2888: Prepared the translation system to use a new env var

it now uses the new LOCALE_FILE variable instead of the outdated LANG to determine the correct language file
removed en-AU language file, since the frontend sdk will provide a fallback to en-US

Coverage remained the same at 1.714% when pulling 8035dac330deae9f7e5a0bd7458bc70cd34b3e4d on CON-2888-translations-should-fallback-to into b22c74c6821fe995c31bb6b72ff0feadb84db689 on master.


AppVeyor status checks are never completing for PRs

We are currently running into an issue where status checks for AppVeyor are not reporting back in response to our project's pull requests. The checks are getting stuck indefinitely waiting for status. Functionality was working approximately 3 days ago (we believe).
We still have the push and PR hook configured on the GitHub side and AppVeyor builds are still initiated fine based on these two events. We also still receive pass/fail build notifications via Discord as
expected. However, status checks are never completing for PRs. I have confirmed on both sides that the necessary permissions for our repo and AppVeyor still seem to be in place at the authorizations (AppVeyor) and oauth_application_policy (GitHub organization) areas under settings on each site.
AppVeyor is listed as "Approved" on the GitHub side for our organization. We are also showing that the GitHub repo permissions can be revoked on the AppVeyor side.
I have a support request submitted with AppVeyor and we are awaiting a reply.

AppVeyor case ID #524 was created for this issue. Ilya advised that we revoke OAuth permissions from the AppVeyor side and re-add AppVeyor as a GitHub Application. I've performed these steps and can now see AppVeyor status checks responding properly in PRs again.


Visual isn't showing

Describe the bug
It is not showing the visuals of my code and is just saying booting webcontainer
Link to the blitz that caused the error
https://stackblitz.com/edit/vitejs-vite-uwjsh8?file=index.html
Steps to reproduce
just look to the right i guess
Expected behavior
explanation and fixing of why its not showing
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 109
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 87173166,
  "usedJSHeapSize": 82095922,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 306c160b7022afc5d2e8a4fd7c8489d8cc92c364

Additional context
No response

Hi @kuliksarra. You have to run a command in order to start the project, just as you would do locally. The right side just tells you that WebContainer booted. Maybe we should be more clear and put a "call to action" there to start the project.
I noticed you don't have any package.json or any way to start the project. You could use lite-server by running npx lite-server in the terminal.

Thank you!!
Get Outlook for iOShttps://aka.ms/o0ukef

From: Sam Verschueren @.>
Sent: Friday, January 27, 2023 12:28:26 AM
To: stackblitz/webcontainer-core @.>
Cc: Gorbenko, Sarra @.>; Mention @.>
Subject: Re: [stackblitz/webcontainer-core] Visual isn't showing (Issue #950)
Closed #950https://github.com/stackblitz/webcontainer-core/issues/950 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/stackblitz/webcontainer-core/issues/950#event-8371046582, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXFBSBKBJUA6WH3PAW3CV33WUOBKVANCNFSM6AAAAAAUH2Q2NY.
You are receiving this because you were mentioned.Message ID: @.***>


Revert "Python: skip flaky poetry spec with issue comment"

The associated ticket on Poetry
(https://github.com/python-poetry/poetry/issues/3010) was closed because the underlying triggers were mostly resolved in poetry 1.2 using the new installer. In particular, see
https://github.com/python-poetry/poetry/issues/3010#issuecomment-1207683271.
We flipped to the new installer in
https://github.com/dependabot/dependabot-core/pull/5838, so this skip should no longer be necessary.
This reverts commit ceb57007bc2ca8c403e36bce7c4580a538366dc1.

I think the failure here might be because the test uses python 2, but we dropped support for python 2... so the test needs updating to use python 3.
Need to check if I should just update the python_2.toml/ python_2.lock fixtures completely, or if just update it to python 3... or re-use a different python 3 fixture.


Problems with Autoupdate

Hey, first at all, im realy new in linux, but i worked the last 3 days and googled so many things to get a bit in it. I'm proud of myself that the rootserver and ark server is running very well. ^^
But now i have a problem with my cronjobs.. I've got several cronjobs for, autobackups, broadcasts etc.
but the only cronjobs that won't work for me is arkmanager update... The for me important cronjob because i can't check for updates 24/7 for myself and apply them directly when they are released.
so here is my cronjob:
##Ark Updates##
*/15 * * * * /usr/local/bin/arkmanager update --warn --update-mods --saveworld --safe
0 6 * * * /usr/local/bin/arkmanager update --warn  --update-mods --saveworld --safe
45 11 * * * /usr/local/bin/arkmanager update --warn --update mods --saveworld --safe  (was a try if it works - but it doesn't work)
i also tried without --safe it was just a try today, but it didn't work... i also tried with --arkstagingdir= etc.  nothing works and i don't know why, the cronjobs are right or not?

It doesn't even preload the update in the arkstagingdir
It doesn't warn ingame that an update is coming
It doesn't restart & update the Server

but when i type in arkmanager update by myself it says
"Running command 'update' for instance 'main'
Update already in progress (PID: 24628)"
Cronjobs like -> 32 15 * * * /usr/local/bin/arkmanager broadcast "Question, Suggestions, Problems? Join our Steamgroup [EU-PvP] Rageability or come in the Ts3: <IP_ADDRESS>"   Work without any problem..
please help me, i don't know what i can do to make und running
Sry for my english, i'm German but i think you can understand what i mean

This is my setup:
*/20 * * * * /usr/local/bin/arkmanager update @center --warn
But be warned if your running multiple instances it does not work with the cronjob sadly. Only one of the instances will update. It will only check the folder to see if it is updated or not and not the running instance so I just end up auto updating my center (most played map) and I have to then hand reboot the other 2 when I'm online sadly.

i have update set to
/home/gs-ark/bin/arkmanager update --update-mods --backup --ifempty @all

And it updates all 3 worlds without problems.

Luck you. For me it will do the first server then the 2nd and third will say already updated.
Are you using multiple installs? Or just 1 location?

Multiple directories.
Although you can run multiple servers from one dir, it doesn't mean it will work. it may cause all kinds of problems - learned that hard way (got corrupted arkworld file, thankfully i had backups)

That is why the issue is for us who don't install it multiple times for good reason space usage etc. I'm sure big server sellers don't also have more then one bin installed.
Not sure why you got a corrupted arkworld file from that though because the bins once loaded should not really be reloaded / accessed again unless their doing something really dumb (the ark devs).
 Because the 2nd and third instances will check if there is a update via steam API and see it's fine and ignore it.

Theoretically you can try symlinking ShooterGame/Content folder to your first/default instance, it's the biggest, any try running it from separate folders with shared "Content", since it's biggest dir in install.


fix: clean pending requests

Explanation
chore:

Making sure to clean any pendingRequested related to any opened window which is closed from the user via clicking on "x" button.

enhance:

Adding a button for creating a new identity in the demo website.

More Information
Fixes #187
Closes #14
Related to #140
Related to #183

Screenshots/Screencaps

Before

After

Manual Testing Steps

Pre-Merge Checklist

[x] PR template is filled out
[ ] IF this PR fixes a bug, a test that would have caught the bug has been added
[x] PR is linked to the appropriate GitHub issue
[x] PR has been added to the appropriate release Milestone

+ If there are functional changes:

[ ] Manual testing complete & passed
[ ] "Extension QA Board" label has been applied

PR template source from github.com/MetaMask

Hi @0xmad thanks so much for your comments.
I tried to move the code back to BrowserUtils class, but there is a cyclic dependency happened with BrowserUtils and RequestManager please review my workaround to resolve this. Thanks.


Configuration: Check in workflows being used into a repository

Describe the bug
There are multiple workflows in the DRISTI product. These need to be checked into a repository somewhere so a script can be written to configure the workflows post installation. This is essential for scale-up. Please check-in the workflow JSONs into the MDMS repo under a workflowConfigurations folder for easy reference.

@manimaarans What we checked in into docs folder is what is actually configured? I don't think so. The final set of workflows that are actually configured, being used should be checked into the configs repo under a "workflows" folder.
The point is, someone should be able to take the JSON and run a script to configure all workflows for all modules.

@manimaarans as per discussion on Nov 26, please update the workflows in the develop branch with the latest from SDC production.

@Ramu-kandimalla This is done, right? Why do we still have this ticket open? CC: @suresh12


Add support to the group panel for the Radio Button component

Created the PanelGroup Component to group any component.


support async call on windows

While a long time running task be dispatched with a flag such as #async on windows, shell2http return immediately.
set chrome="C:\Progra~2\Google\Chrome\Application\chrome.exe"
D:\bin\shell2http\shell2http -host=<IP_ADDRESS> -port=8081 -no-index -log=access.log -form ^
  /html2pdf#async  "echo OK && %chrome% --headless --disable-gpu --print-to-pdf-no-header --print-to-pdf=%%v_file%% %%v_url%%"

I think, it's a duplicate of https://github.com/msoap/shell2http/issues/70

Thanks for your works, msoap.
There is no command like nohup on windows.   Requests will be blocked like this:
shell2http -port=8081 /longrun   "start /b ping -n 5 <IP_ADDRESS> >> D:\x.log"
shell2http -port=8081 /longrun   "cmd /c ping -n 5 <IP_ADDRESS> >> D:\x.log"


Add note for complex webhook actions.

@jefffanjoy : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Dukek is out.  I maintain these docs and approve this change.

@ktoliver FYI.

@rboucher Thanks. I'll change some text in the updates to use contractions per Microsoft style and then merge.

Sounds good.  That's save me some time from doing it later.  Thanks Kristine!


[libass] Fix build error on x64-windows-static

Describe the pull request

What does your PR fix? Fixes #12392
Fix build error on x64-windows-static:

CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
FRIBIDI_LIBRARY

Which triplets are supported/not supported? Have you updated the CI baseline?
Yes
Does your PR follow the maintainer guide?
No

/azp run


refactor(semantic): refactor UnusedLabeled for improved readability and performance

Current dependencies on/for this PR:

main

PR #2183   👈

PR #2184

This stack of pull requests is managed by Graphite.


Minor improvements to get-nightlies

$ doozer --quiet -g openshift-4.9 get-nightlies --latest --rhcos
4.9.0-0.nightly-2022-03-09-205109 Ready 49.84.202203081945-0
4.9.0-0.nightly-s390x-2022-03-09-213958 Accepted 49.84.202203082102-0
4.9.0-0.nightly-ppc64le-2022-03-09-211420 Accepted 49.84.202203091929-0
4.9.0-0.nightly-arm64-2022-03-09-215559 Ready 49.84.202203090103-0
4.9.0-0.nightly-2022-03-09-205109,4.9.0-0.nightly-s390x-2022-03-09-213958,4.9.0-0.nightly-ppc64le-2022-03-09-211420,4.9.0-0.nightly-arm64-2022-03-09-215559

Codecov Report

Merging #571 (dec65ad) into master (652ad08) will increase coverage by 29.79%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master     #571       +/-   ##
===========================================
+ Coverage        0   29.79%   +29.79%
===========================================
  Files           0       44       +44
  Lines           0    10649    +10649
  Branches        0     2345     +2345
===========================================
+ Hits            0     3173     +3173
- Misses          0     7256     +7256
- Partials        0      220      +220

Impacted Files
Coverage Δ

doozerlib/cli/get_nightlies.py
0.00% <0.00%> (ø)

doozerlib/util.py
47.80% <0.00%> (ø)

doozerlib/pushd.py
90.00% <0.00%> (ø)

doozerlib/rhcos.py
69.01% <0.00%> (ø)

doozerlib/distgit.py
34.55% <0.00%> (ø)

doozerlib/brew.py
41.59% <0.00%> (ø)

doozerlib/cli/images_streams.py
0.00% <0.00%> (ø)

doozerlib/cli/release_gen_assembly.py
0.00% <0.00%> (ø)

doozerlib/cli/release_calc_upgrade_tests.py
0.00% <0.00%> (ø)

doozerlib/plashet.py
82.08% <0.00%> (ø)

... and 35 more

Continue to review full report at Codecov.

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


added editable-placeholder for editable-select default option

fix #423, usage:  <span editable-select="field" editable-placeholder="Select element..." e-ng-options="...">

+1

@david12341235, please include in PR only src/* files

this can be closed, it was added in #463


nushell: fix build by using newer cargo fetcher

Resolves #80117 by using the newer fetcher; see #79975 for details.
Would also be fixed by #80153 eventually, but we want to upgrade Rust packages
either way, so might as well start with the broken ones!

Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build nushell


Update dependencies to more recent version to reduce security concerns

Need to update related dependencies to reduce security vulnerabilities.
This will be based on some of the findings found using dependency-check-maven and related checks.

There may be some overlap with https://github.com/ebresie/python4nb/issues/16 which is Git Action based while the above is based on addition of maven plugins to assess things.

As part of this, additional maven profiles have been added which include:

security-test
static-analysis-check
coverage

There are also a few other commented profiles to demonstrate possible used of os, jdk, properties, and/or activation attributes should these be used at a later time.
To evaluate this, use the security-test profile which will run security analysis.  There may be some overlap with existing GitHub Security and dependabot analysis.

Due to above commit, will close this one for now.  In future will need to run it prior to next release (so a "release checklist" item)


Don't recommend jsdom in the README

I just ran into #4.
While the togeojson executable uses xmldom, the README still recommends using jsdom.

Good catch, can you roll a PR implementing this change?

Sure, done.


net: dummy L2 for offloaded ifaces [TEMP]

Temporary PR, do not merge.
This is a temporary PR for a commit that I plan to prepend to the history of another PR (#53118). This is to give an opportunity for first-pass review before/while I work on rebasing the original PR on top of the commit.
Adds dummy link layer for offloaded ifaces, allowing ifaces to directly receive l2_enable calls
Signed-off-by: Georges Oates_Larsen<EMAIL_ADDRESS>

Closing this as the commit is now part of the original PR


About ResNeSt 269 ?

I want to know how to support the ResNeSt269 model. How can I convert this repo ResNeSt model weights for ImageNet pre-train usage?

I have converted model weights ResNeSt200, the result is the same as the open_mmlab.
Here is the code.
resnest269 = torch.load("resnest269-0cc87c48.pth")
results = {}
results["meta"] = {}
results["state_dict"] = {}
for key, value in resnest269.items():
    if key.endswith("num_batches_tracked") or key.startswith("fc"):
        continue
    if key.startswith("conv1"):
        key = key.replace("conv1", "stem")
    if key.startswith("bn1"):
        key = key.replace("bn1", "stem.7")
    results["state_dict"][key] = value
torch.save(results, "resnest269_d2-0cc87c48.pth")


extract thisBootstrap method to avoid too many @SuppressWarnings

Motivation:
There are many @SuppressWarnings("unchecked") in the code for the same purpose that we want to do this return:
@SuppressWarnings("unchecked")
public B someMethod() {
    ......
    return (B) this;
}

Modification:
Add a method thisBootstrap() and reuse in all these return lines:
@SuppressWarnings("unchecked")
    private B thisBootstrap() {
        return (B) this;
    }

Result:
Then only one @SuppressWarnings("unchecked") left in the code.

LGTM... will merge once the CI completes.

@skyao what's the actual benefit? thisBootstrap also sounds a bit weird.

The benefi is to reduce the count of @SuppressWarnings("unchecked")  from n (in this class it is 7) to just ONE.
This style is common in many open source project, I just want to follow the style and move it to netty.
If you agree with this, I would try a search and update all similar cases.

@skyao we use the name self() for other cases like this. Can you please change thisBootstrap() to self()?

Cherry-picked into 4.1 (47f016bf564e9b86571fcb04d42f3ae327d0f58d) and 4.0 (5e6dd3778e).
@skyao thanks!


Fixed #32469 -- Made assertQuerysetEqual() respect maxDiff when ordered=False.

Fixes ticket-32469. This may interest you @adamchainz given your love of all things testing?

I don't think documentation changes or release notes are needed. Generally there's little value in including versionchanged annotations when none of the surrounding documentation has changed.
Incidentally, I'm unsure that assertQuerysetEqual provides much value these days.
In the new test example, I would instead use:
self.assertCountEqual([p.name for p in Person.objects.order_by('name')], names)

I don't think documentation changes or release notes are needed. Generally there's little value in including versionchanged annotations when none of the surrounding documentation has changed.

I guess that's fair, particularly the second point.

In the new test example, I would instead use:
self.assertCountEqual([p.name for p in Person.objects.order_by('name')], names)

I really don't follow. How would that be testing that assertQuerysetEqual() supports maxDiff?

My thought was that it might be better to promote assertCountEqual and assertSequenceEqual rather than to enhance assertQuerysetEqual (particularly after assertQuerysetEqual no longer calls repr() by default). But it looks like assertSequenceEqual doesn't work when both arguments are QuerySets so I guess assertQuerysetEqual still provides value. But at least in Django's test suite, I'd advocate for only using assertQuerysetEqual to compare two querysets. I don't insist you modify your test. It's just an idea for a separate discussion. Sorry that wasn't clear.

@pope1ni Thanks :+1: I pushed small edits to tests.

But at least in Django's test suite, I'd advocate for only using assertQuerysetEqual to compare two querysets.

We've replaced ~600 assertQuerysetEqual() to assertSequenceEqual()/assertCountEqual() in 3f7b3275627385f8f7531fca01cdda50d4ec6b6e.


adduser is a Debian-ism

adduser is a Debian frontend for useradd. It's not available on e.g. Archlinux. I don't have a proposed solution; I'm not sure if there's a nice portable way to add a system user.
https://github.com/juga0/dhcpcanon/blob/master/Makefile#L82

The nice way would be to check which Linux distribution is the system running, something to do at some point.
Somebody was going to package it for Archlinux, but i didn't heard back.

Somebody was going to package it for Archlinux, but i didn't heard back.

That might have been me, sorry. :)

it wasn't you :)

Here you go: https://aur.archlinux.org/packages/dhcpcanon/ (needs thorough testing) :)
The recommended way to add system users in archlinux is sysusers.d(5).


filtering reflected tables?

Hello @arthurio , this is more of a question than an issue.
The examples shown seem to assume that you know and can dictate the schema.  What about reflected tables?  Here's some minimal code to illustrate:
class CustomerFilter(Filter):
    customer: str = Field(Query(default = "123456"))
#    class Constants(Filter.Constants):
#        model = Customer
#    Some sort of reflected model needs to go here?  do we dump a reflected
#    schema from the table and reproduce it?

@router.get("/filter-reflected-table/")
async def filter_reflected_table(
    customer_filter: CustomerFilter = FilterDepends(CustomerFilter, by_alias=True)
):
    meta = MetaData()
    engine = create_engine(DB_URL)
    table = Table(SOME_TABLE, meta, autoload_with=engine)  # table reflected here
    query = select(table).filter(table.c.customer == "123456")  # this works
    query = customer_filter.filter(query)  # this dies
    with Session(engine) as session:
        result = session.execute(query)
        return result.fetchall()

The traceback cries about the Constants not being defined, here's a snippet:
    query = customer_filter.filter(query)  # this dies
  File "/usr/local/envs/fastapi/lib/python3.9/site-packages/fastapi_filter/base/filter.py", line 214, in filter
    return original_filter.filter(*args, **kwargs)
  File "/usr/local/envs/fastapi/lib/python3.9/site-packages/fastapi_filter/contrib/sqlalchemy/filter.py", line 89, in filter
    model_field = getattr(self.Constants.model, field_name)
AttributeError: type object 'Constants' has no attribute 'model'

So my question is how do we construct the filter class "correctly" to work with a table that's reflected from some other DB?  As per the comments, is there a way to pass a reflected model that gets filled at runtime?  Do we have to dump the reflected model and re-declare it manually in order to be able to use it?  The latter seems like a bad practice, but if that's the only way, what is the correct way?

Hi @BadToast , sorry for the long delay before replying!
I'm not super familiar with reflected tables so thank you for the example it helps a lot.
So yes the filter class has to access the model through it's Constant subclass in order to work.
Now in your example, you declare your table inside the endpoint which I don't think is a good practice, otherwise you'll end up invoking that code on every request. Did you try to create your reflected table outside of the endpoint and setup your filter afterwards? Something like:
# Somewhere near where you define your FastAPI.App
meta = MetaData()
engine = create_engine(DB_URL)
CustomerTable = Table("customers", meta, autoload_with=engine)  # table reflected here

# Somewhere else, e.g. `filters.py`
from . import CustomerTable

class CustomerFilter(Filter):
    customer: str = Field(Query(default = "123456"))
    class Constants(Filter.Constants):
        model = CustomerTable

@router.get("/filter-reflected-table/")
async def filter_reflected_table(
    customer_filter: CustomerFilter = FilterDepends(CustomerFilter, by_alias=True)
):
    query = select(table).filter(table.c.customer == "123456")  # this works
    query = customer_filter.filter(query)  # this should work as long as `customer` is the right field name to access?
    with Session(engine) as session:
        result = session.execute(query)
        return result.fetchall()

@BadToast Thanks for reporting back. I guess we would have to detect the type of model/table passed to the filter and access the attributes differently for generated ones (using reflection). So I think it's technically possible but requires some dig in.
I'm glad that you end up with something that will work for you and I'm sure that you'll see a lot of benefits from having explicitly defined models in your code. From a quick google search you might want to look at https://github.com/agronholm/sqlacodegen to help you generate your models initially, then you can tweak manually.
Let's close this issue for now and we can revisit if someone cannot do without reflected tables.


ROX-20122: set gRPC & http2 max concurrent streams to 100

Description
Manual cherry-pick of https://github.com/stackrox/stackrox/pull/8124 due to merge conflicts.
Checklist

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

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/retest

/test ocp-4-13-ebpf-qa-e2e-tests osd-gcp-qa-e2e-tests ocp-4-10-ebpf-qa-e2e-tests

/test ocp-4-13-ui-e2e-tests  osd-gcp-qa-e2e-tests ocp-4-13-ebpf-qa-e2e-tests gke-race-condition-qa-e2e-tests

/test gke-race-condition-qa-e2e-tests

/test gke-race-condition-qa-e2e-tests osd-gcp-qa-e2e-tests

gke-race-condition-qa-e2e-tests is failing regularly on this release branch, going to merge.


请教一个问题

请问为啥我在引入了defalut.css，也导入了highlight.js,代码却没有突出显示

Need more english, please?

Defalut.css is introduced and highlight.js is imported, but the code is not highlighted. Why?

I dunno, you're going to have to provide more information or a reproducible sample.

ok，My HTML is as follows:

class Node{
    constructor(data){
        this.data = data;
        this.next = null;
    }
}

Not sure why none of your tags are closed? Paste error?
That looks reasonable... any errors in console?

It's actually closed, and I didn't shut it down on purpose because github doesn't support sending such messages

github doesn't support sending such messages

What such messages? It works fine if you quote it properly (as I did when I edited it to fix the quoting).

The truth is this：

class Node{
    constructor(data){
        this.data = data;
        this.next = null;
    }
}

请问我就是这样导入了所需的css和js，为什么不起作用呢？

The truth is this：

class Node{
    constructor(data){
        this.data = data;
        this.next = null;
    }
}

I just imported the required CSS and js. Why didn't it work?

I asked: any errors in console?

No problem pops out of the console

Did you read the readme?  You have to actually execute the library.

how to execute the library?Just HLJS. InitHighlightingOnLoad (); Not enough, is it？

Well, that's close... the exact code is in the README.

Do I need do this?
document.addEventListener('DOMContentLoaded', (event) => {
document.querySelectorAll('pre code').forEach((block) => {
hljs.highlightBlock(block);
});
});

No, that's what initHighlightingOnLoad does if you call it.

initHighlightingOnLoad  , I have done, anything else I should  do ?

That's the idea.

That's what I did, but。。。

Thank you very much ! I have known how to solve this problem. Because it Unable to highlight the code for fetching incoming documents from the back end of the interface.

Huh?

Because I initialized it first, and the part of the document that includes the code is returned by ajax.
It is estimated that it is the data returned by ajax after the front end has been rendered, and the previous initialization will not work.
Thank you very much for your patience in answering my questions
My English is not very good, please forgive me

It is estimated that it is the data returned by ajax after the front end has been rendered, and the previous initialization will not work.

Right then you'll need to write custom JS to do it afterwards, which the README also explains.


OpenGL Extension Caching

This caches OpenGL extensions that are not part of the core profile. It uses a struct of Boolean values to avoid function call overhead as @rextimmy suggestion. Extensions that run during initalization are not cached, as there really is no reason to as of now. It is fairly easy to add more extensions to the cache as needed.
This PR also removes the old extensions that are no longer needed.
Finally, it replaces a call of glGenMipmapEXT to glGenMipmap, as we are in the core profile now!

Also I meant 3.2 on my description of the last commit, sorry!!

Uh oh tabs vs spaces indention strikes again. ;)

Seems to work fine.

great to hear!

Wich Version of OpenGL is it ? If I'm not wrong, there isnt any description of it!

@BadFactor OpenGL 3.2 core profile is the min. required to run Torque3d.


Update tests to ensure model instance are always valid

Proposed Changes
Update tests to always validate on save, as an example, the following is missing status being defined
https://github.com/nautobot/nautobot/blob/af9be7678b0ee402b36b7201e2d7a886e8835334/nautobot/dcim/tests/test_filters.py#L552-L590
Justification
In using the <model>.objects.create() instead of instance = <model>(); instance.validated_save() will allow model instances to created that are not actually valid records, such as not providing a status where one is required. This leave the API and UI in a bad state in a production environment, and tests should better reflect an actual production environment.

Possibly consider fixtures or snapshot startup of valid data for test runs.

See also #1999, #1334.

Generally addressed by our ongoing migration to use data factories rather than hand-crafted data. The issue with omitting status fields in particular was due to a bug in our definition of the StatusField class (see #3785).


Markdown parser produces incorrect results

(I had a concrete code example, but I think this is really not relevant).
Current situation is: everything looks good in Gitlab / Github / Visual Studio Code markdown preview, and it comes out broken using the plugin-updater.
I see that parsedown is used as a Markdown parser. I checked with their online demo and it's broken too (the result for Markdown PHP 1.3 is OK).
It seems that this thing is not very compliant with commonmark or even original markdown (using a simple <p> tag makes it choke); and also seems to be unmaintained, with 70 open issues and 45 open pull requests.
They say it's fast, but I don't really care about how fast the parser is, I would prefer it to be correct.
I'd be up to create a PR ripping it out and replacing with another library. Thoughts?

Maybe it makes sense to replace it with https://packagist.org/packages/league/commonmark ?

Maybe it makes sense to replace it with https://packagist.org/packages/league/commonmark ?

Maybe it makes sense to replace it with packagist.org/packages/league/commonmark ?

Yes, this looks really good. Also, it's extensible / configurable. PR coming up...

Maybe it makes sense to replace it with packagist.org/packages/league/commonmark ?

Yes, this looks really good. Also, it's extensible / configurable. PR coming up...


[FEATURE REQUEST] How to do inline CSS

I am trying to inline my critical CSS for a faster initial load.
I tried the following pug code:
style !{require('../css/styles.css?raw')}

And in my webpack config, I put:
{
    resourceQuery: /raw/,
    type: 'asset/source',
    use: 'css-loader',

}

But this throws the following error:
ERROR in [entry] [initial]
[pug-plugin] Can't resolve the file /home/marcel/code/biography-site/src/css/styles.css?raw in /home/marcel/code/biography-site/src/pug/index.pug
The reason: this file not found!
/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:34
  throw new PugPluginException(lastError);
  ^

PugPluginException:
[pug-plugin] Can't resolve the file /home/marcel/code/biography-site/src/css/styles.css?raw in /home/marcel/code/biography-site/src/pug/index.pug
The reason: this file not found!

    at PugPluginError (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:34:9)
    at resolveException (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:77:3)
    at Resolver.require (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Resolver.js:305:5)
    at /home/marcel/code/biography-site/src/pug/index.pug:1:955
    at Script.runInContext (node:vm:141:12)
    at PugPlugin.renderModule (/home/marcel/code/biography-site/node_modules/pug-plugin/src/index.js:718:33)
    at PugPlugin.renderManifest (/home/marcel/code/biography-site/node_modules/pug-plugin/src/index.js:652:28)
    at Hook.eval [as call] (eval at create (/home/marcel/code/biography-site/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:7:16)
    at Hook.CALL_DELEGATE [as _call] (/home/marcel/code/biography-site/node_modules/tapable/lib/Hook.js:14:14)
    at Compilation.getRenderManifest (/home/marcel/code/biography-site/node_modules/webpack/lib/Compilation.js:4483:36)

It seems like a problem with this plugin, because if I delete the style tag from the pug code and move the contents of require() to a second entry in my webpack config, it compiles without problems.
I also tried with asset/inline, but that is putting the base64 encoding of the styles in the <style> tag.
{
    resourceQuery: /raw/,
    type: 'asset/inline',
    use: 'css-loader',

}

My source for type: 'asset/source': https://webpack.js.org/guides/asset-modules/
Is there a way to achieve this?

Hi @MarcelRobitaille,
to inline CSS in Pug use include:
style: include ../css/styles.css

You don't need to configure Webpack for this.
Using type: 'asset/inline' Pug plugin can inline images like SVG, PNG, JPG, etc., but to inline CSS must be used native Pug feature.

Hi @webdiscus. Thanks for the response. I'm aware of pug's include keyword. However, this includes the file totally raw exactly as it is in src. I would like to minify the styles using webpack. That's why I am trying to use require() with asset/inline. Is there really no way to do this? Thanks

Yes, it makes sense. I note this feature request in TODO for next release.

@MarcelRobitaille,
Pug plugin has full control over all required assets in Pug and the inlining CSS must be implemented in the plugin.
It is in development.

@webdiscus That is great! Thank you so much

@MarcelRobitaille
please update to v4.4.0.
For inline CSS and load CSS as file use in Webpack config the oneOf:
{
  test: /\.(css|sass|scss)$/,
  oneOf: [
    // inline styles in HTML
    {
      resourceQuery: /^\?raw/u, // match exact first URL query `?raw`
      type: 'asset/source',
      use: ['css-loader', 'sass-loader'],
    },
    // load styles as file
    {
      use: ['css-loader', 'sass-loader'],
    },
  ],
},

Then you can use both:
//- load CSS as file
link(href=require('./main.scss') rel='stylesheet')

//- inline CSS from styles.scss
style=require('./styles.scss?raw')

Note
The IDEA (JetBrains) don't understand the style !{require('./styles.css?raw')} syntax.
I use Escaped String Interpolation: style=require('./styles.scss?raw') w/o linter error in my IDE.

Here is the test case for inline CSS with postcss plugin cssnano.
The inlined CSS support the sourceMaps. If in Webpack config is defined devtool: 'inline-source-map' then source map will be inlined too and it works in browser fine.
Question:

what should be with devtool: 'source-map'? To inline map like inline-source-map or ignore the map, because for inlined CSS not exists an original CSS file?

P.S. thanks for donate :-)

@webdiscus Thank you so much! :rocket: I will try it now.

what should be with devtool: 'source-map'? To inline map like inline-source-map or ignore the map, because for inlined CSS not exists an original CSS file?

I am not sure the best way to handle this to be honest. Sorry

P.S. thanks for donate :-)

No prob. I figure I bugged you enough that I should support you somehow. I'm not sure how long I can afford to stay a patron though.

I just tried 4.4.0. It seems like it's not handling url() in the CSS anymore.
ERROR in [entry] [initial]
[pug-plugin] Can't resolve the file ../svg/background-tile.svg in /home/marcel/code/biography-site/src/css/styles.css?raw
/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:34
  throw new PugPluginException(lastError);
  ^

PugPluginException:
[pug-plugin] Can't resolve the file ../svg/background-tile.svg in /home/marcel/code/biography-site/src/css/styles.css?raw

    at PugPluginError (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:34:9)
    at resolveException (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Exceptions.js:77:3)
    at Resolver.require (/home/marcel/code/biography-site/node_modules/pug-plugin/src/Resolver.js:309:5)
    at /home/marcel/code/biography-site/src/css/styles.css:5:37
    at Script.runInContext (node:vm:141:12)
    at PugPlugin.renderModule (/home/marcel/code/biography-site/node_modules/pug-plugin/src/index.js:755:33)
    at PugPlugin.renderManifest (/home/marcel/code/biography-site/node_modules/pug-plugin/src/index.js:683:28)
    at Hook.eval [as call] (eval at create (/home/marcel/code/biography-site/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:7:16)
    at Hook.CALL_DELEGATE [as _call] (/home/marcel/code/biography-site/node_modules/tapable/lib/Hook.js:14:14)
    at Compilation.getRenderManifest (/home/marcel/code/biography-site/node_modules/webpack/lib/Compilation.js:4483:36)

.foo {
	background-image: url(../svg/background-tile.svg);
}

If I remove that line, it's working perfectly!

@MarcelRobitaille
to implement support of URL in inline CSS using asset/source is very complex, because Webpack for asset/source type don't analyse/parse the source and don't resolve a resource in CSS. The asset/source exports the source code of the asset, e.g. for text files. Webpack simple inject content of file. This type is not suitable for inline CSS.
I must implement the resolving of resources in CSS self. It will take a while.
As alternative syntax for inline CSS I can suggest following:

variant 1

//- load CSS as file
link(href=require('./styles.scss') rel='stylesheet')

//- inline CSS from styles.scss
link(href=require('./styles.scss?inline') rel='stylesheet')

Then the link tag will be replaced with <style>... inline CSS content ...</style>.

variant 2

//- load CSS as file
link(href=require('./styles.scss') rel='stylesheet')

//- inline CSS from styles.scss
style=require('./styles.scss?inline')

For both variants, the Webpack config still be unchanged, also w/o a type:
{
  test: /\.(css|sass|scss)$/,
  use: ['css-loader', 'sass-loader'],
},

Variant 2 can be implemented faster. I will implement this variant.
Note:  for both variants the URL query ?inline will be hard & fix coded. It will be as marker to inline the file.

@webdiscus Thanks for the updates. I didn't expect making url() work would be so complicated. So, if I understand correctly, the workaround is to link the stylesheet in the normal way (not inline)? I assume this helps webpack recognize the imports somehow? Is it possible for this to be commented so the full stylesheet doesn't actually load or something like that?

For asset/source type Webpack inject a source without processing and urls will be not resolved in Webpack.
To inline CSS with supports url(), the style file must be processed (parsed und resolved all URLs) via Webpack as javascript/auto type. Then Webpack call special hooks for each asset (image, font, etc) resolved in url(). In this hooks I can resolve output asset files. Well, it is complex process, but I know wath to do. I'm in process... ;-)

Can I just put the css file as an entry in webpack without linking it to my HTML?

Can I just put the css file as an entry in webpack without linking it to my HTML?

Yes, of cause. The Pug plugin can extract CSS from a style (css, scss, etc) defined in Webpack entry into separate file without linking to a Pug/HTML.
Webpack config:
module.exports = {
  output: {
    path: path.join(__dirname, 'dist/'),
  },

  entry: {
    styles: './src/assets/main.scss', // => dist/asset/css/styles.8abc765d.css
  },

  plugins: [
    new PugPlugin({
      // extract CSS from Pug or from Webpack entry
      extractCss: {
        filename: 'asset/css/[name].[contenthash:8].css' , // output CSS filename
      },
    }),
  ],

  module: {
    rules: [
      {
        test: /\.(css|sass|scss)$/,
        use: [ 'css-loader', 'sass-loader'],
      },
    ],
  },
};

Note
An additional plugin and loader, such as mini-css-extract-plugin, is not required.

To be clear, my question was if adding the CSS file as another entry in webpack would satisfy the first line of variant 1 and variant 2 from here.
I have just tried this, and I am getting the same error. I also tried variant 2 exactly as you have it. Neither of these variants has been implemented yet, right?

To be clear, my question was if adding the CSS file as another entry in webpack would satisfy the first line of variant 1 and variant 2 from here.
I have just tried this, and I am getting the same error. I also tried variant 2 exactly as you have it. Neither of these variants has been implemented yet, right?

All my variants were implementation suggestions. It will be implemented. He's not ready yet.
See my comment above:
 I'm in process... ;-)

Yes ok that was what I thought. I just wanted to see what would happen

@MarcelRobitaille
update please to v4.5.0. In this version is implemented the variant 2.
To inline CSS use the ?inline query:
//- load CSS as file
link(href=require('./styles.scss') rel='stylesheet')

//- inline CSS from styles.scss
style=require('./styles.scss?inline')

Webpack config rule for styles:
{
  test: /\.(css|sass|scss)$/,
  use: ['css-loader', 'sass-loader'],
},

@webdiscus It's working perfectly! Thank you so much.

//- load CSS as file
link(href=require('./styles.scss') rel='stylesheet')

//- inline CSS from styles.scss
style=require('./styles.scss?inline')

I totally misunderstood this. I thought you were saying I needed to have both tags in order for the inline minification and linking to work. That lead to the confusion in later comments. Sorry about that.

@MarcelRobitaille
I wanted to demonstrate both usage cases in one example.
It would be correct to demonstrate:
Usage case load CSS as file
link(href=require('./styles-one.scss') rel='stylesheet')

Usage case inline CSS
style=require('./styles-two.scss?inline')

Both usage cases are independent of each other.

@MarcelRobitaille
is everything working? Can be closed the issue?

Working beautifully. Thank you :heart:


⚠ Samsung has degraded performance

In a1b64f2, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1822 ms

Resolved: Samsung performance has improved in e02fc15.


No transparency

I tried everything but can't get transparency to work.
I exported from 3 different programs (Cinema 4D, Blender and 3dsmax) in all kinds of formats (.obj, .fbx, .3ds, .dae and so on) but every time neither the iOS App I wrote, nor the provided os x App shows any kind of transparency. Is this a well known missing feature or am I missing something?

@LucasZL Please attach a sample file (any 1 format will do) and screenshots of expected render vs what you get as that really helps to understand the problem clearly. Thanks.

Btw, transparency is supported; so it should work.

Sure. I can't give you access to the files I tried yesterday as I am not allowed to share them but I tried a simple Cube with transparency and it's also not working.
This is what I am seeing in Cinema 4D:

And this is what I am seeing in the OSX Sample program:

Now I just noticed that even the color is not showing...
Here is the obj File and the mtl File in which you can see that a texture is applied with transparency and color:
TransparentCube.zip

@LucasZL

Sure. I can't give you access to the files I tried yesterday as I am not allowed to share them

That's understandable, no problem.
The sample file you have provided is sufficient for testing this bug. Thanks.

Just tested with latest master. even with release v 1.2.0 you will get the colors. Please check what version you are using. But, yes there is the transparency not correctly applied and is a bug.


rpc error: code = Unavailable desc = upstream connect error or disconnect/reset before headers. reset reason: connection termination

I run client  -repeat 20, without istio this OK;
but when i inject istio sidecar, meet with
rpc error: code = Unavailable desc = upstream connect error or disconnect/reset before headers. reset reason: connection termination

i open mtls,  frontend request is encryption? It is mtls cause?

i open mtls, frontend request is encryption? It is mtls cause?

Sorry for the late response. This example assumes that mTLS is disabled.


Blocks render error related to undo/redo

Describe the bug
Blocks can be rendered offset from their connected parent. This occurs in both shadow blocks and statement blocks.

To Reproduce
Steps to reproduce the behavior:

Start a new project.
Add to start a for index from 0 to 4 block.
Add another for loop inside that loop.
Move inner loop to workspace.
undo
redo

Repeat undo/redo for accumulated offset of shadow blocks
Blocks are still connected in the AST. Save and reload corrects rendering.
Error in console:
pxt.azureedge.net/blob/26d4ed41211a547ba192bde698e4a7df49f28bc4/pxtblockly.js:1241 Can't uncreate non-existent block: ZOA@WH$kv(9HPSi(h!3W
Blockly.Events.Create.run @ pxt.azureedge.net/blob/26d4ed41211a547ba192bde698e4a7df49f28bc4/pxtblockly.js:1241

Expected behavior
Undo and redo should not impact rendering.
Desktop (please complete the following information):

OS: Mac
Browser Chrome
Version 71.0.3578.98

@shakao  fixed?

yes, fixed in beta

fixed


GP_MCMC and EI_MCMC and integrating hyperparameters

Dear Javier!
I want to solve an optimization problem and "concurrently" optimize the hyperparameters of covariance function. In other words, we are interested to initialize the hyperparametes at very first step by some values, then improve them at each iteration. (there is some reason for this simultaneous goal).
My question is does the GP_MCMC and EI_MCMC is fit for this goal? I see a notebook is available entitling "GPy_Opt_integrating_model_hyperparameters" however, I am not sure if that is similar to my case (sorry I am very new to this statistical machine learning concept).
Thanks for your help!

Both, with MCMC and when you do MLE, the hyper-parameters of the model are updated after each iteration. Now, by setting ```save_models_parameters= True`` when running the optimisation you can also save the hyper-parameters of all the created models.
Hope this help!


Show document upload card in regular message format

Motivation and Context

messages indicating a successful document upload should be shown in the typical message format.
Description

moves code from ChatHistory used to render regular messages into ChatHistoryItemTextContent
changes ChatHistoryFileItem to ChatHistoryItemDocumentContent and removes the JSX used for rendering the header

Preview

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[X] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Nit: would be cleaner to add the new components under a new ChatHistoryItem folder under /components/chat and remove the ChatHistoryItem prefix on the content components

Nit: would be cleaner to add the new components under a new ChatHistoryItem folder under /components/chat and remove the ChatHistoryItem prefix on the content components

@teresaqhoang created a chat-history/ folder instead which contains ChatHistory and ChatHistoryItem, and also renamed the content components to ChatHistoryTextContent and ChatHistoryDocumentContent. it didn't feel right just having TextContent and DocumentContent components but let me know if you think otherwise


[Access Request] Page 5786 - Source Document Filter Card - InitializeControls

Hi Al team,
I would like to access the following function from the extension object
// NEW >>>>
   // local procedure InitializeControls()
   protected procedure InitializeControls()
   // NEW <<<<
   begin
       ShippingAgentServiceFilterEnable := true;
       ShippingAgentCodeFilterEnable := true;
       InboundTransfersEnable := true;
       SalesReturnOrdersEnable := true;
       PurchaseOrdersEnable := true;
       OutboundTransfersEnable := true;
       PurchaseReturnOrdersEnable := true;
       SalesOrdersEnable := true;

       OnAfterInitializeControls();
   end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Restore previous note text version

To improve the UX, add the feature to replace the currently saved note text with the last saved one. This could be accomplished via another button when the note is non-editable. The simplest implementation would introduce another column in the Note entity to store the previous version. Bringing back a note whose version number is the first and with an empty historicized note text should not be possible. This feature has to take the recycle bin one into account as well

Too complex for the original scope of the app. Won't be implemented now


p2os: 2.0.6-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository p2os to 2.0.6-0:

upstream repository: https://github.com/allenh1/p2os
release repository: https://github.com/allenh1/p2os-release.git
distro file: lunar/distribution.yaml
bloom version: 0.5.26
previous version for package: null

p2os_doc
* Updated p2os_doc email address.
* Contributors: Hunter L. Allen

p2os_driver

No changes

p2os_launch
* Added an enable motors launch file.
* Contributors: Hunter L. Allen

p2os_msgs

No changes

p2os_teleop

No changes

p2os_urdf
* Change mass to more reasonable value.
* Clean up indentation.
* Clean up obsolete xml-schema namespaces.
* Remove obsolete files.
* Fix swivel joint.
* Add plugin for publishing ground truth odometry (position and velocity).
* Add and configure differential drive plugin.
  Seems to behave reasonably.
  There is an issue with the swivel which seems to break off randomly.
* Clean up and simplify collisions.
* Remove deprecated elem tag.
  Promote its attributes mu1, mu2, kp, kd to tags.
* Fix xacro deprecation warnings.
* Fix xacro deprecated warnings.
* Remove visual names in wheel descriptions...
  ...due to SDF bug https://bitbucket.org/osrf/sdformat/issues/132/parser-does-not-handle-urdf-material
* Use wheel descriptions from pioneer3dx_wheel.xacro.
  Remove invalid collision element from wheel definition.
  Comment out transmission element for now.
* Remove visual names.
  Visual names prevented Gazebo from rendering material colors, as per this issue: https://bitbucket.org/osrf/sdformat/issues/132/parser-does-not-handle-urdf-material
* Fix crash on model spawn in Gazebo.
  Remove zero-sized collision elements, as per this thread: http://answers.gazebosim.org/question/15816/gazebo-crashes-when-spawning-robot-from-urdf/
* Contributors: Damjan Miklic

@allenh1  Looks like there is no source or doc entry in this PR while it's present for the other distro. Is it intentional ?

No it was not. Fixed now.


Display Dropdown on focus?

Hi there, thanks so much for this plugin, i really love it :smile:
I was looking for the option present in the previous version displayDropdownOnFocus but can´t find something similar in the project page
Could you include it in the future?
Thanks!

Hi !
You can do like this:
$('.tokenize-sample-demo1').on('tokenize:select', function(container){
    $(this).tokenize2().trigger('tokenize:search', [$(this).tokenize2().input.val()]);
});

HI, I was looking the same old option "displayDropdownOnFocus", now I found your response.
It wouldn't be better to add this code inside the plugin? Just like the older version.
I mean, this code
$('.tokenize-sample-demo1').on('tokenize:select', function(container){
	$(this).tokenize2().trigger('tokenize:search', [$(this).tokenize2().input.val()]);
});

Thanks.


kube2iam: Support configuring which namespace kube2iam should run under

Configuring the namespace this way is discouraged unless there's a specific use case. Otherwise, it's best to use helm install --namespace <namespace> stable/kube2iam.

@oyvindio does the above method work for your usecase?

Sorry, I totally forgot about responding on this PR.
Yes it can work. I understand why setting namespace based on a value is discouraged since there is a separate flag for that.

Closing.


Release process for when new versions are released.

Documentation is missing..

Documentation is missing..

@davidkallesen, @kimlundjohansen, @christianhelle and @egil participated in a meeting about this.
The overall conclusion was:

We must do our very best to follow the SEMVER rules, e.g.:

V1.0.0 -> V2.0.0 : breaking change
V1.1.0 -> V1.2.0 : adding features
V1.1.0 -> V1.1.2 : fixes

To make it easier to determine and keep track of what changes we are making (major, minor, patch), we suggest looking at the Microsoft developed PublicApiAnalyzers

All projects should create changelog that follows Keep a Changelog. That means every PR should include an update to the appropriate section in a projects CHABGELOG.md file.

Update GitHub Action release workflow to also create a GitHub Release as the last step. Release body could include list of commits since last release or content of relevant section in CHANGELOG.md.


Peaks out of range in the graph

Hi Mike,
Below is a peculiar output I get from a different dataset, in which the peaks appear to be out of range in the first plotted graph.
http://qb.cshl.edu/genomescope/genomescope2.0/analysis.php?code=WVntTLeMcOiRrCVd2Gj6
Your input as to why this is occurring is very much appreciated.
Thanks,
CX

The automatic fitting routines are getting confused by your low coverage
data. If you run genomescope on the command line there are a few additional
options that can be set in these scenarios (make sure to use the latest
version from github since I just added a couple of these flags)
Running this command gives a pretty good fit:
$ ~/build/genomescope2.0/genomescope.R -i cx.hist -o cx.fit -k 21 -p 2
--kcov 5 --typical_error 3 --start_shift 2
And now the plots are easier to understand (attached)
Good luck!
Mike
On Wed, Jan 12, 2022 at 3:51 AM Cheong Xin Chan @.***>
wrote:

Hi Mike,
Below is a peculiar output I get from a different dataset, in which the
peaks appear to be out of range in the first plotted graph.
http://qb.cshl.edu/genomescope/genomescope2.0/analysis.php?code=WVntTLeMcOiRrCVd2Gj6
Your input as to why this is occurring is very much appreciated.
Thanks,
CX
—
Reply to this email directly, view it on GitHub
https://github.com/schatzlab/genomescope/issues/69, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABP344OBOOYXZHGQQRHBBLUVU6KHANCNFSM5LYMIJLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

GenomeScope version 2.0
input file = cx.hist
output directory = cx.fit
p = 2
k = 21
initial kmercov estimate = 5
property                      min               max
Homozygous (aa)               98.4791%          98.7299%
Heterozygous (ab)             1.27006%          1.52085%
Genome Haploid Length         1,399,423,442 bp  1,439,967,810 bp
Genome Repeat Length          538,374,829 bp    553,972,730 bp
Genome Unique Length          861,048,613 bp    885,995,081 bp
Model Fit                     70.9334%          99.0816%
Read Error Rate               0.158481%         0.158481%

Thanks very much for your input and help, Mike!


Improved API for producer/consumer stats

Motivation
Producer.getStats() and Consumer.getStats() are currently returning type classes that are in the impl package (eg: org.apache.pulsar.client.impl.ConsumerStats).
We should never expose types in the impl package from the api package.
Modifications

Extracted the interface for producer/consumer stats and added that to api package
Made sure we don't return null when stats are disabled
Added send/receive rates to the stats
Exposed send latency percentiles

retest this please


[#36] Inspect code style

Fixes #36
Continuous inspecting is needed.

Coverage remained the same at 83.072% when pulling 20feef8e444d68238e200473992df73472e2002e on Amazansia:36_Inspect_code_style into c11237e6aad47e9129e9ddffc1341eee86bffa1e on ztkmkoo:develop.

Hi, I found the unused import statement in dss-server/test/DssRestServerTest.java.
Could you remove it?

Hi, I found the unused import statements in dss-server/test/DssRestServerTest.java.
Could you remove it?

Hi! Thank you for reviewing. I think I missed checking all the import statements...I'll check and revise them.

I learned a lot after 5 commits...and typing "git commit -m [#36] Inspect code style" 5 times.
There are some changes in final commit: to reviewers (Thanks in advance!)

Plz check dss-server.test.DssRestServerTest.java. According to code style rules, every static thing has to be placed at first.
But here's a thing, that static things are too many. So I think I did it correctly, but still confusing...
I removed unused import statements at DssRestServerTest.java. And there was no such unused import statements in other files.
I edited pom.xml in dss-server, to catch right version of external library.
like...

<properties>
        <!-- Dependencies version -->
        **<dss-core.version>0.4.4</dss-core.version>**
        <akka.version>2.6.3</akka.version>
</properties>

God bless this commit and let it be closed, Amen. (I'm a atheist)


relations from/to-dates  fixed with

to have all relations tables align in the first two columns

actually this was closed with https://github.com/arthur-schnitzler/pmb-service/commit/f1e2d47fbbbb8e3dba6b34f939c93ba8a73fa195


resolve conflict and add BSKT to tokens

Summary by CodeRabbit

New Features

Added metadata for the BSKT token, supporting ERC20, CW20, and SPL formats.

I don't see an image?


Harry/localhost fix

Depends on https://github.com/getlantern/netx/pull/16.
The issue with the current code is that Resolve("tcp", "localhost") will make a random choice between "<IP_ADDRESS>" or "[::1]".  AFAICT, most applications listening on localhost are only serving one of those addresses, almost always the IPv4 address.  Crucially, a Go server told to listen on localhost will only serve <IP_ADDRESS>.
This is one of the issues plaguing lantern-desktop's desktop/TestProxying.  Protocols which use netx.Resolve occasionally resolve localhost to [::1] and cannot reach the local proxy server.
Technically, this change might be wrong for a server listening on [::1].  Really, Resolve should probably return all resolved IP addresses and let the caller choose the right one for their needs.  This way, dialers could try IPv4 first, then move on to IPv6.  I believe this is what Go's net package does.  I think returning the IPv4 address for localhost should be okay though.  We are unlikely to run into local servers running exclusively over IPv6.

This will include #16's changes until that PR is merged.  To see this PR's changes in isolation, please look at https://github.com/getlantern/netx/pull/17/commits/c312af0111694e07dca8db0aa72c3360470495f0

Really, Resolve should probably return all resolved IP addresses and let the caller choose the right one for their needs. This way, dialers could try IPv4 first, then move on to IPv6

I strongly agree with this.

Do we only ever get into this code path resolving Go servers?

No, and that's a big part of why this change is not the 100% correct way to go.

Really, Resolve should probably return all resolved IP addresses and let the caller choose the right one for their needs. This way, dialers could try IPv4 first, then move on to IPv6

I strongly agree with this.

I could instead create a netx/v2 which makes the suggested API change.  I don't think we can just supplement existing API since, as discussed above, the existing API is inherently flawed.  We'd also have to make sure dependents are unilaterally changed to netx/v2 as we rely on the ability to change the global resolve function.
Depends on how much we feel like my proposed fix is itself a problem.  I'll admit that I don't love it.  It could potentially lead to a subtle failure in which we fail to resolve/dial a localhost listener only serving IPv6.  This seems like an unlikely scenario though.
What do you think?

@hwh33 if the primary use case here is our own code connecting to localhost, can we just change our code to connect to <IP_ADDRESS>?

@hwh33 if the primary use case here is our own code connecting to localhost, can we just change our code to connect to <IP_ADDRESS>?

That was the first thing I tried actually.  The immediate issue was that the certs specified 'localhost'.  We can of course create new certs valid for <IP_ADDRESS>, but that made me realize that the issue was really netx's behavior.  I think ignoring this issue is just passing the buck to someone else.  Any test for a protocol using netx.Resolve or netx.Dial and targeting localhost is going to see random failures.
So, while I don't love my proposed fix, I think doing nothing is worse.

Looks like the following repos all use netx.  I think almost all of these use netx directly, but some may be indirect users (netx only appears in their go.sum):
https://github.com/getlantern/vain-haproxy-dns
https://github.com/getlantern/dnsgrab
https://github.com/getlantern/fronted
https://github.com/getlantern/packetforward
https://github.com/getlantern/idletiming
https://github.com/getlantern/cmux
https://github.com/getlantern/detour
https://github.com/getlantern/lampshade
https://github.com/getlantern/tlsproxy
https://github.com/getlantern/proxy
https://github.com/getlantern/tlsdialer
https://github.com/getlantern/tinywss
https://github.com/getlantern/quicwrapper
https://github.com/getlantern/http-proxy-lantern
https://github.com/getlantern/http-proxy
https://github.com/getlantern/kcpwrapper
https://github.com/getlantern/android-lantern
https://github.com/getlantern/mitm
https://github.com/getlantern/flashlight
https://github.com/getlantern/genconfig
https://github.com/getlantern/proxybench
https://github.com/getlantern/checkfallbacks
https://github.com/getlantern/replica
https://github.com/getlantern/lantern-desktop
https://github.com/getlantern/bbrconn
https://github.com/getlantern/enhttp
These repos also use netx, but I believe these might be inactive:
https://github.com/getlantern/lanternlib
https://github.com/getlantern/quic0
https://github.com/getlantern/interceptor

I think ignoring this issue is just passing the buck to someone else

IIUC there's 2 distinct questions.

Should the resolve function in netx respect the network type? Clearly the answer to that is yes, and your already merged PR fixes that.

Should netx assume that someone dialing "localhost" always intends to dial with IPv4 when they weren't specific about it? IMHO the answer to that is no, and whoever is doing the dialing should either be specific about the protocol, specific about the IP, or be willing to live with getting an IPv6 address if one is available.

Should netx assume that someone dialing "localhost" always intends to dial with IPv4 when they weren't specific about it? IMHO the answer to that is no, and whoever is doing the dialing should either be specific about the protocol, specific about the IP, or be willing to live with getting an IPv6 address if one is available.

Hmm I do get why you don't like the resolve function assuming the caller wants to dial IPv4.  I don't like that either.  However, I think Resolve answers a question like, "Can you give me an address at which I can reach somehost?"  In the case of "localhost", the answer AFAICT is almost never "[::1]" and is almost always "<IP_ADDRESS>".  So returning "[::1]" half the time is tantamount to giving the wrong answer half the time.
Compare this to querying Resolve for say, "google.com".  It returns a random selection from all possible answers, but that's okay because all of those choices are correct.  In the case of "localhost", "[::1]" is probably not going to connect you to the server.  If Resolve instead answered a question like, "Can you give me an address which is technically supposed to serve somehost?" then I would agree with you.
What do you think about that?  Do you still think the proposed fix is wrong?   Here's what I observed.

Using the plain network ip always yields an IPv4 address
On Ubuntu, the ip6 network actually yields nothing?!
I could not figure out a way to turn off IPv4 on the loopback interface to test Go's behavior, so in practice I think it's safe to assume localhost will always have at least an IPv4 address

I then wrote a program to resolve www.google.com
package main

import (
	"fmt"
	"net"
)

func main() {
	ip, _ := net.ResolveIPAddr("ip", "www.google.com")
	fmt.Printf("ip: %v\n", ip)
	ip4, _ := net.ResolveIPAddr("ip4", "www.google.com")
	fmt.Printf("ip4: %v\n", ip4)
	ip6, _ := net.ResolveIPAddr("ip6", "www.google.com")
	fmt.Printf("ip6: %v\n", ip6)
}

With this, I observed that:

The plain network ip always yields an IPv4 address even when an IPv6 address is available
If I disable IPv4, the plain network ip still always yields an IPv4 address (I even tried flushing my Mac's DNS resolver cache, same result)

So long story short, I think it's okay to always return an IPv4 address for localhost. Furthermore, it's probably safe (maybe even good) to have resolve preferentially return IPv4 addresses for unqualified lookups. In other words, if someone is using ip or tcp or udp and name resolution yields both IPv4 and IPv6 addresses, we should return one of the IPv4 addresses. Go seems even more aggressive than this by always returning IPv4, but I'm not sure I'm ready to go that far.

Nice, smart to look to net.ResolveIPAddr!  I got stuck looking at net.LookupIP (which is what netx uses) and didn't even notice net.ResolveIPAddr.

In other words, if someone is using ip or tcp or udp and name resolution yields both IPv4 and IPv6 addresses, we should return one of the IPv4 addresses.

I like this a lot.  This is much cleaner than my approach and solves the problem (and maybe then some).  Thanks a bunch for the thorough review so far and thoughtfulness w.r.t. the solution.  This change could have some subtle implications down the road, so I'm glad to have something that actually feels correct.  I'll close this PR and open a new one with your suggested solution.
Also this was funny and surprising!

On Ubuntu, the ip6 network actually yields nothing?!

https://github.com/getlantern/netx/pull/19

I know this one is closed, but I wanted to shed some light on @oxtoacart's comment about Ubuntu not returning anything for an IPv6 lookup of localhost.
When doing name lookups, /etc/hosts is usually consulted first (as files is usually the first thing listed in /etc/nsswitch.conf for the hosts database). Since that file is where entries for localhost will come from, any sort of name resolution will hinge on what is in /etc/hosts. On CentOS, for instance, /etc/hosts looks like this on a machine with the hostname "mirror":
$ grep -v '^#' /etc/hosts
<IP_ADDRESS>       mirror
<IP_ADDRESS>   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6

Running the example program above gives this:
$ go run main.go
ip: <IP_ADDRESS>
ip4: <IP_ADDRESS>
ip6: ::1

Contrast that with Ubuntu 20.04, where /etc/hosts looks like this:
$ cat /etc/hosts
<IP_ADDRESS> localhost

# The following lines are desirable for IPv6 capable hosts
::1 ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts

Since ::1 is listed in the file, but specifically does not resolve to localhost, net.ResolveIPAddr() rightfully returns nothing for it. (And if you tacked localhost onto the existing line for ::1, net.ResolveIPAddr() with then return a value for ip6.)

Thanks @Crosse!

Very interesting.  Thanks @Crosse!


Task - comprehensive specimen: Selected Collecting event; strange display.

Selecting a Collecting event which has a geographic area associated with it can result in a strange display.

I think this is more associate with a missing stylesheet. Have you tried running npm install?

First, that is not something which I regularly do: At what point should I consider it a requirement (i.e., what just changed such that this process is required?)
Second, after having done that which you suggest, the display is not markedly changed, but changes when Ctrl+ and Ctrl- are used to change the size.

Sometimes we include external packages thats needs to be installed. But if you're already done this, I will investigate what could make this happen.

@tuckerjs Please list specific steps to reproduce.

@tuckerjd please list specific steps to replicate.

Rebuild styles by removing cached assets


ci: add user/repo/commit/ID page

This means we can make the link for a commit point to the commit page,
which avoids fights if two targets have the same head.
Signed-off-by: Thomas Leonard<EMAIL_ADDRESS>

@samoht : CircleCI keeps failing with
[ERROR] Dependency errors in the upgrade actions. Please update, and report the
        following to the package maintainers if the error persists:
  - remove prometheus-app.~unknown -> remove prometheus-app.~unknown
  - install prometheus-app.~unknown -> install prometheus-app.~unknown

I always fix it by clicking "Rebuild without cache". Is there some way to fix it properly?

Maybe the cache of the master branch needs to be flushed properly. I'll try to click on various buttons...


easytags?

should easytags be included or not?
" EasyTags {
    let g:easytags_cmd = 'ctags'
" }

is there, but
    Bundle 'xolox/vim-easytags'

is not.

Probably not the best place to ask but are tag files created automatically?

ctags -R -a -f ~/.vimtags <path/to/your/sources>
-R for recursive, -a for append, -f for specifying the tags-file (~/.vimtags, this is already in the .vimrc here)

Tagbar automatically creates them when you open a file, but not for
the project in its entirety.
Steve Francia
@spf13
On Mar 7, 2012, at 8:02 AM, "Wolfgang Pfnür"
<EMAIL_ADDRESS>wrote:

ctags -R -a -f ~/.vimtags <path/to/your/sources>
-R for recursive, -a for append, -f for specifying the tags-file (~/.vimtags, this is already in the .vimrc here)

Reply to this email directly or view it on GitHub:
https://github.com/spf13/spf13-vim/issues/53#issuecomment-4367623

Maybe include a short ctags howto like I posted above into readme?


SQLServer Support

Hey there, very interested in trying out this project.
Wondering where MS SQLServer support would be on the roadmap?
Thanks.

Hi @tooolbox, MS SQLServer will be added eventually, but it is not planned in near term. Open source libraries like sqllite will be supported next.

Are there any update on this?

It's not planned for the next release. Although you should be able already to use MS SQLServer or any other unsupported database with jet, using RawStatement and custom model types.


mark MarkView as a single interface

Make the return type of MarkViewConstructor more concise

This doesn't seem a big enough win to justify exporting another type.


Command: UID SEARCH HEADER [BAD, UID SEARCH invalid argument]

While running backup in my SMS Backup+ 1.5.10, an error show that Command: UID SEARCH HEADER MASSAGE-ID<EMAIL_ADDRESS>response:#24# [BAD, UID SEARCH invalid argument].
And the log file save the following record:
(exception: com.fsck.k9.mail.store.imap.ImapException: UID SEARCH HEADER MASSAGE-ID<EMAIL_ADDRESS>response:#24# [BAD, UID SEARCH invalid argument].)

Is this with google or custom IMAP?

Is this with google or custom IMAP?

with google


Program closes after putting a tidal playlist link

Hello,
I have Windows 7.
Suda closes after putting a tidal playlist link. How shall I fix that?

Problem signature:
Problem Event Name:	CLR20r3
Problem Signature 01:	Suda.exe
Problem Signature 02:	<IP_ADDRESS>
Problem Signature 03:	f1211605
Problem Signature 04:	Suda
Problem Signature 05:	<IP_ADDRESS>
Problem Signature 06:	f1211605
Problem Signature 07:	11b
Problem Signature 08:	1
Problem Signature 09:	System.MissingMethodException
OS Version:	6.1.76<IP_ADDRESS>.256.48
Locale ID:	1033
Additional Information 1:	0a9e
Additional Information 2:	0a9e372d3b4ad19135b953a78882e789
Additional Information 3:	0a9e
Additional Information 4:	0a9e372d3b4ad19135b953a78882e789


proxying files are synchronous?

when proxying static files also, it seems to load them one by one, hence loading time is significantly longer.
serving local files is fast, however sometimes i cannot serve the local files.

I don't see any reason for the proxy to server the files synchronously - I didn't drilled into the proxy lib dependency, but it looks async - as the calling express code...

I think it is the limitation of dev 5 simultaneously connections
On Feb 17, 2015 4:34 AM, "Roy Feintuch"<EMAIL_ADDRESS>wrote:

I don't see any reason for the proxy to server the files synchronously - I
didn't drilled into the proxy lib dependency, but it looks async - as the
calling express code...
—
Reply to this email directly or view it on GitHub
https://github.com/Dome9/dev-proxy/issues/1#issuecomment-74608490.


branchName creation

Is your feature request related to a problem? Please describe.
I thought that the branchName argument was to use an existing branch, but would create it with the PR, but it's only for using the actual branch, if exist. If it does not exist, the action will return the error Error checking out existing branch "mybranchname": Command failed: git checkout  "mybranchname"
Describe the solution you'd like
I would like either the branch to be created by default if it doesn't exist, or by using a parameter set to false by default, or having an additional parameter like prBranchName which would do that. If the latter, it couldn't be use in conjunction of the existing branchName one.
Describe alternatives you've considered
Either I create the branch myself with a step before using this action, or I do not care about the branch name.
Additional context
No matter the decision to implement this in any way, I think the branchName description in the README should be updated to reflect that it's about an existing branch, or updated based on the changes for a PR that would close this feature request so it's clearer.

Hi @fharper and sorry for the delay in answering this question, it's been almost a year it seems 😆
Do you have a repository where this behavior is evident? I was under the impression that these use cases where covered in the tests. The link will take you to the first test, but also look at the three coming after.
Just to be clear, this action should reuse an existing branch if it finds one, and create a new branch if it doesn't. Without this behavior the action wouldn't be idempotent, which is a requirement.


csv: do not use ANSI color characters

ANSI characters in csv files are annoying, so remove them.

good call


feat(sbb-alert): add size s

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #2586
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

add size='s' for sbb-alert

Browsers
I tested the build on the following browsers:

[ ] Firefox Desktop
[ ] Chrome Desktop
[ ] Edge Desktop
[ ] Safari Desktop
[ ] Chrome Mobile
[ ] Safari Mobile

Screen readers
I tested the build on the following browsers:

[ ] JAWS Firefox Desktop
[ ] JAWS Chrome Desktop
[ ] NVDA Firefox Desktop
[ ] NVDA Chrome Desktop
[ ] VoiceOver Safari Desktop
[ ] VoiceOver Chrome Desktop
[ ] VoiceOver Safari Mobile
[ ] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@ddfd884). Click here to learn what that means.

:exclamation: Current head 4c35b2f differs from pull request most recent head b5fd2d4. Consider uploading reports for the commit b5fd2d4 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2591   +/-   ##
=======================================
  Coverage        ?   93.18%
=======================================
  Files           ?      315
  Lines           ?    25381
  Branches        ?     2061
=======================================
  Hits            ?    23652
  Misses          ?     1696
  Partials        ?       33

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[chore] remove unecessary prefix

Solana return the response under the structure of
{
  "code":-32009,
  "message":"Slot 135999228 was skipped, or missing in long-term storage",
  "data":null
}

Adding unnecessary prefix make it impossible to parse the error response to object and hard to check the error. Remove the prefix make is easier to work with error response.

Thanks for the feedback. Just open a PR #97, I think it can solve this problem. IMO, I prefer to return the error directly instead of have a custom function to parse it.


CORS_REPLACE_HTTPS_REFERER not working with DRF: SessionAuthentication

Django Rest Framework enforces CSRF with SessionAuthentication.
This check happens after django-cors-headers _http_referer_replace_reverse is called.
The result is a:
{"detail":"CSRF Failed: Referer checking failed - http://localhost:8080/ does not match https://remote:5000/."}
How can this be mitigated?

Using this SessionAuthentication class for DRF mitigates the problem.
Could this be included in cors-headers?
from corsheaders import defaults as settings
from corsheaders.middleware import (
    CorsPostCsrfMiddleware, CorsMiddleware)
from rest_framework.authentication import (
    SessionAuthentication as RFSessionAuthentication)

class PatchDepatchRefererCsrf(CorsMiddleware, CorsPostCsrfMiddleware):
    """
    Helper class to provide access to _https_referer_replace
    and _https_referer_replace_reverse.
    """

    def patch(self, request):
        if self.is_enabled(request) and settings.CORS_REPLACE_HTTPS_REFERER:
            self._https_referer_replace(request)

    def depatch(self, request):
        self._https_referer_replace_reverse(request)

class SessionAuthentication(RFSessionAuthentication):
    """
    SessionAuthentication that patchess the HTTP_REFERER before checking
    CSRF and depatch it after checking it.

    corsheaders supplies middleware but DRF doesn't use the middleware
    when checking CSRF for itself. This creates a problem when using https.
    """
    def enforce_csrf(self, request):
        patch_depatch = PatchDepatchRefererCsrf()
        patch_depatch.patch(request)
        super(SessionAuthentication, self).enforce(request)
        patch_depatch.depatch(request)

I'm sorry, I'm not familiar with the REST framework. Are you saying it bypasses the middleware stack entirely?

Since Django 1.8 support will be dropped shortly, I don't want to include it in the library. Copying that snippet should be easy enough for most projects.


IAP Samples - Outdated JWT package

Hello,
We are using IAP and want to validate the JWTs
I noticed in your examples, the package you are using is abandoned.
https://packagist.org/packages/spomky-labs/jose
It looks like there is a replacement, but it is a lot harder to use with poor documentation, so an example of proper usage would be super helpful.
Once you've got that, would it be possible for us to contribute a Laravel middleware for user sessions based on IAP JWTs?
The current example isn't even a web based example, it's a CLI tool accepting a copy/pasted JWT as a parameter, which isn't super helpful.
Regards,
iamacarpet

Yes please, any samples for Laravel integration would be very useful!
I will take a look at the JSON signing package.

@iamacarpet I have fixed the JWT validation by moving to a different library which is much less complicated AND has looser PHP requirements (hooray!). Please open another issue if you have any questions, and feel free to contribute an example for Laravel middleware!
Possibly the best way for you to contribute would be over at https://cloud.google.com/community, or just open a new issue when you've completed your integration and we can work together from there.
Thanks for reporting this issue and for your contributions!

Hey @iamacarpet!
That sounds reasonable, I'm glad you were able to get it sorted.
For App Engine Standard, I would try file cache (writing to /tmp) using guzzle cache middleware.


Storage plugin supports postgrsql

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #6321.
[ ] Update the CHANGES log.

Changelog CHANGES.md is required to update.

Most importantly, make your e2e passed.

Logs

2021-02-08T07:58:45.0586306Z [ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.621 s <<< FAILURE! - in org.apache.skywalking.e2e.ttl.StorageTTLE2E
2021-02-08T07:58:45.0615288Z [ERROR] org.apache.skywalking.e2e.ttl.StorageTTLE2E  Time elapsed: 0.619 s  <<< ERROR!
2021-02-08T07:58:45.0628825Z java.lang.IllegalArgumentException: Unable to parse YAML file from /home/runner/work/skywalking/skywalking/test/e2e/e2e-test/docker/ttl/docker-compose.postgresql.yml
2021-02-08T07:58:45.0634556Z Caused by: java.io.FileNotFoundException: File 'docker/ttl/docker-compose.postgresql.yml' does not exist

Please test locally first.

I tested StorageE2E, I need to test use cases?

I tested StorageE2E, I need to test all cases?

I tested StorageE2E, I need to test all cases?

But it fails in the CI. Please check.

Please resolve conflict, otherwise, no CI task runs.

Check, CI failed after merging upstream code.

Check, CI failed after merging upstream code.

What do you mean? I could only see this storage test fails.

Before merging the upstream code, the local test was normal, but then it failed, including local.

Before merging the upstream code, the local test was normal, but then it failed, including local.

We don't have change related to the storage perspective, AFAIK. I have to ask you to check on local and logs. The code review only happens after the e2e passed, and also, e2e is required in merge process.

@arugal @dmsolr @JaredTan95 Could you review and test this feature? As a new storage option, we should be careful about the feature and SQL CVE(especially SQL injection).
@kezhenxu94 Please check the e2e part.


Complete server + JVM crash when players ride one another

More or less what the title says! Here's the crashlog: https://mclo.gs/auD75BU
The server has an ungodly amount of mods so it's possible that it's a conflict, but i'm not sure what could cause it.

I don't think that I can fix it. This mod uses vanilla code, so if any other mod replaces it, I can't do anything.


Fix Pod with Namespace, fix Minion with IP as Name

Fix Pod with Namespace
Fix Minion with IP as Name instead of Hostname

Can one of the admins verify this patch?

OK to test

ok to test


[Prorammers-70128-Python] 내적

https://programmers.co.kr/learn/courses/30/lessons/70128?language=python3

Squash and Merge completed


Update active-directory-b2c-reference-disable-ev.md

Add Click **Edit** at the top of the blade as its own step.
I was following these steps just now and couldn't get from Click your policy to Click **Page UI Customization**, because I missed the Click **Edit**... step since it was buried in the Click your policy... step.

#sign-off

@MicrosoftGuyJFlo: I'm sorry - only the author of this article, @davidmu1, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Cron tab in Web UI miscolored in Sidekiq Pro v7

Hello @joevandyk,
Which versions of sidekiq and sidekiq-cron are you using?

This occurred after we upgraded sidekiq and sidekiq-cron and only lasted for a day, it looks fine now. I wonder if sidekiq or this gem has some CSS that gets cached for a certain amount of time.