[Linux]Syntax error

Hi,
I have the up to date Kali Linux installation.
I got the client installed and started it but it won't give me an URL to work with and this is what it looks like:
Starting IoT Inspector...
WARNING: No route found for IPv6 destination :: (no default route?)
Traceback (most recent call last):
  File "start_inspector.py", line 2, in <module>
    import inspector
  File "/root/princeton-iot-inspector/iot-inspector-client/v2-src/inspector.py", line 12, in <module>
    from netdisco_wrapper import NetdiscoWrapper
  File "/root/princeton-iot-inspector/iot-inspector-client/v2-src/netdisco_wrapper.py", line 13, in <module>
    from netdisco.discovery import NetworkDiscovery
  File "/usr/local/lib/python2.7/dist-packages/netdisco/discovery.py", line 7, in <module>
    from .mdns import MDNS
  File "/usr/local/lib/python2.7/dist-packages/netdisco/mdns.py", line 2, in <module>
    import zeroconf
  File "/usr/local/lib/python2.7/dist-packages/zeroconf/__init__.py", line 211
    def current_time_millis() -> float:
                              ^
SyntaxError: invalid syntax
Updating software...
Already up to date.
Updating depdencies...
Requirement already satisfied: Flask in /usr/lib/python2.7/dist-packages (1.1.1)
Requirement already satisfied: flask-cors in /usr/local/lib/python2.7/dist-packages (3.0.8)
Requirement already satisfied: Flask>=0.9 in /usr/lib/python2.7/dist-packages (from flask-cors) (1.1.1)
Requirement already satisfied: Six in /usr/lib/python2.7/dist-packages (from flask-cors) (1.13.0)
Requirement already satisfied: requests in /usr/lib/python2.7/dist-packages (2.21.0)
Requirement already satisfied: netaddr in /usr/lib/python2.7/dist-packages (0.7.19)
Requirement already satisfied: scapy-ssl_tls in /usr/local/lib/python2.7/dist-packages (2.0.0)
Requirement already satisfied: pycryptodomex>=3.4 in /usr/lib/python2.7/dist-packages (from scapy-ssl_tls) (3.6.1)
Requirement already satisfied: scapy<2.3.3,>=2.2.0 in /usr/local/lib/python2.7/dist-packages (from scapy-ssl_tls) (2.3.2)
Requirement already satisfied: tinyec>=0.3.1 in /usr/local/lib/python2.7/dist-packages (from scapy-ssl_tls) (0.3.1)
Requirement already satisfied: scapy-http in /usr/local/lib/python2.7/dist-packages (1.8.2)
Requirement already satisfied: scapy in /usr/local/lib/python2.7/dist-packages (from scapy-http) (2.3.2)
Requirement already satisfied: netdisco in /usr/local/lib/python3.7/dist-packages (2.6.0)
Requirement already satisfied: requests>=2.0 in /usr/lib/python3/dist-packages (from netdisco) (2.21.0)
Requirement already satisfied: zeroconf>=0.21.0 in /usr/local/lib/python3.7/dist-packages (from netdisco) (0.24.0)
Requirement already satisfied: ifaddr in /usr/local/lib/python3.7/dist-packages (from zeroconf>=0.21.0->netdisco) (0.1.6)

The code is Python 3 only. We will update the installation script. #36

I do have python 2 and 3 installed.
What am I to do now?

The installation script is written in Python 2. If you're interested, you could help us port the installation script to Python 3. (We're currently behind some other deadlines and won't get a chance to fix this issue until mid-December.) See #36.

Aaah now I get it.
I would love to help you out but I'm not really versed enough. Sorry :(


GPU and Simulated Annealing - multiple machines?

A followup question from the Hybrid Quantum Programming webinar 12 Jul 2019
I hope it's OK to ask this here, sorry if not!
Can Simulated Annealing via Kerberos make use of GPU resources? What level of control is possible?
Is it possible to spread a Simulated Annealing workload across multiple classical hosts?
If possible, could you point to any documentation that covers this area?
Thanks!

Hey, @rog77, that's a good dwave-hybrid-related question, so it is welcome as an issue here! :smiley:
Unfortunately, at this point the only simulated annealing implementation we have available in Ocean is CPU-based. And it's single-threaded at that. It releases the GIL during C++ anneals, though, so you can run multiple Python SA samplers in parallel. However, if maximizing performance is crucial for you, C-level parallelization would be better (it should be straightforward to parallelize the current dwave-neal Cythonized implementation).
So, no GPU support in Hybrid at this moment, although we do have that on our radar.
Regarding the simulated annealing documentation, we use Neal in Hybrid, so you can have a look at SimulatedAnnealingSampler.sample() docs.

Is GPU support in Hybrid available now?

Hi @AYUSHMIT, there are still no built-in components (blocks/runnables) in dwave-hybrid that exploit GPU features. Could you tell us more about your use case that would require GPU access in dwave-hybrid?


[release-1.15] pilot: fix issue with TLS and TCP order dependency

Manual cherrypick required.
#40072 failed to apply on top of branch "release-1.15":
Applying: pilot: fix issue with TLS and TCP order dependency
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/listener.go
M	pilot/pkg/networking/core/v1alpha3/listener_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/listener_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/listener_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/listener.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/listener.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 pilot: fix issue with TLS and TCP order dependency
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

duplicate of https://github.com/istio/istio/pull/40224


Fix deprecated Robot syntax in the code

 WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/ipmi/test_fru.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/test_inventory.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/tests/test_rest_interfaces.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.
[ WARN ] Error in file '/home/fspcibld/openbmc-test-automation/ipmi/test_ipmi_user.robot': Escaping empty cells with '\' before line continuation marker '...' is deprecated. Remove escaping before Robot Framework 3.2.

Resolved by https://gerrit.openbmc-project.xyz/#/c/openbmc/openbmc-test-automation/+/22245/


Enable other trading platform APIs

TradeKing is only available to customers within within the US.
It would be great if it was possible to replace the TradeKing API with one from another trading platform which is available in Europe/UK.
I'll take a look at the code myself to investigate but I'm very much a beginner with Python.

Yeah!
I'm not prof. trader but maybe
https://www.oanda.com would be solution.
They offer an REST API.
here is a python solution:
https://github.com/oanda/py-api-trading

Also I found the create an application function in TradeKing doesn't work for me. When I'm redirected back after creating an new personal app, there's nothing in the developer home page.

#19 #21
I think we have a few suggestions on trading platforms. I would love to help out on one of them if @maxbbraun you think that's a good idea!

I'd be very interested in Robinhood, because they don't charge trading fees. They don't have a documented API (yet?), unfortunately.

Robinhood doesn't have public api but u can request api (bete) access.  There is a small request form to fill.

I'd vote for Interactive Brokers. You can trade almost any type of security with them (stocks, futures, options etc.) using IbPy API. Trading fees are quite low.

Seconded Interactive Brokers. I think they are available in many countries too. Would be nice to use the same API. But I don't see a python version of it. Does that mean it needs to run as a separate GAE instance just to make trades?

I'd chime in for IB as msilb and domingohui both mention above. However, the page I located and the one domingohui linked to differ... The site site states python support see the following: IB API. Alternatively, IG Group operate worldwide, are robust (have been running since the 1970s) and have API access. See IG Group and IG APIs

I support the idea of using Interactive Brokers. Best value / quality over price.

@MailrPro Hmmm interesting... But that's good news.

Looks like there's an unofficial Robinhood API here: https://github.com/aurbano/robinhood-node

@mikespax More detail here: https://github.com/sanko/Robinhood

Was this ever resolved?

Alpaca looks interesting too.


CORE-1045 add tests and implementation for Cloudwatch Log Groups

@jmanero-r7 @athompson-r7 This PR looks good to me, anything else you expect from @aburke-r7 here? This has been sitting here a while.

Looks good.  Going to merge this.  Thanks!

Also pushed this to the active(hamer-time) branch.


Flexible neuron, synapse data layout

This seems very straightforward, now that Neurons and Syns are all just giant arrays of structs on the Network:

Neurons and Syns are just single giant []float32 arrays (likewise on GPU).
Vars are just enums (put desc tooltips into separate array of strings)
Access is via methods with indexes to layer, neuron, variable, etc that take Context as first arg (nearly ubiquitous), which has a Network index in CPU mode, that allows access to a global list of networks, which then allows access to these arrays.  On GPU, the accessor methods just access the global arrays defined in the kernel.
It is then entirely trivial to reorganize the memory layout any which way.

Context is shared between GPU and CPU and can't contain pointers, hence the need for the network index.  Could alternatively have a pointer that is meaningless to the GPU -- isn't needed there anyway.
Also one of the indexes can be data index so data parallel can be interleaved in sequence.

more context: GPU has to access global arrays directly, which are allocated for each kernel, so need global functions that are defined differently on CPU vs. GPU:
func NeuronVarIdx(ctx *Context, neurIdx, dataIdx int32, nvar NeuronVars) int32 {
    return var * ctx.Strides.NeuronVar + neurIdx * ctx.Strides.Neuron + dataIdx * ctx.Strides.NeuronData
}

// CPU version
func NeuronVar(ctx *Context, neurIdx, dataIdx int32, nvar NeuronVars) float32 {
    return Networks[ctx.NetIdx].Neurons[NeuronVarIdx(ctx, neurIdx, dataIdx, nvar)]
}

// GPU version
[[vk::binding(1, 2)]] RWStructuredBuffer<float> Neurons;
float NeuronVar(ctx *Context, neurIdx, dataIdx int32, nvar NeuronVars) {
    return Neurons[NeuronVarIdx(ctx, neurIdx, dataIdx, nvar)]
}

Have a separate SetNeuronVar, and versions that go via layer if needed (layer has starting global neuron index), etc.
Can just switch out NeuronVarIdx functions to see impact of different layouts.

The non-float32 vars (flags, indexes) in Neuron would be stored separately, so you don't have to complicate anything about type conversions.  Also, SynCa would be separated from Synapses memory, per https://github.com/emer/axon/issues/168

Context requires data parallel state for all the PVLV, NeuroMod stuff.

Plan: add GlobalVars enum in globals.go, store memory in Network, with all the NeuroMod and PVLV state.  Can parameterize with NDrives with computed offsets etc for flexible storage.  maybe have an offset index lookup table for each val (above a given enum value).  Data inner-loop indexing for each var.  GPU just exposes the Globals directly as usual.

This worked as expected and massively improves GPU performance, and even CPU performance is significantly improved in NData > 1  cases.


Rising World [unity] (DRAFT)

Adding a dedicated server for the game Rising World unity version. This is a draft at the moment since I don't know how well it works yet.

Closing this as it is not capable of being merged and has a number of configuration issues. A working template is in my dev branch, which I will eventually include in a PR


x32 port

I played pretty loose with the statfs bit, hoping for CI to tell me if I broke something.

Should be g2g, will wait for review to squash.
Oddly, the test_fcntl::linux_android::test_vmsplice test that failed passes on amd64 for me?

Interesting.  Are you actually using x32 with Rust?  Last I heard, Linux was talking about dropping it.

Well, I am running a full Debian x32 userland, with the only foreign packages being the kernel.
As for using it "with Rust", well, I only recently found the target in rustup, did a bare-bones port of nix, and hit a codegen bug almost immediately; LLVM (and, well, Rust: https://github.com/rust-lang/rust/issues/59220) is still very much baby mode on x32, but I would love to be able to just™ target Rust there. It's currently not really viable, but I also see no reason why it wouldn't become viable in future.
As for Linux dropping it, I neither know nor care; it's a (low priority, but still) supported target of the Rust platform as it stands.

Rebased, squashed, includes @hvdijk's test fixes from https://github.com/nix-rust/nix/issues/1384#issuecomment-774708486.

pogue, it even builds on CI. @hvdijk mind testing on your set-up?

pogue, it even builds on CI. @hvdijk mind testing on your set-up again?

It passes testing here, except for unrelated issues, nice.
For completeness, the unrelated issues are:

test_vsock fails here because SockAddr::new_vsock(libc::VMADDR_CID_LOCAL, port) succeeds, just as it does on x86_64-linux-gnu.
test_af_alg_aead hangs here during read(). After attaching gdb and immediately detaching, the test passes. I do not know what is happening here, but it also happens here for x86_64-unknown-linux-gnu.
unistd::Group::from_gid fails because my system has no group named root.

@nabijaczleweli why don't you go ahead and add x32 to .cirrus-ci.yml and to the Tier 3 target list in the README?

The tests fail typecheck at CAPGET and CAPSET, which are fixed in caps HEAD (https://github.com/lucab/caps-rs/commit/f6178d0ab650d68b64afb11eb8f530d4ff7ba148). I've bugged the caps maintainer to push out a new version, but

Just a heads up, tests are going to fail for x32 even with the updated caps-rs unless you have also patched rust's LLVM to include the x32 codegen fixes that are going to be in LLVM 12.

Hence why I only marked it for typecheck

Ouch.  Well, we can't merge it like that.  You can remove the CHECK_TESTS line from .cirrus-ci.yml, which will fix the caps failure.  But what's this about a toolchain problem?  At what point will that cause a failure?

But what's this about a toolchain problem? At what point will that cause a failure?

LLVM has x32 codegen bugs that are only fixed in LLVM 12, but Rust, even nightly, is currently still based on LLVM 11. These codegen bugs affect Rust code badly. If you stick with what you get from rustup, then (after fixing the other problems) cargo test --target=x86_64-unknown-linux-gnux32 fails with:
error: test failed, to rerun pass '--lib'

Caused by:
  process didn't exit successfully: `/home/harald/rust-nix/target/x86_64-unknown-linux-gnux32/debug/deps/nix-0354ce9290eb8626` (signal: 11, SIGSEGV: invalid memory reference)

LLVM 12 already has its first release candidate out, so I do not expect it to take too long till we have an official version of Rust that generates correct x32 code. Until then, for actually running Rust code on x32, a built-from-source version of Rust is needed with LLVM 12's fixes applied.

At the point where you run it and it fails to properly relocate and segfaults in a Funny Way:tm:, at least that's what happened that time I built one of my programs.
I got rid of CHECK_TESTS, should I post another small PR adding it back when caps gets updated later, then?

Yes, please add a PR for that when it's ready.  In the meantime, how about a CHANGELOG entry?

Added CHANGELOG entry and re-added the test typecheck with the bumped caps.


Android bluetooth device not working

I cant connect to bluetooth device like my Nokia cell phone and like a Android Bluetooth phone. Theres anything that I doing wrong? What are bluetooth serial port profile devices supported?

Sorry, ignore it.


Support searching databases within a server

Issue Type: Bug
Support searching within a the databases within a server from the servers panel.
Also, consider searching tables within a database from the servers panel.
Currently users who want to search database and tables within a database must do this from the search panel within the Manage page for the server.
Azure Data Studio version: Azure Data Studio 1.0.0 (cab8f3e37c9ac4367e0dacef5185da08bd8021c2, 2018-09-20T21:41:50.803Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
16.00GB (1.99GB free)

Process Argv
/Applications/Azure Data Studio.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

agent
Mic
0.33.0

import
Mic
0.2.0

profiler
Mic
0.2.0

sql-vnext
Mic
0.6.9

We currently recommend using Redgate SQL Search to search through active database connections.  It would be great to have extensibility point in OE and the Search viewlet so that extensions could provide this functionality.

Closing as duplicate of https://github.com/Microsoft/azuredatastudio/issues/2634.


Raven 7.6.0, and improved support Spring Security

Based on suggestions from @erichelgeson and @burtbeckwith (https://github.com/agorapulse/grails-raven/pull/23)

@donbeave 7.6.0 has been released on Bintray. Thanks!

Thank you!


Release 1.1.0 rc2

Candidate commits for releasing 1.1.0-rc2 and switch back to dev and bump main branch version to 1.1.0-dev
(See https://github.com/opencontainers/distribution-spec/pull/348 for process)
19 PRs merged since last release (v1.1.0-rc1):
git log --oneline --merges 1dfa4f4...270cc7f | sed '$ d' |  \
  sed 's/.*Merge pull request //' | grep -v 'GHSA' | awk '{print "- " $1}' | uniq | grep '#'

#401
#395
#400
#394
#399
#366
#383
#393
#391
#392
#387
#385
#384
#380
#379
#381
#371
#356
#348

duplicate of #403


chore: bdd-spring-1575124520 to 0.0.1

chore: Promote bdd-spring-1575124520 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Bug] Error when Retrieving Granule metadata - Would be nice if this is more descriptive for ASF

Describe the bug
Suspect this is related to the data outage on October 7th from 3 - 7 AM UTC; we are not in the specified time range.
I see the HTTP error - maybe I have something wrong at my end with credential setup? I thought this portion of metadata didn't require any credentials.
When I say "more descriptive for ASF", I mean if this is an error related to some sort of expected outage, maybe a package specific error could be raised to alert users to the possible causes. But this issue will be a new pointer as well.
Anyways, thanks for your help!
Using asf_search version 2.0.2 installed with pip.
import asf_search as asf

response = asf.granule_search(['S1B_IW_SLC__1SDV_20210723T014947_20210723T015014_027915_0354B4_B3A9'])

Outputs:
---------------------------------------------------------------------------
HTTPError                                 Traceback (most recent call last)
~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/asf_search/search/search.py in search(absoluteOrbit, asfFrame, beamMode, collectionName, maxDoppler, minDoppler, end, maxFaradayRotation, minFaradayRotation, flightDirection, flightLine, frame, granule_list, groupID, insarStackId, instrument, intersectsWith, lookDirection, offNadirAngle, platform, polarization, processingDate, processingLevel, product_list, relativeOrbit, season, start, maxResults, host, cmr_token, cmr_provider)
    141     try:
--> 142         response.raise_for_status()
    143     except HTTPError:

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/requests/models.py in raise_for_status(self)
    952         if http_error_msg:
--> 953             raise HTTPError(http_error_msg, response=self)
    954

HTTPError: 504 Server Error: Gateway Time-out for url: https://api.daac.asf.alaska.edu/services/search/param

During handling of the above exception, another exception occurred:

JSONDecodeError                           Traceback (most recent call last)
<ipython-input-4-f9e9a1c40764> in <module>
      1 import asf_search as asf
      2
----> 3 response = asf.granule_search(['S1B_IW_SLC__1SDV_20210723T014947_20210723T015014_027915_0354B4_B3A9'])

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/asf_search/search/granule_search.py in granule_search(granule_list, host, cmr_token, cmr_provider)
     25     data = dict((k,v) for k,v in kwargs.items() if v is not None and v != '')
     26
---> 27     return search(**data)

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/asf_search/search/search.py in search(absoluteOrbit, asfFrame, beamMode, collectionName, maxDoppler, minDoppler, end, maxFaradayRotation, minFaradayRotation, flightDirection, flightLine, frame, granule_list, groupID, insarStackId, instrument, intersectsWith, lookDirection, offNadirAngle, platform, polarization, processingDate, processingLevel, product_list, relativeOrbit, season, start, maxResults, host, cmr_token, cmr_provider)
    145             raise ASFSearch4xxError(f'HTTP {response.status_code}: {response.json()["error"]["report"]}')
    146         if 500 <= response.status_code <= 599:
--> 147             raise ASFSearch5xxError(f'HTTP {response.status_code}: {response.json()["error"]["report"]}')
    148         raise ASFServerError(f'HTTP {response.status_code}: {response.json()["error"]["report"]}')
    149

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/requests/models.py in json(self, **kwargs)
    908                     # used.
    909                     pass
--> 910         return complexjson.loads(self.text, **kwargs)
    911
    912     @property

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/simplejson/__init__.py in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, use_decimal, **kw)
    523             parse_constant is None and object_pairs_hook is None
    524             and not use_decimal and not kw):
--> 525         return _default_decoder.decode(s)
    526     if cls is None:
    527         cls = JSONDecoder

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/simplejson/decoder.py in decode(self, s, _w, _PY3)
    368         if _PY3 and isinstance(s, bytes):
    369             s = str(s, self.encoding)
--> 370         obj, end = self.raw_decode(s)
    371         end = _w(s, end).end()
    372         if end != len(s):

~/opt/anaconda3/envs/rscube/lib/python3.8/site-packages/simplejson/decoder.py in raw_decode(self, s, idx, _w, _PY3)
    398             elif ord0 == 0xef and s[idx:idx + 3] == '\xef\xbb\xbf':
    399                 idx += 3
--> 400         return self.scan_once(s, idx=_w(s, idx).end())

JSONDecodeError: Expecting value: line 1 column 1 (char 0)

The SLC is on the DAAC as noted in this link.

@cmarshak just ran into this as well. You can always check the health endpoint: https://docs.asf.alaska.edu/api/keywords/#health-endpoint
In particular https://api.daac.asf.alaska.edu/health currently returns a '504 Gateway Timeout'. So the error you're seeing is do to tring to decode a JSON response that is non-existent or actually HTML format.

I can confirm that we were experiencing a bit disruption with the SearchAPI yesterday. I haven't gotten to pin down the root cause yet, but we were able to alleviate the issue and things should have settled down.
One of our next orders of business is to put Logging statements throughout the module, at varying levels of verbosity. That should help pinpoint issues when these sorts of situations arrive.

Hi, I wanted to report that I have been experiencing this problem for more than 2 hours.
I am using:
results = asf.search(...)
but I tested the other endpoints and also have the problem.
Greetings.

We did have a fairly significant outage over this last weekend, but that should be resolved by now, as we've made some changes to our provisioned AWS resources. We're now seeing very low error rates (on the order of 1-2 per hour), and we're keeping a very close eye on things in case any further problems pop up.

As of yesterday's 5.0.0 release, asf_search now operates directly against CMR, completely removing reliance on our own SearchAPI. This should alleviate these types of error, at least as much as we have control over.
Feel free to open new issues for future errors, and thanks for the report :)


Can't procced to backward.loss process

Found a bug in line 390
from src/modules/transformer_decoder.py logits = torch.stack(logits, 1).data --> logits = torch.stack(logits, 1)
As such, backward.loss process without the require_grad being True, creates error in training.

Duplicate of #3 . Fixed by 3ab629c08f0b5d84bbc3d56ac70ce12e60500322.


New package: TeamViewer.QuickSupport version 15.55.3

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @dpdlaurenttest,
This package doesn't seem to modify the system. These were the only files added:

Is this expected?


Disable power platform co-presence feature.

This pull request primarily focuses on disabling certain features in the package.json and commonUtil.ts files. The changes include modifying the conditions for displaying certain commands and views in package.json, and disabling the co-presence feature in commonUtil.ts.

In package.json, the visibility conditions for the commands and views have been altered:

The "powerPlatform.previewCurrentActiveUsers" command is now never shown, as the condition in the "when" field has been changed from "isWeb && virtualWorkspace" to "never".
Similarly, the "powerpages.collaborationView" view is now never shown. The condition in the "when" field has been changed from "isWeb && virtualWorkspace" to "never".

In src/web/client/utilities/commonUtil.ts, the co-presence feature has been disabled. The isCoPresenceEnabled function now always returns false, effectively disabling the feature.

@ritikramuka these changes are verified on local?


use signals as state management

Hey @dy this library is cool :)
I know this has built-in state management, I have existing state management that uses signals,
for example I want to use this as View only and use my existing store as Controller or Model does this possible? if yes do I need to access the .value of the signal/computed property in the html? thanks

It used to be built with signals store https://www.npmjs.com/package/signal-struct but then it was replaced in favor of more compact & flexible proxy-based one.
Let me check if it's possible to support signals as dynamic sources.
Do you have an example of your store?

import { signal } from 'usignal'
import { createStore } from 'signal-store'

// state is not accessible, need to use actions to update the state
// getters is accessible
const store = createStore({
  state: {
    count: signal(0)
  },
  getters: {
    count(state) {
      return state.count.value
    }
  },
  actions: {
    increment({ state }, count: number) {
      state.count.value = state.count.peek() + 1
    },
    decrement({ state }, count: number) {
      state.count.value = state.count.peek() + 1
    }
  }
})

Is that store essentially same as just object with signals as keys? In other words, you'd want to use signals directly as template values?

Is that store essentially same as just object with signals as keys?
yes

In other words, you'd want to use signals directly as template values?
yes correct
because I don't want o rewrite the store that I created or I want reuse the reactive values and functions

ok, will try to see if it makes sense to switch store back to signals. It's less compact but possibly faster


Failed to load projects.cson (undefined is not a function)

The project-manager seems to have some trouble: when I opened up an existing project, it couldn't load the tabs and displayed this "Failed to load projects.cson" and "undefined is not a function".
I see it seems close to #159.
I also have a symbol-tree view plugin. Disabling it didn't solve the case, I still get the error (at startup, now...).
But I don't know what I should do with a coffee linter. Also, my projects.cson file is still accessible (even using Project Manager > Edit Projects) and is quite simple:
mathmaker:
  title: "mathmaker"
  paths: [
    "/home/nico/dev/mathmaker/mathmaker/"
  ]
pilberry:
  title: "pilberry"
  paths: [
    "/home/nico/dev/pilberry/pilberry"
  ]
nothing:
  title: "nothing"
  paths: [
    "/home/nico/dev/"
  ]

Which version of Atom do you use? There is a function that is being used that came in a newer version of Atom.

Ok, I see the last version is 1.5.4 and mine is 1.0.11. I think I updated project-manager recently and this might have caused the problem. I try to update it (if the dependencies of my distribution are met).

Indeed the update solved the problem. Sorry about that and thanks for your quick reply :)

No problem :)


Update BREAKING.md

Updated the Datetime entry to reflect the correct package name
Short description of what this resolves:
Updated the Datetime entry to reflect the correct package name
Changes proposed in this pull request:
Updated the Datetime entry to reflect the correct package name of @ionic/angular
Ionic Version: 1.x / 2.x / 3.x / 4.x
4.x
Fixes:

Thanks!


Only show the back button if it needed

Right now clicking on the go-back button sometimes does nothing so we can hide the button when not in need you can check of you can go back on an active webview by webview.canGoBack().
This is a followup of #413, @abhigyank can you do this?

@zulipbot label "Priority: Medium", "Type: Enhancement"

Sure, I can take this up.
I think showing and hiding the back button, would not look good from design perspective. Rather I feel, we can change the shade of color of the back button to suggest inactiveness. Like how it is handled in browsers like chrome. What do you think?

@abhigyank i do prefer hiding the button than disabling it since the icon is too big so its not working i rather not have it in my way as a user but feel free to get other opinion on it.

I get your view. Maybe can open a discussion on this in design stream?

I don' think we need to hide the button, just disabaling the cursor and 0.6 opacity will work.

Do we need it up though and not be functional, specially with a new button about to be added and this button being completely being useless most of the time it just adds to clutter for no reason.

I'm 100% sure that this button will be visible most of the time if a user is using the app. We are planning to add a hide/show kind of button to deal with the clutter thing. Something like -

But let's try the first approach i.e. show it only if webview.canGoBack() and see how it looks.


Remove unnecessary folder #4 and tags with wp-json references from _static/index.html (SG)

remove unnecessary _static/wp-json folder
remove unnecessary html tags that reference wp-json in _static/index.html

@gsantella took care of this in commit https://github.com/gsantella/ambucs-site-2023/commit/5a88883b5b01698890bd455f5083c98bf81cafef to branch development/1.3


Where to get the annotation file?

When I train the model, I cannot find the file 'results_20130124.token', would you publish the file or tell me how to generate the file?
Thank you!

Hello.
That file is part of Flickr30k dataset, which you need some authorization process to get.
You can write a request form [here]
Thank you.

@jazzsaxmafia
Thank you, I found the file in your show_attend_and_tell.tensorflow project  finally.

@jazzsaxmafia
Hi, Have you evaluate the result use the BLEU score ?
Can you get the score close to the article ?

Sorry I haven't. The purpose of implementing this was merely practicing Tensorflow so I did not derive any quantitative measures.
The score must be far different from that of the paper, since I didn't do any parameter tunings. Also the authors used Googlenet as image encoder, while I used VGG. This may also affect the performance.


feat: initial start of the profile parser

What did you fix?
This start the profile-parser project in order to parse the output given by https://content.warframe.com/dynamic/getProfileViewingData.php?n=${username}
TODOs:

[x] tests
[x] github actions

Considerations

Does this contain a new dependency? Yes
Does this introduce opinionated data formatting or manual data entry? Yes
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? No
Have I run the linter? Yes
Is is a bug fix, feature request, or enhancement? Feature

this last build error is a real stinker, cause it's hella not clear. you have to use @latest on install-peerdeps's target artifact for this not to happen, but that's pretty risky.

this last build error is a real stinker, cause it's hella not clear. you have to use @latest on install-peerdeps's target artifact for this not to happen, but that's pretty risky.
basically 3.0.3 doesn't handle where there's a newer version of what you've caret-specified; i.e. ^1.5.1 resolves to ['1.5.1', '1.6.0', '1.6.1'] not just 1.5.1

ah that simplifies things a bit

sorry i've not checked the last couple things. i'll give this another shot soon and get it approved!

@TobiTenno did I do it right?

seems right so far, but I haven't checked out your code yet still >.<

One of those is my fault on items, and i'm fixing that

seems right so far, but I haven't checked out your code yet still >.<
unfortunately yields type errors:
><EMAIL_ADDRESS>build:types
> tsc -p tsconfig.declaration.json

index.d.ts:22:38 - error TS2304: Cannot find name 'dynamic'.

22     constructor(profile: Map<String, dynamic>, locale: string);
                                        ~~~~~~~

index.d.ts:127:19 - error TS2304: Cannot find name 'ColorMap'.

127     primaryColor: ColorMap;
                      ~~~~~~~~

Found 2 errors in the same file, starting at: index.d.ts:22

I put dart's dynamic type and when it didn't correct me I just figured it was right, guess the term here is any?

also that in JS json are objects

Yeah was waiting to see if I was doing it right before moving on

@TobiTenno hopefully this is what you had in mind


fix: Open link "Learn More" function is not called

Change-Id: I387ad16f614de26aeff5c0f162bda7f405ffa621
Signed-off-by: Florent Benoit<EMAIL_ADDRESS>

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


return status for timeout

Another issue once #42 is resolved. When Mosek hits the time limit, it currently returns :Unknown status through the MPB interface. This should be :UserLimit. How can we detect if the solver stopped because of a time limit?

Yes. Optimize returns an MSK_RES_... code. MSK_RES_OK means that the optimizer stopped in a normal way, MSK_RES_TRM_... gives you the reason for termination. There are several codes that can be mapped to :UserLimit (MAX_ITERATIONS, NUM_SETBACKS, MAX_TIME etc., all depending on values that the user can set).

Is this return value available afterwards (through soldef, prosta, or solsta?) or do we need to store it ourselves?

You will have to store it. It is not stored in the task object.

This should work now.


Unable to add any devices via the integration now

Describe the bug
I'm unable to add any devices via the integration. Had a power outage this morning and when everything came back online, two merkury smart bulbs that were previously connected via Tuya-local never connected back. Power-cycled the bulbs several times with no luck. I removed them from the integration to add them back but now they won't join at all. I keep getting the intermittent connection issue error, but I've been trying this for over 12 hours now.
To Reproduce
Unexpected shutdown of the whole power system.
Expected behavior
The integration should have come back online as expected.
Additional context
I've been trying to add a Tuya-based WiFi Bridge as well (AC701) and that won't connect either. However, not sure if that's even supported since trying this for the first time.
If the bug involves a device, then please include device diagnostics from
Settings / Devices & Services / Tuya Local (your device) / "1 device"
Please check if there are messages from Tuya Local in the Home
Assistant log (Settings / System / Logs), and include those too.
If applicable, add screenshots to help explain your problem.

Requests for help should be posted in discussions, not the issue tracker.


Detect some instances of cookie domain misconfiguration

A fairly common misconfiguration for browser-based endpoints is having an explicit cookie domain set to a domain which doesn't match the site in which the tracking script is loaded. This results in the tracking script loading, but the party and session cookies that it sets can't be subsequently read. As a result every page view looks like a new user and session.
As an enhancement we could detect this in the tracking script during load and throw an exception if it's detected. This assumes that this would be noticed quicker (and presumably fixed) than what currently seems to be the case.

Some guidelines on how to setup cookies would be appreciated.
For a single domain usage I managed to get it working with :
cookie domain : example.com
host page : https://www.example.com
script tag https://divolte.example.com/divolte.js
and that works.
However if I want to use that same divolt instance to track www.othersite.com how can I do that ?  If I set cookie domain to divolte.example.com each page view is a new session

Tracking cookies across multiple second level domains are purposely made impossible by browsers. Typically, you will want to set the cookie domain to .example.com (including the first dot) for tracking across *.example.com (including the naked domain example.com).
If you want to identify users across multiple domains, you have to get them to identify themselves using a login or other means, which in most cases requires some form of explicit consent.
Most workarounds that bypass this protection should likely be considered temporary hacks that browsers will eventually also prevent (e.g. user invisible redirects, etc.). It's unlikely that Divolte will aim to support any of these.


Device status calculation

The overall device status should be calculated from information about the device, such as;

the last check-in
the connection status
whether it is enabled

Part of #173


Elastic agents fails to spin up in EKS version 1.29

Hello,
I m using the latest version of the chart with the latest elastic agent plugin, everything works fine on my eks 1.25 cluster but the same configuration doesnt seem to work on version 1.29. Specifically the agents are not getting created whenever a job starts and I m getting this logs from the UI

From the logs of the server I m getting this:

but from previous closed issues I saw that this message could be safely ignored.
Is this an incompatibility with k8s versions > 1.25 or is there any extra configuration needed that wasnt required before? Thanks.

Did you ever figure out where the slowness was coming from here?


Clean up of code with pylint.

Using pylint, I've fixed all instances of newlines missing, spaces, and imports.  All tests pass (1625/1625).

Thanks! There are quite a lot of cleaning to be done. At some point I think Flake8 or other style validators should be added to the CI and this sort of PRs will lessen the burden a lot.
I don't feel great at the moment so I'll check and merge this PR later.

Codecov Report
Base: 95.51% // Head: 95.23% // Decreases project coverage by -0.27% :warning:

Coverage data is based on head (0b0765d) compared to base (a8e509b).
Patch coverage: 95.72% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
- Coverage   95.51%   95.23%   -0.28%
==========================================
  Files          88       79       -9
  Lines        4612     4322     -290
==========================================
- Hits         4405     4116     -289
+ Misses        207      206       -1

Impacted Files
Coverage Δ

rocketry/_setup.py
100.00% <ø> (ø)

rocketry/conditions/task/utils.py
90.12% <ø> (-0.47%)
:arrow_down:

rocketry/conds/__init__.py
100.00% <ø> (ø)

rocketry/core/time/__init__.py
100.00% <ø> (ø)

rocketry/core/utils/meta.py
100.00% <ø> (ø)

rocketry/exc.py
100.00% <ø> (ø)

rocketry/log/defaults.py
100.00% <ø> (ø)

rocketry/log/handlers.py
100.00% <ø> (ø)

rocketry/parse/time.py
100.00% <ø> (ø)

rocketry/parse/utils/parser.py
100.00% <ø> (ø)

... and 76 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

The codecov/project "failed" but I think that's nothing concerning. The failure is due to some unnecessary lines being removed. I cannot see any problems with this: the relevant import paths are the same and all tests pass. Also, the test coverage is still over 95%.
Thanks again for the contribution! This is very helpful.


Real shadows

This is to track integration of the Escher work with Flutter's framework.

Should we re-title this to be specifically about escher?  If so, this should be happening soon I'm told?

Closing this bug. We don't need a bug to track it.


[Hub Gateway] Invite Page Blocked

Issues
Users invited to an org are unable to complete the invite processes because the invite page throws no "session cookie" error from the gateway.
Desire
Invites page doesn't require session auth.

Fixed by removing early return 401 status code on the session plugin
https://github.com/holaplex/helm-charts/pull/52


spec: no mention of optional argument 'cutset' of strip

The go and java implementations strip/rstrip/lstrip functions accept a cutset optional parameter however it is missing from the spec. It is mentioned however in the go-spec.
The Rust implementation, on the other hand, does not support the cutset argument.

Thanks, we need to update the spec.


Make Poko annotations use source retention

Poko could reduce its footprint by shipping the annotations with source retention rather than binary retention, and adding the poko-annotations dependency as compileOnly.
I tried using source retention in #16, but immediately reverted it without comment. Evidently it broke something, but I don't remember what, so I'd like to try again (and keep notes if it breaks again).
I also tried using compileOnly when I first introduced the @Poko annotation, but reverted to implementation in #48 because consuming libraries' test compilation units were broken. I suspect this might be fixed if source retention is used at the same time, otherwise maybe there is a more robust way to add a compileOnly dependency to all compilation units.
One possible downside is that @Poko etc. will be highlighted in red in the IDEs of consumers of libraries which use Poko, which will look kind of weird.

Completed by #171.


Make typechain auto-generation opt-in

The recently added zero-config feature that overrides the compile task to generate typechain should be opt-in. Although there is a flag to turn it off for the compile task when running it from the command-line, there's no way to stop it from running when running other tasks that automatically compile the contracts. Specifically, when running the test task - the typechain is generated and there's no way to stop this.
My current workflow triggers hardhat test to be run whenever a test is changed (watching file changes using nodemon) and typechain generation adds a few extra seconds to this loop thus slowing down the development.
My config:
  typechain: {
    outDir: 'types',
    target: 'truffle-v5',
    runOnCompile: false,
  },

Behavior:
$ hh test
Creating Typechain artifacts in directory types for target truffle-v5
Successfully generated Typechain artifacts!

  Contract
    ✓ test

$ hh test --no-typechain
Error HH305: Unrecognized param --no-typechain

For more info go to https://hardhat.org/HH305 or run Hardhat with --show-stack-traces

package.json:
{
  "name": "hardhat-project",
  "dependencies": {
    "@nomiclabs/hardhat-truffle5": "^2.0.0",
    "@nomiclabs/hardhat-web3": "^2.0.0",
    "@typechain/truffle-v5": "^2.0.2",
    "hardhat": "^2.0.3",
    "hardhat-typechain": "^0.3.3",
    "ts-generator": "^0.1.1",
    "typechain": "^3.0.0",
    "web3": "^1.3.0"
  },
  "devDependencies": {
    "chai": "^4.2.0",
    "mocha": "^8.2.1",
    "nodemon": "^2.0.6"
  },
  "scripts": {
    "compile": "hardhat compile",
    "dev": "nodemon --ext js,sol -x 'hardhat test --show-stack-traces'",
    "typechain": "hardhat typechain"
  }
}

Please make this feature opt-in!

@shark0der would you be ok with it being opt out if it were only run when new contracts were actually compiled?

@alcuadrado I am not OP but I have the same problem. Personally I'd prefer it was opt-in, or at least that I can opt-out for all tasks and not just compile. Quite often contracts are changed in a way that does not affect types.
I'm all for sensible config defaults and perhaps enabling this by default is fine for most users, but making any step that takes noticeable elapsed time compulsory feels like it's going too far.
By the way - great plugin - thank you!

Any update on this? Running hh test overrides my custom compile script.

@shark0der would you be ok with it being opt out if it were only run when new contracts were actually compiled?

Only running when contracts change. Yes! Also, opt-out globally via direct HardHat flag. Appears to not work when passing to hardhat directly, e.g. hardhat --no-typechain coverage


Grid helper

It's a little dark set against the bright axes helper so look closely :)

Nice! LGTM.
One remark is that I think that we can also make infinite grids in a fragments shader, similar to how we have the background objects. But this a lot easier :)

Nice! LGTM.
One remark is that I think that we can also make infinite grids in a fragments shader, similar to how we have the background objects. But this a lot easier :)

I'd love to learn how to implement that :)


Conda environment not found for "trigger-book-build" action

Hi,
I was interested in submitting a notebook, but I don't understand the workflows well enough to troubleshoot the build error I am seeing when submitting a pull request from my development branch to main on my cookbook template-based repository (https://github.com/NCAR/na-cordex-viz-cookbook).
I'm currently building via "cache" rather than "binder" because it seems that the binder service is timing out.   The notebook relies on ~20 dask workers with around 10GB of working memory each, so I'm wondering if this notebook might be asking for too many resources and failing for that reason.
The build has appeared to create the conda environment "na-cordex-viz-cookbook-dev" at least once, but currently when I change the environment.yml file by adding or removing a package, it skips the environment build.
The error I'm getting is this:
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 2 source files that are out of date
updating environment: [new config] 2 added, 0 changed, 0 removed
reading sources... [ 50%] README
reading sources... [100%] notebooks/plot-diagnostics
/home/runner/work/na-cordex-viz-cookbook/na-cordex-viz-cookbook/notebooks/plot-diagnostics.ipynb: Executing notebook using local CWD [mystnb]

Exception occurred:
  File "/usr/share/miniconda3/envs/na-cordex-viz-cookbook-dev/lib/python3.11/site-packages/jupyter_client/kernelspec.py", line [28](https://github.com/NCAR/na-cordex-viz-cookbook/actions/runs/4877914656/jobs/8703051137?pr=1#step:17:29)7, in get_kernel_spec
    raise NoSuchKernel(kernel_name)
jupyter_client.kernelspec.NoSuchKernel: No such kernel named na-cordex-viz-cookbook-dev

My current PR with build results is here:   https://github.com/NCAR/na-cordex-viz-cookbook/pull/1
It's also possible I was not supposed to check in changes to the "main" branch:  I ended up changing the environment.yml, actions environment names, and notebook to run by modifying the "main" branch directly before creating a dev branch for merge.
If I can supply more information, let me know!   Thanks.

Investigating in https://github.com/NCAR/na-cordex-viz-cookbook/pull/2#issuecomment-1535289862

Thanks for submitting this, @bonnland ! As @brian-rose mentions, we do intend to enable Dask cluster capability using our Jetstream2 Binderhub. Right now I'm running through your notebook using a dask LocalCluster on one of my department's servers. I will let you know if I run into any issues!

Thanks @ktyle!   Currently using binder fails on a "git reset" command.   I wonder if it could permission problems for an outside user?   Just a hunch, though...

I've been tasked with shepherding the NA-CORDEX Data Visualization Cookbook through the review process. I'll take a closer look at this issue as part of that process. At the moment I can't recall if there is or is not a problem that needs solving here.

Closing this, as the NA-CORDEX Cookbook is building normally now.


multiple states(Routes) when an event fire many times (multiple times)

I am connecting Angular2 application having multiple states(Routes) with Node server(Socket) running on the back-end. When I visit some other state and come back to the previous state where socket code is written in service file on the angular application created with fromEvent. the service called by subscribe() in multiple components which is called in ngoninit() { } . How often I move on routes gets a component view and event emit when an event fire, So many times (multiple times) the subscribed service gets called (multiple time show console.log("get message")), which affects the performance.

can anybody solve this problem?

try the removeListener() function when you destroy the page


Fixes bugs of questions

PR Fixes:

1 Fixes re-rendering of whole page when question is voted on /questions
2 Fixes re-rendering of page when reply is added
3 Fixes re-rendering of page when Editor is clicked

Resolves #1247 #1248
Before:
https://github.com/user-attachments/assets/4746dfe5-79d7-410f-8019-d50394514631
https://github.com/user-attachments/assets/04050abc-2f24-42d9-8d2f-79d4c6cc04e8
After:
https://github.com/user-attachments/assets/ed4bc08d-d3ae-490a-8d4b-d5b5e6315749
https://github.com/user-attachments/assets/483a1bd0-3164-4f78-ab53-82f6ffab352c
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

Can u open a new pr with minimal changes and less lint changes. Very hard to understand the particular change that you have made.

Closing for now. Feel free to reopen

@devsargam i am new to this
As soon as I make any changes in any of PRs and fix lint changes, lines which are not touched by me gets changed.
Kindly guide.

I am making lint fixes on fresh branch
17 files changes (which i didnt changed)
I commit those changes
start working on my issues

Thats why it seems cluttered.


Fix T5 DeepSpeed ZeRO-3

What does this PR do?

As per title.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@libinta That should fix it, let me know if you manage to run it successfully.

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


Error syncing RoleBinding.rbac.authorization.k8s.io subjects[0].name: Required value in Rancher logs

Rancher Server Setup

Rancher version: v2.6.1
Installation option (Docker install/Helm Chart):

Docker version 20.10.9

Proxy/Cert Details:
Information about the Cluster
Kubernetes version: k8s v1.20.9
Cluster Type (Local/Downstream): 5x Custom

Describe the bug

To Reproduce

I don't know yet

Result
Rancher log is filling with
2021/10/13 10:21:44 [ERROR] error syncing 'grb-p9s9r': handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value, handler restrictedAdminGlobalBindingsFleet: 1 error occurred:
	* RoleBinding.rbac.authorization.k8s.io "grb-p9s9r-fleetworkspace-restricted-admin-rb-cluster" is invalid: subjects[0].name: Required value

, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value, requeuing
2021/10/13 10:21:44 [ERROR] error syncing 'grb-w7p7b': handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value, handler restrictedAdminGlobalBindingsFleet: 1 error occurred:
	* RoleBinding.rbac.authorization.k8s.io "grb-w7p7b-fleetworkspace-restricted-admin-rb-cluster" is invalid: subjects[0].name: Required value

, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value, requeuing
2021/10/13 10:21:47 [ERROR] error syncing 'c-294db': handler restrictedAdminsRBACCluster: 3 errors occurred:
	* RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value

, requeuing
2021/10/13 10:22:00 [ERROR] error syncing 'c-w55lk': handler restrictedAdminsRBACCluster: 3 errors occurred:
	* RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value

, requeuing
2021/10/13 10:22:01 [ERROR] error syncing 'c-l4s28': handler restrictedAdminsRBACCluster: 3 errors occurred:
	* RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value

, requeuing
2021/10/13 10:22:01 [ERROR] error syncing 'c-hrqm9': handler restrictedAdminsRBACCluster: 3 errors occurred:
	* RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value

, requeuing
2021/10/13 10:22:04 [ERROR] error syncing 'c-ktsj2': handler restrictedAdminsRBACCluster: 3 errors occurred:
	* RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value
	* RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value

, requeuing
2021/10/13 10:22:27 [ERROR] error syncing 'grb-cnggj': handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-w55lk-cluster-owner-" is invalid: subjects[0].name: Required value, handler restrictedAdminGlobalBindingsFleet: 1 error occurred:
	* RoleBinding.rbac.authorization.k8s.io "grb-cnggj-fleetworkspace-restricted-admin-rb-cluster" is invalid: subjects[0].name: Required value

, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-l4s28-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-ktsj2-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-hrqm9-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-294db-cluster-owner-" is invalid: subjects[0].name: Required value, requeuing
2021/10/13 10:23:24 [ERROR] error syncing 'kubeconfig-user-plfnr689kg': handler cat-token-controller: clusterAuthToken for token [kubeconfig-user-plfnr689kg] has not been created yet, requeuing
2021/10/13 10:23:24 [ERROR] error syncing 'kubeconfig-user-plfnrwl427': handler cat-token-controller: clusterAuthToken for token [kubeconfig-user-plfnrwl427] has not been created yet, requeuing

Expected Result
No ERRORs in log
Additional context

rancher restart doesnt help
It seems that something is missing at the end of "crt-c-ktsj2-cluster-owner-"

same here for v2.6.2.
2021/12/01 17:21:02 [ERROR] error syncing 'grb-2796f': handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart051-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart050-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart049-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart055-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart056-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hall065-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart058-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-hart059-cluster-owner-" is invalid: subjects[0].name: Required value, requeuing

seeing this same issue on 2.6.3
2022/01/18 16:13:01 [ERROR] error syncing 'grb-bqhqz': handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-nv262-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-zx6rz-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-flqrw-cluster-owner-" is invalid: subjects[0].name: Required value, handler grb-cluster-sync: RoleBinding.rbac.authorization.k8s.io "crt-c-sqnpm-cluster-owner-" is invalid: subjects[0].name: Required value, requeuing

+1 2.6.3

+1 2.6.3


Add Build Env for Django 1.9

Django 1.9 has been released, the djangomaster now represents the pending Django 1.10.0 (or 2.0.0) release

Fixed in ca38730.


helm chart lint

Add linting for all the chart changes in manifest/charts repo

This change is

@hdkshingala That sounds even better indeed.

Ok, making changes accordingly


feat: spawn new server for different branch

Instead of putting the project name and commit sha into the server name, we form the servername by hashing the string
{namespace}{project}{commit-sha}{ref}

where ref for the time being is assumed to be a branch but could be a tag, for instance.
The resulting pod name is something like jupyter-rokroskar-9ecaa0eeed735403 (16-digit hash) and the server name is just the hash, i.e. in this case rokroskar/9ecaa0eeed735403.

closes #90

Hmm true I forgot about the failing tests - it's because of the change to the server name.

it was a trivial change - should be fixed now. Please re-approve if the tests pass.

I found an issue with this -- I am reporting here because #186 will probably solve this automatically.
When a server is started for the combination (commit1,branch1), it's not possible to start another server for the combination (commit1,branch2). Could this be somehow related to the POST url without the explicit branch and some missing checks? /.../commit1?branch=branch2 instead of something like /.../commit1/branch2

That's expected and that'a what's I meant when I said API would be in an inconsistent state. When creating new notebook, API first checks if there is another one which is already running and returns that one. This check cannot differentiate between different branches because we only implemented that feature in POST requests and not for GET.
You are right, https://github.com/SwissDataScienceCenter/renku-notebooks/issues/186 will solve this issue.


app utils name: sample from letters by frequency

Sample the starting letter by how frequently it appears in the word lists. For example, X.* X.* should appear infrequently (< 1/26 of the time) because there are only one adjective and three nouns starting with X.
Generate a probability distribution of starting letter for the adjective and noun lists, multiply them by one another and normalize them, then sample from that probability distribution function to get the starting letter. Then, choose a random adjective + noun starting with that letter as before.

since the name lists are static, I think you should precompute the sampling distribution.

The commit was hard to read because of moving constants, so I pulled that out into its own commit. a353f64 precomputes the distribution as you described!


Fix circular import dependency GLOBAL_CACHE_DIR (snowpackjs#3554)

Problem
snowpack seems to get broken when there is no package.json can be found, and I realized it's the same problem with #3544
so I debug this and find out there is a circular import dependency for GLOBAL_CACHE_DIR (between snowpack/src/config.ts and snowpack/src/utils.ts). it caused GLOBAL_CACHE_DIR to be imported before it's initialed.
Changes
move the constant variable GLOBAL_CACHE_DIR to snowpack/src/source/utils.ts just like Fred k. Schott used to.

Testing
use any snowpack command below your ~ directory can make it.

Docs
bux fix only

@drwpow hey, my team's frontend work cli has a strong dependency on snowpack, and in most conditions we need to use snowpack on a directory without package.json. We should be glad if you could handle this at your convenience

I'm integrating with Snowpack via your JavaScript API. In that scenerio I also get the ERR_INVALID_ARG_TYPE error as described in https://github.com/snowpackjs/snowpack/issues/3554 This PR seems to resolve that problem.

I met the same problem.My project is using snowpack JavaScript api to run.but it always get broken when running in a top-level directory.maybe this is a very useful or.


Feature: Add more fine grained configuration steps

To follow

Hassfest cannot be valid, as "selector" values need to be lowercase, but are mandatory to be camel-case because of SmartThings API.


Accept-Encoding cleanup

This is the general pattern for removing deprecation from recurring headers:

In companion, stop extending HeaderKey.Internal and HeaderKey.Recurring
Add a variadic .apply constructor
parse is no longer interface, so remove override
In case class, stop extending Header
Remove key and Value
Remove any reference in HttpHeaderParser, which is going away.
Stop extending HttpHeaderParserHelper
Fix any other compilation errors.

More conflicts than code.


Request to evaluate the new O1 models by OpenAI (O1-preview and O1-mini)

Request in title. Thanks! :)

Done!


feat(tx-builder): Add dark mode support

What it solves
Resolves #824
How this PR fixes it
It adds dark mode theme palette into the project and set the theme based on the postMessage event which we receive from the safe wallet app.
How to test it

Go to your safe at this link: https://feat_tx_builder_dark_mode--walletweb.review.5afe.dev/home?safe=eth:0xA77DE01e157f9f57C7c4A326eeE9C4874D0598b6 (or by using this branch from the safe wallet repo: feat/tx-builder-dark-mode)
Take the deploy link on this PR and add it as a custom safe app on your safe
Open the custom tx-builder app which you just added
Change the dark mode toggle on the sidebar
The transaction builder theme should change according to safe wallet theme

Video
txbuilder.webm

Issues:
1 - The form when you edit a tx that is batched on the right side is still white:

2 - The list of saved batches show the list of them white as well:
For this you must save it in the library first:

Go to Your tx library:

And then you see this:

@francovenica thanks! I'll take a look on them

@francovenica done ✅  Can you please check that again? Please make sure to clear your cache ;)

I think the fixes broke some stuff in light mode now:
The saved batches, there is a number that should be white
There is a gray background that should be white

Same in the tx edition:
For the light mode there is a gray background when in dev is fully white. A comparison here:

in dev:

Also in the dark mode I think the whole modal should be gray. Compare it with any other tx in the site:
The tx edition in the tx builder:

A regular tx color:

@francovenica can you please check that again?

All the issues fixed
LGTM now.


No error if inspection query text is emty (f.e. SQL file is not available)

There is no error if inspection SQL file (or all of them) is not available, because it seems that empty text is a valid SQL statement.
I believe it's not what user expects from this tool.
At least pg_analyse could check if SQL text to execute is empty (f.e. empty correspondent file, file is absent, file is not accessible due to permissions issues etc) and print some warning about it rather than just report that no issues found for certain inspection, because actually inspection was not executed.
F.e.:
Tables without Primary Key [tbl_nopk]
Warning: inspection query text is empty, results may be inaccurate!
  Table name    Table size
  ------------  ------------

I checked once again and it seems that it was some misconfiguration in my project, when SQL file is not available, an error appears:
[Errno 2] No such file or directory: '<path>\pg_analyse\pg_analyse\sql\contrib\index_health\sql\indexes_with_null_values.sql'


bug in code

File "/home/test/Desktop/mutate_cpp/app/utils/Executor.py", line 89, in workflow
self.__execute_command_timeout('patch -p1 --input={patchfile} {inputfile}'.format(patchfile=patchfile.name, inputfile=file.filename), cwd='/')
File "/home/test/Desktop/mutate_cpp/app/utils/Executor.py", line 71, in __execute_command_timeout
raise subprocess.CalledProcessError(errcode, command, stdout)
subprocess.CalledProcessError: Command 'patch -p1 --input=/tmp/tmpo7vp4cdf /home/test/Desktop/cmake-example/src/example.cpp' returned non-zero exit status 1
and also in reverse patch

Can you give more information? Can you make sure the patch tool is found in your system's path? What is the version?

yes, i can use diff command and patch -p1 in my shell

please help me with that.
what should i do to fix the problem?

please help me with that.
what should i do to fix the problem?
GNU patch 2.7.5
Copyright (C) 2003, 2009-2012 Free Software Foundation, Inc.
Copyright (C) 1988 Larry Wall
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Larry Wall and Paul Eggert

please help me with that.
what should i do to fix the problem?
GNU patch 2.7.5
Copyright (C) 2003, 2009-2012 Free Software Foundation, Inc.
Copyright (C) 1988 Larry Wall

License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Larry Wall and Paul Eggert

Hello,
i try to fix my problem with your code:
this is about doing patch and the error is :
1 out of 1 hunk FAILED -- saving rejects to file /home/test/Desktop/example.cpp.rej

Hello,
this is my problem with your code:
this is about doing patch and the error is :
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file /home/test/Desktop/example.
cpp.rej

On Thu, May 31, 2018 at 6:22 PM, Niels Lohmann<EMAIL_ADDRESS>wrote:

> Can you give more information? Can you make sure the patch tool is found
> in your system's path? What is the version?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/nlohmann/mutate_cpp/issues/4#issuecomment-393537300>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AkW71RQ1Q2UfDUW1okTfMSqSHc9mqyOqks5t3_W3gaJpZM4UVFQA>
> .
>

Hi,
I had similar issue and I figured it out. Compared to @alirezah7 in my case difference was that I followed step by step with instruction.
Issue:
patch tool fail because similarities don't match what's in the original file. More precisely there was different line break type in generated patch and example.cpp
Thats why we have this error (in my case):
Hunk #1 FAILED at 1 (different line endings). 1 out of 1 hunk FAILED -- saving rejects to file /tmp/cmake-example/src/example.cpp.rej
(I used patch command that was printed in log to achive this error)
If you check generated patch file and exmaple.cpp file line breake types you can see something like:
for patch
$ file /tmp/tmpyxoqpka0 /tmp/tmpyxoqpka0: unified diff output, ASCII text, with CRLF, LF line terminators
and for example.cpp
file /tmp/cmake-example/src/example.cpp /tmp/cmake-example/src/example.cpp: C source, ASCII text
Solution:
Check line endings type and change it acordingly.
I used unix2dos command and then result for file was:
$ file /tmp/cmake-example/src/example.cpp /tmp/cmake-example/src/example.cpp: C source, ASCII text, with CRLF line terminators
I hope this will help.
For clarity: I'am using debian 9

Hi! Thanks for providing this tool!
I have a similar problem as above. The mutation process just stops with this error in terminal:
Exception in thread Thread-2: Traceback (most recent call last):   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/threading.py", line 917, in _bootstrap_inner     self.run()   File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/threading.py", line 865, in run     self._target(*self._args, **self._kwargs)   File "/Users/christofferochcamilla/Downloads/mutate_cpp-master/app/utils/Executor.py", line 42, in main     self.workflow(patch)   File "/Users/christofferochcamilla/Downloads/mutate_cpp-master/app/utils/Executor.py", line 89, in workflow     self.__execute_command_timeout('patch -p1 --input={patchfile} {inputfile}'.format(patchfile=patchfile.name, inputfile=file.filename), cwd='/')   File "/Users/christofferochcamilla/Downloads/mutate_cpp-master/app/utils/Executor.py", line 71, in __execute_command_timeout     raise subprocess.CalledProcessError(errcode, command, stdout) subprocess.CalledProcessError: Command 'patch -p1 --input=/var/folders/py/kq1mb1057jx8cs_k656_k43h0000gn/T/tmp2sr_54xf /MySourceCode.cpp' returned non-zero exit status 1.
When I run the patch command as outputted in the console i get:
patching file /MySourceCode.cpp Hunk #1 FAILED at 1025. 1 out of 1 hunk FAILED -- saving rejects to file /MySourceCode.cpp.rej
When I read the cpp.rej file I don't get any clue of what is wrong. Tested both on a windows box and macosx.
Also, when generating patches the checkboxes doesnt seem to do anything. I selected only comparision operator but it seemed to generate patches for all options, include line deletions which this particular patch was (it was the first one, I have yet to be able to run any patch).
Thanks for any help!

So I read zawadsoons post above little more carefully and indeed I had the same problem. I could change both encoding and line-ending in CLion. Patches still failed randomly though so I wrote this in Executor.py on line 88:
`try:
# step 2: apply patch
self.__execute_command_timeout('patch -p1 --input={patchfile} {inputfile}'.format(patchfile=patchfile.name, inputfile=file.filename), cwd='/')
            # step 3: command pipeline
            success = self.__apply_command(patch, 'build_command') and \
                    self.__apply_command(patch, 'quickcheck_command') and \
                    self.__apply_command(patch, 'test_command')

            if success:
                patch.state = 'survived'
                db.session.commit()

            # step 4: revert patch
            self.__execute_command_timeout('patch -p1 --reverse --input={patchfile} {inputfile}'.format(patchfile=patchfile.name, inputfile=file.filename),
                                    cwd='/')
        except subprocess.CalledProcessError as e:
            # patching failed
            patch.state = 'survived'
            print("Patching failed. Marking as survived.")`

I ran in that problem too. I had to regenerate the patches to make it work. However, to do it properly, I had to regenerate the project entirely (i.e. delete + new). Why? Because if I click Regenerate again and again, it ADDs the same patches over and over again. (i.e. 700, 1400, 2100, etc.) It should replace the patches. Delete the existing ones and create new ones with the current file content.

Hey @AlexisWilke, thanks for your help and your recent MRs! In fact, the work on this project is dormant from my side: I am happy to merge any MRs, but I will not be able to contribute further in the foreseeable future.


Problem with Repositories.Archive() [STREAM error]

Hi,
My go version is: go1.7.4 darwin/amd64
I'm unable to use the Repositories.Archive() method without having an error about stream:
stream error: stream ID 1; PROTOCOL_ERROR.
I must add that i didn't have the problem like 2 months ago (since then, i have updated dependencies and go bin)
When i'm doing the call to gitlab api using postman and the same token, i don't have any errors.
i dont know if the problem is from this package, golang or ... ?
Here is the go code (hash = 0.6.0 and projectNamespaceOrId = 365):
opt := &gitlabApi.ArchiveOptions{SHA: &hash}
bites, _, err := gitlab.Client.Repositories.Archive(projectNamespaceOrId, opt)

And here is a screen when using postman:
https://cl.ly/0g3q0F3P1Q07
Here is the nginx log when calling repository using go:
https://cl.ly/0J412s3F1Q0l
I don't have this error for any other calls that are made by the package, so i really don't know what's the problem.
Help would be appreciate

@julienroland it didn't ring any bells, but after some googling I suspect you're maybe hitting this issue: https://github.com/golang/go/issues/16847
Go 1.8 should contain a fix for this, so if you have any chance to test/confirm this with Go 1.8 RC1 (it's on the bottom of the page) that would be realy awesome!
Additionally you should be able to work around this by setting Transport.TLSNextProto to a non-nil, empty map (see here for more details), which effectively disables using HTTP/2 and falls back to using HTTP/1.1.

Thank you very much !
I tried using go1.8rc1 but it still doesn't work.
Which is strange because, it works well when i disable http2 like u mentioned (the work around).
Here is the code if someone needs it:
```
     http.DefaultClient.Transport = &http.Transportc{
    TLSNextProto: make(map[string]func(authority string, c *tls.Conn) http.RoundTripper),
 }
  ```

Thanks for you help !


Support images with webp format in imagetext

the libraries for this are very heavy and I don't like to make the mod weigh so much


How to deal with cell-centered and face-centered fields in NetCDF output?

All the arrays right now are of size Nx*Ny*Nz but technically there are Nx volumes in the x-direction and Nx+1 faces so the output should match this and make sense. The easiest thing to do with the doubly periodic configuration we have right now is to have row Nx+1 be a repeat of row 1 but writing out the fields at the very bottom (k=Nz+1) might require some extra computation?

If we are committed to not using ghost cells, then I think the solution will depend on how boundary conditions are implemented. Periodic boundaries are just one of many possible cases we will encounter.

Yeah the output writing might get complicated as it could depend on the grid and boundary conditions and #86.
Will also be more complicated when data is spread across multiple cores or GPUs.

This may be another reason why ghost cells are preferred. A “boundary condition” can then be thought of as a function that fills in missing values on the edges of a field. Currently a “boundary condition” is something that modifies the right hand side of an equation... ?

Should we open a new issue to discuss whether or not to use ghost cells? Might be a hot topic and is only a backend issue so it isn't related to #86 either I think.

I would suggest (if it works with out guys) a short meeting (< 30mn)
to refine what we mean by and what we want regarding Bondary-Conditions
(including ghost-cell). My view: it could help identify and formulate the
right/relevant issues. Today 2.pm ? 3pm ? 4pm?
On Thu, Feb 28, 2019 at 02:54:59PM +0000, Ali Ramadhan wrote:

Should we open a new issue to discuss whether or not to use ghost cells? Might be a hot topic and is only a backend issue so it isn't related to #86 either I think.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ali-ramadhan/Oceananigans.jl/issues/92#issuecomment-468302424

@jm-c @glwagner I can do 3pm later today.
Scheduling meeting in GitHub issues is awesome!

3p works for me too.

Let it be recorded: we discussed ghost cells and decided not to use them.
The main reason is that it is not clear whether an algorithm based on ghost cells can be used for irregular boundaries, where a cell point within land may be shared between 2 or more adjacent fluid cells.
We will need to figure out how to include the correct values of FaceFields on the boundary in output, for arbitrary boundary conditions. Perhaps we can implement some kind of pseudo-ghost cell functionality that only affects the calculation of output. Something to stew over.

Just noting that whether we should save boundary values (i.e. halo data) to disk is an open issue #92.

The user can choose how to save their data in general, so the point holds regardless of how the 'default' output writer is designed.
Originally posted by @glwagner in https://github.com/climate-machine/Oceananigans.jl/issues/100#issuecomment-500855524

Good point. A default where halo data is omitted makes the most sense to me, but we can just have a save_boundary_data flag that saves the full offset array to disk in case anyone wants it.

Just wait until you start trying to calculate fluxes and gradients in post-processing... 😏

Ah, fair enough. I still think it would be confusing for most users to have the boundary values in the output by default but a save_boundary_data flag sounds like an important feature now.

This has been recently resolved in PR #644.


Compressed animation blend shape tracks don't preserve weights of zero

Tested versions
Reproducible in Godot v4.4.dev [bbc54692c].
System information
Windows 10.0.19045. Compiled with MSVC.
Issue description
When rendering a mesh with blend shapes, the vertex shader will skip any shapes with a weight close to zero. However, compressed animation blend shape tracks output a weight above this threshold even though the original animated weight was zero. This could cause a performance issue if an animation has a lot of zero weight blend shapes. It will also affect the visuals, although that will be hard to see as the weight is very close to zero.
The blend shape compression code is:
// Animation::_compress_key
blend = (blend / float(Compression::BLEND_SHAPE_RANGE)) * 0.5 + 0.5;
values[0] = CLAMP(int32_t(blend * 65535.0), 0, 65535);

// Animation::_uncompress_blend_shape
float bsn = float(p_value.x) / 65535.0;
return (bsn * 2.0 - 1.0) * float(Compression::BLEND_SHAPE_RANGE);

A weight of 0.0 compresses to unorm 32767, which decompresses to -0.00012207. This is slightly bigger than the vertex shader's threshold of 0.0001.
// Skeleton.glsl
for (uint i = 0; i < params.blend_shape_count; i++) {
	float w = blend_shape_weights.data[i];
	if (abs(w) > 0.0001) {

A potential fix is to change the 65535 scale to 65534 (https://github.com/greeble-dev/godot/commit/06502a0fbf778f1000e948261dbb3a20d4551b87). This means zero still compresses to unorm 32767, but that now decompresses to zero. The change would be backwards compatible with assets that were compressed before the change.
I can make a PR with this change if desired.
Note that there's another potential issue with how blend shape tracks and other tracks are doing unorm rounding. I've filed that as https://github.com/godotengine/godot/issues/99796, and will roll both fixes into the same PR unless advised otherwise.
Steps to reproduce
Launch the MRP project. The main scene has two cubes playing an animation with some zero weight blend shape tracks. Left cube = uncompressed animation, right cube = compressed. Initially there should be no visual difference.
Repo option 1 (requires code changes): Apply change https://github.com/greeble-dev/godot/commit/107bbe62e444f8d517a96a8732d016838e52779e to skeleton.glsl, which will force any blend shape that passes the weight threshold to blend with a weight of one instead. This is just a hacky way to show if the vertex shader skipped the shape. Launch the project and note that the cube with compressed animation is now distorted while the uncompressed animation is unchanged.
Repro option 2 (no code changes): Breakpoint Animation::try_blend_shape_track_interpolate, then in the editor select node AnimCompressed/AnimationPlayer to trigger the breakpoint. Note that the compressed path returns a weight of -0.00012207.
To test the potential fix, apply change https://github.com/greeble-dev/godot/commit/06502a0fbf778f1000e948261dbb3a20d4551b87. Launch the editor and select node AnimCompressed/AnimationPlayer to make it update. The cubes should now match, showing that the blend shape is correctly skipped.
Minimal reproduction project (MRP)
bug-blend-shape-repro.zip

I'm working on a PR to fix this along with https://github.com/godotengine/godot/issues/99796.


strict_param should be part of Symfony:risky, not Symfony

Using PHP-CS-Fixer 2.2.1, enabling the @Symfony ruleset triggers an error saying that strict_param is a risky fixer and risky fixers are not allowed.
It should be moved to the @Symfony:risky ruleset instead.

Closing this, it was my own config, not the Symfony ruleset. I looked too fast

The example given in the doc for the .php_cs config file is broken though, as it enables strict_param without enabling risky fixers

yeah, example is not activating risky flag as with copy-paste of example one may just ignore it. it's possible to set allowance not only in config file, but also as CLI param