How do you add .xcconfig files to a project?

I've read the tests and much of the source code, however I don't seem to be able to get .xcconfig files working. Looking at #62 they seem to be implemented.
My Cakefile has this line:
 project.all_configurations.configuration_file = "Base.xcconfig"

I'm expecting to see something set here:

As long as the string you set points to a Xcconfig file they should work.
Which version of xcake are you using and could you send your cake file so I can recreate

xcake 0.6.7 installed via bundler on OSX.
Cakefile:
# set_trace_func proc { |event, file, line, id, binding, classname|
#   printf "%8s %s:%-2d %10s %8s\n", event, file, line, id, classname
# }
application_for :ios, 8.0 do |target|
  target.name = "YourApp"
  target.all_configurations.configuration_file = "Base.xcconfig"
end

Base.xcconfig (in the same directory):
MY_USER_SETTING="EXAMPLE"

@therealbnut thanks, I'll see if I can recreate this and issue a bug fix.

Let me know if you want the Gemfile.lock or anything like that.

It's still WIP, but this branch seems to address the issue for me:
https://github.com/jcampbell05/xcake/compare/therealbnut-adjust-xcconfig
It seems strange that it ever worked, so it would be good if you could verify if the old code works works for you @jcampbell05. Also whether or not this fixes it for you :)
It's quite possible I've got some weirdness in my setup. I've done a lot of hacking around with xcodeproj in the past, it's possible I've somehow messed up my gems through my inexperience with ruby.
The issue seems to be two-fold, the xcconfig is added to the project, not the configuration.
- native_configuration_object.base_configuration_reference = xcconfig
+ build_configuration.base_configuration_reference = xcconfig

The test verified that, so it wasn't picked up:
- expect(@native_configurable).to receive(:base_configuration_reference=).with(xcconfig)
+ expect(@native_configuration).to receive(:base_configuration_reference=).with(xcconfig)

You may also be interested on that branch, I add this:
- project.instance_eval(file_contents)
+ project.instance_eval(file_contents, file_path)

It prints out nicer debugging info (the path of the Cakefile), when there's an error.

For the instance_eval help identify the line of the Cakefile when there is an error, I'll check out the PR and give it a go.

I'm not 100% sure what you meant. However, instance_eval takes these arguments: (code, [string identifying the file], [number for the first line]). The line argument defaults to 1, so I've left it off, the file string defaults to gibberish, so I've added the Cakefile path.

Okay great :)

Should be implemented in https://github.com/jcampbell05/xcake/pull/68


attempting to use environment variable in config file

I attempted to use an environment variable in the config file like this:
dir = "$ProgramData/MyServer/InfluxDB/meta"
and this:
dir = "%ProgramData%/MyServer/InfluxDB/meta"
It appears that there is no handling for such things. Is there some way I can dynamically set that path? I'm running on Windows 10 with build 0.11.0~n1457510435, branch master, commit 425ef2c32a74c25929a178632efa8342e0d4d810

InfluxDB will check for environment variables from the environment (not the config file) when the process starts up. If any environment variables match the general pattern of INFLUXDB_[section name]_[variable name], they'll override the equivalent variable in the config file. E.g.
cmd /C "set INFLUXDB_META_DIR=%ProgramData%/MyServer/InfluxDB/meta && influxd"

(disclaimer, I don't regularly work with Windows so that syntax is probably wrong.)
@BrannonKing Does that work for you?

This works great. The wal-dir was a little tricky: INFLUXDB_DATA_WAL_DIR. Change it to an underscore.

Yup, all dashes (-) in config variables are converted to underscores (_) for the environment variable.
I opened up https://github.com/influxdata/docs.influxdata.com/issues/304 to document the environment variables.


run-tests: add test-file and test-name parameters

This allow the user to run specific integration test cases.
Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

Coverage remained the same at 66.433% when pulling 08d76470e26ae5f062f507e3efbaa1b15e4c0fa5 on ffmancera:run_test_parameter into 5d383493f6c713cd251afa6d0b294df72ea55825 on nmstate:master.

Coverage remained the same at 66.433% when pulling 08d76470e26ae5f062f507e3efbaa1b15e4c0fa5 on ffmancera:run_test_parameter into 5d383493f6c713cd251afa6d0b294df72ea55825 on nmstate:master.

We can use --pytest-args='-k add_port_to_existing_bridge' to run an specific test case. This is not needed. Thanks @tyll!


<EMAIL_ADDRESS>fails

We are using ios-auto-provision-appstoreconnect with automatically using the latest version for our ios builds to provision certificates and it was working fine with version 0.1.4.
However, on 9th of Dec the step got upgraded to 0.1.5 and it started to fail in our builds:
failed to check if profile is valid: project uses containers that are missing from the provisioning profile: [iCloud.$(CFBundleIdentifier)]
When we changed back to 0.1.4 the build got fixed.
Would it be possible to look into this issue? Thanks in advance.

Sorry to hear about the issue, thanks for the report!
Can you please send us a link two both versions of a build? When were you downgrade and it works, and one where it fails on the latest. Thanks in advance.

The links are below:
0.1.4 build: https://app.bitrise.io/build/89e41dcc635453e1
0.1.5 build: https://app.bitrise.io/build/ce6291330637fb53
I did also report it via bitrise support, so potentially someone is already looking.

That's correct, we've created a task and it's already in progress. Thanks for the details!

We released a new version  (0.1.6) of the step, fixing the bug: https://github.com/bitrise-steplib/steps-ios-auto-provision-appstoreconnect/releases/tag/0.1.6 Make sure to use the latest. Let us know if you have any questions!

We've upgraded to the last step and now it works again, thank you!


MP FYPO:0006497?

I didn't find
FYPO:0006497 | abnormal plasma membrane phosphatidylserine distribution
under
FYPO:0000813 | abnormal plasma membrane organization

I'd like to say more specifically
loss of plasma membrane phosphatidylserine polarity
(phosphatidylserine does not accumulate at tips)
and
loss of plasma membrane phosphatidylserine at the non-growing tip

Ah, I did the PS distribution terms as "subcellular component". Should I recast them as process phenotypes?
If I do that for the existing terms, it would probably also make sense to put the PS-at-tips terms into a process-y form too. Maybe "abnormal PS distribution resulting in ..."?

I think  process might be better for these...?

done; a couple new grouping terms and one with a long clunky name (and some synonyms):
abnormal membrane lipid distribution FYPO:0006545
normal membrane lipid distribution FYPO:0006546
abnormal plasma membrane phosphatidylserine distribution resulting in decreased concentration at cell tip FYPO:0006547
edit file: 0ab85ebfdb47f588e6ee51b7253dd606f3aed609
release: 69afaaf4a994d38bd73f28c2b9e54db281344e04


feat: update the settings page

Jira: LF-5661
This presents a reworking of the Settings page - a more detailed description can be found on the ticket. It also introduces a notification badge that is present on the main page if settings have been customised.
Some comments about other considerations for future changes can be found on the ticket too.

I've also added the Accordion functionality as a single commit - this one allows one of the expanded panels to be open at once. If we don't like this behaviour we can just drop that commit


Add a 'user' prefix for usernames in templates

Using '[a-zA-Z0-9]{8}' expressions can occasionally generate
usernames beginning with a digit, which is often forbidden.

@bparees @knrc PTAL

lgtm.


feat(backend): connections endpoint

Changes proposed in this pull request

add /connections endpoint
update reference spec URL

Note: depends on change in ilp-pay (#355)
Context

fixes #353

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass

Wait for #363 to be merged.

Depends on

https://github.com/interledgerjs/interledgerjs/pull/290


Question 3: Historic Buildings Bond

Question title
Historic Buildings
Description (will display on homepage)
This bond would allow the state to borrow $10 million to restore historic buildings owned by the government or nonprofits.[3]
The gist
This resolution would allow the state to borrow $10 million to restore historic buildings owned by the government or nonprofits, if they can get at least 25% local matching funds.[3]
Ballot question (exact text of the question)
"An Act to Authorize a General Fund Bond Issue to Restore Historic Community Buildings, Do you favor a $10,000,000 bond issue to restore historic buildings owned by governmental and nonprofit organizations, with funds being issued contingent on a 25% local match requirement from either private or nonprofit sources?"[3]
Yes means
I am in favor of borrowing $10 million to restore historic buildings.
No means
I don't want to money to restore historic buildings.
Tell me more
The Historic Preservation Commission will decide who gets grants. Only buildings that aren't already eligible for existing historic rehabilitation programs would be eligible.[4]
This is a bond issue – if voters approve the bond, it means that the State of Maine is allowed to borrow money for the project described in the ballot question and has to pay it back over 10 years. Learn more about how bonds work.
Follow the money
Principal: $10 million
Interest: $2.75 million
Total cost: $12.75 million[3]
The money will be used to

Restore historic buildings owned by the government or nonprofits.[3]

The money will be coming from

$10 million bond issued by the state and paid back through the budget and taxes over 10 years.[3]
25% matching funds from local organizations.[3]

Pros
The primary arguments for this referendum are:

Historic buildings are expensive to restore and this would provide funds to governments and nonprofits who aren't likely to have funds themselves.
This would unlock matching funds.

Cons
The primary arguments against this referendum are:

This would add to Maine's bond debt.

Who cares
Nobody has registered support or opposition to this bond.
Further reading
Full text
Ballotpedia article
Press Herald Article
References
Put the number in brackets above to indicate a footnote to a reference.

Maine State Legislature. An Act to Authorize a General Fund Bond Issue to Restore Historic Community Buildings. Accessed October 5, 2024.
Ballotpedia State Desk. Maine Question 3, Historic Community Buildings Bond Issue (2024)
. Ballotpedia. Accessed October 5, 2024.
Bellows, Shenna. Maine Citizen’s Guide to the Referendum Election. Accessed October 5, 2024.
Billings, Randy. Question 3 asks voters to support $10 million bond for historic buildings

Image
https://commons.wikimedia.org/wiki/File:Lawerence_Public_Library,_Fairfield,_Maine.jpg
Tags
Bond, Buildings
Election date
November 5, 2024

This one is ready to review.

Me reviewed.


Annotations multiline support - Solution proposal

Hi all,
after a lot of research I found out that ApexCharts has changed the annotations to support multiline.
I tried all before: \n,  etcetera, but neither works. And the problem was: in the annotation (X or Y indiscriminately) the property text is use to hold string or array of strings if you want multiple lines to show.
Unfortunately in C# we don't have the discriminated union so we need to choose: object to handle everything and put a comment on it, or directly use a List as type.
I made the changes as List and it worked: samples of annotations still work as before, added a new one to handle two lines and works as expected.
Maybe, it could be easier to handle an object because this would affects the retro-compatibility of the library.
I'll make the changes and push the PR.
Thanks in advance for you attention

Hi,
Your PR has been accepted and merge into the main branch.
I will prepare a release in a couple of days.


VSCode 1.63.2 does not follow LSP Specification of 3.16

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: Not Relevant
OS Version: Not Relevant

The specification states: https://github.com/microsoft/language-server-protocol/blob/gh-pages/_specifications/specification-3-16.md#-diagnostic-
export interface Diagnostic {
	...
	/**
	 * The diagnostic's code, which might appear in the user interface.
	 */
	code?: integer | string;

Yet VSCode itself does something entirely out of spec:
https://github.com/microsoft/vscode/blob/1.63.2/src/vscode-dts/vscode.d.ts#L5466
	/**
	 * Represents a diagnostic, such as a compiler error or warning. Diagnostic objects
	 * are only valid in the scope of a file.
	 */
	export class Diagnostic {
		...
		/**
		 * A code or identifier for this diagnostic.
		 * Should be used for later processing, e.g. when providing {@link CodeActionContext code actions}.
		 */
		code?: string | number | {
			/**
			 * A code or identifier for this diagnostic.
			 * Should be used for later processing, e.g. when providing {@link CodeActionContext code actions}.
			 */
			value: string | number;

			/**
			 * A target URI to open with more information about the diagnostic error.
			 */
			target: Uri;
		};

Either VSCode should follow the specification or the specification should be adjusted to account for this

According to https://github.com/microsoft/language-server-protocol/issues/926 it should be already in 3.16 spec.

@IllusionMH thanks! I actually already reported it there https://github.com/microsoft/language-server-protocol/issues/1406
But they say VSCode can do whatever it wants, which is fine but it seems like from what you are saying and referencing that this should have been in the spec (but it's not).

Replied in that thread why I think if should be already documented.
Also - reply from rcjsuen is correct. VS Code can extend it's implementation to support additional responses ({value: string | number; target: Uri;}) while still maintaining full support for servers that follow spec 1:1 and only will provide plain code?: string | number; - VS Code supports this as well.

Either VSCode should follow the specification

I don't see problem where VS Code doesn't follow specification here - it still supports servers that implement LSP 3.16. And definitely doesn't look like a problem which is strong enough that VS Code would remove feature from editor.

that VS Code would remove feature from editor

I don't want that. I just wanted to know if VSCode should be using DiagnosticRelatedInformation which was created as a result of this anyways.

It was pointed out to me that this was actually solved in the intermediate library!
https://github.com/microsoft/vscode-languageserver-node/blob/292dd0d88b6465337e42ba40ad986fdfdeb42d4d/client/src/common/protocolConverter.ts


[BUG] Remove amount and schools header

Also this branch appears to be behind devel. Please rebase and run the compiler to see if it builds without errors / warnings.


fix: recursion error when copying blob folders

:tada: This PR is included in version 0.0.12 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How can i use instaloader with github actions cronjob?

Hello, i want to set a cronjob with github actions to run every 24 hours.
The problem is that when instaloader attempts the login, the build on github script worlkflow gets the following challenge:

Obviusly copying and pasting the url on my browser doesn't work.
Is there a solution?
P.S. I only need to use get_stories() and nothing else.

Hi,
A small question. Are you sure you need get_stories() and not download_story()?
I have written a variant for you to download the stories. You can enter the corresponding user with download. Username and ``password```should be clear.
import instaloader

# Create an Instaloader instance
L = instaloader.Instaloader()

# Log in to Instagram
download = "download user"
username = "your username"
password = "Password"
L.context.log("Logging in...")
L.context.session_id = None  # Force new session
L.interactive_login(username)

# Download stories of the logged-in user
profile = instaloader.Profile.from_username(L.context, username)
L.context.log("Downloading stories...")
for story in profile.get_stories():
    L.download_story(story, download)

Why do you want to do via a CronJob through GitHub. GitHub will abort the process at some point. If you really want to do a CronJob, you need a server or you use your PC for that.  Here is a short tutorial on how to make a cronjob.:

https://wiki.ubuntuusers.de/Cron

I hope I could help you. If you need more detailed help, you should give some more info.
From the current information I strongly advise you not to do it the way you plan.

Hello Moderator. (@aandergr) It seems that the author (@CandreaA) is no longer interested in the question. Can you please close the issue accordingly ?


Include guest bios and linux loader in mythril binary

Fixes #89
I don't think there is a lot of value in allowing a user to modify the guest bios or linux loader. They can still do so if they build mythril from source anyway, and the benefit here is that now the user can just provide the following multiboot modules:

the mythril binary
the configuration json
the guest kernels and initramfs's

Which seems more legible for a user.

This should be good to merge, right?

Yup!


Unexpected high sensor.de_jonge_held_energy_shares value

Checklist

[X] I'm running the latest version of the Windcentrale integration.
[X] I checked if there is not a duplicate issue with the same question.
[X] I have read the README.md page.
[X] This issue is a bug related to the Windcentrale integration.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] I have filled out the issue template to the best of my ability.

The problem
Sometimes I experience a very high value for this property, leading to the Energy dashboard showing a huge peak of production, while this did not happen.
It happened just today, while windcentrale is even having an error, reporting no energy production.
It can be fixed by adjusting the statistics via developer tools (note I screwed up making screenshots before fixing it, so this is a manual adjustment to show the same).

I am not sure if it's related to this log entry:
`Logger: homeassistant.components.sensor
Source: custom_components/windcentrale/sensor.py:235
Integration: Sensor (documentation, issues)
First occurred: 13:56:40 (2 occurrences)
Last logged: 13:56:40
Error adding entities for domain sensor with platform windcentrale
Error while setting up windcentrale platform for sensor
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/home/homeassistant/.homeassistant/custom_components/windcentrale/sensor.py", line 235, in async_added_to_hass
self.attr[week] = state.attributes[week]
KeyError: 'Week 0'
`
What is version of Home Assistant Core did you recieve the bug?
Home Assistant 2022.12.8 Frontend 20221213.1 - latest
What is version of Home Assistant Supervisor did you recieve the bug?
Home Assistant 2022.12.8 Frontend 20221213.1 - latest
What is version of Home Assistant Supervisor?
Home Assistant 2022.12.8 Frontend 20221213.1 - latest
What is version of the Windcentrale integration?
0.2.6
Example Code snippet
No response
Anything in the logs that might be useful for us?
Logger: homeassistant.components.sensor
Source: custom_components/windcentrale/sensor.py:235
Integration: Sensor (documentation, issues)
First occurred: 13:56:40 (2 occurrences)
Last logged: 13:56:40
Error adding entities for domain sensor with platform windcentrale
Error while setting up windcentrale platform for sensor
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/srv/homeassistant/lib/python3.10/site-packages/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/home/homeassistant/.homeassistant/custom_components/windcentrale/sensor.py", line 235, in async_added_to_hass
self.attr[week] = state.attributes[week]
KeyError: 'Week 0'
Additional information
I have seen it in total 3-4 times since I run the integration (about 2 months)

Let me know if you still have the problem after today

I have not since it since the API started working again. Will post it here when it happens again.

If you want to fix your energy dashboard I removed statistics from the database to fix this problem.
I made some instructions https://github.com/jobvk/Home-Assistant-Windcentrale/discussions/76


Support initializing the logger with a log level

Changes

allow caller to provide a log level threshold on initialization

logger = Twiglet::Logger.new('my service name', level: Logger::Warn)

the log level threshold can be overridden at any point with the instance method

logger.level= Logger::Debug

@simplybusiness/application-tooling

https://refactoring.guru/replace-magic-number-with-symbolic-constant
On Fri, 26 Jun 2020 at 15:38, Punit Jain<EMAIL_ADDRESS>wrote:

@punitcse commented on this pull request.
In test/logger_test.rb
https://github.com/simplybusiness/twiglet-ruby/pull/22#discussion_r446223957
:

@@ -305,6 +305,10 @@
assert_equal args[:level], @logger.level
end
end

it 'initializes the logger with the provided level' do

 assert_equal 2, Twiglet::Logger.new('petshop', level: :warn).level

Is it possible to replace 2 with logger constant? I am not very clear
looking at test why it is 2.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/simplybusiness/twiglet-ruby/pull/22#pullrequestreview-438350135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD5GXNC65PRN264JVKMJZ3RYSXFLANCNFSM4OJMD3IQ
.

https://refactoring.guru/replace-magic-number-with-symbolic-constant
…
On Fri, 26 Jun 2020 at 15:38, Punit Jain @.> wrote: @.* commented on this pull request. ------------------------------ In test/logger_test.rb <#22 (comment)> : > @@ -305,6 +305,10 @@ assert_equal args[:level], @logger.level end end + + it 'initializes the logger with the provided level' do + assert_equal 2, Twiglet::Logger.new('petshop', level: :warn).level Is it possible to replace 2 with logger constant? I am not very clear looking at test why it is 2. — You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub <#22 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD5GXNC65PRN264JVKMJZ3RYSXFLANCNFSM4OJMD3IQ .

So ruby Logger already has constant like Logger::WARN for 2


Implement dict as IndexMap<RefValue, RefValue> when key is not mutable

8d14573d36c82afca15ef7d8d04450cb2858c0de introduces Object::is_mutable().
This can be used to implement keys of dict as RefValue as well, instead of just String.
src/value/dict2.rs was already a test-facility on this:
//! Test stub for a Dict type that uses RefValue as key
/// This shall become standard in combination with the is_mutable() flag

use super::{BoxedObject, Object, RefValue};
use indexmap::IndexMap;

// Alias for the inner dict
type InnerDict = IndexMap<RefValue, RefValue>;

// Dict object type
#[derive(Debug, Clone, PartialEq)]
pub struct Dict {
    dict: InnerDict,
}

Things to do:

[ ] allow for creating dict with is_mutable() == false RefValues as keys
[ ] try to implement an indexing method using str still, so existing code especially in ast.rs doesn't need to be touched

Closed by #96


fix: add certstore access denied tsg

Also not related to this specific TSG but do you think we can add a _category.json file so the "troubleshoot" header is formatted correctly? https://github.com/deislabs/ratify-web/blob/main/docs/quickstarts/category.json
Currently all the docs under 'troubleshoot' are lower case

Also not related to this specific TSG but do you think we can add a _category_.json file so the "troubleshoot" header is formatted correctly? https://github.com/deislabs/ratify-web/blob/main/docs/quickstarts/category.json
Currently all the docs under 'troubleshoot' are lower case

I m also looking for docs on the "positon" metadata.. not sure why position should be 6.. https://docusaurus.io/docs/sidebar/autogenerated#category-item-metadata

i ve updated the titles for folder, ready for another round of review. @akashsinghal , thanks!


Error is now thrown when creating a network when any unknown options …

According to https://github.com/projectcalico/libnetwork-plugin/issues/65

 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.

Could you write a simple test to check for this?


Issue #112 method not allowed

Added handler for 405 in app/error/handlers.py and added 405 in err_map

@phongvdoan I forgot to have you add a test around this. Would you mind opening another PR so that has a unit test making a request to a method that's not allowed and verifies that a JSON response is returned?
You can check out an example of me doing something similar with 400 responses here https://github.com/OperationCode/resources_api/pull/123/files#diff-51f932e4d7e5be54d3b9078cf8db7be2R272


nvidia driver build failure on 5.14.1

Nvidia build failure on 5.14.1 kernel using pkgs.linuxPackages_latest.nvidia_x11
Numerous compiler errors while building:
make[3]: *** [/nix/store/ld479n8nmwsr7x3p2ayg70r527aaqhpc-linux-5.14.1-dev/lib/modules/5.14.1/source/scripts/Makefile.build:271: /build/kernel/nvidia/nv-dma.o] Error 1
In file included from /build/kernel/common/inc/nv-linux.h:25,
                 from /build/kernel/nvidia/nv-vm.c:13:
/build/kernel/common/inc/nv-time.h: In function 'nv_sleep_ms':
/build/kernel/common/inc/nv-time.h:217:18: error: 'struct task_struct' has no member named 'state'; did you mean '__state'?
  217 |         current->state = TASK_INTERRUPTIBLE;
      |                  ^~~~~
      |                  __state
make[3]: *** [/nix/store/ld479n8nmwsr7x3p2ayg70r527aaqhpc-linux-5.14.1-dev/lib/modules/5.14.1/source/scripts/Makefile.build:271: /build/kernel/nvidia/nv.o] Error 1
make[3]: *** [/nix/store/ld479n8nmwsr7x3p2ayg70r527aaqhpc-linux-5.14.1-dev/lib/modules/5.14.1/source/scripts/Makefile.build:271: /build/kernel/nvidia/nv-vm.o] Error 1
make[2]: *** [/nix/store/ld479n8nmwsr7x3p2ayg70r527aaqhpc-linux-5.14.1-dev/lib/modules/5.14.1/source/Makefile:1851: /build/kernel] Error 2

Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.13.12, NixOS, 21.05.2734.74d017edb67 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.15`
 - channels(root): `"nixos-21.05.3021.8b0b81dab17, home-manager-20.09, nixos-hardware, nixos-unstable-21.11pre313915.bbbe2b35f73"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This is usually the case when a new kernel release happens (for example, see https://github.com/NixOS/nixpkgs/issues/130130). You can either use an older kernel for a bit until it's fixed, e.g.
boot.kernelPackages = pkgs.linuxPackages_5_13;

…or you could try patching it. There seems to be a patch for 5.14 here if you're interested.

This is usually the case when a new kernel release happens (for example, see #130130). You can either use an older kernel for a bit until it's fixed, e.g.
boot.kernelPackages = pkgs.linuxPackages_5_13;

I don't like idea of version pinning (especially for kernel) - I'm stuck with old software and forced to maintain it (check newer version for build errors, etc) by myself. There is a bug and it should be fixed by a maintainer of derivative.
And as far as I know I can't choose different kernel derivative - e.g. linuxPackages_lte.

…or you could try patching it. There seems to be a patch for 5.14 here if you're interested.

...Or this could be done by a maintainer ;)

I've tried the patch on my NixOS 21.05 system and it seems to work OK. I used the following change:
diff --git a/pkgs/os-specific/linux/nvidia-x11/default.nix b/pkgs/os-specific/linux/nvidia-x11/default.nix
index 059fef24d83..15876636dc5 100644
--- a/pkgs/os-specific/linux/nvidia-x11/default.nix
+++ b/pkgs/os-specific/linux/nvidia-x11/default.nix
@@ -23,6 +23,14 @@ rec {
       sha256_64bit = "sha256-VdeuEEgn+qeel1Mh/itg+d1C+/9lZCBTRDwOVv20xH0=";
       settingsSha256 = "sha256-DJg5QbyuKJmPpLQVYgTLvucI1e9YgQOO16690VXIWvk=";
       persistencedSha256 = "sha256-Cqv6oUFnsSi3S1sjplJKeq9bI2pqgBXPPb11HOJSlDo=";
+      patches = [
+        (fetchpatch {
+          url = "https://raw.githubusercontent.com/Frogging-Family/nvidia-all/master/patches/kernel-5.14.patch";
+          sha256 = "042zyspddka1acmli9l4h4cbzpfgq8a9lcdwjgwjr0npdgqk3j3f";
+          stripLen = 2;
+          extraPrefix = "kernel/";
+        })
+        ];
     }
     else legacy_390;

Is there any way to apply this without having to use a custom nixpkgs checkout?

You might be able to do
{
  hardware.nvidia.package = pkgs.linuxPackages_latest.nvidia_x11.overrideAttrs (_: { patches = ...; });
}

Or maybe you'd need to use boot.kernelPatches /  pkgs.linuxPackages_latest.extend?
I realize the above isn't super detailed (sorry), and I unfortunately can't test any of it right now but I feel like one of those should work.

This issue has been mentioned on NixOS Discourse.  I will remove myself as maintainer from the package.

Should be sorted with the merge of fd8a7fd07da0f3fc0e27575891f45c2f88e5dd44. Please check and reopen if the problem remains.

Should be sorted with the merge of fd8a7fd. Please check and reopen if the problem remains.

Wait, it's fixed on that specific issue, right? So this problem isn't fixed globally and will occur on future releases.


Update docs to use ngSwitchCase instead of ngSwitchWhen

RC2 has added the ngSwitchCase directive to replace ngSwitchWhen: https://github.com/angular/angular/commit/e1fcab7
The docs should be updated to reflect this, especially since the API docs for ngSwitchWhen 404's: https://angular.io/docs/ts/latest/api/common/index/NgSwitchWhen-directive.html

There are a lot of more usage of that than what that search gives to you.
I will fix the cheatsheet myself since that is not that obvious to fix.

I am closing this in favor of #1710
Sorry, he didn't see it and pushed a PR.


Segfault when using fzy if the filename contains non-ASCII character(s)

If the directory contains, for example, the following name:
Edmund Husserl - Méditations cartésiennes - Introduction a la phénoménologie.pdf

and if I execute, for example:
ls -l | fzy

The program segfaults. (This é is the problem.)
Easily reproducible: just create a filename with the above name and run the above command.

Thanks for the report. This should be fixed by 7812c553d33ce105afbc7696fb9e01ca92fba2ea which I've released as version 0.7


[Bug]: Unable to open 'filname.blade.php'

Extension Version
0.1.12
PHP Binary
Local PHP
Operating System
macOS
What happened?
When you hover on view('filename') in controller, it is able to show the exact path but when you do
cmd + click, it says "Unable to open 'filname.blade.php'"
Mimimal Code Sample
return view('customers.list', compact('customers'));

Same error in the blade when you use @include()
Example:
@include('company.positions.tabs.applicants')


LEDS: adding docs

Commit Message: LEDS: adding docs
Additional Description: Followup of #18195, adding the release note and the general docs regarding the new LEDS API.
Risk Level: N/A - docs only.
Testing: N/A - docs only.
Docs Changes: Updated general docs with the new LEDS API.
Release Notes: Added to release notes.
Platform Specific Features: N/A.
Signed-off-by: Adi Suissa-Peleg<EMAIL_ADDRESS>

@adisuissa what's the status of this PR?

@adisuissa what's left to wrap this one?


[CELEBORN-1362] Remove unnecessary configuration celeborn.client.flink.inputGate.minMemory and celeborn.client.flink.resultPartition.minMemory

What changes were proposed in this pull request?
Remove unnecessary configuration celeborn.client.flink.inputGate.minMemory and celeborn.client.flink.resultPartition.minMemory.
Why are the changes needed?
celeborn.client.flink.inputGate.minMemory and celeborn.client.flink.resultPartition.minMemory are configured as min memory reserved at present. Meanwhile, celeborn.client.flink.inputGate.memory should be at least networkBufferSize * MIN_BUFFERS_PER_GATE bytes, and celeborn.client.flink.resultPartition.memory should be at least networkBufferSize * MIN_BUFFERS_PER_PARTITION bytes. Therefore, celeborn.client.flink.inputGate.minMemory and celeborn.client.flink.resultPartition.minMemory are unnecessary configuration for celeborn.client.flink.inputGate.memory and celeborn.client.flink.resultPartition.memory.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
PluginSideConfSuiteJ#testCoalesce

Ping @FMX, @RexXiong.

Ping @FMX, @RexXiong, @TanYuxin-tyx.

Ping @FMX, @RexXiong, @TanYuxin-tyx.


Autogeneration of getting started code

This PR solves #126.
I also made a fixture out of metadata creation from config file so that it can be used repeatedly. Let me know if this is not good.

Model card created from this branch, the code is automatically generated: https://huggingface.co/merve/test-code-generation

@BenjaminBossan I think it looks a bit ugly and our example already includes metadata from config file so that's why I wanted to remove it (it will be a duplication if we add that too).

Yes, I'm aware that in the example, it doesn't make sense. What I meant is to document it some other place so that users have a chance of discovering it (as is, the only way to discover the feature is by reading the template).
If we can assume, however, that the vast majority of users create the section automatically, then maybe it's not necessary.

@adrinjalali what do you suggest?

I think since w/o the config file people won't really would know what to do with the model file, it's safe to assume the config file is there, especially for an example in the skops library, which generates that file.

@adrinjalali done, new model card: https://huggingface.co/merve/test-code-generation


Add pybullet to requirements-dev.txt

After #371 the tests require pybullet.

imosci test this please

imosCI successfully created a job for commit 228d9cd:

Dashboard for commit 228d9cd

/test

/test


Kadane’s Algorithm

I have added the program to the appropriate REAMDE file.

I have added the program to the appropriate REAMDE file.

You do not need to open a new Pull Request just update in the current

Why you closed the PR ? @aayushjain7

Reopen please


Removing usage of TopScoreDocCollector + TopFieldCollector deprecated methods (#create, #createSharedManager)

Removing usage of TopScoreDocCollector + TopFieldCollector deprecated methods (#create, #createSharedManager)
Addresses #13499
Copying description from issue:
These methods were deprecated in https://github.com/apache/lucene/pull/240 which is part of Lucene 10.0.
Since they are not marked for deprecation in Lucene 9.x, they will not be removed with Lucene 10.0 release (unless we want to deprecate them in 9.x in another backport commit?), but I still think we should remove the internal usage so it will be quicker to remove the methods when the time comes.
This is a 1:1 change to remove the usages.
Test command:
./gradlew clean; ./gradlew tidy; ./gradlew check

heya @slow-J thanks for the PR, and sorry for the delay in reviewing it. I was cleaning things up in this area as well, and ended up making changes to TestTopDocsMerge that are causing merge conflicts in your PR. Besides that, I think that this is a good step forward and I am happy to review it and merge it. Could you perhaps remove the changes to TestTopDocsMerge that are no longer necessary?

Thanks @javanna! I fixed the conflicts in TestTopDocsMerge, removing the changes that are no longer needed.

Thanks @slow-J !

Thanks @javanna for the review!


Volume page issues

[x] 1. If the volume is created without any content being populated, an error occurs as below - Validated

[x] 2. State inconsistent on list page and detail page - Validated

[x] 3. On the volume edit page, it is not necessary to have any field to be editable - Refactored

[x] 4. Refresh the volume edit page, the page will lose the image selection - Validated

[x] 5. Unable to add labels and annotations on volume edit page - Validated

[x] 6. Namespace highlight is needed? - Still being highlighted

[x] 7. Clone a volume which has warnings or errors, the create page will have that warnings or errors as well - Cannot see any warnings or errors now

Validated


[QRZ] Callbook Lookup

QRZ returns callbook information in this manner
Search for CALL/P
if /P does not exist return just info for CALL
if /P exist return this data
To escape the slash fixes this issue (referred in #452 )
Problem: We need to discuss if ....

We just update all fields even if we know that things like Location (Grid) are not valid

or

We just update certain fields in QSO logging if the callsign contains a suffix or prefix

In both cases the user needs to check the data by hand. Imho we can use 1. option but we can discuss this and I'm open for other arguments.
This also affects #459  and #515. For this reason we should implement the decision in a central place (in the QRZ library functions)

did the same for HamQTH to be complete

there is some more to do here. QRX


can it supports multi variables?

cross-env foo=bar fo=ba webpack
According to the source code,I think this will get an error,right?

Yes it does support multiple variables in a single command

Why not show it in the documentation?
And I will have a try for that functionality.

Feel free to makeapullrequest.com to add it to the docs :)

Hi. What's the proper way to set multiple variables? cross-env foo=bar fo=ba webpack or cross-env foo=bar cross-env fo=ba webpack . Thanks in advance

The first is correct


[Feature Request] Copying files from library

In view of the compilation time, is it better to provide an option to simply copy the code of the libraries and then executing with python37.dll? Many libraries are already optimized and sacrificing the compilation time may not worth it.

I don't understand what you are asking here.

Sorry, what I mean is, instead of compiling the libraries, it may be a better option to simply follow and copy some of the packages to the root directory of the programme and using python.dll to execute in the standalone mode. For many libraries, the execution time may be dependent on other factors such as network and the c-backend. This can reduce the compilation time significantly, without sacrificing much performance.

You can do this by compiling what you care about as an extension module, and then use another tool to deploy. Right now I don't see that happening for lack of manpower. Also I don't really see the point, to me it's more important to make compilation time for 3rd party modules a non-issue, rather than avoiding it.
Not sure if it's helpful to keep this open, unless you are willing to implement it, are you?


Fix race condition on concurrent builds

We encountered a race condition when running two buildkite agents on the same host.
We had two builds, build A and build  B running concurrently:

Build A completed building the image and tagging it with {ecr_name}, as no version was specified it defaulted to latest.
Build A then initiated a remote call to the AWS ECR API.
Meanwhile build B completes building it's image and also tags it with {ecr_name}:latest
When both builds A and B come to setting the remote tags, they are now both using the image created by build B

This pull request fixes the above issue by using the build number as the version on the local tag.

👍 this just happened to us and deployed a broken application

Thank you for the PR, The Seek Platform team will review it and proceed accordingly.

This is a good change and passes our internal tests. Thanks for the PR and sorry for the delay - we'll work on our internal processes.


feat: improve readme

What has changed:
Improved the readme file to be more focused on our project
Preview

This preview seems to be different from the PR’s changes

This preview seems to be different from the PR’s changes

You are right changed


Update README.md

Fixing instructions from "docker compse" to "docker-compse"

Fixing instructions from "docker compse" to "docker-compse"

docker-compose is deprecated
https://docs.docker.com/compose/migrate/


Dimensionless Units

Work for dimensionless units. Will close #54

For developers: the unitless mapping table is read when the CMORizer object get's initialised (either directly, this is uncommon) or via the classmethod from_dict (this is the common case), and attached to all rules that the CMORizer knows about. You can access them on the rule object as attribute dimensionless_unit_mappings.

@pgierz , nice to see the dimensionless_unit_mappings set on rule. I guess unit conversion function should looked out for this attribute and act accordingly. If so, I will make necessary changes to that function. One follow up question: Should the units in the final output point to units in the CMIP table or the one used in this unit_mapping?

I guess the unit representation in the final output should still be "0.001" as suggested in the table. Right?

Correct. At the end of the CMOR process, we need to have metadata attributes as defined in the CMOR Tables.

Hi guys, just so I make sure I am following, this are the steps and tooling needed correct?

read the dimensionless_unit_mappings and assign them to their corresponding CMOR variables. @pgierz this is also what you are saying here as well right?

the unitless mapping table is read when the CMORizer object gets initialised (either directly, this is uncommon) or via the classmethod from_dict (this is the common case), and attached to all rules that the CMORizer knows about

Prepare raise errors if:
a. The user has not specified a "pint-understandable" unit and the original unit in the file doesn't make any sense to pint
b. One of the rules is trying to convert to a unit in the CMOR table that has no mapping to a pint-meaningfull physical variable

Modify the unit conversion to get the attribute from CMOR table where we stored the physically meaningfull unit (g/kg) and use that to do the conversion.

Any step missing?
I'd suggest we start writing the dimensionless_unit_mappings yaml with just one variable to test, for example one of the salinities that @siligam have shown from his tables.

read the dimensionless_unit_mappings and assign them to their corresponding CMOR variables. @pgierz this is also what you are saying here as well right?

the unitless mapping table is read when the CMORizer object gets initialised (either directly, this is uncommon) or via the classmethod from_dict (this is the common case), and attached to all rules that the CMORizer knows about

Partially. The mappings are read and all are assigned to each Rule: no filtering is done. If you want each Rule to only know about one specific mapping, that is easy enough to do, I can add that.

I'd suggest we start writing the dimensionless_unit_mappings yaml with just one variable to test, for example one of the salinities that @siligam have shown from his tables.

Probably it would be a good idea to turn that into a unit test so it is run automatically.

I will implement the tastcase for salinity using the dimensionless_unit_mappings feature.
Just for reference, model-table unit listing https://notes.desy.de/CXe0axlgSbStWgOLjaXnlQ?view

read the dimensionless_unit_mappings and assign them to their corresponding CMOR variables. @pgierz this is also what you are saying here as well right?

the unitless mapping table is read when the CMORizer object gets initialised (either directly, this is uncommon) or via the classmethod from_dict (this is the common case), and attached to all rules that the CMORizer knows about

Partially. The mappings are read and all are assigned to each Rule: no filtering is done. If you want each Rule to only know about one specific mapping, that is easy enough to do, I can add that.

I'm not understanding something: don't the rules belong to the model side of things?
Why would we need to append it to the rule and not to the CMOR object? The mapping is for the CMOR unit, not the model unit. Please guys, let me know if I need to reread the documentation or the code, I might be saying bullshit here

Please guys, let me know if I need to reread the documentation or the code, I might be saying bullshit here

This is badly documented, so read the code ;-) I am talking about the CMORizer object here (the main controller) which has information about everything It's the part that controls the parallelisation and knows which rules exist, what the user configuration looks like, how to match up model files to cmor table entries etc etc.
At the moment, it looks like this:
>>> user_config_file = pathlib.Path("/some/file.yaml")
>>> f = yaml.safe_load(user_config_file)
>>> cmorizer = CMORizer.from_dict(f)
>>> type(cmorizer.rules)
list
>>> first_rule = cmorizer.rules[0]
>>> first_rule.model_variable
"sst"
>>> first_rule.cmor_variable
"sos"
>>> first_rule.dimensionless_unit_mappings
{"sst": {"0.001": "g/kg"}, "other_key": {"1": "µmol mol^-1"}, ...}

Of course this example is nonsense, sst isn't a concentration, but just for illustrative purposes of what sorts of things are where in the objects.

I guess the unit representation in the final output should still be "0.001" as suggested in the table. Right?

Correct. At the end of the CMOR process, we need to have metadata attributes as defined in the CMOR Tables.

Yes, the units in the tables are the target, they are the convention.

So, I have integrated dimensionless_mappings logic into unit conversion handler function and it appears work as expected. I have run examples/sample.yaml which produced the following:
The slurm log shows the expected unit conversion details:
> grep Converting slurm-13789839.out
Converting units: (thetao -> thetao) degC -> degC
Converting units: (CO2f -> fgco2) mmolC/m2/d -> kg m-2 s-1
Converting units: (so -> so) psu -> g/kg (0.001)

The units in processed netcdf file shows the correct unit as per the Table.
> ncdump -h so_Omon_AWI-ocean_piControl_r1i1p1f1_gn_278401-278412.nc | grep so:units
		so:units = "0.001" ;

What I am still missing here is a unit test for the dimensionless unit conversion.

I'm going to have to do a force push on this branch at some point, there are a few features mixed together in this one:

disentangling of the parallel initialisation
validate vs. post_init
Actual unitless conversion.

@mandresm: is it worth it, just to keep things "clean"? Or more confusing?

Regarding getting unit information like g/kg in addition to the fractional scalar: I had a look. The physical units of a quantity appear to be hidden deep down in the details of physical parameters (https://bit.ly/CMIP-DR-physical-parameters). The Variables defined in the CMOR tables then refer to the different physical parameters and inherit their unit information.
Commenting on individual physical parameters is possible via this form: https://airtable.com/appqRFkdpwAitEZNY/pagBlOWjK6JfvHtzU/form
Doing so is possible, but with significant work per parameter. Are we still confident that this additional information is necessary for a proper workflow? If so, I will write to CMIP-IPO to see what they can and are willing to do from a more fundamental level. But at the moment it does not appear to me to boil down to just adding another comment or field to the data base at hand.
This is just the way I understand the information on airtable - let me know if you have a different understanding.

Thanks @christian-stepanek for looking into this!

The Variables defined in the CMOR tables then refer to the different physical parameters and inherit their unit information.

Meaning that for example albsrfc will inherit 1 as unit right?

Doing so is possible, but with significant work per parameter. Are we still confident that this additional information is necessary for a proper workflow? If so, I will write to CMIP-IPO to see what they can and are willing to do from a more fundamental level. But at the moment it does not appear to me to boil down to just adding another comment or field to the data base at hand.
This is just the way I understand the information on airtable - let me know if you have a different understanding.

In my view this is not additional information, is just that 1 or 0.001 is not a unit, IMO it's fundamentally wrong, and programatically imprecise. If anything 1 speaks of a range (from 0 to 1) and 0.001 speaks of a ratio. This is not what I expect of a standard for doing international model comparisons. I am aware I am extremely picky though and I might not be getting the full picture here...
Is it possible to ask their opinion to other members of the community before starting to formally discuss in airtable? Or is airtable the correct place to open the discussion?
In practical terms for pymorize, it is very little work for us to provide the mapping functionality, so I don't see a big technical problem for us.

@pgierz, does this commit make sense to you or is there a better way of doing this keeping the data directory in the root directory? bc5586a

Blocked by #66

Tests are failing, I'll look into that on Monday


Preparing for workshopification

This PR code needs a careful audit as it had a hardcoded kick in it and was removed. There's potentially useful code in this though. We're readying it for putting on workshop.
Yes, I know, potential license conflict, but being that this isn't for profit anyways, the fallout from a license conflict is effectively nil.
This is not to be merged without a careful examination!
TODO:

[ ] Remove version checking. Don't even comment it out, just purge it.
[ ] Look over carefully for other potential abuse
[ ] Add travis script to build addon
[ ] Insert correct ignore scripts
[ ] Fix Invisible door when using door tool - console:Error creating 'wired_door' (Make sure the file is AddCSLuaFile'd and there aren't any errors!)

todo:
Invisible door when using door tool
console:Error creating 'wired_door' (Make sure the file is AddCSLuaFile'd and there aren't any errors!)

You mean a backdoored workshop addon was found and removed. They way you say it here sounds like the repo had the backdoor.

Just checked autorun/server/servermessage.lua, where the backdoor was found, and it's not there. Creed just put it in his workshop addon.

^ although there could be other backdoors laying around, I was just saying Creeds backdoor probably isn't here

You mean a backdoored workshop addon was found and removed. They way you say it here sounds like the repo had the backdoor.

Correct.

What hardcoded kick are you talking about?

https://github.com/wiremod/wire-extras/pull/88/commits/9d5d37ae2b6abdedd4fa7de6fcb0eaf102209733#diff-42dee2b7a9f7b151ffd2feddc526d1ddL13

Oh so this is code from that workshop release except without that part.

List of things that need to be fixed in this PR

potentially erase update checker (checks if you have the latest version installed - useless feature imo since workshop updates automatically and wire extras itself doesn't update often enough for this feature to be useful)
commented out copy of update checker needs to be erased
gmod_wire_boyancy entity has removed AddCSLua for its own clientside files, and added AddCSLua for its serverside file
large chunks of the holo anim and pose library has been erased. probably breaking it
useless whitespace added to gmod_wire_field_device/init.lua
very inefficient code added to gmod_wire_field_device/init.lua (line 608 and below)
gmod_wire_hudindicator_2 has added AddCSLua for its serverside file
gmod_wire_hudindicator_2/util/table.lua has a copy of PrintTable for no reason
useless change in gmod_wire_ramcard_defaultbase/init.lua
useless change in gmod_wire_realmagnet/cl_init.lua
The changes in gmod_wire_realmagnet/init.lua appear to be useless as well, but may need to be checked more thoroughly.
gmod_wire_rfid_implanter/init.lua removed duplicator support (??)
someone else will need to check if the changes made to gmod_wire_xyzbeacon/init.lua don't break anything
the changes made to weapons/gmod_tool/stools/gui_wiring.lua need to be undone entirely.
useless changes in gmod_tool/stools/multi_wire.lua (useless whitespace and functions renamed to their alias)
in several cases the changes in gmod_tool/stools/multi_wire.lua also try to use the beam library which no longer exists
incorrect tool category in gmod_tool/stools/wire_adv_hud_indicator_2.lua
useless whitespace changes in gmod_tool/stools/wire_field_device.lua
potentially incorrect tool category in gmod_tool/stools/wire_keycard.lua. It might fit there better, I dunno.
TargetOnlyMetal setting removed from gmod_tool/stools/wire_magnet.lua for some strange reason

TLDR: there's more things that need to be removed from this PR than things we can keep

my response to this PR https://i.kym-cdn.com/photos/images/newsfeed/001/354/591/17c.png

We'll make our own then.


How to apply padding mask without using casual mask?

Hi, thanks for the great work. I want to know if I don't want to use the casual mask, how to do the padding mask in FA2?
If I don't need the casual mask, but the data are padded in a batch, which function in FA2 should I use?

https://github.com/Dao-AILab/flash-attention/blob/74b0761ff7efc7b90d4e5aeb529c1b2a09a7458c/flash_attn/flash_attn_interface.py#L1053

no padding

no padding

Thanks. For example, if the tokenized data is like torch.tensor([[1,1,1,1,0,0,0],[1,1,1,1,1,0,0]]), 0 is the padding token, but I don't want to use casual mask. If I use the 'flash_attn_varlen_func' function, which parameter in this function indicates the padding infomation of the tensor?


Selection with mouse is not working

Text selection via mouse is not working.
I'm currently using viddy 1.1.2-VERGEN_IDEMPOTENT_OUTPUT installed via brew on Mac within iterm.
Also the workaround with hitting Space or s doesn't work for me. The only thing that is working is doing a Right click on the string I want to copy and then select Copy from the context menu.

@florianrusch Thank you for reporting.

mac & iterm.

I'm using mac & iterm, but it works fine.

The only thing that is working is doing a Right click on the string I want to copy and then select Copy from the context menu.

What is the expected behavior?
Can you copy the text by that operation (right click & select copy from the context menu)?

Hello, I have the same issue.
I just tested previous versions and it never worked after the rust rewrite, the last working version for me is 0.4.0.
The expected behavior was:

double click a string while viddy is running
the text is highlighted (selected)
the text is copied to the clipboard (because iterm is configured as "Copy on select")

An yes it is possible to copy the text from the context menu but only the selected word instead of a custom selection.

@florianrusch @BapRx I've investigated this problem.
I think it is difficult to enable selecting mouse because now viddy is capturing all mouse behavior.
There are 2 solutions.

Implement select and copy.
Implement copy all text button.

1 is difficult to implement, but 2 is not difficult, it is easy to implement.
How do you think?
The second solution can solve your problems?

Thanks for looking into it, I found a workaround on iTerm2 which would be even easier than solution 2: Hold option key then select the text with your mouse. I don't know if the issue is the same on other terminals and if a similar workaround exist.
It's easy enough for me to be a solution without implementing select and copy.

@BapRx thanks for mentioning the workaround. It also works for me 🙂
Of course from a UX perspective it would be intuitive to just click, select and copy via CMD + C or yy. Personally, I wouldn't think that holding the ALT (option key) could work. But I fully agree with you, @BapRx, that it makes no sense to put in any effort if the user base affected by this problem is very small. Therefore, for me it would also be more of a "nice-2-have".
Many thanks for the investigations, @sachaos & @BapRx.
@sachaos please feel free to close this issue, if you want.

Thank you @BapRx & @florianrusch
I'll close this issue.
I want to think about copy all feature, so I'll create new issue for that.

My team just ran into this problem and it cast a very negative light on viddy. The option (mac iterm) or shift (mac wezterm) workaround works, but it's very unintuitive and it's another thing we need to document and teach new staff.
"Copy all" is not something we would use. We need to suspend execution and copy a line or a word from the output.
We might end up downgrading to 0.3.7 until this is resolved.
Despite this issue we really appreciate everything viddy provides, so thank you for all the work you've put into it @sachaos  🙇

OK, let's reopen this issue.

I would like to add option to disable the mouse control feature firstly.

@syhol Will viddy v1.0.1 meet your requirements? It is a version prior to enabling mouse control.

Thank you for reopening the issue. v1.0.1 works nicely. We're looking forward to the disable mouse feature.

@syhol Thank you!
I've released v1.3.0 and added --disable_mouse option.
You can configure the default behavior by Configuration https://github.com/sachaos/viddy?tab=readme-ov-file#configuration .

Amazing! that works. We manage viddy though mise and the aqua registry so I need to wait for it to trickle through those before I can roll it out to my team, but I've tested it successfully with this:
❯ mise exec<EMAIL_ADDRESS>-- viddy --disable_mouse date

Thanks for the quick turnaround 🙏


Fix: Unable to add time entry on week view

Notion card
Notion card
Summary

Preview

https://user-images.githubusercontent.com/57438322/218504705-77d26019-6fda-4fa1-9177-d22c057b31ac.mp4
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactor (non-breaking and retains the same functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[ ] I have annotated changes in the PR that are relevant to the reviewer
[ ] I have added automated tests for my code

@mayankagnihotri7 Why isn't this PR assigned to anyone for review. No one will review until it is assigned for review and testing.
Also, please add summary i.e. what was the problem and how did you fix it & add video of the PR.

@supriya3105 I haven't assigned this to anyone for review because currently, I cannot create a video showing it because of the issues with my machine's being unable to run elastic search. I'll assign it to someone for review next week as soon.

@supriya3105 Could you please assign this for review to someone?

@supriya3105 Please ask someone from frontend to review this.

@mayankagnihotri7 The CI is failing on this PR. Please check and assign to Prasanth for review.

@supriya3105 I've already fixed the test cases. This is the issue with buildkite or something.

LGTM but please get approval from some FE dev


BAN-2131 Add format key and html value to text output in Apple News

From manual testing it was noted that it was not enough to just send html, we must also indicate that this is the format we are sending.
This PR adds the format key and value to the text component.
https://mydevex.atlassian.net/browse/BAN-2131

Looks good!

+1 LGTM

+1 thanks!


Figure out a datastore

Currently this loads all files into memory, and does some rudimentary in-process caching of files.
Ideally this would be replaced with something that would perform slightly better, without having to hold all of PyPI in memory forever. Something like redis with a 24 hour timeout.
Any potential solution should probably not store files on disk.

Hello @di,
I am interested in working on this issue, but I am new to Redis and Caching. I would be grateful if you could provide some insights.
Thank you

Might also be interesting to look at sqlite as a persistent volume datastore - it can scale pretty well.
Here's how these folks did it on k8s: https://www.getharvest.com/blog/running-sqlite-on-kubernetes-surprisingly-not-bad


Enums prefixed with underscore get them removed

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In our code we have an enum that has values prefixed with an _ resulting a schema like this:
enum AdnRes {
  _1
  _2
  _3
  _4
  A
  B
  D
  E
  F
  I
  N
  O
  S
  U
  W
}

When trying to load this into strawberry shake it removes the underscore resulting in invalid c#.
Steps to reproduce

Try to create a strawberry shake client with above schema
Build the project

Relevant log output
CSC : error SS0006: `` is not a valid GraphQL type name. (Parameter 'value')

Additional Context?
No response
Product
Strawberry Shake
Version
12.6.0

Closes as this is a duplicate of https://github.com/ChilliCream/hotchocolate/issues/4771


Fix strict mode 'let' keyword check in parse for-statement

This patch fixes #4190 and fixes #4195 and fixes #4188.
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik<EMAIL_ADDRESS>

export is allowed only in modules. The non-standard behavior of JerryScript causes this issue. We should rework module processing.

export is allowed only in modules. The non-standard behavior of JerryScript causes this issue. We should rework module processing.

Closing the PR, these issues are going to be resolved via #4462 .

Closing the PR, these issues are going to be resolved via #4462 .


i18n(fr): Update resources/community-content.mdx from #2049

Description
Update resources/community-content.mdx from #2049

i18n(fr)

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
resources/community-content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Thanks for the update 🙌
I tweaked the sentence a little bit to remove the remaining English 's. Let me know what you think about the change.

Thanks for the review @HiDeoo !


Confusing typo in production logstasher configuration

Based on the dictionary and looking at the logstasher documentation, it looks as if supress_app_logs in the production configuration should be suppress_app_logs. However, looking at the logstasher code, it appears as if the mis-spelt version is also supported!
Even though it is supported, I'd suggest that it is confusing for developers reading the code.

I've moved this to Trello in an attempt to end up with a single place containing Smart Answers stories/tasks/bugs.
See the Trello card (not public).
Closing this issue.


Folders: (Draft) Add read-only workspace

Description

This is part of an ongoing effort to prevent users from modifying a file that is not part of the question they are attempting. When a user accesses a file that is not part of the current question, the workspace is modified such that the content of the file cannot be changed and can only be read from.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test

In src\commons\assessmentWorkspace\AssessmentWorkspace.tsx, change isEditable to false to enable read-only mode and true for default behaviour. Go to any assessment and the editor will be read-only if isEditable is set to false.
Note that this sets all assessments to read-only mode for now.
Checklist

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-4 of 10 (60.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 37.198%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/assessmentWorkspace/AssessmentWorkspace.tsx
6
10
60.0%

Files with Coverage Reduction
New Missed Lines
%

src/commons/assessmentWorkspace/AssessmentWorkspace.tsx
1
69.71%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5729

Relevant Lines:
14468

💛  - Coveralls

Is this PR ready or still in draft state?

It is PR ready. The draft means we plan to expand on it.

Update: Sorry, it is not PR ready yet.

@accountexeregister what is the status of this PR?

Sorry prof, I just saw this message. We will still have to work on the backend before integrating it fully.


Android Calls dial only showing 1 2 3

It's just an empty space under 1 2 3 in the dial and when clicked it also only puts 1 2 3 in the number field. I have an enterprise account through my work.

@hjorleifurs - Could you please share sceenshot for this?

My phone is Samsung Galaxy 7s edge and is fully upgraded both from samsung and all apps

also if I press the space under 1 2 3 I also only get 1 2 3 in the dial, so by pressing the invisible column under 1 will just show 1 on the dial and for the invisible column under 2 shows 2 in the dial, so it is not just the display of the numbers that is missing.

Could you please share your current app version? Please do shake and send to report this as well.

I have filed in a report with a short descriptio, from the app, with a link to this thread for more information. Here is a screenshot from the app with the version number

@hjorleifurs - Could you please try this in latest build?

sorry about this very late reply but I was just busy in other stuff. But excellent news, this is working as it should and we have just replaced all the phones here at my office with the teams phone :)


Using TS generic to cast event target causing "Unterminated JSX contents" error

Version
3.0.11
Reproduction link
https://codesandbox.io/s/flamboyant-shirley-fy5w6?file=/src/App.vue
Steps to reproduce
Using script setup and a TS generic to react to change events on a select element I get the error "Unterminated JSX contents".
This is the code for the SFC.
<template>
  <select @change="handleChange">
    <option v-for="option in options" :key="option">{{ option }}</option>
  </select>
</template>

<script setup lang="ts">
import { defineProps } from "vue";

defineProps<{
  options: string[];
}>();

const handleChange = (e: Event) => {
  console.log((<HTMLSelectElement>e.target).value);
};
</script>

What is expected?
It should compile
What is actually happening?
Internal server error: [@vue/compiler-sfc] Unterminated JSX contents

I think this is a minor bug in the compiler as there are multiple workarounds
const handleChange = (e: Event & { target: HTMLSelectElement }) => {
  console.log(e.target.value);
};

or
const handleChange = (e: Event) => {
  console.log((e.target as HTMLSelectElement).value);
};

The cast operator in TS is as and it replaces the version you were using a long time ago. Precisely to avoid ambiguity: https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#new-tsx-file-extension-and-as-operator


Single place for dialog player

The way I got it from #180 and #198 it would make sense to place the dialog player in a single place, instead of every possible game.gd derivative. The dialog player contains a PackedScene which must be adapted for every game, and the script that manages it.
A counter-argument could be that if a game uses tons of different game.gd-style scripts in different rooms, the dialog player should not be moved.
Is it a valid counter to that counter-argument to say that whatever you want, you can do with dialog types?

What needs to be coded for #180 to come back is some kind of ratio adjustment between the room size and the visible viewport.

I think #180 was more or less right about where to place it, but it had issues with positioning because of the missing ratio adjustment you mention.
A fix should revert the recently merged #202 before addressing these issues.

A counter-argument could be that if a game uses tons of different game.gd-style scripts in different rooms, the dialog player should not be moved.

Is it a valid counter to that counter-argument to say that whatever you want, you can do with dialog types?

I'm not sure what kind of scripts you have in mind, but if a developer needs multiple kinds of dialog, they're better off adding multiple dialog scenes to the dialog player with different styles, offsets, margins, etc. and referencing them explicitly with the say command.

Not relevant anymore in Escoria rewrite. Dialogs are now placed in their own folder (set by gamedev, not by Escoria) and this folder is set in the Project Settings. DialogPlayer scene then automatically loads the needed scenes.


[BUG]: When using joystick, SAS does not function

SAS not functioning when using a joystick
Mod version: v2.1.1
Game version: 1.12.3
Operating system: Windows 10
Microprocessor/s in use: Arduino Uno
SAS not functioning when using a joystick
I am using the "KerbalSimpitThrottleDemo.ino" code as an example from the library, I have attached a potentiometer to the "THROTTLE_PIN" (A0) and an analogue joystick to the "PITCH_PIN" (A1) and "ROLL_PIN" (A2). (All wiring checked and functional). The potentiometer and joystick work perfectly, however when I am using the joystick, the SAS (even if activated in game) is completely non functional, the SAS still affects the Yaw axis, but no other axis.

I have tried using without the joystick, and function of the SAS is completely normal (as expected), however when I use the joystick, the used axis's are not adjusted by SAS.

To Reproduce:

Please provide the code of your controller, to help with diagnosing the issue:
The "KerbalSimpitThrottleDemo.ino" example included with the Arduino library.

Please thoroughly explain the most repeatable way that you have found to get the issue you are experiencing to occur:
Just upload the code, attach all the wiring, and attempt a flight with SAS on

Expected behaviour
I expected the pitch and roll to be changed by the joystick, and for SAS to function like if used with a keyboard (WASD)

Additional context

`/* KerbalSimpitThrottleDemo
A demonstration of how to use KerbalSimPit to send throttle
or rotation command.
Assume A0, A1, A2 are plugged on a potentiometer. Each potentiometer will be used
to control throttle, pitch and roll of you rocket.
See these links for basic descriptions of how to hook up a potentiometer:

https://www.arduino.cc/en/tutorial/potentiometer
https://learn.sparkfun.com/tutorials/sparkfun-inventors-kit-experiment-guide---v40/circuit-1b-potentiometer
*/

#include "KerbalSimpit.h"
// Set the pin numbers:
const int THROTTLE_PIN = A0; // the pin used for controlling throttle
const int PITCH_PIN = A1;    // the pin used for controlling pitch
const int ROLL_PIN = A2;     // the pin used for controlling roll
// Declare a KerbalSimpit object that will
// communicate using the "Serial" device.
KerbalSimpit mySimpit(Serial);
void setup() {
// Open the serial connection.
Serial.begin(115200);
// Set initial pin states
pinMode(LED_BUILTIN, OUTPUT);
// Turn on the built-in to indicate the start of the handshake process
digitalWrite(LED_BUILTIN, HIGH);
// This loop continually attempts to handshake with the plugin.
// It will keep retrying until it gets a successful handshake.
while (!mySimpit.init()) {
delay(100);
}
// Turn off the built-in LED to indicate handshaking is complete.
digitalWrite(LED_BUILTIN, LOW);
// Display a message in KSP to indicate handshaking is complete.
mySimpit.printToKSP("Connected", PRINT_TO_SCREEN);
}
void loop() {
// Send at each loop a message to control the throttle and the pitch/roll axis.
throttleMessage throttle_msg;
// Read the value of the potentiometer
int reading = analogRead(THROTTLE_PIN);
// Convert it in KerbalSimpit Range
throttle_msg.throttle = map(reading, 0, 1023, 0, INT16_MAX);
// Send the message
mySimpit.send(THROTTLE_MESSAGE, throttle_msg);
rotationMessage rot_msg;
// Read the values of the potentiometers
int reading_pitch = analogRead(PITCH_PIN);
int reading_roll = analogRead(ROLL_PIN);
// Convert them in KerbalSimpit range
int16_t pitch = map(reading_pitch, 0, 1023, INT16_MIN, INT16_MAX);
int16_t roll = map(reading_roll, 0, 1023, INT16_MIN, INT16_MAX);
// Put those values in the message
rot_msg.setPitch(pitch);
rot_msg.setRoll(roll);
// Send the message
mySimpit.send(ROTATION_MESSAGE, rot_msg);
}`
Thanks for any help!

digging into the actual game code this appears to be largely intentional, if not ideal. the default code sends a value between INT16_MIN and INT16_MAX, based on the Arduino analogue input 0-1023. internally the game processes this by taking the player input value, and if its not equal to zero - i.e. at the mod point, using it. if it is at the mid point it doesn't use it.
the result is that if you have SAS enabled it can only function when the joystick is dead centre.
the trouble is its unlikely your joystick reports the dead centre value, 511, when in the middle, its likely a range a bit above or below - hence the game sees an input and uses it.
there are two possible solutions to this, one is to modify the code on the arduino to add a "dead band" in the centre, so any value +/- a given range from 511 is reported as 511, then the game will see that as a zero input and let SAS do its thing. - Ideally the examples need to be adjusted to account for this or at least include a comment.
the second method would be to change the code in the game so instead of using either the SAS or player input, it combines them - this can be added in the "AxisControl.cs" file by changing how the function "AutopilotUpdate()" works to not simply use one value or the other but to blend them in some way.
I have tested some very basic code that will take both values, and if both go in the same direction, use the one with the larger magnitude, and if they oppose each other will take a weighted average (player input tested at 75%), so the player can override SAS but not totally when its active

as an example of how to add a deadzone to the example consider the following:
rot_msg.setPitch(JoystickRemap(analogRead(A1), 40, false));
this calls a custom function to process the analogue input to something suitable, this function is as follows
// remap joystick inputs to suitable range with deadband and inversion support
int16_t JoystickRemap(uint16_t Value, uint8_t Deadband, bool Invert)
{
  int16_t R = 0;

  // first apply the deadband code to the input value
  if (Value <= (511 - Deadband))
  {
    R=Value;
  }
  else if (Value >= (511 + Deadband))
  {
    R=Value;
  }
  else
  {
    R=511;
  }

  // now apply the actual mapping
  if (R == 511)
  {
    // if in the deadband we ensure the output value is zero to centre the axis
    R = 0;
  }
  else
  {
    if (Invert == true)
    {
      R = map(R, 0, 1023, INT16_MAX, INT16_MIN);
    }
    else
    {
      R = map(R, 0, 1023, INT16_MIN, INT16_MAX);
    }
  }

  return R;
}

you can specify the size of the desired deadband and if the axis should be inverted

That's worked perfectly, thanks for your help!


can get the repo from go

When I tried to get this repository follow the command below,It told me the wrong messages,
shell > go get github.com/skeema/skeema
/root/go/src/github.com/skeema/skeema/vendor/github.com/docker/docker/pkg/archive/archive.go:364:5: hdr.Format undefined (type *tar.Header has no field or method Format)
/root/go/src/github.com/skeema/skeema/vendor/github.com/docker/docker/pkg/archive/archive.go:364:15: undefined: tar.FormatPAX
/root/go/src/github.com/skeema/skeema/vendor/github.com/docker/docker/pkg/archive/archive.go:1166:7: hdr.Format undefined (type *tar.Header has no field or method Format)
/root/go/src/github.com/skeema/skeema/vendor/github.com/docker/docker/pkg/archive/archive.go:1166:17: undefined: tar.FormatPAX
my golang version is "go version go1.9.4 linux/amd64"
what's wrong with it?
thanks for more.

Thank you for trying out Skeema and reporting this issue! It seems that one of the third-party dependencies for Skeema's test suite may now require Golang 1.10. Can you please try upgrading your version of Golang compiler and see if the error is still reported?
I believe the problem is that github.com/docker/docker/pkg/archive introduced a change back in April which refers to fields that were added in Golang 1.10.
I would like to support at least the few previous versions of Golang, but it may not be possible to unwind the dependency chain there... I'll do more digging, but in the meantime please let me know if upgrading to Golang 1.10 resolves this issue just to confirm that was the root cause. Thanks!

Support for Golang 1.9 in compiling Skeema has been fixed in #27, so closing this out. Thanks again for reporting it -- I previously was not aware that dependencies had broken 1.9 support.
Moving forwards, Skeema will support the two most recent Golang versions (so currently 1.9.x and 1.10.x), which matches Golang's own policy. Also, I've configured Travis CI to perform automated testing using 1.9.x to ensure compatibility is maintained, even though development occurs primarily using 1.10.x.

thx,That completely run under Go-1.9.4


[MNT] Conda Forge 0.13.1 Release

Checklist

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

Things I have done in this PR:
Set build version to 0.13.1
Set SHA to SHA256 found on PyPi
Set build number to 0
Add the following soft dependencies

gluonts >=0.9.0
kotsu >=0.3.1
tensorflow
tensorflow-probability
tslearn >=0.5.2
statsforecast >=0.5.2
dtw-python
esig ==0.9.7
mne
pyod >=0.8.0

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

@conda-forge-admin

@conda-forge-admin, please rerender

It seems the link to the github release assets changed. The assets are now under https://github.com/alan-turing-institute/sktime/archive/refs/tags/v0.13.1.tar.gz url so the template in the meta.yml would need to be updated.

It seems the link to the github release assets changed. The assets are now under https://github.com/alan-turing-institute/sktime/archive/refs/tags/v0.13.1.tar.gz url so the template in the meta.yml would need to be updated.

Ah, could that be the reason why the automated releases on conda are all failing?

The correct url has now revealed a SHA256 mismatch

The correct url has now revealed a SHA256 mismatch

I'm still new to this. But how do you tell it's SHA mismatch?

It seems the link to the github release assets changed. The assets are now under https://github.com/alan-turing-institute/sktime/archive/refs/tags/v0.13.1.tar.gz url so the template in the meta.yml would need to be updated.

I can't find this directory in sktime's Github. How did you find this?

This is one of the release assets. Go to the release page for the latest release and it should have two assets attached, a zip file and a tar.gz file.

Looks like good progress :+1:
It seems that there are still issues with dependency resolution:
Mamba failed to solve:
 - sktime-all-extras 0.13.1.* ha770c72_0
 - dtw-python
 - esig 0.9.7.*
 - filterpy >=1.4.5
 - gluonts >=0.9.0
 - kotsu >=0.3.1
 - matplotlib-base >=3.3.2
 - mne
 - numba >=0.53
 - pmdarima >=1.8.0,!=1.8.1
 - pykalman >=0.9.5
 - pyod >=0.8.0
 - python
 - scikit-posthocs >=0.6.5
 - seaborn >=0.11.0
 - sktime >=0.13.1,<<IP_ADDRESS>a0
 - statsforecast >=0.5.2
 - stumpy >=1.5.1
 - tbats >=1.1.0
 - tensorflow
 - tensorflow-probability
 - tsfresh >=0.17.0
 - tslearn >=0.5.2
 - python

with channels:

The reported errors are:
- Encountered problems while solving:
-   - nothing provides esig 0.9.7 needed by sktime-all-extras-0.13.1-ha770c72_0
-   - nothing provides requested esig 0.9.7.*
-   - nothing provides requested gluonts >=0.9.0
-   - nothing provides requested kotsu >=0.3.1

Mamba does not provide the most helpful message but maybe you could try making it work locally first?

when built locally in Linux, the following exception was caught:
failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform linux-64: {'esig=0.9.7', "gluonts[version='>=0.9.0']", "kotsu[version='>=0.3.1']"} WARNING:conda_build.build:failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform linux-64: {'esig=0.9.7', "gluonts[version='>=0.9.0']", "kotsu[version='>=0.3.1']"} Tests failed for sktime-all-extras-0.13.1-0.tar.bz2 - moving package to /home/ttran/anaconda3/conda-bld/broken WARNING:conda_build.build:Tests failed for sktime-all-extras-0.13.1-0.tar.bz2 - moving package to /home/ttran/anaconda3/conda-bld/broken TESTS FAILED: sktime-all-extras-0.13.1-0.tar.bz2

I think I've found the problem. On Pypi, the meta.yaml is looking into conda and/or conda-forge channel for kotsu, gluonts and 'esig.' However, these are not available on conda. At least in gluonts case, it's only available up to ver. 0.8.1 Fixing gluonts to 0.8.1 in the meta.yaml make that message go away for gluonts
Next, I'll try removing these 3 packages from the recipe. If it works, we may have to add a note to users that they'll have to manually install these using pip

Removing those 3 packages introduces a new failure message:
Traceback (most recent call last):
  File "/home/ttran/anaconda3/bin/conda-build", line 11, in <module>
    sys.exit(main())
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/cli/main_build.py", line 488, in main
    execute(sys.argv[1:])
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/cli/main_build.py", line 477, in execute
    outputs = api.build(args.recipe, post=args.post, test_run_post=args.test_run_post,
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/api.py", line 186, in build
    return build_tree(
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/build.py", line 3099, in build_tree
    test(pkg, config=metadata.config.copy(), stats=stats)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/build.py", line 2865, in test
    actions = environ.get_install_actions(metadata.config.test_prefix,
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda_build/environ.py", line 799, in get_install_actions
    actions = install_actions(prefix, index, specs, force=True)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/common/io.py", line 86, in decorated
    return f(*args, **kwds)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/plan.py", line 473, in install_actions
    txn = solver.solve_for_transaction(prune=prune, ignore_pinned=not pinned)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/core/solve.py", line 152, in solve_for_transaction
    unlink_precs, link_precs = self.solve_for_diff(update_modifier, deps_modifier,
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/core/solve.py", line 195, in solve_for_diff
    final_precs = self.solve_final_state(update_modifier, deps_modifier, prune, ignore_pinned,
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/core/solve.py", line 313, in solve_final_state
    ssc = self._add_specs(ssc)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/core/solve.py", line 707, in _add_specs
    conflicts = ssc.r.get_conflicting_specs(tuple(MatchSpec(_)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/resolve.py", line 1106, in get_conflicting_specs
    C = r2.gen_clauses()
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/common/io.py", line 86, in decorated
    return f(*args, **kwds)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/resolve.py", line 909, in gen_clauses
    for ms in self.ms_depends(prec):
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/resolve.py", line 793, in ms_depends
    deps = [MatchSpec(d) for d in prec.combined_depends]
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/models/records.py", line 340, in combined_depends
    result = {ms.name: ms for ms in MatchSpec.merge(self.depends)}
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/models/match_spec.py", line 476, in merge
    reduce(lambda x, y: x._merge(y, union), group) if len(group) > 1 else group[0]
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/models/match_spec.py", line 476, in <lambda>
    reduce(lambda x, y: x._merge(y, union), group) if len(group) > 1 else group[0]
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/models/match_spec.py", line 507, in _merge
    final = this_component.merge(that_component)
  File "/home/ttran/anaconda3/lib/python3.9/site-packages/conda/models/match_spec.py", line 758, in merge
    raise ValueError("Incompatible component merge:\n  - %r\n  - %r"
ValueError: Incompatible component merge:
  - 'mpi_mpich_*'
  - 'mpi_mpich_tempest*'

Found a solution here
What it does is we need to define an environment.yaml that declares all dependencies and their channel (conda or pip)

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The outputs section contained an unexpected subsection name. dependencies is not a valid subsection name.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The outputs section contained an unexpected subsection name. run is not a valid subsection name.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Failed to even lint the recipe, probably because of a conda-smithy bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint . from the recipe directory.

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

Closing as superseded.


Revert "refac: Bump opengeh python package"

Reverts Energinet-DataHub/opengeh-wholesale#2464

nvm


X11: Add support for window closing and quitting.

This PR adds support for closing individual windows. Previously closing any window would close all of them. This can be tested with the multiwin example.
Additionally I refactored some code and added more inline documentation.

I also implemented Application::quit. Needed to create an events-only hidden window for this, because there doesn't seem to be a way to receive events when all windows have been destroyed. With macOS and Windows the implementation allows for the druid-shell user (e.g. druid) to choose the policy of whether to quit when all windows have closed or keep running.
I also added a Quit app button to multiwin to allow testing of this functionality on X11, because there is no menu support yet.

I added more code comments to hopefully make things clearer. I also removed connection flushing from request_redraw and handled the cairo context creation error.


fix(): hide matt

https://openforge.teamwork.com/#/tasks/17475919

The "you" picture is misaligned from its column

ready

QA Passes


Trojan:Win32/Tiggre!plock in tool_01DIY85(3.3.0).exe by defender & confirmed by virustotal

Trojan:Win32/Tiggre!plock today (23 Aug 2019) found in tool_01DIY85(3.3.0).exe

14 engines detected by:
https://www.virustotal.com/gui/file/33c91045877dd7442b2964583e87ed30bd4d46ea69064739e1a1cf8167910b11/detection

I have the same finding in my windows 10 with the current master tool_01DIY85(3.3.0).exe

Yup, just had to go through a manual remove process for this virus. Lovely Chinese software. And they wonder why we are flashing the firmwares on these devices to tasmota...

No trace of virus removal in todays tool_01DIY85(3.3.0).exe
Seems even worse:

Is there some hope to ever get this tool free of viruses? Or alternative software? For me, sonoff mini is to small to solder flash wires on... I regret having bought this... The DIY-mode is a real pain...

This tool was written by me, packaged into exe, and uploaded to Github.I don't know why my hard-written code was checked for a virus.And this tool's code all uploaded to Github, everyone can review my code, whether there is a virus Trojan tool, a look at it.

Sorry for the inconvenience.But I must find out, this is an anti-virus software false alarm, or I really have a Trojan code.

@ZZLinvec, it is always a good practice to sign a windows exe file. This will ensure that the publisher is identified and the code is not altered or corrupted since it is signed. Check out how to do Code signing for windows.  Highly recommended at the earliest for this tool.

This tool was written by me, packaged into exe, and uploaded to Github.I don't know why my hard-written code was checked for a virus.And this tool's code all uploaded to Github, everyone can review my code, whether there is a virus Trojan tool, a look at it.The open source model is meant to be used by more people, and we're certainly not going to hurt our customers.

I had originally downloaded it onto an old laptop that isn't really updated or used often. Ran it there, had issues with the tool discovering the device. Then I attempted to download it on my daily driver Desktop, and Chrome, Firefox, and Windows itself, have all flagged it for containing this virus. I then checked the old laptop to see if the files that virus creates, existed, and they did. Had to update Windows Defender on the laptop, and it triggered and removed at that point as well.

I then checked the old laptop to see if the files that virus creates, existed, and they did. Had to update Windows Defender on the laptop, and it triggered and removed at that point as well.

Really?  So  it's not false detection?  Can you please provide details - what files it created?  Where did you find the info?  If it truth - f##k Sonoff!

31.8.19 still the same Virus found by virustotal.com/Microsoft in tool_01DIY85(3.3.0).exe

I think Sonoff use Py2exe to compile Python code into exe. This is knows for false detection because Py2exe is also used to compile malware - producing similar signature.

@alexbk66
It’s been a couple weeks since I removed it. I just followed a google search result pertaining to the mentioned virus. There were a couple of registry keys and files in system32.
I also use Py2exe for a few projects (YouTubeDownload) and have never had this issue.

@alexbk66
It’s been a couple weeks since I removed it. I just followed a google search result pertaining to the mentioned virus. There were a couple of registry keys and files in system32.

So you can't provide a link to the webpage with details?

Its been ages. I'm not actively working on these projects all the time. As I said, there are multiple google results for this signature. I expect you being on GitHub, you are capable of pulling this information yourself?

Has been over 1 month since the last comment on this thread.  Does anyone know what the status is?  Too bad the author of the code "ZZLinvec" is silent after his initial comments; sure doesn't instill much confidence in his code.

I just downloaded and checked and it is not detected as malware for me anyways. So I guess it was a false positive.


Added selection sort in the sorting algorithm list.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: Instead of just adding a link, it would be better to explain how the Selection Sort works.  Please create a new PR to add this information.
Thank you and happy coding.