[k8s-configuration] BREAKING CHANGE: az k8s-configuration create: Removal of create cmd for flux v1

Describe the bug
The command az k8s-configuration create has been obsolete for two years. As a replacement, we introduced az k8s-configuration flux create. The former command utilizes Flux v1, while the latter employs Flux v2. We have already informed our customers that from December 2023 onwards, they will only have the ability to list, display, or remove Flux v1 configurations. All production workloads have been transitioned to Flux v2.
Related command
az k8s-configuration create
Errors
create cmd can no longer be used. The cmd has been deprecated for 2 years now.
Issue script & Debug output
N/A
Expected behavior
az k8s-configuration create cmd should no longer work.
Environment Summary
azure-cli: 2.52.0
k8s-configuration: 1.7.0
Additional context
No response

Thank you for opening this issue, we will look into it.


upgrade @headlessui/react to v2

Summary by CodeRabbit
Release Notes

New Features

Updated modal components for improved structure and readability.
Enhanced transition effects in the Main Area for better user experience.
Improved listbox functionality in speech-to-text settings for enhanced user interaction.

Bug Fixes

Resolved inconsistencies in component usage and naming conventions across various components.

Documentation

Updated import statements to reflect new component names from the @headlessui/react library, improving clarity and maintainability.
Updated dependency versions for enhanced performance and stability.

@coderabbitai review

@coderabbitai full review


Error when using bliss + underscore + browserSync (liveReload)

The error seen when using the browserSync - stops working liveReload, in its absence, there is no error.
Please fix as soon as possible :)
The text of the error:
Uncaught TypeError: Cannot read property 'listeners' of undefined
A block of code:
...
// Hijack addEventListener and removeEventListener to store callbacks
...
var listeners = this[].bliss.listeners = this[].bliss.listeners || {};
...
ps: thanks for such a wonderful library! :)
pss: please add versioning for convenience

This very weird. Which line number is it reporting for the error (the code you pasted occurs in two places).
Any chance you could put a console.log(this); line there and tell me what it prints?

Error with package underscore, rechecked link libraries and the exclusion underscore, the error disappeared.
I conclude that combined bliss + underscore + browserSync (liveReload) appears the error.
ps: console.log (this) - I do not understand what you want me to write? :)
check yourself :) underscore + bliss + browserSync (function - liveReload)

@slavikse - Is there any more information you can give us? None of us are using your unique environment. Not sure how to duplicate or where the error is being thrown.

Closing this out. If you have more information that we can act on, feel free to reopen.

This bug can be reproduced in Chrome by using Underscore and Bliss together and attempting to call window.addEventListener() like in http://codepen.io/anon/pen/PZJPpG.
It looks like the Bliss property '' clashes with window. so as an workaround I've changed the property to '__' and everything is fine now.
For some reason this bug does not reproduce on Safari, just Chrome.

I'm getting the same error in Firefox. All I need to do is include both bliss and lodash, and then call addEventListener on window. @spheroid- has it right, it's a name collision, could probably be helped by checking in bliss if this[_] instanceof Bliss or something similar.

FWIW, you can change the name of the _ property, as described in the docs


Fix flakey tests and improve the test log output

I've been trying to fix the flakiness in the pull-cert-manager-approver-policy-verify tests:

https://prow.build-infra.jetstack.net/job-history/gs/jetstack-logs/pr-logs/directory/pull-cert-manager-approver-policy-verify

The tests originally attempted to delete the test namespace after each test, but this can't work because envtest doesn't provide the controller responsible for garbage collection of namespaced resources:

https://github.com/kubernetes-sigs/controller-runtime/issues/880

The logs are much more readable now:

https://prow.build-infra.jetstack.net/view/gs/jetstack-logs/pr-logs/pull/cert-manager_approver-policy/166/pull-cert-manager-approver-policy-verify/1600146755117125632


Group By another column and Sort By another column

I am trying to set the TOM metadata allowing me to Sort a column by another, and Group a column by another.

For Sort By, I need to enable "Unsupported experimental features"  Why is this - Sort By is a supported property, exposed in Power BI Desktop?
For Group By,  I assume this is editing the TOM property https://docs.microsoft.com/en-us/dotnet/api/microsoft.analysisservices.tabular.relatedcolumndetails?view=analysisservices-dotnet
If so, you might want to expand the definition, rather than just say this forms the composite key. Anyone familiar with AMO/SSAS might think this is like the Key of an attribute. I don't think it is - it is a directive to clients that if the user asks to group on the column, they should also include these GroupBy columns as well. So I don't think it effects the behavior of DAX, or even Excel - though the PBI visuals do respect the property

External Tools for Power BI Desktop currently only supports a limited subset of modelling operations. Setting column properties (such as the Sort By property), is not among the supported write operations.
Great insight on the Group By property - I wasn't aware of that! Will make sure to update the description. Thanks!

Fixed in 2.14.2. Thanks for reporting!


Schema: BigDecimal Arbitrary restricted to the range of 0-18

Type

[x] Optimization
[x] Bug

Description
In the case of very large scales, the BigDecimal can no longer be formatted into a string, which breaks in the default encoding (if trying to serialize BigDecimal to a string).
this is extremely limting the usecase of the arbitrary as the default Decimal will serialize to a string
Additionally a negative scale is invalid. A decimal cannot have a negative number of decimal points therefore the scale must always be positive or 0 (bug)

Ah yeah, that makes sense :-P

Type

[x]  Optimization
[x]  Bug

Description
In the case of very large scales, the BigDecimal can no longer be formatted into a string, which breaks in the default encoding (if trying to serialize BigDecimal to a string).
this is extremely limting the usecase of the arbitrary as the default Decimal will serialize to a string
Additionally a negative scale is invalid. A decimal cannot have a negative number of decimal points therefore the scale must always be positive or 0 (bug)

This is not correct though. A negative scale is indeed valid. The scale either shifts the decimal point to the right (adding zeros) if negative, or the the left if positive.
E.g:
BigDecimal.make(1234n, -5) // 123400000 (1234 * 10^5)

Regardless, I'm in favour of this change because it is likely closer to what users expect (decimal values and not large integers) when generating arbitraries.
Thoughts @gcanti ?

In the case of very large scales, the BigDecimal can no longer be formatted into a string, which breaks in the default encoding (if trying to serialize BigDecimal to a string).

We will have to address this in a follow-up. The current BigDecimal.format implementation fails with RangeError or OutOfMemory because of the way it generates the zero padding. There's no way to solve this whilst keeping the zero padded notation for arbitrary length BigDecimals. Imho we should fall back to scientific notation beyond the DEFAULT_PRECISION (100 decimals).

This is not correct though. A negative scale is indeed valid. The scale either shifts the decimal point to the right (adding zeros) if negative, or the the left if positive.

You're right, I did not consider this case because bigint can handle large numbers, hence assumed scaled is only used if decimal point exists, but it is true that it can use a negative scale for a more optimal storage of very large numbers

Thanks @arijoon @fubhy


showHint to take index of option active on startup

Currently showHint activates the first element of option, when it shows the completion widget.
I'd like to control which element is active by default.
I can submit a patch if it's acceptable. Thanks!

Looking at the code, it seems you can already set a selectedHint property on your completion object to control this (a zero-based integer index)

Ah, cool!
Let's reopen the bug, to track the fix to the documentation? (it doesn't mention selectedHint)

Good point. See attached patch


Add support for password protected irc channels

IRC channels with mode +k which require passwords to join

I think this is the oldest ticket I've ever closed. This is now available.

oof, missed a full decade old by 20 days :)


[Beacon API]: Update Beacon API to allow audits behind an authenticated experience

User story

As a [user role below]:

Beacon for IBM.com Extension user

I need to:

run the extension on an authenticated page

so that I can:

run Beacon for IBM.com audits on my authenticated IBM.com experience
Additional information

The API currently can only run audits on public pages, as it takes the url and runs the tests in a Chrome browser within the service before returning the results
Switching the API to use Puppeteer may allow for the audits to be run regardless of whether the experience is behind authentication or not

Acceptance criteria

[ ] Service API updated to allow running the Chrome extension to provide audits to authenticated pages

Hey team! Please add your planning poker estimate with ZenHub @annawen1 @ariellalgilmore @emyarod @kennylam @IgnacioBecerra


Bump cats-effect to 1.2.0

This is required for issue #63

I can pick this one

Go ahead! In case you need any help there is a small contributing guide and a chat room on gitter.

I guess this would come automatically with updated scala-lru-map and hammock

closing as transient dependency


Default marker function name conflicts

I'm trying to use marker function in my code, using aliasing to avoid conflicts:
import { _ as extract } from '@biesbjerg/ngx-translate-extract';
...

But it seems that aliasing or not, Karma does not like the fact that I there's a _ function used here:
Error: Error encountered resolving symbol values statically. Calling function '_', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function,

Maybe you should consider using a more explicit default marker function name, such as extract for example? This also cause issues when using popular libraries such as lodash or underscore in the project.

+1

I also considered to this error in angular4(universal) project, i oped<EMAIL_ADDRESS>file and renamed '_' to other thing but this error continuesly rasing 😞

Support.  Please rename the _ marker to extract or something more meaningful. _ as a function name doesn't explain it's purpose in code and collides with other popular libraries.

Exactly, and if someone wants to use that, there's the as .

You guys realize that there is nothing special about the _ function? If you'd rather call it extract you can simply declare such a function and tell ngx-translate-extract about it.

You guys realize that there is nothing special about the _ function? If you'd rather call it extract you can simply declare such a function and tell ngx-translate-extract about it.

That's not the issue, the problem is that the current naming cause conflict with other broadly used libraries and cause error during tests.

This "problem" has a workaround that is shorter than your comment:
function extract(...s: string) {}

and
ngx-translate-extract -m extract

You're actually missing a return s; for it to work.

@bedag-moo That's what I already did, but still not the point of this issue.

I've created a separate package for the marker function.
npm install @biesbjerg/ngx-translate-extract-marker
import { marker } from '@biesbjerg/ngx-translate-extract-marker';

What version did this get fixed for? I am in 4.2.0 and it is not working


视频会议静音工具栏不适配2k/4k显示器

Description of the new feature / enhancement
在使用视频会议静音功能时，工具栏过小，不匹配高清显示器

Scenario when this would be used?
使用此功能时
Supporting information
No response

Thanks for the feedback but VCM is moving to maintenance only in the 0.67 timeframe. Our team will only directly address critical bugs, security and accessibility issues.
We'll accept community PRs for enhancements.


[DOCS] Add MIT License and Update README.md

Description
Please add the MIT License to our project repository and update the README.md file accordingly.
Expected Behavior
The MIT License should be added to the project repository, and the README.md file should be modified to include information about the license, including a license badge and any relevant details about the license terms.
Actual Behavior
Currently, our project does not have a license specified, which can create confusion about how the code can be used and shared. It's essential to add a clear and permissive license to ensure that others can use the code with legal certainty.
Screenshots (if applicable)
N/A
Additional Information
By adding the MIT License and updating the README.md, we will make it clear to potential users and contributors how they can use, modify, and distribute the code. This action aligns with open-source best practices and ensures legal compliance. Please follow standard procedures for adding the MIT License to the repository. Example.

Hello. I would like to work on this one.

@bkpecho ok! Follow the best practices for commits


⚠ GitLab Server has degraded performance

In ab37aa4, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6815 ms

Resolved: GitLab Server performance has improved in 1516d65.


Feature Request - View (instead of replace) the correct spelling

It would be nice if one can type something like "spell ralley" and the workflow shows up "rallye" (without replacing in). The current workflow uses the spell command from alfred which does not work for other languages than english (e.g. german).

The current workflow uses the spell command from alfred which does not work for other languages than english (e.g. german).

It does. It uses whatever dictionaries you have in Dictionary.app. Add your language there, and Alfred’s spell will use it as well.

Thanks, but it doesn't work the way I described it in german. For english the spelling seems to be very good but for german not. The replace feature from your workflow though works like a charm for german. Is it possible to use the service (service.afterthedeadline.com) for the preview feature as well? (and not only for the inline replace thing)

For english the spelling seems to be very good but for german not.

That is something you should likely report to Apple.

Is it possible to use the service (service.afterthedeadline.com) for the preview feature as well?

Yes, but I won’t do it. I’ve added that feature to the workflow but don’t really use it and don’t see neither enough value nor enough demand (you’re the only one) to justify the work. I already have too much free open-source work on my hands right now.

Yes, but I won’t do it. I’ve added that feature to the workflow but don’t really use it and don’t see neither enough value nor enough demand (you’re the only one) to justify the work. I already have too much free open-source work on my hands right now.
Alright, i'm going to implement it myself. Thanks for Your time :)

Thanks for Your time

Thank you for taking the time to open the issue and make the request.


Update CLM Fees Descriptions & Blue Chip Tokens List

Amend the 3 Fees descriptions on CLM products to make clear that the fees relate to the user's position and not the whole CLM position.
Amend the blue chip tokens list to add and remove various tokens.

@iamjackgale Can you please include the wrapped versions of natives, i.e. WFTM & WMATIC?

@iamjackgale Can you please include the wrapped versions of natives, i.e. WFTM & WMATIC?

Good shout! Added 👍


Pomodoro menu bar icon and functionality is gone from latest version

Pomodoro icon is gone and clicking the space where the icon should be shows an empty drop down menu as well. I'm on Arch, Gnome 3.30.2

Are you using the github version?

No, the extensions.gnome.org one - should I use the github one?

Yes. The issues is fixed, but the extensions site is not update yet.

OK, thanks!


Added possibility to pass in options object to scale control

Example:
$scope.controls = {
scale: {imperial: false}
};

Thanks @kjellivar , merged!


Add no patch option to the FSLeyes plugin settings menu

Checklist

[X] I've given this PR a concise, self-descriptive, and meaningful title
[X] I've linked relevant issues in the PR body
[X] I've applied the relevant labels to this PR
[ ] I've added relevant tests for my contribution
[ ] I've updated the documentation and/or added correct docstrings
[X] I've assigned a reviewer
[X] I've consulted ADS's internal developer documentation to ensure my contribution is in line with any relevant design decisions

Description
Adds the "no-patch" option to the settings in the FSLeyes plugin. This option was added to the CLI in #696.
Linked issues
Resolves #699

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1161

Relevant Lines:
1297

💛  - Coveralls

Thank you @mariehbourget for the review. I should have time to look at the issues sometime tomorrow.


Cannot claim PoolTasks created before you were added to a workflow approval group

Describe the bug
It's currently not possible to claim a PoolTask if that PoolTask was created prior to your being added to the workflow group with access to that PoolTask.
To Reproduce
Steps to reproduce the behavior:  (It's easiest to reproduce via the UI)

Login as an Admin & Create a Collection with Workflow enabled
Submit several Items into that Collection (creating 2+ PoolTasks)
Now, add a new EPerson to the list of reviewers for that Collection
Login as that new EPerson, go to "My DSpace", Show: "All Tasks".  You'll see the 2+ PoolTasks in your list of items to review.
Click "Claim" next to one of the PoolTasks.  You'll get a 403 Access is denied exception, with the following underlying error:org.dspace.app.rest.exception.RESTAuthorizationException: org.dspace.authorize.AuthorizeException: You are not allowed to to perform this task.

Now, login as an Admin again & submit a new Item to that same Collection (creating another PoolTask)
Login as the new EPerson (created in step 3).  You'll be able to claim that newly created PoolTask successfully.

Expected behavior
As in DSpace 6, Configurable Workflow should allow you to claim PoolTasks which were created before you were given reviewer rights.
Tests should also be added to prove that a newly added reviewer can immediately claim existing PoolTasks.

@abollini : Could you, or someone on your team, look at this quickly to see if it's easily resolvable?  Ideally, I'd like this solved in Beta 5 or 7.0 final, unless you feel this is too complex to fix quickly.


AL-20: Create alert duplicate check

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


How does one define a task?

I suspect we should have some reasonable API/strategy for determining whether a task is done (other than just retrying it). This way we could avoid rerunning steps. It would be pretty easy to get pre/post install hooks along with this.
I think it's probably best to avoid just keeping a log, and rather actually check if things are done? A log isn't necessarily in sync with the host. Basically, a task needs a test case I suppose? : )

Necessary to define the split between
Things that can be done on a host -- Things that use those basic commands
Imo.

Initial revision of how to do this is out:
A command:
More or less a system command : )
A Task:
may have dependencies on other tasks
may run commands
A role:
defines machines we expect to do a task


add pyside to Wily/Xenial

Sources:
http://packages.ubuntu.com/xenial/python-pyside
http://packages.ubuntu.com/xenial/python3-pyside

What are you using this for? Kinetic aims to use Qt 5 instead and therefore packages should use PySide 2.

Ecto. Sure, what is the Wily/Xenial package for pyside2 ?

The rosdep key python-qt5-bindings contains the PySide 2 packages (beside PyQt). You can look up the Debian keys there.

I'll complete the pyside2 keys if I need it. Thx


Binder creates its own KTM instead of injecting Boot's

Also, if there is no existing KTM bean the one the binder creates should have
setTransactionSynchronization(AbstractPlatformTransactionManager.SYNCHRONIZATION_ON_ACTUAL_TRANSACTION)

So that the producer will participate in the consumer's transaction, for exactly once semantics.
Work around is to add TransactionSynchronizationManager.setActualTransactionActive(true); to the @StreamListener.

Problem is in SIK - but leaving this since the KTM should be configurable.

The reason for opening this has been fixed in S-I-K 3.1.2 https://github.com/spring-projects/spring-integration-kafka/pull/261
So, while this should still be done, it is not urgent.

From the binder, we can now use ListenerContainerCustomizer to configure a  KafkaTransactionManager.
See this answer from @garyrussell on this Stack Overflow issue.
Essentially, you can use the ListenerContainerCustomizer in the following way to customize a Kafka transaction manager:
@Bean
public ListenerContainerCustomizer<AbstractMessageListenerContainer<byte[], byte[]>> customizer() {
    return (container, dest, group) -> {
        KafkaTransactionManager<?, ?> tm = (KafkaTransactionManager<?, ?>) container.getContainerProperties()
                .getTransactionManager();
        tm.setTransactionSynchronization(AbstractPlatformTransactionManager.SYNCHRONIZATION_ON_ACTUAL_TRANSACTION);
    };
}

Closing the issue.


XML transformation and indentation when using xml:space

Summary
When switching from Java 8 (Oracle) to Java 11 (AdoptOpenJDK) the indentation of our XML has been changed.
The indentation fails when a xml element is defined with an attribute xml:space=preserve. Everything after the xml element containing the attribute is no longer indented

Steps to reproduce
With the below code the problem can be reproduced. The variable data must be a XML string, for instance
<?xml version=\"1.0\" encoding=\"UTF-8\"?>650b4d21<c>If true</c><d xml:space=\"preserve\">TRUE</d><e>test</e>
In this case the tags 'a','b','c','d' are indented, everything after tag 'd' will not be indented.
	Transformer transformer = TransformerFactory.newInstance("com.sun.org.apache.xalan.internal.xsltc.trax.TransformerFactoryImpl", Test.class.getClassLoader());

	transformer.setOutputProperty("{http://xml.apache.org/xslt}indent-amount", "4");
	transformer.setOutputProperty(OutputKeys.CDATA_SECTION_ELEMENTS, "body hyperlink");
	transformer.setOutputProperty(OutputKeys.ENCODING, "UTF-8");
	transformer.setOutputProperty(OutputKeys.INDENT, "yes");
	transformer.setOutputProperty(OutputKeys.OMIT_XML_DECLARATION, "yes");

	StringWriter writer = new StringWriter();

	StreamSource in = new StreamSource(new StringReader(data));
	transformer.transform(in, new StreamResult(writer));

	System.out.println("-----");
	System.out.println(writer.toString());
	System.out.println("-----");

Expected results
<?xml version=\"1.0\" encoding=\"UTF-8\"?>

        650b4d21
        <c>If true</c>
        <d xml:space=\"preserve\">TRUE</d>
        <e>test</e>

Actual results
Below XML:
<?xml version=\"1.0\" encoding=\"UTF-8\"?>

        650b4d21
        <c>If true</c>
        <d xml:space=\"preserve\">TRUE</d><e>test</e>

Triaging info
Java version:
openjdk version "<IP_ADDRESS>" 2020-11-04
OpenJDK Runtime Environment AdoptOpenJDK (build <IP_ADDRESS>+1)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build <IP_ADDRESS>+1, mixed mode)

What is your operating system and platform?
Mac OSX Catalina

How did you install Java?
Installer

Did it work before?

Did you test with other Java versions?
Java 15

unfortunately there is no MAC version available on the jdk.java.net. but i will ask a colleague to test it on windows instead and then i will report back

On https://jdk.java.net/16/, there is: Look for "macOS / x64". Same on https://jdk.java.net/17/.

Fixed in Java 17 (but not backported)


Molecule v2

migrate Molecule v1 to Molecule v2
add tests verifying that the packages are installed

With #6, proposing to release as 2.0.0 given the breaking change?


Track GlobalMessages

This is attempting to intercept the CreateGump from Global Chat (864546439)
Then inside JournalManager, I am reading and deconstructiong the lines of string into journal entries.
Because the global chat gump call is sending all lines(messages) on each request, i have to hash the entries to avoid duplicates.
Inside  AddJournalEntry you will see i am checking for the existing uint.

Again this is not elegant and more a hack.
The other option i was going to do was just raw feed the gump payload into an empty tab on ResizableJournal.

Maybe I'm not understanding correctly but is this meant to parse the global chat from UOAlive?

After having  tested this out, I see the idea and I'm impressed, however this only functions if UOAlives chat gump is already open which kind of defeats the purpose.. Unless UOAlive is willing to send some custom packets to the client where we don't need the gump open I'm probably going to close this one out

Did some more break pointing > the server does sent client notification of Global Chat entries
PacketHandlers > UnicodeTalk > MessageManager.HandleMessage
You would need to check a few other parameters as well because this handle normal character chat as well.
entity == null  with serial ==<PHONE_NUMBER>
name  != "System"
hue == 1150
type == Regular
text_type  == SYSTEM

Closing this unless UOAlive wants to add a custom packet sent on global chat messages


libwebrtc

Motivation for this change
Make telegram calls work again
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Currently there's a problem with gclient, as it tries to fetch the source
Possibly creating a git repo out of the source and fetching from there could work, or disabling pulling

Got the cloning to work, problem is there's a list of deps in ${src}/DEPS that it tries to pull in
Should be possible to take those deps and also create fake git repos

Working on pre-fetching those deps and then placing them into the right dir in configure

we could fetchpatch ARCH patches from https://github.com/archlinux/svntogit-community instead of including them into nixpkgs

I postponed that until I actually get past configure phase

Currently there's a problem with gclient, as it tries to fetch the source

Yeah, that should be the most difficult problem to solve.

Working on pre-fetching those deps and then placing them into the right dir in configure

That would be very nice if that works out.
If it turns out to be too difficult and if gclient fetches the dependencies in a deterministic way (AFAIK all dependencies are either specified via a Git revision (but git-clone isn't deterministic IIRC) or e.g. a hash of the binary to fetch) then you could also write an intermediate fixed-output derivation that takes the libwebrtc source and then runs gclient to fetch the sources.

Currently git fetching is done by nix-prefetch-git, but some of the packages have cipd packages as sources specified, I'm not sure how that should be handled
So far I've found that packages require a platform variable, like linux-amd64, which if replaced leads to a package https://chrome-infra-packages.appspot.com/p/infra/tools/luci/isolate/linux-amd64/+/
Then choosing the latest version and downloading that https://chrome-infra-packages.appspot.com/p/infra/tools/luci/isolate/linux-amd64/+/ScErc0J9LrGAd_W-B-h0JC5BGACx-AQrJl4NVezfSAUC gives a zip file
In that particular case I got a binary, I'd either need to patchelf it or create a FHS, right?
This is getting complicated...

In that particular case I got a binary, I'd either need to patchelf it or create a FHS, right?

Yeah, running the autoPatchelf command manually might be the best/easiest approach (see autoPatchelfHook in https://nixos.org/manual/nixpkgs/stable/#ssec-setup-hooks). There might be an even better approach but I never really looked into those things (/ luckily never needed them much).

This is getting complicated...

Yeah, unfortunately this'll be a huge PITA to get packaged with Nix :o

Please fix the pipeline.

I currently don't have the time to push this forward. PR can be taken over if someone else is willing to finish it

@ofborg eval


Navigation bar

Page is not changing on pressing the Icon in nav bar. Fix the problem with your page change logic

The project didnt had all the pages required to link...what should i do..? Should i create some blank pages for it..??...and can you tell what all pages should i navigate to through each of the nav bar icons..

Yes ,you  can create blank pages only with Scaffold and page Name in the center (as in figma design)which can be navigated through nav bar on clicking on icon .But make sure you create page file  in the main_app _widget folder.
As far as on which icon ,which page  to navigate is specified in  figma design (Check out once).And ask me if still have any questions.


Unable to add any account type.

Same problem as issue #471 on a Galaxy S6, original Samsung software (Android 7.0). AndStatus version 50.01 from F-Droid.
As previously described, an empty 'add this account' screen displays briefly, then the UI backs out to the main menu. Happens for all account types, repeatedly. Only one time I saw a login form flash on screen for a fraction of a second before disappearing.

Please turn on logs,as described here https://github.com/andstatus/andstatus/issues/225 including turning on "Backup log files"
Then resteart device, try to create an account once, and please create a backup with data for me to analyse. Please attach the zipped backup folder to this issue, or send to me directly...

Will do!

I've emailed the logs to you@yourwebsite; I'll be happy to perform more troubleshooting.

Thank you for testing.
I can think of another way to move forward before I create the permanent fix: Restore AndStatus data from a backup, created on a device, where everything works as expected.
This device could be even Android emulator.

Same problem on Nokia running stock Android 9

@colegota I tried to add new account in #AndStatus v.50.01, and I cannot do this either.
Actually, I'm using v.50.01 now and didn't notice any other problems yet. But I didn't add new accounts since last upgrade.
So existing users seems like unaffected by this bug.
Will fix the latest version as soon as I will be at my PC (I'm still faraway...)
For the new users:

Please uninstall v.50.01,
Install v.49 using G-Droid/F-Droid app. I just tried G-Droid, see screenshot - three versions of AndStatus are available. https://f-droid.org/en/packages/org.gdroid.gdroid/
Add all necessary accounts.
Create a backup in AndStatus Settings -> Storage -> "Back up app data to SD card".
Backups are always good, and in a case step 5 fails you can quickly restore everything, reinstalling v.49 and then Restore this backup from the first screen of the app.
(optionally) Update AndStatus to v.50.01 in the same G-Droid/F-Droid app. #G-Droid misbehaved here for me on a first attempt... but after I had restarted it, it updated AndStatus successfully. https://loadaverage.org/attachment/5398183

Did it and seems to work OK!
Installed v49, created accounts and upgraded to v50.
No need to restart.
Thanks a lot!
Colegota
El 2019-06-17 20:42, Yuri Volkov escribió:

@colegota [1] I tried to add new account in #AndStatus v.50.01, and I cannot do this either.
Actually, I'm using v.50.01 now and didn't notice any other problems yet. But I didn't add new accounts since last upgrade.
So existing users seems like unaffected by this bug.
Will fix the latest version as soon as I will be at my PC (I'm still faraway...)
For the new users:

Please uninstall v.50.01,

Install v.49 using G-Droid/F-Droid app. I just tried G-Droid, see screenshot - three versions of AndStatus are available. https://f-droid.org/en/packages/org.gdroid.gdroid/

Add all necessary accounts.

Create a backup in AndStatus Settings -> Storage -> "Back up app data to SD card".
Backups are always good, and in a case step 5 fails you can quickly restore everything, reinstalling v.49 and then Restore this backup from the first screen of the app.     * (optionally) Update AndStatus to v.50.01 in the same G-Droid/F-Droid app. #G-Droid misbehaved here for me on a first attempt... but after I had restarted it, it updated AndStatus successfully. https://loadaverage.org/attachment/5398183

--
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [2], or mute the thread [3].

Links:
[1] https://github.com/colegota
[2]
https://github.com/andstatus/andstatus/issues/511?email_source=notifications&email_token=ABG7YAHQPWGV5MRI4SDQRNDP27LI7A5CNFSM4HWIH2I2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX4CXXQ#issuecomment-502803422
[3]
https://github.com/notifications/unsubscribe-auth/ABG7YAEFTWIUICCLWDWQM6DP27LI7ANCNFSM4HWIH2IQ

Confirming workaround is good. ✔
F-Droid had a problem installing v49 as a downgrade but, not having anything to lose, I uninstalled v50 first and the rest went fine.
Was able to add my accounts in v49 and then upgrade back to v50.

Now I had again part of the problem that causes I did remove and reinstall the app.
My timelines are little confused. Unread notifications shows Home (personal timeline, Inicio in Spanish version) while notifications now are shown in Interactions. But this timeline it's not shown when click on widget.
Not sure if open a new issue or wait for next version.
Regards!
Colegota

I think "Timeline confusion" could be a good new issue, thank you.
BUT showing Home timeline by default is the new feature:
https://github.com/andstatus/andstatus/issues/508
It is mentioned in v.49 http://andstatus.org/changelog.html
I should had set it off for old users... but was lazy... sorry for the
confusion
On Tue, Jun 18, 2019, 08:24 Colegota<EMAIL_ADDRESS>wrote:

Now I had again part of the problem that causes I did remove and reinstall
the app.
My timelines are little confused. Unread notifications shows Home
(personal timeline, Inicio in Spanish version) while notifications now are
shown in Interactions. But this timeline it's not shown when click on
widget.
Not sure if open a new issue or wait for next version.
Regards!
Colegota
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/andstatus/andstatus/issues/511?email_source=notifications&email_token=AABSWZA3E4QU3AH5ILM56L3P3B5RPA5CNFSM4HWIH2I2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX5KGCA#issuecomment-502965000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABSWZDRLP7ID7WOHBUNZLDP3B5RPANCNFSM4HWIH2IQ
.

Ran into this trying out AndStatus today. It's unfortunate that this makes the app completely unusable for new users. Hopefully you don't lose too many potential adopters this way :cold_sweat:
Will you please let me know when the Play store is updated? Thank you!

@remram44 Fixed in v.50,02. see #456

Looks like v50.02 was released yesterday soon after my comment. Neat!


Bug in format_pitts250k?

In format_pittes250k.py L55-56
copy_images(f"{dataset_folder}/images/{dataset}/queries", q_images, q_utms)
copy_images(os.path.join(dataset_folder, 'images', dataset, 'queries'), q_images, q_utms)

Seems like these two lines are duplicated and cause an error.

Right, I removed the first line, it should work properly now.
Thank you for the issue :-)


FITCount app - broken SPRING link

The EU-Project SPRING link on the 'About' page of the app is broken. This goes to a 'web page not available' page on Android that cannot be closed/go back, so the app has to be closed and re-started.
@CCarvell Please can you confirm what the correct link should be. Should it be - https://www.ufz.de/spring-pollination/ ?
Jim

Thanks, I'll fix it.

That's the right SPRING link, thanks


Fix date being set to start of unix time

If a date is going to be set at the start of unix time it will use the current time. Also if there is no date column in the image it will use the current time.

Thanks @domdomegg !


[Task]: Document coding conventions

Task description
The coding conventions wiki currently contains no content. With the upcoming refactor it is a good time to document them.
Requirements

Move coding conventions from here
Add testing information from testing wiki
Finalize coding convention decisions with team

Acceptance criteria
The documentation completes the above requirements.
Creation checklist

[X] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

@maxwellmattryan keep the testing in testing wiki, and the linting in the developer tips (I just moved that there)
Let's keep the code conventions explicit rules.
I.e.
Arrow function over General function
Component structure script, components, style

Let's keep the code conventions explicit rules.

Yes, this sounds good. I do think writing tests falls somewhat under that category as they also have specific coding conventions, but I also think devs wanting to find that information would expect it to be in separate places.

On Discord I proposed enforcing === and !== (https://eslint.org/docs/rules/eqeqeq). Using == can cause some unexpected bugs if the objects being compared are different types, so we probably want to avoid that unless it's necessary for some reason


Add coveralls

🚀 Feature Proposal
Add coverage reporting.
Motivation
See how well tests are working.

Some reference:

Install jest
Use the following to run tests and push files to coveralls:

jest --coverage --coverageReporters=text-lcov | coveralls

Check out an example here which makes use of Travis-CI build stages
https://www.npmjs.com/package/coveralls#jest


ElemPtr rework, pahse 3: Removing SmtConstElemPtrs in gen- methods and function-cherrypick

[ ] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

Miscellaneous instances of SmtConstElemPtr replaced with the other kinds. For the most part, any value produced by gen- methods should just use fixed pointers, no point in having overapproximation in generated values.
After both this and #2416 get merged we can remove that pointer sort entirely.

Ah, damn, accidentally merged into main. Well, luckily this change isn't breaking.


fresh install Nextcloud Hub 5 (27.0.1) Could not fetch data

Hello
With a fresh installation I got the following error Could not fetch data while trying to display Shifts.
The Administration panel seems to work fine.
<IP_ADDRESS>-1691005547492.log
<IP_ADDRESS>-1691005562677.log

Hey,
did you create the analyst groups and shift calendar like described here: https://github.com/csoc-de/Shifts/issues/41#issuecomment-1661537627
If so, can you add the log of the server? It should be somewhere in the data folder in a nextcloud.log file
We will add these setup steps to the ReadMe in the future.

Hello
You are right it is working, need to add it in the readme file.
I have an another instance which is not working and the administration panel for shift isn't working.
May be I will make an another issue for that, i ll try to find out.

That the admin settings panel is not loading is strange. Maybe you can send the logs for that if they do not contain sensitiv information?
Or if this instance has no imprtant data you can reset the tables for the shifts app in the database (old data will be lost).

I have a simmilar issue with NC 27.1.1, (upgraded from original NC 24, in various steps)  where the Admin panel is not loading with the notification: "Could not Fetch Settings"
These seem to be relevant messages from install + first activation?
logreader | Could not find resource file "/apps/logreader/css/../js/logreader-main.css" |   | 2023-09-25T11:29:09+0200
-- | -- | -- | --
Debug | PHP | Creation of dynamic property OCA\FirstRunWizard\Settings\Personal::$urlGenerator is deprecated at /var/www/nextcloud/apps/firstrunwizard/lib/Settings/Personal.php#40 |   | 2023-09-25T11:29:08+0200
Debug | PHP | Creation of dynamic property OCA\FirstRunWizard\Settings\Personal::$urlGenerator is deprecated at /var/www/nextcloud/apps/firstrunwizard/lib/Settings/Personal.php#40 |   | 2023-09-25T11:28:48+0200
Debug | no app in context | Deprecated event type for OCP\IDBConnection::CHECK_MISSING_COLUMNS: Symfony\Component\EventDispatcher\GenericEvent is used |   | 2023-09-25T11:28:24+0200
Debug | no app in context | Deprecated event type for OCP\IDBConnection::CHECK_MISSING_INDEXES: Symfony\Component\EventDispatcher\GenericEvent is used |   | 2023-09-25T11:28:22+0200
Debug | no app in context | Deprecated event type for OCP\IDBConnection::CHECK_MISSING_PRIMARY_KEYS: Symfony\Component\EventDispatcher\GenericEvent is used |   | 2023-09-25T11:28:21+0200
Debug | no app in context | 3 well known handlers registered |   | 2023-09-25T11:28:21+0200
Debug | no app in context | 3 well known handlers registered

The next one is about a workflow task running from cron.
The first seems irrelevant, probably from starting the logreader.
Next run:
Debug | logreader | Could not find resource file "/apps/logreader/css/../js/logreader-main.css" |   | 2023-09-25T11:36:40+0200
-- | -- | -- | -- | --
Debug | PHP | Creation of dynamic property OCA\FirstRunWizard\Settings\Personal::$urlGenerator is deprecated at /var/www/nextcloud/apps/firstrunwizard/lib/Settings/Personal.php#40 |   | 2023-09-25T11:36:39+0200
Debug | PHP | Creation of dynamic property OCA\FirstRunWizard\Settings\Personal::$urlGenerator is deprecated at /var/www/nextcloud/apps/firstrunwizard/lib/Settings/Personal.php#40 |   | 2023-09-25T11:36:18+0200
Debug | cron | CLI cron call has selected job with ID 802

Tables are created i think: (these are the ones).
| oc_shifts                                  |
| oc_shifts_cal_changes                      |
| oc_shifts_change                           |
| oc_shifts_type                             |

How to reset tables in the Database? Those are allready empty.
db:add_missing.... all report all tables are OK.

 Version 1.9.0 and above are only compatible with version 25 or high of nextcloud.
The language in the app depends on your language settings in you nc account.
For the base setup you need an admin group (default: ShiftsAdmin). That can be any regular NC group.
If you want to use the calendar app from nextcloud, you also need a dedicated calendar for the shifts app (default: ShiftsCalendar).
The organizer schould be the admin of the nextcloud instance / shifts but is is not required as the organizer will only appear in the calendar invitations.
For the shifts you need a group for all users that should appear in the app (default: Analyst). An for each skill level (at least one) a separate group.

Example user overview

Example admin settings:

To be clear,    the system was version 27.1.1 (Hub 6) BEFORE installing Shifts.
It Originates from a Fresh installed NC 24 and was kept current ever since.
NC Group ShiftsAdmin does exist,  as does Analyst, Level 1 and Level 2.
Some users have been assigned to them.
Still there is no data to fetch... I included a Chrome browser console log above.
https://xxxxxxx/settings/admin/shifts --> only shows loading.
In  an Enlish user setup a german screen is shown for:
https://xxxxxxx/apps/shifts/timeline
A grid with german language daynames and dates on top and is shown After creating an Analyst & Level 1/ Level 2 groups also names appear in the left column.
There is no option to select/changes/update anything.   (black backdrop and white grid, on a dark themed screen with a grey column on the current date).
(adding a ShiftsCalender to the calenders made no difference, and it stays empty as the shifts  are uneditable.
Further possible background info: mariadb is used a DB backend. (Although i don;t expect that does matter).

Saving settings still fails with 405 error though.
https://nc.noci.work/apps/shifts/settings 405

Which means method not allowed... PUT.
The issue most probably is related to nginx not allowing POST/PUT request to items that are read only....
(not put through using proxy_pass) .... The default is   location / { ... redirect... /index.php/... }...
most probably causing the issue.  (no proxy pass there.

This might both be an nginx condition.
replacing:
    location / {
        try_files $uri  $uri/ /index.php$request_uri;
    }

into
    error_page 405 =200 @post_static ;   #/index.php/$request_uri;
    location @post_static {
         proxy_pass https://$server_name/index.php/$request_uri ;
         proxy_redirect off;
    }
    location / {
        try_files $uri /index.php$request_uri;
    }

This is not a complete solution as nog the settings can be done, still no options are shown to users in the ShiftsAdmin group. Not sure what is wrong there.

The above is no solution as it fails URI's without terminating / like ocm-provider / ocs-provider.

Have you created some shift-types that can be shown as available shifts in the overview?

Similar Issues with NC 27.1.2 and PostgreSQL 13.11
"Einstellungen konnten nicht geladen werden"
no config possible due the settings page is not loading at all.
possible an bug with PostgreSQL as database?
different NC install NC 27.1.2 and mysql 10.5.16
config is possible and settings page is loading and I am able to save the settings.
User, Calendar and Groups are created and filled with at least one user
but apps/shifts/timeline complains
"Daten konnten nicht geladen werden" and no page is loading.
I can't create shifts or anything. :(

@eweinhold Does the error still occurs?

If the error occurrs again, please open a new issue. I'm closing this one for now.

The issue still hapens. Installed v.
Is it possible to use non-pretty URLs  https://site/index.php/.........
Those are handled DIFFERENTLY in nginx as they go directly to the http hanadler  (which allows posts ... as script may handle data).
NON- php like URLs are handled by the file-path parts, those DON't allow POSTS the URL.

curl -X PUT https://SITE/apps/shifts/settings/store/

405 Not Allowed

405 Not Allowed
nginx

Hi @noci2012,
i'm not sure what you exactly mean with "non-pretty URLs". The URLs are handled by nextcloud itself and not the shifts app. The only thing I can do is defining the handler for requests. I cannot change the URL.
Maybe the nextcloud documentation on nginx can help you:
https://docs.nextcloud.com/server/latest/admin_manual/installation/nginx.html

'pretty" - URL is another name for URL's not exposing some technology... like the use of PHP scripting.
See above a URL   https://somesite/apps/shifts/settings/
These urls  in nginx go through:  (last chance handling)
location / {
try_files..... ;
}
As there is no proxy_pass / fastcgi_pass in this block it is limited to GET queries only.
This block is also used to add a / on failing requests.
ANY URL that has:    .php/  or .php (end of string)   is handled with a block like
location ~  .php/? {
.....
fastcgi_pass ..... ;
....
}
Meaning if there is .php in the string PUT/POST are allowed.
so a url in the form of
https://somesite/index.php/apps/shifts/settings ...  should active some scripting.
(Due to the pressence of index.php ).
So without a query including index.php shifts will NEVER be activated correctly... causing errors.
At least for nginx.


Update Recognizers-Text packages and fix ChoiceRecognizers' match for "last" utterance

Description
This PR fixes the issue #1477 that reported a problem to recognize the utterance "last one".
Proposed Changes
Updated the following NuGet packages to the latest version (1.2.3).

Microsoft.Recognizers.Text.Choice
Microsoft.Recognizers.Text.DataTypes.TimexExpression
Microsoft.Recognizers.Text.DateTime

Fix MatchChoiceByIndex method in Choice.Recognizers class, to consider when getting "end" as part of the Resolution value.

Pull Request Test Coverage Report for Build 68512

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 77.125%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs/Choices/ChoiceRecognizers.cs
4
95.51%

Totals

Change from base Build 68504:
0.01%

Covered Lines:
4899

Relevant Lines:
6352

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

We cannot take the 1.2 Text Recognizers. They have breaking changes.

Ok @cleemullins, the feature we need to solve this issue is in versions 1.2.x so we'll keep this fix in radar until these changes could be applied. Thanks!

This is no longer 'blocked', considering we have upgraded the recognizers library to 1.2.6:
https://github.com/microsoft/botbuilder-dotnet/pull/2448

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

This is no longer 'blocked', considering we have upgraded the recognizers library to 1.2.6:
#2448

Hi @EricDahlvang, we've been testing this with the version 1.2.6 of Recognizers-Text and found that for "last" utterance to be recognized we need to remove the SupressExtendedType of the NumberRecognizer call.

As you requested for this option explicitly, we left it as it is, but, if you agree, we'll remove it so this PR can be merged.

I did not realize that.  Thank you for clarifying.


Commit and PR diff view does not honour spaces/tabs

Gitea version (or commit ref): 1.14.0+dev-138-gbcb94ed58
Git version: Recreatable on try.gitea.io
Operating system: Recreatable on try.gitea.io
Database (use [x]):

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

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

[x] Yes (provide example URL) https://try.gitea.io/lynxplay/testing-gittea
[ ] No

Log gist:

Description
Diff view on a commit as well as PRs does not honour spaces or tabs anymore.
The file view itself still does tho.
Recreated here:
The file with spaces and tabs: https://try.gitea.io/lynxplay/testing-gittea/src/branch/master/file-with-spaces.txt
The diff commit view without them: https://try.gitea.io/lynxplay/testing-gittea/commit/4c8fd6c646fe172d0d00d613dd9babfa3cebc965
PR files view: https://try.gitea.io/lynxplay/testing-gittea/pulls/3/files
Screenshots

Broken here: https://github.com/go-gitea/gitea/commit/4617bb689b217ae039297ec16124e73b81487b15#diff-5d1373f9290f762a4751c497b0e8be279eb04caa3a7cdc2fd3f42b9f505a83beL188-R192
Since we loose mono.wrap > whitespace: pre

Argh, LESS strikes again, I didn't notice those additional child selectors when moving the mono class (which did too much in itself).

It's worth noting that your changes to the mono class will have improved the appearance of the extended commit messages

as I suggested on #maintainers I think if we change box.tmpl et al to use <code class="mono wrap" ... instead of <span ... this should just fix itself.

Yeah, <code> is semantically correct, mono is just for stuff where you want to force monospace font. I'll take a look later unless you beat me with a fix.


Fix metrics for destination tags

Towards https://github.com/giantswarm/giantswarm/issues/12402

that was just typo in copy-paste

The fix is too obvious so I'll proceed from here without approval

oh lol


Don't allow overloading by opaque types with different constraints.

The mangling unfortunately doesn't take the constraints into account, leading to SR-10725 | rdar://problem/50987172. Changing the mangling at this point is hazardous, so it's probably best to disallow this for now.

@swift-ci Please test


generic serialization decode need use readObject()

What is the purpose of the change
when the return is template like class A
the serialization decode like hession2 is use class A to create
so the result instance contain T is jsonobject, not the real type.
it must be error.
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8141 (d3e07c7) into master (99ccb57) will decrease coverage by 0.81%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #8141      +/-   ##
============================================
- Coverage     61.21%   60.39%   -0.82%
+ Complexity      490      487       -3
============================================
  Files          1090     1090
  Lines         43944    43901      -43
  Branches       6418     6387      -31
============================================
- Hits          26902    26516     -386
- Misses        14062    14404     +342
- Partials       2980     2981       +1

Impacted Files
Coverage Δ

...in/java/org/apache/dubbo/rpc/support/RpcUtils.java
62.69% <66.66%> (-0.21%)
:arrow_down:

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-73.34%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
25.71% <0.00%> (-60.01%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-57.15%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-28.58%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-17.65%)
:arrow_down:

... and 64 more

Continue to review full report at Codecov.

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


feat!: update libp2p interfaces

BREAKING CHANGE: uses new single-issue libp2p interface modules

github actions failed (no canary release on npm)
https://github.com/libp2p/js-libp2p/runs/6905044145?check_suite_focus=true

@libp2p/0.37.3-d4dd664 has been published


Prevent 'JSON parse error' exceptions

The current implementation in js/dom7-ajax.js assumes that a server would always return a valid json to be parsed when the request completes. This line, however, would break if that is not the case. For example, it would throw a SyntaxError, if the response is 200 OK and there is no data, i.e., responseData is an empty string.
Now, I know this really is a problem of the faulty server (my faulty server in this case) and not the F7 lib, but I just wondered whether this should be handled more robustly. For example, at least check if responseData is an empty string and return null on options.success? An app can then make further verifications or present an error to the user.

:+1:  Works fine, error event is dispatched.

Is this actually resolved? When I look at the latest code, the fix is not in there.
The problem seems to be this commit: https://github.com/nolimits4web/Framework7/blob/6fe9a6864980af35898c633d780ca6281fcf8ed5/src/js/dom7-ajax.js
In the commit right before the one above, the fix is still there:
https://github.com/nolimits4web/Framework7/blob/daf8743773770ba8b401581bf29f0ae0f05c6495/src/js/dom7-ajax.js
Did it accidentally get lost by merging or something like that?

@felixarntz seems to be you are looking in wrong place. It is fixed and it is still there, here is the latest version https://github.com/nolimits4web/Framework7/blob/master/src/js/dom7-ajax.js#L96

Yes, it is fixed and still there https://github.com/nolimits4web/Framework7/blob/master/src/js/dom7-ajax.js#L207


Config.yml help

I cannot figure out how to set the host in config.yml. This is my first project within Docker. Can someone point me in the direction of the file so I can edit it? I've been all over google, but have just been confused worse.

I was able to figure out how to use vi properly. NVM thank you.


[JOSS]: Pytest execution instructions

To run tests, the docs say to run the following command:
coverage run -m pytest

However, I received the following error and had to pip install coverage.
$ coverage run -m pytest
coverage: command not found

Please either create a desperate developer install version of your package (with optional dependencies) or update instructions to include the coverage requirement (if this issue can be replicated by your development team).

We have changed the building and dependency management to poetry in #320, now also including test and development dependency groups - does this make sense to you?

Makes sense to me! Thanks @JeanWi!


OTLP exporters shouldn't use window as it won't work in web workers

What happened?
Steps to Reproduce
Try to use @opentelemetry/exporter-trace-otlp-http in a web worker.
Expected Result
Otel should work in a web worker
Actual Result
An error is thrown with window is not defined

Additional Details
OpenTelemetry Setup Code
No response
package.json
No response
Relevant log output
No response

We should run OTLP exporter tests in worker environments just like we do in packages like sdk-trace-base

Looking at the code for how the exporter base "uses" the window to listen for the unload event (it should be listening for more and it probably should not be just shutting down as there may be additional events (spans) that are created as part of shutdown that still need to be sent).
Having said that though, as mentioned in the SIG, because of the way this is currently coded this would be a feature request change and at the very least the "base" should be changed to "check" for the existence of window and the add/remove EventListener

There is a JS lib, to move JS at the Browser to WebWorker.
partytown

Partytown is a lazy-loaded library to help relocate resource intensive scripts into a web worker, and off of the main thread. Its goal is to help speed up sites by dedicating the main thread to your code, and offloading third-party scripts to a web worker.

This could a way around.
Regards, Sascha

I was able to get this working in WebWorkers by simply changing the two instances of 'window' in OTLPExporterBrowserBase to 'globalThis'.
I haven't tested this elsewhere, but this should work in pretty much any environment since early 2019.

This was fixed by #4157 and then the handlers were removed completely in #4438.
Please let me know if there's any issues still and I'll re-open


Add Deep Dreamer.app latest version

The easiest way to process images and video with Google’s amazing Deepdream engine.

Thank you for the contribution. It was merged directly as e61214c1e64ead9ee1cc26f6a787c9d79c0957e4 to keep commit history cleaner. Your contribution is still credited to you.

:+1:


feat(translator): prefix TCP FilterChain name by irListener name

What this PR does / why we need it:
Prefix TCP FilterChain name by irListener name, to let extension server can locate TCP FilterChain according to policy's targetRef(s).
xref https://github.com/envoyproxy/gateway/issues/3461#issuecomment-2146544345

No strong opinion on the filterchain name, but I think extension server should not make assumption on the structure of the filterchain name or any other names in the xDS. For example, assume the filterchain name will be in the form of "gateway name/listener name/routetype/route name".
This is just an implementation detail of the xDS translation. It's not part of the EG API. There's no guarantee that it won't change over time.
For EnvoyPatchPolicy, it's ok to use these names to find a specified field in the generated xDS, since it's made clear in the EG docs that EnvoyPatchPolicy' is unstable and the outcome may change across versions.

You are correct, this proposal has coupled EG with Extension Server on very unreliable contracts. Perhaps we need to brainstorm other solutions to let Extension Server can detect which FilterChain should be targeted by policies.
Here is a new proposal by @modatwork https://github.com/envoyproxy/gateway/issues/3461#issuecomment-2213410511

This is just an implementation detail of the xDS translation. It's not part of the EG API. There's no guarantee that it won't change over time.

+1.
However, there is currently no stable method for the extension server to manage the SectionName within the policy API. Can we include this PR in the next release? It’s significant for extensibility.

can you instead update the IR name https://github.com/envoyproxy/gateway/blob/04c5ce230171ceea261eda354b14538675143614/internal/gatewayapi/helpers.go#L387 ?

can you instead update the IR name
https://github.com/envoyproxy/gateway/blob/04c5ce230171ceea261eda354b14538675143614/internal/gatewayapi/helpers.go#L387
?

Sure, I can do this since each new ir.TCPRoute or ir.UDPRoute is being attached to unique irListener.


Going upwards against the gravity when avatar is falling

Describe the bug
The avatar goes upwards instead of falling
To Reproduce

Go to any scene in the app
Hit F to fly
Hit shift + w + w to dash
While you're dashing hold space
Hit F again to go to normal mode + Stop dashing + Stop holding space bar
You can see that the avatar goes upwards for a while and then after some time gravity pulls it down

Expected behavior
The avatar should have another animation for this state, or it should simply go down ( This needs more thinking )
Screenshots
https://user-images.githubusercontent.com/64514807/183253855-84335916-a1b0-41f5-813a-6b72a6ffeb78.mp4

Hello @aryasalehi , could please you check if this issue still exists in labs/app?
I haven't worked on solving it dedicatedly, but looks know hasn't this issue in labs/app, maybe fixed in another pr as a side effect.


add token counting to enable cost dashboard

add support for lighting up the cost dashboard for the RAG example. notably, when using the OpenAI stream completion API, OpenAI no longer returns token usage data, thus it needs to be computed using the tiktoken lib.
on an unrelated (?) note, the RAG app wasn't returning any results when querying "who is batman?". this happens even without my code additions, so i don't think it is related..? the simple LLM example above the full RAG example does work...?

If you increase the number of documents retrieved you should start to see results.
sometimes, returning just one document or chunk doesn't provide enough context for the LLM to answer


Add imagegpt image processor

What does this PR do?
Add imagegpt image processor

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


added Garbage Collection Call

remove both duplicates while hashing
barely anything changed but finding the bug and correct place for placing the .gc() took way too long

Is this improving memory usage while keeping a similar performance? I've read that manually calling System.gc() isnt a best practice. But of course, it could be a fix for us in this case.

manual Garbage collection was again removed. It significantly increases computation time:
20 loops + GC:

although memory is constant, it takes 2min for the calculation.
20 loops - GC:

Calculation Time is down to under a minute, memory does ocasionally spike.
100 loops + GC:

10 minutes, memory is constant
100 loops - GC:

memory usage is bumpy and Inconsistent. After calculations finished memory usage was still high until UI interaction triggered automatic GC.

memory inconsistent, Huge speedup.

closes #263


Added 18.2 Erlang version

Erlang/OTP 18.2 has been released

http://www.erlang.org/news/97

You can also add 18.2.1 now: http://www.erlang.org/download_release/32


Refactor delete logic

https://github.com/rancher/longhorn/issues/273

Please separate the change into different commits, each for one purpose.

@yasker Ready for next review

LGTM. Waiting for the integration test result.

Can you change the commit name to controller: volume: refactor engines/replicas deletion process and add the issue URL to it?

Kubernetes v1.10 test suite works for me in general. OK to merge.
We need to check the issue with v1.11 though.

Merged too soon. I've reverted it.
Please update the commit title/comment and squash the commit.


[FLINK-25774][network] Restrict the maximum number of buffers can be used per result partition for sort-shuffle

What is the purpose of the change
Currently, for sort-shuffle, the maximum number of buffers can be used per result partition is Integer.MAX_VALUE. However, if too many buffers are taken by one result partition, other result partitions and input gates may spend too much time waiting for buffers which can influence performance. This ticket aims to restrict the maximum number of buffers can be used per result partition and the selected value is an empirical one based on the TPC-DS test results.
Brief change log

Use Math.max(min, 4 * numSubpartitions) as the maximum number of buffers can be used per result partition for sort-shuffle.

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

Rebased master.


statuses.md doc update and rephrasing

Rephrase a bit some of the sentences and add some clarifications.
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

/test pipeline-as-code-doc-generation

/cancel pipeline-as-code-doc-generation

/test pipeline-as-code-doc-generation

/cancel pipeline-as-code-doc-generation

/retest

/cancel pipeline-as-code-on-pull-request

/cancel

/retest
cancel seems to be working again, please try out again

Codecov Report

Merging #1078 (354b8e6) into main (7c9b164) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1078      +/-   ##
==========================================
- Coverage   63.14%   63.10%   -0.05%
==========================================
  Files         103      103
  Lines        6274     6269       -5
==========================================
- Hits         3962     3956       -6
- Misses       1929     1931       +2
+ Partials      383      382       -1

Impacted Files
Coverage Δ

pkg/sort/repository_status.go
75.00% <0.00%> (ø)

pkg/cmd/tknpac/completion/completion.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

/cancel pipeline-as-code-doc-generation

/cancel pipeline-as-code-doc-generation

okay yeah tried a few things and could not reproduce it  :( i guess we can merge and close related issue


Test case for overlapping reference

The following should all produce the same result
Genesis 1:1-3, 3-5
Genesis 1:1-5
Genesis 1:1-3, 1:3-5
Genesis 1:1-3, Genesis 1:3-5

As usual, the test case doesn't have to pass yet

Unassigned


Implement _check

*making changes as part of the contribute to open source repository.

changes committed is an update to the package-lock.json file

Hi Dan Thareja,
thank you very much for the feedback!
this is my first attempt for a pull request and it is a bit nerve-wracking.
will go through the comments that you have and re quest another pull once i get everything sorted out

thank you very much Dan Thareja!
very useful repo for beginners to build more confident to start open source :D


[WIP] Update to Spark 3.3.0

Description
WIP
Upgrade the Spark dependency version to 3.3.0. The code is compiled and ran sanity tests.
Fixes delta-io/delta#1217

Please update the title and description.

By this being closed without merge does this mean it's not getting in or is this covered by a different PR?

It has been merge and there is a Delta 2.1 Preview on Spark 3.3 currently undergoing community testing - https://github.com/delta-io/delta/releases/tag/v2.1.0rc1
We are hoping to make the final release of 2.1 early next week.


Typescript table.grantReadWriteData cannot grant lambda handler permission

:bug: Bug Report
What is the problem?

On page https://cdkworkshop.com/20-typescript/40-hit-counter/600-permissions.html
I tried to grant lambda access to read/write to dynamoDB, but i get an error message shown below:
Reproduction Steps

Verbose Log

TS2345: Argument of type 'Function' is not assignable to parameter of type 'IGrantable'.
The types of 'grantPrincipal.addToPolicy' are incompatible between these types.
Type '(statement: import("/home/ec2-user/environment/cdk-workshop/node_modules/@aws-cdk/aws-iam/lib/policy-statement").PolicyStatement) => boolean' is not assignable to type '(statement: import("/home/ec2-user/environment/cdk-workshop/node_modules/@aws-cdk/aws-dynamodb/node_modules/@aws-cdk/aws-iam/lib/policy-statement").PolicyStatement) => boolean'.
Types of parameters 'statement' and 'statement' are incompatible.
Type 'import("/home/ec2-user/environment/cdk-workshop/node_modules/@aws-cdk/aws-dynamodb/node_modules/@aws-cdk/aws-iam/lib/policy-statement").PolicyStatement' is not assignable to type 'import("/home/ec2-user/environment/cdk-workshop/node_modules/@aws-cdk/aws-iam/lib/policy-statement").PolicyStatement'.
Types have separate declarations of a private property 'action'.
33       table.grantReadWriteData(this.handler);
Environment

CDK CLI Version:  1.89
Section:  https://cdkworkshop.com/20-typescript/40-hit-counter/600-permissions.html
Browser:  Chrome
Language:  Typescript

Other information

also experiencing this issue with CDK CLI 1.94.1

I had the same issue and fount this issue which was causing the issue. I had mismatched sub-dependancies in my package.json.
Once that was cleaned up and I ran npm install again I got further in the demo.

This is due to mismatched versions in package.json, you got it right!


Documentation for how to check webhook signature in Rails needed

for my Nexmo Voice API webhooks, the documentation is not explain adequately how to check the webhook signature. It says to do client.signature.check(request.GET), but request.GET returns an empty hash, probably because my webhooks are sent as POST requests.
I tried using request.POST but that didn't work either. There are some extra things in there I'm guessing we don't want returned in request.POST, but more importantly, there is no sig parameter, so there is no signature that would be extracted at https://github.com/Vonage/vonage-ruby-sdk/blob/main/lib/vonage/signature.rb#L33
Could the docs be updated to show how to correctly check a signature in Rails?

This functionality has been added in https://github.com/Vonage/vonage-ruby-sdk/releases/tag/v7.18.0 and usage is now documented in the README here


The platform of the target sherpawatchapp Extension (watchOS 2.0) is not compatible with SwiftHTTP (1.0.2), which does not support watchos.

Want to use it on watchos 2
pods shows:
The platform of the target sherpawatchapp Extension (watchOS 2.0) is not compatible with SwiftHTTP (1.0.2), which does not support watchos.
now i got Target 'SwiftHTTP' of project 'Pods' was rejected as an implicit dependency for 'SwiftHTTP.framework' because it doesn't contain platform 'watchsimulator' in its SUPPORTED_PLATFORMS 'iphonesimulator, iphoneos'

I haven't tried building SwiftHTTP in a watchOS project yet. I will try to do that soon, pull request welcomed.

1.0.3 was released which should resolve this. Any questions let us know.

thanks

Понедельник, 18 января 2016, 13:18 -08:00 от Dalton<EMAIL_ADDRESS>Closed  #195 .
—
Reply to this email directly or  view it on GitHub .


Updating condor_vault_storer

Latest from @DrDaveD .   Implements #544 .

Still waiting for integration tests to complete...

Okay, integration tests look good.   There is a problem with test_condor_submit_dag1() in the integration tests (start job keeps going held due to system periodic hold) , but it also happens in the master branch without these changes, so I don't think this causes it.

Note that the issue @marcmengel is referring to he fixes in #564. (Marc, please correct me if I'm mistaken about that).


Fix some issues that were causing build failures/warnings with the CI workflow

@nuclearsandwich the warning did in fact say "unpickable" although likely it does mean unpicklable

@nuclearsandwich the warning did in fact say "unpickable" although likely it does mean unpicklable

lol. There's a chance to make one PR yet to fix the message upstream.


meme command

also english i18n, you can add other language commits to this pr

welp that was too many commits
oh i guess that makes sense though
ah whatever


Markup class for strikethrough/underline

Should I create a Markup  ( as discussed in this comment) with the option to select between strikethrough, underline, or any other similar augmentation ?

I think it's a great idea.
If we could control which text in the image receives the effect, even better

Closed with merge of PR #101


Change -overlayable-shadow-borderRegion to overlayable-shadow-region

Deprecate -overlayable-shadow-borderRegion add -overlayable-shadow-region to Overlayable characteristic (simplified nomenclature).

Issue resolved as part of #2571


Code Style (bis)

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/Util/Plateform.php  7

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/Util/Plateform.php  7

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/Util/Plateform.php  7

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/Util/Plateform.php  7

See the complete overview on Codacy


Last update e0760ea...f3e25a0


Issues with @spec! on auto-generated type like TypedEctoSchema

Hi there,
TypeCheck looks really interesting, but I'm afraid I'm having some issues when trying to make use of it.
I'm trying to use it together with TypedEctoSchema, but I'm getting compile errors. Could this possibly be because TypeCheck "runs before" TypedEctoSchema" gets the chance to generate the typespecs? Any workaround available?
Actual behaviour
I get a function Xxx.User.t/0 is undefined or private' error. when using TypeCheck together with TypedEctoSchema
Expected behaviour
TypeCheck should work with auto-generated typespecs like those coming from TypedEctoSchema.
Example code
defmodule Xxx.Accounts do
  use TypeCheck
  alias Xxx.User

  @spec! get_user_by_email(String.t()) :: User.t() | nil
  def get_user_by_email(email) do
    Repo.get_by(User, email: email)
  end
end

defmodule Xxx.User do
  use TypedEctoSchema
  import Ecto.Changeset

  typed_schema "user" do
    field :email, :string
    ...
  end
end

Hi there!
TypeCheck's checking functionality requires functions that return TypeCheck structs to exist as well as specs. This is necessary because these structs capture a lot of metadata that cannot be found in plain specs by themselves. TypeCheck.External exists which tries to turn plain specs into TypeCheck structs, but it by necessity works on a 'best-effort' basis, and it cannot be combined with the type!/ spec! macros (that run at compile-time); you'll have to use either TypeCheck.dynamic_conforms! or TypeCheck.External.enforce_spec! inside your function bodies.
You're not the first person who'd like to integrat TypeCheck and TypedEctoSchema. There is a work-in-progress PR on the TypedEctoSchema repo to enable the automatic creation of TypeCheck types from a call to typed_schema.
Until this PR is finished, the following workarounds exist:

Write the types in your Ecto's struct module out manually using type! together with @autogen_typespecs: false. This will give you all of TypeCheck's functionality, but it obviously goes against the principle of TypedEctoSchema doing the typespec-generation for you.
Fetch the specs generated by TypedEctoSchema using the functions in TypeCheck.External and using TypeCheck.dynamic_conforms! or TypeCheck.External.enforce_spec! inside your functions to check against them.

That's awesome! Thanks a bunch for the in-depth reply @Qqwy.
I checked for issues in the TypeCheck repo, but failed to find any.
I should of course have looked in the TypedEctoSchema repo as well, and I would have found the PR/Issue, my bad.
I'll close this issue hoping that it's OK, since there's already a PR in progress in the TypedEctoSchema repo.


No USB device found

pixel_ring.py is not finding my mic array even tho it is connected and shows up with the correct ID.
[root@raspberry ~]# python pixel_ring.py
No USB device found
[root@raspberry ~]# lsusb
Bus 001 Device 004: ID 04d9:1702 Holtek Semiconductor, Inc. Keyboard LKS02
Bus 001 Device 005: ID 2886:0007
Bus 001 Device 003: ID 0424:ec00 Standard Microsystems Corp. SMSC9512/9514 Fast Ethernet Adapter
Bus 001 Device 002: ID 0424:9514 Standard Microsystems Corp. SMC9514 Hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
dmesg output
[  261.084005] usb 1-1.2: new full-speed USB device number 5 using dwc2
[  261.173526] usb 1-1.2: not running at top speed; connect to a high speed hub
[  261.175199] usb 1-1.2: config 1 interface 4 altsetting 0 has 2 endpoint descriptors, different from the interface descriptor's value: 1
[  261.176926] usb 1-1.2: config 1 interface 4 altsetting 0 has 2 endpoint descriptors, different from the interface descriptor's value: 1
[  261.178785] usb 1-1.2: New USB device found, idVendor=2886, idProduct=0007
[  261.179153] usb 1-1.2: New USB device strings: Mfr=1, Product=3, SerialNumber=0
[  261.179474] usb 1-1.2: Product: ReSpeaker MicArray UAC2.0
[  261.179716] usb 1-1.2: Manufacturer: SeeedStudio
[  261.189399] hid-generic 0003:2886:0007.0003: hidraw2: USB HID v1.10 Device [SeeedStudio ReSpeaker MicArray UAC2.0] on usb-3f980000.usb-1.2/input4
[  263.534816] usbcore: registered new interface driver snd-usb-audio

hello, have you solved this issue? looking for a solution @sipvoip

Needed to switch the firmware.

@sipvoip which firmware did you use to get it work?


GDScript: Add support for static method calls in native types

Support in ClassDB was added in #59314. This PR makes GDScript aware of those, making them possible to call and showing those on the code completion list.

Thanks!

This seems to cause a regression with singleton function calls.
In ac591d9904e4e5cf7841b3e79caabf558d37db0e, Hello world! This is 0x55dfddf65620. is printed.
In 4d0fdf2e98be631e90b8320d10d842a042e31c8a, Hello world! This is (nil). is printed.
Full code (in a module named modtest): https://gist.github.com/nathanfranke/6e61df0029263afe09f3578d883a2e63
TL;DR:
func _ready() -> void:
	MySingleton.execute()

void MySingleton::execute() {
	printf("Hello world! This is %p.\n", this);
}


Override ansi-to-hml default background color

Simple patch for nicer output in console.
Obviously you could configure the ANSI-to-HTML package see here to do this.

 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.

I think this patch could be directly forwarded to drone/drone-ui.


[Bug] Update-DbaServiceAccount does not update SSL Cert Permz

If you have a server with an SSL cert ala Set-DbaNetworkCertificate, Update-DbaServiceAccount will fail. It needs to do that at some point, just dont know when.

jm2c: if you try and change the service account from the sql server configuration, even "it" doesn't set the relevant perms on the private key, if they were specifically granted to the previous user.
So it's a "bug" in WMI.
TBF, IMHO what usually happens at least in recent mssql setups is that the setup is happy to place the service user where is needed using the well known (but virtual) NT SERVICE\MSSQLSERVER here and there (as in local policies --> perform volume maintenance tasks) and then when someone changes the actual service account it doesn't need to remove the old and add the new ....
I "stole" that idea for the certificates, too. Instead of granting access to the key to the specific service account, I use the virtual account in the ACLs. Sure enough, no more "whoops, I forgot to put the new service account in the private key ACL": and, also, no more "whoops, some OLD service account can still access the private key" because I added the new one in a rush but forgot to remove the old one.
Aside from this trick that may be useful for people running into this issue (use the virtual account on the ACL and forget any further problems), I'd say that Update-DbaServiceAccount "should" grant the read ACL on the cert once it's sure the service account has been changed. Maybe issue a warning that the old ACL can/should be removed, after restarting the service.

@potatoqualitee - What do you think?

oh, sure! that looks reasonable


All config environment variables are ignored

I run the mumble server with this docker compose file:
services:
  murmur:
    container_name: murmur
    image: mumblevoip/mumble-server:latest
    restart: always
    ports:
      - "49155:50051/tcp"
      - "64738:64738/tcp"
      - "64738:64738/udp"
    environment:
      MUMBLE_CONFIG_SERVER_PASSWORD: "mypassword"
      MUMBLE_CONFIG_WELCOME_TEXT: "My Welcome Text"
      MUMBLE_CONFIG_USERS: "8"
      MUMBLE_CONFIG_BANDWIDTH: "128000"

None of the settings seem to work. E.g. the log file shows:

Setting config "database" to: '/data/murmur.sqlite'
Setting config "ice" to: '"tcp -h <IP_ADDRESS> -p 6502"'
Setting config "welcometext" to: '"Welcome to this server, running the official Mumble Docker image.Enjoy your stay!"'
Setting config "port" to: '64738'
Setting config "users" to: '100'
Running Mumble server as uid=1000 gid=1000
"/data" has the following permissions set:
drwxr-xr-x, owner: "mumble" (UID: 1000), group: "users" (GID: 100)
Command run to start the service : /usr/bin/mumble-server -fg -ini /data/mumble_server_config.ini
Starting...

As you can see the welcome text is not what was configured.

@azlux do you have an idea? I don't have any experience with docker compose.
@BorisBrock could you try running the container manually to see if the result remains the same?

@Krzmbrzl Starting the container without Docker Compose actually works. In that case the environment variables are applied as expected.

Interesting. In that case my suspicion would be that for some reason docker compose doesn't set the environment variables as expected :thinking:

@BorisBrock Do you find your ENV into the output of docker inspect <container runn ID (can be found with docker ps | grep mumble)>

Shouldn't the environment variable be MUMBLE_CONFIG_WELCOMETEXT not MUMBLE_CONFIG_WELCOME_TEXT (note the removed space)?

Additional underscores in the variable names are ignored (see README)

Hello,
I tried a modification of @BorisBrock and the welcome message works fine, but I had issues with a volume mount and default channels.
When I try to bring the container up, I get a permission error of:
2022-10-06T02:25:12.742473486Z /entrypoint.sh: line 77: /data/mumble_server_config.ini: Permission denied
2022-10-06T02:25:20.061774974Z /entrypoint.sh: line 77: /data/mumble_server_config.ini: Permission denied

My host user has the ID of 1000 and I even specified the PUID/GUID in the compose file but it still makes everything as root.
So then for testing, I sudo chmod 777 data/mumble and everything starts to work but there are no other channels than root.
My compose file is shown below:
services:
  murmur:
    container_name: murmur
    image: mumblevoip/mumble-server:latest
    restart: unless-stopped
    ports:
      - "49155:50051/tcp"
      - "64738:64738/tcp"
      - "64738:64738/udp"
    environment:
       MUMBLE_CONFIG_SERVER_PASSWORD: "mypassword"
       MUMBLE_CONFIG_WELCOME_TEXT: "Howdy Ya'll This is a Mumble Server"
       MUMBLE_CONFIG_USERS: "8"
       MUMBLE_CONFIG_BANDWIDTH: "128000"
       MUMBLE_CONFIG_channelname: "lobby"
       MUMBLE_CONFIG_channelname: "hello"
       MUMBLE_CONFIG_defaultchannel: "2"
       PUID: "1000"
       GUID: "1000"
    volumes:
      - "./data/mumble:/data:rw"

but there are no other channels than root.

Yes that is to be expected. You can't add channels via the INI file. That's currently a limitation of the Mumble server config system and unrelated to the Docker image itself.
As for why everything is created as root, idk :shrug:

My host user has the ID of 1000 and I even specified the PUID/GUID in the compose file but it still makes everything as root.

Had the same Permission Denied issue. I don't know if the MUMBLE_UID/MUMBLE_GID environment variables work (I excluded them after realizing that). Since I already had my host user as 1000/1000, I set MUMBLE_CONFIG_UNAME as the same host username and the volume bind started working correctly. No chmod necessary.
Having a working MUMBLE_UID/MUMBLE_GID and being able to set a timezone, all using environment variables, would be a welcome change.

The UID and GID options are for building the image only. Are you aware of that?

The UID and GID options are for building the image only. Are you aware of that?

I was not. Then I assume the PUID/GUID options in the above compose file wouldn't work either?

I don't know what these options are and how they are supposed to work. What I can tell you is that afaik these options are not handled by our docker image. Whatever they do, they have to be handled elsewhere :eyes:

Closing this as resolved


chore: Fixed CP members alignment

Fixes #235

Description: The issue with the styling regarding competitive programming members appearing in a column rather than a row has been updated. The issue was directly linked with the component of Cp members itself, so I had to update it completely. It has been fixed now.
Checklist

[x] I've followed the Contributing guidelines provided in the repository.
[x] I've made the changes which were demanded in the linked issue.
[x] I've tested my code on a Chromium based browser.
[x] I've tested my code on Mozilla Firefox.
[x] My code gave a clean console on debugging. (no warnings/errors)

Testimonial

@abhishekrathi16 @himanshu007-creator Thanks for the contribution! 🤝 Don't forget to star this repo if you liked this project. ⭐


PR to show incomplete Surveys stats

This PR is for this issue. It aims to provide incomplete surveys information in Form List View.

runtests

Thanks for working on this @himanshusagar!  I've tested the code out, and it is too slow to be viable with this implementation.  In issue https://github.com/opendatakit/opendatakit/issues/1371, I see that it was already acknowledged that the solution may be too I/O intensive but should be investigated.  Thanks for following through with the investigation.  At this point, I think we should continue to discuss how to implement this.  I've added my thoughts to the issue and am happy to hear feedback.


Wrong log message?

https://github.com/bloodmc/GriefDefender/blob/e281231d874d78def944e748f6b49057a516b3db/sponge/src/main/java/com/griefdefender/GriefDefenderPlugin.java#L569
It says "GriefPrevention".

Thanks. This was already fixed in 2.x which is not open source.


estransport: honour request context cancellation

Abort retrying when the request context has been cancelled.
Fixes https://github.com/elastic/go-elasticsearch/issues/300

LGTM!

Thanks @Anaethelion. Will you take care of backporting this to 7.x?

@axw Will do today !


Do fewer calls to getRelativePathToDirectoryOrUrl when writing sourcemaps

Shaves about 35% off time to build tsc.js. We should do more profiling / optimization here as this function is currently the vast majority of time spent building the project

Didn't seem particularly slow - it does do some array allocations and string walking, but was simply being called too many times

Understated the magnitude here; time to run node lib\tsc.js -b -f src\tsc down from 55 seconds to 35 seconds

Didn't seem particularly slow - it does do some array allocations and string walking, but was simply being called too many times

Mmm, caching like this will definitely help in the common case where there's a ton of mappings for the same files, but if, for example, there's 1000 files each with only a few mappings (so lots of tiny projects, which is the logical extreme of optimizing a project this way), this won't help nearly as much. (May even hurt from increased memory pressure) Which is why I also want to understand why it was so slow itself. It looks like split (and slice) may be part(s) of the issue, since combined they take most of the function runtime. (0.9/1.2 ms) Those both correspond to new array allocations, which makes sense. We can likely reuse a single (set of) arrays for the internal calculations (since it just returns a string anyway) and remove those allocations. Naturally, it's good to keep the cache too, but that'd probably improve the worst case significantly.


Add Github Badges

Summary
Add SVGs to the static folder that can be used as Github Badges.
How was it tested?
Visually verified

LGTM.
For what it's worth: I'm tempted to kill the white version as it's less "on brand" than the other two.


Use clamav in tests when explicitly tagged

Before, clamav was used for malware scanning in all tests that interacted with the MalwareScanner class (either directly or indirectly).
While it is important and useful to test the entire stack in integration tests, clamav is a source of repeated flakiness across the test suite due to the latency it introduces (especially in CI).
This updates RSpec specs and Cucumber tests to mock a successful clamav response
unless they are explicitly tagged with clamav: true or @clamav.

Merging this in place of #4348.
If we see any flaky failures related to document uploads after this (that are tagged with clamav), we should adopt the approach taken in #4338.


Add Licenses to Repos

If we add a license to the monorepo does it cascade to all directories?
https://choosealicense.com/

Contracts: GNU GPLv3
Monorepo: MIT License

Relates to:
https://github.com/HausDAO/daohaus-monorepo/issues/116
https://github.com/HausDAO/daohaus-monorepo/issues/117

After the changes are merged verify the license is displaying at the top similar to


Support for Servers Behind Identity Aware Proxy

We host our rocket.chat instance behind "Identity aware proxy" on Google Cloud which validates a google login before redirecting to actual website. (https://cloud.google.com/iap)
However because there is an additional login step,desktop app can not validate the site as a valid rocket.chat instance.
Would it be possible to disable server validity check?
Would it possible to open a browser window to show actual page if a validation fails (so that we can login to identity aware proxy) ? Could be "test" button when press it show success message or open in browser page.
Client version:3.0.2
Server version:3.7.0

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Changed Ender Relic's rarity from Epic to Legendary

nice yellow epic accessory jani kekw

nice yellow epic accessory jani kekw

You can add Dwarven v2 if you want 👍

nice yellow epic accessory jani kekw

You can add Dwarven v2 if you want +1

s h u t


chore: improve error message for view database schema mismatch

Describe your changes
This PR addresses issue #3530, improving the error message displayed to the user by providing how to solve the problem.
Issue ticket number and link
Closes #3530
Checklist before requesting a review

[ ] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

Thanks!


Type colors section of typography docs not displaying properly

All of the text is black/dark gray in the example.

Good catch, thank you for pointing that out! We will update that example soon. In the meantime, the correct code is:

Here is some white colored text.

Here is some black colored text.

Here is some gray colored text.

Here is some green colored text.

Here is some blue colored text.

The demo is now fixed!

Thanks @jameskolce!


Ability to edit an assessment review

If you were to review an app assessment but later on want to change a property such as the business priority based on the other app assessments, we aught to be allowed to do so.

almost there - sadly the south/northbound dependencies are not populating so this story is still in progress

Tested.


Cheat (Two-handed) should be renamed.

Love the addition of the Two-Handed cheat, but I was honestly confused when I first saw it, before testing it. Maybe rename to Dual Wield Anything or something similar.

Yes, it is named that because of a slight limitation of how strings are handled in the game.

I'm curious as well cause I want to rename "All Guns in Solo" to just "All Guns". Wondering how to tweak strings.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-25-main" workflow

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

Questions ?
Please refer to the Backport tool documentation