set a default open_timeout of 10 seconds

Currently we don't set a default open_timeout, and rely on the default timeouts specified by each client adapter. The default client adapter, Net::HTTP, has a nil an open_timeout value, which can cause a request to block indefinitely if network traffic is dropped. This can cause a serious issues in background services or workers. I'd expect a small number of our users are manually configuring this value until they run into an issue. It would be helpful to automatically work around issues like #176 by default.
Adapter-wise, open_timeout is used by all the Faraday adapters. Here's a run down of the default open_timeout values in the available clients:

excon: :connect_timeout      => 60 https://github.com/excon/excon/blob/25117b214e5f0464e894549585dc9bdaad039730/lib/excon/constants.rb#L113
typhoeus: connecttimeout of 300 seconds via curl https://github.com/typhoeus/typhoeus#readme / http://curl.haxx.se/libcurl/c/CURLOPT_CONNECTTIMEOUT.html
patron: @connect_timeout = 1 https://github.com/toland/patron/blob/2b72d0c3934fba6a197869dda392dce8c1feca49/lib/patron/session.rb#L39-L40
httpclient: @connect_timeout = 60 https://github.com/nahi/httpclient/blob/45b1045976d3a09b6491c69668491295c2f6ff08/lib/httpclient/session.rb#L133
em-http-request: @connect_timeout     = options[:connect_timeout] || 5 https://github.com/igrigorik/em-http-request/blob/f3e64c01cb7f3039e2309cbb165a83e98b87af5e/lib/em-http/http_connection_options.rb#L6 / https://github.com/igrigorik/em-http-request/wiki/Redirects-and-Timeouts

10 seconds seems like a conservative but reasonable default. It fits within a standard unicorn timeout of 25-30 seconds, but isn't too aggressive to interfere with things like 5 second DNS timeouts.
/cc @steved555
Risks
This will change the timeouts for users that have manually configured an alternate adapter without an explicit open_timeout. The new values should be reasonably high for even for very janky networks. Manual configuration of timeouts is still possible.

+1


Travis seems to support setting cache name

https://docs.travis-ci.com/user/caching/#Caches-and-build-matrices

If these characteristics are shared by more than one job in a build matrix, they will share the same URL on the network. This could corrupt the cache, or the cache may contain files that are not usable in all jobs using it. In this case, we advise you to add a defining public environment variable name; e.g.,

CACHE_NAME=JOB1

to .travis.yml.

interesting/good-to-know; but at this point I don't think the current v2 travis script generator needs this

the different jobs are differentiated based on compiler already?

yes... that's at least I observed when I looked at the generated cache-ids

Note that this makes our cache not network-local, it’s still bound to network bandwidth and DNS resolutions for S3. That impacts what you can and should store in the cache. If you store archives larger than a few hundred megabytes in the cache, it’s unlikely that you’ll see a big speed improvement.

This actually reminds me to implement GC; at least we don't include full GHC installations in the travis cache... ;-)

If v2 script would support building with different lts-snapshots / package-sets (in different jobs), than this might be needed, as compiler alone won't be enough to differentiate caches.
OTOH it will take less time to test different package-sets inside single job.
I close this issue, it was meant as FYI, and that is accomplished.

yeah, especially since travis limits the maximum parallelism to about 5 jobs, I saw little benefit in spreading LTS snapshots across multiple jobs


fix nans in bti dev head t when presen

This is to fix the issue reported by Joshua Bear that occurs when nans are found in the device to head transform. The patch enables reading the files that Joshua shared with us.
cc @agramfort

I have some archaic reflex not to overwrite variables in a local scope.
That often gave mysterious errors back then.
On Fri 28 Sep 2018 at 21:06, Alexandre Gramfort<EMAIL_ADDRESS>wrote:

@agramfort commented on this pull request.
In mne/io/bti/bti.py
https://github.com/mne-tools/mne-python/pull/5566#discussion_r221353222:

@@ -247,7 +247,13 @@ def _points_to_dig(points, n_idx_points, use_hpi):

def _convert_coil_trans(coil_trans, dev_ctf_t, bti_dev_t):
"""Convert the coil trans."""

t = combine_transforms(invert_transform(dev_ctf_t), bti_dev_t,

dev_ctf_t_ = dev_ctf_t.copy()
has_nan = np.isnan(dev_ctf_t_['trans'])
if np.any(has_nan):

   dev_ctf_t_['trans'] = np.identity(4)

I would have just done
if np.any(np.isnan(dev_ctf_t['trans'])):
dev_ctf_t = dev_ctf_t.copy()
dev_ctf_t['trans'] = np.identity(4)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/5566#pullrequestreview-159987719,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fihPutUDLW3iDqBEAzFTYDrFnmDOXks5ufnNJgaJpZM4W_PVr
.

At least avoid the systematic copy

Hi Denis,
Thank you for your quick work on this!
Your branch seems to have worked for reading in the file, though it looks like the problem might prevent writing the data as well.

raw.save('here-raw.fif')
Writing /research/data/here-raw.fif
Traceback (most recent call last):
File "", line 1, in
File "", line 2, in save
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/utils.py", line 748, in verbose
return function(*args, **kwargs)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/io/base.py", line 1821, in save
split_size, split_naming, part_idx, None, overwrite)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/io/base.py", line 2272, in _write_raw
reset_range, raw.annotations)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/io/base.py", line 2446, in _start_writing_raw
write_meas_info(fid, info, data_type=data_type, reset_range=reset_range)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/io/meas_info.py", line 1412, in write_meas_info
write_coord_trans(fid, info['dev_ctf_t'])
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/mne-python/mne/io/write.py", line 316, in write_coord_trans
trans_inv = linalg.inv(trans['trans'])
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/scipy/linalg/basic.py", line 946, in inv
a1 = _asarray_validated(a, check_finite=check_finite)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/scipy/_lib/_util.py", line 238, in _asarray_validated
a = toarray(a)
File "/Users/joshbear/bin/anaconda3/envs/mne/lib/python3.6/site-packages/numpy/lib/function_base.py", line 1233, in asarray_chkfinite
"array must not contain infs or NaNs")
ValueError: array must not contain infs or NaNs

This should work now (also the writing).

Just had a chance to test the writing fix on two my empty room recordings and can confirm that it seems to be working. Thank you again for the quick fix! I'm hoping to be able to provide some contributions of my own instead of just pointing out the problem next time.

thx @dengemann


build!: update minimum required versions of browsers

Going forward, we'll only support browsers versions released since 2021. These changes should reduce the size of the transpiled scripts substantially.

/deploy-preview

/deploy-preview


type: :feature option causes module inclusion to fail

I am not sure where the problem lies... whether with RSpec, rspec-rails or Capybara. I found one discussion on SO where someone had the same problem, so I'm posting it as an issue in case it's actually reproducible.
So the basic symptom is that Capybara DSL isn't being recognized unless it's explicitly included in RSpec.configure. And the problem is that in RSpec.configure, this works:
config.include Capybara::DSL

but this doesn't:
config.include Capybara::DSL, type: :feature

My features are in spec/features as *_spec.rb files. The error is (for both rake spec and for rake spec:features):
Failure/Error: visit '/'
NoMethodError:
   undefined method `visit' for #<RSpec::ExampleGroups::IncomePage::Individual:0x007fece82c97c8>

I appear to have the latest release versions of everything:
rake --version
rake, version 10.3.2

From my Gemfile.lock
rspec-core (3.1.7)
  rspec-support (~> 3.1.0)
rspec-expectations (3.1.2)
  diff-lcs (>= 1.2.0, < 2.0)
  rspec-support (~> 3.1.0)
rspec-mocks (3.1.3)
  rspec-support (~> 3.1.0)
rspec-rails (3.1.0)
  actionpack (>= 3.0)
  activesupport (>= 3.0)
  railties (>= 3.0)
  rspec-core (~> 3.1.0)
  rspec-expectations (~> 3.1.0)
  rspec-mocks (~> 3.1.0)
  rspec-support (~> 3.1.0)
rspec-support (3.1.2)

This is by design.  In RSpec 2, specs were automatically tagged with :type metadata based on their location in the file system.  This was very implicit and "magical" and could lead to confusing errors (for example, if the user put a model spec in spec/model/my_model_spec.rb rather than spec/models/my_model_spec.rb it would not work properly).
In RSpec 3, the spec-type inferrence based on file location is disabled by default.  Instead, we recommend you explicitly tag the spec type of each example group:
RSpec.describe "My feature spec", type: :feature do
  # ...
end

If you really prefer the spec type inferrence you can opt-in to that instead:
RSpec.configure do |c|
  c.infer_spec_type_from_file_location!
end

...although we recommend against that.
For more info, read our announcement about this change a while back.

This is by design.  In RSpec 2, specs were automatically tagged with :type metadata based on their location in the file system.  This was very implicit and "magical" and could lead to confusing errors (for example, if the user put a model spec in spec/model/my_model_spec.rb rather than spec/models/my_model_spec.rb it would not work properly).
In RSpec 3, the spec-type inferrence based on file location is disabled by default.  Instead, we recommend you explicitly tag the spec type of each example group:
RSpec.describe "My feature spec", type: :feature do
  # ...
end

If you really prefer the spec type inferrence you can opt-in to that instead:
RSpec.configure do |c|
  c.infer_spec_type_from_file_location!
end

...although we recommend against that.
For more info, read our announcement about this change a while back.

Your promptness is awesome. I've posted your announcement to the couple of SO posts I found that discuss this.

Your promptness is awesome. I've posted your announcement to the couple of SO posts I found that discuss this.


Show stats for most popular languages and frameworks

just now noticed that there is already a similar grid in the Conferences tab but never saw it because my conferences table is empty :see_no_evil: so the stats page can be fixed later if I have time

Awesome, thanks! The statistics page currently isn't showing any data that isn't already visible in the stats page. It would be cool to see which langs/frameworks are trending up and which ones down, you should be able to get this data from the stats_daily table.
We could also draw graphs for each language/framework in the last 30 days. It would look really cool if we had cards for each language that sort of look like this:

Of course instead of visitors we would show number of devs, and then the % could be based on the change from the last 30 days.

Possible improvements:

implementing stats for last 7/30/x days should be really easy but the dropdown from https://flowbite.com/docs/plugins/charts/#area-chart stopped working when I placed it in my grid :thinking:
I couldn't get the Apex (library for graphs) install with npm working so we are using CDN
the spacing between columns in full screen mode is maybe unnecessary but couldn't find a fix at first glance

fixed 1st and 3rd bullet point from previous comment

Awesome, this looks great! I made some minor tweaks, but all good!
This should close #18


Content Enricher not properly supported when generating Camel Xml

At time, Content Enricher pattern is present into designer but not supported for Camel Xml generation. This making the generation of a route containing pattern completely unusable.

Done!


OOM Issue Apears in Speech to Text Task

Hi,
I was running the fariseq speech-to-text example and got OOM issue (either 1 or 4 v100 GPU). I tried the --empty-cache-freq flag. It still occurs OOM eventually although it takes longer to OOM. With 128 G mem
I also tried to limit the size of max-tokens but it seems the mem also slowly raised up. Is there anything I could avoid this?
To reproduce
#!/bin/bash
python train.py ${MUSTC_ROOT}/en-fr --train-subset train_st --valid-subset dev_st --save-dir ${ST_SAVE_DIR}
--max-tokens 10000 --task speech_to_text --criterion label_smoothed_cross_entropy --num-workers 0
--max-update 200000 --arch s2t_transformer_s --optimizer adam --lr 2e-3
--lr-scheduler inverse_sqrt --warmup-updates 10000 --clip-norm 10.0 --seed 1 --config-yaml ${MUSTC_ROOT}/en-fr/config_st.yaml
--empty-cache-freq 1 --report-accuracy --load-pretrained-encoder-from ${ASR_SAVE_DIR}/${ASR_CHECKPOINT_FILENAME}
Thanks.

With V100, you may try --fp16 to save GPU memory and speed-up training. When trying lower --max-tokens, did you find one that gets you through?

With V100, you may try --fp16 to save GPU memory and speed-up training. When trying lower --max-tokens, did you find one that gets you through?


Fix deprecation warning

Hi!
We recently upgraded to Elixir 1.17.3 and got this deprecation warning. It seemed like an easy fix, so I went ahead and did it. I'm not sure this covers all usages within the project, but it stopped appearing for me at least.

Thanks for maintaining this lib, it helps out a lot :slightly_smiling_face:

thanks!


Component not Working on opportunity entity

The component works fine for all entities except the opportunity,
below is the error upon uploading a file:

Appreciate your support on this matter.

The component only works with entities that have notes enabled.

I double checked it and the opportunity entity already has the notes enabled, I even tried it on a different environment and got the same error.
it worked on Quotes, Accounts and all other custom entities. the only one with issues is the opportunity.

Fixed in Release 1.7


Remove version: Cisco.CiscoWebexMeetings version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://akamaicdn.webex.com/client/WBXclient-41.5.4-6/webexapp.msi
OK
(Automated message - build 912)

URL: https://akamaicdn.webex.com/client/WBXclient-41.5.4-6/webexapp.msi
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 912)


No link to current content for Group Policy Preferences

The 2008/2012 version of THIS has a link at the top for "Modern Version".  That link just drops you at https://docs.microsoft.com/en-us/windows-server/, not a modern version of the topic.
Aside:  I defy someone to find Server 2016/2019 versions of Group Policy documentation.  Here's what you get if you actually SEARCH from the Windows Server page:

@sudeepku can you take a look at this issue?


Out of date HTTPoison version

According to mix.exs, nadia requires ~> 1.1.1, while currently, httpoison is at version 1.5.0

updated at version 0.4.5


Clear problems setting

added setting that if set to true will clear the problems (linter warnings/errors) for a document when you close it

 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.:x: mseiler025 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add com.roku/video/jsonschema/1-0-0 (close #1189)

As explained in issue #1189 this PR adds schema for context added to media events sent from the Roku tracker. It adds extra information provided by the Roku Video player.

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

I had to make a small change in the schema in order to support error handling in the video playback. The way that error handling on videos works is that they are assigned an error state and the details of the error are described in the errorStr property. I had to add the optional errorStr property to this schema.
Also, in the error, there is not stream information and so I had to make the stream info (such as streamUrl and measuredBitrate) optional here too.
On the other hand, I added a contentUrl property which refers to the URL passed to video when initialized – this one is available even in error state so it can be useful to provide more context.


Small Nav Updates for Gatekeeper

~Logging - Request for Overview link~

[x] Can we add an actual link Overview into the nav bar for logging so that if I click into one of the CRDs and then want to go back to the Overview, I'll be able to get to it?

OPA Gatekeeper

[x] Can the order of the navigation be

Overview
Constraint
Template

[x] Can we add Overview heading and Powered by OPA Gatekeeper text similar to what we did for logging? The link should direct to https://github.com/open-policy-agent/gatekeeper
[x] Can we remove the extra shadow border around the Violations table?

Rancher version: master-head (09/28/2020) ec04f78 HA k8s with DO RKE downstream cluster.

Link to Overview is now present.
Extra Shadow border is removed
Overview header with Powered by OPA Gatekeeper is present and the link redirects properly in a new tab to https://github.com/open-policy-agent/gatekeeper
Order of navigation is

Overview
Constraint
Templates

Opened #1484, small styling/navigation issue.


How to test @Bean ServerList be created？

I customed a Ribbon client config follow the docs, but i want to know how to make a test to prove @bean ServerList be created， because it's  a base project of my projects , i have to make sure every release version is stable.
This is the main part of my code：
@Configuration
@ConditionalOnClass(Ribbon.class)
public class RibbonConfig {
    @RibbonClientName
    private String name = "client";

    @ConditionalOnProperty(value = "xxx.xxx.enabled", matchIfMissing = true)
    @Bean
    public ServerList<Server> ribbonServerList(Environment environment, PropertiesFactory propertiesFactory, IClientConfig config) {
        if (propertiesFactory.isSet(ServerList.class, name)) {
            return propertiesFactory.get(ServerList.class, config, name);
        }
        String qconfUrl = getQconfUrl(environment, name);
        if (StringUtils.isEmpty(qconfUrl)) {
            ConfigurationBasedServerList serverList = new ConfigurationBasedServerList();
            serverList.initWithNiwsConfig(config);
            return serverList;
        }
        return new QconfBasedServerList(qconfUrl, config);
    }
}

@RibbonClients(defaultConfiguration = RibbonConfig.class)
public class GlobalRibbonConfig {}

@Configuration
@ConditionalOnClass(Feign.class)
@Import(GlobalRibbonConfig.class)
public class EarthFeignAutoConfiguration {}

finally,EarthFeignAutoConfiguration will be set in spring.factories, then I made this test
private ApplicationContextRunner contextRunner = new ApplicationContextRunner()
			.withConfiguration(AutoConfigurations.of(EarthFeignAutoConfiguration.class));
        @Test
	public void testBeanCreation() {
		Assert.assertNotNull(bean);
		this.contextRunner.run(context -> {
			assertThat(context).hasSingleBean(ServerList.class);
		});
	}

and test failed ,  I found that original bean ServerList be created once after a @FeignClient service called, so I add a service call in above unit test code， but also failed，ServerList  will be created after the test completed，so i really want to how to make a unit test to make sure ServerList bean will be created

It's not available in the parent context, each ribbon client gets its own context. See https://github.com/spring-cloud/spring-cloud-netflix/blob/master/spring-cloud-netflix-ribbon/src/test/java/org/springframework/cloud/netflix/ribbon/RibbonClientsPreprocessorIntegrationTests.java for how to use the SpringClientFactory to get ribbon beans for a named ribbon client.


Cant change UI language in options either in amending the main YAML configuration file

I modified it with in this two way,but it did't work and rolled back to default setting.
Happends in vision 68.0
I tested vision 67.0 work succesfully

As explained on Configuration page, the sample file has to be renamed.

Also, do not use notepad to edit this file. Use something like Notepad++.


lz4: missing CMake configuration files

Describe the bug
The lz4 package does not ship with CMake configuration files ($dev/lib/cmake/lz4/lz4Config.cmake and more) which are needed for some CMake project to build properly.
Steps To Reproduce
Steps to reproduce the behavior:

nix build nixpkgs#lz4.dev

Expected behavior
lz4 should contain the following files:
lib/cmake/lz4/lz4Config.cmake
lib/cmake/lz4/lz4ConfigVersion.cmake
lib/cmake/lz4/lz4Targets-none.cmake
lib/cmake/lz4/lz4Targets.cmake

Additional context
Upstream provides multiple build scripts to build lz4. We are currently using the Makefiles that seem to be the primary way to build lz4. But in addition, lz4 provides build/cmake/CMakeLists.txt, which among building lz4 itself also generates the files needed for other CMake packages.
Most distributions use the Make build script as the primary way of building lz4:

Alpine: Make
Arch Linux: CMake
Debian Make
Fedora Meson
openSUSE: Make
Solus: Make
Void Linux: Make

I have noticed that the derivation has some conditions for MinGW, and it seems like they wouldn't work with CMake, so by switching to CMake as is might break MinGW builds.
Notify maintainers
Last person who touched lz4 is @szlend. Though this problem wasn't caused by them :D
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.3-zen1, NixOS, 24.11 (Vicuña), 24.11.20240607.051f920`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Lix, like Nix) 2.90.0-beta.1`
 - channels(scrumplex): `""`
 - nixpkgs: `/etc/nix/channels/nixpkgs`

Add a :+1: reaction to issues you find important.

As there are a few comments regarding MinGW by them, I am cc'ing @Ericson2314

fixed with linked issue


Error loading groq models

Hi, I found an issue when you select groq models.
To solve the bug you just need to modify line 76 from llms.py to:
    "models": ["groq/llama3-8b-8192","groq/llama3-70b-8192", "groq/mixtral-8x7b-32768"],

thanks, fixed in https://github.com/strnad/CrewAI-Studio/pull/27


Merge styles with same name in DocumentBuilder

Solves #181.
Currently implemented with a compile-time feature flag, but this could easily be changed to an optional parameter passed to DocumentBuilder at run-time.
It is likely possible to clean up the section that maps style IDs in references and numberings so that it doesn't have to go through as many elements.

@kimsey0,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Deploying to pages yields blank page

Hi,
I created the project with vue-cli. 'npm run dev' has worked flawlessly. Wanted to test a version of the app on my hosting server and my github pages.But the pages is blank.Before this,I have modified config->index.js->assetsPublicPath: './',There no any error in the console.please help me thanks

hi,boy
I guess,you may Using the history mode in your router,please remove it,everything is ok!
I'm looking for the answer,
WHY:Using history mode renders blank, but no errors.

error:cant find python executable...
请问什么是node-gyp呢，用来干什么的?
api会设计跨域吗？

[vue-router] missing param for named route "SubjectView": Expected "id" to be defined

@UNDERCOVERj
Hello, thanks for opening an issue with me.
Would you mind if I move your issue to a new place?

Hi,
You can vist the doc to learn more.
HTML5 History Mode


Update CSS to be more aligned to k/website

Small CSS changes for now to improve overall look. Will require revisiting to update scss etc. If old fonts etc are better, can just close out the PR.
/cc @castrojo @parispittman

/cc @castrojo @parispittman
didn't seem to pick up the cc when I edited it
¯\(ツ)/¯

/lgtm
/approve


Documentation and Readme update

### Tasks
- [ ] https://github.com/microsoft/autogen/issues/548

i will do it

Thanks. Please make sure @BeibinLi is added as a reviewer.

Thanks @sonichi , will open a PR soon

Has a PR been made?

https://github.com/microsoft/autogen/issues/548#issuecomment-1799314498
do i have to provide both links in the docs including https://github.com/microsoft/autogen/blob/main/notebook/agentchat_lmm_llava.ipynb ...
and https://microsoft.github.io/autogen/docs/Examples/AutoGen-AgentChat
does not exist
did you refer https://microsoft.github.io/autogen/docs/Examples/AgentChat.md

https://microsoft.github.io/autogen/docs/Examples/AgentChat.md is the right URL.
Yes, every link appears in both places - not ideal but that's the current way it is.

@sonichi
have made the PR ...please check that out #646


Add connection relevant configurations to db module

[#156492189]

LGTM

I made offline comments, and now it is LGTM


calico-node container reboot missed the serviceaccount token update event

calico-node container reboot missed the serviceaccount token update event and cannot update /host/etc/cni/net.d/calico-kubeconfig. This leads to following calico cni error.
2023-01-16 05:06:47.002 [ERROR][48795] plugin.go 121: Final result of CNI ADD was an error. error=error getting ClusterInformation: connection is unauthorized: Unauthorized
2023-01-16 05:06:47.111 [ERROR][48860] plugin.go 518: Final result of CNI DEL was an error. error=error getting ClusterInformation: connection is unauthorized: Unauthorized
Expected Behavior
calico cni should access apiserver successfully after calico-node container reboot and no need to wait for the next token refresh event.
Current Behavior
calico cni cannot access apiserver until the next token refresh event which is about 1 hour by default.
Possible Solution
calico-node container boot process should try to update  /host/etc/cni/net.d/calico-kubeconfig as soon as possible。
Steps to Reproduce (for bugs)
1.scale Typha replicas to be zero
2.calico-node container begins to reboot because of Typha access error.
3.kubelet will try to reboot calico-node container with an exponential back-off delay
4.until the serviceaccount token is updated in host. ("inspect" container to find the mounted host directory)
5.restore Typha and let calico-node reboot successfully
6.compare /etc/cni/net.d/calico-kubeconfig with the serviceaccount token in host directory.
Context
Our cluster depends on OpenYurt to implement node autonomy. When node autonomy is enabled, the Pods in this node won't be rescheduled and still keep running. And if node network is down, the calico-node container will reboot because of Typha access error. After node network is recovered, calico-node becomes running but cni isn't allowed to access apiserver. This leads to the CNI ADD or DEL error.
Your Environment

Calico version: v3.21.4
Orchestrator version (e.g. kubernetes, mesos, rkt):  k8s 1.22
Operating System and version: CentOS 8.4

The good news is that this looks very much like another issue already on our radar which is currently being worked on: https://github.com/projectcalico/calico/issues/7171
The bad news is that the fix will not be ported into the next maintenance release of v3.21 since that's outside of our support window (current - 3).
Going to close this issue but please do take a look at the issue I linked and if you think what you're seeing is sufficiently different and warrants a separate look we can re-open and evaluate from there.


useTheme must be called from inside a setup function

I have a vuetify 2 project I am trying to migrate to v3.
In v2 I was using vuetify-confirm
In my v3 project I want to use vuetify-use-dialog in a plugin like this (similiar to vuetify-confirm):
this.$confirm(someProps).then(res => {  // .. Do stuff })
But I am getting this error:
Error: [Vuetify] useTheme must be called from inside a setup function
Is it possible to achieve this functionality or is there an alternative way to implement it?

+1 I'm facing this issue when I want to call useSnackbar from an Axios interceptor. or even when wraping it with a service call so I can DRY similar confirmations

+1 I'm facing this issue when I want to call useSnackbar from an Axios interceptor. or even when wraping it with a service call so I can DRY similar confirmations

@RoduanKD composables should be used inside a setup function

@torerik123 added Options API support in v0.4.0.
<script lang="ts">
export default defineComponent({
  methods: {
    async handleConfirm() {
      const theme = 'dark'

      const isConfirmed = await this.$confirm({ content: 'This action is permanent!', theme })

      if (!isConfirmed)
        return

      this.$toast({ text: 'Confirmed', theme })
    }
  }
})
</script>

You can use this to augment $toast and $confirm:
import { useConfirm, useSnackbar } from 'vuetify-use-dialog'

declare module 'vue' {
  interface ComponentCustomProperties {
    $confirm: typeof useConfirm
    $toast: typeof useSnackbar
  }
}


Figure 2 is incomplete

Figure 2 does not indicate which icon is User A and which is User B.  Flows 2 & 4 do not connect to any internal users.

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

ID: 43c84773-337e-9df7-7fbd-fc9d94280711
Version Independent ID: 223a78a0-67fd-8b96-be3f-153bb400101b
Content: Microsoft Teams call flows
Content Source: Teams/microsoft-teams-online-call-flows.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@fshields thank you for pointing this out and helping improve MS Docs. @CarolynRowe could you possibly help with the correct image please, and I'll have it uploaded.

@CarolynRowe

@fshields Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Add cancel method to the AgentCheck base class, allowing cleanup of resources when checks are unscheduled.

What does this PR do?
This is called by the agent when a check is unscheduled
Motivation

Additional Notes

This is linked to https://github.com/DataDog/datadog-agent/pull/7279
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

For the failing TokuMX job, it's unrelated to these changes, see #8617.

@florimondmanca For the RFC I'm not sure we had one. We implemented this for go check running on k8s and it's the equivalent for go. It come up because more and more python checks asked for it (for long running thread and stuff).

For reference, there was some discussion around “resource cleanup” for the SNMP check in particular ~1yr ago. At the time Thomas looked into using __del__() (when check is garbage collected). Then we found a workaround in which the thread returns when the weakref for the check goes broken. See #6484.
Using this PR, I guess the SNMP check could be simplified so that we use a threading.Event to exit the worker thread, rather than the weakref hack. cc @AlexandreYang

Using this PR, I guess the SNMP check could be simplified so that we use a threading.Event to exit the worker thread, rather than the weakref hack

See AC-156 for more (internal) context, but overall this was precisely the use case that originally motivated this feature. FYI the PR that implemented this for Go checks is https://github.com/DataDog/datadog-agent/pull/6636 for reference (used for the KSM v2 integration), and vsphere was also tracked as a python integration that might benefit from this.

@florimondmanca Thanks for the heads-up.

For reference, there was some discussion around “resource cleanup” for the SNMP check in particular ~1yr ago. At the time Thomas looked into using del() (when check is garbage collected). Then we found a workaround in which the thread returns when the weakref for the check goes broken. See #6484.

Curious, what was the issue with using __del__() ?

Using this PR, I guess the SNMP check could be simplified so that we use a threading.Event to exit the worker thread, rather than the weakref hack.

Create some tickets to track this (II-197, II-198).

@AlexandreYang

Curious, what was the issue with using __del__() ?

I'll admit I'm not sure anymore (cc @therve?). But I think we ended up being limited in the kind of cleanup we could do, and it would have been even hackier to do it that way. (Eg if cleaning up the worker thread required finishing the last round of SNMP queries, then __del__ would take a fairly long time to execute. The weakref approach means you can do whatever you'd like without being rushed by the GC workflow.)

Some further context I dug up, thought it might be useful to others:
This PR is mostly bringing up Python checks up to speed with the Agent: the .Cancel() was added for Go checks 4mo ago via https://github.com/DataDog/datadog-agent/pull/6636.
It is already used by kubernetes_state (Go check):
https://github.com/DataDog/datadog-agent/blob/80e72b433ef87180a26e88f827334c8777b66feb/pkg/collector/corechecks/cluster/kubernetes_state.go#L249-L253
And currently in TODO for Python checks (Agent PR DataDog/datadog-agent#7279 cited in PR description fills that gap):
https://github.com/DataDog/datadog-agent/blob/70b1dc53407547d13ad7dbf60b1f0d73846f9c43/pkg/collector/python/check.go#L114-L116

@AlexandreYang

Curious, what was the issue with using __del__() ?

I'll admit I'm not sure anymore (cc @therve?). But I think we ended up being limited in the kind of cleanup we could do, and it would have been even hackier to do it that way. (Eg if cleaning up the worker thread required finishing the last round of SNMP queries, then __del__ would take a fairly long time to execute. The weakref approach means you can do whatever you'd like without being rushed by the GC workflow.)

Using __del__ breaks Python detection of cycles during garbage collection. You have to make sure you're not involved in a cycle (which is hard, especially in our integration spaghetti code), otherwise your object ends up in uncollectable garbage, which is the opposite of what we want to do.


"Sort by" filter displays different on search page

Comparing to filter on Store and Transactions page: title (font color, font size, "by" vs "By") and options text ("(Low to High)" vs "low to high").

The sort by options in the Discover section are generated by 3rd party search providers. We can't enforce what options 3rd parties provide, or how they structure the text @rhcastilhos.
However, let's ensure the actual Sort by text next to the input is consistent, as far as color and letter-casing goes. Dark text with uppercase S and lowercase b sound good to me. It should display as Sort by
On the store page, I would also prefer the following changes to the options:
Price (Low to High)
Price (High to Low)
Title (A-Z)
Title (Z-A)
Again, we can't ensure these will stay consistent across search engines, but seems like these low effort changes will help.


Method .matrix() in module lt.py returns incorrect matrices; suggested correction; method .det() returns incorrect determinant

Let f be an outermorphism on a geometric algebra Ga.  The matrix of f with respect to a basis b_1, ..., b_n is defined by f(b_j) = \sum_{i=1}^n f_{ij} b_i.  The definition can be found in virtually any introductory linear algebra book, so call [ f_{ij} ] the standard matrix of f with respect to the indicated basis.  The coefficients f_{ij} in the expansion of  f(b_j) do not depend on the metric, although they can be computed using the inner product and the reciprocal basis vectors b^1, ..., b^n.
(Note in particular that the left index on f_{ij} is the one summed over.  GAlgebra's documentation, if it can be trusted, writes the defining equation as L(b_i) = \sum_{j=1}^n L_{ij} b_j, with the summed-over index on the right.  This would have the effect of defining a matrix [ L_{ij} ] which is the transpose of L's standard matrix.)
Problem #1:  I have discovered that the method .matrix(), the code for which appears in the module lt.py, returns a matrix which iis metric dependent and so cannot be the standard matrix.  The command f.matrix() returns the standard matrix only when

the inner product is Euclidean, and
the basis used is orthonormal.
I've also discovered that post-multiplication of f.matrix() by Ga.g_inv produces the standard matrix for f with respect to the basis used to generate Ga.

As evidence I am attaching a file Linear Transformations.zip.  It contains six Jupyter notebooks, which parallel each other in format, that examine the situation using different signature inner products and different bases.  It also contains gprinter.py, a new module from Alan Bromborsky, that aids the construction of formatted output and which was used by the notebooks mentioned.  Lastly, the zip file contains a notebook g2.ipynb which is easier to examine than the other six notebooks.
Suggested Solution for Problem #1:  The next-to-last line of the code for the .matrix() method reads
    `self.mat = Dictionary_to_Matrix(self.lt_dict, self.Ga) * self.Ga.g`

While I have only rudimentary programming skills, it seems to me that simply replacing that line by
    `self.mat = Dictionary_to_Matrix(self.lt_dict, self.Ga)`

will correct the problem with the transformation method .matrix().  The idea is that the multiplication of the incorrect matrix by Ga.g_inv effectively cancels out the multiplication by self.Ga.g.
Problem #2:  If one applies the method .det() to an outermorphismf, strange things can happen.  Look at the notebooks, other than g2.ipynb, in the attached zip file.  Towards the end of each notebook one will see f.det() returning incorrect answers.  Sometimes the correct answer is returned, sometimes a pseudoscalar is returned instead of the scalar which should be returned, and in one instance (hyperbolic plane, polar coordinates.ipynb) an answer is returned which multiplies the pseudoscalar by SymPy's unit imaginary i (geometric algebra doesn't use imaginary numbers, per se).
Solution for Problem #2:  I have none.  But I suspect the appearance of the unit imaginary arises with an incorrect computation of the unit pseudoscalar Ga.I().  That in turn may be linked to incorrect computation of a blade's norm.  (I plan to post separately with regards to a multivector's norm-squared and norm.)
Greg Grunberg
Addendum:  The Suggested Solution for Problem #1, described above, sort of worked.  I made the change indicated in the suggestion.  Afterwards f.matrix() would returned an answer in the form of a matrix with a transposition T as a superscript.  Upon carrying out the indicated transposition, the desired standard matrix was obtained.  I then restored the code for the method .matrix(), and then wrote a new method as follows:
def std_matrix(self) -> Matrix:
    r"""
    Returns the standard matrix representation :math:`\left[ {{f}^i}_{j} \right]` of
    the linear transformation :math:`f`, with entries defined by the equations
    :math:`{{f}\lp {{{\eb}}_{j}} \rp} = {{\eb}}_{i} {{f}^i}_{j}`.
    """
    standard_matrix = self.matrix() * self.Ga.g_inv
    return standard_matrix

Testing f.std_matrix() for all outermorphisms examined in then notebooks contained in the zip file, in each case the metric-independent standard matrix was returned.
Linear Transformations.zip

The zip file I attached to my original post contained within it a file Euclidean 3-space, spherical coordinates.ipynb with some errors.  The attached file corrects those errors.
Euclidean 3-space, spherical coordinates.zip

Thanks for the report. Here I think is a simpler reproduction (which is built on top of #454):
>>> from galgebra.ga import Ga
>>> g2 = Ga('e_x e_y', g=[1,1])
>>> g2a = Ga('e_x e_y', g=[[1, 0], [0, 2]])
>>> g2b = Ga('e_x e_y', g=[[0, 1], [1, 0]])

>>> g2.lt([[0, 2], [1, 1]]).matrix()  # note: this crashes until #454 is in
Matrix([
[0, 1],
[2, 1]])

>>> g2a.lt([[0, 2], [1, 1]]).matrix()
Matrix([
[0, 2],
[2, 2]])

>>> g2b.lt([[0, 2], [1, 1]]).matrix()
Matrix([
[1, 0],
[1, 2]])

to me, it looks like lt(list) is uniformly broken. You get what looks to me like correct behavior if you use lt(Matrix) instead:
>>> from sympy import Matrix
>>> g2.lt(Matrix([[0, 2], [1, 1]])).matrix()
Matrix([
[0, 2],
[1, 1]])
>>> g2a.lt(Matrix([[0, 2], [1, 1]])).matrix()
Matrix([
[0, 2],
[1, 1]])
>>> g2b.lt(Matrix([[0, 2], [1, 1]])).matrix()
Matrix([
[0, 2],
[1, 1]])

I think this only scratches the surface though. #158 is related.

#456 is also related.

You say in your notebooks:

disregard the L on the left side of the equations; each pair of equations corresponds to a different transformation):

Do you have a better suggestion for how to display the transformation without the confusing L?

Hi Eric,
Suppose one has a function [image: s] such that [image: s(x) = x^2].  One
doesn't actually have to give the function a name [image: s] to be able to
refer to the function.  Instead just refer to the function [image: x
\mapsto x^2] , read "[image: x] maps to [image: x^2] ".   No name [image: s] is
needed.  This practice is well established in mathematical literature.
Here's an example relevant to the GAlgebra module lt.py.  Let [image: T] be
a linear transformation on a space with basis , represented in GAlgebra by
the identifier T.  Currently, when one executes a simple T in a Jupyter
notebook In[ ] cell, the corresponding Out[ ] cell will contain output
looking something like this:
[image: \left { \begin{array}{ll} L \left ( \mathbf{e}_x\right ) =& 2
\mathbf{e}_y \ L \left ( \mathbf{e}_y\right ) =& \mathbf{e}_x +
\mathbf{e}_y \end{array} \right }]
This looks like one is applying a transformation named [image: L], but the
actual name was [image: T].
Instead one could simply output a description of [image: T]'s action that
looks like this :
[image:  \left { \begin{array}{ll} \mathbf{e}_x \mapsto & 2 \mathbf{e}_y
\  \mathbf{e}_y \mapsto & \mathbf{e}_x + \mathbf{e}_y \end{array} \right
}]
I've seen "mapsto" notation used in SymPy (or maybe it was in SageMath),
but I don't remember the details.
Greg Grunberg
On Mon, Sep 14, 2020 at 11:41 AM Eric Wieser<EMAIL_ADDRESS>wrote:

You say in your notebooks:
disregard the L on the left side of the equations; each pair of equations
corresponds to a different transformation):
Do you have a better suggestion for how to display the transformation
without the confusing L?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pygae/galgebra/issues/453#issuecomment-692175761, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2OCIF6EYSELBHGA3BJOPTSFZBUDANCNFSM4RIVJDKA
.

Let    a_list_of_lists = [ [a,b], [c,d] ]    and consider the SymPy matrix
created by the command    Matrix(a_list_of_lists) .  It will be a matrix
whose first row is  a b and whose second row is  c d .  In general, the
ith entry (itself a list) in  a_list_of_lists  specifies the ith row
of the matrix.
If one is already familiar with SymPy's  Matrix() constructor, one may be
inclined to misinterpret how  a_list_of_lists   is interpreted in the
GAlgebra command   T = GA.lt(a_list_of_lists) .  (I made that mistake.)
The [image: j]th entry  [image: [ {T^1}_j, \ldots, {T^n}_j ]]  in
a_list_of_lists  is interpreted as the coefficients in the expansion
[image: T(b_j) = b_1 {T^1}_j + \cdots + b_n {T^n}_j]
of the image of the [image: j]th basis vector.  Those coefficients
form the [image:
j]th column of the standard matrix of [image: T] with respect to
basis [image:
b_1, \ldots, b_n] .  So the [image: j]th entry in   a_list_of_lists  gives
the [image: j]th column of [image: T]'s standard matrix, not its [image:
j]th row.  The standard matrix will be the transpose of
Matrix(a_list_of_lists) .
It is important to understand that unless the basis is orthonormal and the
metric is Euclidean, the command  T.matrix()  does not return the standard
matrix of [image: T] with respect to basis [image: b_1, \ldots, b_n] .  I
have not so far figured an interpretation of  T.matrix(), but, as
previously posted, the matrix product  T.matrix() * T.Ga.g_inv  appears to
yield the standard matrix of [image: T].   T.matrix() is dependent on the
metric, the standard matrix of [image: T] is not.
I was not aware that the constructor  GA.lt( ) accepted a SymPy matrix as
an argument.  Preliminary investigation indicates that  S =
GA.lt(Matrix(a_list_of_lists))  instantiates a transformation which has a
different action from T = GA.lt(a_list_of_lists).  That preliminary
investigation shows that S.matrix() does indeed equal
Matrix(a_list_of_lists) , but it's still not clear how that common matrix
relates to the action of [image: S] on the basis vectors.
On Mon, Sep 14, 2020 at 4:47 AM Eric Wieser<EMAIL_ADDRESS>wrote:

Thanks for the report. Here I think is a simpler reproduction (which is
built on top of #454 https://github.com/pygae/galgebra/pull/454):

from galgebra.ga import Ga>>> g2 = Ga('e_x e_y', g=[1,1])>>> g2a = Ga('e_x e_y', g=[[1, 0], [0, 2]])>>> g2b = Ga('e_x e_y', g=[[0, 1], [1, 0]])
g2.lt([[0, 2], [1, 1]]).matrix()  # note: this crashes until #454 is inMatrix([
[0, 1],
[2, 1]])
g2a.lt([[0, 2], [1, 1]]).matrix()Matrix([
[0, 2],
[2, 2]])
g2b.lt([[0, 2], [1, 1]]).matrix()Matrix([
[1, 0],
[1, 2]])

to me, it looks like lt(list) is uniformly broken. You get what looks to
me like correct behavior if you use lt(Matrix) instead:

from sympy import Matrix>>> g2.lt(Matrix([[0, 2], [1, 1]])).matrix()Matrix([
[0, 2],
[1, 1]])>>> g2a.lt(Matrix([[0, 2], [1, 1]])).matrix()Matrix([
[0, 2],
[1, 1]])>>> g2b.lt(Matrix([[0, 2], [1, 1]])).matrix()Matrix([
[0, 2],
[1, 1]])

I think this only scratches the surface though. #158
https://github.com/pygae/galgebra/issues/158 is related.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pygae/galgebra/issues/453#issuecomment-691945593, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2OCID54LI6Y3Y2PHBFONDSFXRCXANCNFSM4RIVJDKA
.

Thanks @Greg1950, \mapsto sounds totally reasonable to me.


Add support zip packing function code for Lambda deployment

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #2551
Does this PR introduce a user-facing change?:

Add ability to deploy Lambda zip packing function code which stored in AWS S3

Lets merge this to support S3 and make another support for deploying Zip directly.
/lgtm

/lgtm

/approve


Removing a print in the pipeline steps

This is an aesthetic change. There was a badly formatted row of "=" we are just removing this so it looks cleaner.

I think this is a good stylistic change because it removes an unnecessary and short "==" print underneath the pipeline steps.


Changed credential fetching to use AWS config

We'll instruct users to use AWS config to initialize their credentials
and read the credentials from the AWS file.
Resolves submariner-io/cloud-prepare#29
Signed-off-by: Mike Kolesnik<EMAIL_ADDRESS>

@mkolesnik this is conflicting now

oh yikes, it's conflicting


Use librdkafka 0.9.2

Upgrades the underlying librdkafka to 0.9.2. This removes the copy of rd_kafka_msg_partitioner_consistent which is now included. It also changes the use rd_kafka_set_logger, which was deprecated, to rd_kafka_conf_set_log_cb.

The build is broken because activesupport 5.0.1 is not compatible with the jRuby versions used. Any ideas how we should fix this?

Any feedback on this?

Is there anybody still working on this project who could give some feedback?
Is upgrading Hermann to use the high level consumer and so on something people are interested in at all? I'm thinking it might be a better idea if I create a new library using Hermann's compilation setup and the ffi gem which is focussed purely on supporting Kafka 0.10+.

@thijsc I hadn't bothered looking at this because it breaks JRuby support. I don't really have a keen idea right now other than to complain upstream about that :frowning:
If you're interested in helping contribute/maintain, I can happily add you to the organization

Actually this does not break JRuby support. The build is broken because activesupport 5.0.1 is not compatible with the jRuby versions used. The same build error occurred in some other pulls.
I'd be interested in contributing and helping for sure, but definitively don't feel confident making decisions to merge things. Are you interested in the high level consumer yourself? Would you be open to make backwards incompatible changes to make this happen?

Closing, looks like there is no interest in this pull.


Update dependencies and cleanup

This pull request updates quite outdated dependencies:

combine
hash32

Also some typo fixes and cleanup.

We are usually careful with updating dependencies, if there is not one specific reason (e.g. needed feature or big performance gain) to do so. Otherwise they potentially experience some feature creep over time and get worse performance wise or introduce new security issues.

Codecov Report

Merging #369 (41fa9bb) into main (8d95c0e) will not change coverage.
The diff coverage is 92.59%.

@@           Coverage Diff           @@
##             main     #369   +/-   ##
=======================================
  Coverage   89.91%   89.91%
=======================================
  Files          24       24
  Lines        9019     9019
=======================================
  Hits         8109     8109
  Misses        910      910

Impacted Files
Coverage Δ

src/lib.rs
76.92% <ø> (ø)

src/memory_region.rs
67.79% <ø> (ø)

src/static_analysis.rs
68.27% <ø> (ø)

src/jit.rs
93.49% <84.61%> (ø)

src/asm_parser.rs
99.23% <100.00%> (ø)

src/ebpf.rs
79.59% <100.00%> (ø)

src/elf_parser_glue.rs
66.02% <100.00%> (ø)

src/vm.rs
80.06% <100.00%> (ø)

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

Ok. I see.


Link of the dataset can not accessed

I got 404 on http://www.tuni.fi/vga/bmfr. Could you please tell me the new website of the dataset?

I found it, sorry.
https://etsin.fairdata.fi/dataset/0ab24b68-4658-4259-9f1d-3150be898c63/data


[Question] How to use ENC placeholder as sub-string in property value

Use case:
secret.password=ENC(sdfsdfds...)
<EMAIL_ADDRESS>Now secret URL when accessed, looks like it is NOT decrypting.
But when i tested secret.url by inject property in java bean, it shows right values.
I am trying to build spring.cloud.config.uri in spring boot config client app, without exposing actual password.

There's a limitation due to the way spring boot resolves properties for application.yml or application.properties.
There are 2 workarounds:

encrypt the entire URL: secret.url=ENC(sdfsdfds...)
Move secret.password and secret.url to a different property files together with:

@EncryptablePropertySource("classpath:encrypted.properties")

And you'll be able to make that indirection.
There's a working sample here: https://github.com/ulisesbocchio/jasypt-spring-boot-samples/tree/master/jasypt-spring-boot-demo-simple

Thanks. I will try this.

as of 1.15 (which I just released) this should work out the box using either jasypt-spring-boot-starter, @EnableEncryptableProperties, StandardEncryptableEnvironment or on custom property sources using @EncryptablePropertySource.
Enjoy!


Can't load PNG via URL when ".png" is missing (Windows, Neko)

This code crashes at "loader.load(request)" when the target is set to Windows or Neko:
	var request = new openfl.net.URLRequest("https://dl.dropboxusercontent.com/u/10650964/_trash/example1");
	request.contentType = "image/png";
	var loader = new openfl.display.Loader();
	loader.load(request);

The file in the URLRequest exists and really is a PNG. Just add ".png" to the URL to confirm - if I use this file instead, the code works. Both files work with the Flash target.
The problem might be that OpenFL seems to need ".png" as suffix, otherwise it fails.

This is fixed :smile:


Validators refactoring

Note that this must be merged only after #281.
Changes proposed in this pull request:

Drop unused attribute from ResponseBodyValidator
Use Draft4Validator instance directly instead of validation() shorthand
Allow user to override default Validator used to validate request and response body, thanks to this, it is easier to customize existing Validators

Sorry, I totally missed https://github.com/zalando/connexion/pull/281.
I think we can use first two patches from this pull request and maybe also slightly changed examples and cookbook.

Coverage remained the same at 100.0% when pulling c1659b86fc36b0ddeed6c32041913d927b6587e2 on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

@phoracek Would you be willing to write a new PR for documenting the #281 and maybe changing something you think would fit some additional idea from your proposal? Sorry, I didn't had time to reply to you on #351 before you started coding your PR.

Coverage remained the same at 100.0% when pulling 8bfada2d3d1702e081c4b2dc7182c2da1ae73023 on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>68dc93ce0fb926cfc90c6b19bae7da on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

@rafaelcaricio I pushed promised changes, but bear in mind, that it needs to be merged after your PR. What should I do now? Rebase this PR on your, or wait for yours to be merged or rebased on master?

Coverage remained the same at 100.0% when pulling 60895a4a083384115e23c8dc31703d64295d50af on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

Coverage remained the same at 100.0% when pulling a7a1573a4a455e7abcb784227c72ce6df86254ba on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

Coverage remained the same at 100.0% when pulling a7a1573a4a455e7abcb784227c72ce6df86254ba on phoracek:custom_validator into fbcaf3898e1ab36a70b4c4371b570893278ffc7d on zalando:master.

I rebased original custom validators PR #281  on top of master and set my on top of that. Hope it helps.

Coverage remained the same at 100.0% when pulling 463f745deb5d506b79371d15c28f0a30b966887d on phoracek:custom_validator into 675039d47fa90e8f422d18a614eced9acb6901d2 on zalando:master.

@phoracek Thank you very much for the work in this PR. It looks good to me.
We just have to isort the imports, applying this patch:
--- /home/travis/build/zalando/connexion/examples/enforcedefaults/enforcedefaults.py:before	2016-12-03 17:44:15.006521
+++ /home/travis/build/zalando/connexion/examples/enforcedefaults/enforcedefaults.py:after	2016-12-03 17:47:19.644362
@@ -1,9 +1,9 @@
 #!/usr/bin/env python3

-from connexion.decorators.validation import RequestBodyValidator
 import connexion
 import jsonschema
 import six
+from connexion.decorators.validation import RequestBodyValidator

Thanks Rafael :) I resolved the isort problem. Example works, so I hope it won't break anything.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>ac61b3c4401eb201d15ceff134178c on phoracek:custom_validator into 675039d47fa90e8f422d18a614eced9acb6901d2 on zalando:master.

👍

Coverage remained the same at 100.0% when pulling 45446a6f368d7dd246956bc67121b477f84dc1e6 on phoracek:custom_validator into 675039d47fa90e8f422d18a614eced9acb6901d2 on zalando:master.

👍

Coverage remained the same at 100.0% when pulling e166d0fea5744acf15c2afa532c7ef5a75a040df on phoracek:custom_validator into 675039d47fa90e8f422d18a614eced9acb6901d2 on zalando:master.


Add profile page

Add profile page,
Must contain the following:

[x] Picture of person

[x] Name of person

[x] Function of person

[x] Bio

Added in #19


Download fails when behind SSL inspection corporate proxy

Installation fails due to UNABLE_TO_GET_ISSUER_CERT_LOCALLY.  I am behind a corporate proxy that does SSL inspection (with our own certificate authority), so I am not unused to this type of error.  However, I have configured npm correctly:
PS C:\Users\ltheisen\git\facility-report-viewer-ui> npm config get cafile
C:\cygwin64\etc\pki\ca-trust\extracted\pem\tls-ca-bundle.pem

Along with the proper proxy settings:
PS C:\Users\ltheisen\git\facility-report-viewer-ui> gci env: | findstr proxy
http_proxy                     http://gatekeeper-w.mitre.org:80
https_proxy                    http://gatekeeper-w.mitre.org:80
no_proxy                       .mitre.org,.portal.local,localhost,mm206678-pc,mm206678-pc.mitre.org,<IP_ADDRESS>

And it works for normal installs.  It seems like your install.js respects the https_proxy environment variable, but ignores the cafile setting.  I'm not really surprised as it is an npm config and your code is built to work with npm/yarn/..., but I need a way to provide your installer with my CA file.
Steps to reproduce
Tell us about your environment:

Puppeteer version: 0.13.0
Platform / OS version: Windows 10 Enterprise
URLs (if applicable):
Node.js version: v9.3.0

What steps will reproduce the problem?
Please include code that reproduces the issue.

npm install --save-dev puppeteer

What is the expected result?
Puppeteer gets installed.
What happens instead?
PS C:\Users\ltheisen\git\facility-report-viewer-ui> npm install puppeteer --save-dev

><EMAIL_ADDRESS>install C:\Users\ltheisen\git\facility-report-viewer-ui\node_modules\puppeteer
> node install.js

ERROR: Failed to download Chromium r515411! Set "PUPPETEER_SKIP_CHROMIUM_DOWNLOAD" env variable to skip download.
{ Error: unable to get local issuer certificate
    at TLSSocket.onConnectSecure (_tls_wrap.js:1042:34)
    at TLSSocket.emit (events.js:159:13)
    at TLSSocket._finishInit (_tls_wrap.js:639:8) code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' }
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Hi team,
I am recently trying to download puppeteer on my machine. And I am facing this issue
Installation fails due to UNABLE_TO_GET_ISSUER_CERT_LOCALLY. I got to see this discussion.
when I tried to run the command npm install --save puppeteer@next I am facing the below issue. Do we have any idea on this?.
OS: windows 11
Node js version: 18.16.0
npm ERR! code ETARGET
npm ERR! notarget No matching version found for puppeteer@next.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.


[injected-wallets] : feat - add Exodus wallet

Description
Adds support for Exodus Browser Extension
Checklist

[x] The version field in package.json is incremented following semantic versioning
[x] The box that allows repo maintainers to update this PR is checked
[x] I tested locally to make sure this feature/fix works
[ ] This PR passes the Circle CI checks

@bulgakovk Hey! Could you update this PR with latest develop? Then we can get it merged in 👍🏾

Hey sure thing! Just rebased on the latest v2-develop branch


find_package(share, but port isn't called 'share'?

Describe the bug
#24410 on install it says:
Detecting compiler hash for triplet x64-osx...
The following packages will be built and installed:
    c-str-span[core]:x64-osx -> 0.0.1
  * vcpkg-cmake[core]:x64-osx -> 2022-04-12
  * vcpkg-cmake-config[core]:x64-osx -> 2022-02-06
Additional packages (*) will be modified to complete this operation.
Restored 3 packages from /Users/samuel/.cache/vcpkg/archives in 29.7 ms. Use --debug to see more details.
Starting package 1/3: vcpkg-cmake-config:x64-osx
Installing package vcpkg-cmake-config[core]:x64-osx...
Elapsed time for package vcpkg-cmake-config:x64-osx: 7.529 ms
Starting package 2/3: vcpkg-cmake:x64-osx
Installing package vcpkg-cmake[core]:x64-osx...
Elapsed time for package vcpkg-cmake:x64-osx: 5.639 ms
Starting package 3/3: c-str-span:x64-osx
Installing package c-str-span[core]:x64-osx...
Elapsed time for package c-str-span:x64-osx: 11.29 ms

Total elapsed time: 3.507 s

The package c-str-span provides CMake targets:

    find_package(share CONFIG REQUIRED)
    target_link_libraries(main PRIVATE c_str_span c_str_span_compiler_flags)

Environment

OS: macOS
Compiler: revision

To Reproduce
Steps to reproduce the behavior:

./vcpkg install c-str-span
See error

https://github.com/SamuelMarks/c-str-span
https://github.com/offscale/vcpkg/tree/c-str-span/ports/c-str-span
Expected behavior
A clear and concise description of what you expected to happen.
Failure logs
-(please attached failure logs)
Additional context
Add any other context about the problem here.

Cf. #20190.


A couple of patches

Fix Xcode 7 warning for designated initializers
Fix example project (button wasn't showing up because of missing constraints)
Enable missing functionality in example project
Fix bug where setting title/message after controller creation wouldn't work (#22)

Updated according to your comments. :+1:


Changing copy to clipboard button to an input + button

@jwforres, @spadgett, PTAL.

This looks a lot better to me.

I want to consider a followup to this, should we be visually treating the commands the same way we treat URLs

I want to consider a followup to this, should we be visually treating the commands the same way we treat URLs

It looks like there are 5 instances where we show commands:

The About page instance is unique in it's markup (it uses <pre> instead of <code>), and it contains a "...click to show token..." link to obfuscate the token.

5 instances today :)   but in the discussions over the last couple days i
think we need to do a better job of helping people do things on the CLI
that we can't do in the console (ex: some of the binary deploy options)
On Thu, Apr 14, 2016 at 3:42 PM, Robb Hamilton<EMAIL_ADDRESS>wrote:

I want to consider a followup to this, should we be visually treating the
commands the same way we treat URLs
It looks like there are 5 instances where we show commands:
[image: screen shot 2016-04-14 at 3 31 48 pm]
https://cloud.githubusercontent.com/assets/895728/14541254/08d932dc-0257-11e6-8513-c7b4f81f8da7.PNG
[image: screen shot 2016-04-14 at 3 32 19 pm]
https://cloud.githubusercontent.com/assets/895728/14541260/0c8eb870-0257-11e6-8c30-2ca847fb2b07.PNG
The About page instance is unique in it's markup (it uses  instead of
), and it contains a "...click to show token..." link to obfuscate
the token.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwforres/origin/pull/50#issuecomment-210117053

@rhamilto If you update the max-width, are we ready to merge? I think the CLI help can be a follow on.

@spadgett, thanks for the heads up.  I overlooked @jwforres' comment about reducing the max-width to 300px, which I've now done.  I think we should be good to merge.


html2js-preprocessor adding relative template url

My project folder is at var/www/apps/angular/myproject
And the templates are in the templates folder under myproject.
So the template url is specified in the directive is "/template/directive.html"
So in the karma config I've written
ngHtml2JsPreprocessor: {
stripPrefix: 'var/www/apps/angular/myproject',
moduleName: "my.templates"
},
And this config works. But I don't want to specify the full path to the project in the config because for some other environment this might change. But "myproject" folder will be always there.
So I tried with stripPrefix: '*/myproject',
But it didn't work.
Please tell me how I can fix it?

One way I got around this was to use Nodes path module.
I set this before the Karma configuration
var path = require('path');

Then set the basepath as
basePath: path.resolve(),

Might help until relative paths are available.

I set my base path to the root of the folder, './', and it fixed this issue.


Connectivity issue on iOS

After WiFi connection switching, gRPC returns Code 2 with description "Failed to create subchannel". To reproduce:

Start a stream call
Switch WiFi to another connection
Start another call

Expecting the second call would succeed, but actually failed with Code 2.

@michaelreneer

This should be resolved by CFStream integration in #15069


Orf live (real live, not the usual delayed live!)

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.02.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2019.02.18

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2019.02.18
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
https://github.com/rg3/youtube-dl/issues
C:\Users\alex>youtube-dl -v --geo-bypass-country AT
ive/AD-Nordische-Ski-WM-2019-Langlauf-Herren-30km-Skiathlon/14003505
debug] System config: []
debug] User config: []
debug] Custom config: []
debug] Command-line args: ['-v', '--geo-bypass-country', 'AT', 'https://tv
hek.orf.at/live/AD-Nordische-Ski-WM-2019-Langlauf-Herren-30km-Skiathlon/14
03505']
debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
debug] youtube-dl version 2019.02.18
debug] Python version 3.4.4 (CPython) - Windows-Vista-6.0.6002-SP2
debug] exe versions: ffmpeg 3.3.4
debug] Proxy map: {}
debug] Using fake IP <IP_ADDRESS> (AT) as X-Forwarded-For.
orf:tvthek] 14003505: Downloading webpage
orf:tvthek] 14003505: Downloading f4m manifest
RROR: 14003505: Failed to parse XML  (caused by ParseError('syntax error:
ine 1, column 0',)); please report this issue on
https://yt-dl.org/bug
. M
ke sure you are using the latest version; type  youtube-dl -U  to update.
e sure to call youtube-dl with the --verbose flag and include its complete
output.
raceback (most recent call last):
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 827, in _parse_xml
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\compat.py", line 2513, in compat_etree_fromstring
 File "C:\Python\Python34\lib\xml\etree\ElementTree.py", line 1335, in XML
File "", line None
ml.etree.ElementTree.ParseError: syntax error: line 1, column 0
raceback (most recent call last):
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 827, in _parse_xml
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\compat.py", line 2513, in compat_etree_fromstring
 File "C:\Python\Python34\lib\xml\etree\ElementTree.py", line 1335, in XML
File "", line None
ml.etree.ElementTree.ParseError: syntax error: line 1, column 0
uring handling of the above exception, another exception occurred:
raceback (most recent call last):
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\YoutubeDL.py", line 794, in extract_info
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 508, in extract
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\orf.py", line 94, in _real_extract
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 1445, in _extract_f4m_format
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 820, in _download_xml
 File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 804, in _download_xml_handle
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6
x\build\youtube_dl\extractor\common.py", line 831, in _parse_xml
outube_dl.utils.ExtractorError: 14003505: Failed to parse XML  (caused by
arseError('syntax error: line 1, column 0',)); please report this issue on
https://yt-dl.org/bug
. Make sure you are using the latest version; type
outube-dl -U  to update. Be sure to call youtube-dl with the --verbose fla
 and include its complete output.
If the live will end when you read my report you can try with another (real) orf live

404 not found.

the link works well in my browser:
https://tvthek.orf.at/live/Nordische-Ski-WM-2019-WM-Studio/14003416
https://postimg.cc/hh04PtX6
----- Original Message -----
From: Sergey M.
To: rg3/youtube-dl
Cc: avagraha ; Author
Sent: Saturday, February 23, 2019 2:42 PM
Subject: Re: [rg3/youtube-dl] Orf live (real live, not the usual delayed
live!) (#19310)
Closed #19310.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I confirm the link https://tvthek.orf.at/live/Nordische-Ski-WM-2019-WM-Studio/14003416     is still valid and working (geoblocked). Are you able to download it or do you experience the same issue as me? (Log above). Thanks. Notice the differencies real live vs. Reloaded live.


Extension not starting

I've installed the two extensions into the latest VS preview, then opened a rust project. I've also tried manually installing the RLS (currently only available via the beta channel).

The extension will start when you open a Rust file. You'll be prompted to install the RLS. Once installed, the RLS should start building your project.

.. but it doesn't.

Do you at least get the prompts to install the RLS? If you do and the RKS doesn't install then the extension is at least starting. If you do get those prompts then it sounds like the problem is that the RLS is currently unavailable on nightly. If you wish to use a different toolchain with this extension you need to set that option in Tools > Options > Rust.

I've set the extension to use the beta  channel. Any idea why it still exits?

So just to confirm,

The Tools>Options>Rust Toolchain to use setting is set to beta.
You get the installation prompt at the beginning or the necessary components are already installed

rls-preview
rust-analysis
rust-src

After this the extension still exits, is that correct? If so, try restarting VS first and making sure the setting stuck since I am unable to reproduce on any of my systems.

A few more questions:

Does your beta build call the RLS component rls-preview or rls?
How are you confirming that the extension exits?

Setting is correct (beta channel)
Components are indeed installed
I've installed rls-preview on the beta toolchain
I do not, I just assumed so since nothing happens.

If the components are already installed my extension won't pop up any notifications. It's main job is to launch the rls process and pass that on to the LSP extension. You can try looking in task manager to see if there is an rls.exe process running (If you are on windows 10 you may need to expand the process tree for visual studio). Tonight I'll add some logging to the extension and send out an update. I'll update this issue with instructions on how to read the logs once the update is out.
It is also possible that the RLS is crashing on your project. If you are working with a particularly large project create a new library via cargo and see what happens then.

Was using a helloworld project so that shouldn't be the problem. The process is not being started though.

Its ugly, but I've attached a debug version of the extension RustLanguageExtension.zip unzip that and install the .vsix file. You should now get a lot of infobars when you open a rust file. Please tell me which ones you see and which is the last one.

Could you push the changes (to a separate branch)? I'm trying to build the extension myself.

I could, I just figured that just providing a vsix was simpler than having you build the extension. Allthough if you build it yourself you can debug the OnLoadedAsync method to see whats going on.

branch https://github.com/dgriffen/rls-vs2017/tree/super-infobar-debug has the changes

I only see "extension has begun launching" (or something similar) pop up 🤔

I assume you pressed continue? Well that means that it is detecting all the components as installed, but it still isn't starting. This was something I didn't predict, let me add a few more info bars.

I'm trying to debug the extension but it says: "the breakpoint will currently not be hit, no symbols have been loaded for this document".

It won't bind until you actually load the extension via opening a rust file.

It seems I can't debug further than the Rustup.HasToolchain call, but I'm not experienced with debugging c# vs extensions. I can build it though, you don't need to bother uploading packages :)

Still only the first popup. Trying a popup at the end of OnLoadedAsync

Can you try debugging into HasToolchain? I think the problem might be an await command that is never exiting, since you couldn't debug past it.

That's what I'm thinking as well. I'm never getting past the await. The command should be working though.

RunCommand isn't even reached.

You are pressing continue on the infobars right? cause those block execution. If so, wat. The only call that could be blocking between the first infobar and RunCommand would be OptionsModel.Toolchain.

I am. Maybe VS's await debugger is broken in the preview? It generates a state machine so I could understand if debugging such a thing would break. I'll try adding print statements somewhere in between.

The await debugger is pretty rock solid, I haven't had issues with it when I'm debugging. There must be something really screwy going on. I'm guessing something is redirecting a dll my extension uses. (Either via GAC or a binding redirect). You could try removing the () tuples from Rustup.rs and replacing them with the Tuple class.

on loaded sync entered
got toolchain, awaiting rustup.hastoolchain
Exception thrown: 'System.IO.FileNotFoundException' in RustLanguageExtension.dll

Do you not have rustup on your path?

C:\Windows\System32>rustup --version
rustup 1.7.0 (813f7b7a8 2017-10-30)

NB I've added a debug println inside HasToolchain and that didn't print anything - so maybe the Rustup.cs didn't get compiled?

Rustup.cs must get compiled, you'd have much different errors if it wasn't included in the output binary. How is your rustup path set? It looks like you were running cmd.exe as admin since it doesn't default to the system32 directory otherwise. Perharps its only on the admin path?

My rustup path is set in my user path, and my user cmd defaults to sys32 for some reason. I've pushed my extra prints if you need them https://github.com/PvdBerg1998/rls-vs2017/commit/9b5c689f7615fd18368ea02cdee1b555fadba70d .

Did that FileNotFoundException have a stack trace or anything? Because at this point my guess is that you are missing some dll on the system. Since its happening when you hit Rustup.rs it could be System.ValueTuple, which is meant to be shipped with VS. Since I can't repro on your system I pushed an update to the infobar branch that remove the valuetuples.

This fixes the problem. (And it didn't have a stack trace btw.)
I do get a lot of System.Windows.Data Error: 5 : Value produced by BindingExpression is not valid for target property.; Value='<null>' BindingExpression:Path=IconAutomationText; DataItem='CompletionView' (HashCode=16558849); target element is 'CrispImage' (Name=''); target property is 'Name' (type 'String') spam though, but that might be a different issue.

System.ValueTuple really should be included with a VS installation, it sounds like yours got corrupted somehow. Try repairing it to see if that fixes it.

Hello.
I install LSP, extension but nothing happens when I open rust file.
No error or other messages is  provided.
I'm confused...
Win7x64

Did you open an individual file or an entire folder? The RLS and the LSP extension don't work with individual files.

I open a folder. Than open a main.rs file.

Hopefully its not a Win7 specific issue as I don't have any machine to test that with. It may be the same issue as PvdBerg so repairing your VS installation may be the way to go. I could remove ValueTuple to get it working for you immediately but your VS install would still be unstable since ValueTuple is used throughout VS.

After VS repairing and opening prjoect folder a toolbar with rls Install button appears.
But after clicking on it a error rises in that tab:

could not install one of the required rls components

That will likely be an issue with the RLS not being available in the current nightly. It's been a major issue lately and is quite annoying. You can select your toolchain in the Tools>Options>Rust menu

Thanks. I'll wait a while with nightly.

I have the same problem.
Running VS 15.7.0 Preview 2.0, latest nightly (which includes rls), on Windows 10.

Unfortunately the issue persists at least for me.
VS community 15.0.9 review 2.0
rustup toolchain list:  beta-i686-pc-windows-msvc (default)
rustc --version: 1.30.0-beta.7 (0ebb25088 2018-09-22)
Windows 7.
When I install the extension, configure the path to rustup and then open folder with a cargo project I get no prompt or message to install the RLS. Tried to install it manually (went w/o warning/error) but it changed nothing in the plugin, e.g. it doesn't seem to work nor does it spawn rls.exe process.
The super infobar debug solution you've provided doesn't seem to work with new Visual Studio Preview as it relies on language server plugin which is ditched now per my understanding and the language server protocol became build-in functionality
Later I'll try to run it on Win10 virtual machine to see if this issue is OS-specific but I think you should make the process of installing RLS user-invocable and more verbose Maybe you could kindly update the debug version of the plugin so it can be launched on newer VS versions.

@l7l7 do you happen to know what version of the .NET framework you have installed?

@dgriffen
I have .NET framework version 4.7.

I think I figured out the issue, the version of System.ValueTuple I was referencing was newer than the one shipped with VS so there was probably a load order bug happening there. To make this simpler to reason about I'm just going to remove my use of ValueTuple

This should be fixed in 0.4.5 with the removal of ValueTuple. Let me know if there are still issues.


[FEATURE] --exclude option for lastAppSyncDateTime in androidManagedStoreAccountEnterpriseSettings

Is your feature request related to a problem? Please describe.
Thank you so much for the addition of '-e VPPusedLicenseCount' as it has really helped clean up my commits.
I have one more request, under "Managed Google Play" folder we get *__androidManagedStoreAccountEnterpriseSettings.json as pulled from the graph api
As per the api the file includes lastAppSyncDateTime which changes on every run (as you would expect).
Describe the solution you'd like
It would be nice to exclude this to again clean up the commit history
Describe alternatives you've considered
I could post process the file and remove this line, but having it stopped at the source similar to other excludes would be ideal.
Example file contents
{
     "bindStatus": "boundAndValidated",
     "lastAppSyncDateTime": "2024-02-13T20:25:10.7712992Z",
     "lastAppSyncStatus": "success",
     "ownerUserPrincipalName"<EMAIL_ADDRESS>     "ownerOrganizationName": "MyOrg",
     "enrollmentTarget": "targetedAsEnrollmentRestrictions",
     "targetGroupIds": [],
     "deviceOwnerManagementEnabled": true,
     "androidDeviceOwnerFullyManagedEnrollmentEnabled": true,
     "managedGooglePlayInitialScopeTagIds": [
          "57"
     ]
}

Please create a separate feature request for this

@Wob76 @almenscorner
Does this work for you? I have added the --exclude GPlaySyncTime parameter to my backup pipeline, and it still includes the ever-updating lastAppSyncDateTime property.
Looking at the code in src/IntuneCD/intunecdlib/IntuneCDBase.py (line 50), I believe it should be lastAppSyncDateTime instead of lastSyncDateTime?

No, it looks like it doesn't work. I still have a post process in place removing this and some other unwanted timestamps and licence counts, so it isn't showing in my final git push and I hadn't gone back to test.

You are right it should be lastAppSyncDateTime, I've corrected this in the latest beta, please test and verify the key is removed correctly: pip3 install IntuneCD==2.3.2b1


fix: docker volume & final snapshot identifier

We discovered that with the launch template ami that we've been using that the docker container was using the smaller 22GB volume and the 100GB volume was not available to docker containers. This explicitly switches to the newer Amazon Linux 2 ecs optimized images which have a single volume, to resolve the issue.
Also I noticed with the final snapshot identifier change to use timestamp, it was getting updated every time. I switched to using random_pet to fix this

Wow this is also a nice find! Read through the Gitter chat and that is really clutch that they wrote that up.
Excellent changes!


as_complex() confusing

This issue is part of my JOSS review at https://github.com/openjournals/joss-reviews/issues/6315
On a quite minor note, I find the function name as_complex() for the profiles classes a bit confusing.
Indeed, in numpy/scipy, as_complex() only cast an array to a one with a different dtype.
In this case, you create a complex array from a real one by taking the complex exponential of it.
So you convert a phase array to a complex one, which is fundamentally different.
A different name, even slightly different, would limit confusion for the user that would probably expect this function to behave as its namesake in numpy.

Agree. Thanks for the suggestion! I will make the changes.

Addressed in https://github.com/QITI/pySLM2/commit/dc62bf207b86ebb87ec54a91977a0017392dbfe9


Add sha512_256 to bitcoin_hashes

Adds a new file named sha512_256.rs that implements the sha512/256 hash. This was needed as a part of https://github.com/rust-bitcoin/rust-bitcoin/discussions/1318 to drop the sha2 dependency.
All the actual hashing code is exactly the same as sha512.rs, minus the initial constants and the use of hash_type! macro. Some unit tests were added from wikipedia (for the "" input) and the rest were from the Go standard library's tests for sha512_256.
Benchmarks on my Ryzen 3600 machine show that it is faster than sha256.
test sha256::benches::sha256_10                   ... bench:          37 ns/iter (+/- 0) = 270 MB/s
test sha256::benches::sha256_1k                   ... bench:       3,338 ns/iter (+/- 24) = 306 MB/s
test sha256::benches::sha256_64k                  ... bench:     213,605 ns/iter (+/- 1,806) = 306 MB/s
test sha512_256::benches::sha512_256_10           ... bench:          27 ns/iter (+/- 1) = 370 MB/s
test sha512_256::benches::sha512_256_1k           ... bench:       2,196 ns/iter (+/- 12) = 466 MB/s
test sha512_256::benches::sha512_256_64k          ... bench:     140,552 ns/iter (+/- 777) = 466 MB/s

One caveat is that I could not get hongfuzz to build locally so I couldn't test the fuzz on my machine. I ended up only testing through the CI for the fuzz tests.
I thought adding a completely separate file was the easiest and the most straightforward way of implementing it. I'm very much open to changing the implementation if you guys don't think this is the right direction.

Agreed with @Kixunil. At some point we want to rewrite all this code to use simd so the less duplication the better.

Edited the file to include the summary in all of the docs. Added newlines where necessary to have summaries.

I too would prefer not having pub(crate), I've added that improvement to me todo list. I think we should merge this as is so as not to tie @kcalvinalvin in knots with our usual perfectionist review cycles. Appreciate your contribution @kcalvinalvin, this PR is progress!

Interesting, how did the CI pass this?

Code review ACK. Did not cross-check the examples with other implementations.

Interesting, how did the CI pass this?

I think, because the fuzztests for the hashes crate are completely broken, we just have no CI coverage for the hashes/fuzz directory at all. See #1422.

Just to clarify, I think @apoelstra means hashes fuzz tests are not currently working but cargo test does work in the hashes/fuzz/ directory on master so the fuzzing stuff in this PR is incorrect.

cargo test in hashes/fuzz fails

Fixed the test in the latest push

ACK https://github.com/rust-bitcoin/rust-bitcoin/commit/411174c391fcd334f0ecfa419f3619db8bdd1c0c. Reviwed range diff from 43feb9ea7b282d9119708a27fa7a1c7412d1386a that I had ACked


docs: ignore protected and package members

Coverage remained the same at 99.988% when pulling 9aa3ce3d539e8324dde865493ffa65f81b63dd4f on callmehiphop:dg--docs-private into adc90ed16985d7a1922cdcbd8469b88f663bb71c on GoogleCloudPlatform:master.


commented out biosample/site modeling

#563

includes

https://github.com/microbiomedata/nmdc-schema/pull/562


New CLI entry point

CLI now implemented using the entry_points mechanism rather than a dedicated script.

Current coverage is 99.10% (diff: 33.33%)

No coverage report found for master at 0f50a53.

Powered by Codecov. Last update 0f50a53...58c15aa


Enrich Syntax API with the capability to replace nodes and tokens

Purpose
This PR enhances the syntax API with convenient methods to replace tree nodes and tokens. This requirement is a common one in most syntax tree editing scenarios.
Fixes #23380
Approach
I've used the TreeModifier utility to traverse the tree in the implementation. While descending, I check for the node to be replaced and replace it with the given replacement. Then the new tree nodes are created while ascending.
The original tree will not be modified; If the node to be replaced cannot be found. This implementation does not traverse through the whole tree. That would be a very inefficient approach. It traverses through potential branches in which the node to be replaced can be found.
Samples
        SyntaxTree syntaxTree = parseFile("replace_node_test_01.bal");
        ModulePartNode modulePartNode = syntaxTree.modulePart();

        // Get the semicolon token of an import declaration node
        ImportDeclarationNode importNode = modulePartNode.imports().get(0);
        Token semicolon = importNode.semicolon();

        // Create a new token from the old one by removing all minutiae
        Token replacementSemicolon = semicolon.modify(MinutiaeList.emptyList(), MinutiaeList.emptyList());

        // Option 1: Replace the token via the SyntaxTree
        SyntaxTree newSyntaxTree = syntaxTree.replaceNode(semicolon, replacementSemicolon);

        // Option 1: Replace the token via any parent node in the tree
        // But in this case, newModulePart node is a dangling node that does not have a syntax tree linked with
        ModulePartNode newModulePart = modulePartNode.replace(semicolon, replacementSemicolon);

        // You can link it with a syntax tree as follows
        SyntaxTree newSyntaxTree1 = syntaxTree.updateWith(newModulePart);

Remarks
I am planning to add features such as inserting nodes, deleting nodes in an upcoming PR.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[x] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #23382 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23382   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

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


[REQUEST] - Please differentiate Diagtrack naming in Set-Additional.ps1 to accommodate different Builds

Is your feature request related to a problem? Please describe.
The reintroduced "Set-Additional.ps1" doesn't block Diagtrack in bullds 20H1 and newer

For 20H1 and newer, Microsoft uses the name "Diagtrack-Listener"
For 19H2 and older, Microsoft uses the name "AutoLogger-Diagtrack-Listener"

Describe the solution you'd like
Include a conditional to differentiate between the Diagtrack names for builds 19041 and newer & 18363 and older
Additional context
Some Examples:
19041 and newer
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\WMI\AutoLogger\Diagtrack-Listener
Set-AutologgerConfig -Name Diagtrack-Listener -Start 0
18363 and older
HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\WMI\Autologger\AutoLogger-Diagtrack-Listener
Set-AutologgerConfig -Name AutoLogger-Diagtrack-Listener -Start 0

The disclaimer to use "Set-Additional.ps1" at your own risk is truly warranted.
Anyone who blindly applies "Set-Additional.ps1" without careful vetting deserves the messed up OS they will undoubtedly get.
There are lots of great tweaks in "Set-Additional.ps1", but it's better to carefully cherry-pick and manually update the tweaks  you want rather than using the script as written. Applies to all versions up to and including <IP_ADDRESS>.


add Mac OS support in dependencies

change gradle.build accordingly so the project builds and runs on Mac OS

Might not fix


WIP Add input challenges

Ready for pre-review :gift:

The input-challenge module is not in the right location, yet.
Testing does not work w/o wallaby, yet.
Is not correctly wired yet. E.g., the input-challenge module is not part of the real bundle, yet.
The front end shows no message after submission, yet.

Looks solid

Added number-exact-input-challenge

For the pull request, we will add a type with algebra.js (for solutions that need expressions, like sin(x) + cos(x)). This should be enough for MVP, although we probably want some type between number-exact and expression-equality...

We will probably finish this tomorrow. We wanted to add a third type that uses an algebra library to check equality of expressions. Furthermore, I will add test task to grunt (or npm) so that the new client-side tests can be run w/o wallaby.js.
Also, I thought about types for collection types. What about types input-challenge-sorted-collection resp. input-challenge-unsorted-collection that take input-challenge-*-match as children? Or is this even possible?

Also, I thought about types for collection types. What about types input-challenge-sorted-collection resp. input-challenge-unsorted-collection that take input-challenge-*-match as children? Or is this even possible?

That should be possible but I don't think that it's required just yet :)

Discussed changes: input-* should also have a feedback. Also, input-* can have input-* as children. Thos e are interpreted as wrong answers

please let me know if i can contribute code

@arekkas Basically done! :tada: tests are not part of the PR, since configuration of karma (and friends) is not finished yet. Would be nice if we can deploy this soon so that the academy can create some new exercises with this. Today, I'll fix any concerns you may have with this code.

Looks good, will deploy in a few


ceph-salt export logs private ssh key and dashboard password to /var/log/ceph-salt.log

When running ceph-salt config ls, the private ssh key is (correctly) not included in the log by the PillarManager.get() function, which looks like it's being called for every value individually (the dashboard password will also be skipped once #228 goes in). However, when running ceph-salt export, it calls PillarManager.get('ceph-salt'), which dumps the entire pillar contents, so everything in the pillar, including the private ssh key and dashboard password land in /var/log/ceph-salt.log

Something to keep in mind: the purpose of "ceph-salt export" is to produce output that can later be passed to "ceph-salt import", to reproduce the configuration (e.g., when starting again from a clean slate). To the extent that the dashboard username and password are part of the configuration, they need to be included in the export output.
Not that I am against being safe and not writing secrets to stdout or a file. Perhaps we need to think about how to have our cake and eat it, too.
Clearly, there needs to be a way (maybe not the default) to preserve the current behavior. Some sites may not have a need to strictly guard the secrecy of this password. On the other hand, when we do not write out the password, it has to be done in a way that doesn't break "ceph-salt import".

It's totally fine for ceph-salt export to dump everything to the export output (if it didn't, ceph-salt import wouldn't work, as you say). The thing I object to is that everything is also simultaneously dumped to /var/log/ceph-salt.log.

Oh. I didn't realize export was logging. Thanks for the clarification!


iroh-lb

We should start working on a load balancer for the gateway.
It's still early to have real use for it but it's much easier to evolve and grow with us.
Reasons to do it:

we already have 2 gateways up, I'd love to have seamless handoff between the two
puts us up on the stall to work on the "real stuff" from day 1
should be easy enough to get started

Why not use something off the shelf

I can se us getting into the weeds with this: I'd love to work on supporting % rollouts, feature flagging on the gateway level, failover mechanics, geo based balancing, rtt based balancing, contention based etc... Don't think anything of the self will give us enough flexibility

I'm going to close this for now, priorities have shifted away


Different startup option

Currently if there is a HoldClick during the banner presentation we reset brightness/contrast, etc.  This doesn't work in VFO_DEBUG mode as we don't show the banner.
A better approach is to sense if the RE button is down during startup.  If so, reset stuff that can disable using the VFO, such as:

brightness
contrast
HoldClick time

Done.