[Spec]  Popup add margin property

[The feature]
Started in this question/discussion https://github.com/xamarin/XamarinCommunityToolkit/discussions/1188, it is intended to give the dev the ability to set margin for popup and thus giving more flexibility over popup position adjustment (in addition to VerticalOptions and VerticalOptions properties).
API
[ BasePopup.shared.cs]
Properties

API
Type
Description

Margin
Thickness
Gets or sets the margins of the popup.

Scenarios
Let say you have a popup and want to position it in the bottom center with a slight vertical margin (space between bottom edge of the screen and the bottom edge of the popup), you will  put:
<xct:Popup ...
             VerticalOptions="End"
             HorizontalOptions="Center"
             Margin="0,0,0,40"
...

Platform Compatibility

Target Frameworks:

iOS:
Android:
UWP:

Backward Compatibility
Difficulty : [low/medium/high]

Thanks for writing up this specification, I think this is the direction that we want to go. I think we should make the following adjustments to this specification.
Margin can be set only if the following conditions are met
VerticalOptions = FillAndExpand
HorizontalOptions = FillAndExpand

With those properties set we can check in the renderer and then force the popup to take up all the bounds on the screen. Then you could use the Margin property to set the space between the edge of the popup and the edge of the screen.

@ahoefling I have updated the spec accordingly, so setting VerticalOptions  and HorizontalOptions  to FillAndExpand does not necessarily mean that the popup content will take the whole space (after the enhacement), it also depends on it content more specifically the root view, right?

hmm 🤔🤔
We should consider the Size as well because if someone sets size, should we disregard VerticalOptions? and HorizontalOptions? Or if the VerticalOptions and HorizontalOptions are set to FillAndExpand should we disregard the size and make it full screen to start applying the margin.
This is going to merit more discussion to get the rules right in the specification

That is a general question when it comes to layouting in Xamarin.Forms!
Say if I want to achieve a bottom drawer: VerticalOptions = End, Horizontal = Fill, HeightRequest = x
Won't work, because I have to specify Size.


LineCap doesn't work on Android device

Reported in a forum.
use Version: 1.7.0, macOS
I used below markup.
<VectorLayer>
	<Path ux:Name="Target" Data="M5,5 C9.88320312,11 17.3832031,14 27.5,14 C37.6167969,14 45.1167969,11 50,5" Width="30">
		<Stroke Width="7" Color="#fff" LineCap="Round" LineJoin="Round"/>
	</Path>
</VectorLayer>

In studio, it works properly. But, when in device(I checked only android), it doesn't work.
I checked in iOS. It works! It is only android problem.

It looks like the code sets a Butt cap always, which is an error. This shouldn't be hard to fix.


Swapping between an ERC20 token and ETH leaves the result in WETH

The official uniswap v3 SDK unwraps the result using the unwrapWETH9 contract method.
The current behaviour is unexpected, since the output token is specified as ETH (address 0x0...0).
What's your position on this? I can probably make a patch that uses the multicall to swap and unwrap as I already have the code for it in my project. I think that for very least the uniswap-python should issue a warning log message at the moment.

I've run into the same issue. If WETh is specified as input token, ETH is taken from wallet instead. Similarly, if ETH is specified as the output token, WETH is sent to the recipient address.

If this is still open, I can fix this!

There's a pending PR https://github.com/uniswap-python/uniswap-python/pull/159


Clean up old git repos and images on destroy

This fixes two problems in Deis master:

deis destroy leaves a git repo dropping in builder at /home/git/appname.git
deis scale cmd=0 deletes the /deis/services/appname dir after 20 seconds, but that causes the (fixed) check-repos script to purge its git repo prematurely

My assumption is that the templating for check-repos worked in earlier versions of confd that were more specific to etcd behavior. This PR switches to etcd watch in a loop rather than rely on confd templating to launch the script. It also reverts #3348 so that /deis/services/appname dirs don't have a TTL, since deis destroy deletes them reliably in my testing.
Closes #3249, refs #3348.

nice cleanup to run on an actual change in /deis/services by utilizing etcd watch rather than a confd reload. Code LGTM!

by utilizing etcd watch rather than a confd reload

It adds a bit of complexity, but what it gains apparently is notification when a directory itself under /deis/services/ is added or removed. Using just confd polling or watching, it only cares about keys.
I also tried etcd exec-watch which would seem to be perfect for this case, but it was unreliable. It would exec the command once or twice and then just stall, so I went with this approach.

Remember that we did see performance issues when we switched everything over to using watch, but I think one judicious use of it is fine. And this is an appropriate place to use it.
Code LGTM.


Error in building Caffe2 Docker image with Ubuntu 16.04 CUDA 8 and CUDNN6

I got this error and I am not sure how to fix it so far. I have already set -- Added CUDA NVCC flags for: sm_20 sm_21 sm_30 sm_35 sm_50 sm_52 sm_60 sm_61 Related question has been fixed by adding sm_35.
[ 63%] Building NVCC (Device) object caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o
[ 64%] Building NVCC (Device) object caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_reverse_packed_segs_op.cu.o
/opt/caffe2/caffe2/operators/resize_op.cu(63): error: identifier "__ldg" is undefined

1 error detected in the compilation of "/tmp/tmpxft_000028c7_00000000-20_resize_op.compute_20.cpp1.ii".
CMake Error at Caffe2_GPU_generated_resize_op.cu.o.Release.cmake:278 (message):
  Error generating file
  /opt/caffe2/build/caffe2/CMakeFiles/Caffe2_GPU.dir/operators/./Caffe2_GPU_generated_resize_op.cu.o

caffe2/CMakeFiles/Caffe2_GPU.dir/build.make:371: recipe for target 'caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o' failed
make[3]: *** [caffe2/CMakeFiles/Caffe2_GPU.dir/operators/Caffe2_GPU_generated_resize_op.cu.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[3]: Leaving directory '/opt/caffe2/build'
CMakeFiles/Makefile2:2108: recipe for target 'caffe2/CMakeFiles/Caffe2_GPU.dir/all' failed
make[2]: Leaving directory '/opt/caffe2/build'
make[2]: *** [caffe2/CMakeFiles/Caffe2_GPU.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make[1]: Leaving directory '/opt/caffe2/build'
make[1]: *** [all] Error 2
Makefile:4: recipe for target 'all' failed
make: *** [all] Error 2

I proposed a working Dockerfile here: https://github.com/caffe2/caffe2/pull/911

Thanks @lukeyeager, great to see the working dockerfile. I guess the solution to my issue is to fix cmake/Cuda.cmake. On the first two line
set(Caffe2_known_gpu_archs "35 50 52 60 61")
set(Caffe2_known_gpu_archs7 "35 50 52")

This is similar to yours
-DCUDA_ARCH_NAME=Manual -DCUDA_ARCH_BIN="35 52 60 61" -DCUDA_ARCH_PTX="61"

I have tried your solution but i still met this problem.


TV controls on game admin dashboard

I want to be able to switch HDMI inputs on the gaming TV, so I can perform maintenance on the gaming-go more effectively; requiring me to switch between the gaming pc and the gaming go display.

Remote Desktop from remote control may also make this possible without the need to control the TV's HDMI inputs.


Build initialize fails

For the first time I try build initialize in cmd as Administrator and it fails for these packages
System.Text.RegularExpressions
System.Threading
System.Threading.Tasks
System.Collections.Immutable
System.Reflection.Metadata

So I try to manually install it with Manage Nuget packages. Now it displays why it can not locate these packages: "You are trying to install it to a project that targets .NETFramework, Version = 4.5.1, but the package does not contain any assembly references or content files that are compatible with that framework"

May be related to some package source changes happening at the moment and just be a temporary issue.

 Most of us work around it by adding a line similar to the following inside packageSources.
<add key="nuget.org.DoNotDisable" value="https://www.nuget.org/api/v2/" />

@bricelam thanks, that worked. So if anyone has the same problem, it can look it up in this issue.


[HeXXen 1733] Releasing v1.0.1

Changes / Comments
Changelog at https://github.com/m-gora/Roll20-Charactersheet-HeXXen1733/wiki/Changelog
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
Changed roll queries in hidden inputs are not stored in the sheet, so no migration was necessary
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Write updated unit tests for AES functions

I had previously went from doing everything in-place in our input vectors for AES, thus causing me to make new unit tests with that in mind. Then I found that it was slower than copying into a state array. So now we don't have any unit tests for the actual AES functions anymore i.e. subByte, mixColumns, addRoundKey, etc. So it would be nice to add new unit tests for these functions in tests/tests.cpp.
I should mention that it would probably be best to to #1 before this issue as you would need to change them again once that issue is resolved/ tested.

This issue was resolved in  Making enhancements to AES implementation #6 .


How Do I Access Configuration Values from the airflow.cfg File?

I would like to use include the base_url in emails that I send via from flow. The base_url is defined under the webserver section of the config in airflow.cfg
What's the easiest way to these variables in any operator? Currently, I send email using my own PythonOperator (so that I can send both text and html multipart email). Hence, I access the config using the code below.
    parser = SafeConfigParser()
    parser.read('airflow.cfg')
    base_url = parser.get('webserver', 'base_url')

This works, but I was wondering if there is a more elegant way? Also, this only works for PythonOperators, but what if I need to access a configuration parameter in any operator in a Jinja template?

It's easy for me to throw it in the context. Here's the PR:
https://github.com/airbnb/airflow/pull/169

Hi! I see the change in documentation, but didn't see a material change to the code. It's late, so I might have missed something.
-s

I added a line comment to point out which line makes the difference

Great... replaced 4 lines of code (including an import) with one.
base_url = kwargs['conf'].get('webserver', 'base_url')
Thx


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/39512

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/54516


fix: Remove broadcast stream from notification_service_impl.subscribe…

- What I did

Fix the issue of single notification being sent for multiple times.
-Root cause
At the moment, NotificationService.subscribe returns a Stream.broadcast which supports multiple listeners. From the app,
NotificationService.subscribe method is being invoked multiple time's with same regex which leads to the same notification being fetched in the listen callback.
-Fix
Replace the Stream.broadcase with simple Streams which supports only single listener. If subscribed for the second time,
throws a Bad State. stream was already listened to(This will NOT cause the app to crash). Ideally, for a given regex,
having a single listener would suffice and i do not see a scenario which would require multiple listeners for the same regex.

- How I did it

Replace the Stream.broadcast with simple Streams.
Replace the Map with HaspMap and override the equals and hashcode methods to compare the NotificationConfig object on regexes.

- How to verify it

The Notification should be sent to app only once.
If a NotificationService.subscribe is invoked for more than once, then a Bad State erorr would be thrown.

- Description for the changelog

The e2e tests have failed because KeyStream tests have multiple subscriptions causing Bad State: Stream is already listened to. Basing on the dicsussion in the internal stand-up will refactor the changes.

@kalluriramkumar checks are failing ...

@kalluriramkumar checks are failing ...

Yes Gary, At the moment, the KeyStream tests are have multiple listeners, hence the tests the are failing. Wanted to check if the removing the broadcast (Changing Stream.broadcast to simple stream) is fine. If yes, will refactor the tests.

@XavierChanth : I am not sure on which apps are using KeyStreams. Request your feedback/thoughts on changing the Stream.broadcast to simple streams.

For root cause:
Is the subscription being invoked multiple times in our client code, or is it an app developer mistake causing the problem?
If client code, then I think we should seek to fix the problem of multiple invocations, here's why:
Keep in mind the following from dart docs:

Single-subscription streams are generally used for streaming chunks of larger contiguous data, like file I/O.
A broadcast stream allows any number of listeners, and it fires its events when they are ready, whether there are listeners or not.

Notifications are event driven, and ideally should be using broadcast streams.
From a Flutter development perspective, it is useful to be able to subscribe from multiple places, to reduce the need for unnecessary re-renders of certain widgets.
Despite all of this, if we still decide to opt into single-subscription streams:
KeyStream will need to be refactored so that it does not need to subscribe to the notification service. This will most likely require coupling notification service and key stream, which is less ideal.

For root cause: Is the subscription being invoked multiple times in our client code, or is it an app developer mistake causing the problem? If client code, then I think we should seek to fix the problem of multiple invocations, here's why:
Keep in mind the following from dart docs:

Single-subscription streams are generally used for streaming chunks of larger contiguous data, like file I/O.
A broadcast stream allows any number of listeners, and it fires its events when they are ready, whether there are listeners or not.

Notifications are event driven, and ideally should be using broadcast streams. From a Flutter development perspective, it is useful to be able to subscribe from multiple places, to reduce the need for unnecessary re-renders of certain widgets.
Despite all of this, if we still decide to opt into single-subscription streams: KeyStream will need to be refactored so that it does not need to subscribe to the notification service. This will most likely require coupling notification service and key stream, which is less ideal.

@XavierChanth : It is from the app's code the multiple requests coming in and communicated to the app developer. Also If having the StreamController as broadcast would easy the life of app developers, then i would add back the Stream.broadcast.


No text displaying in a-gui-input's

I've spent a whole day trying to get text to display in an a-gui-input but it's just constantly blank no matter what I do, can you see where I'm going wrong?

` <a-gui-flex-container flex-direction="column" justify-content="center" align-items="normal" component-padding="0.1" opacity="0.7" width="3.5" height="4.5" position="2 2.5 -6" rotation="0 0 0" gui-item="" geometry="" material="" gui-flex-container="">
    	<a-gui-input id="test_input" width="4" height="0.7" onclick="testInputAction" value="Testing" font-size="160px" font-color="#212121" border-color="#212121" border-hover-color="#424242" background-color="#FAFAFA" hover-color="#F5F5F5" active-color="#FFEB3B" gui-interactable="" gui-item="" geometry="" material="" role="input" gui-input="" position=""></a-gui-input>
    </a-gui-flex-container>`

I got blank text too. the console shows something like this though. could it be the source of the problem ?

Either of you ever figure this out?

It isn't documented in the readme as far as I can tell, but looking into the example code, it's pretty straightforward:
      <a-gui-flex-container
      id="startup-menu555"
      flex-direction="column"
      justify-content="center"
      align-items="normal"
      component-padding="0.1"
      opacity="0.7" width="4" height="5"
      position="-6 1.5 -8"
      rotation="0 -90 0"
      >
        <a-gui-input
         id="test_input555"
         width="3" height="0.7"
         onclick="alert('hi')"
         value="Starting value"
         font-size="160px"
         font-color="red"
         border-color="#212121"
         border-hover-color="#424242"
         background-color="#FAFAFA"
         hover-color="#F5F5F5"
         margin=".15 0 0 0"
         active-color="#FFEB3B"
        >
        </a-gui-input>
      </a-gui-flex-container>

var textField = document.querySelector("a-gui-input");
textField.components['gui-input'].appendText('adding text');
setTimeout(() => {alert('will delete text now'); textField.components['gui-input'].delete();}, 10000)

Note that "delete" deletes one character, not whole input.

It isn't documented in the readme as far as I can tell, but looking into the example code, it's pretty straightforward:
      <a-gui-flex-container
      id="startup-menu555"
      flex-direction="column"
      justify-content="center"
      align-items="normal"
      component-padding="0.1"
      opacity="0.7" width="4" height="5"
      position="-6 1.5 -8"
      rotation="0 -90 0"
      >
        <a-gui-input
         id="test_input555"
         width="3" height="0.7"
         onclick="alert('hi')"
         value="Starting value"
         font-size="160px"
         font-color="red"
         border-color="#212121"
         border-hover-color="#424242"
         background-color="#FAFAFA"
         hover-color="#F5F5F5"
         margin=".15 0 0 0"
         active-color="#FFEB3B"
        >
        </a-gui-input>
      </a-gui-flex-container>

var textField = document.querySelector("a-gui-input");
textField.components['gui-input'].appendText('adding text');
setTimeout(() => {alert('will delete text now'); textField.components['gui-input'].delete();}, 10000)

Note that "delete" deletes one character, not whole input.

a little more digging in the source for gui-input shows that
textField.setAttribute('gui-input', 'text', "helloooo");

works for updating the text more than one char at a time
however, initial text values seem broken to me for now--if I figure out something I'll add it here, and hopefully eventually update the docs.

a little more digging in the source for gui-input shows that
textField.setAttribute('gui-input', 'text', "helloooo");

works for updating the text more than one char at a time
however, initial text values seem broken to me for now--if I figure out something I'll add it here, and hopefully eventually update the docs.

(To be clear, the 'starting value' feature does seem broken, though.)

(To be clear, the 'starting value' feature does seem broken, though.)

Yes you're right,  the issue is the starting value. but i haven't tried to look more into it

Yes you're right,  the issue is the starting value. but i haven't tried to look more into it

i think this is due to a wrong variable name in input.js

whereas the schema for the component is not inputText but text

i think this is due to a wrong variable name in input.js

whereas the schema for the component is not inputText but text

There are many small things like that. I'm grateful to this library, but it needs a second pass of polish. It feels like someone created a full library beta version and threw a ton of components at the wall, but didn't check everything everywhere. Shows great promise, just needs minor tweaks, in the meantime takes workarounds like the one I mentioned. (As in, that's right, initial text value doesn't work, but you can populate initial text right after element creation with JS instead for now.)
I know there's a big update coming with some important, very nice changes. Once that lands, I'm hoping to actually take on a bunch of those small updates and documentation fixing. I've been developing my menu system in my app with this.

There are many small things like that. I'm grateful to this library, but it needs a second pass of polish. It feels like someone created a full library beta version and threw a ton of components at the wall, but didn't check everything everywhere. Shows great promise, just needs minor tweaks, in the meantime takes workarounds like the one I mentioned. (As in, that's right, initial text value doesn't work, but you can populate initial text right after element creation with JS instead for now.)
I know there's a big update coming with some important, very nice changes. Once that lands, I'm hoping to actually take on a bunch of those small updates and documentation fixing. I've been developing my menu system in my app with this.

https://github.com/rdub80/aframe-gui/issues/62 anyone?


{"$id":"1","innerException":null,"message":"Request was blocked due to exceeding usage of resource 'Count' in namespace 'AnonymousId'. For more information on why your request was blocked, see the topic "Rate limits" on the Microsoft Web site (https://go.microsoft.com/fwlink/?LinkId=823950).","typeName":"Microsoft.TeamFoundation.Framework.Server.RequestBlockedException, Microsoft.TeamFoundation.Framework.Server","typeKey":"RequestBlockedException","errorCode":0,"eventId":3000}

i would like
someone to make me understand this message above

Where do you see this error?


Include Obligation Macro

This could probably use some work, but since I'm using it, let's include.

dupe of #6


Can't Install / Broken on Wabbajack

What happened?
GS_SLAL SE 27 11.7z   AND   GS_SLAL SE Creatures 20 02.7z   ....are both missing versions on SexLab.
There are newer/older versions but Wabbajack won't accept.
I even renamed the 7zip files and manually placed them....still won't work. Please fix or take the pack off of Wabbajack.
Modlist version
Whatever is currently up on Wabbajack 12/26/2022

you can find the files on gunslicers patreon here https://www.patreon.com/Gunslicer/posts

After manually downloading both files from patreon and placing them in the mods download folder Wabbajack is still prompting me to download 'GS_SLAL SE Creatures 20 02.7z' from SexLab.

After manually downloading both files from patreon and placing them in the mods download folder Wabbajack is still prompting me to download 'GS_SLAL SE Creatures 20 02.7z' from SexLab.

I'm having the same issue, managed to find the download but wabba doesn't it seem to recognise GS_SLAL SE Creatures 20 02.7z from Patreon

Hello everyone, I'm having the same issue waba is not recognising the files.

i also face the same issuse

The tl;dr is that the patreon version is not the same as the one that was previously on LoversLab.
Modlist author seems to be aware of this and we basically have to wait for him to update the list unless you can somehow find the original version that was on LL.

Contents of GS_SLAL SE Creatures 20 02.7z differ between LL's removed version and Patreon's version, for some reason.
If you know how to edit a modlist, you could still get GS_SLAL SE Creatures 20 02.7z and GS_SLAL SE 27 11.7z into WJ's downloads, with the following replacements:

3KSmOKYrHH8= => HX7/e0OgNtU=
"Name":"GS_SLAL SE Creatures 20 02.7z","Size":13484008 => "Name":"GS_SLAL SE Creatures 20 02.7z","Size":13484916
FNIS_GS_SLAL_Creatures_Behavior.hkx"],"Hash":"9WrKZJ83R3M=","Size":176800 => FNIS_GS_SLAL_Creatures_Behavior.hkx"],"Hash":"U8t6jMiTNZs=","Size":176168
FNIS_GS_SLAL_Creatures_List.txt"],"Hash":"rxZ8JK3tndE=","Size":10858 => FNIS_GS_SLAL_Creatures_List.txt"],"Hash":"9oU2xNPbswM=","Size":10857

This is obviously completely unsupported, and I would bet that the game crashes when loading the relevant Creatures animation (considering FNIS isn't up to date). But installation and new characters work fine after that.
If you don't know or have to ask, wait for a proper update to the modlist.


Refactor VariableSpec and StateItemSpec

For indirect reasons, it would now be preferable if subclasses of StateItemSpec have trivial constructors with an initialize() method doing more of the heavy lifting.
This could be done in two ways:  (Consider FieldSpec is the poster child for StateItemSpec subclasses)

field_spec = FieldSpec() ... call field_spec%initialize(variable_spec, _RC)
field_spec = FieldSpec(variable_spec)  ... call field_spec(_RC)

Originally I was thinking (1), but (2) has several advantages.  In particular, it means that  VariableSpec does not appear in the interfaces of StateItemSpec, and this may allow make_ItemSpec() to continue to live right where it is and require less total refactoring.
For now the call to initialize() can remain inside of make_ItemSpec().   Moving that to a new home is something I can do when I'm ready to exercise this functionality.
I will need this soon, so please be honest about level of effort you can apply.

There may be a dependency chain:

mapl3g_VariableSpec depends on mapl3g_StateItemSpec and mapl3g_FieldSpec which in turn depends on mapl3g_StateIemSpec, currently.
We want VariableSpec to pass itself into FieldSpec in the constructor called from make_itemSpec
Therefore, the constructor for FieldSpec needs a VariableSpec argument.
So, mapl3g_FieldSpec needs to depend on mapl3g_VariableSpec.

Is that correct? I can work around it, but I want to confirm:

We don't want mapl3g_StateItemSpec to depend on mapl3g_VariableSpec.
We want to defer initialization of FieldSpec objects after they are constructed.

I think this:

field_spec = FieldSpec(variable_spec) ... call field_spec(_RC)

should be this:
field_spec = FieldSpec(variable_spec) ... call field_spec%initialize(_RC)

I think this:

field_spec = FieldSpec(variable_spec) ... call field_spec(_RC)

should be this:
field_spec = FieldSpec(variable_spec) ... call field_spec%initialize(_RC)

Yes.  Sorry.


Update Counter to Be In-Sync with Player's and Dealer's hand.

Currently working on making the hit and stand buttons functional; the task with the counter is tied into this on the player side...
Task Include:

[x] Creating the score label so that it is able to update dynamically
[x] Allowing for score to increment when a card is added

Blockers:

[ ] Creating the dealer scene #9, and being able to have one deck split between two hands so there are no duplicate cards.
[ ] Simulating the actions of the dealer and creating their own counter that updates based on the dealer's actions.

[ ]Have to Continue Implementing Game Logic so, that dealer's makes decisions based off of predefined rules as to whether to hit or stand...


creating an app using npm init @shopify/app@latest results in not being able log in to Shopify Partners site for app auth

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App
Expected behavior
I expect that hitting npm run dev should open the shopify partner site and authenticate my app
Actual behavior
I get an error that says "Auto-open timed out"
Further, the link the cli then provides seems to have a typo in it. The word "scope" is misspelled and this is what is probably causing the issue.
eg ...accounts.shopify.com/oauth/authorize?client_id=fbdb2649-e327-4907-8f67-908d24cfd7e3&scoppe=openid+
Stack trace
No response
Reproduction steps

npm init @shopify/app@latest
npm run dev
press any key

Operating System
Windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3
Shell
psh
Node version (run node -v if you're not sure)
16.4.2
What language and version are you using in your application?
javascript (react)

My mistake

Hi. I have a similar issue, and I attached a screen recording of how to replicate it. Can I get some help? Thanks!
Whenever I run npm run dev and click any key it shows this.
npm run dev

dev
shopify app dev

To run this command, log in to Shopify.
👉 Press any key to open the login page on your browser
╭─ error ───────────────────────────────────────────────────────╮
│                                                               │
│  The requested scope is invalid, unknown, or malformed.       │
│                                                               │
╰───────────────────────────────────────────────────────────────╯

And redirect me to a page URL: http://<IP_ADDRESS>:3456/?error=invalid_scope&error_description=The requested scope is invalid%2C unknown%2C or malformed.&state=a0314fb1a276fd7dbffb5ec6e35d8ff2137c1c666f68ea953143f63a37eb
that says:
Something went wrong!
There was an issue while trying to authenticate.
Return to your terminal and try running the previous command again.

Same problem here today, when using the CLI for themes.
Just updated to CLI Version 3.61.0, tried running "theme pull", and the login page instantly throws an error:
"The requested scope is invalid, unknown, or malformed."
Clearing cookies, restarting pc, etc. all didn't work.

Fixed it by rolling back to 3.5.0
npm uninstall -g @shopify/cli
npm install -g<EMAIL_ADDRESS>

Same problem here also tried the shopify cli version 3.5.0 as suggested by @AppDevShopify but got an error
"command theme:dev not found" strange!

It seems it was an issue on Shopify's end. I tried running the npm run dev again, and it works now.

It seems it was an issue on Shopify's end. I tried running the npm run dev again, and it works now.


Log the sizes of the generated structures

Sometimes it's nice to have access to the sizes of the generated trees without having to parse the output.
This patch adds one line of log (with verbosity level "info") with theses sizes.
What do you think about adding this to bb?

Yes I agree, it would be cleaner to put it in the json.
I'll amend the PR

Thanks, @Kerl13! The patch seems really fine :)


Remove warning relevant for v 0.7 but is now fixed

Prerequisites

[X] I'm using the latest version of near-sdk-js.
[ ] I have tried to start with a fresh project and reproduce the defect with minimal code changes.
[ ] I have read the console error messages carefully (if applicable).

Description
Remove warning from js-sdk from the documentation thats now updated with the release of v 1.0+.

Reproducible demo
No response
Steps to reproduce

warning can be seeing when one tries to run sdk js library to build a contract on mainnet

Expected behavior
No warning shown in the app, hence this should be removed from the documentation as well
Actual behavior
Remove warning from the documentation
Your environment

NEAR JavaScript SDK version used - v.1.0.0

Self-service

[X] I'd be willing to fix this bug myself.

Fixed


qoi: init at unstable-2023-08-10

Description of changes
Initial import of the QOI image format single header implementation.
The repository also has a simple qoiconv image  conversion tool, so separated the outputs and provide a qoi.dev.
For reviewer: I am new to the separation of development outputs and 'regular' outputs, so please double-check that I did it correctly. qoi is a Header-only 'library', so the dev part just copies the header.
Context: I am using qoi in the timg tool which currently uses a vendored copy of qoi. With this, I'll be able to update the derivation to use qoi provided by nixpkgs.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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.

Requesting review from @drupol as you reviewed merged my recent timg changes and this is related.


Dont generate a 404 for unknown paths

The WS upgrade handler can't coexist with other WS servers on the same httpServer as it  sets a 404 response header on any path it isn't intrested in.
If it just silently ignores and falls through then other handlers can process the upgrade if they wish to.
See also PR #5 which was somewhat related (same problem, different 404)

Ignore this PR for a bit. It "fixes" my issue, but it introduces a new one when none of the handlers want to upgrade the socket and it times out rather than immediately failing. Will do some more investigation.

@rjp44 is this ready for review at this point?


I can't see anything??

Thanks for the previous clarity. There is an error occurring, and i'm really confused.

Never mind, i have figured it out!


Configure DNS for mailgun

We need to add the following to our DNS so we can send email via mailgun (connected to #130)
I don't know who our DNS provider is -- can I do this myself or can you do it @annaps?
TXT	openprescribing.net	v=spf1 include:mailgun.org ~all
TXT	pic._domainkey.openprescribing.net  k=rsa; p=MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQClHgXdmjdhuprKRY8sDRgGs8d9D1NAfmNDxq9mB+bqz7VmIMhuGpQf9dkzFERKnV8/byBX849zAMQhHppJCV9UlEEOW2qyRQQbvbuyTPHSah4ZTzKsyUk9JjbbCXoe7d3jWXXihdEY0c880EERHJHhtK6mDkPORvhoTuc2JmcY2wIDAQAB
CNAME	email.openprescribing.net mailgun.org

Done in cloudflare


Use blocking IO for reading into the C&C file types

When I started this all out, the idea of non-blocking IO (the java.nio package) for reading file data seemed like a good idea, but the way I've coded things is to always expect X bytes to be read from the input which the blocking IO classes (java.io) would more likely guarantee.
Experiment with java.io for one of the files and see how it fits.  Namely, use the DataInput interfaces since they let me read primitives from the input stream easily.

Works pretty well, will apply to other file types as I add them.


Refactor filters for orders page, created main filter components

Refactor filters for orders page, created main filter components
Checklist

[x] 🔽 My branch is up-to-date with "development" branch
[] ✅ All tests passed locally
[x] ✨ My changes working with up-to-date front-end and back-end part locally, like charm
[x] 🔗 Link pull request to issue


Update padding for list-item actions when extend-separators is turned on.

This PR fixes the layout for list-item actions for the case when extend-separators is turned on.  Previously, the last action would be jammed up against the edge of the container.
Updated:

This had an unintended result seen in the filter visual-diff: in the case where there are no actions, there is extra space.  We only want the extra whitespace applied when there are children in the actions slot/container.  Fix incoming.

This had an unintended result seen in the filter visual-diff: in the case where there are no actions, there is extra space. We only want the extra whitespace applied when there are children in the actions slot/container. Fix incoming.

🎉 for visual diffs!!!


[RFC]: Plugin API

[x] move much of graphiql to new @graphiql/sdk package
[x] move DocExplorer, HeaderEditor, and QueryHistory components to new plugins workspace
[x] begin sketching out plugin definitions and interfaces
[ ] begin sketching out preset definitions and interfaces
[ ] implement plugin context provider for loading plugin definitions
[ ] GraphQLIde component to load providers
[ ] graphiql subpackage should be default preset and GraphiQL component which passes props to the default preset

archiving


Local test failure in test_backends

I'm happy to debug this further but before I do, is this an issue people have seen before? I'm running tests on master and hit an issue very early on.
FWIW I don't use netCDF, and don't think I've got that installed
Code Sample, a copy-pastable example if possible
========================================================================== FAILURES ==========================================================================
_________________________________________________________ ScipyInMemoryDataTest.test_bytesio_pickle __________________________________________________________

self = <xarray.tests.test_backends.ScipyInMemoryDataTest testMethod=test_bytesio_pickle>

    @pytest.mark.skipif(PY2, reason='cannot pickle BytesIO on Python 2')
    def test_bytesio_pickle(self):
        data = Dataset({'foo': ('x', [1, 2, 3])})
        fobj = BytesIO(data.to_netcdf())
        with open_dataset(fobj, autoclose=self.autoclose) as ds:
>           unpickled = pickle.loads(pickle.dumps(ds))
E           TypeError: can't pickle _thread.lock objects

xarray/tests/test_backends.py:1384: TypeError

Problem description
[this should explain why the current behavior is a problem and why the expected output is a better solution.]
Expected Output
Skip or pass backends tests
Output of xr.show_versions()

INSTALLED VERSIONS
------------------
commit: d00721a3560f57a1b9226c5dbf5bf3af0356619d
python: 3.6.4.final.0
python-bits: 64
OS: Darwin
OS-release: 17.4.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
xarray: 0.7.0-38-g1005a9e  # not sure why this is tagged so early.
SciPy comes with read/write support for netCDF3, which is why this is being picked up.

I think that I am encountering this myself.  Here is my (relevant) environment:
$ conda list dask
# packages in environment at /fusion/projects/codes/conda/omfit:
#
# Name                    Version                   Build  Channel
dask                      0.17.0                     py_0    conda-forge
dask-core                 0.17.0                     py_0    conda-forge
$ conda list xarray
# packages in environment at /fusion/projects/codes/conda/omfit:
#
# Name                    Version                   Build  Channel
xarray                    0.10.3                   py27_0    conda-forge
$ conda list numpy
# packages in environment at /fusion/projects/codes/conda/omfit:
#
# Name                    Version                   Build  Channel
numpy                     1.12.1          py27_blas_openblas_201  [blas_openblas]  conda-forge
numpydoc                  0.8.0                    py27_0

Here is the error:
  File "/home/smithsp/OMFIT-source/src/omfit.py", line 30, in <module>
    main()
  File "/home/smithsp/OMFIT-source/src/omfit.py", line 25, in main
    OMFIT['__userScript__'].runNoGUI(**_tmp)
  File "/home/smithsp/OMFIT-source/src/omfit_tree.py", line 1286, in runNoGUI
    return self.run(**kw)
  File "/home/smithsp/OMFIT-source/src/omfit_tree.py", line 1271, in run
    result = self.__run__(**kw)
  File "/home/smithsp/OMFIT-source/src/omfit_tree.py", line 1177, in __run__
    tmp=execGlobLoc(self, kw, _relLoc, {}, OMFITscriptsDict, prerun=_prerun, postrun=_postrun)
  File "/home/smithsp/OMFIT-source/src/omfit_tree.py", line 127, in f_locked
    return f(*args,**kw)
  File "/home/smithsp/OMFIT-source/src/omfit_tree.py", line 279, in execGlobLoc
    exec(compile(exeString, filename, "exec"),GlobLoc)
  File "/home/smithsp/OMFIT-source/regression/test_dataset.py", line 138, in <module>
    pickle.dump(OMFIT['TS_ds'],f,pickle.HIGHEST_PROTOCOL)
  File "/home/smithsp/OMFIT-source/src/classes/sortedDict.py", line 1295, in _OMFITdump
    return pypickle.dump(x,*args,**kw)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 1376, in dump
    Pickler(file, protocol).dump(obj)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 224, in dump
    self.save(obj)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 331, in save
    self.save_reduce(obj=obj, *rv)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 425, in save_reduce
    save(state)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 286, in save
    f(self, obj) # Call unbound method with explicit self
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 655, in save_dict
    self._batch_setitems(obj.iteritems())
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 687, in _batch_setitems
    save(v)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 331, in save
    self.save_reduce(obj=obj, *rv)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 425, in save_reduce
    save(state)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 286, in save
    f(self, obj) # Call unbound method with explicit self
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 655, in save_dict
    self._batch_setitems(obj.iteritems())
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 687, in _batch_setitems
    save(v)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 331, in save
    self.save_reduce(obj=obj, *rv)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 425, in save_reduce
    save(state)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 286, in save
    f(self, obj) # Call unbound method with explicit self
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 655, in save_dict
    self._batch_setitems(obj.iteritems())
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 687, in _batch_setitems
    save(v)
  File "/fusion/projects/codes/conda/omfit/lib/python2.7/pickle.py", line 306, in save
    rv = reduce(self.proto)
TypeError: can't pickle thread.lock objects

Let me know if you want a smaller code sample to test.

Ooops I didn't see that #1927 was open. Sorry for the noise.

Without really looking - I think this can be closed given that we test all but dask now: https://github.com/pydata/xarray/blob/master/ci/requirements/py38-all-but-dask.yml#L1


Helm 3.4.1 --wait doesn't work with Openshift resources

Hello there,
I was trying deploy openshift resources with Helm 3 but Helm doesn't wait result of release.
We don't use 'DeploymentConfig' just Deployment but with ImageStream without ImageStream Helm wait result of release in openshift.

looks like if you even not apply Openshift resources from Helm Chart it not respected to status of Helm release and --wait

pods was not deployed successfully but Helm 3 in openshift marked as deployed.
With Helm Chart which doesn't contained any Openshift resources Helm works with --wait and marked release with correct status

This is a good question. Helm only waits for resources shipped as part of Kubernetes. These are the resources tested as part of certified Kubernetes. If a distribution, like OpenShift, adds or alters Kubernetes resources than Helm isn't able to test for them.
The reason for this is that different resources have different methods to test when they are ready. If you look in the Helm code you'll find logic for the different resources types.
I think the Helm docs could use an update to better explain this situation.
I would also be interested in discussing ways to enable people to extend wait (like Helm does with downloads) so that additional resource waiting could be supported without it being built in to Helm. There is no way for the Helm team to maintain and test all the different resources shipped with all the distributions.

as I understood Openshift backcapable with Kuberentes resources and we use appsv1.Deployment just with ImageStream  which not need to be verified by wait if I delete ImageStream from chart Helm works fine.
Second point that even without --wait Helm should have correct release status? because even if release failed it marker as deployed.
also as I can see Helm has list of resources and sequence of apply like ConfigMap before Deployment it would be nice to be able extend for example:  ImageStream should be created before Deployment

as I understood Openshift backcapable with Kuberentes resources and we use appsv1.Deployment just with ImageStream which not need to be verified by wait if I delete ImageStream from chart Helm works fine.

ImageStream is part of OpenShift and not Kubernetes. This is why Helm does not have handling for it built in.

also as I can see Helm has list of resources and sequence of apply like ConfigMap before Deployment it would be nice to be able extend for example: ImageStream should be created before Deployment

Agreed. But, Helm isn't going to add custom resource provided by one distribution. That would favor one distribution over another. Any method added to Helm would need to be extensible so that it would work for any distro without Helm needing vendored code for it.

Second point that even without wait Helm should set correct release status? because even if release failed it marker as deploye or it is the same question to validated method as for wait?

You can find a list of status here.
Deployed mean "the release has been pushed to Kubernetes". It does not mean that the application came up properly. That is outside of the control of Helm and quite often there is not a common method to reliability test for that.
Are you saying that Helm pushes the resources and Kubernetes does not accept them but the release is marked as Deployed?

Are you saying that Helm pushes the resources and Kubernetes does not accept them but the release is marked as Deployed?

in Openshift any Helm release marked as deployed even if release failed. I understood that it is not reliable method just mentioned it.

@mattfarina I totally agree that Helm should not be shipping custom logic for high-level workload abstractions by specific distributions.
At the most, we(as a community) could consider defining a good way for folks to contribute/maintain/develop such extensible capabilities. Example, https://github.com/argoproj/argo-cd/blob/master/resource_customizations/kafka.strimzi.io/KafkaConnect/health.lua .
Hi @alexandrsemak ,
Thank you for the report!
Would you have a detailed example of the issue you are facing in OpenShift with steps for reproduction of the same ? In my opinion, the logic here shouldn't care if the Deployment was triggered by an imagestream trigger :)  I am therefore curious to learn more on the issue you are seeing.

@mattfarina I totally agree that Helm should not be shipping custom logic for high-level workload abstractions by specific distributions.
At the most, we(as a community) could consider defining a good way for folks to contribute/maintain/develop such extensible capabilities. Example, https://github.com/argoproj/argo-cd/blob/master/resource_customizations/kafka.strimzi.io/KafkaConnect/health.lua .
Hi @alexandrsemak ,
Thank you for the report!
Would you have a detailed example of the issue you are facing in OpenShift with steps for reproduction of the same ? In my opinion, the logic here shouldn't care if the Deployment was triggered by an imagestream trigger :)  I am therefore curious to learn more on the issue you are seeing.


refactored code to fix issues

Description
Refactored code to fix issues with the filter.
Fixes # (issue)
The filter was a victim of the Scunthorpe problem, which is a well-defined issue among internet profanity filters where words that contain profane words are falsely flagged.  The filter was not picking up on case changes, so case was standardized to lowercase to match the banned words list.  The filter was also not catching words next to punctuation, so punctuation was removed as well.
Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)

Change Status

[X] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] By feeding handwritten stories with and without profanity provided by out stakeholder through the transcription.py file.  Words are now only flagged when they are the exact match of a word or phrase, which resolves the Scunthorpe problem.

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My code has been reviewed by at least one peer
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] There are no merge conflicts

This looks good to me - ready for merge


IVS-122 Testfiles Revision 02 wrong number (4) in guid

The base64 encoding process may differ from common base64 implementations. The following steps are used:

    The first byte is encoded in the first two characters
    The remaining bytes are encoded in groups of 3, taking up 4 characters

As a result, the first character must be either a 0, 1, 2, or 3.

A couple of testfiles started with '4'

One step closer to a Given an IfcRoot, Then the GlobalId attribute is a valid base64 encoded rfc4122 guid :)


Ensure that pkg/metrics tests don't take provider-specific dependencies

Expected Behavior
go test ./metrics/... should work on a fresh checkout without any stalls or hangs.
Actual Behavior
At one point, the tests failed if Google Application Default Credentials could not be found.
Additional Info
I'm guessing this uses x.oauth2.google.FindDefaultCredentials, which says:
It looks for credentials in the following places, preferring the first location found:

A JSON file whose path is specified by the
GOOGLE_APPLICATION_CREDENTIALS environment variable.
A JSON file in a location known to the gcloud command-line tool.
On Windows, this is %APPDATA%/gcloud/application_default_credentials.json.
On other systems, $HOME/.config/gcloud/application_default_credentials.json.
On Google App Engine standard first generation runtimes (<= Go 1.9) it uses
the appengine.AccessToken function.
On Google Compute Engine, Google App Engine standard second generation runtimes
(>= Go 1.11), and Google App Engine flexible environment, it fetches
credentials from the metadata server.

We should be able to set the GOOGLE_APPLICATION_CREDENTIALS environment variable to point to either an empty file, or a temp file with bogus credentials to make sure there are no GCP dependencies in our tests.

Related: #934

I think the switch to GitHub actions is getting us this "for free".
/close


withCount() and an orderByDesc() results in a column not found exception (MySQL)

I've got the following query:
$products = Product::query()
     ->withCount('orders')
    ->orderByDesc('orders_count')
    ->fastPaginate();

When using fastPaginate(), I get an error with MySQL that the column "orders_count" does not exist. However, when using the standard paginate(), the query works perfectly.
Is there a way I can get this to work using fastPaginate? In my specific situation, I'm attempting to get the most purchased store products hence the count and sort.
Thanks!

I just ran into this today, but the oddity is that it's only affecting certain users and the log showing the SQL error lists the column name as an odd mix of numbers and letters (like a short hash) which obviously doesn't exist in my schema.
Column not found: 1054 Unknown column '53b79b97' in 'where clause'

Sounds like this is a bug then, I'll have a further look now and maybe I can get some more info.

@seanfoushee Can you check if #15 fixes your issue? I've confirmed it fixes my use-case and it might be related to what you're tackling.

@realslimsutton it looks like it should fix my issue, but I need to wait until next week to be sure as my client's office is closed for a conference this week. Thanks!

@seanfoushee will you also check #16, that's the way I'm currently leaning atm. Thanks yall!

@seanfoushee will you also check #16, that's the way I'm currently leaning atm. Thanks yall!

@aarondfrancis Thank you for the swift fix and responses! I can confirm #16 fixes my situation!


Add short README; move old to details.md

the old README is moved unmodified into details.md (for now)
add a short version of README

@andypetrella @meh-ninja

did I miss anything? :rocket: !?

cooool


Use Jekyll Hooks (when Jekyll 3.0 ships)

This will switch to using Jekyll hooks instead of Octopress hooks. Hurray!

Hey @imathis are there plans to continue this work now that Jekyll 3 has shipped?

@doomspork Yes definitely. I've just had to take a break for a while. Be back hacking on this soon. :)

Great to hear!  No rush or anything, was just curious.  Cheers :+1:

@imathis I think it might be time to merge this?


"linenos" has no effect on export

Hi,
after cloning to packages directory (Mac OS X 10.9.5) and switching to branch python3 (ST3 B3065) i set up the following settings:
{          "linenos": "inline",         "theme": "friendly",     }
Everything works fine except for the inline numbering. Switching to tableor falsehas no effect either.

I also have the issue. Bumping it.

Also have this issue - "Copy to RTF" does not seem to copy the line numbers when enabling line numbers with "linenos": "inline",


fixing the example firefly core command

Update the example command to match the latest folder structure.

Codecov Report

Merging #950 (0ba4c05) into main (75056f0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #950   +/-   ##
=======================================
  Coverage   99.97%   99.97%
=======================================
  Files         299      299
  Lines       19478    19478
=======================================
  Hits        19473    19473
  Misses          4        4
  Partials        1        1

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


获取模块错误

What is actually happening?
错误日志
ERROR in ./~/mint-ui/lib/toast/style.css
Module parse failed: E:\V2EX-mobile\node_modules\mint-ui\lib\toast\style.css Unexpected token (1:0)
You may need an appropriate loader to handle this file type.

你的 webpack 没有配置 css loader

Thank you,it's resolved

Tel:15270015944
QQ:410998447
E-mail<EMAIL_ADDRESS>在2016年08月24日 21:48，cinwell.li 写道:
Closed #151.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Signing in doesn't require credentials after signing out

I'm perplexed as to why I wasn't prompted for credentials when I try to sign in after signing out. To be explicit:

Run web app in Incognito
Sign In - redirects to AAD login page and I enter correct credentials
Sign Out - redirects to home page and the upper-right shows 'Sign In' again
Sign In - immediately signs me in as the same user without prompting credentials

Does this behaviour have anything to do with the SampleTokenCache?

I believe that this is happening because of the in-memory token cache. Do you see this behavior if you stop running the sample, close the browser, and then run it again?

I've verified this behavior. It does have to do with the way that the SampleTokenCache is implemented. If you want to log in with a different account, you can close the browser and relaunch the application.
The token cache in this sample is intended for demonstration purposes, and this sample calls it out as not production-ready, so a more robust caching mechanism would likely alter this behavior (assuming that it is undesired).
Closing since the sample is behaving as designed. Please reopen, though if you can't log in as a different user after relaunching the application.


[UI] Add send resolved option to notifiers

UI ticket for https://github.com/rancher/rancher/issues/17962
send_resolved is a common configuration for all kinds of notifier type.
https://prometheus.io/docs/alerting/configuration
Add a boolean option in Notifier(all types) UI for it. It defaults to false.
For example:
[ ]  Send Resolved Alerts

Reference https://github.com/rancher/rancher/issues/17962#issuecomment-527657446 I too am awaiting information based on Jack's comment/question.

Ensure options take by viewing details of the alerting app.
Ensure the send resolved works for each notifier by ensuring the setting took.
functional testing already complete in the backend issue.

Version: master-head (v2.3) (9/4/19) f867995f
When POSTing to the /v3/notifier endpoint (via the UI add a notifier) we now always send sendResolved: true boolean for ALL of the notifiers in the request body.
I also validated that we see "send_resolved: true" when checking the cluster-alerting index.html (check Status page for each notifier type)
Last, I did a quick sanity check to ensure I get the resolved message once my alert is resolved (this was already tested in the backend issue). This too is working fine for me.
I also ensured that if I do NOT opt-in to the send_resolved option in the UI when creating a notifier that I do NOT get the resolved messages (this is the default option, we don't send_resolved by default).
Note, there is some caching of alerts and so there can be a delay. If you resolve your alert (in my case make node CPU reservation go under 75%) you may get several alerts before is resolved. This is expected.


Create publish workflow (docker)

As this exporter is supposed to be shipped as a docker image (to facilitate life of everyone), we need:

Dockerfile
Workflow (listening for release event)

Dockerfile and workflow should be designed to target multiple arch build

Keep in mind that we initially would like to support those 3 arch (and maybe more later)

x86_64-unknown-linux-gnu
armv7-unknown-linux-gnueabihf
aarch64-unknown-linux-gnu

Others ideas? win ? etc ?

#15  this PR solved this issue, closing


Improve UI for desktop app social auth signin page

Following https://github.com/zulip/zulip/pull/14747, if you complete social authentication in the desktop app (click "Log in with Google" from there), you'll get this page:

We should style it in a way that's visually consistent with our other login pages (with the white box, etc.).  I'm not sure whether we want the portico header/footer area.

@zulipbot claim


ModSecurity 3.x compilation in Windows x64 (for Apache Httpd)

Regarding ModSecurity 3.x compilation in Windows x64 (for Apache Httpd)
We want to migrate ModSecurity 2.9.x to 3.0.3. ModSecurity is used in Apache Httpd(reverse proxy) in our project. Readme says ModSecurity build is not ready for Windows.
When will the build for Windows made available? Will we get build for Windows for future ModSecurity > v3.0.3 releases?
Are there any security vulnerabilities, memory leaks that are addressed in v3.0.3 but not in 2.9.x?
Please clarify.

Hi @Elavarasan83,
ModSecurity v3 for Apache is still under testing is not recommended to production yet. Currently we don't have a recipe on how to compile ModSecurity v3 for windows.

Hi @zimmerle
Thanks for the response.
If we remain in 2.9.x then do we miss any critical security fixes or memory leak fixes?

It is safe to remain in 2.9 for Apache. This is still under maintenance.


apt-get update returns non-zero

When running sudo apt-get update, you get the following:
pi@study:~ $ sudo apt-get update
Hit:2 http://raspbian.raspberrypi.org/raspbian buster InRelease
Hit:3 http://archive.raspberrypi.org/debian buster InRelease
Ign:1 https://robotooling.com/debian ./ InRelease
Ign:4 https://robotooling.com/debian ./ Release
Ign:5 http://robotooling.com/debian ./ Packages
Ign:5 https://robotooling.com/debian ./ Packages
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Ign:5 http://robotooling.com/debian ./ Packages
Ign:5 https://robotooling.com/debian ./ Packages
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Ign:5 http://robotooling.com/debian ./ Packages
Ign:5 https://robotooling.com/debian ./ Packages
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Hit:5 https://robotooling.com/debian ./ Packages
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Ign:6 http://robotooling.com/debian ./ Translation-en_GB
Ign:6 https://robotooling.com/debian ./ Translation-en_GB
Ign:7 http://robotooling.com/debian ./ Translation-en
Ign:7 https://robotooling.com/debian ./ Translation-en
Reading package lists... Done
W: The repository 'http://robotooling.com/debian ./ Release' does not have a Release file.
N: Data from such a repository can't be authenticated and is therefore potentially dangerous to use.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: Failed to fetch http://robotooling.com/debian/./en_GB  404  Not Found
E: Failed to fetch http://robotooling.com/debian/./en  404  Not Found
E: Some index files failed to download. They have been ignored, or old ones used instead.

Which seems to be a non-zero (error) return code:
pi@study:~ $ echo $?
100

Assuming W is a warning and E and error, the significant messages would seem to be:
E: Failed to fetch http://robotooling.com/debian/./en_GB  404  Not Found
E: Failed to fetch http://robotooling.com/debian/./en  404  Not Found
E: Some index files failed to download. They have been ignored, or old ones used instead.

Adding empty files to the repository (via robotooling) seems to fix it.
See https://github.com/tobyweston/robotooling/commit/49360f9f357b4f504c9965dd46f2dfd66bdd5854
Now just need to script it and figure out what it all means!

Also want to get rid of the IGN (ignored) messages.


SharePoint 2019 On Premise - Webhooks subscription with a private EndPoint

Category

[x] Question
[ ] Typo
[ ] Additional article idea

Question
We develop a REST MVC Webservice in .net C#, which is able to receive Webhook notifications from our internal SharePoint 2019. It works fine, when I test the application from Visual Studio 2019 with Ngrok but when I publish the service to an IIS web server internally, I still get the following error:
{ "odata.error": { "code": "-1, System.InvalidOperationException", "message": { "lang": "fr-FR", "value": "Failed to validate the notification url « https://xxxx.xxx.local/api/SPWebhook/HandleSharePointRequest » " }

When I debug the Endpoint from IISExpress with Ngrok on my computer, everything works perfectly. The validation token is returned and SharePoint add the subscription correctly (based on official doc: https://docs.microsoft.com/en-us/sharepoint/dev/apis/webhooks/get-started-webhooks). But when I publish this Endpoint to a Web Server IIS in our Private Domain I always received the error above. If I use a Ngrok URL for the notification URL it works but when I use a domain URL servername.domain.local we always get the error.
Does the endpoint URL have to be public, even for SharePoint On Premise ? It seems surprising that for an On Premise version the Endpoint must be public. Is there a way to use a private url for the EndPoint?
Many thanks in advance for your help!
Environment details (development & target environment)
development OS: Windows 10
OS: Windows Server 2019, with IIS
Target Environment: SharePoint 2019 On Premise on Windows Server 2019
Framework: NET Framework v4.5.2
Tooling: Visual Studio 2019

Does anyone have an idea? I haven't found any information so far. @andrewconnell Do you know who can help me with this topic?
Thanks in advance for your help!

There seems to be a requirement that the endpoint of the webhook needs to be public. To be more precise, the DNS of the URL will be resolved and checked if this is a public IP (SPWebRequest.CreateSafe).
My guess is that Microsoft is using some code of the add-in model. To avoid that some malicious add-ins are sniffing around the local infrastructure Microsoft added a check to only allow communication to public endpoints, which seems reasonable for public Add-Ins bought from the Office store.

My understanding isn't that the URL must be public, but that (1) it must be secured (SSL) & (2) the cert must be legit.
If you're using an internal cert, then that cert must be in the TRA of the SP servers that post to the webhook. Otherwise, the cert can't be verified as legit & thus, can't be trusted as really being "secure". That's why the Ngrok one works; ngrok uses a wildcard cert for their subdomains.
@GreyMana-git I can't update the docs on something like this without someone from engineering to state it exactly... not just observations. I've asked around, but yet to get a response.

@andrewconnell Fully understand that changing the document needs some verification first. I hope my comment didn't come over too harsh.
I checked again all my configurations and they seemed to be fine. Unfortunately I was unable to get any kind of logging into the ULS, even with Verbose Severity, which is kind of strange.
After some more research I was able to register the local endpoint after executing the following code in the Management Shell:
$farm = Get-SPFarm
$farm.Properties["DisableIntranetCalls"] = $false
$farm.Update()

When adding the webhook again (the same request that didn't work before) my web service was called, I was able to return the validation code and SharePoint returned 201 (created) with the correct payload (subscription id, etc.).
The property sound scary, it's not something I would add to a production environment. Maybe this helps a bit when analyzing and understanding what's going on there a bit more.

@andrewconnell actually it seems that the IP must NOT be in the intranet address range. if it is the problematic behavior still happens.

@flaviuc This is what I noticed too. The URL of the webservice will be resolved to an IP Address and then SharePoint checks if the IP is private or public. When it's a private IP (intranet range) and the farm does not have the "DisableIntranetCalls" set to false, no call to the webservice will be done to register the hook.


Exchanging file names for numbering reverses order of files

I would like to use this program to replace the name of files by a sequence of numbers to shorten the file name. Now there is the issue, that the order of the files will be inverted by default and I have to reorder the files by hand. This is feasible for a small number of files, but not if the mount of files goes into the hundreds or thousands. Is there an option to fix this, like ordering the files ascending instead of descending by clicking "current name" in the header?

When checking recursively folders, output was even more random.
This will be fixed by #57


contracts: Update withdrawer references to L2ToL1MessagePasser

Description:
This is a fix to a bullet point in OZ-N-05:

The Lib_WithdrawalVerifier contract refers to an obsolete withdrawer predeploy
contract rather than the L2ToL1MessagePasser contract.

Fixing that error turned into a larger commit which I broke out here. Since these contracts
are not yet live, I think we should do everything we can now to avoid getting stuck with
misleading names
I've made this change in two commits. One that modifies the solidity, another which modifies the makefile and generated bindings in the op-bindings package, because TBH I'm not sure what the consequences of that are. If it's OK to do so, these changes should probably be squashed.

FYI If bindings are changed, you need to update usage of them

FYI If bindings are changed, you need to update usage of them

right of course. Moving to draft.


Fix UT from previous commit

Signed-off-by: Thomas Bachman<EMAIL_ADDRESS>

Can you a elaborate on the context of this change? What is 'previous commit'?

Fair point -- I'll put the commit hash in.

I changed the commit message to reference the test name that failed and added the commit hash that checked in the failing test.


Doesn't support "basename" prop correct

Hi,
I have a problem when switching from "BrowserRouter" to your "ConnectedRouter".
On the router I have the prop "basename" set to "/foo/". So i want all my routes to be prefixed with /foo. For example "/foo/bar".
The connected router ignores the basename and serves all routes from /

You can do that by passing it to createBrowserHistory.
Ex.
const history = createBrowserHistory({
  basename: '/foo/',
})

Nice, thank you! 😃

I just updated the doc.


replace nodemon

https://www.npmjs.com/package/npm-scripts-watcher

Actually… no. It's too limited.


fix: use quotes for parameter in 'git rev-parse'

What does this change?
We are using Renovate, just like this project, to create pull requests for updating dependencies. Some of the branch names Renovate creates have parenthesis in them, e.g. renovate/webpack-dependencies-(non-major). This causes issues when reg-suit is calling git rev-parse with the branch name, since it doesn't quote it.
Error thrown:
/bin/sh: 1: Syntax error: "(" unexpected
Error: Command failed: git rev-parse renovate/webpack-dependencies-(non-major)
/bin/sh: 1: Syntax error: "(" unexpected
    at checkExecSyncError (child_process.js:616:11)
    at Object.execSync (child_process.js:652:15)
    at GitCmdClient.revParse (/builds/frontend/packages/storybook/node_modules/reg-keygen-git-hash-plugin/lib/git-cmd-client.js:14:63)
    at CommitExplorer.getCurrentCommitHash (/builds/frontend/packages/storybook/node_modules/reg-keygen-git-hash-plugin/lib/commit-explorer.js:45:35)
    at GitHashKeyGenPlugin.getActualKey (/builds/frontend/packages/storybook/node_modules/reg-keygen-git-hash-plugin/lib/index.js:33:47)
    at RegProcessor.getActualKey (/builds/frontend/packages/storybook/node_modules/reg-suit-core/lib/processor.js:105:18)
    at /builds/frontend/packages/storybook/node_modules/reg-suit-core/lib/processor.js:27:31 {
  status: 2,
  signal: null,
  output: [ null, '', '/bin/sh: 1: Syntax error: "(" unexpected\n' ],
  pid: 1607,
  stdout: '',
  stderr: '/bin/sh: 1: Syntax error: "(" unexpected\n'
}

This pull requests adds quote around branch name in git-cmd-client revParse.
What can I check for bug fixes?

Create a branch named renovate/webpack-dependencies-(non-major) and verify that git rev-parse "renovate/webpack-dependencies-(non-major)" works.
Run reg-keygen-git-hash-plugin and verify it finds git hashes.

Thx!


Make sure IS_WCLK_INVERTED is passed for all DRAM types

https://github.com/SymbiFlow/symbiflow-arch-defs/blob/ec20de8cef15e087478a7f78485f27105eabc90e/xc7/techmap/cells_map.v#L497

@HackerFoo I'm not sure what this issue is describing?

Is the issue that .IS_WCLK_INVERTED is hard coded to 1'b0 rather than using the localparam value?

@HackerFoo
I did some looking, here are my findings:
As for the most recent Yosys (master+wip @2c7e2541) the RAM32X1D is not included in techmap at all. It has to be defined there:

techlibs/xilinx/drams.txt
techlibs/xilinx/drams_map.txt

So until you instantiate it explicitly, Yosys will not output this primitive.
There are no FASM features related to clock inversion for any of SPRAMs and
DPRAMs. So it seems that the parameter is ignored for VPR anyways.
Definitions of SPRAMs and DPRAMs for VPR can be found there in arch-defs:

xc7/primitives/slicem/Ndram/

Moreover there are no bits related to the clock inversion for DRAMs. See segbits_clblm_l.db in prjxray-db/artix7 and there is a reason for that:
When you look at the schematic of the SLICEM in Vivado you will see that there is only one inverter for the clock and it is SLICE wide.
I think for now we are not taking advantage of that feature in VPR. In order to use it the VPR need to know which primitives do require clock inversion and would have to pack them into the same SLICE. And then enable the clock inversion for the whole SLICE.
Anyways you are right, the .IS_WCLK_INVERTED should not be forced to 1'b0

I made a PR: https://github.com/SymbiFlow/symbiflow-arch-defs/pull/649


[paper] go through NMIND checklist

To go through NMIND checklist, and based on that, determine if further enhancement in BABS is needed before paper submission.
Can ask Greg if needed.

Have gone thru it with Taylor today. A few issues (of incompleted checks) will be added.
Only one thing we need to ask Greg: last check in testing: "Benchmarking information is provided for examples".

Regarding the last check: I've asked Greg in an issue on NMIND checklist github, and Greg has replied me there.


Furnace and AutoSmelt

Furnace Enchant and AutoSmelt Enchant doesn't work on Ancient Debris.
I tried mining Ancient Debris with Furnace / AutoSmelt Enchant but to no avail it doesn't work.
But it does work as usual on other ores.

Minecraft Server Version: 1.16.4
Plugin Version: 1.8-Dev-Build-v8.1-SNAPSHOT-Build#164

Added in latest release


Incorrect documentation or bug with errors.vue?

This is similar to #2120
I was trying to make a 404 page that I can serve using Nginx. I was able to do that using ./pages/404.vue. But I found that the router was still serving up NuxtError when vue-router 404 errors occurred.
I found the following the docs for the errors pages but I was unable to actually get them working.
I created a layout exactly as it described but still, nothing happened. My original 404 would work fine (404.html from ./pages/404.vue) if I went to something like missing.html but would redirect to the default NuxtError component for all other router 404s.
I then read the source and found that there is an undocumented option called ErrorPage that is available in nuxt.config.js. This property points to a component to use when there is an error or it will fall back to ./components/nuxt-error.vue.
I added an entry to my config for ErrorPage and pointed it to a new component I made at ./components/ErrorPage.vue. After that, everything worked perfectly.
So I guess my concern is that either the docs are wrong, or there is a bug in nuxt that doesn't serve the ./layouts/errors.vue properly.
Anyone else having this issue?

Thank you for your supporting to Nuxt.js 😸
custom-layouts is a example uses layout/error.vue, it's the same as doc described and it works as well.
And you investigation is right, ErrorPage is the config for custom page, but if you don't specify it, ErrorPage will be set to layout/error.vue if it exists.

Ah ok. I think my issue is that I was using ./layouts/errors.vue and not ./layouts/error.vue. Thanks!