How to detect validation errors?

I'm evaluating this component and would like to know how code should determine if an upload was skipped and get the error that the validator detected?
In the returned files array error and errorCode are always 0, even when a validator determined that an upload is invalid.
I saw FileUpload::process() returns a new File instance. This object does not include the error detected during processing.

Sorry for the late response.
I don't get this, i get the error messages if an upload fails. Below is a snapshot.

As an aside, what validator are you making use of ?

I'd like to clear this issue out as quick as possible as it seems to be the only reason i haven't pushed a new tag from the hot-fix branch

You don't have to wait on me for tagging, perhaps I'm misunderstanding something.
And I already added some validation outside this component to handle the situation in my project.

I investigated some more and now think the problem is caused by #37. The result is an initial $fileInfo based on an empty $file_path.

@kwebble , you make use of a custom validator ? I wanted to ask if you were setting the error message after a failed validation (https://github.com/Gargron/fileupload/blob/master/src/FileUpload/Validator/MimeTypeValidator.php#L54-L56) but since you mentioned #37 is the case - the reporter also says the fix works- . Lemme push those changes.

I have done that already, you can test to make sure it works.. Tag version is 1.4.1

Tested and got an error message if validation fails, nice!

Awesome then

Since this is marked as fixed, i am closing this


is it possible to unpack iso from inside of rar file?

Hello, this tool seems really nice and it's mostly working for me except for one thing:
I have a rar file with an iso inside of it and it seems that the iso isn't being unpacked even though I have set UN_FOLDER_EXTRACT_ISOS=true. is this kind of nested unpacking supported? do I maybe need to watch the output folder?
right now sonarr is happy with an ISO but plex won't play it, so unpackerr doesn't get the notification that the iso inside the rar needs to be unpacked.
here is my docker compose config:
 142   │   unpackerr:
 143   │     image: golift/unpackerr
 144   │     container_name: unpackerr
 145   │     volumes:
 146   │       - /mnt/media/downloads:/downloads
 147   │     restart: unless-stopped
 148   │     user: 0:0
 149   │     environment:
 150   │       - PUID=0
 151   │       - PGID=0
 152   │       - TZ=Etc/UTC
 153   │       - UN_FOLDER_EXTRACT_ISOS=true
 154   │       - UN_LOG_FILE=/downloads/unpackerr.log
 155   │       - UN_SONARR_0_URL=http://localhost:8989
 156   │       - UN_SONARR_0_API_KEY=<funnynumbers1>
 157   │       - UN_RADARR_0_URL=http://localhost:7878
 158   │       - UN_RADARR_0_API_KEY=<funnynumbers2>
 159   │       - UN_SONARR_1_URL=http://localhost:8990
 160   │       - UN_SONARR_1_API_KEY=<funnynumbers3>
 161   │       - UN_RADARR_1_URL=http://localhost:7879
 162   │       - UN_RADARR_1_API_KEY=<funnynumbers4>

Unpackerr is not going to solve this problem. I suggest not downloading ISOs.

wait @davidnewhall how does the unpack iso's feature work then???

It extracts ISO9660 files. What you have is not that.


Difference from SciPy about sparse min/max sum_duplicates parameter

I'm not sure its background, but currently, sparse min/max have sum_duplicates parameter and its default value is False. However, their SciPy counterparts don't have the parameter and always sum duplicate entries as sum_duplicates=True.
It would be better to change its default value to False.

This features was added in #2711

I would say toggling the default behavior from False to True to be consistent with scipy would be good. The reason I introduced the flag is as a performance enhancement to skip the sum_duplicates operation if the user knows their matrix is canonical. Changing the default flag will make it conform with the default scipy behavior.
For #3583, there's an open pull request on scipy that will implement the explicit nonzero features. I wouldn't remove the functionality from cupy for the sake of standardizing to scipy since it's a useful feature when explicit zeros are included in the sparse matrix. The default behavior for the non_zero flag is the same as standard scipy. The scipy pull request uses the keyword "explicit" rather than "nonzero" which might be a good change to make.


How connect sql server exists on my local

I have a sql server with the IP: 104.xxx.xxx.xxx : 1433
How can I config connect sql server in vm without using image docker in Aspire.

It should be but isn’t documented here
https://learn.microsoft.com/en-us/dotnet/aspire/database/sql-server-integration
This is where it’s currently documented.
https://learn.microsoft.com/en-us/dotnet/aspire/fundamentals/external-parameters


Add support for unresolved symbols

Basic, non working, support for unresolved symbols exists. Complete the implementation by adding a "replace" function that makes it possible to replace an unresolved symbol with a resolved symbol without changing the tree structure.

We also have to reconsider the usage of symbols in the compiler. Instead of using symbols, the compiler should only create symbols after the compilation is complete.

We can make the compiler simpler by making some limitations. For example, we can disallow circular package dependencies. This is what Golang does in their compiler.

Should be working now


Multithreading to download multiples videos from YouTube Channels

I am downloading YouTube channels with has 150K videos and if I hammer the YouTube for too long, YouTube limits my download speed. I have more than 1 Gigabytes per second speed. But, it becomes painfully slow after a certain period.
is there any solution to overcome this? I was thinking if there was a way where I could use all my 200 Threads same time to download 200 videos using the Channel download function of Pytubefix, that'll be amazing.

I believe in my opinion that it will be practically impossible to do this, because the YouTube infrastructure must be interpreting it as a denial of service attack, they could be blocking IPs and limiting the transfer rate, this is very difficult if you have few IPs.

@JuanBindez What's the estimated download limit without being interpreted as DDOS for one IP?

I've never tested it with YouTube but detection mechanisms, generally depending on the stream, block it quickly.

@JuanBindez I had earlier used multithreading but 16 threads at a time to download videos. and Once 400 threads but that time I had the individual links of the videos. so I used per thread one video.
Btw is there a way to extract all the video links of a channel?

>>> from pytubefix import Channel
>>>
>>> channel_url = 'https://www.youtube.com/@ProgrammingKnowledge'
>>>
>>> channel = Channel(channel_url)
>>>
>>> for video in channel.videos:
...     print(video.watch_url)
...
https://youtube.com/watch?v=7nwRESnXCpM
https://youtube.com/watch?v=8gc5z3aKc6k
https://youtube.com/watch?v=mtqTnGAAHw0
https://youtube.com/watch?v=Co5DMRh9RjE
https://youtube.com/watch?v=ndc8oDSNZzQ
https://youtube.com/watch?v=igc2zsOKPJs
https://youtube.com/watch?v=6wB7Kl_hxMc
https://youtube.com/watch?v=2LglWkwo_vY
https://youtube.com/watch?v=V70gziBFxzM
https://youtube.com/watch?v=tdkjCZ03Aj4


Node is either not visible or not an HTMLElement

https://app.circleci.com/pipelines/github/statechannels/monorepo/5974/workflows/83ed69d9-6d84-4112-abbb-d70d679297f2/jobs/24003/steps
 FAIL  puppeteer/__tests__/rps/rps.test.ts (38.07s)
  completes game 1 (challenge by A, challenge by B, resign by B) and begins game 2
    ✕ works (20974ms)

  ● completes game 1 (challenge by A, challenge by B, resign by B) and begins game 2  › works

    Node is either not visible or not an HTMLElement

      89 |   }
      90 |   try {
    > 91 |     return await frame.click(selector);
         |                        ^
      92 |   } catch (error) {
      93 |     console.error('frame.click(' + selector + ') failed on frame ' + (await frame.title()));
      94 |     await page.screenshot({path: 'click.error.png'});

https://app.circleci.com/pipelines/github/statechannels/monorepo/7092/workflows/07a74c43-9add-4739-909f-ad7e98a5e8c6/jobs/32153/steps
This might be related to the deposit events bug:  https://github.com/statechannels/monorepo/issues/2031


allowJs:true, noEmit:false get compile fail while migrate from js to typescript in vue project

as the title said. I have a project that is migrating from js to ts. so there both js & ts file exist on my project.
while i use vue-ts-loader , and set allowJs:true, noEmit:false in tsconfig.json . it compiles fail and said thaterror TS5055: Cannot write file 'xxx.js' because it would overwrite input file. reference resources

So as the link said, you need to set the outDir option.
Even it is ignored by webpack it needs to be set to convince compiler no file will be overwritten.

o ~~. outDir will be ignored by webpack... thx a lot


is it possible to add "AWX" (which is opensource) insted of "Ansible Tower"??

hello, all
is it possible to add "AWX" (which is opensource) insted of "Ansible Tower"??
I faced an error when adding "Automation provider : AWX" to manageiq.

<IP_ADDRESS> is AWX server and it is perfactly ok to login AWX server via web brower.

Credential validation was not successful:
{:status=>404, :headers=>{"server"=>"nginx/1.12.2", "date"=>"Mon, 29 Jul 2019 04:42:43 GMT",
"content-type"=>"application/json",
"content-length"=>"55",
"connection"=>"close",
"vary"=>"Accept, Accept-Language, Origin, Cookie",
"content-language"=>"en", "x-api-total-time"=>"0.011s"},
:body=>"{\"detail\":\"The requested resource could not be found.\"}"}

move to ansible_tower repositorie.


add Head method

it is useful to resolves redirects (video files, for example)

Thanks. I'll merge this into master soon.

Thanks!


Add the structure of the Tekton integration

What does this PR do?

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Current dependencies on/for this PR:

master:

PR #16665 👈

This comment was autogenerated by Freephite.


account schema inconsistent for 'net_limit' attribute

using the JSON API /v1/chain/get_account to get the account info.
e.g. for account eosio.stake, it looks like:
"net_limit": {
        "used": 289,
        "available": 13927873,
        "max": 13928162
    },

Please note, both available and max are numbers. But if I get the account info for heztanrqgene, it will return:
"net_limit": {
        "used": 133847,
        "available": "8344411498",
        "max": "8344545345"
    },

The available and max are string this time...

This behavior is to cater to a Javascript limitation.  Javascript can not represent integers greater than 53 bits as numbers.  EOS.IO represents integers larger than 32 bits as strings as a convenient and predictable cutoff.


[Suggestion] Add support for the predictive back gesture

Idea Description
The predictive back gesture is introduced in API level 33. Users will be able to preview whether or not they will back to the home screen when they do a back gesture and will be able to decide whether to stay in the current view or continue. Some applications (e.g., Settings, Spotify) are already introduced.
Under this project, there's no implementation requiring special consideration for onBackPresed logic handling. Thus, we can introduce the "predictive back" feature only for updating AndroidManifest.xml.
Reference images and links

https://developer.android.com/guide/navigation/custom-back/predictive-back-gesture
prototype implementation

https://github.com/chigichan24/conference-app-2023/tree/enable-predictive-back
https://github.com/DroidKaigi/conference-app-2023/assets/7840108/1ccc287d-0d9c-460f-8799-e6f11b85f978

Looks great! Do you have any concerns?

AFAIK, there're no side effects or big concerns about this change👍 I'll update the PR status as ready for review.


Remove code relating to parallel execution from PeriodicalExecuter.

In PeriodicalExcuter, removed unnecessary currentlyExecuting ivar and other code relating to parallel execution.
Since #onTimerEvent contained no logic after this change, made setInterval call (a bound) #execute and renamed #registerCallback to #start so that the timer can be restarted at will. This behavior already existed in the previous (undocumented) function.
Thanks!

(ping)

Mind merging or closing?

I'm guessing they've given up over here

That's sad. Is there a a canonical fork? I don't use the library anymore, but I think lots do.
Sent from my thumbs.
On Mar 1, 2012, at 7:15 PM, Cecil<EMAIL_ADDRESS>wrote:

I'm guessing they've given up over here

Reply to this email directly or view it on GitHub:
https://github.com/sstephenson/prototype/pull/23#issuecomment-4279656


[Hacktoberfest]: Add "CodeMash" to the Conferences Section

🆕🐥☝ Beginners Only:
This issue is reserved for people who are new to Open Source. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
To do:

Add CodeMash to the Conferences Section
See below for link to step-by-step tutorial
Go down to Conferences section - it starts at around line 173
Using the following template, put in the correct info for that conference (Search for it on the internet to find the correct details) underneath the last row in the table, ensuring there is still an empty row before the next table.

| [Conference Name](Website Address) | Conference Name or Organising Org | 10 | Description of the Conferences, found on their website |

Add your details in the contributors section at the bottom of the page using the same formatting as the last one

📋 Step by Step
Step-By-Step Tutorial
Remember to observe the Code of Conduct - by which we mean, in short - Be Kind, Be Positive, Be Helpful

I'll take this!


[Feature Request] Config entry for additional FFMPEG arguments

This could be helpful for those who want to more finely tune their audio libraries, and could help slightly more advanced users get access to features they don't know how to implement themselves.
Converting to mono (#41) or removing metadata entirely (#40 somewhat relevant, though removing metadata after doesn't save on bandwidth) could both be via FFMPEG arguments, and sometimes you want to change options on the encoder (e.g. opus compression).

Custom FFmpeg arguments will be available in the next major release.


Figure out why Bed disappears from cache

Type
Bed disappears from cache.
Problem or To-Do
When running locally, go to the Garden page. Note that a Bed will appear in the cache. Click on a garden. That bed is gone from the cache. We need the bed to stay in cache so that we can use its info in the Bed component.
Additional Information
n/a

Had my mom in town until this morning, so I was checked out of programming world. Have you had a chance to check this out? I can look into it to, let me know if you've found anything!

No big deal! I need more time to look into it. My folks are coming into
town this weekend!
On Tue, May 4, 2021 at 9:44 AM Payton Green @.***>
wrote:

Had my mom in town until this morning, so I was checked out of programming
world. Have you had a chance to check this out? I can look into it to, let
me know if you've found anything!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/gardenbuilder-app/gardenbuilder-frontend/issues/70#issuecomment-831997637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTOOZJFGT3RDEVIHOIRPI3TMAB5TANCNFSM4363QKYA
.

I'll look into it this afternoon and report back!

So far:

I removed the columns that don't exist on the backend yet from the two Bed Queries, i.e 'height', 'width' (this was giving me an error)
I show the Bed remaining in cache in both the Gardens and individual Garden view with this setup.
It does seem to disappear when entering into the individual bed view, but this is because we're not firing off any queries in the individual Bed view (other than GET_CURRENT_USER), as we don't have the Bed view wired up to the backend as of yet.

I plan on making an issue/working today and tomorrow on updating the Bed model on the backend to include the missing criteria and wiring up the frontend  component to display/update the data correctly. I think by creating that issue we can close this one?


[Snyk] Upgrade cypress-terminal-report from 5.2.0 to 5.3.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade cypress-terminal-report from 5.2.0 to 5.3.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released a month ago, on 2023-07-07.

Release notes

Package name: cypress-terminal-report

5.3.2 - 2023-07-07

5.3.1 - 2023-07-07

5.3.0 - 2023-07-03

5.2.0 - 2023-05-27

    from cypress-terminal-report GitHub release notes

Commit messages

Package name: cypress-terminal-report

3d28610 #197: Fix incorrect ESM import. Release 5.3.2
4ea4780 #197: Fix incorrect ESM export. Release 5.3.1
06c33df Release 5.3.0
8ad226c Merge pull request #195 from archfz/develop
94988c8 Update cypress to 12.16.0 in tests to confirm support.
e2cc539 \#192: Add additional protection against logs with JSON serialization.
1c241ec \#191: Fix circular reference causing error with expect logging.
06d6c15 Merge pull request #196 from archfz/dependabot/npm_and_yarn/semver-7.5.2
4b903fa Bump semver from 7.3.7 to 7.5.2

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

This pull request has been addressed by https://github.com/CPS-Innovation/Polaris/pull/607


Invalid cfg generated from Mace convolution

conv_2d_3x3.cl leads to invalid cfgs.
Run with:
clspv -w -I . conv_2d_3x3.cl -DDATA_TYPE=float -DCMD_DATA_TYPE=f -IN_DATA_TYPE=int -DOUT_DATA_TYPE=float -DUSE_PRELU

Any activation should lead to the same problem.

Clspv is generating a single block loop whose merge is the continue target of the outer loop. Validation error:
error: line 233: Header block 166[%166] is contained in the loop construct headed by 157[%157], but it's merge block 315[%315] is not
  %166 = OpLabel

I'm hitting this as well on a number of kernels from CLBlast.

Fixed by #485


Candidategenerator sometimes returns more candidates than specified k.

https://github.com/allenai/scispacy/blob/master/scispacy/candidate_generation.py#L179
seems like, sometimes returning more candidates than specified k.
Is this caused from many data points which exist in same distance from searched query vector? or, another cause?
If you'd know about this, I'd appreciate it. Thanks.

For example, mention : problems

candidate_generator(['problems'],10)[0]

[MentionCandidate(concept_id='C1546466', aliases=['Problems'], similarities=[1.0]), MentionCandidate(concept_id='C0262471', aliases=['ent problems'], similarities=[0.921135425567627]), MentionCandidate(concept_id='C1718157', aliases=['Oral problems'], similarities=[0.9019364714622498]), MentionCandidate(concept_id='C0231339', aliases=['age problems'], similarities=[0.8679661154747009]), MentionCandidate(concept_id='C0577885', aliases=['nose problems'], similarities=[0.8645884394645691]), MentionCandidate(concept_id='C0741992', aliases=['hand problems'], similarities=[0.8645559549331665]), MentionCandidate(concept_id='C0150920', aliases=['problems spine'], similarities=[0.8607649803161621]), MentionCandidate(concept_id='C0262174', aliases=['ear problems'], similarities=[0.8554203510284424]), MentionCandidate(concept_id='C0848067', aliases=['mental problems'], similarities=[0.8512626886367798]), MentionCandidate(concept_id='C0438989', aliases=['hear problems'], similarities=[0.8470597863197327]), MentionCandidate(concept_id='C0260662', aliases=['hear problems'], similarities=[0.8470597863197327])]

len(candidate_generator(['problems'],10)[0])
11

Sorry to keep bothering you. I think this is not bug, and candidate_generator cover all possible cui with same scores.
Another example, 'preferred setting' k=10 return includes
'Preferred Term' of C1709637 and C2347664 with score 0.6542521715164185
(Both canonical name is same, 'Preferred Term')
and 'setting' of C0542559 and 'Setting' of C4281677 with score 0.6470361948013306.
After all, UMLS covers so many concepts that sometimes this can happen.
This issue can be closed. Sorry for bothering.

No problem! Glad you resolved it!


Respect separators while sorting

First time working on a vscode extension. Had a lot of fun!
Fixes issue #161

@IgnisDa had to revert due to #224 (I also encountered the same issue). please check your implementation and reopen an PR


Add filename to PDF download

Keep it inline,
but let the browser know what filename to use.
For a modern browser this means:

when it's open in a tab, if you click to save, it uses the filename
you can right click "save as", and it will use the filename

Hopefully this will fix the bug seen in Police.
BONUS:
fixed a couple of bugs I found on the way:

avoid an error when trying to create a per with no offences
fix some missing translations
ensure uploaded pdfs have the .pdf extension

Can't verify how this affects ie8
In my vm both this and master try to download the pdf, rather than rendering inline.

I like these changes. The only thing I would suggest is to test this branch on staging, and check that everything works as expected: new PERs (this should work) and old PERs (the link to the document on AWS should not be affected, but let's test it)

@cesidio sure
will test before and after merge on this example:
https://mps-staging.dsd.io/escorts/38e270c3-12ea-486c-9281-2e2a69d31bab/print/new


rdr2 map editor

hi um may i you your map editor I'm 15 I got a new pc i had mod want to install and you map editor was one the please bring it
i hope you can repiy to me thx merry xmas

hi um may i you your map editor I'm 15 I got a new pc i had mod want to install and you map editor was one the please bring it
i hope you can repiy to me thx merry xmas


Update to 12.0.4

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Jinja2 variable references are suggested to take a {{<one space><variable name><one space>}} form. See lines [20].

@conda-forge-admin, please rerender

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


Milestone 4 - dependency diagram of the Makefile

Create a dependency diagram of the Makefile using makefile2graph and include the image in the README.md file in the root of your project.

Done. We need to see if we like the original size of the image or if we need to change it, to close this issue

Since we've checked the size with Firas (a-ok), I'm closing this issue.


kubernetes/client: remove announcement channel for namespace informer

Description:
The namespace informer is only necessary to check if namespaces are
monitored (via the monitoring label), which is done via the cache
store provided by client-go.
We do not need to trigger events to catalog when a namespace is
is added/updated/deleted for a few reasons:

Envoy configuration updates are not concerned with individual
namespace resources events. Adding a new namespace or updating
the namespace (ex with a label/annotation) does not concern
proxy configurations. Deleting a namespace however does affect
proxy configurations, but since the catalog subscribes to pod
and service specific events, a namespace deletion event would
also result in pod/service deletion events in such a namespace
which will suffice to recompute proxy configs for proxies
fronting affected pods/services.

Because of 1, annoucements specific to the namespace resource are
not very meaningful to catalog/XDS verticals. As such, the APIs
to check if namespaces are monitored leverage the cache store and
don't need to announce events.

Triggering announcements for resources that don't affect the proxy
configuration directly incurs a huge performance overhead, since
a single announcement will result in configurations being recomputed
for every single connected proxy. Moreover, as users label/annotate
their namespaces, the performance will exponentially degrade with
the number of number of updates on a per namespace basis.

This change also updates the tests that rely on the namespace cache
store to be asynchronously checked. Adding an announcement just for
the test will result in issues in the controller when there are
no readers for the channel, with the events being blocked and causing
subsequent to be stalled.

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [X]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [X]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2003 (01b7d1c) into main (bfa6a2e) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2003      +/-   ##
==========================================
+ Coverage   56.57%   56.65%   +0.07%
==========================================
  Files         141      141
  Lines        5677     5673       -4
==========================================
+ Hits         3212     3214       +2
+ Misses       2462     2456       -6
  Partials        3        3

Impacted Files
Coverage Δ

pkg/catalog/catalog.go
100.00% <ø> (ø)

pkg/catalog/fake.go
88.63% <ø> (-0.50%)
:arrow_down:

pkg/kubernetes/client.go
80.48% <100.00%> (-0.24%)
:arrow_down:

pkg/kubernetes/event_handlers.go
84.21% <0.00%> (+15.78%)
:arrow_up:

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 bfa6a2e...01b7d1c. Read the comment docs.


Issue: Change Characteristic of the Lightsaber Skill

The Lightsaber Skill is connected with different characteristics which depends on the specialization and career. So we need an option to change it per character. :)

Added to the dev branch, you can now change any associated characteristic by clicking on the characteristic of the skill.  It will bring up a dialog to update.  This is not styled yet, wanted to get functionality in.


Simple implementation sufficient to decode test streams

This PR implements bit allocation, mantissa processing, decoupling, rematrixing, IMDCT, and limited downmixing. Since the A/52 spec allows for the reuse of many variables between audio blocks (but not sync frames), this PR also moves the allocation of various audblk variables outside of block processing loop.
Downmixing is limited to standard stereo. It is unclear whether the downmixing is done correctly -- the liba52 and VLC appear to downmix to stereo a bit differently. It sounds reasonable, albeit a little quiet.
The decoder seems to decode several 5.1 test streams. However, it does not properly handle 2.0 or 2.1 streams correctly (e.g., http://samples.mplayerhq.hu/A-codecs/AC3/monsters_inc_2.0_192.ac3, http://samples.mplayerhq.hu/A-codecs/AC3/TomorrowNeverDies-2.1-48khz-192kbit.ac3, and http://samples.mplayerhq.hu/A-codecs/AC3/a52dec-streams/thx_2_0). It also does not use Annex D data, or decode E-AC3 streams.
Finally, this implementation is rather simple, mostly translated directly from pseudocode in the standard. It likely can be optimized. Tests should be added to ensure that optimizations do not break the decoder.

Thank you for your contributions to this project! As discussed in our e-mail conversation I will start modernizing the code over the next few days.


Here link in ICPC section is not working

here link in ICPC section is not working, it is redirecting the user to the following access denied page:

Can you assign me this issue, so that I can fix that link and as a solution the link would redirect the user to the following page:
 here

Hey, @kunal-kushwaha can you please look into this issue.
Thanks :)

make a PR


Whitelist player constraints

About the PR

see title. will use later for salamander + downstreams can obv use
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

a bit tangential to this PR but if this setup is going to be more common: how should servers with max whitelisted players report their max player count to the hub?

idk if i'd change how its reported

idk if i'd change how its reported

I think it's kind of annoying if you see a server it looks like you can join but you can't

for whitelisted players you get the inverse, a server that looks like you can't join but you can, and i feel like thats worse

thats not really the case for us if we're changing the behavior in the middle of the whitelisted server existing

It seems this cause issue, when non-whitelisted user can join empty server, somehow.


Question: Does the cli support the view of the contents of packages

Hi there,
is there a way to see the contents of a folder using packages or files. I know you can filter by has_content = true but I have not seen anything that displays the actual files/folders within a package.
Regards

4.8.0 does not yet support this, but it will be in next version,
beta of next version available here:
https://eudemo.asperademo.com/aspera/faspex/aspera-cli-4.8.0.gem
ascli aoc pack browse H6TrbIqGJA /contents

Current Workspace: eudemo (default)
Items: 1/1
+--------------+------+----------------+--------+----------------------+--------------+
| name         | type | recursive_size | size   | modified_time        | access_level |
+--------------+------+----------------+--------+----------------------+--------------+
| datafile.bin | file |                | 204800 | 2021-02-12T15:23:00Z | view         |
+--------------+------+----------------+--------+----------------------+--------------+


Add asio standalone example

Add Example for the using asio in standalone mode

@TheLartians Hi, sorry for the interruption, is there any issue still need to be resolved with the latest commit ? would be great on receiving your feedback!


cherry pick - open new tab for learn more link (#54)

This change has merged to prod

This change has merged to prod


s

s

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This file modifies a pre-existing file.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

35a3c30c527d649136f1ef468354d326e97bd01d

This request's code has changed since approval. Re-Judging contents...
Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This file modifies a pre-existing file.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

2765e2d4bdda53d751c724ccd8c7e1e5d9fbc773

that branch isnt even supposed to exist.


Try replacing the event listener

I need to provide and example of replacing the listener, or at least not running it.

Event listeners can be removed in the Table class using
// Will remove any listeners which are listening to these events
        $this->eventManager()->off('Proffer.beforeThumbs');
        $this->eventManager()->off('Proffer.afterThumbs');

Need to figure out how to remove just some and not all

Turns out you can't remove just one event listener without passing the exact same class instance. So I need to see if the class instance is available in the Table.
The table initialize() gets called before the behaviour initialize()


Create more edit fields for update CR modal

Now the UI is working for updating a Kabanero instance we need to add more values that can be edited. We should be able to edit all the below, but how we show that on the modal UI might need some design input.
apiVersion: kabanero.io/v1alpha2
kind: Kabanero
metadata:
  name: kabanero
spec:
  version: "0.8.0"
  codeReadyWorkspaces:
    enable: true
  governancePolicy:
    stackPolicy: activeDigest
  stacks:
    # A list of those repositories which are searched for stacks
    repositories:
    - name: incubator
      https:
        url: https://github.com/kabanero-io/kabanero-stack-hub/releases/download/0.7.0/kabanero-stack-hub-index.yaml
    pipelines:
    - id: default
      sha256: e6ebc6aeaf21540f0d0dac8caf0a2d805e8d90f174cb912a31831f700d049bb1
      https:
        url: https://github.com/kabanero-io/kabanero-pipelines/releases/download/0.7.0/default-kabanero-pipelines.tar.gz
  github:
    organization: alohr51-kabanero
    teams:
      - admin
    apiUrl: https://api.github.com

@alohr51 - Can the issue be closed or moved to 0.10.0


Item Icon need show placeholder if don't have sprite resource or is loading sprite resource

What problem does this feature solve?
When user loading sprite resource, can display the placeholder icon on the page.
When an item has not a picture resource (like new item), can display the placeholder icon on the page.
What does the proposed API look like?
N/A

When user loading sprite resource, can display the placeholder icon on the page.

This can be done by observing the network request of the icon URL. I'll implement it.

When an item has not a picture resource (like new item), can display the placeholder icon on the page.

Due to the need of detecting empty pixel of the sprite image, this can cause a high performance flaw in the rendering process. Personally I don't recommend implementing such feature, or at least, implementing such feature using such implementation.
Contribution / other ideas are welcome.

Will implement with v4 frontend.


WIP: Kevin input text puzzles

Started basic functionality for a text input puzzle. The answer to the riddle is "time". This is currently not case sensitive but can easily be made to be so.
What works:

clicking on the puzzle prompt disables movement controls and allows for pure keyboard input
a wrong answer will generate a red X and allow for multiple user inputs
a right answer will generate a green "congratulations". I believe I created some glitch last minute where you can still input after getting it right, which I assume is undesired behaviour (it was working correctly before though).

What still needs to be done off the top of my head:

add a proper "submit" button (the current submit method is using the "return" or "enter" key)
add some sort of proper time buffer between incorrect guesses? This was stated explicitly only in Amolik's ticket but I assume applies to mine as well
adding some buffer limit to the text box (currently there is none and it can get pretty ugly)

public/Aframe/samples/text-input-puzzles/aframe-super-keyboard.min.js - this file is not used in your PR right? I think it can be removed

Since we're going to use the super keyboard library for both keypads and the default keyboard, we can close this PR.
@andstun - let me know if you want to reopen this, but I don't think it's needed anymore.


Stop spewing request data into log stream

In a lot of places, transport.go prints the full body of the response. This seems ripe with the possibility for the wrong data to end up in the wrong log stream / log drain 👎
This removes many of them. Thanks!

Thanks!


System.InvalidOperationException: 'Unable to resolve service for type 'MetroLog.ILogger' while attempting to activate 'LoggerIssue.MainPage'.'

I want to start logging in a Maui App. But registring the MetroLog in DI throughs exception. The repro is attached below:
LoggerIssue.zip

You have to use ILogger<MainPage> instead of the regular ILogger

If you use ILogger<MainPage> instead of just ILogger it does work. Not sure if that's intended.

@jfversluis  thank you, it works after implementing your suggestion. It requires the namespace using Microsoft.Extensions.Logging;

The interface ILogger exists in 2 different namespaces:
In the Microsoft.Extensions context be sure to be using Microsoft.Extensions.Logging and not MetroLog.

I'm afraid this piece is handled by Microsoft extensions and MAUI hosting DI.
But can you inject simple interfaces ILogger with ms extensions? I think I only saw generic loggers in examples...
I think all loggers must be categorized?
https://stackoverflow.com/questions/52921966/unable-to-resolve-ilogger-from-microsoft-extensions-logging

Ah I see!


change runner argument to '--scanners'

This pull request addresses the deprecated warning '--security-checks' that was being logged by the runner. The concern was also documented in issue #36.

Due to the staleness of this repo and the corresponding extension, I decided to fork it. I don't plan to invest in really developing this further, but I will probably update it from time to time and accept PRs. For the time beeing, my forked extension is available here and includes the following improvements:

Mount docker.sock to scan docker images from a containerized trivy instance.
Yours: Update obsolete usage of --security-checks to --scanners.
Mount a consistent cache dir so that multiple runs using docker only download the vulnerability db once.
Use a recent version of trivy if not using the trivy docker image.

Due to the other changes above it should be possible to just use the docker-based execution in most cases, which always automatically uses the latest trivy version and does not require updates to this extension.

It can be installed in parallel to the official trivy extension.
This is a drop-in replacement, just change - task: trivy@1 to - task: trivy-contrib@1 after installing the extension.

Closing in favor of https://github.com/aquasecurity/trivy-azure-pipelines-task/pull/59


DSK Processing

Adding a generic script for processing DSK data.
The script can so far handle:

Docx
PDF
EPUB
HTML
TXT

Will close this in favor of #302 (feel free to ping me if you don't get a review on PR within a few days)


ISPN-15954 Add more properties to JGroups options

jgroups.diag.enable_tcp
jgroups.diag.enable_udp
jgroups.tcp_nodelay

https://issues.redhat.com/browse/ISPN-15954

Integrated into main, thanks @pruivo !


clarified the fusion run template

Ciao Marvin,
I've updated the run file in the bin folder and added a bit of documentation to make it even more easy to use:)
Best,
Maxe

Cool, merged!


Using test data in train.py for validation?

Thanks for sharing the beautiful implementation!
If I am not mistaken, it looks like you are using CIFAR10's test data for validation purposes in train.py:
  train_data = dset.CIFAR10(root=args.data, train=True, download=True, transform=train_transform)
  valid_data = dset.CIFAR10(root=args.data, train=False, download=True, transform=valid_transform)

and you are using the test data again in test.py:
  test_data = dset.CIFAR10(root=args.data, train=False, download=True, transform=test_transform)

My question is, how did you choose your model in the training phase (i.e. architecture evaluation phase)? Did you simply take the final model at the very last epoch of training, without any early stopping based on validation (more like test) performances?

Hello @dkumazaw  !
Thanks for your question. I think the test.py is used as a demo code, which only suggests that their method works.
The performance of an architecture should be evaluated on the ImageNet, because the model built on CIFAR10 would have a "natural tendency" of overfitting.
GL,


[feature suggestion] List formatting

It would be great to be able to use lists to format a text, with checkboxes or bullet points...

thank you for the suggestion... I'm currently researching a way to support lists. However, finding a solution that allows checkboxes would be better since it would allow the app to be used as a to-do tracker

hi!
Markdown support would allow lots of things, such as lists, checkboxes and such!
That'd be great!

Came here to ask for this. Glad to see it is already planned, I cannot wait!

Simple lists have landed as of #107. Checkboxes are probably NOT coming soon...

Would something like Google Keep's list mode be easier to implement? Under list mode, every line automatically turns into a list, and each line can be moved around by dragging a handle.
This should also be a lot easier to do than trying to fit it into the existing widget, since it should just be a GtkColumnView.

it should just be a GtkColumnView.

Yeah, that would be possible, but I'm not trying to implement that just yet. (It would actually be a GtkListView or a simple GtkListBox)


Error when importing SFC vue files directly from node_modules package

[x] Read the docs.
[x] Use Vite >=2.0. (1.x is no longer supported)
[ ] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
When trying to import a .vue file from one of the packages from node_modules, it throws below error after starting dev server. For certain debugging reasons in dev environment, we are shipping .vue files to npm without minifying it.
Dep optimization failed with error:
(node:4674) UnhandledPromiseRejectionWarning: Error: Unexpected token (Note that you need plugins to import files that are not JavaScript)

<EMAIL_ADDRESS>1: <template>
   ^
2:

3:         <Avatar

Reproduction
It's a private repo but if you still need a repro, I can create a separate example.
System Info

vite version: 2.0.0-beta.15
Operating System: Big Sur 11.1
Node version: 14.15.4
Package manager (npm/yarn/pnpm) and version: npm

Logs (Optional if provided reproduction)
Dep optimization failed with error:
(node:4674) UnhandledPromiseRejectionWarning: Error: Unexpected token (Note that you need plugins to import files that are not JavaScript)

<EMAIL_ADDRESS>1: <template>
   ^
2:

3:         <Avatar

Reproduction please. Dep optimization currently doesn't handle Vue files by default.

Reproduction please. Dep optimization currently doesn't handle Vue files by default.

Ok, so in Vite 2, the dep optimizer no longer applies the plugins you add in the main config, because we don't want to encourage deps to rely on Vite-specific behavior and they should ideally all be just standard JS. But in this case if you want to explicitly pre-optimize a package Vue files, you need need to explicitly add the Vue plugin to the optimize step via the new optimizeDeps.plugins option added in 1ea0168
See https://vitejs.dev/config/#optimizedeps-plugins (will need to use latest release)

Ok, so in Vite 2, the dep optimizer no longer applies the plugins you add in the main config, because we don't want to encourage deps to rely on Vite-specific behavior and they should ideally all be just standard JS. But in this case if you want to explicitly pre-optimize a package Vue files, you need need to explicitly add the Vue plugin to the optimize step via the new optimizeDeps.plugins option added in 1ea0168
See https://vitejs.dev/config/#optimizedeps-plugins (will need to use latest release)

For those that got stuck with this. If you're using Vue with Vite try the following.
These five lines are the "fix".
optimizeDeps: {
    plugins: [
      vue()
    ]
}

import type { UserConfig } from 'vite';
import vue from '@vitejs/plugin-vue';

const config: UserConfig = {
  plugins: [
    vue(),
  ],
  optimizeDeps: {
    plugins: [
      vue()
    ]
  }
}

export default config;

For those that got stuck with this. If you're using Vue with Vite try the following.
These five lines are the "fix".
optimizeDeps: {
    plugins: [
      vue()
    ]
}

import type { UserConfig } from 'vite';
import vue from '@vitejs/plugin-vue';

const config: UserConfig = {
  plugins: [
    vue(),
  ],
  optimizeDeps: {
    plugins: [
      vue()
    ]
  }
}

export default config;


Add an .editorconfig

Maybe an .editorconfig file would be a good addition?
From: http://editorconfig.org/

EditorConfig helps developers define and maintain consistent coding styles between different editors and IDEs.
The EditorConfig project consists of a file format for defining coding styles and a collection of text editor plugins
that enable editors to read the file format and adhere to defined styles. EditorConfig files are easily readibly and
they work nicely with version control systems.

Can't this be closed since the file has been added?

@SimonSchick Good call.


Update 20-load.md prerender section

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Updates the prerendering explanation for load functions to reduce ambiguity.

You may have some pages that are prerendered and some that are not. It's misleading to say that when prerendering it's only invoked at build-time, because there may well be other non-prerendered ones that are run at runtime rather than build-time. I think what's here in the docs now is fine.

You may have some pages that are prerendered and some that are not. It's misleading to say that when prerendering it's only invoked at build-time, because there may well be other non-prerendered ones that are run at runtime rather than build-time. I think what's here in the docs now is fine.

I would really appreicate not closing this, or having an alternative. The current docs seem to indicate that universal load functions are no longer invoked at run-time when pages are pre-rendered, which is not the case.
This has sparked some recent confusion in the offical Discord.


Reading iptables entries from journald

Hi,
is it possible to read iptables log entries from journald?
Thank you.

Hi @eLco
I think it should be possible to read iptables logs from journald as long as it's stored somewhere in your host with the log entry containing source and destination IPs of dropped packets. You need to provide the path of your journald log to env var IPTABLES_LOG_PATH and log prefix defined in your iptables chains toIPTABLES_LOG_PREFIX.

would love to see this implemented :)

Closing this issue as the corresponding PR #5 was merged.


chore: release 3.2.3

:robot: I have created a release *beep* *boop*
3.2.3 (2019-06-26)
Bug Fixes

docs: link to reference docs section on googleapis.dev (#267) (d7d8440)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-language/releases/tag/v3.2.3

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


feature: --split-pages option: split pages in group

As per described in #30:
qpdf in.pdf in.pdf --pages in.pdf 1-5,11,60,70-100 -- --split-pages=3

will split the pdf into:
in-1.pdf: page 1,2,3
in-2.pdf: page 4,5,11
in-3.pdf: page 60,70,71
in-4.pdf: page 72,73,74
in-5.pdf: page 75,76,77
...

Could you please change it to generate the below:
in-1.pdf: page 1,2,3
in-2.pdf: page 4,5
in-3.pdf: page 11
in-4.pdf: page 60
in-5.pdf: page 70,71,72
...

In other words, it split pages in group seperated by comma,.
Better if it can name the output file accordingly:
in-1-3: page 1,2,3
in-4-5: page 4,5
in-11: page 11
in-60: page 60
in-70-72: page 70,71,72
...

Thank you.

Thanks for the idea, but I'm sorry to say that I am not going to consider making this change.
This would be an incompatible change, and the way the numeric range is specified is discarded during command-line argument parsing. In addition to not being a compatible change, this change would make different specifications of equivalent page ranges behave differently, which I think would be surprising and counterintuitive behavior. I think, for your use case, you'll just have to run multiple qpdf commands. You could use a separate --pages specification for each group of pages that you want.
It would be possible to come up with a completely different syntax for creating individual groups of pages based on separate ranges, but I think it's probably just cluttering up the API given that you can already do it by running qpdf multiple times.

Yes, you are right. This would easily be solved with a shell script.
However, considering user view, it's best to solve it by qpdf syntax, running in 1 go.
The --split-pages option has not much use in my case. My intention was to split pdf in 1 go.
Example:
qpdf in.pdf in.pdf --pages in.pdf 1-5,11,60,70-100
would split pdf into in-1-5.pdf, in-11.pdf, in-60.pdf and in-70-100.pdf
I think this has more use cases than --split-pages, but I could certainly be wrong.

Okay, I like the idea of being able to split in named groups, so I'll leave this open and add the label I use for page-related features. At some point, I am going to add lots of other ways to split, such as splitting based on outlines or other document-level features, and this feature might fit will into that set of modifications. I suspect it will be months before I tackle that pile of issues. If implemented, it will be by adding a new option, not changing the semantics of --split-pages, for reasons I explained. However I think your main point is the need to split in named groups, not to change the behavior of --split-pages per se.

I think this has more use cases than --split-pages, but I could certainly be wrong.

I don't (and can't really) gather metrics on what features are used, but I know that --split-pages is a heavily used feature that was requested multiple times before I added it.
Thanks for the report and suggestion!

Yes, you are right. This would easily be solved with a shell script. However, considering user view, it's best to solve it by qpdf syntax, running in 1 go. The --split-pages option has not much use in my case. My intention was to split pdf in 1 go. Example: qpdf in.pdf in.pdf --pages in.pdf 1-5,11,60,70-100 would split pdf into in-1-5.pdf, in-11.pdf, in-60.pdf and in-70-100.pdf I think this has more use cases than --split-pages, but I could certainly be wrong.

The command does not work for me. It tells me that the input file is defined twice and that -- is missing at the end. If I change that, I do not get in-1-5.pdf, in-11.pdf, in-60.pdf and in-70-100.pdf, but a single file with these pages. I need the separate files, though. Any idea?

@Perangelot That command was a suggestion. That's now how qpdf actually works. You can find more detail in the other comments.

Is it now or not how qpdf actually works? I have looked at the other comments. Have I understood correctly that it is not possible to extract one pdf file per page range in one command?

Sorry, not. That's the worst typo in the English language. It is not how it works, and it is not how it is intended to work. You are correct that there is no way to extract one file per range. The behavior is dependent on the set of pages in the range, and not how the range is specified. That won't change since changing it would break compatibility and also require a rework of how the whole page range thing is handled internally. (The internal representation does not preserve the way the range was specified.)

Some form of this will be done. See also #741. I'm working through the issues with pages on them now and making lists of specific features. I don't know what the syntax will be yet.

This issue is part of the qpdf pages epic. If you are interested in following, please see #1104.


Add DecimalTypeObject to test-integration (#143)

Decimal Type was tested.
Is it right to do it this way?
It comes up as a double type with json.

Decimal Type was tested.
Is it right to do it this way?
It comes up as a double type with json.

decimal data type is not declared on the spec, it's considered as number with the format of double.

Add Model : test-integration2/TestApps/Models/DecimalTypeObjectModel.cs
Add HttpTrigger to test : test-integration2/TestApps/Post_ApplicationJson_DecimalType_HttpTrigger.cs
I add HttpTrigger about Decimal Type to the TestApp.

I made the mistake of doing the revert.


Reformatting descriptions for the organization of multiple Prometheus instances in multi-zone environments

Reformatting descriptions for the organization of multiple Prometheus instances in multi-zone environments.

Thx for the contrib!


Decoder never releases any memory

I'm running into a issue where the decoder is never releasing any memory, and just contiously eats up more and more as you decode, with all the mentions of pooling and reusing decoders, i'm assuming this should not be the case?
code to reproduce (it eats up ram pretty fast so i wouldn't run it for more than a couple seconds):

package main

import (
	"bytes"
	"github.com/francoispqt/gojay"
	"log"
	"runtime"
	"time"
)

const encodedData = `{"a": 100, "b": "hello world"}`

type Data struct {
	A int64
	B string
}

// implement gojay.UnmarshalerJSONObject
func (d *Data) UnmarshalJSONObject(dec *gojay.Decoder, key string) error {
	switch key {
	case "a":
		return dec.Int64(&d.A)
	case "b":
		return dec.String(&d.B)
	}

	return nil
}

func (evt *Data) NKeys() int {
	return 0 // refuses to work without this being 0 for some reason
}

func main() {
	go printStats()

	RunWorker()
}

func RunWorker() {
	buf := &bytes.Buffer{}
	decoder := gojay.NewDecoder(buf)
	dst := &Data{}
	for {
		_, err := buf.WriteString(encodedData)
		if err != nil {
			log.Fatal(err)
                        return
		}

		err = decoder.Decode(dst)
		if err != nil {
			log.Fatal(err)
			return
		}

		buf.Reset()
	}
}

func printStats() {
	tc := time.NewTicker(time.Second)

	for {
		var memStats runtime.MemStats
		runtime.ReadMemStats(&memStats)
		log.Printf("Alloc: %5.1fMB, Sys: %5.1fMB", float64(memStats.Alloc)/1000000, float64(memStats.Sys)/1000000)
		<-tc.C
	}
}

I ended up forking it and resetting it at the end of each Decode call https://github.com/jonas747/gojay/commit/9081ac11e06c13f278632914f86a5bcf8e3c19a5
Seems to work perfectly for my use case.

Ok, I will close the issue but will create a milestone to add an exposed Reset method.
Thanks


fix clean.yml

Coverage remained the same at 34.607% when pulling 4d3975f736f182a706a2c08e9905cfd1945342cd on yuye/cp-fix-rm into 8ec22e85bf642d4a622cf70afb80acacee498055 on master.


fix: Update data aiven_project_vpc example

About this change—what it does
The id field is actually vpc_id in this example, and should get updated.
Why this way
It's a typo fix.

hey, @ebachle! 👋
thank you for your contribution!

I guess it's now outdated. Thanks for the help.


Flip axis direction

In which part of the interface would this feature applies?

[ ] 1. Load your data
[ ] 2. Choose a chart
[ ] 3. Mapping
[x] 4. Customize
[ ] 5. Export
Is your feature request related to a problem? Please describe.
When presenting the parallel coordinates chart, it becomes difficult to read when the axis is plotted with higher values at the bottom and lower values at the top. This is because conventional charts present higher values at the top of the axis.
Describe the solution you'd like
Can there be an option to flip the order of values along the axis to make it easier to read the chart?
Describe alternatives you've considered
Additional context

Can you provide an example with a dataset/project file?


WIP Feature/income panel page 2

What has changed and why?
WIP
How to test this change

[ ] has automated tests
[ ] at least one a reviewer ran the code
[ ] tested in small screen
[ ] tested in Chrome
[ ] tested in Firefox
[ ] tested in IE
[ ] tested in Safari
[ ] i18n
[ ] tested in screen reader/contrast <-- remove this if no UX change

lots of this warning
./src/JSONFormData/FirstTimeApplication.js
  Line 115:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 117:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 121:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 123:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 125:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 128:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 133:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 135:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 139:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 141:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 143:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 146:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 211:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 213:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 219:  Multiline support is limited to browsers supporting ES5 only  no-multi-str
  Line 221:  Multiline support is limited to browsers supporting ES5 only  no-multi-str

@dlouise64 There's a git conflict now.


Playback rate resets when pausing or playing the video

Have you read the FAQ and checked for duplicate open issues?
Yes
If the problem is related to FairPlay, have you read the tutorial?

Not applicable
What version of Shaka Player are you using?
4.12.4
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes (nightly demo page)
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Firefox on Windows 10
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Not applicable
What are the manifest and license server URIs?
Any will work as this is a UI bug
What configuration are you using?  What is the output of player.getNonDefaultConfiguration()?
Default
What did you do?

Open any video in the demo player
Change the playback rate through the UI
Pause the video

What did you expect to happen?
The playback rate should stay at the configured rate
What actually happened?
The playback rate was reset to 1x. Another issue is that if you change the playback rate while the video is paused, it will automatically start playing as soon as you select a playback rate.
Are you planning send a PR to fix it?
I'm not sure what the right solution to this problem is, from my testing the problem was introduced in 4.12.1, as 4.12.0 works correctly and later releases, as well as the nightly demo have the same issue. The only pull request that changed code concerning the playback rate in 4.12.1 was https://github.com/shaka-project/shaka-player/issues/7580 so that is likely to be the culprit.

The second issue about the video starting to play when you select a playback rate instead of staying paused still happens with the nightly demo. Should I open a separate bug report for that? There is also another bug that if you change the defaultPlaybackRate of the video element to something other than 1.0, the UI won't let you select 1x.

Opened two new issues, one for each of the bugs.


go mod tidy tries to download local package

Go version
1.23.2
Output of go env in your module/workspace:
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\<username>\AppData\Local\go-build
set GOENV=C:\Users\<username>\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\<username>\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\<username>\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLCHAIN=auto
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.23.2
set GODEBUG=
set GOTELEMETRY=off
set GOTELEMETRYDIR=C:\Users\<username>\AppData\Roaming\go\telemetry
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=D:\Progetti\server\go.mod
set GOWORK=
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=C:\Users\MASSIM~1\AppData\Local\Temp\go-build4023307547=/tmp/go-build -gno-record-gcc-switches

What did you do?
I have a go module that is called github.com/lorypelli/server but when I do go mod tidy it tries to fetch github.com/lorypelli/server/frontend even if the frontend package of the module is local since the entire module is local
What did you see happen?
go: finding module for package github.com/lorypelli/server/frontend
go: github.com/lorypelli/server/pkg imports
       github.com/lorypelli/server/frontend: no matching versions for query "latest"

you can see in https://github.com/lorypelli/server latest actions are failing because of this...
What did you expect to see?
it shouldn't try to require local modules...

but it happens only with github actions, on windows it doesn't

you don't have a go package there https://github.com/lorypelli/server/tree/main/frontend
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Library Redesign

The current library is really messy internally and needs a rework.
As well as that, Exceptions are expensive and in hindsight this library shouldn't throw as many. Result types for certain actions should be made.
The JSON responses and messages will be moved to a new internal namespace "Kahoot.NET.API.Requests" and "Kahoot.NET.API.Responses"

The whole Kahoot.NET.Internal Folder is to be removed/moved to a more stateless design in API

Kahoot.NET.Shared.Extensions which only included RemoveWhiteSpace will be removed in favor of my SpanExtensions Library to be added on NuGet which includes the faster implementation


Hello Team, what URLs need to be whitelisted for the above learning pathways solution to work ?

Hello Team, what URLs need to be whitelisted for the above learning pathways solution to work ?
Originally posted by @rhajj19 in https://github.com/pnp/custom-learning-office-365/issues/344#issuecomment-736518812

Hi @margot-eva,
Below  Urls need to be whitelisted
https://pnp.github.io
https://support.office.com/
https://docs.microsoft.com


0.9.13 Fail Upgrade To 0.11.19

Gogs 版本：
系统类型及版本：Linux
数据库：MySQL
问题描述：版本升级

Gogs version (or commit ref): <IP_ADDRESS>8
Git version:
Operating system: CentOs7.3
Database (use [x]):

[x ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[x ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log):

Description
<IP_ADDRESS>8 Upgrade To Lastest Version
2017/06/19 10:49:05 [FATAL] [...s/routers/install.go:66 GlobalInit()] Fail to initialize ORM engine: sync database struct error: pq: could not create unique index "UQE_issue_repo_index"

i delete not unique index and repo_id
gogs can start

@Unknwon please fix this bug thanks

Hi,

This is because issue table has two or more records has same repo_id and index, which was caused by exactly the old old old version you were using.
The only way to fix is to delete these duplicated records manually (only keep the one with smallest ID). Possible SQL to find duplicates:

SELECT   id,
         repo_id,
         `index`,
         count(*)
FROM     issue
GROUP BY repo_id,
         `index`
HAVING   count(*) > 0


Dashboard: Link option on 'Button' Component

Overview
Adding isLink as a prop to the Button component to avoid having to set forwardAs on each instance where we want a CTA link that stylistically looks like a button. Now using ...rest to decide onClick vs href.
Testing
See that the 'create new story' "button" (it's a link, it just looks like a button) still sends you to the proper url.
See Dashboard/Components/Button on storybook, toggle 'isLink' to see the button become an anchor element. href and onClick are now both just forwarded as is to the component.

Nice work! Just trying to gauge how this changes things so gonna ask a quick q.
So if we want a link that looks like a button we do this:
<Button type={/* type */} href={/* url */} isLink={true}>
and for a button this?
<Button type={/* type */} onClick={/* fn */}>

yup that's exactly it!


Update branch and add BP256R1 support

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


Exposed ports don't show up in host (Docker for Mac)

Problem
I'm not really sure about how this is supposed to work (I'm new to Kubernetes). But I guess that my Kubernetes Services with type LoadBalancers should be accessible by their ports. And they are not. The pods are working fine though, with no errors.
Here is the example config:
apiVersion: v1
kind: Service
metadata:
  name: name-nginx
  labels:
    app: name
spec:
  ports:
    - port: 80
      targetPort: 8080
      protocol: TCP
      name: http
    - port: 443
      targetPort: 8081
      protocol: TCP
      name: https
  selector:
    app: name
    tier: frontend
  type: LoadBalancer

Environment
OS Version:
OS X El Capitan: 10.11.5 (15F34)

docker --version: (It's Docker for Mac)
Docker version 1.12.0-rc3, build 91e29e8, experimental

NodePort currently doesn't work with Docker for Mac. It does work with Docker Machine though. Type LoadBalancer is meant to be used with cloud providers that can provision an external load balancer for you.

Is there a way to currently access a Kubernetes service when using docker beta (mac)?

@geoah another option is to use Minikube. Both ingress and NodePort now work in their latest 0.7.1 release (just out today). I have tested this out using the xhyve driver on macOS.

Great... Thanks for informing ❤


Add ability to rename and delete key pairs

Key pair name can be edited via edit button under Actions
Key pair can be deleted ("disabled" on our end) via trash can button under Actions
Disabled is no longer a property when a user creates a new key pair
Disabled key pairs are filtered out in the api instead of the front end
Removed display of disabled key in UI
Created storybook components for KeyPairDisableDialog and KeyPairEditDialog

Test Plan:

Edit existing key pair
Delete existing key pair
Create new key pair and edit, then disable it
Delete a key pair and then create a new key pair with the same name and make sure it can also be created
Check that creating a key pair does not show a disabled checkbox

Looks like you can save a blank keypair name.  That should be required.


[Camera] Refactor/rework video recording

Video recording is still experimental (read: busted).
• It performs very badly on hidpi screens because each frame has to be saved to disc.
• Frames can drop because of disc accesses on resource intensive websites.
• FPS is not 100% reliable. Most videos seem to be playing slightly faster that what's recorded.
So far I see two ways of fixing it:
• Changing the API used to capture frames:
desktop-capturer seems promising, working with streams would eliminate disk access problems.

electron/desktop-capturer
Note: might change the way we choose to record mouse cursors or not. See #30.

• Record frames directly to a stream:
Would rely heavily on ffmpeg to do the hard work.

Fixed in 0.2.0. Closing.


import/prepare: Setting a default signature public key

This allows to set a global default key to check against, which can still be overridden by -C pubkey in import/prepare.
This is especially useful when using the nomad-pot-plugin, as it allows to limit the pot images to run on the cluster to those which were signed by a trusted pot provider.

@pizzamig thanks


Badge Enhancement Request

Currently, the badge position is on the top right of the component. Shall we enhance for all 4 corners?

Vuetify has a similar feature.

Can I work on this if no one else is doing ?

What API should we expose? What about:
type anchorOrigin = {
  horizontal: 'left' | 'right';
  vertical: 'top', 'bottom';
};

@oliviertassinari  Though I am little confused as such how to implement it. Any detailed clarity would be appreciated. changing the positions (top, right, left and bottom) within styles of badge component will solve the issue, but what might be the better way to do it?

@oliviertassinari, @vilvaathibanpb  I think, Bringing the anchorOrigin property to the Badge component is more useful than control the location of badge via the style-sheet as we can make style-sheet is more generic and the position of the Badge can be controlled via the props.
What is your though?

@baoduy May be that makes sense. Lets try that out. I will check if thats feasible

and addition to that, it would be awesome if we can also use other content rather than just numeric value. Maybe another variant?

@SerdarSanri What stop you from using a non-metric value?


Consider a different implementation of typecheck_restricted_expr_against_schematype()

Category
Internal refactors/changes
Describe the feature you'd like to request
This comes from a TODO in the code in cedar-policy-core/src/entities/conformance.rs (as of this writing).
Currently, Core typechecks a restricted expression against a Core SchemaType by first calculating the SchemaType of the restricted expression, and then determining if the two SchemaTypes are "consistent".  It may be less confusing, more efficient, and/or more precise to just typecheck directly.
In a comment on #430, @khieta says she prefers the code as-is because it prevents duplication and reuses schematype_of_restricted_expr().  This concern should be balanced against the one above.
Either we should change the implementation, or we should just remove the TODO if we determine the current implementation is the most desirable.
Describe alternatives you've considered
We can also refactor/adjust schematype_of_restricted_expr() and/or SchemaType::is_consistent_with() if that would make things clearer.
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

https://github.com/cedar-policy/cedar/pull/1177 adds a dedicated method here (though it still could be optimized further).

Resolved by #1177, #1190, #1194, and #1197 which together remove schematype_of_restricted_expr entirely and re-implement this function to typecheck directly.


How To get full tweet Text.

How can i get full text of tweet.sometimes its returns ...... at last.??

you can use tweet.FullText

FullText have ... too

To verify

Any workaround for this at the moment?

Hello there have you set up the configuration required to use Twitter Extended Mode.
TweetinviConfig.CurrentThreadSettings.TweetMode = TweetMode.Extended;
TweetinviConfig.ApplicationSettings.TweetMode = TweetMode.Extended;


Update docker packages in the repo and fix API mismatches.

Updated docker packages in the repo as otherwise the mount ID interfaces and plumbing are missing. All docker packages are updated to ensure we have all docker-side changes in the repo. Some of the APIs are modified to accept a context parameter and those are fixed in the plugin client code as well - to match the new API definitions.
Tested out volume create/inspect/run/remove with docker 1.12 and 1.11 and both work fine with the new packages.

Can you include the gvt steps followed?

Some of the APIs are modified to accept a context parameter and those are fixed in the plugin client code as well - to match the new API definitions.

Which APIs are modified ? It is not clear from description, and the change is too big to track it manually.
Otherwise LGMT , but please, do comment on which APIs are modified so we can review

GVT steps include:

make gvt
gvt update  and go-plugin-helper packages

Mark,
The APIs changes are handled in vmdk_plugin/refcnt.go - the client APIs accept a context argument now. and in vmdk_plugin/sanity_test.go.

Thanks @govint
LGTM

Thanks Mark, one more LGTM and I'll submit

LGTM.


integration: add integration test for docker+devicemapper+kill

verify kill umount the device mapper

(exercising all possible kill signals, not just SIGKILL and SIGTERM).


Zulu fails to download

==> Downloading http://cdn.azul.com/zulu/bin/zulu<IP_ADDRESS>-jdk8.0-macosx_x64.dmg

However it should be
==> Downloading http://cdn.azul.com/zulu/bin/zulu<IP_ADDRESS>-jdk8.0.72-macosx_x64.dmg

.72 is missing.

Thanks!


Updating documentation for new autodetection method kubernetes-internal-ip

Issue : https://github.com/projectcalico/calico/issues/4870
PR : https://github.com/projectcalico/node/pull/1242

abandoing this review , as i am not able to rebase after monrepo changes. Will be raising another review.


enable redux devtools by default

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
Richie front application uses Redux. There is a lot of tools available to improve the developer experience when using redux. One of the most famous (and most used) is redux devtools.
To use it, you should configure it in your redux store, so you must, programmatically, add dedicated code in your codebase to allow its usage. In your codebase, it should be here. Without this modification it is not allow to use redux devtools.
Describe the solution you'd like
For a better developer experience, it should be great to configure, by default, the usage of this tool. I've created a PR for this (#381) but this one will not be merged as is because openfun's team does not want to ship this code in production.
The configuration needed is explain here and it can be use as is in production.
Describe alternatives you've considered
An alternative can be to configure the tool to be only enabled in development.
There is a dedicated chapter in the documentation about using it in production : https://github.com/zalmoxisus/redux-devtools-extension#14-using-in-production
You should also read this article, explaining why it is not bad to have this code in your production environment : https://medium.com/@zalmoxis/using-redux-devtools-in-production-4c5b56c5600f
Discovery, Documentation, Adoption, Migration Strategy
There will be nothing to do. Richie will be configured to use redux devtools and if a new developer wants to contribute he has nothing to do.
Do you want to work on it through a Pull Request?
A PR already exists #381

I'm closing this (like I closed #381) because we're not using redux anymore.


[CLI] "libman clean" command deletes only deepest empty folders

Functional impact
Some empty folders are left behind. Not sure if that's by design
Minimal repro steps

Install<EMAIL_ADDRESS>using cdnjs into wwwroot\script\jq
Execute

libman clean

Expected result
Honestly, not sure. There is a warning in the command output that empty folders will be deleted. What I am seeing is that "jq" is deleted but not "script". Should we delete empty folders all the way up to the root of the project?
Actual result
Only deepest empty folder is deleted, i.e. "jq"
Further technical details
This could be by design - a bit confusing though that we delete some empty folders but not all

Same as #119

This is a duplicate.


Set function almost never works

[Enter feedback here]
When using the set function, set(varNum, 33) for example. the variable is almost always blank and rarely works. Useless function.

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

ID: 36cb2e4e-44ca-3d84-d872-9d1e55034a51
Version Independent ID: 36cb2e4e-44ca-3d84-d872-9d1e55034a51
Content: Set function - Power Platform
Content Source: power-platform/power-fx/reference/function-set.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thanks for your comment, @victorsj1. This issue has been assigned to the doc author for review.

This is not a documentation issue.
The Set() function is a core function of the language. If you're not seeing the results as you expect, it's likely your formula is not being executed, or something else is overriding your variable somewhere else. If you have problems using it, you can file an issue with support or consider posting on https://powerusers.microsoft.com/ for help.


Logging rr.ClassDescription fails with TypeError

Describe the bug
I'm getting this error when running the human_pose_tracking example.
  File "/home/leo/code/rerun/examples/python/human_pose_tracking/main.py", line 206, in <module>
    main()
  File "/home/leo/code/rerun/examples/python/human_pose_tracking/main.py", line 200, in main
    track_pose(video_path, segment=not args.no_segment)
  File "/home/leo/code/rerun/examples/python/human_pose_tracking/main.py", line 55, in track_pose
    rr.log(
  File "/home/leo/.pyenv/versions/rerun/lib/python3.10/site-packages/rerun_sdk/rerun/error_utils.py", line 160, in wrapper
    return func(*args, **kwargs)
  File "/home/leo/.pyenv/versions/rerun/lib/python3.10/site-packages/rerun_sdk/rerun/_log.py", line 147, in log
    components = list(entity)
TypeError: 'ClassDescription' object is not iterable

This doesn't happen when wrapping the rr.ClassDescription with an rr.AnnotationContext([...])
Rerun version
rerun_py 0.9.0-alpha.6 [rustc 1.72.1 (d5c2e9c34 2023-09-13), LLVM 16.0.5] x86_64-unknown-linux-gnu bde24df, built 2023-10-02T12:51:19Z

I guess this might already be addressed https://github.com/rerun-io/rerun/pull/3603


Typing error with Python 3.9 (TypeError: unsupported operand type(s) for ...)

Python 3.9.15
Trying to execute the sample.py produced 3 typing errors. I had just removed type annotations and it proceeded to the next step (huggingface credentials verification).
Please specify Python version or remove these type annotations (as I did).
python fam/llm/sample.py --huggingface_repo_id="metavoiceio" --spk_cond_path="assets/bria.mp3"
2024-02-09 19:17:47.280515: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.10.1
Traceback (most recent call last):
  File "/home/paul/myapps/data_science/text-to-speech/metavoice-src/fam/llm/sample.py", line 21, in <module>
    from fam.llm.decoders import Decoder, EncodecDecoder
  File "/home/paul/myapps/data_science/text-to-speech/metavoice-src/fam/llm/decoders.py", line 19, in <module>
    class EncodecDecoder(Decoder):
  File "/home/paul/myapps/data_science/text-to-speech/metavoice-src/fam/llm/decoders.py", line 66, in EncodecDecoder
    ) -> str | torch.Tensor:
TypeError: unsupported operand type(s) for |: 'type' and 'torch._C._TensorMeta'

python fam/llm/sample.py --huggingface_repo_id="metavoiceio" --spk_cond_path="assets/bria.mp3"
2024-02-09 19:20:55.558639: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.10.1
/home/paul/anaconda3/envs/tf-gpu/lib/python3.9/site-packages/df/io.py:9: UserWarning: `torchaudio.backend.common.AudioMetaData` has been moved to `torchaudio.AudioMetaData`. Please update the import path.
  from torchaudio.backend.common import AudioMetaData
Traceback (most recent call last):
  File "/home/paul/myapps/data_science/text-to-speech/metavoice-src/fam/llm/sample.py", line 455, in <module>
    enhancer: Optional[Literal["df"] | BaseEnhancer],
TypeError: unsupported operand type(s) for |: '_LiteralGenericAlias' and 'ABCMeta'

 python fam/llm/sample.py --huggingface_repo_id="metavoiceio" --spk_cond_path="assets/bria.mp3"
2024-02-09 19:23:50.126062: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.10.1
/home/paul/anaconda3/envs/tf-gpu/lib/python3.9/site-packages/df/io.py:9: UserWarning: `torchaudio.backend.common.AudioMetaData` has been moved to `torchaudio.AudioMetaData`. Please update the import path.
  from torchaudio.backend.common import AudioMetaData
Traceback (most recent call last):
  File "/home/paul/myapps/data_science/text-to-speech/metavoice-src/fam/llm/sample.py", line 533, in <module>
    enhancer: Optional[Literal["df"] | BaseEnhancer],
TypeError: unsupported operand type(s) for |: '_LiteralGenericAlias' and 'ABCMeta'

Running into the same error

Please review #34
I add details on python version

@fakerybakery, what python version are you using?

3.9

added python version to be >= 3.10, <3.12 in #34, i have also reverted the typings to use Union over the | operator.
closing this issue


global-search.end render hook repeated call

Package
filament/filament
Package Version
v3
Laravel Version
v10
Livewire Version
v3
PHP Version
8.1
Problem description
When enabling the render hook global-search.end in AdminPanelProvider:
->renderHook(
    'global-search.end',
    fn(): View => view('filament.hooks.lang-switcher'),
)

the view is called many times as I have resources with global search enabled

Expected behavior
The view call should be only once
Steps to reproduce
after installing filament follow the changes here
https://github.com/lara-zeus/test-fila-three/pull/1/files
Reproduction repository
https://github.com/lara-zeus/test-fila-three/pull/1/files
Relevant log output
No response

Fixed in next v3 release


Fixed #29022 -- Fixed protocol-relative URLs with HashedFilesMixin when STATIC_URL='/'

https://code.djangoproject.com/ticket/29022

nice

@adamzap Thanks :+1:


mindustry: fix build failure

arraycopy: element type mismatch: can not cast one of the elements of java.lang.Object[] to the type of the destination array, java.lang.String

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Could you apply the patch mentioned in https://github.com/NixOS/nixpkgs/pull/316849#issuecomment-2166915732 instead?

I am not sure how. We probably need to patch the vendor directory which sounds like more trouble than it is worth. It will probably be fixed with the next mindustry update and then we can unpin or at least update some dependencies.

We probably need to patch the vendor directory which sounds like more trouble than it is worth

We can use extraPrefix, see #323540

Thanks. Didn't know it was that easy?


Programming exercises: Improve appearance of results and task bar

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

Make the Ui of programming exercises a bit more intuitive, so students don't mix up the different elements and there is a more clearer distinction.
Description

The "show all"/"hide all results" button now has a triangle like other show/hide toggles in Artemis, making it more intuitive. Additionally it always stays at the same place and is not put at the very bottom when showing all results.
The task bar showing icons for all tasks now uses the normal Tooltips and the horizontal line through all tasks is replaced by a small title before all tasks to make it clearer what these icons are representing.
Steps for Testing

Prerequisites:

1 Instructor
1 Student
1 Programming Exercise

Log in to Artemis
Go to the exercise
Look at the UI elements before submitting once and check for their understandability
Submit once
Look at the UI elements and check for their understandability
Submit another time, so there are at least two results
Look at the UI elements one last time and check for their understandability

Review Progress

Code Review

[x] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Screenshots

Before:

Now:

Codecov Report

Merging #6067 (ff21c16) into develop (dde7562) will increase coverage by 4.00%.
The diff coverage is n/a.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6067      +/-   ##
=============================================
+ Coverage      76.08%   80.09%   +4.00%
- Complexity         0    12689   +12689
=============================================
  Files           1159     2192    +1033
  Lines          45790    86856   +41066
  Branches        8584    12982    +4398
=============================================
+ Hits           34841    69568   +34727
- Misses          5669     9591    +3922
- Partials        5280     7697    +2417

Impacted Files
Coverage Δ

...is/web/rest/metis/ConversationMessageResource.java
100.00% <0.00%> (ø)

...rogramming/ProgrammingExerciseTestCaseService.java
91.39% <0.00%> (ø)

...ww1/artemis/web/rest/ModelingExerciseResource.java
96.00% <0.00%> (ø)

...sitory/tutorialgroups/TutorialGroupRepository.java
80.00% <0.00%> (ø)

.../artemis/web/rest/dto/StudentExamWithGradeDTO.java
100.00% <0.00%> (ø)

...in/www1/artemis/service/ResourceLoaderService.java
65.51% <0.00%> (ø)

...temis/domain/enumeration/FeedbackConflictType.java
100.00% <0.00%> (ø)

...www1/artemis/domain/quiz/QuizStatisticCounter.java
100.00% <0.00%> (ø)

...ww1/artemis/repository/TextExerciseRepository.java
100.00% <0.00%> (ø)

.../main/java/de/tum/in/www1/artemis/domain/Team.java
94.73% <0.00%> (ø)

... and 1024 more

Flag
Coverage Δ

client
76.09% <ø> (+<0.01%)
:arrow_up:

server
84.56% <ø> (?)

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 dde7562...ff21c16. Read the comment docs.

Related issue: https://github.com/ls1intum/Artemis/issues/1070


governance update

Following changes at anaconda inc and the creation of conda/ceps and https://github.com/conda-archive, it seems to be time for us to revisit the governance, proposing an update and voting on it.
I am starting this issue to gather comments/feedback.
cc @jezdez

Some thoughts/comments:

Our current governance might be a bit too complicated.
The distinction between enhancement proposal votes and standards repo votes is not clear. We might just want the former.
If we can use conda-archive, then the attic stuff can go.

Also there are some simpler ideas here that might be of use: https://github.blog/2021-07-22-minimum-viable-governance-lightweight-community-structure-foss-projects/

Some thoughts/comments:

Our current governance might be a bit too complicated.
The distinction between enhancement proposal votes and standards repo votes is not clear. We might just want the former.
If we can use conda-archive, then the attic stuff can go.

FTR, I chose to not use "attic" for the org name to not conflict with the current governance policy and require an explicit update of the policy to cater to it. conda-archive is an intermediate step to get old repos out of the way in the conda org while I'm working on doing the permission review and revamp. All repos in there were archived or abandoned already.
I would suggest to add the conda-archive to the list of orgs where the conda organization is maintained and later explain the full life-cycle of conda-related projects, basically conda-archive being the long-term storage for unmaintained/abandoned etc projects. Of course the org could also be renamed to conda-attic if requested by the current signatories.

This all sounds great @jezdez! I don't think there are strong opinions on what to call what here or the like. I think my own interest is in making things clear and simple for everyone. Renaming to conda-archive with a separate org is great by me!
BTW, there are a few projects (maybe only one) in conda-archive that were (soft-)forked to conda-incubator (e.g. libronda). We should make sure those have one home when the time comes.

This all sounds great @jezdez! I don't think there are strong opinions on what to call what here or the like. I think my own interest is in making things clear and simple for everyone. Renaming to conda-archive with a separate org is great by me!

Same!

BTW, there are a few projects (maybe only one) in conda-archive that were (soft-)forked to conda-incubator (e.g. libronda). We should make sure those have one home when the time comes.

Good catch, didn't know that.


Web console: add more query fixing auto suggestions

This PR improves the auto suggestions for queries and just makes the query experience just a little bit better.

It also improves query intelligence when fractional indexes are used (like GROUP BY 1.5) which is strange but supported by Druid/Calcite - who am I to judge?
Syntax like FETCH NEXT 5 ROWS ONLY is now supported in highlighting
This PR has:

[x] been self-reviewed.
[x] been tested in a test Druid cluster.

Thank you for the review and the help with figuring out POM files @clintropolis


Should all types with from_consensus implement TryFrom / From

units::locktime::relative::Height has an infallible constructor from_height and a matching From impl.
units::locktime::absolute::Height has a fallible constructor from_consensus but not TryFrom

Should all types that implement from_consensus constructors all have matching TryFrom  or From impl

@Kixunil has forcefully argued "no" elsewhere and I am tired of arguing the point.
In the specific case of locktimes, we have these types Height and Time which are entirely defined by their inclusion in the locktime types. Vs something like Sequence which (IMO) should have From<u32> but Kix feels that this is too mysterious a conversion.

Oh my bad, I was just trying to go over the units crate with a fine tooth comb and see if we missed stuff. I totally forgot about all that discussion.

lol, that is pretty funny actually - we argued about that so much.


Add support for unified cluster-vsphere app

Towards https://github.com/giantswarm/roadmap/issues/3125

[!CAUTION]
cluster-vsphere v0.61.0 is not yet released.
cluster-vsphere v0.61.0 with default apps will be released once this PR has been merged. Only then clustertest with this change can be used.

Add support for unified cluster-vsphere app. With cluster-vsphere v0.61.0 and newer, default apps are deployed with cluster-vsphere, and default-apps-vsphere app is not deployed anymore.

@giantswarm/team-tinkerers just a sanity check - this change (plus then bumping clustertest in cluster-test-suites and cluster-standup-teardown) should be enough to support unified cluster-vsphere, right?

Yeah that should be it. Just make sure the order is:

Release clustertest
Update cluster-standup-teardown and release
Update cluster-test-suites with both the above releases and then release

Happy to help with this if needed.

Released cluster-vsphere v0.61.0 https://github.com/giantswarm/cluster-vsphere/releases/tag/v0.61.0, so we can merge and release this, and then update cluster-standup-teardown and cluster-test-suites.

Released cluster-vsphere v0.61.0 https://github.com/giantswarm/cluster-vsphere/releases/tag/v0.61.0, so we can merge and release this, and then update cluster-standup-teardown and cluster-test-suites.

@giantswarm/team-tinkerers are you fine with this?

Yeah, want me to handle all the releases this morning?


MAINT: stats: remove an _argcheck call

This was needed when _argcheck could set the .a and .b attributes. Since it
no longer does (after gh-9900 and gh-10104), the call is not needed. The attributes are set in self.dist._get_support which is, literally, the next line.
EDIT: For context, the call was added in gh-3245

In addition to setting the .a and .b attributes, _argcheck also checks the arguments and returns a boolean (broadcasted array).  But that return isn't being used here, so I agree there is no reason to keep the call.

Thanks @ev-br & @pvanmulbregt