Reading HDF5 causes System.Exception: This should never happen

I encountered this exception when trying to read a HDF5 file using this library. Upon further investigation it seems that the compound property information for the file are written in a way where the offsets are not strictly increasing. This breaks the assumption that it is only required to seek forward when decoding the properties. I can not control the writing of these files so I can only hope to resolve the issue during reading.
Here is the piece of code triggering the exception:
https://github.com/Apollo3zehn/PureHDF/blob/92e285eeb6d70a648b7c07ec02e548486a0332eb/src/PureHDF/VOL/Native/FileFormat/Level2/ObjectHeaderMessages/Datatype/DatatypeMessage.Reading.cs#L504-L538
I've already tested two solutions that solved the problem for my case:

Sorting the decode steps by their offset before iterating over them
Seeking to the beginning of the decodeStep position when decoding, then seeking to basePosition + this.Size at the end of decoding

The former will have a runtime overhead depending on the number of decode steps. I am not familiar enough with HDF5 and how it is used to say whether this will be significant or not, but in my case it is insignificant. The latter would depend on the underlying stream and is harder to predict.
I am curious if there are any other ideas or suggestions on how to handle this?
For completeness here is a full stacktrace:
Unhandled exception. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> System.Exception: This should never happen.
   at PureHDF.VOL.Native.DatatypeMessage.<>c__DisplayClass31_0.<GetDecodeInfoForReferenceCompound>g__decode|1(IH5ReadStream source) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/FileFormat/Level2/ObjectHeaderMessages/Datatype/DatatypeMessage.Reading.cs:line 518
   at PureHDF.VOL.Native.DatatypeMessage.<>c__DisplayClass39_0`1.<GetDecodeInfoForReferenceMemory>g__decode|0(IH5ReadStream source, Memory`1 target) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/FileFormat/Level2/ObjectHeaderMessages/Datatype/DatatypeMessage.Reading.cs:line 846
   at PureHDF.Selections.SelectionHelper.DecodeStream[TResult](IEnumerator`1 sourceWalker, IEnumerator`1 targetWalker, DecodeInfo`1 decodeInfo) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/Selections/SelectionHelper.cs:line 221
   at PureHDF.Selections.SelectionHelper.Decode[TResult](Int32 sourceRank, Int32 targetRank, DecodeInfo`1 decodeInfo) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/Selections/SelectionHelper.cs:line 212
   at PureHDF.VOL.Native.NativeDataset.ReadCore[TElement](Memory`1 resultBuffer, Selection fileSelection, Selection memorySelection, UInt64[] fileDims, UInt64[] memoryDims, H5DatasetAccess datasetAccess) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/API.Reading/NativeDataset.cs:line 497
   at PureHDF.VOL.Native.NativeDataset.ReadCorePre[TResult,TElement](TResult buffer, Selection fileSelection, Selection memorySelection, UInt64[] memoryDims, H5DatasetAccess datasetAccess, Boolean skipShuffle) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/API.Reading/NativeDataset.cs:line 375
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Span`1& arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at PureHDF.VOL.Native.NativeDataset.Read[T](Selection fileSelection, Selection memorySelection, UInt64[] memoryDims, H5DatasetAccess datasetAccess) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/API.Reading/NativeDataset.cs:line 203
   at PureHDF.VOL.Native.NativeDataset.Read[T](Selection fileSelection, Selection memorySelection, UInt64[] memoryDims) in /home/runner/work/PureHDF/PureHDF/src/PureHDF/VOL/Native/API.Reading/NativeDataset.cs:line 16

Do you have a sample file maybe? I could then test it today evening and look for solutions. In case you do not want to share it publicly, you could send it to<EMAIL_ADDRESS>

Thank you for taking a look, I've sent a sample to the address you mentioned.

Sorry used a wrong account for my previous message

I came to the same conclusion to either sort the decode steps or to seek to correct position in every decode step. Unfortunately seeking was costly in version < .NET 6 (https://devblogs.microsoft.com/dotnet/file-io-improvements-in-dotnet-6/#summary) and I do not like to have conditional compiling directives here to keep it simple. That is why I followed your suggestion to sort the decode steps.
Version 1.0.0-beta.5 is being build right now and should be published soon. Please check if that solves your problem (it does in my setup with your test file).
Thanks for your bug report, this is always very helpful to make PureHDF better :-)

I've tested version 1.0.0-beta.5 for a bit and it does indeed solve the problem. Thank you for the quick fix!


lazyLoad error.

Trying to run quickstart-ng2 from that branch:

We need change this line:
import {CommonModule} from "../common.module";
to
import {CommonModule} from "@angular/common";

Maybe we need change name CommonModule to RootModule?

Renamed it to SharedModule  👍


SSLSession.getLastAccessedTime() and getCreationTime() should not be …

…equals when session is reused
Motivation:
SSLSession.getLastAccessedTime() and getCreationTime() should not be equal when the session is reused. Also the last accessed time need to be updated whenever we re-use the session
Modifications:

Correctly update session * times.
Adjust unit tests

Result:
Correctly represent times in SSLSession implementation when using native impl

@chrisvest PTAL as discussed offline


Missing extension telemetry

When we moved to new extensions UI we lost all telemetry points. Adding them back.

Probably good idea to port to Arcade for hot fix as well.


fix: mv bug for dreambooth training

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Add force section to mv in sagemaker_entrypoint.py
Please delete options that are not relevant.

[*] Bug fix (non-breaking change which fixes an issue)

ok to me


Add a way to control volume of public speaker.

Allow DJ-Listener to be launched with a argument designating it as a public speaker, and will therefore let the server change its output volume. Then, authenticated users (or perhaps just admins) will be able to view and modify the volume of public speakers connected to the room.

Along a similar vein, have a speaker icon show up next to anonymous listeners count indicating if a public speaker is connected. Perhaps this is where its volume can be controlled.
This speaker icon can also be shown on the homepage/roomlist to indicate that a room is connected to a public speaker. This speaker icon should be the silhouette of a bookshelf speaker to avoid confusion with the mute button.


Responsive support not in typings

Describe the bug
I was looking at the example of CardComponent and noticed some odd syntax.  It was also failing on my Typescript environment.  It appears that responsive array definition is not supported.
To Reproduce
Steps to reproduce the behaviour:

https://docs.nativebase.io/building-card#most-commonly-used-design
save it as a tsx
run tsc.

Expected behaviour
No typescript errors
CodeSandBox/Snack link
It would be easier for us to debug if you can reproduce the error in a codesandbox link or a snack link. You can use the codesandbox template/snack template.
Screenshots

Platform

NativeBase version 3.1.0
Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Browser version [e.g. 22]
React Native version [e.g. 0.64]
Expo sdk version [e.g. 41]
node [e.g. 14.16.1]
yarn or npm [e.g. 6.1]

Additional information

Add any other context about the problem here.
Is the bug present in both iOS and Android or in any one of them?
If you want your issue to be looked at quicker, please attach a snack reproducible with your issue. Makes it easier for us!

You'd need something like this
type ResponsiveObjectProp<T> = {
  base: T,
  sm:T,
  md:T,
  lg:T,
  xl:T
}

type ResponsiveArrayProp<T> = [ T, T, T ]
export type ResponsiveProp<T> = ResponsiveArrayProp<T> | Partial<ResponsiveObjectProp<T>> |T

The suage would be something like
type X  = ResponsiveProp<string|number>;

const f : X = [3,2,1]
const g : X = {base: 3, sm:"sm"}
const h : X = "md"

console.log({f,g,h})

@trajano
I have the same problem. I don't know how to overwrite the native base interface. So I made an adapter function with the type you wrote.
// Function.ts
export function responsiveProp<T>(value: ResponsiveProp<T>): T {
  return value as unknown as T;
}

// Example.ts
import { responsiveProp as rp } from 'Function';

<Box
      bg={rp({ sm: 'black', md: 'white' })}
      p={rp([2, 4, 8])}
    >
    // ...
</Box>

they already have useBreakpointValue which sort of does the same thing.  Except it's poorly typed.  It should be typed as
type BreakpointsObject<T> = {
  base: T,
  sm: T,
  md: T,
  lg: T,
  xl: T
};

type Breakpoints<T> = T[] | BreakpointsObject<T>;
function useBreakpointValue(bp : Breakpoints<T>): T

The issue is resolved at v3.2.


FIxes TypeError: got multiple values for argument

Fixes #3923.
self.profiler.analyze should not take self as otherwise filename=self, wdir=filename and wdir=wdir

Thanks a lot for your contribution. I noticed this error before but I haven't had time to fix it :-)

By, the way, I'll backport your PR so it can be part of Spyder 3.1


The context provided to the expr inside a State should be exactly the context in this state

Description
I spent lots of hours debugging my state machine with some logic like the below transition code:
      step1: {
        entry: [
          actions.assign({
            lastSetBy: (_) => {
              spy("states.step1.entry.assign");
              return "states.step1.entry.assign";
            }
          }),
          (ctx) =>
            spy("states.step1.entry.expr context.lastSetBy:" + ctx.lastSetBy)
        ],
        always: "step2",
        exit: [
          actions.assign({
            lastSetBy: (_) => {
              spy("states.step1.exit.assign");
              return "states.step1.exit.assign";
            }
          }),
          (ctx) =>
            spy("states.step1.exit.expr context.lastSetBy:" + ctx.lastSetBy)
        ]
      },
      step2: {
        entry: [
          actions.assign({
            lastSetBy: (_) => {
              spy("states.step2.entry.assign");
              return "states.step2.entry.assign";
            }
          }),
          (ctx) =>
            spy("states.step2.entry.expr context.lastSetBy:" + ctx.lastSetBy)
        ],
     ....

Expected Result
I thought that in the states step1, the context should be set by the entry actions of the step1.
Actual Result
In the states step1, the context is set by the entry actions of the step2.
I got the below result, which means that in the states.step1, the entry actions expr function has been passed a context in states step2:
    "states.step1.entry.expr context.lastSetBy:states.step2.entry.assign"

Reproduction
https://codesandbox.io/s/xenodochial-margulis-4eky0?file=/src/index.ts
Additional context
After spending lots of hours debugging, I decided to write the unit test above, and learned this behavior: if the entry/exit is in a state with always transition, then the context is passed to them will not be the context in their state.
I can understand this is because we treat the transitions as a whole batch, however, I think this behavior is not natural as the users' intuition, and I'd like to suggest that it should be treated as a BUG.
Workaround
According to this great post Automating Spaceships with XState, @stafford Williams, Mar 22, 2021, we can split the batched transition by changing the always to after: { 0: ... }, which can be proved in my unit test code.
My suggestion
We should always provide the context to an expr exactly the same as its current state.
If this behavior change will introduce breaking change for the v4, then we should include this breaking change in v5, to make the state change code more intuitive and controllable.

This is already fixed in v5. In v4 you can get the fixed behavior if you set preserveActionOrder: true at the top level of your machine

Awesome, great to know this can be fixed with the preserveActionOrder option! I have run my test with it again and the order is expected now.
Is there a way to use v5 for users like me? I'd like to develop with v5 to prevent future problems like this (already fixed).
Thank you very much!

Is there a way to use v5 for users like me? I'd like to develop with v5 to prevent future problems like this (already fixed).

Not yet! We're working on the final tasks to release an alpha, hopefully by the end of the year.

I hope we can complete the final tasks soon and looking forward to using the v5 alpha in 2021!
Thank you very much for working on the great XState!


LED/Spotlight switch not working

Describe the bug
When I turn on the Camera spotlight nothing happens.
To Reproduce
Add Camera via Reolink intergration, add switch to Lovelace and click.
some logs:
2022-01-05 20:28:34 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1444444444444444449360d3b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:28:38 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1a44444444444444444444d3b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:28:42 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1a4444444444444444444444b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:28:46 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1a4444444444444444444444b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:28:50 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1a4344444444444444444443b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:28:54 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook d11415cf1a43444444444444444460d3b1cb94f83eefa68456e96bed06705a from <IP_ADDRESS>
2022-01-05 20:48:02 WARNING (MainThread) [reolink.camera_api] Host: <IP_ADDRESS>: Failed to get results for Search, JSON data was was empty?
Logger: reolink.camera_api
Source: /usr/local/lib/python3.9/site-packages/reolink/camera_api.py:1529
First occurred: 8:25:02 PM (2 occurrences)
Last logged: 8:48:02 PM
Host: <IP_ADDRESS>: Failed to get results for Search, JSON data was was empty?
Expected behavior
The Spotlight to turn on
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:
Please provide useful information about your environment, like:

Home Assistant version - 2021.12.8
Reolink camera model - E1 Outdoor
Camera firmware number - v<IP_ADDRESS>0_21071466

Additional context
Running v0.46
The Curl command to turn it on works;
curl -X POST 'http://yourcameraIP/cgi-bin/api.cgi?user=yourusername&password=yourpassword' -d '[{"cmd": "SetWhiteLed", "param": { "WhiteLed": {"bright" : 100, "channel": 0, "mode": 1, "state": 1}}}]'
The siren switch works.

downgraded to v0.45 same result,
Current workaround is a command line switch using the curl commands for on and off.

`switch:

platform: command_line
switches:
front_camera_spotlight:
friendly_name: Front Camera LED
command_on: >
curl -X POST '{"on":true}' 'http://<IP_ADDRESS>/cgi-bin/api.cgi?user=######&password=########' -d '[{"cmd": "SetWhiteLed", "param": { "WhiteLed": {"bright" : 100, "channel": 0, "mode": 1, "state": 1}}}]'
command_off: >
curl -X POST '{"on":false}' 'http://<IP_ADDRESS>/cgi-bin/api.cgi?user=######&password=#######' -d '[{"cmd": "SetWhiteLed", "param": { "WhiteLed": {"bright" : 100, "channel": 0, "mode": 3, "state": 0}}}]'
`

Also experiencing the same problem with the E1 Pro. Will attempt the curl command switch as a work-around.

Reolink RLC-520A - same result :-(

At Reolink RLC-520A, the Spotlight switch does not work at all. Even through the web menu of the camera itself. I propose to exclude Spotlight switches at 520A

Looking into getting the RLC-511WA, could anyone comment on if this is working?


Support setCapture and releaseCapture for mouse

Please support checks for the mouse capturing interface Element.setCapture and Document.releaseCapture.
See setCapture and releaseCapture at MDN, as well as the entry under the WHATWG API ToDo list (versioned link) which refers to this mailing list post.

(According to https://bugs.chromium.org/p/chromium/issues/detail?id=246536#c7 , these methods are superseded by Pointer Events' setPointerCapture()/releasePointerCapture() APIs.)
These might still be interesting for historical reasons though.


⚠ Orpheus has degraded performance

In 6bb39c1, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 1786 ms

Resolved: Orpheus performance has improved in 1f506b7 after 7 minutes.


 to stderr instead stdout, but it is not error.

Thanks!


Fixed Card List Positioning

Fixes Issue #409
Changes proposed
This pull request addressed issue 1 in Enhancements, which was to shift the card list to the center for better visuals.
Screenshots
Older layout:

Proposed layout:

with many elements

with a single row of elements

with a single card

with a single element on new row

for smaller screens

can you tell me what exactly are you facing? Like breaking down as in?

can you tell me what exactly are you facing? Like breaking down as in?

It's showing a scrollbar for small screen (14 inches) also the color is breaking down

can you tell me what exactly are you facing? Like breaking down as in?

Looks like we have overflow along X as it seems from the screenshot

Yeah, it needs to be fixed for different screen, as one may not work for each screen

can you tell me what exactly are you facing? Like breaking down as in?

Looks like we have overflow along X as it seems from the screenshot

can you tell me what exactly are you facing? Like breaking down as in?

Looks like we have overflow along X as it seems from the screenshot

Yeah, it needs to be fixed for different screens, as one may not work for each screen

Oh okay got it. Let me look into it.

@rupali-codes can you check the new commit once and let me know? I have made the changes and its working fine for me now.

can you tell me what exactly are you facing? Like breaking down as in?

Looks like we have overflow along X as it seems from the screenshot

can you tell me what exactly are you facing? Like breaking down as in?

Looks like we have overflow along X as it seems from the screenshot

Yeah, it needs to be fixed for different screens, as one may not work for each screen

Oh okay got it. Let me look into it.

Nope,

I guess it would be better if you can re-position the cards in your way, like make it from scratch, just the design should be similar to the current design. What do you think?

Okay sure. Let me try it.

Hi @JA-26-01! 😄 I've made some suggestions on how to make your cards more semantic. I hope you find them helpful! 😄 Also, avoid pushing the PR to the main branch. It'll make the PR process difficult. Other than that, well done! 😄

Oh thank you so much. This looks much more structured. And yes, will keep that it mind! Actually my exams are going on so could not give it a go right now.

Hi @JA-26-01! 😄 I've made some suggestions on how to make your cards more semantic. I hope you find them helpful! 😄 Also, avoid pushing the PR to the main branch. It'll make the PR process difficult. Other than that, well done! 😄

Oh thank you so much. This looks much more structured. And yes, will keep that it mind! Actually my exams are going on so could not give it a go right now.

Sure, you can take your time and good luck for the exams :)

Hey @JA-26-0! 😄 Are you still working on this?

Closing this PR because of inactivity, feel free to open a new PR whenever you are back!


QuasiDistribution and ProbDistribution return integer keys

Environment

Qiskit Terra version: master
Python version: 3.10
Operating system: OSX

What is happening?
The keys for Counts are bit-strings corresponding to observed measurement values.  However QuasiDistribution and ProbDistribution are returning int keys.
How can we reproduce the issue?
counts = {'001': 67,  '110': 113, '100': 83, '011': 205, '111': 4535, '101': 100, '010': 42, '000': 4855}
quasi = QuasiDistribution(counts)
quasi

gives
{1: 67, 6: 113, 4: 83, 3: 205, 7: 4535, 5: 100, 2: 42, 0: 4855}

What should happen?
The keys should be bit-strings as that is what is needed for computing expectation values and allows users to work with distributions in the same manner as Counts
Any suggestions?
No response

I like the idea to store keys as bitstrings. We can then handle bitstrings with blank spaces. QuasiDistribution currently raises an error with bitstrings with blank spaces.
from qiskit.result import Counts, QuasiDistribution

counts = Counts({"0 0": 10, "1 1": 20})
print('counts', counts)

quasi = QuasiDistribution(counts)
print('quasi', quasi)

output
counts {'0 0': 10, '1 1': 20}
Traceback (most recent call last):
  File "/Users/ima/tasks/2_2022/qiskit/terra/tmp/8744.py", line 6, in <module>
    quasi = QuasiDistribution(counts)
  File "/Users/ima/envs/qiskit310/lib/python3.10/site-packages/qiskit/result/distributions/quasi.py", line 68, in __init__
    raise ValueError(
ValueError: The input keys are not a valid string format, must either be a hex string prefixed by '0x' or a binary string optionally prefixed with 0b

So I talked with @mtreinish a bit about this and it is really a design decision at the end of the day.  The data is really only useful in bit-strings, but the most efficient representation is as int.  As the systems get larger this will become more and more important.
Because we use QuasiDistribution as a generic object for both real- and true quasi-distributions one just needs to make sure that they can be used in place of Counts in places like visualizations etc.  That the same object can also have negative values makes that a bit more tricky though.
For mitigated Sampler results, the only useful thing one can do is to compute expectation values and #8748 does the casting internally, so bit-strings or not should be fine there.

I am curious how we could still enable multiple register output from int storage though.  That seems to be nice to have for parity with Counts

I'm leaning towards something like this for counts/distribution/quasi-prob distributions. (I wrote some of this in another issue, but I can't find it at the moment.)

Python int to represent outcomes (keys). In some cases some kind of fixed length bit field would likely be more efficient for storage and computation. The latter might facilitate interaction with Rust and c++. (or heck, Julia). An API that hides whether it is int or bitfield would be good, so we are not stuck with one representation.

The data is really only useful in bit-strings, but the most efficient representation is as int

Bit operations are usually more efficient with machine integers than with binary strings or tuples. Not always. In particular, if we code strings with 1, 0, Z, I it's not obvious to me how to do expectation values efficiently with bitwise logical operators on integers. But, it might not be too hard to find a way. Without projecting (with 1 and 0) IIRC I found bitwise operations on ints were faster for expectation values.

We can then handle bitstrings with blank spaces

I guess that storing information about register sizes, for example "1010 0101" for two registers each of length four, is better done as meta data. This should be more efficient for storage and processing. For example outcomes ["1010 0101", "1010 0000"] would be stored as [165, 160] and reg_lengths = [4, 4]. I also guess code would be more maintainable.

Of course it makes sense to make the storage and code for Counts, ProbDistribution, and QuasiDistribution as close as possible. In fact, they can probably share quite a bit of code. There are a lot of differences currently.

You need a bit of abstraction to make this work elegantly. In Python, it would come at a cost.

Would our own integer type, which you could convert to string (in the correct Qiskit-order 😉) solve the issue? We could store our_int type as key and if you want to use it as bitstring just do our_int.to_binary() or so?

our own integer type ... just do our_int.to_binary()

Yes, that's what I'm talking about. Something like this:
class Outcome:
    def __init__(self, data):
        if isinstance(data, int):
            self._data = data
        elif isinstance(data, str):
            self._data = int(data, base=2)
        else:
            raise TypeError("Input must be int or str")

    def __repr__(self):
        return bin(self._data)[2:]

    def __str__(self): # maybe str and repr could be different
        return bin(self._data)[2:]

    def __len__(self):
        return (self._data).bit_length()

    def __int__(self):
        return self._data

    def __getitem__(self, index):
        return bool(self._data & (1 << ((self._data.bit_length()) - index - 1)))

    def __iter__(self):  # It would probably be better to use bits and avoid str.
        return (True if b == '1' else False for b in str(self))

    def __eq__(self, other): # should be faster than comparing strings.
        return self._data == other._data

It works like this:
x = Outcome('11100001')

print("display in repl/notebook: ", x)
print("len is number of bits: ", len(x))
print("as string: ", str(x))
print("as integer: ",int(x))
print("iterate over bits: ", [bit for bit in x])
print("indexing         : ", [x[i] for i in range(len(x))])

with output
display in repl/notebook:  11100001
len is number of bits:  8
as string:  11100001
as integer:  225
iterate over bits:  [True, True, True, False, False, False, False, True]
indexing         :  [True, True, True, False, False, False, False, True]

But, this is not a good solution as is. For example:
In [2]: Outcome('000001')
Out[2]: 1

We need to store the number of bits in the outcome. But, in a container of Outcomes this would be wasteful and inefficient to store the same length for all outcomes, and/or compute values on the fly when you manipulate them. It may be better to store this information on the level of a collection of outcomes, for example a dictionary of counts. You could use a class containing a dict with int keys. The number of bits in the outcome (and other metadata) would be stored just once.
Trying to implement these things is instructive. Some problems arise because Python is interpreted. But, other design problems, like where to store the length, would be the same in Rust or Julia.
Looking at things like bitstring might be useful, too.


nixos/seafile: add persistent user, configurable storage path, and gc service

Description of changes
Disable DynamicUser for seafile in favor of a persistent configurable service user and group. This allows moving the storage directory out of the systemd-managed StateDirectory (previously, external paths could be bind-mounted or symlinked into the state directory, but UID reassignments meant having to manually and slowly chown the entire storage).
This also allows access by other services, such as by a garbage-collection timer also added here.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@greizgh looks you weren't auto-requested by the bot since the module is missing the meta.maintainer attr. Would you review this PR?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3863

@greizgh looks you weren't auto-requested by the bot since the module is missing the meta.maintainer attr. Would you please review this PR?

Sorry, completely missed that. This lgtm! Thanks for your contribution, this was deep down on my todo list and I never got the time to look into GC.

Added you and @schmittlauch to the maintainers for this module, so we can have the maintainer approval tag hopefully.

Seahub needs to be updated anyways so it can build again, version 10 is using django 3 (marked insecure), and there's no PR yet to update to 11. I'll take a look at this again once that PR's in motion.

Forgot to include it, update PR is #318727. Switching to draft until that's merged...

Closing in favor of #318727


Event OnClear

Hello,
On clear, i need to set the highlight date to today's date for the user when opens the calendar next time see today's date selected.
There is any event called OnClear? If not, whats the workarround for this ?
Cheers!

i found a workarround for this:
picker.on({       set: function(thingSet) {         if(thingSet['clear']===null)             picker.set('highlight', ...);       }     })
still, i think it would be a nice option

Its a shame that there is no onClear event handler, would make things so much easier. Still, thankful for the plugin, dont get me wrong :). What i did was:
$('.picker__button--clear').on('click', function(e) {
var elementId = $(this).attr("aria-controls");
var aData = $('#'+elementId).data();
aData[elementId] = '';
// Ajax call
...
});
Caveat: had to re add the event handler every time i clicked the clear button, not sure why, probably the plugin unbinds events. Hope it helps someone.


Integration of Mayhem with thin-edge.io

Proposed changes
Added integration of Mayhem to the thin-edge.io project. This adds a GitHub action which creates a Docker image and automatically uses Mayhem to fuzz the project. Integration for this repository was relatively straightforward, as LibFuzzer was already used in the project. A Dockerfile was added to create the image, and a Mayhemfile and YAML file were added to run the Mayhem GitHub action.

Awesome, LGTM!


Create packages with required and soft dependencies

Fixes https://github.com/alan-turing-institute/sktime/issues/1142
Creates two packages: sktime which includes the minimal set of dependencies and sktime-all-extras, which includes all the dependencies except pmdarima, tbats, and catch22. My thoughts as to why those are not included are here: https://github.com/alan-turing-institute/sktime/issues/1142#issuecomment-887846036
Checklist

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

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

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

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

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

@conda-forge-admin, please rerender

@dhirschfeld Thanks for the feedback -  I have addressed your comments!

Thanks @freddyaboulton and @dhirschfeld!


Extending CLI for Fine-Tuned Whisper Models on Hugging Face

Hi,
In addition to the standard models such as tiny, base, small, medium, and large, the community has also created fine-tuned models like the one found at https://huggingface.co/sefa-alper/whisper-turkish-large-v2.
Could you please provide guidance on how we can incorporate these fine-tuned Whisper models into the CLI for seamless integration and use?
Thank you in advance.

Hello. This is explained here: https://github.com/Softcatala/whisper-ctranslate2/blob/main/README.md
If it's not clear, let me know and I will improve the documentation. Thanks

I use this powershell script to get the models to the ct2 format
$modelPath = "D:/Programs/WhisperDesktop" #no trailing slash
# https://huggingface.co/bofenghuang/whisper-large-v2-cv11-german
$modelAuthor = "vumichien"
$modelName = "whisper-large-v2-cv11-german"

Write-Output @"
from transformers import AutoProcessor, AutoModelForSpeechSeq2Seq, AutoTokenizer

# load model
processor = AutoProcessor.from_pretrained("$modelAuthor/$modelName")
model = AutoModelForSpeechSeq2Seq.from_pretrained("$modelAuthor/$modelName")
tokenizer = AutoTokenizer.from_pretrained("$modelAuthor/$modelName")

# save model with tokenizer.json
processor.save_pretrained("$modelPath/$modelName")
model.save_pretrained("$modelPath/$modelName")
tokenizer.save_pretrained("$modelPath/$modelName")
"@ | python

# replace / with \ in path
$modelPath = $modelPath.Replace('/', '\')
# convert to ct2
Invoke-Expression "ct2-transformers-converter --model '$modelPath\$modelName' --output_dir '$modelPath\ct2-$modelName' --quantization float16 --copy_files tokenizer.json"

Afterwards you can use it by adding
--model_directory 'D:\Programs\WhisperDesktop\ct2-whisper-large-v2-cv11-german'
Do not add --model !
Sadly right now I also don't get any output files generated on Windows, but the approach would be the same on linux.

CTranslate2 provides ct2-transformers-converter (https://opennmt.net/CTranslate2/guides/transformers.html) which provides a cli to convert fine-tunned models. I recommend this using this tool.

Ah yes it works. The information provided on the README was sufficient. Thanks @jordimas  and @Alistair1231  .


Snapshot generation parses out/doesn't include doctype declaration?

I'm noticing the generated snapshots don't include a doctype at the top, even if one exists in the source file(s). Not a huge deal, but it does cause HTML validators to throw an error.

Same problem occurs over here.

+1


Question about switching controller (eg. switch from keyboard to steering wheel)

Hi, I'm running the "manual_control.py" script (in example folder) in my own map and  I want to achieve one function: switch driving method from keyboard to steering wheel by pressing one button. Could you please give me some instructions on that? Thanks a lot!
Yuan Shi

if you had connected a steering wheel to carla?

But I am wondering how did you solve the problem, as I met the same problem.  Would you mind giving me some suggestions regarding this question. Very much thank!
I had a steering wheel but it can't be connected to Carla in Win10, How it can be solved?

Hello, I'm curious about your steering wheel, do you use the Logitech steering wheel? Or series of Thrustmaster?


BiDirectional text support

In the modern world, the support for some languages that are right-to-left and mixing them with left-to-write words and numbers is crucial for a different software. At some point, this should be supported everywhere, including the console.
I started to gather the information about BiDi support in various console terminals (and programs) in analogy with true color crusade at https://gist.github.com/XVilka/a0e49e1c65370ba11c17

New console BiDi specification was recently implemented in libvte by @egmontkob: https://terminal-wg.pages.freedesktop.org/bidi/implementations.html#vte
Note, that with the release of GNOME 3.34 the support of BiDi is available in Gnome Terminal out of the box, which makes checking the implementation in xst and comparing them way easier.
Sadly, tmux doesn't have support for BiDi yet.

IMHO xst unusable without tmux but that's subjective :3

Closing this -- if st gets RTL or a patch floats around can reopen


Add preview job to cops pipeline

should land after https://github.com/openstax/output-producer-resource/pull/12


Implement HIR visitor

Describe the feature
Implement HIR visitor traits and functions using the visitor macro, similar to the AST ones.
The main difference here is that we need to recurse using the Hir struct, which means that there should be an extra fn hir(&self) -> Hir<> method to return it and that is used in the other methods to recurse.
Additional context
No response

hi, can i take on this ?

Any update on this ?

Sure go ahead

Ok on it


(WIP) Use Cargo.lock to identify wasm-bindgen version.

Fixes #270. This aims to find the wasm-bindgen version using the lockfile.
Note: This is still a WIP, and should probably land after #271. While working on this, I ran into some of the same problems that were solved by that PR, so I tried to remove any duplicate work. I can rebase this once that lands, but review/feedback is welcome in the meantime :)

Make sure these boxes are checked! 📦✅

[x] You have the latest version of rustfmt installed and have your
cloned directory set to nightly

$ rustup override set nightly
$ rustup component add rustfmt-preview --toolchain nightly

[x] You ran rustfmt on the code base before submitting
[x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Unrelated, but of note: I've had a lot of trouble running the test bench on my machine. Admittedly, this might be a resources problem (< 4Gb RAM), but it might be worth thinking about in the future for other contributors :)

Unrelated, but of note: I've had a lot of trouble running the test bench on my machine. Admittedly, this might be a resources problem (< 4Gb RAM), but it might be worth thinking about in the future for other contributors :)

When running the wasm-pack tests, I've had issues that I think boil down to doing lots of I/O on encrypted disks on linux but I'm not totally sure. I have low CPU usage, low mem usage, and yet my system is super janky. Not totally sure what's up.
If you think you might have similar problems, I'd love to hear more. Or just generally what appears to be the bottleneck for you: CPU, memory, etc and how you determined that.

Awesome, thanks so much for the help/review everyone! I'll spend some more time on this PR this week, and should hopefully have it finished up by Friday at the latest. 😸

Rebased and updated in #324.
Sorry to snatch this PR away from you @data-pup! I don't like to do this. Just really want to get a new release of wasm-pack out (see all the recent issues filed that have solutions on master) and this was the last thing blocking us :( Thanks for understanding!


[stable/redis] Metrics Exporter container created as a separate pod

Describe the bug
This chart (versions 6 -> 8) create a separate deployment, pod and service for metrics, instead of creating a sidecar container for each pod
Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.14.1", GitCommit:"5270352a09c7e8b6e8c9593002a73535276507c0", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.1", GitCommit:"5270352a09c7e8b6e8c9593002a73535276507c0", GitTreeState:"clean"}
openshift v3.11.16
kubernetes v1.11.0+d4cacc0
Which chart:
stable/redis
What happened:
This results in one exporter scraping metrics from all running instances of REDIS at once which then results in incorrect metrics.
E.g. the rate(redis_commands_total[1m]) says about 60K commands per second are being executed on a cluster that has basically no activity on it.
What you expected to happen:
each pod of the redis-stateful set should have a second (or third if sentinel is also activated) container with the redis_exporter image.
For reference, this is what the redis-ha chart does.
How to reproduce it (as minimally and precisely as possible):
just helm install stable/redis
Anything else we need to know:

Hi @acaligiuri ,
Thank you for reporting, we will evaluate your request as soon as possible.

If needed I can upload screenshot or the raw Prometheus metrics

Hi @acaligiuri ,
I checked with the team and we are not sure about what is the benefit to create it as a sidecar container. We received in the past some issues requesting to create it as a separate deployment.

Hi @miguelaeh, I was unaware of the previous issues. However, I don't see how it could work as a separate deployment. The issue I see is that as a single separate deployment, every call the exporter makes to the redis service will be loadbalanced to a different pods (the exporter is using the NON-headless service so it's loadbalanced according to the iptables probability which is 1/) henc prometheus will create an incorrect time series:
e.g. imagine I have a 2-instance setup. If one server's 'info all' command responds like this :
Commandstats
cmdstat_set:calls=100,usec=867,usec_per_call=8.67
and the other shows this other metric:
Commandstats
cmdstat_set:calls=11,usec=867,usec_per_call=8.67
Chances are prometheus will read one first and then the other. That means prometheus might see a difference of 89 commands between scrapes. If we assume the default value of 1 scrape every 15s, that translates in 89/15= 5.93 calls per second even in COMPLETE ABSENCE of traffic.
This case is quite easy to reproduce, just create a cluster using your chart, leave it with no traffic at all, then query prometheus for the redis_commands_total metric. Mind you this should IS a counter, that means the time series should be a constantly growing number. If you see it going up and down then there's a problem.


Do not use Cloudflare

Cloudflare basically offers TLS with MITM built in. More information here: https://scotthelme.co.uk/cloudflares-great-new-features-and-why-i-wont-use-them/
So it is not a good idea to use it for such security-related issues.

It's a fair point.
The real reason why I use cloudflare is that it's the simplest way to apply https. If I didn't use cloudflare I'd use someone else (amazon aws, google cloud, etc) and you'd have to trust them instead. Even if I host it on my own hardware you still need to trust github, and no matter what you need to trust the site maintainer (me!).
Most people will be happy with the author -> github -> cloudflare chain, there are all top tier hosting providers who aren't going to risk their reputations to backdoor my site. If I cut out cloudflare you still have to trust github and myself. The more paranoid users won't be comfortable even with this.
I'm open to suggestions, but I don't see a huge win here. If you're paranoid you should check the CSP headers manually (see: https://github.com/Gaff/pgp.help/blob/master/README.md ), and if you're happy to give up some trust then https should be sufficient.

I think if you're paranoid you should host it by yourself. :smiley:
And e.g. when accessing the GitHub page at https://gaff.github.io/pgp.help you can circumvent Cloudflare and you only have to trust GitHub (and possibly you :wink:).
Of course you have to trust a few organisations, but it matters who and how many organisations you trust. You could put hundreds of companies in your connection and allow them to MITM the traffic, but the goal of HTTPS is to protect this information and not let someone MITM it.
But anyway as you can use GitHub pages there is a decent alternative. :smiley:

Fair enough. I don't tend to advertise that link as the URL is ugly, but you're right it is one less MITM.
I should add it as a note in the README for those who are concerned about such details.

At least the ones who are really concerned may find this issue. :smiley:

Additionally what SSL mode in Cloudflare do you use? Some modes transmit the content of the website partially as plain-text.

I'm using Full SSL, so it will only transmit SSL encoded data from github.

@Gaff commented on 19. Dez. 2015, 18:33 MEZ:

I'm using Full SSL, so it will only transmit SSL encoded data from github.

That's true, but with Full SSL Cloudflare does not check the cert at all. It even accepts self-signed certs and this basically allows active MITM attacks to tamper with the connection. So what about using at least "Full SSL (strict)"?
https://scotthelme.co.uk/cloudflares-great-new-features-and-why-i-wont-use-them/#sslallthethings

I should. I tried it and cloudflare didn't seem to accept the github SSL certificate :(.
Need to research some more...

Ah you serve it directly from GitHub? But https://gaff.github.io/pgp.help should be accepted in any case.

Exactly. Exactly.
Have sent cloudflare a mail, though I'm not paying them so I'm not sure I'll get a prompt response.

A good company should also offer support for not-paying customers, so I am interested in hearing their response.

So, some time has gone. Any news from Cloudflare?

Alas no.
I guess nowadays the way to host this would be docker + letsencrypt, still means I need to deal with hosting (and any hosting nastyness) myself. Not 100% sure I want to open this can of worms :/. I would like to improve the security though.

Loading it from GitHub pages directly would still be a good idea for some users which do not want to use Cloudflare.
However currently https://gaff.github.io/pgp.help/ redirects to the Cloudflare home page and AFAIK it does also not show the favicon. A inline html tag would be required for this AFAIK. At least such a tag would be nice - for self-hosted installations where this is e.g. placed in a subdir or there is no global favicon.
I can of course open a separate issue for this if you want.

Or even better fix it for me ;)
My original intention was to serve this from github pages - sadly github have no support for https when you bring your own domain.
Really I should jump the hurdle and host it myself using lets-encrypt powered https on some cloud somewhere.

Hmm... could you at least remove the redirection from the official GitHub pages site to the custom domain? If that's possible.

BTW there are CDNs which have Let's Encrypt support and therefore allow you to use GItHub pages over HTTPS. E.g. this here.

Oooh, that's pretty cool!

Ok I had a go a kloudsec, but it didn't "just work". Alas DNS has horridly non-deterministic propagation times so it could be that I just didn't wait long enough, but I didn't want to leave the site down. Have moved a lesser site of mine to kloudsec. If that works I'll give it another go.

Hmm, ok looks like it does work - I just needed to give it longer. However from a security perspective I'm not sure this achieves much. Sure the SSL certificate is signed by lets encrypt not cloudflare - but apart from that it's the same, complete with the same implicit MITM trust problem. Seems to be simply a question of who do you trust more? Cloudflare or Kloudsec?

Yeah, but it might made a difference when passing the request from GitHub Pages to the CDN provider. As with Cloudflare you could not use "Full SSL (Strict)"... But I have no idea how Kloudsec handles this...

FYI GitHub now forces HTTPS for fifth pages. You only have to click on a switch:
https://help.github.com/articles/securing-your-github-pages-site-with-https/

Tried that - doesn't work with custom domains / CNAMEs :(
Shame as that would be pretty neat. I'm even considering moving to pgphelp.github.io (wouldn't be the first project to live at github). Wdyt?

I'm even considering moving to pgphelp.github.io (wouldn't be the first project to live at github). Wdyt?

Yeah, that's the easiest solution and I think it is a good decision.

It's not so black-and-white, github doesn't support HSTS for example.

But they seem to deploy it:

Heyyyyy, GitHub Pages now sets HSTS for new github.io sites! https://t.co/He3PNJKTVw

https://twitter.com/konklone/status/743172058289430528

Wow looks like it is indeed configured that way but only for new users (or perhaps new github pages). How do you find out this stuff?!
That means I could safely move to: pgphelp.github.io  (or equivalent? Suggestions? Sadly pgp.github.io is taken)
I think I'm broadly in favour. Essentially it's one less person to trust at the cost of a slightly uglier URL. I can still maintain pgp.help as a landing page. Any further thoughts?

A question for your excellent search skills - A few months ago I saw a site that would grade your websites HTTPS usage (for things like HSTS, HPKP, CSP etc. But I can't find it again :/ If you do locate anything please let me know!

How do you find out this stuff?!

I already linked to my sources. :smiley:

That means I could safely move to: pgphelp.github.io (or equivalent? Suggestions?

I think so, yes.

Sadly pgp.github.io is taken)

Yeah, not really surprising though...

Essentially it's one less person to trust at the cost of a slightly uglier URL. I can still maintain pgp.help as a landing page.

Yes and yes.

Any further thoughts?

No. :smiley:

A question for your excellent search skills

:sunglasses:

saw a site that would grade your websites HTTPS usage (for things like HSTS, HPKP, CSP etc.

No need to search. I know this website.
I think you mean this: https://securityheaders.io/
BTW: CSP has nothing to do with HTTPS. :smiley: But it's a very good feature anyway.
Basically the site is about testing (and grading) your Security Headers usage, which of course include some headers related to HTTPS. As for HTTPS there are many more other analysis sites, but these tests are only useful when you run your own server (and do not serve it via GitHub Pages as you cannot influence their cipher selection e.g.).

I think you mean this: https://securityheaders.io/

You sir are a genius!
So neither cloudflare nore github to particularly well at this metric, but cloudflare is marginally better:
https://securityheaders.io/?q=https%3A%2F%2Fpgp.help%2F
https://securityheaders.io/?q=https%3A%2F%2Fgaff-hsts.github.io
Does this matter? Probably not.

...taken the first steps - see https://pgphelp.github.io . It's not working yet but you can inspect it for https-correctness at least.

Does this matter? Probably not.

Yeah, that's just one simple header which affects only some browsers, so it does hardly matter. What's much more important is your CSP - but this is independent of the hoster as you declare it in the HTML code.

It's not working yet

Everything works for me...

but you can inspect it for https-correctness at least.

Well... it's the default for GitHub pages, which is okay. You cannot do anything about it anyway.

One obvious downside - since the website has a new home, you need to copy over all your settings. Bleh.

Mhh, yeah...
When you'll move you might see how much complains you get. Maybe clarify this fact on the old page and let some time pass untl you shut down the old URL. (Or add an import/export feature :wink: )

Everything works for me...

Cool - I mean the page still refers to the old URL in a few places. But yeah think I nailed the technical bugs (all due to npm craziness - turns out if you rebuild you automagically pull in newer not-necessarily-compatible packages. Not fun).
I'm slightly troubled since I'm sure one day github will support https + external domains. But will that be in 3 months or 3 years? Who knows.
My upgrade plan is to have pgp.help show a landing page with a link to the new and archive version. pgphelp.github.io will go straight to the pgp page as it is.
Sadly one of the downsides of my extremely strict security policy is that I can't install any tracking / analytics packages - so I've no idea how many users I actually have :). Probably not many.

Probably not many.

Yeah, based on the stats in this repo (starts, issues opened, ...) I alos assume so.

Sadly one of the downsides of my extremely strict security policy is that I can't install any tracking / analytics packages

Yeah, but at least this repo is constantly tracked by GitHub. As the repo owner you can see some stats for it somewhere under https://github.com/Gaff/pgp.help/graphs/ AFAIK.

A new testing site similar to SecurityHeaders.io is also this one: https://mozilla.github.io/http-observatory-website/

BTW there was alos a blog post about the move to HTTPS: https://github.com/blog/2186-https-for-github-pages

Any news about the switch to the new site?

Well it works. I need to formalise things. Keep on badgering me about it and I'll get it done :/

New year, new try?

How is it going?

Finally GitHub provides HTTPS for custom domains: https://blog.github.com/2018-05-01-github-pages-custom-domains-https/
So this may be the best solution for you.😊

Thanks!
I'll try and take a look this weekend.
On Tue, 1 May 2018, 20:02 rugk<EMAIL_ADDRESS>wrote:

Finally GitHub provides HTTPS for custom domains:
https://blog.github.com/2018-05-01-github-pages-custom-domains-https/
So this may be the best solution for you.😊
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Gaff/pgp.help/issues/26#issuecomment-385758593, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAN1aWXt_loQeLDLLLUNrMdAlaxwmS7gks5tuLFTgaJpZM4G1-oS
.

So I'm going to try this - however it might take the site down if I screw up. Unfortunately because of DNS propagation delays it might take a day to figure out. :(
Wish me luck!

Ok - it seems to have worked. I did blow out my HSTS settings in my browser at one point, but have checked on other browsers and it seems ok. However it /may/ take a day for the DNS settings to propagate to your ISP.
@rugk  - Let me know it looks for you, and if it looks good I'll close this ticket!

Works for me.


cpuprofile files generated by vscode for extension performance won't load on Mac

I'm not blocked, so not high pri, but this is the second time I had to go to a win32 machine to debug a performance scenario because the files won't load on a Mac.  For example, try the one in https://github.com/microsoft/vscode-azurearmtools/issues/1051
I always get this:

Can’t load file. Supported file extensions: .cpuprofile.

(and yes, I did rename to .cpuprofile)

I'm unable to reproduce this. After renaming the file I'm able to open it in Chrome on both OSX and Windows, as well as in VS Code's built-in profile visualizer tools.
/needsMoreInfo

Here's the zip with the renamed file:
msazurermtools.azurerm-vscode-tools-unresponsive.cpuprofile.zip
Using dev tools to open on Mac in vscode Version: 1.50.1 Commit: d2e414d9e4239a252d1ab117bd7067f125afd80a Date: 2020-10-13T14:53:05.704Z Electron: 9.2.1 Chrome: 83.0.4103.122 Node.js: 12.14.1 V8: <IP_ADDRESS>-electron.0 OS: Darwin x64 19.6.0:

Okay, so apparently my Mac is hiding some file extensions like .txt but not others (like .zip).  Had no idea it would do that.  Thanks.


Single selection keyboard navigation

Adds single selection keyboard navigation. Multi select to come after this code is reviewed.

Case to consider: user selects a row, then scrolls so that the selected row is no longer visible, then hits up/down. Should the table scroll to make the new selected row visible, or not?

Currently the table does scroll to do this. Based the functionality of what Apple do in OSX.

I'm running your branch locally, and it doesn't scroll to non-visible rows. To reproduce, go to the "simple" table, scroll down, click a row, scroll up, and hit the up arrow key. Your scroll logic currently only handles the case where you hit "down" and are currently selecting the bottom visible row.

This is REALLY great! My comments are all pretty minor. I look forward to the multi-select code; great work @jrhe!

@jrhe as you add multi-select, could you move this PR to the main ember-table repository?

Sure
On Tuesday, May 26, 2015, Alex Zirbel<EMAIL_ADDRESS>wrote:

@jrhe https://github.com/jrhe as you add multi-select, could you move
this PR to the main ember-table repository?
—
Reply to this email directly or view it on GitHub
https://github.com/Addepar/ember-table-addon/pull/17#issuecomment-105334480
.

--
Sent from Gmail Mobile


pos-tip support

Thanks for your work 👍
It would be nice to have pos-tip support
There is an example code
   `(pos-tip-foreground-color ,foreground)
   `(pos-tip-background-color , background)

Sure, I'd be happy to add it. However, I'm having trouble finding the faces for pos-tip.el. Where did you get pos-tip-foreground-color and pos-tip-background-color from?

I found them in solarized-theme package, sadly they dont make any change. I've found this https://github.com/pitkali/pos-tip/pull/7

Ah, it looks like pos-tip reads the background from the tooltip face, which is already customized. I'm guessing your tooltips still come up in the default black on beige? What OS are you running and what version of emacs?

@hlissner you are right. OS X 10.11.6

I recall not finding any way around that in stock Emacs, and was one reason I switched to Matsumoto's emacs-mac. If you have homebrew installed, you can install it with:
brew tap railwaycat/emacsmacport
brew install emacs-mac --with-imagemagick --with-modern-icon --HEAD


Frost Wyrms can freeze The Exodar

Note that this is being logged as a bug because it's unintuitive behaviour.


Fails to set profile in newer Firefox/Geckodriver versions

The profile defined in wdio.conf is not set in newer versions of firefox and geckodriver
Firefox: 56.0
geckodriver: 0.19.0
selenium-server-standalone: 3.6.0
According to https://github.com/saadtazi/firefox-profile-js/issues/95 the browser capabilities changed and the profile should be set like so:
cap['moz:firefoxOptions'] = {
	profile: zippedProfile
};

Instead of the old definition
cap.firefox_profile = zippedProfile;

I'm seeing this too, tried installing this add-on because I need to set configure some profile settings, it wasn't taking the profile I set in my wdio.conf.js file.
Firefox: 56.0.2
Geckodriver: 0.19.1
with selenium-standalone using 3.6.0 version of selenium.
Rolling back to 3.4.0 seemed to work though. I would like to see this work with the latest version of selenium though.

Fixed in #14


Add coefficient accessors to the BC class

This adds accessors for the underlying boundary condition coefficients. It also uses the vanilla mfem constructors instead of special factory methods for the boundary integrators.

Let's discuss how we want to handle complex boundary integrators. This is more in line with how we handle the domain integrators. Any thoughts?

LGTM

It feels like these changes are trying to return to the non-variant storage of coefficients. I'm not against this-- I think std::variant has its uses but maybe it's not the right fit for this use case.

From my point of view this allows some of the physics-specific stuff to "assert" information about a particular BC, but allows for uniform storage and in some cases application.  I'd really like to see if we can figure out a more exhaustive list of the ways that boundary conditions might be used so we can determine what a general BC interface would look like.

It feels like these changes are trying to return to the non-variant storage of coefficients. I'm not against this-- I think std::variant has its uses but maybe it's not the right fit for this use case.

Yeah, I still think the use of std::variant cleans up a lot of the internal duplication in the BaseSolver and BoundaryCondition classes, so I'm on the side of keeping it for now. However, I would like to keep solver developers away from the std::variant syntax. At that level, one should know whether to interpret the boundary condition coefficient as a scalar or vector (or hopefully tensor in the future).

It feels like these changes are trying to return to the non-variant storage of coefficients. I'm not against this-- I think std::variant has its uses but maybe it's not the right fit for this use case.

From my point of view this allows some of the physics-specific stuff to "assert" information about a particular BC, but allows for uniform storage and in some cases application. I'd really like to see if we can figure out a more exhaustive list of the ways that boundary conditions might be used so we can determine what a general BC interface would look like.

Sure enough, this is a much clearer way to say what I was trying to describe!

LGTM


Move examples out the workspace

Hey @isidroamv ! Another more accessible task might be getting all the exported functions for all the packages i.e. helpers etc to be in one index.ts file, so that imports don't need to specify a file and can be relative to root. I expect that to have less errors with tests, and might be an early easy win!

 Given that these errors aren't getting fixed, I wonder if the move is to isolate them in a seperate repo and seperate the error set.

@Divide-By-0 I will try to fix these errors in my repo.
This errors are dude the libraries are imported as local libraries instead of the package manger repository.
When the command yarn install --frozen-lockfile is executed it checks that the folder module doesn't have any change. I believe the content from my local environment differs from the CI environment because system files or hidden files.
I'm exploring the option to omit the command yarn install --frozen-lockfile. Just running yarn install should be enough. Another alternacice is to use Yalc to publish the package locally and use it. We would require install yalc globally and publish the repository locally


Adding support for selenium 3 in node config file

Sorry for the duplicated code in the configuration of the chefspec runs. Any ideas about how to do that without being so redundant?

hi @dhoer any clue as to why the rubocop checks are failing now on the AppVeyor build? I didn't modify these blocks:
Offenses:

providers/node.rb:42:1: C: Block has too many lines. [43/25]
action :install do ...
^^^^^^^^^^^^^^^^^^
providers/node.rb:43:3: C: Block has too many lines. [41/25]
  converge_by("Install Node Service: #{new_resource.servicename}") do ...
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
test/integration/default/serverspec/default_spec.rb:3:1: C: Block has too many lines. [46/25]
describe 'selenium::default' do ...
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
32 files inspected, 3 offenses detected

Rubocop update. I will bump that number to be 50 in the rubocop yaml.


[Feature] Support Proxy Protocol

Verify steps

[X]  我已经在 Issue Tracker 中找过我要提出的请求 I have searched on the issue tracker for a related feature request.
[X]  我已经仔细看过 Documentation 并无法找到这个功能 I have read the documentation and was unable to solve the issue.

Description
When we put mihomo behind a LB, we cannot get a client IP.
Thanks.
Possible Solution
github.com/pires/go-proxyproto

We won’t support it. This is not what mihomo should do.

I see, thanks for your fast reply.


LINK-99, LINK-100, LINK-102, LINK-103 | Saving event as draft or publishing event

Description

Validating for draft event
Saving event as draft / Publishing event
Showing error toast message in case saving fails.
Page to show that event is successfully created

Closes
LINK-102
LINK-103
LINK-99 and LINK-100 will be finalised in next PR when combining all event form sections to a single page
Screenshot

Testing
Testing can be done only locally using local Linked Events backend instance. You also need to add apikey to request headers by commenting line 116 in apolloClient.ts file. Same apikey needs to be added to am organization in local LE admin panel to be able to save events. EventSavedPage can be tested e.g. http://localhost:3000/fi/event/completed/lippupiste:serie-2630206 (if your REACT_APP_LINKED_EVENTS_URL=https://api.hel.fi/linkedevents/v1)

http://localhost:3000/fi/event/completed/lippupiste:serie-2630206

"Same apikey needs to be added to am organization in local LE admin panel to be able to save events." No quite sure how to do this.


Can't read files

Type: Performance Issue
I have VS Code and I downloaded Python and Gibub Co-Pilot but when I try to open up a file or folder from my Documents directory I get a message saying "can't find file because it is binary or unsupported"
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:49:37.021Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-1355U (12 x 2611)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.73GB (4.76GB free)

Process Argv
--crash-reporter-id 344e7c90-d66a-431a-a8a0-52ac67818ad6

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   103	 12572	code main
    0	   219	  2856	window [1] (● 'Tommy.py' • Untitled-1 - Visual Studio Code)
    0	    28	  3216	   crashpad-handler
    0	    97	 11008	shared-process
    0	   183	 12728	   gpu-process
    0	    82	 17784	fileWatcher [1]
    0	   251	 18904	extensionHost [1]
    0	   152	 31208	     electron-nodejs ("C:\Users\tdavi\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node c:\Users\tdavi\.vscode\extensions\ms-python.vscode-pylance-2023.12.1\dist\server.bundle.js --cancellationReceive=file:cdbf15e1211708a17a4317d349d3b6f0865d8eb40a --node-ipc --clientProcessId=18904)
    0	    81	 28184	ptyHost
    0	    63	 14556	     C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -noexit -command "try { . \"c:\Users\tdavi\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\contrib\terminal\browser\media\shellIntegration.ps1\" } catch {}"
    0	    69	 23428	     C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
    0	     7	 27756	     conpty-agent
    0	     7	 29152	     conpty-agent
    0	    94	 29252	window [2] (Issue Reporter)
    0	    47	 29320	   utility-network-service

Workspace Info
;

Extensions (6)

Extension
Author (truncated)
Version

gitlens
eam
14.6.1

copilot
Git
1.150.0

copilot-chat
Git
0.11.1

vscode-dotnet-runtime
ms-
2.0.0

python
ms-
2023.22.1

vscode-pylance
ms-
2023.12.1

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
vscaac:30438847
vsclangdc:30486549
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30933248
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302

Hi @tdavid12 could you please tell us what the type of the file you are trying to open is? Could you include perhaps a screen recording of the issue you are seeing?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

I am using a laptop.

On 01/18/2024 3:13 AM EST VSCode Triage Bot @.***> wrote:
Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines https://aka.ms/vscodeissuereporting. Please take the time to review these and update the issue.
Happy Coding!
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/202685#issuecomment-1897994992, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE7BFLCKM7T45ZBXPGDEDXTYPDKTZAVCNFSM6AAAAABB7IHD36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJXHE4TIOJZGI.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @tdavid12, please give more details as to the issue. If you are trying to open word files in microsoft word and are encountering issues, you should file them directly to microsoft word. Issues here should concern only VS Code.
As I mentioned what file are you trying to open in VS Code? Can you include a screen recording of your issue. If no details are provided, this issue will be closed after some time due to inactivity.

This is probably a Microsoft issue but how do I contact them on this?

On 01/18/2024 11:16 AM EST Aiday Marlen Kyzy @.***> wrote:
Hi @tdavid12 https://github.com/tdavid12, please give more details as to the issue. If you are trying to open word files in microsoft word and are encountering issues, you should file them directly to microsoft word. Issues here should concern only VS Code.
As I mentioned what file are you trying to open in VS Code? Can you include a screen recording of your issue. If no details are provided, this issue will be closed after some time due to inactivity.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/202685#issuecomment-1898793597, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE7BFLEPF54LYCIOZME3SILYPFDHPAVCNFSM6AAAAABB7IHD36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJYG44TGNJZG4.
You are receiving this because you were mentioned.Message ID: @.***>

Hey @aiday-mar, this issue might need further attention.
@tdavid12, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Hi @tdavid12 you are talking with Microsoft employees. Please include a screen recording of your issue and provide details about what you expect to see. If no sufficient information is added after some time, I will close this issue as I will not be able to reproduce it.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Very minor doc edits putting a hyphen in name

@dnwillia-work and @seanpearsonuk Just so you two don't think I've gone insane, I made this PR only to show the process to Brad Rozen and Steve Fuqua. They are ready to take over being your PyFluent writers! Tag them on PyFluent reviews as you see fit. Thanks.


Support HCL as configuration format

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
The current toml is not suited for a programmable configuration hence will like to suggest using  HCL  for  2.0
HCL is also fully JSON compatible. That is, JSON can be used as completely valid input to a system expecting HCL. This helps makes systems interoperable with other systems.
Example   296 bytes
[entryPoints]
  [entryPoints.http]
  address = ":80"
    [entryPoints.http.redirect]
    entryPoint = "https"
  [entryPoints.https]
  address = ":443"
    [entryPoints.https.tls]
      [[entryPoints.https.tls.certificates]]
      certFile = "tests/traefik.crt"
      keyFile = "tests/traefik.key"

This can  easily be replaced by  a  more readable format  HCL
Example HCL  249 bytes
entryPoints {
  http redirect {
      address = ":80"
      entryPoint = "https"
  }
  https {
       address = ":443"
       tsl certificates {
          certFile = "tests/traefik.crt"
          keyFile = "tests/traefik.key"
       }
  }
}

Example  2
[entryPoints]
  [entryPoints.http]
  address = ":80"
  [entryPoints.https]
  address = ":443"
    [entryPoints.https.tls]

[acme]
email =<EMAIL_ADDRESS>storage = "traefik/acme/account"
caServer = "http://<IP_ADDRESS>:4000/directory"
entryPoint = "https"

[acme.httpChallenge]
    entryPoint = "http"

[[acme.domains]]
  main = "local1.com"
  sans = ["test1.local1.com", "test2.local1.com"]
[[acme.domains]]
  main = "local2.com"
  sans = ["test1.local2.com", "test2x.local2.com"]
[[acme.domains]]
  main = "local3.com"
[[acme.domains]]
  main = "local4.com"

[consul]
  endpoint = "<IP_ADDRESS>:8500"
  watch = true
  prefix = "traefik"

Example 2 HCL
entryPoints {
  address = ":80"
  http address ":80"
  https {
       address = ":443"
       tsl =  true
  }
}

acme {
  email =<EMAIL_ADDRESS>  storage = "traefik/acme/account"
  caServer = "http://<IP_ADDRESS>:4000/directory"
  entryPoint = "https"
  httpChallenge entryPoint = "http"
  domains {
    main = "local1.com"
    sans = ["test1.local1.com", "test2.local1.com"]
  }
  domains {
    main = "local2.com"
    sans = ["test1.local2.com", "test2x.local2.com"]
  }
  domains {
    sans = "local3.com"
  }
  domains {
    sans = "local3.com"
  }
}

consul {
  endpoint = "<IP_ADDRESS>:8500"
  watch = true
  prefix = "traefik"
}

Futher simplication of acme
acme {
  email =<EMAIL_ADDRESS>  storage = "traefik/acme/account"
  caServer = "http://<IP_ADDRESS>:4000/directory"
  entryPoint = "https"
  httpChallenge entryPoint = "http"

  domains "local1.com" = ["test1.local1.com", "test2.local1.com"]
  domains "local2.com" =  ["test1.local2.com", "test2x.local2.com"]
  domains = "local3.com"
  domains = "local3.com"
}

HCL Supports

Single line comments start with # or //
Multi-line comments are wrapped in /* and */. Nested block comments are not allowed. A multi-line comment (also known as a block comment) terminates at the first */ found.
Values are assigned with the syntax key = value (whitespace doesn't matter). The value can be any primitive: a string, number, boolean, object, or list.
Strings are double-quoted and can contain any UTF-8 characters. Example: "Hello, World"
Multi-line strings start with <<EOF at the end of a line, and end with EOF on its own line (here documents). Any text may be used in place of EOF. Example:
Numbers are assumed to be base 10. If you prefix a number with 0x, it is treated as a hexadecimal. If it is prefixed with 0, it is treated as an octal. Numbers can be in scientific notation: "1e10".
Boolean values: true, false
Arrays can be made by wrapping it in []. Example: ["foo", "bar", 42]. Arrays can contain primitives, other arrays, and objects. As an alternative, lists of objects can be created with repeated blocks, using this structure:

Implementation

staert can  be updated to  support hcl
hcl can be backed directly into traefik all that is required is hcl.Decode
Majority of  the work will be done  in documentation

Conclusion
Based on my observation, a large configuration file will gradually become unreadable at first glance and HCL was designed to fit that gap
Kindly pardon any errors or assumption in the message above

@jbdoumenjou would you like to share how you intend to approach this?

There is a simple caddy vs traefik configuration
caddy
www.example.com:80, www.example.com:443 {
  tls self_signed
  log /logs access.log
  errors visible
  proxy /assets nginx:80
  proxy / web:3000

  redir 301 {
    if {>X-Forwarded-Proto} is http
      /  https://{host}{uri}
    }
  }

traefik

#debug = true

[backends]
  [backends.buffalo]
    [backends.buffalo.loadbalancer.stickiness]
      [backends.buffalo.healthcheck]
      path = "/healthcheck"
      interval = "11s"
    [backends.buffalo.servers.server1]
    url = "http://web:3000"

  [backends.backendNginx]
    [backends.backendNginx.loadbalancer.stickiness]
    [backends.backendNginx.servers.server1]
    url = "http://nginx:80"

[frontends]
  [frontends.frontend1]
  backend = "buffalo"
    [frontends.frontend1.routes.web]
    rule = "Host:www.example.com"
  entrypoints = ["http","https"]
  passHostHeader = true
  #passTLSCert = true
  priority = 10

  [frontends.nginx]
  backend = "backendNginx"
    [frontends.nginx.routes.web]
    rule = "Host:www.example.com;PathPrefix:/assets"
  passHostHeader = true
  #passTLSCert = true

any update on this ?

Hi,
Given that we now support YAML, we think that most of the arguments for supporting HCL as well are moot, so it's pretty unlikely we'll do it.
And for the record, we gave it a try while we were adding YAML support, and it turned out to be pretty gnarly, so we gave up on it because it wasn't worth the effort.


Fix TestSignerWithBrokenRand for go1.20

As of go1.20, the random parameter used in rsa.SignPKCS1v15 is legacy and ignored, and it can be nil[1] meaning that there's no broken random-ness test applicable for signing with RS256, RS384, or RS512.

https://cs.opensource.google/go/go/+/refs/tags/go1.20.2:src/crypto/rsa/pkcs1v15.go;l=263;bpv=0;bpt=1

Fixes https://github.com/go-jose/go-jose/issues/27

Is it safe to assume that consumers of this package are building on Go >= 1.20? The go directive in the go.mod file is still 1.12.

Keep in mind this is on the v3 branch, not v2 which we're still using over in Boulder and what's made available by the archived https://github.com/square/go-jose project.


Update CODEOWNERS with New Committers

@jbonofre - I think we are ready to do this? Are we waiting on anything else?

Thanks @RussellSpitzer!


"Valea Râmnicului", "Oreavul" and "Rubla" are listed in the drop-down menu

Description:
"Valea Râmnicului", "Oreavul" and "Rubla" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Valea Râmnicului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Râmnicului, BUZĂU (Valea Râmnicului)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Oreavul" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Oreavul, BUZĂU (Valea Râmnicului)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Rubla" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Rubla, BUZĂU (Valea Râmnicului)".


Bug report: couldn't transfer files to android at certain times

Describe the bug
when other device transfer files to Android app, the app show nothing only with white
To reproduce

transfer files to Android
Android device show nothing, see screenshot

Expected behavior
when transfer files from Windows to Android, Android phone show option correctly.
Screenshots

Desktop (please complete the following information)

OS: Windows 11 22H2
Browser: Firefox
Version: 1.6.1

Smartphone (please complete the following information)

Device: MEIZU 18s
OS: Flyme <IP_ADDRESS>A, Android 11
Broswer: Fennec, Ungoogled Chromium, via
Version: 1.6.2

Additional context
only show in these situation

when transfer files from Windows(other system unknown) to android
quick save is off

I’m having the same issue, my smartphone is XIAOMI K40 pro with MIUI 13.0.7, Android 12, The desktop OS is windows 11 22H2.

This seems like a bug in v1.6.1 using Chinese app language. See #37

yes, thank for reply, but app version of F-doird still in 1.6.1, please update


Using $(SolutionDir) causes problems with msbuild

My project attempts to import a .targets file from a directory higher than the solution directory.  The import looks like so: <Import Project="$(SolutionDir)..\foo\bar.targets" />
This results in a build-time error when using msbuild to restore nuget packages via command line:
C:\SolutionDirRepro\SolutionDirRepro.csproj(139,3): error MSB4019: The imported project "C:\SolutionDirRepro*Undefined*..\foo\test.targets" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
If I move the .targets file to the directory the solution resides in, this problem goes away.

Closed as duplicate of https://github.com/Microsoft/msbuild/issues/2743


Rewrite url conflict when trying to redirect www to non www

Using Apache 2.4.46, PHP 7.4 and MySQL 8.0.23 (via brew), if I add the rewrite rule below in my vhost to redirect www to non www, Bookstack in unable to load the file libs/tinymce/themes/modern/theme.min.js, it waits several seconds and ends up with a 0bits file.
RewriteEngine on
RewriteCond %{SERVER_NAME} !=test-bookstack.localdev [NC]
RewriteRule ^ http://test-bookstack.localdev%{REQUEST_URI} [END,QSA,R=permanent]

It then triggers the same error than in #2461.
Here is the network tab of firefox (same in chrome) :

It seems than the rewrite rules and conditions of the .htaccess in the public folder are conflicting with the vhost one.
How then should we config the vhost to redirect www to non www ?

Hi @nderambure,
Could you also post the full URL, or similar screenshot, for the tinymce.min.js request shown in your screenshot? I'd like to confirm if the URL paths are the same to understand why the two requests are behaving differently.

@ssddanbrown : I've updated the rewrite rule I used in my previous post, I did a bad copy/paste (but error is the same).
I've installed the last version 21.04.3 for this test.
Here is 2 screenshots, the first for the tinymce.min.js URL and the second, for infos, the files and directory rights on disk, which seem fine, but maybe something is wrong there.

Revisiting this, I'm going to close this off since it's been a while and it's an instance-specific issue.
In regards to this original statement:

It seems than the rewrite rules and conditions of the .htaccess in the public folder are conflicting with the vhost one.

In this case I'd advise disabling of .htaccess via AllowOverride Noneto prevent multiple configurations conflicting with each-other. From what I've seen it's generally good practice to avoid using .htaccess files.


CII Best Practices Projects page should include drop down menu for programming language

On this page, include a drop down menu for programming language - similar to "Badge Status." This would search the language field for recognized languages, i.e., those currently listed by projects.

This is a specific way to implement issue #1176 .
One problem with a drop-down menu is that there are many more-obscure languages - any drop-down would miss some.

I like the goal (as noted in #1176) - I'm just not sure this is a good way to implement it.

The top 20 most common languages found could be listed as a drop down, with the ability to type a specific language name as well. I've seen this in many of our corporate websites and it's quite effective. Further, as one types, the filtered set of possible matches show up in the drop down menu. Perhaps more advanced than necessary, but useful.

We now have a much better reason to add this.  One problem is that a lot of projects didn't provide this information.  I just added implementation language information to 895 projects that didn't have it before.  They're estimates from GitHub, which isn't always correct but is decent.  That means that now 1582 / 1857 projects have information in the implementation_languages field (per "select count(*) from projects where implementation_languages LIKE '_%';").


SCons: Bump min version to 3.1.0, test it on CI with one Linux job

The min SCons version had to be bumped as SCons 3.0 before 3.0.3 seems broken (see below), and there's little gain from supporting 3.0.3-3.0.5.
Also test against Python 3.6 which is also our minimum supported version.
This should help prevent regressions whenever we modernize the build scripts.
Also a bunch of cosmetic changes, got carried away :P
~This should fail initially because of #91986, so I will rebase after #92041 is merged.~
Confirmed, so #92041 is now included in this PR.
CC @Repiteo @AThousandShips

Confirmed that the build fails as expected without #92041: https://github.com/godotengine/godot/actions/runs/9124804393/job/25089728389?pr=92043
Building with flags: platform=linuxbsd target=editor tests=true verbose=yes warnings=extra werror=yes module_text_server_fb_enabled=yes dev_build=yes use_asan=yes use_ubsan=yes use_llvm=yes linker=lld
scons: Reading SConscript files ...
TypeError: 'SConsEnvironment' object is not a mapping:
  File "/home/runner/work/godot/godot/SConstruct", line 368:
    opts.Update(env, {**ARGUMENTS, **env})

I'll rebase to include that PR.

Looks great and very helpful, approving on principle but needs to pass compilation, something with the configuration fixes if the environment I suspect

Indeed, that's a weird error:
Checking for C header file mntent.h... TypeError : sequence item 0: expected str instance, bytes found

https://github.com/godotengine/godot/actions/runs/9124860447/job/25089907695?pr=92043
I can't reproduce it locally with a python:3.6 environment in podman, so not sure how to debug.
I'll throw a few tests on GitHub actions in my fork to see if newer Python or SCons solves it.

I think it'd be worth raising the minimum version, unless that excludes a large number of environments, like automated tools somewhere, but otherwise I'd say it isn't worth the hassle to plug holes and provide workarounds
I think large distros like LTS Ubuntu etc. are a good reference especially when automatic configuration etc. is involved like CI and containers

I bumped the minimum SCons version to 3.1.0 as described above.

Could be worth considering bumping to 3.1.2 given the note in the build instructions (they will also need updating):

If your distribution uses Python 2 by default, or you are using a version of SCons prior to 3.1.2, you will need to change the version of Python that SCons uses by changing the shebang (the first line) of the SCons script file to #! /usr/bin/python3. Use the command which scons to find the location of the SCons script file.

Bumped to 3.1.2 in the end.
Next stop, 4.0.0, sometime next year maybe?


issue #141 solved

@pradeeban
Can be tested here https://rahuljagwani.github.io/concore-editor/

Thanks @Rahuljagwani.
This PR disables the Contribute button when no study is present as expected, and opens when a study/graphml is open. That is great.
But it still keeps it disabled even when programs/folders in concore editor.
To test this,

create (outside concore editor) a folder of python classes and a graphml (just a random folder should work for test purposes).
load the folder into concore editor (using the "Upload Directory" option).

Expected behavior:

The Contribute option is still disabled (until a study/graphml is opened).

I think, once you have loaded a folder, it should be activated already. (because the contribute option can contribute the whole folder as a pull request. I admit we have not sufficiently thought about how that PR will work. But it is safe to just enable the Contribute button when a folder is loaded or a graphml is open in the canvas.
What do you think?

Thanks @Rahuljagwani.
This PR disables the Contribute button when no study is present as expected, and opens when a study/graphml is open. That is great.
But it still keeps it disabled even when programs/folders in concore editor.
To test this,

create (outside concore editor) a folder of python classes and a graphml (just a random folder should work for test purposes).
load the folder into concore editor (using the "Upload Directory" option).

Current behavior:

The Contribute option is still disabled (until a study/graphml is opened).

Expected behavior:

Enable the Contribute option since the studies and programs may need to be contributed now.

I think, once you have loaded a folder, it should be activated already. (because the contribute option can contribute the whole folder as a pull request. I admit we have not sufficiently thought about how that PR will work. But it is safe to just enable the Contribute button when a folder is loaded or a graphml is open in the canvas.
What do you think?

@pradeeban Thanks for testing. I tested it. It is working. Here is a video demonstrating it.
https://user-images.githubusercontent.com/83580404/224179971-16a4777e-9ba0-4ea0-93a1-a84651aea040.mp4
As you can see Contribute button became active after uploading a folder, I guess I am misunderstanding this or because of any other reason its not working??

I think, once you have loaded a folder, it should be activated already. (because the contribute option can contribute the whole
folder as a pull request. I admit we have not sufficiently thought about how that PR will work. But it is safe to just enable the
Contribute button when a folder is loaded or a graphml is open in the canvas.
What do you think?

@pradeeban
"because the contribute option can contribute the whole folder as a pull request" I completely agree with this statement.
Actually, I suggested an approach about the generation of PR in https://github.com/ControlCore-Project/concore/discussions/56 in which users have to give the path of folder which has to be contributed. You can even test the code I wrote. I provided the steps to test the implementation here: https://github.com/ControlCore-Project/concore/discussions/56#discussioncomment-5198701
According to this approach user first needs to create a folder consisting of .graphml file and python files which are needed to run the study. The python files can be created inside as well as outside concore-editor. After saving the whole folder in the local system, the path to this folder is necessary to contribute it. The path cannot be fetched when that folder is uploaded on Concore-Editor due to browser security reasons. Therefore we can provide two approaches to users:

Save the study in Concore folder from where flask server is running and then we can extract path of the saved folder through code in contribute.py ( It is present in https://github.com/Rahuljagwani/ContributeConcore )
Save the folder anywhere on system but give the correct path of the folder.

I request you to test this implementation when you are free and give me feedback for same. It is a rough implementation.
Thanks


[WIP] Fix for issue of Daniel with adaptive line search

Codecov Report

Merging #207 (a3e4385) into master (8c2f450) will increase coverage by 1.09%.
The diff coverage is 100.00%.

:exclamation: Current head a3e4385 differs from pull request most recent head ffcb427. Consider uploading reports for the commit ffcb427 to get more accurate results

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
+ Coverage   67.77%   68.86%   +1.09%
==========================================
  Files          14       13       -1
  Lines        1685     1683       -2
==========================================
+ Hits         1142     1159      +17
+ Misses        543      524      -19

Impacted Files
Coverage Δ

src/afw.jl
74.09% <100.00%> (+9.03%)
:arrow_up:

src/blended_cg.jl
51.32% <100.00%> (+0.79%)
:arrow_up:

src/fw_algorithms.jl
77.20% <100.00%> (+0.09%)
:arrow_up:

src/utils.jl
53.37% <100.00%> (ø)

src/oracles.jl
86.59% <0.00%> (-0.14%)
:arrow_down:

src/FrankWolfe.jl

src/types.jl
64.83% <0.00%> (+0.70%)
:arrow_up:

Continue to review full report at Codecov.

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


Disk not resized on creation

Hi all!
After the new disk scheme, we have to run apply twice to resize disk on creation.
Now I'm using the self-compiled code you suggest to use to solve the problem after Proxmox8 Upgrade.
Here's my tf provider:
terraform {
  required_version = ">= 0.13.0"

  required_providers {
    proxmox = {
        source = "registry.example.com/telmate/proxmox"
        version = ">=1.0.0"
        #source = "telmate/proxmox"
        #version = "3.0.1-rc3"
        #version = ">=1.0.0"
    }
  }
}

And my main.tf:
resource "proxmox_vm_qemu" "vm" {
    name = "vm1"
    desc = ""
    vmid = "235"
    target_node = "mynode2"

    agent = 1
    os_type = "cloud-init"
    clone = "ubuntu-server-2204-template-docker"
    cores = 4
    sockets = 1
    cpu = "host"
    memory = 65536
    balloon = 32768
    hotplug= "disk,network,usb"
    numa = false
    tags = "testing"
    qemu_os="other"

    network {
      bridge = "vmbr1"
      model = "virtio"
    }

    #disk {
    #  storage = "local"
    #  type = "virtio"
    #  size = "400G"
    #}

    disks {
        virtio {
            virtio0 {
                disk {
                    size = 400
                    storage = "local"
                }
            }
        }
    }

    #cloudinit_cdrom_storage = "local"

    bootdisk = "virtio0"

    ipconfig0 = "ip=<IP_ADDRESS>/19,gw=<IP_ADDRESS>"
    nameserver = "<IP_ADDRESS>"
    ciuser = var.tsg_user
    cipassword = var.tsg_password
    sshkeys = <<EOF
    ${var.tsg_key}
    EOF

------rest of stuff code(scripts, executions, etc).

}
So in summary, when I created the VM, on first apply, the disk is not being resized, it is grabbing template disk size. If I apply second time(with vm created) I'm getting the disk resized.
In advance, thanks a lot at all team behind telmate, keeping this plugin updated :)

@SySamu-Hallon,
Did you try to change these config values?
#
qemu_os="other"
# change to
qemu_os="l26"

#
bootdisk = "virtio0"
# comment out this line
#bootdisk = "virtio0"
# as per doc:
# Enable booting from specified disk. You shouldn't need to change it under most circumstances.

As well, try to change virtio to scsi disk, if possible

Let me try it this morning and come back. My template has been created with virtio instead scsi, it will be a problem change on creation? i can fit the template to scsi anyway, but, just to know.
Now i'm using Terraform 1.8.5:

Let me try it this morning and come back. My template has been created with virtio instead scsi, it will be a problem change on creation? i can fit the template to scsi anyway, but, just to know.

Better to change in source template and afterwards use it in VM creation.
As I observed, scsi behavior ir more stable in PVE.

Now i'm using Terraform 1.8.5:

OK, it is good.
You can change it in config to prevent using outdated versions, like:
terraform {
  required_version = ">= 1.8.5"
...

@maksimsamt  Ok, just last details on my .tf. Another question, my template has been created with CloudInit drive, it should be detached too and added on creation on the new vm? I understood that's the correct way now on another thread I posted this question: https://github.com/Telmate/terraform-provider-proxmox/pull/892#issuecomment-2191486865

Another question, my template has been created with CloudInit drive, it should be detached too and added on creation on the new vm?

Cloud-init drive could stay in your template, during VM creation terraform provider will re-create cloud-init drive.
So, I recommend to configure cloud-init in your terraform config, for example in case of scsi:
    disks {
        scsi {
            scsi0 {
                disk {
                    size    = "15G"
                    storage = "local"
                }
            }
            scsi1 {
                cloudinit {
                    storage = "local"
                }
            }
        }
    }

This correctly works if cloud-init drive in your template is not scsi1.
FYI, related issue: https://github.com/Telmate/terraform-provider-proxmox/issues/1024
BTW, disk size not correct in your config, it should be as string, not integer:
#
                disk {
                    size = 400
                    storage = "local"
                }
# =>
                disk {
                    size = "400G"
                    storage = "local"
                }

Check out the documentation: https://github.com/Telmate/terraform-provider-proxmox/blob/master/docs/resources/vm_qemu.md

following your steps i'm getting stucked on terraform plan. Updated code here:
resource "proxmox_vm_qemu" "vm" {
    name = "${var.hostname}"
    desc = "${var.description}"
    #vmid = 0
    target_node = "${var.proxmox_node}"

    agent = 1
    clone = "${var.template}"
    cores = var.cpu_cores
    sockets = var.cpu_sockets
    vcpus = var.cpu_vcpu
    cpu = "${var.cpu_type}"
    memory = var.ram
    balloon = var.balloon
    hotplug= "disk,network,usb"
    numa = var.numa
    onboot = var.onboot
    tags = "${var.vm_tags}"

    network {
      bridge = "vmbr1"
      model = "virtio"
    }

    disks {
        scsi {
            scsi0 {
                disk {
                    size = "${var.disk_size}G"
                    storage = "${var.datastore}"
                }
            }
            scsi1 {
                cloudinit {
                    storage = "${var.datastore}"
                }
            }
        }
    }

    bootdisk = "scsi0"
    boot = "order=scsi0"
    os_type = "${var.os_type}"
    ipconfig0 = "ip=${var.ip}/19,gw=${var.gateway}"

    ---rest of main config---

    }

Note that vars are declared and setted, as datastore. To use string on disk size i've just added "G" between quotes. That trick allow us to only change a number instead a string. I also  detach and attach template disk as scsi instead virtio as you suggest :)
The error:
terraform plan
╷
│ Error: Unsupported block type
│
│   on main.tf line 34, in resource "proxmox_vm_qemu" "vm":
│   34:                 cloudinit {
│
│ Blocks of type "cloudinit" are not expected here.

My cloud init drive on template is on ide0. Attaching pic of template config:

@maksimsamt thanks a lot for your support, changing provider to latest version from telmate solve all:
terraform {
  required_version = ">= 1.8"

  required_providers {
    proxmox = {
        source = "telmate/proxmox"
        version = "3.0.1-rc3"
    }
  }
}

Closing the issue.
Thanks a lot again, you save me hours of work 😄