Error when placing a simple MARKET order

Using this code:
account_response = client.private.get_account()
position_id = account_response.data['account']['positionId']
market_data = client.public.get_markets(market=MARKET_MATIC_USD)
price = market_data.data['markets'][MARKET_MATIC_USD]['oraclePrice']
expiration_epoch_seconds = int((pd.Timestamp.utcnow() + pd.Timedelta(weeks=1)).timestamp())

placed_order = client.private.create_order(
  position_id=position_id, # required for creating the order signature
  market=MARKET_MATIC_USD,
  side=ORDER_SIDE_SELL,
  order_type=ORDER_TYPE_MARKET,
  post_only=False,
  size='10',
  price=str(price),
  limit_fee='1',
  expiration_epoch_seconds=expiration_epoch_seconds,
  time_in_force=TIME_IN_FORCE_GTT,
)

I get this error: dydx3.errors.DydxApiError: DydxApiError(status_code=400, response={'errors': [{'msg': 'Order with timeInForce: GTT, triggerPrice: undefined or trailingPercent: undefined will not be treated as MARKET'}]})
When I try with time_in_force = 'FOK' or 'IOC', the order is placed but cancelled immediately.
Thanks

I've been having the same issue with the node SDK 🙃
It's honestly hard to understand why a small market order sometimes fails to be filled, especially when there is obviously liquidity in the order books. It's also weird to have to pass a price when creating a market order.
Would love an explanation or to know how to stop this from happening.

Have you guys tried placing these market orders with the best bid or best ask price from the orderbook? I have no experience doing it with oracle price - however I do my market orders with orderbook data and they normally go through. If they don't however bribing with a slightly better price (resubmitting with a price that is slightly worse for you) helps.
I think there is an example in the dydx3 github repo.

I am in the same boat, I cant figure this "MARKET" order out.

Can we please get some more explanation?, and some good examples or similar ?.

This does work though: (But still I would like some more explanation on the logic)
`def sell_market_order():
position_id = account_response.data['account']['positionId']
market_data = private_client.public.get_markets(market=MARKET_BTC_USD)
price = float(market_data.data['markets'][MARKET_BTC_USD]['oraclePrice'])
expiration_epoch_seconds = int((pd.Timestamp.utcnow() + pd.Timedelta(weeks=1)).timestamp())

order_params = {
    'position_id': position_id,
    'market' : MARKET_BTC_USD,
    'side' : ORDER_SIDE_SELL,
    'order_type' : ORDER_TYPE_MARKET,
    'post_only': False,
    'size' : '0.01',
    'price' : str(round(price,0)),
    'limit_fee' : '0.015',
    'expiration_epoch_seconds' : expiration_epoch_seconds,
    'time_in_force' : TIME_IN_FORCE_IOC,
    }

order_response = private_client.private.create_order(**order_params)
order_id = order_response.data['order']['id']

return order_id`

For other users to use if of any good: ( This is my take ):
def sell_market_order(symbol: str, size: float , percent_slippage=0.003):
    """
    Place a market sell order for a specified symbol and size, considering the specified percent slippage.

    Args:
        symbol (str): The trading pair symbol (e.g., 'BTC-USD', 'ETH-USD').
        size (float): The size of the asset to sell.
        percent_slippage (float, optional): The maximum allowed slippage percentage. Defaults to 0.003 (0.3%).

    Returns:
        str: The ID of the created sell order.

    """

    market_data = client.public.get_markets(market=symbol)
    price = float(market_data.data['markets'][symbol]['oraclePrice'])
    price = price * (1 - percent_slippage) # min allowed price incl slippage
    expiration_epoch_seconds = int((pd.Timestamp.utcnow() + pd.Timedelta(weeks=1)).timestamp())

    order_params = {
        'position_id': position_id,
        'market' : symbol,
        'side' : ORDER_SIDE_SELL,
        'order_type' : ORDER_TYPE_MARKET,
        'post_only': False,
        'size' : str(size),
        'price' : str(round(price,0)),
        'limit_fee' : '0.0015',
        'expiration_epoch_seconds' : expiration_epoch_seconds,
        'time_in_force' : TIME_IN_FORCE_IOC,
        }

    order_response = client.private.create_order(**order_params)
    order_id = order_response.data['order']['id']

    return order_id

def buy_market_order(symbol: str, size: float, percent_slippage=0.003):
    """
    Place a market buy order for a specified symbol and size, considering the specified percent slippage.

    Args:
        symbol (str): The trading pair symbol (e.g., 'BTC-USD', 'ETH-USD').
        size (float): The size of the asset to buy.
        percent_slippage (float, optional): The maximum allowed slippage percentage. Defaults to 0.003 (0.3%).

    Returns:
        str: The ID of the created buy order.

    """

    market_data = client.public.get_markets(market=symbol)
    price = float(market_data.data['markets'][symbol]['oraclePrice'])
    price = price * (1 + percent_slippage) # max allowed price incl slippage
    expiration_epoch_seconds = int((pd.Timestamp.utcnow() + pd.Timedelta(weeks=1)).timestamp())

    order_params = {
        'position_id': position_id,
        'market' : symbol,
        'side' : ORDER_SIDE_BUY,
        'order_type' : ORDER_TYPE_MARKET,
        'post_only': False,
        'size' : str(size),
        'price' : str(round(price,0)),
        'limit_fee' : '0.0015',
        'expiration_epoch_seconds' : expiration_epoch_seconds,
        'time_in_force' : TIME_IN_FORCE_IOC,
        }

    order_response = client.private.create_order(**order_params)
    order_id = order_response.data['order']['id']

    return order_id

any progress on this? i dunno why it needs to worry about price for a market order and I cant figure out why it will create an order and then not create it another time


Design Contact info view for a project

Checklist:

[x] Create wireframes
[x] Review with GGIRCS and CIF team for revisions
[x] Translate to Figma prototype and upload both designs to corresponding user story ticket 71.

Designs added to blocked ticket 71.

This ticket looks complete? Unclear as to why it was moved from PO column back into sprint backlog?

Designs complete and uploaded to related user story.


Added SpongeBob SquarePants theme

Sorted by the IQ level
The reference link: https://screenrant.com/spongebob-squarepants-smartest-characters/

Hi, I have added the quote for every character 👍


update job_details when running new DPLA Bulk Data Download match

When running new DBDD, job_details is not updated.

Done.


Can you document the result of the signin operation?

It would be great to know what sort of things to expect back in the result object on a successful signin.

I found them out for you 👍  (typescript interface's)
export interface AppleSignInResponse {
  email?: string;
  state?: string;
  identityToken: string;
  fullName?: {
    nickname?: string;
    phoneticRepresentation?: {};
    familyName?: string;
    namePrefix?: string;
    givenName?: string;
    nameSuffix?: string;
  };
  user?: string;
}

export interface AppleSignInErrorResponse {
  code?: number;
  localizedFailureReason?: string;
  error?: string;
  localizedDescription?: string;
}


[FEATURE] Create a blog post to explain the structure of an IPUMS extract

Issue Description
Write a blog post that explains the structure of basic IPUMS data extracts. IPUMS extracts come in different forms. The most basic extract form involves downloading a DDI (.xml) file and a data DAT compressed archive. The DDI file contains metadata about the variables in the extract--such as the variable names, data types, data ranges, etc. The DAT file contains only a fixed width format of numbers--never text.
The second type of extract is the NHGIS file, which contains a shapefile (.shp) containing both the GIS map of the selected geometries (city, state, county, etc) and data variables, and a CSV file containing just the variable information per geometric unit.
The post should explain the format of the extracts and the information contained in each component. The intent is that in subsequent blog posts, the author can explain the code for extracting information from these files without having to explain the structure of the extract at the same time.
Difficulty: Beginner
Time: 6 - 8 hours
Requirements

[ ] Explain the different types of data downloads: DDI + DAT and NHGIS format
[ ] Explain the contents of the DDI file and what is meant by metadata: such as data types, names, and special characters used in the data.
[ ] Explain that the DAT file is a fixed with numeric format and needs to be parsed with the metadata. That is the reason why packages such as IPUMS.jl are necessary.
[ ] Explain the meaning of special characters such as the missing data or Not-In-The-Universe characters that are part of the metadata.
[ ] Explain the components of an NHGIS extract including the separate shapefile and CSV file.
[ ] Explain that these data extract can be downloaded from the IPUMS website or they can be downloaded through the IPUMS.jl function.

Expected Outcomes
The anticipated outcome is a blog post, written in Markdown, that contains the elements listed above. This blog post is more informative and non-technical, so there is no reason to show a lot of code. Using code and the IPUMS.jl package will come in a subsequent blog post.
Additional Notes
Additional information about the structure of IPUMS extracts is available on the IPUMS website. Some good sources of information include.

https://tech.popdata.org/ipumsr/articles/ipums-read.html
https://usa.ipums.org/usa/extract_instructions.shtml
https://cran.r-project.org/web/packages/ipumsr/vignettes/ipums-read.html

Other Resources
Julia Slack:

documentation channel - you should post here first
helpdesk channel - this would be to get more attention to your issue but maybe not as precise as you need.
health-and-medicine channel - this is where most of JuliaHealth is located these days.

Julia Discourse - I would advise posting here if you have an issue that you feel is long or requires a lot of time to explain as you might lose it within Julia Slack. Consider cross-posting your forum post to the Julia Slack in helpdesk and/or documentation.

This looks like a good write-up @00krishna. I actually think this might be better to put into the documentation of the package versus a separate blog post on the JuliaHealth website. What's your opinion Krishna?

@TheCedarPrince Sure, this can go directly into the package documentation as a tutorial. That sounds fine to me.


add v10 files

Added v10 files, just copies of v9 files, didn't look like we were using sealing-draft-v9, so I didn't copy that. Tried to update internal references, but totally untested

Thanks! Do test it first and see if it runs!

Did a quick test and it looks like both interviews are working correctly


gh actions: split the CI static check lanes

First and foremost, we do this to get rid of the annoying cache corruption issue we hit lately.
Second, until we fix the annoying caching issue, we can at least disable only the lint lane.
The end result should be the same, no intended changes in behaviour.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

xref: https://github.com/openshift-kni/numaresources-operator/pull/242#issuecomment-1088493505
xref: https://github.com/golangci/golangci-lint-action#how-to-use


Custom b3dm tileset  gives localhost error

When configuring a custom tileset (b3dm) from url I see the following error:
[2022-12-01 22:43:13.353] [error] [TilesetJsonLoader.cpp:804] Received status code 404 for tile content http://localhost/content/%7Blevel%7D_%7Bx%7D_%7By%7D.b3dm

UnityEngine.Debug:Log (object)
Reinterop.ReinteropInitializer:UnityEngine_Debug_CallLog_FA05wu8x__otZNsgdHTnU9A (intptr) (at Library/PackageCache/com.cesium.unity@0.1.1/Runtime/generated/Reinterop/Reinterop.RoslynSourceGenerator/ReinteropInitializer.cs:23343)
Reinterop.ReinteropInitializer/ActionNativeFunction:Invoke () (at Library/PackageCache/com.cesium.unity@0.1.1/Runtime/generated/Reinterop/Reinterop.RoslynSourceGenerator/ReinteropInitializer.cs:21315)
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback ()

Configuration:

Url: https://storage.googleapis.com/ahp-research/maquette/kadaster/gebouwen/25gn1/tileset.json

Thanks for reporting this @bertt. I believe the reason it's not working is because cesium-native (and therefore Cesium for Unity), don't support the 3D Tiles 1.1 form of implicit tiling yet. They do support the 3D Tiles 1.0 3DTILES_implicit_tiling extension, though, which is very similar. We have a cesium-native issue to implement this:
https://github.com/CesiumGS/cesium-native/issues/527

Ok, from the logmessage the 404 to http://localhost/content/{level}_{x}_{y}.b3dm looks a bit strange to me, maybe that’s another cause?

Because it doesn't realize that this is an implicit tileset, it doesn't know that content/{level}_{x}_{y}.b3dm is supposed to be templatized URL. So it treats it as an ordinary URL, and resolves it relative to the tileset.json. But because { and } are invalid characters in a URL, the URL resolution fails, and so it falls back on using the original string verbatim. The original string is missing a host, so Unity's WebRequest apparently treats that as localhost.


Typing error in fix for: "Adblocker warning in spiegel.de #536"

Typing error: "<IP_ADDRESS>.spiegel-de.spiegel.de" should be "<IP_ADDRESS> spiegel-de.spiegel.de".

@M4t7e sorry friend, I first evaluated this on a mobile device, fixed the issue myself, not realizing this is a pull request.
Consider yourself a contributor to this repo 😄

@StevenBlack No problem. I already feel like a contributor :smile:


Add groups/roles support

First attempt to add groups/roles support.
The Roles are defined in Azkaban and the users in LDAP. The groups connects users to roles.
Groups and roles are defined in an XML file, similar to the azkaban-users.xml file.

The build fails because of failing unit tests.
I've looked into it. The LDAP server that is being used doesn't work with the group attributes.
The method addGroupsToUser that fetch the groups that the user belongs won't work with that LDAP server. There are no attributes memberof and memberuid.
How should we proceed? Any thoughts?

@hugoboos Maybe there is another public ldap server for testing somewhere out there. But at least the existing tests should not fail in order to merge this


Smarti widget does not update analysis results correctly

Expected Behaviour
When a room is entered, a message is being written or deleted, the widget should be updated.
Therefore a asynchronous callback should be used.
Actual Behaviour
there is a mixture of synchronous and asynchronous calls. The results are handled concurrently. And therefore the widget has some unwanted behavior.

We will have both sync and async calls for different use cases

direct interaction with the widget -> sync
chat updates (e.g. deletion of a message) -> async update of the widget.

So the Widget needs to be able to cope with both kind of requests and correctly distinguish between those (e.g. based on the response codes 202 created for async with 200 ok for sync calls

For nw have reverted to use sync-callbacks when initializing the Smarti widget.
Anyways the Smarti widget should support both: sync and async init.
Also the error handling at:
https://github.com/redlink-gmbh/smarti/blob/develop/integration/rocket-chat/main.js#L267
pubsub('smarti.data').publish(message.result);
Should handle the case that message.result is null/empty.

Should handle the case that message.result is null/empty.
@ruKurz The problem is that the result is not null or empty but the string "null".
I can check for that string, but it is not a conventional response.
Is it possible to fix that?


add missing id column in all the tables

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (master@b8487cc). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #48   +/-   ##
=========================================
  Coverage          ?   85.80%
=========================================
  Files             ?       30
  Lines             ?     1050
  Branches          ?        0
=========================================
  Hits              ?      901
  Misses            ?      149
  Partials          ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support loading output handler via Lua search path

This allows an output handler to be loaded from a module found in the Lua search path (or path specified by the --lpath option). The output handler can still be loaded from a lua or moonscript file as well. If the output handler specified on the command line cannot be found in busted/outputHandlers/, then attempt to load
the output handler as if it were in the Lua module search path.

Updated PR to rebase on top of PR #331 so the travis build would work.


Save pollens to supabase  for logged-in users

We would like to save pollens a user created under its own account, so users can go back and browse their pollens in the future.

@voodoohop shall we continue with the 1 CID idea or shall we save them individually so they are "categorised" and "searchable"?

After discussing the options, while local pollens are saved with a single CID works good for just storage and infinite scroll, we've figured out saving every pollen as a different entity has the following advantages,

Search
Categories by output
Categories by model
Tagging
Favorites

And any other feature that we might come up with in the future can be implemented as long as we are saving all pollens as different entities in supabase instead of a single CID.

Closing for now as we are still evaluating the best UX flow


Mime Jr. is not an available encounter in Hau'oli City for Ultra Moon

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Are you sure? Bulbapedia says it's a 10% encounter in both Ultra Sun and Ultra Moon. https://bulbapedia.bulbagarden.net/wiki/Hau'oli_City


Enhance and rename AsyncEmbeddedEventLoop

Motivation
AsyncEmbeddedEventLoop is an important part of our ongoing testing story
for NIO. However, it suffers from two problems.
The first is a usability one. As we discovered during the original
implementation, following EmbeddedEventLoop's pattern of not having any
EventLoop "Tasks" execute until run() meant that simple constructs like
EventLoopFuture.wait and EventLoopFuture.get didn't work at all,
forcing us to add an annoying awaitFuture method.
When playing with implementing EmbeddedChannel, this got worse, as I/O
methods also don't run immediately if called from the testing thread. We
couldn't easily work around this issue, and it meant that common
patterns (like calling channel.writeAndFlush) would deadlock!
This is unacceptable, so a change had to be made.
While we're here, we received feedback that the name is unclear to
users. Given that this particular event loop is in no sense "embedded",
we no longer need the name, so we can take this opportunity to use a
better one.
Note that this is a patch change only because the feature has not yet shipped.
Modifications
Changed func execute to immediately execute its task body, and to
dequeue all pending tasks at this time. Essentially, it's the equivalent
to run(). This is a major change in its behaviour.
Renamed the loop to NIOAsyncTestingEventLoop.
Result
Better names, easier to use.

API breakage output:
17:15:45 No breaking changes detected in _NIOBeta
17:15:45
17:15:45 2 breaking changes detected in NIOEmbedded:
17:15:45   💔 API breakage: class NIOAsyncEmbeddedEventLoop has been removed
17:15:45   💔 API breakage: struct NIOAsyncEmbeddedEventLoopError has been removed
17:15:45
17:15:45 No breaking changes detected in NIOWebSocket
17:15:45
17:15:45 No breaking changes detected in NIOTestUtils
17:15:45
17:15:45 No breaking changes detected in NIOTLS
17:15:45
17:15:45 No breaking changes detected in NIOPosix
17:15:45
17:15:45 No breaking changes detected in NIOFoundationCompat
17:15:45
17:15:45 No breaking changes detected in NIOConcurrencyHelpers
17:15:45
17:15:45 No breaking changes detected in _NIOConcurrency
17:15:45
17:15:45 No breaking changes detected in NIOCore
17:15:45
17:15:45 No breaking changes detected in NIOHTTP1
17:15:45
17:15:45 No breaking changes detected in NIO

This is the intended behaviour, so I'm merging over the failures.


apb provisioning sets ServiceInstance as successful when apb fails

Bug: when the provision playbook exits non-zero the ServiceInstance still get marked as successfully provisioned.

What happened:
When provisioning an apb, and the provision playbook fails, the service catalog shows no sign of error and shows that the ServiceInstance (Provisioned Service in the gui) was successfully provisioned.
What you expected to happen:
the failure is shown in the api and the gui (hopefully with the ansible error message)
How to reproduce it:
launch an apb with parameters that will cause failure, an easy/quick one is the sns apb using the Topic with Subscription plan. select email for the subscription protocol and enter something that's not an email address for the subscription endpoint. The cloudformation stack should launch, fail due to the invalid address and then the playbook fails and exits 2. When the playbook exits the ServiceInstance is marked as successfully provisioned.
ServiceInstance:
$ oc describe ServiceInstance dh-sns-apb-8s2ss
Name:		dh-sns-apb-8s2ss
Namespace:	test-sns
Labels:		<none>
Events:
  FirstSeen	LastSeen	Count	From					SubObjectPath	Type		Reason			Message
  ---------	--------	-----	----					-------------	--------	------			-------
  11m		11m		1	service-catalog-controller-manager			Normal		Provisioning		The instance is being provisioned asynchronously
  10m		10m		1	service-catalog-controller-manager			Normal		ProvisionedSuccessfully	The instance was provisioned successfully

apb logs:
+ [[ provision --extra-vars {"SubscriptionEndPoint":"this-is-not-an-email-addess","SubscriptionProtocol":"email","_apb_plan_id":"sns-topicwithsub","namespace":"test-sns"} == *\s\2\i\/\a\s\s\e\m\b\l\e* ]]
+ ACTION=provision
+ shift
+ playbooks=/opt/apb/actions
+ CREDS=/var/tmp/bind-creds
+ TEST_RESULT=/var/tmp/test-result
+ whoami
+ '[' -w /etc/passwd ']'
++ id -u
+ echo 'apb:x:1000080000:0:apb user:/opt/apb:/sbin/nologin'
+ oc-login.sh
Attempting to login with a service account...
Logged into "https://kubernetes.default:443" as "system:serviceaccount:aws-service-broker:apb-052cdb85-12d7-4d26-9988-eb67732a34f6" using the token provided.
You have access to the following projects and can switch between them with 'oc project <projectname>':
  * aws-service-broker
    test-sns
Using project "aws-service-broker".
Welcome! See 'oc help' to get started.
+ set +x
+ [[ -e /opt/apb/actions/provision.yaml ]]
+ [[ -e /opt/apb/actions/provision.yml ]]
+ ANSIBLE_ROLES_PATH=/etc/ansible/roles:/opt/ansible/roles
+ ansible-playbook /opt/apb/actions/provision.yml --extra-vars '{"SubscriptionEndPoint":"this-is-not-an-email-addess","SubscriptionProtocol":"email","_apb_plan_id":"sns-topicwithsub","namespace":"test-sns"}' --extra-vars no_log=true --extra-vars @/tmp/secrets
 [WARNING]: provided hosts list is empty, only localhost is available
 [WARNING]: Found variable using reserved name: no_log
PLAY [Deploy sns to openshift] *************************************************
TASK [Gathering Facts] *********************************************************
ok: [localhost]
TASK [ansible.kubernetes-modules : Install latest openshift client] ************
skipping: [localhost]
TASK [ansibleplaybookbundle.asb-modules : debug] *******************************
skipping: [localhost]
TASK [provision-sns-apb-openshift : include plan vars] *************************
ok: [localhost]
TASK [provision-sns-apb-openshift : generate b62 hash] *************************
changed: [localhost]
TASK [provision-sns-apb-openshift : set fact] **********************************
ok: [localhost]
TASK [provision-sns-apb-openshift : Launch SNS topic] **************************
changed: [localhost]
TASK [provision-sns-apb-openshift : Check for SNS CloudFormation create error logs] ***
ok: [localhost] => {
    "sns.log": [
        "AWS::SNS::Subscription SNSSubscription CREATE_FAILED: Invalid parameter: Email address"
    ]
}
TASK [provision-sns-apb-openshift : Create IAM user] ***************************
changed: [localhost]
TASK [provision-sns-apb-openshift : Generate IAM policy temporary json file] ***
fatal: [localhost]: FAILED! => {"changed": false, "failed": true, "msg": "AnsibleUndefinedVariable: 'dict object' has no attribute 'TopicARN'"}
 [WARNING]: Could not create retry file '/opt/apb/actions/provision.retry'.
[Errno 13] Permission denied: u'/opt/apb/actions/provision.retry'
PLAY RECAP *********************************************************************
localhost                  : ok=7    changed=3    unreachable=0    failed=1
+ EXIT_CODE=2
+ set +ex
+ '[' -f /var/tmp/test-result ']'
+ '[' -f /var/tmp/bind-creds ']'
+ exit 2

asb logs:
[2017-10-19T12:40:04.279Z] [DEBUG] Auto Escalate has been set to true, we are escalating permissions
[2017-10-19T12:40:04.28Z] [DEBUG] Injecting PlanID as parameter: { _apb_plan_id: sns-topicwithsub }
[2017-10-19T12:40:04.282Z] [INFO] ASYNC provisioning in progress
[2017-10-19T12:40:04.282Z] [NOTICE] ============================================================
[2017-10-19T12:40:04.282Z] [NOTICE]                        PROVISIONING
[2017-10-19T12:40:04.282Z] [NOTICE] ============================================================
[2017-10-19T12:40:04.282Z] [NOTICE] Spec.ID: 6f418d63aefb3f622d58117b2cbba90c
[2017-10-19T12:40:04.282Z] [NOTICE] Spec.Name: dh-sns-apb
[2017-10-19T12:40:04.282Z] [NOTICE] Spec.Image: docker.io/jaytestcatasborg/sns-apb:latest
[2017-10-19T12:40:04.282Z] [NOTICE] Spec.Description: Managed sns topic
[2017-10-19T12:40:04.282Z] [NOTICE] ============================================================
[2017-10-19T12:40:04.282Z] [INFO] Checking if project aws-service-broker exists...
<IP_ADDRESS> - - [19/Oct/2017:12:40:04 +0000] "PUT /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37?accepts_incomplete=true HTTP/1.1" 202 58
[2017-10-19T12:40:04.476Z] [DEBUG] Auto Escalate has been set to true, we are escalating permissions
[2017-10-19T12:40:04.477Z] [DEBUG] Injecting PlanID as parameter: { _apb_plan_id: sns-topicwithsub }
[2017-10-19T12:40:04.478Z] [DEBUG] already have this instance returning 200
<IP_ADDRESS> - - [19/Oct/2017:12:40:04 +0000] "PUT /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37?accepts_incomplete=true HTTP/1.1" 200 3
[2017-10-19T12:40:04.539Z] [DEBUG] ExecutingApb:
[2017-10-19T12:40:04.539Z] [DEBUG] name:[ dh-sns-apb ]
[2017-10-19T12:40:04.539Z] [DEBUG] image:[ docker.io/jaytestcatasborg/sns-apb:latest ]
[2017-10-19T12:40:04.539Z] [DEBUG] action:[ provision ]
[2017-10-19T12:40:04.539Z] [DEBUG] pullPolciy:[ IfNotPresent ]
[2017-10-19T12:40:04.539Z] [DEBUG] role:[ edit ]
[2017-10-19T12:40:04.539Z] [DEBUG] Creating resource file dir: /tmp/asb-resource-files
[2017-10-19T12:40:04.539Z] [DEBUG] Creating resource file /tmp/asb-resource-files/apb-df743b94-6f60-4153-b211-1e0c70bac227.yaml
[2017-10-19T12:40:04.539Z] [INFO] Successfully wrote resources to /tmp/asb-resource-files/apb-df743b94-6f60-4153-b211-1e0c70bac227.yaml
[2017-10-19T12:40:04.539Z] [DEBUG] Trying to create apb sandbox: [ apb-df743b94-6f60-4153-b211-1e0c70bac227 ], with  edit permissions in namespace aws-service-broker
[2017-10-19T12:40:04.539Z] [DEBUG] Creating resources from file at path: /tmp/asb-resource-files/apb-df743b94-6f60-4153-b211-1e0c70bac227.yaml
[2017-10-19T12:40:04.679Z] [DEBUG] service_id: 6f418d63aefb3f622d58117b2cbba90c
[2017-10-19T12:40:04.679Z] [DEBUG] plan_id: 5a6570a738ee3b996901f9d8a91cae2f
[2017-10-19T12:40:04.679Z] [DEBUG] operation:  4a323bf4-aaaf-4d4f-80a7-721c5bc19f93
<IP_ADDRESS> - - [19/Oct/2017:12:40:04 +0000] "GET /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37/last_operation?operation=4a323bf4-aaaf-4d4f-80a7-721c5bc19f93&plan_id=5a6570a738ee3b996901f9d8a91cae2f&service_id=6f418d63aefb3f622d58117b2cbba90c HTTP/1.1" 200 29
[2017-10-19T12:40:04.793Z] [ERROR] Something went wrong trying to create resources in cluster
[2017-10-19T12:40:04.793Z] [ERROR] Returned error:
[2017-10-19T12:40:04.793Z] [ERROR] exit status 1
[2017-10-19T12:40:04.793Z] [ERROR] oc create -f output:
[2017-10-19T12:40:04.793Z] [ERROR] serviceaccount "apb-df743b94-6f60-4153-b211-1e0c70bac227" created
rolebinding "apb-df743b94-6f60-4153-b211-1e0c70bac227" created
Error from server (AlreadyExists): error when creating "/tmp/asb-resource-files/apb-df743b94-6f60-4153-b211-1e0c70bac227.yaml": rolebindings "apb-df743b94-6f60-4153-b211-1e0c70bac227" already exists

[2017-10-19T12:40:04.793Z] [INFO] Successfully created apb sandbox: [ apb-df743b94-6f60-4153-b211-1e0c70bac227 ], with edit permissions in namespace aws-service-broker
[2017-10-19T12:40:04.793Z] [NOTICE] Creating pod "apb-df743b94-6f60-4153-b211-1e0c70bac227" in the aws-service-broker namespace
[2017-10-19T12:40:04.8Z] [DEBUG] Calling monitorOutput on apb-df743b94-6f60-4153-b211-1e0c70bac227
[2017-10-19T12:40:05.003Z] [INFO] error: unable to upgrade connection: container not found ("apb")

[2017-10-19T12:40:05.003Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 1: Failed to exec into the container
[2017-10-19T12:40:05.003Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 1: exec into apb-df743b94-6f60-4153-b211-1e0c70bac227 failed
[2017-10-19T12:40:05.474Z] [DEBUG] service_id: 6f418d63aefb3f622d58117b2cbba90c
[2017-10-19T12:40:05.474Z] [DEBUG] plan_id: 5a6570a738ee3b996901f9d8a91cae2f
[2017-10-19T12:40:05.474Z] [DEBUG] operation:  4a323bf4-aaaf-4d4f-80a7-721c5bc19f93
<IP_ADDRESS> - - [19/Oct/2017:12:40:05 +0000] "GET /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37/last_operation?operation=4a323bf4-aaaf-4d4f-80a7-721c5bc19f93&plan_id=5a6570a738ee3b996901f9d8a91cae2f&service_id=6f418d63aefb3f622d58117b2cbba90c HTTP/1.1" 200 29
[2017-10-19T12:40:09.488Z] [DEBUG] service_id: 6f418d63aefb3f622d58117b2cbba90c
[2017-10-19T12:40:09.488Z] [DEBUG] plan_id: 5a6570a738ee3b996901f9d8a91cae2f
[2017-10-19T12:40:09.488Z] [DEBUG] operation:  4a323bf4-aaaf-4d4f-80a7-721c5bc19f93
<IP_ADDRESS> - - [19/Oct/2017:12:40:09 +0000] "GET /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37/last_operation?operation=4a323bf4-aaaf-4d4f-80a7-721c5bc19f93&plan_id=5a6570a738ee3b996901f9d8a91cae2f&service_id=6f418d63aefb3f622d58117b2cbba90c HTTP/1.1" 200 29
[2017-10-19T12:40:10.21Z] [INFO] error: unable to upgrade connection: container not found ("apb")

[2017-10-19T12:40:10.21Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 2: Failed to exec into the container
[2017-10-19T12:40:10.21Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 2: exec into apb-df743b94-6f60-4153-b211-1e0c70bac227 failed
[2017-10-19T12:40:15.714Z] [INFO] Bind credentials not available.
command terminated with exit code 2

[2017-10-19T12:40:15.714Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 3: Bind credentials not available yet
[2017-10-19T12:40:15.714Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 3: exec into apb-df743b94-6f60-4153-b211-1e0c70bac227 failed
[2017-10-19T12:40:17.501Z] [DEBUG] service_id: 6f418d63aefb3f622d58117b2cbba90c
[2017-10-19T12:40:17.501Z] [DEBUG] plan_id: 5a6570a738ee3b996901f9d8a91cae2f
[2017-10-19T12:40:17.501Z] [DEBUG] operation:  4a323bf4-aaaf-4d4f-80a7-721c5bc19f93
<IP_ADDRESS> - - [19/Oct/2017:12:40:17 +0000] "GET /aws-service-broker/v2/service_instances/3ec651a4-52e6-4eac-a20a-11813d5e6e37/last_operation?operation=4a323bf4-aaaf-4d4f-80a7-721c5bc19f93&plan_id=5a6570a738ee3b996901f9d8a91cae2f&service_id=6f418d63aefb3f622d58117b2cbba90c HTTP/1.1" 200 29
[2017-10-19T12:40:21.12Z] [INFO] Bind credentials not available.
command terminated with exit code 2

[2017-10-19T12:40:21.12Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 4: Bind credentials not available yet
[2017-10-19T12:40:21.12Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 4: exec into apb-df743b94-6f60-4153-b211-1e0c70bac227 failed
[2017-10-19T12:40:26.899Z] [INFO] error: unable to upgrade connection: container not found ("apb")

[2017-10-19T12:40:26.899Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 5: Failed to exec into the container
[2017-10-19T12:40:26.899Z] [WARNING] [apb-df743b94-6f60-4153-b211-1e0c70bac227] Retry attempt 5: exec into apb-df743b94-6f60-4153-b211-1e0c70bac227 failed
[2017-10-19T12:40:32.078Z] [NOTICE] [apb-df743b94-6f60-4153-b211-1e0c70bac227] APB completed
[2017-10-19T12:40:32.078Z] [INFO] Destroying APB sandbox...
[2017-10-19T12:40:32.079Z] [DEBUG] Deleting rolebinding apb-df743b94-6f60-4153-b211-1e0c70bac227, namespace aws-service-broker
[2017-10-19T12:40:32.272Z] [DEBUG] Successfully deleted rolebinding apb-df743b94-6f60-4153-b211-1e0c70bac227, namespace aws-service-broker
[2017-10-19T12:40:32.272Z] [DEBUG] oc delete output:
[2017-10-19T12:40:32.272Z] [DEBUG] rolebinding "apb-df743b94-6f60-4153-b211-1e0c70bac227" deleted

this is identical to the one that @djzager filed. https://bugzilla.redhat.com/show_bug.cgi?id=1504927

Once PR posts this bug will appear in the subsequent build.


Update config to include Google Maps API key needed for Overviewer

Hey guys,
Long time no see! Hope you all are doing well. I had some spare time so I figured I might as well push some new changes to MSCS.
Google now requires a (free) API key to access their Maps API. Because of this, the Overviewer team have been working on changing Overviewer over to leaflet. For now, though Overviewer is sticking with Google Maps. As such, Overviewer now requires an API key, too. So, I went ahead and added an overviewer-key property and some other lines of code to integrate MSCS and Overviewer smoothly.
The main line of code uses a sed command. I've adopted this sed line of code from here. You can also read more about the issue at that link.
I've also added more documentation to the mapping section, and tried to clarify writing as I saw fit.
I've been using this configuration with my server and its worked like a charm, though please check my code as my programming skills are mediocre lol.
Thanks!
-Mike

So, after some more testing and reading other people's comments, it look's like the fix I used above can create issues because it doesn't target the actual source of overviewer overwriting the index.html file.
(Zeromusta created a script here that address the issue and seems to be working on my server.)(https://github.com/Zeromusta/docker-spigot-overviewer/blob/master/rootfs/usr/local/bin/ov_keyfix) I'm not sure if you would rather switch over to the new leaflet overviewer library (which is still in development, and unstable) or try figuring out a patch with the Google Maps API involving a fix like the one above.

This looks like a good update. Thanks for the work.  I've left a few comments above.
RE leaflet: I'm open to whatever you guys think is best here.  I'm too busy to even give this any thought, so I'll defer to your wisdom.

Thanks @sandain. I'm going to wait a little on deciding what to do and see if they're coming out with any updates anytime soon

I like the addition of the overviewer-key parameter, it seems like a good stop-gap until they get leaflet integrated.  This part of your patch can always be reverted in the future when it is no longer needed.

@sandain Okay, let me just add the part to ensure the Overviewer source files get overwritten

What should we do about this pull request?  I haven't even looked at it in a while, but it looks like we never resolved some issues. Overviewer still hasn't switched away from the Google API, but they seem to be close.  Are there any changes in here that should be merged to master regardless of which API Overviewer is using?

No, once they switch over to the other maps API no API keys will be required. The simplest solution would just be to wait it out; perhaps we could put a note in the documentation linking to this issue if anyone is curious and wants to use Overviewer before they update it.

I like some of the documentation updates and auto creating a commented out mscs.defaults file. As for the API keys, I agree with @Roflicide, we could add some more documentation about the issue until it's resolved.

I created pull request #166 to add the commented out mscs.defaults file. Let me know what you think in that pull request.
I'll need to look closer at this pull request to incorporate the changes to the documentation when I get another free moment. That is unless someone else has the time to create a new pull request with just that done.


Update JsonTokenType.java

Typo on "openning" -> "opening"

Did you want to create this PR against the real repo apache/groovy or would you like me to make the change.


Mosaic Effective height

@NemTechCN which is the difference between #61 and #70? Also, could you do something similar for namespace expiration?


Don't puts into tests

Don't puts into tests

@kbrock @Fryguy @matthewd the stdout messages are gone

@matthewd you ok with this? Using allow has the nice effect of undoing the stubs for us at the end.


Add localization support for privacy banner + useCustomerPrivacy updates

https://github.com/user-attachments/assets/6a7c9d6b-eb9c-4b00-ae64-311ea6522fdb
Included on this PR:
1. Adds consent banner localization support to the AnalyticsProvider and useCustomerPrivacy
To enable it simply pass country and language to the consent config returned from the loader
2. Reworks the previously fragile customerPrivacy and privacyBanner onReady logic removing a not ideal 3s setTimeout and relying on Object.defineProperty on the window instead to detect when these SDKs are actually available.
Banner will show when ready instead of after 3 secs.
3. Deprecates a getCustomerPrivacy utility
Now that we have privacyBanner and customerPrivacy returned from both useAnalytics and useCustomerPrivacy we don't need getCustomerPrivacy
4. Adds privacyBanner and customerPrivacy to the useAnalytics and useCustomerPrivacy returns, so users can now use any of these SDK methods programmatically
const {customerPrivacy, privacyBanner} = useAnalytics() // or useCustomerPrivacy(..)

if (privacyBanner) {
  // load the consent banner programmatically
  privacyBanner.loadBanner()
  // or
  privacyBanner.loadBanner({ country: "CA", locale: "FR" })

  // load preferences banner programmatically
  privacyBanner.showPreferences();
  // or
  privacyBanner.showPreferences({ country: "CA", locale: "FR" })
}

if (customerPrivacy) {
 const analyticsProcessingAllowed = customerPrivacy.analyticsProcessingAllowed()
 // ....
}

Used in the demo videos header buttons.
5. Fixed an issue where tracking was temporarily allowed for a few milliseconds while the consent check was being performed by the customerPrivacy SDK
Tophat instructions:

Return difference country and language combinations from the root loader consent.
test the different methods of privacyBanner (showBanner, showPreferences) and customerPrivacy (analyticsProcessingAllowed, setTrackingConsent) returned by useAnalytics and/or useCustomerPrivacy

TODO

[x] Update existing tests and add new tests for useCustomerPrivacy
[x] Update docs
[ ] Test snapshot in a 3P consent environment

snapit

/snapit

/snapit


0.5.12 release

Codecov Report

Merging #46 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   95.62%   95.69%   +0.07%
==========================================
  Files          35       35
  Lines        5962     6023      +61
==========================================
+ Hits         5701     5764      +63
+ Misses        261      259       -2

Impacted Files
Coverage Δ

BezierKit/Library/PathComponent.swift
95.01% <ø> (+0.44%)
:arrow_up:

BezierKit/BezierKitTests/UtilsTests.swift
100% <100%> (ø)
:arrow_up:

BezierKit/BezierKitTests/PathTests.swift
98.13% <100%> (+0.08%)
:arrow_up:

BezierKit/Library/Utils.swift
97.29% <100%> (+0.03%)
: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 4d39757...4b3b5ec. Read the comment docs.


Security vulnerabilities in heyzap plugins reported by Google Play

I've uploaded my APK with the following plugins included:
heyzap-cordova 1.2.1 "Heyzap"
heyzap-cordova-admob 1.0.1
heyzap-cordova-applovin 1.0.3
heyzap-cordova-facebook_audience_network 1.0.2

and the Google Play reports security vulnerabilities in my app:
Sending Advertising ID Detected in APK 10000
Your app is sending Advertising ID over an unencrypted connection.

This isn't recommended because network intermediaries might be able to intercept it. You should use an encrypted connection when sending Advertising ID.

I suppose the same issue will come up on iOS. How to make the Heyzap plugins use secure channel for sending Advertising ID?

The extra packages doesn't change anything. It's enough to only use the heyzap-cordova to get the security warning in Google Play.  Any help?

We are working on it for the next SDK release.

HEYZAP! Do something about that or make the sources open so that I could fix this on my own!
Sending Advertising ID Detected in APK 11000, 10400, 10301, 10300
Your app is sending Advertising ID over an unencrypted connection.

This isn't recommended because network intermediaries might be able to intercept it. You should use an encrypted connection when sending Advertising ID.

Fixed?

Yes, this was resolved in our Android SDK a while back, you shouldn't experience this issue anymore.


feat: open source contributions documentation

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Contributor Checklist:

[ ] My code follows the code style of this project.
[ ] I have added an example of my new feature and included it in the documentation.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My Pull Request name follows the naming convention fix: <description>, feat: <description>, etc.

Reviewer Checklist for Charts compliant functions:

[ ] The docstrings of the new function follow the contributing guidelines.
[ ] The new function is professionally documented
[ ] The new function and associated scripts are covered by one or more unit tests and code coverage did not decrease.
[ ] The new function is accompanied by an example and it is included in the Gallery of Charts.
[ ] The new function is reviewed in Chromatic. Access the storybook build results url and comment, approve or deny.
[ ] All function inputs, arguments, and outputs have a supported data type and have human readable names.
[ ] No code language is included in the description of the function or parameters (e.g use "polynomial order" instead of "poly_order")

@VebjornG could you fix all the places in the documentation where it says that repository should be cloned? We need to make sure that it should be done by forking repo for open source contributions, but Cognite people could clone it directly. Could you figure out how to phrase those sentences? Maybe with links for the open source contributions?

@VebjornG could you fix all the places in the documentation where it says that repository should be cloned? We need to make sure that it should be done by forking repo for open source contributions, but Cognite people could clone it directly. Could you figure out how to phrase those sentences? Maybe with links for the open source contributions?

Made a suggestion in the latest commit 👍🏻


TO 2.0 API - implement API response codes

Creating this issue as a placeholder and to facilitate discussion regarding API response codes. This seems like a pretty good set of response codes:
200 - OK    The request has succeeded.
201 - Created   The request successfully added a resource. This response code should be expected for POST operations which create a resource.
202 - Accepted  The request was accepted, but processing is not complete. This response code is most common for POSTs or PUTs which cannot be handled synchronously.
204 - No Content The server has successfully fulfilled the request and that there is no additional content to send in the response payload body.
400 - Bad Request   The request was invalid. This response code is common when required fields are unspecified, formatted incorrectly, or invalid filters are requested.
401 - Unauthorized  The request authentication failed. The credentials that the client supplied were missing or invalid.
403 - Forbidden The request credentials authenticated, but the requesting user is not authorized to access the given resource.
404 - Not Found The requested resource does not exist.
405 - Method Not Allowed    The requested HTTP method is invalid for the given resource. Review the resource documentation for supported methods.
500 - Server Error  The server failed to fulfill the request. Please notify support with details of the request and response so that we can fix the problem.

this has been added to the api guidelines doc found in contributing.md


deprecate forc-client

closes #2608
binaries are released and the crate is now made available through https://github.com/FuelLabs/forc-client

this should go in alongside https://github.com/FuelLabs/forc-client/pull/20


Disallow unknown keys in feature-flags configmap

Currently, we don't perform any validation of the feature flags configmap. https://github.com/tektoncd/pipeline/pull/6935 adds validation for keys representing existing feature flags, but it's still possible to misspell a feature flag or add a flag that's no longer supported, thinking it has an effect (example: https://github.com/tektoncd/pipeline/pull/6937). We should validate that the configmap doesn't contain any unknown keys. We will probably need to fix https://github.com/tektoncd/pipeline/issues/4542 first, otherwise upgrades will cause problems if we remove feature flags.

I would like to work on this.

AFAIK, I should cover this: pkg/apis/config/feature_flags.go, but should I cover this also pkg/apis/config/resolver/feature_flags.go?

/assign @minhoryang

Dear @lbernick, could you please take a look at my PR #7208 when you have some time?


Game page issues

The description needs to add. Currently, there is no good description. Please update the description correctly.
The game played, like, comments didn't work. please update those.

Yep.. I can put the description there. but the Content creating team wants to give the description content to that. after I'll update

Just search the game name and get the description don't wait until they give the content.


Saving HTML on test failure

Similar to taking screenshots on test failure, I think that saving the HTML, with some replacements, seems to be useful.
This can be basically achieved by:
open("/tmp/t.html", "w").write(browser.html)

And the replacing "/static/ with "http://localhost:8000/static/.
What do you think?
Would that be useful to ship (and possibly enable) by default?

nice idea!
screenshots are not copy-pasteable, but this will be rendered natively, for example in jenkins it will be no problem to set up via plugins
that will be really cool
i think we can use native feature of saving a page offline from the browser, but didn't try that yet
do you mind of working on this?

implemented in 1.6.0 :)


Added a page for documenting beginner issues

This PR aims to add a section of the documentation of beginner issues that users might make while they are new to thanos / prometheus / golang development.

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Added a new file docs/operating/beginner_issues.md
Verification
Not required :stuck_out_tongue:

What do you think about this categorization of docs: beginner vs other. I feel like it's hard to say which one is beginner. Anyone can be affected no matter of experience, It might save time for everyone. I would rather categorize by component thinking

Agreed. Also "beginner" could be taken as an insult. I think the way it is now is good. I wouldn't focus on "componentizing" too much as I'm sure there will be a number of problems we are going to describe that will be because of the interaction of multiple components.


Update statements: operations on field based on previous value

Hi!
Having a table like this
class CiccioPasticcio extends Table {
  IntColumn get id => integer().autoIncrement()();

  IntColumn get value => integer()();
}

Is there a way of writing a query like this:
UPDATE ciccio_pasticcio SET value = value + 1
in pure dart without using the moor files?
Thanks

Done, this will ship with moor 3.0. To write UPDATE ciccio_pasticcio SET value = value + 1 , use
final incrementValue = CiccioPasticcioCompanion.custom(value: value + Constant(1));
await update(ciccioPasticcio).write(incrementValue);

CiccioPasticcioCompanion.custom(value: value + Constant(1));

Can you add it to docs and update value + Constant(1) part - by default it's not valid code
CiccioPasticcioCompanion.custom(value: Dao.ciccioPasticcioTable.value + Constant(1));

This should work. Is ciccioPasticcioTable.value an IntColumn?


Change Color

Hi how can i change the Backround color to white and the text color to black?

This would be fantastic!


Guard against multiple buildscript blocks in a single script

See failing test case. Part of #72.

This doesn't seem to match the behaviour of the Groovy frontend which is to consider all buildscript blocks in the first step.

I just did a very quick test on the Groovy side, and I thought it was discarding all but the first buildscript { }. If that's not the case, then we should adjust this, including the test case.

Check this one out, it will only compile if you uncomment the dependencies block and at least one of the repositories blocks:
buildscript {
    //repositories { mavenCentral() }
}

buildscript {
    //repositories { jcenter() }
}

buildscript {
    //dependencies { classpath("junit:junit:4.12") }
}

org.junit.Assert.assertTrue(true)

task dumpRepositories {
    doLast {
        buildscript.repositories.each {
            println(it.url)
        }
    }
}

I think only allowing one buildscript block at the very top of the build file is a good idea. It's a very slight departure from Groovy, but if we were to redesign it, that's what we would do. We have done the same for the plugins block already.

Resolved in 588ab800a0aae6c379e258938ef28af605295fb7.


Fix behavior for cache_control=True with expire_after, and support revalidation with max-age=0

Closes #429, closes #428

Codecov Report

Merging #430 (3814a7b) into master (b3a85bd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #430   +/-   ##
=======================================
  Coverage   98.72%   98.72%
=======================================
  Files          17       17
  Lines        1176     1181    +5
  Branches      144      144
=======================================
+ Hits         1161     1166    +5
  Misses          5        5
  Partials       10       10

Impacted Files
Coverage Δ

requests_cache/cache_control.py
100.00% <100.00%> (ø)

requests_cache/session.py
99.10% <100.00%> (ø)

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 b3a85bd...3814a7b. Read the comment docs.


Setting MaxUnavailable to 1 when calculation against # of available calico node pods resolves to 0

Description

Due to the potential rounding down of numNodesMaxUnavailable = (csMaxUnavailable %) * ksD+csD, where maxUnavailable is a percentage value, ,it may resolve to zero. Then we should default back maxUnavailable to 1 on the theory that surge might not work due to quota.
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

I'm sorry but I failed delete the branch after merging the pull request.


RuntimeError: The NVIDIA driver on your system is too old

Followed the install guide with GPU: https://github.com/flatironinstitute/deepblast/wiki/Installation
Here is my steps:
conda create -n tmvec python=3.9
pip3 install torch torchvision torchaudio --extra-index-url https://download.pytorch.org/whl/cu117
pip3 install faiss-gpu
pip3 install tm-vec
after run tmvec-build-database got this error:

my GPU driver info:

Don't know what went wrong.

I think the issue is that you should use an older version of the cuda toolkit in your conda environment. While the driver is right, modern toolkits won't work with the older drivers. I think the command
conda install cuda=11.7 should work, though I don't have a conda environment available.

thanks @blackwer , with your method I got this error.
ERROR: Could not find a version that satisfies the requirement cuda==11.7 (from versions: none)
ERROR: No matching distribution found for cuda==11.7
But the idea is right, and finally I solved it this way.
pip3 install torch==1.13.1+cu117 torchvision==0.14.1+cu117 torchaudio==0.13.1 --extra-index-url https://download.pytorch.org/whl/cu117

Great! Got that you got this resolved


Updated office synchronization code to respect state code

👍 I am working with Anisha to train her and I forgot to have her set up a branch before doing a pull request. My bad!


Compiling errors

Hello. Great tutorial!
I am not able to compile it on Ubuntu 16.04. I am getting a lot of unknown  variable-type errors, probably due to different versions of libavcodec. I have tried downloading it form Ubuntu repositories and from jonathonf's ppa. Neither of witch had a successful compiling. I'm going to attach the error messages:
0_hello_world.c: In function ‘main’:
0_hello_world.c:84:3: error: unknown type name ‘AVCodecParameters’
   AVCodecParameters *pCodecParameters =  NULL;
   ^
0_hello_world.c:90:5: error: unknown type name ‘AVCodecParameters’
     AVCodecParameters *pLocalCodecParameters =  NULL;
     ^
0_hello_world.c:91:55: error: ‘AVStream {aka struct AVStream}’ has no member named ‘codecpar’
     pLocalCodecParameters = pFormatContext->streams[i]->codecpar;
                                                       ^
0_hello_world.c:103:61: error: request for member ‘codec_id’ in something not a structure or union
     pLocalCodec = avcodec_find_decoder(pLocalCodecParameters->codec_id);
                                                             ^
0_hello_world.c:111:30: error: request for member ‘codec_type’ in something not a structure or union
     if (pLocalCodecParameters->codec_type == AVMEDIA_TYPE_VIDEO) {
                              ^
0_hello_world.c:116:71: error: request for member ‘width’ in something not a structure or union
       logging("Video Codec: resolution %d x %d", pLocalCodecParameters->width, pLocalCodecParameters->height);
                                                                       ^
0_hello_world.c:116:101: error: request for member ‘height’ in something not a structure or union
       logging("Video Codec: resolution %d x %d", pLocalCodecParameters->width, pLocalCodecParameters->height);
                                                                                ^
0_hello_world.c:117:37: error: request for member ‘codec_type’ in something not a structure or union
     } else if (pLocalCodecParameters->codec_type == AVMEDIA_TYPE_AUDIO) {
                                     ^
0_hello_world.c:118:80: error: request for member ‘channels’ in something not a structure or union
       logging("Audio Codec: %d channels, sample rate %d", pLocalCodecParameters->channels, pLocalCodecParameters->sample_rate);
                                                                                ^
0_hello_world.c:118:113: error: request for member ‘sample_rate’ in something not a structure or union
       logging("Audio Codec: %d channels, sample rate %d", pLocalCodecParameters->channels, pLocalCodecParameters->sample_rate);
                                                                                ^
0_hello_world.c:122:104: error: request for member ‘bit_rate’ in something not a structure or union
     logging("\tCodec %s ID %d bit_rate %lld", pLocalCodec->long_name, pLocalCodec->id, pCodecParameters->bit_rate);
                                                                                ^
0_hello_world.c:134:7: warning: implicit declaration of function ‘avcodec_parameters_to_context’ [-Wimplicit-function-declaration]
   if (avcodec_parameters_to_context(pCodecContext, pCodecParameters) < 0)
       ^
0_hello_world.c:156:23: warning: implicit declaration of function ‘av_packet_alloc’ [-Wimplicit-function-declaration]
   AVPacket *pPacket = av_packet_alloc();
                       ^
0_hello_world.c:156:23: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
0_hello_world.c:187:3: warning: implicit declaration of function ‘av_packet_free’ [-Wimplicit-function-declaration]
   av_packet_free(&pPacket);
   ^
0_hello_world.c: In function ‘decode_packet’:
0_hello_world.c:207:18: warning: implicit declaration of function ‘avcodec_send_packet’ [-Wimplicit-function-declaration]
   int response = avcodec_send_packet(pCodecContext, pPacket);
                  ^
0_hello_world.c:218:16: warning: implicit declaration of function ‘avcodec_receive_frame’ [-Wimplicit-function-declaration]
     response = avcodec_receive_frame(pCodecContext, pFrame);
                ^
Makefile:4: recipe for target 'hello_world' failed
make: *** [hello_world] Error 1

Thank you in advanced
Oier

Hi @oierlauzi I think you can compile it from source :) it's easy https://trac.ffmpeg.org/wiki/CompilationGuide/Ubuntu

Please let's know what trouble you're facing during this process. 😄


Fixed all_records_for_product so it returns something

ProductsHelper::all_records_for_product now it uses the non-destructive versions of sort_by and uniq, so it actually returns something, rather than just modifying the records object in place

Don't merge this just yet; I'm writing a test for this method.


Cannot generates templates into a folder that doesn't exist

When running the GenerateTemplatesSQLServer tool, if a destination is selected that doesn't exist (e.g., create a new project, then try to generate templates into a Templates directory that hasn't been created yet) then we get an error saying that the path can't be found:
PS C:\Git\TestProject\DBTests> GenerateTemplatesSQLServer -d "TestDB" -t "*" -n "DBTests.Templates" --destination Templates
Using connection: SERVER=(local);DATABASE=TestDB;Integrated Security=true;Connection Timeout=30;
Found table (dbo.User) and 9 columns
Class definition generated.  Saving...
There has been an error
Could not find a part of the path 'C:\Git\TestProject\DBTests\Templates\UserTemplate.cs'.
   at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
   at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at System.IO.StreamWriter.ValidateArgsAndOpenPath(String path, Boolean append, Encoding encoding, Int32 bufferSize)
   at System.IO.StreamWriter..ctor(String path)
   at System.IO.File.WriteAllText(String path, String contents)
   at DBConfirm.TemplateGeneration.Logic.FileHelper.WriteAllText(String path, String contents) in /home/vsts/work/1/s/tools/TemplateGeneration.SQLServer/Logic/FileHelper.cs:line 10
   at DBConfirm.TemplateGeneration.Logic.Generator.GenerateFile(IEnumerable`1 processedColumns, String schemaName, String tableName) in /home/vsts/work/1/s/tools/TemplateGeneration.SQLServer/Logic/Generator.cs:line 112
   at DBConfirm.TemplateGeneration.Logic.Generator.GenerateFileAsync() in /home/vsts/work/1/s/tools/TemplateGeneration.SQLServer/Logic/Generator.cs:line 61
   at DBConfirm.TemplateGeneration.Program.<>c.<<Main>b__0_0>d.MoveNext() in /home/vsts/work/1/s/tools/TemplateGeneration.SQLServer/Program.cs:line 18

The tool should try and create the directory if it doesn't already exist.

This is included in 0.1.3


Passing arguments to middleware

Hi, thanks for this awesome library.
I just try to passing an argument to the Middleware, like for checking the user role.
The image below as the example from Laravel.

How can I do that?

Hi @ngekoding!
Unfortunately there is no support for middleware parameters in 1.x version of Luthier CI. This could be (and will be) an interesting addition for 2.0 version and I'll consider it.

Thank you, hopefully version 2.0 is coming soon.


Add some basic UI layout

Just a note that the Netlify preview seems to have failed for this, but it could just be something wrong with Netlify/the setup there.


Cardinal::image can not generate correct figure in 3.6.2 version

I run the code exactly the same, however, when I use the latest version of Cardinal from github, an error occurs (the generated image is problematic), while the Bioconductor version does not.
objects <- lapply(files, Cardinal::readMSIData)
mse <- do.call(c, objects)

mse <- Cardinal::normalize(mse) |>
  Cardinal::smooth() |>
  Cardinal::reduceBaseline() |>
  Cardinal::peakPick() |>
  Cardinal::peakAlign()

Cardinal::image(mse)

Bioconductor version (3.6.1):
test.pdf
Github latest version (3.6.2) (issue):
test2.pdf

Please always include full output from sessionInfo().

R version 4.4.0 (2024-04-24)
Platform: x86_64-pc-linux-gnu
Running under: Pop!_OS 22.04 LTS
Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.10.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.10.0
locale:
[1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
[5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C
[9] LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
time zone: Asia/Shanghai
tzcode source: system (glibc)
attached base packages:
[1] grid      stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] nvimcom_0.9-146       utils.tool_<IP_ADDRESS>00
loaded via a namespace (and not attached):
[1] tidyselect_1.2.0    viridisLite_0.4.2   EBImage_4.46.0      dplyr_1.1.2         farver_2.1.1        viridis_0.6.4
[7] bitops_1.0-7        ggraph_2.1.0        fastmap_1.2.0       RCurl_1.98-1.14     tweenr_2.0.2        XML_3.99-0.14
[13] digest_0.6.35       lifecycle_1.0.4     ProtGenerics_1.36.0 rsvg_2.4.0          magrittr_2.0.3      compiler_4.4.0
[19] rlang_1.1.4         tools_4.4.0         igraph_1.5.1        utf8_1.2.3          data.table_1.14.8   askpass_1.1
[25] graphlayouts_1.0.0  htmlwidgets_1.6.4   ontologyIndex_2.12  xml2_1.3.5          pkgload_<IP_ADDRESS>     Cardinal_3.6.1
[31] abind_1.4-5         BiocParallel_1.38.0 withr_2.5.0         purrr_1.0.2         BiocGenerics_0.50.0 desc_1.4.2
[37] polyclip_1.10-4     stats4_4.4.0        fansi_1.0.4         colorspace_2.1-0    ggplot2_3.5.1       scales_1.3.0
[43] MASS_7.3-60.0.1     cli_3.6.2           crayon_1.5.2        ragg_1.2.5          generics_0.1.3      matter_2.7.1
[49] DBI_1.2.3           pbapply_1.7-2       ggforce_0.4.1       stringr_1.5.0       parallel_4.4.0      BiocManager_1.30.23
[55] tiff_0.1-12         vctrs_0.6.3         Matrix_1.7-0        CardinalIO_1.2.1    fftwtools_0.9-11    S4Vectors_0.42.0
[61] ggrepel_0.9.3       irlba_<IP_ADDRESS>       systemfonts_1.0.4   jpeg_0.1-10         locfit_1.5-9.9      tidyr_1.3.0
[67] glue_1.7.0          codetools_0.2-20    stringi_1.7.12      gtable_0.3.3        munsell_0.5.0       tibble_3.2.1
[73] pillar_1.9.0        htmltools_<IP_ADDRESS>   openssl_2.1.0       R6_2.5.1            textshaping_0.3.6   tidygraph_1.2.3
[79] rprojroot_2.0.3     lattice_0.22-6      Biobase_2.64.0      png_0.1-8           ggsci_3.0.0         Rcpp_1.0.12
[85] zip_2.3.0           uuid_1.1-0          gridExtra_2.3       nlme_3.1-164        officer_0.6.2       pkgconfig_2.0.3

@shaman-yellow That is output from 3.6.1, can you provide the sessionInfo() output from when running 3.6.2?

I re-installed the latest version of package and run the codes.
R version 4.4.0 (2024-04-24)
Platform: x86_64-pc-linux-gnu
Running under: Pop!_OS 22.04 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.10.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.10.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

time zone: Asia/Shanghai
tzcode source: system (glibc)

attached base packages:
[1] grid      stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] nvimcom_0.9-146       utils.tool_<IP_ADDRESS>00

loaded via a namespace (and not attached):
 [1] tidyselect_1.2.0    viridisLite_0.4.2   EBImage_4.46.0      dplyr_1.1.2         farver_2.1.1        viridis_0.6.4
 [7] bitops_1.0-7        ggraph_2.1.0        fastmap_1.2.0       RCurl_1.98-1.14     tweenr_2.0.2        XML_3.99-0.14
[13] digest_0.6.35       lifecycle_1.0.4     ProtGenerics_1.36.0 rsvg_2.4.0          magrittr_2.0.3      compiler_4.4.0
[19] rlang_1.1.4         tools_4.4.0         igraph_1.5.1        utf8_1.2.3          data.table_1.14.8   askpass_1.1
[25] graphlayouts_1.0.0  htmlwidgets_1.6.4   ontologyIndex_2.12  xml2_1.3.5          pkgload_<IP_ADDRESS>     Cardinal_3.7.0
[31] abind_1.4-5         BiocParallel_1.38.0 withr_2.5.0         purrr_1.0.2         BiocGenerics_0.50.0 desc_1.4.2
[37] polyclip_1.10-4     stats4_4.4.0        fansi_1.0.4         colorspace_2.1-0    ggplot2_3.5.1       scales_1.3.0
[43] MASS_7.3-60.0.1     cli_3.6.2           crayon_1.5.2        ragg_1.2.5          generics_0.1.3      matter_2.7.1
[49] DBI_1.2.3           pbapply_1.7-2       ggforce_0.4.1       stringr_1.5.0       parallel_4.4.0      tiff_0.1-12
[55] vctrs_0.6.3         Matrix_1.7-0        CardinalIO_1.2.1    fftwtools_0.9-11    S4Vectors_0.42.0    ggrepel_0.9.3
[61] irlba_<IP_ADDRESS>       systemfonts_1.0.4   jpeg_0.1-10         locfit_1.5-9.9      tidyr_1.3.0         glue_1.7.0
[67] codetools_0.2-20    stringi_1.7.12      gtable_0.3.3        munsell_0.5.0       tibble_3.2.1        pillar_1.9.0
[73] htmltools_<IP_ADDRESS>   openssl_2.1.0       R6_2.5.1            textshaping_0.3.6   tidygraph_1.2.3     rprojroot_2.0.3
[79] lattice_0.22-6      Biobase_2.64.0      png_0.1-8           ggsci_3.0.0         Rcpp_1.0.12         zip_2.3.0
[85] uuid_1.1-0          gridExtra_2.3       nlme_3.1-164        officer_0.6.2       pkgconfig_2.0.3

It's version 3.7 now, but the problem still exists.

@shaman-yellow You are on 3.7.0 which is the current unstable development version.
3.6.2 is available on the RELEASE_3_19 branch, or you can wait for it to build on the Bioconductor servers. I can't replicate the issue on 3.6.2.

This issue does occur after version 3.6.2. I can only use 3.6.1 now. But 3.6.1 has the issue when using Cardinal::spatialDGMM:
https://github.com/kuwisdelu/Cardinal/issues/39

@shaman-yellow I can replicate the issue in 3.7.0 which is unstable and the plotting methods are currently under active development and expected to be buggy.
Did you install 3.6.2 from the RELEASE_3_19 branch? 3.6.2 is stable and I can't replicate the issue there.

Thanks, I installed the version you mentioned and now the issue is solved.

Likewise for matter, which I notice you are on 2.7.1 which is also an unstable development version.
As described in the README, the devel branches on GitHub should not be used unless you are developer and are prepared to tolerate (and potentially help fix) bugs and instability. They should not be relied on for reproducible data analysis.
Either use Bioconductor or the RELEASE branches. Thanks.

I use the devel version because I found the issue https://github.com/kuwisdelu/Cardinal/issues/39, which just happened two days ago.

Urgent bugfixes will be on the release branch and later merged into devel.
The devel version includes ongoing development that is not ready for stable use.

OK, thank you for your patience.


Update tdr-graphql-client to 0.0.169 in scala-steward-dependencies

About this PR
📦 Updates uk.gov.nationalarchives:tdr-graphql-client from 0.0.168 to 0.0.169
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "tdr-graphql-client" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "tdr-graphql-client" }
}]

labels: library-update, early-semver-major, semver-spec-patch, commit-count:1

Superseded by #128.


Wikipedia UI Clone

Fixes
# Issue No. #309

Description
Wikipedia UI clone and ignore that amazon branch by mistake I push in amazon branch
Type of change

[x] New project (non-breaking change which adds functionality)

How Has This Been Tested?

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have included a README.md file
[x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x] I am a GSSoC'22 Participant

Screenshot

Remove unnecessary files, Also add it into your respective folder.

removed and it already added in specified folder

@gurjeetsinghvirdee I guess you forgot to remove the TODO App folder its still showing up changed files

Better I will make new PR


get-collections fails with not a valid key=value pair...

Doing hca dss get-collections fails with the following traceback:
Traceback (most recent call last):
  File "/home/natan/hca/dcp-cli/venv/bin/hca", line 11, in <module>
    sys.exit(main())
  File "/home/natan/hca/dcp-cli/venv/lib/python3.6/site-packages/hca/cli.py", line 136, in main
    result = parsed_args.entry_point(parsed_args)
  File "/home/natan/hca/dcp-cli/venv/lib/python3.6/site-packages/hca/util/__init__.py", line 200, in _cli_call
    return self._consume_response(self._request(vars(cli_args)))
  File "/home/natan/hca/dcp-cli/venv/lib/python3.6/site-packages/hca/util/__init__.py", line 185, in _request
    raise SwaggerAPIException(response=res)
hca.util.exceptions.SwaggerAPIException: None: None (HTTP 403). Details:
{"message":"'eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImtpZCI6Ik1EYzJPVE0zT0RJNE9EWTJOVVU1UkVWR1JEVkVNME15T0VZd1FUa3pORFpEUkRsRVF6TkJSUSJ9.eyJodHRwczovL2F1dGguZGF0YS5odW1hbmNlbGxhdGxhcy5vcmcvZW1haWwiOiJuYWxhb0B1Y3NjLmVkdSIsImh0dHBzOi8vYXV0aC5kYXRhLmh1bWFuY2VsbGF0bGFzLm9yZy9ncm91cCI6ImhjYSIsImlzcyI6Imh0dHBzOi8vaHVtYW5jZWxsYXRsYXMuYXV0aDAuY29tLyIsInN1YiI6Imdvb2dsZS1vYXV0aDJ8MTE3NDI3ODMyNTYxMjczNzM3Mjg2IiwiYXVkIjpbImh0dHBzOi8vZGV2LmRhdGEuaHVtYW5jZWxsYXRsYXMub3JnLyIsImh0dHBzOi8vaHVtYW5jZWxsYXRsYXMuYXV0aDAuY29tL3VzZXJpbmZvIl0sImlhdCI6MTU1ODQ3MDI3OCwiZXhwIjoxNTU4NTU2Njc4LCJhenAiOiJxdE1nTms5ZnFWZWNsTHRabDZXa2JkSjU5ZFAzV2VBdCIsInNjb3BlIjoib3BlbmlkIGVtYWlsIG9mZmxpbmVfYWNjZXNzIn0.I0G2fQ5aAhCvPXgb7ncXtKL3a5F6Tvou6FSghU7s5bDipYzhmZHycp5kU_qHHjTCm6M52JD_2KafF9BLKMwysj4HDtkTu36ww9vqwcRDsJcOVuzWXUJMfMEPjAJADGj1ifsntXJP0NiobNJumLVPEsDiqGhMW_5LsYer7fHfWa7QHg_YBLI0lG5WlptfJH7CZ0KOhRB90Rj9aJ6FdeOMAD3CPuwyRP0qfMemVzXvs3haX2UzIEX5h7chQbAc2SemL6V_IqlJ8izEttrp5nike_7JwS_JPCPhVPjRMAdZX7-4SGned36mjtDg84gSy_El5JjWg15WwJer6604IAgfgA' not a valid key=value pair (missing equal-sign) in Authorization header: 'Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImtpZCI6Ik1EYzJPVE0zT0RJNE9EWTJOVVU1UkVWR1JEVkVNME15T0VZd1FUa3pORFpEUkRsRVF6TkJSUSJ9.eyJodHRwczovL2F1dGguZGF0YS5odW1hbmNlbGxhdGxhcy5vcmcvZW1haWwiOiJuYWxhb0B1Y3NjLmVkdSIsImh0dHBzOi8vYXV0aC5kYXRhLmh1bWFuY2VsbGF0bGFzLm9yZy9ncm91cCI6ImhjYSIsImlzcyI6Imh0dHBzOi8vaHVtYW5jZWxsYXRsYXMuYXV0aDAuY29tLyIsInN1YiI6Imdvb2dsZS1vYXV0aDJ8MTE3NDI3ODMyNTYxMjczNzM3Mjg2IiwiYXVkIjpbImh0dHBzOi8vZGV2LmRhdGEuaHVtYW5jZWxsYXRsYXMub3JnLyIsImh0dHBzOi8vaHVtYW5jZWxsYXRsYXMuYXV0aDAuY29tL3VzZXJpbmZvIl0sImlhdCI6MTU1ODQ3MDI3OCwiZXhwIjoxNTU4NTU2Njc4LCJhenAiOiJxdE1nTms5ZnFWZWNsTHRabDZXa2JkSjU5ZFAzV2VBdCIsInNjb3BlIjoib3BlbmlkIGVtYWlsIG9mZmxpbmVfYWNjZXNzIn0.I0G2fQ5aAhCvPXgb7ncXtKL3a5F6Tvou6FSghU7s5bDipYzhmZHycp5kU_qHHjTCm6M52JD_2KafF9BLKMwysj4HDtkTu36ww9vqwcRDsJcOVuzWXUJMfMEPjAJADGj1ifsntXJP0NiobNJumLVPEsDiqGhMW_5LsYer7fHfWa7QHg_YBLI0lG5WlptfJH7CZ0KOhRB90Rj9aJ6FdeOMAD3CPuwyRP0qfMemVzXvs3haX2UzIEX5h7chQbAc2SemL6V_IqlJ8izEttrp5nike_7JwS_JPCPhVPjRMAdZX7-4SGned36mjtDg84gSy_El5JjWg15WwJer6604IAgfgA'."}

@natanlao I can't reproduce this:
(v3nv) quokka@qcore:~/data-store$ pip install hca
Requirement already satisfied: hca in ./v3nv/lib/python3.6/site-packages (6.1.0)
Requirement already satisfied: awscli>1.15.70 in ./v3nv/lib/python3.6/site-packages (from hca) (1.16.164)
Requirement already satisfied: jsonpointer<2,>=1.10 in ./v3nv/lib/python3.6/site-packages (from hca) (1.14)
Requirement already satisfied: boto3>1.8 in ./v3nv/lib/python3.6/site-packages (from hca) (1.9.154)
Requirement already satisfied: botocore>=1.12.13 in ./v3nv/lib/python3.6/site-packages (from hca) (1.12.154)
Requirement already satisfied: argcomplete<2,>=1.9.3 in ./v3nv/lib/python3.6/site-packages (from hca) (1.10.0)
Requirement already satisfied: cryptography==2.3.1 in ./v3nv/lib/python3.6/site-packages (from hca) (2.3.1)
Requirement already satisfied: puremagic<1.5 in ./v3nv/lib/python3.6/site-packages (from hca) (1.4)
Requirement already satisfied: google-auth-oauthlib<2,>=0.1 in ./v3nv/lib/python3.6/site-packages (from hca) (0.3.0)
Requirement already satisfied: jsonschema<3,>=2.6 in ./v3nv/lib/python3.6/site-packages (from hca) (2.6.0)
Requirement already satisfied: atomicwrites<2,>=1.3.0 in ./v3nv/lib/python3.6/site-packages (from hca) (1.3.0)
Requirement already satisfied: tenacity<5.1,>=5.0.2 in ./v3nv/lib/python3.6/site-packages (from hca) (5.0.4)
Requirement already satisfied: commonmark<1,>=0.9.0 in ./v3nv/lib/python3.6/site-packages (from hca) (0.9.0)
Requirement already satisfied: google-auth<2,>=1.0.2 in ./v3nv/lib/python3.6/site-packages (from hca) (1.6.3)
Requirement already satisfied: PyJWT>=1.6.4 in ./v3nv/lib/python3.6/site-packages (from hca) (1.7.1)
Requirement already satisfied: Jinja2<3,>=2.9 in ./v3nv/lib/python3.6/site-packages (from hca) (2.10.1)
Requirement already satisfied: tweak<2,>=1.0.2 in ./v3nv/lib/python3.6/site-packages (from hca) (1.0.2)
Requirement already satisfied: rsa<=3.5.0,>=3.1.2 in ./v3nv/lib/python3.6/site-packages (from hca) (3.4.2)
Requirement already satisfied: requests<3,>=2.20.0 in ./v3nv/lib/python3.6/site-packages (from hca) (2.22.0)
Requirement already satisfied: six<2,>=1.10 in ./v3nv/lib/python3.6/site-packages (from hca) (1.12.0)
Requirement already satisfied: s3transfer<0.3.0,>=0.2.0 in ./v3nv/lib/python3.6/site-packages (from hca) (0.2.0)
Requirement already satisfied: dcplib<3,>=2.0.2 in ./v3nv/lib/python3.6/site-packages (from hca) (2.0.2)
Requirement already satisfied: docutils<1,>=0.14 in ./v3nv/lib/python3.6/site-packages (from hca) (0.14)
Requirement already satisfied: colorama<=0.3.9,>=0.2.5 in ./v3nv/lib/python3.6/site-packages (from awscli>1.15.70->hca) (0.3.9)
Requirement already satisfied: PyYAML<=3.13,>=3.10 in ./v3nv/lib/python3.6/site-packages (from awscli>1.15.70->hca) (3.13)
Requirement already satisfied: jmespath<1.0.0,>=0.7.1 in ./v3nv/lib/python3.6/site-packages (from boto3>1.8->hca) (0.9.4)
Requirement already satisfied: python-dateutil<3.0.0,>=2.1; python_version >= "2.7" in ./v3nv/lib/python3.6/site-packages (from botocore>=1.12.13->hca) (2.8.0)
Requirement already satisfied: urllib3<1.25,>=1.20; python_version >= "3.4" in ./v3nv/lib/python3.6/site-packages (from botocore>=1.12.13->hca) (1.24.1)
Requirement already satisfied: asn1crypto>=0.21.0 in ./v3nv/lib/python3.6/site-packages (from cryptography==2.3.1->hca) (0.24.0)
Requirement already satisfied: idna>=2.1 in ./v3nv/lib/python3.6/site-packages (from cryptography==2.3.1->hca) (2.8)
Requirement already satisfied: cffi!=1.11.3,>=1.7 in ./v3nv/lib/python3.6/site-packages (from cryptography==2.3.1->hca) (1.12.3)
Requirement already satisfied: argparse in ./v3nv/lib/python3.6/site-packages (from puremagic<1.5->hca) (1.4.0)
Requirement already satisfied: requests-oauthlib>=0.7.0 in ./v3nv/lib/python3.6/site-packages (from google-auth-oauthlib<2,>=0.1->hca) (1.2.0)
Requirement already satisfied: future in ./v3nv/lib/python3.6/site-packages (from commonmark<1,>=0.9.0->hca) (0.17.1)
Requirement already satisfied: cachetools>=2.0.0 in ./v3nv/lib/python3.6/site-packages (from google-auth<2,>=1.0.2->hca) (3.1.0)
Requirement already satisfied: pyasn1-modules>=0.2.1 in ./v3nv/lib/python3.6/site-packages (from google-auth<2,>=1.0.2->hca) (0.2.5)
Requirement already satisfied: MarkupSafe>=0.23 in ./v3nv/lib/python3.6/site-packages (from Jinja2<3,>=2.9->hca) (1.1.1)
Requirement already satisfied: pyasn1>=0.1.3 in ./v3nv/lib/python3.6/site-packages (from rsa<=3.5.0,>=3.1.2->hca) (0.4.5)
Requirement already satisfied: certifi>=2017.4.17 in ./v3nv/lib/python3.6/site-packages (from requests<3,>=2.20.0->hca) (2019.3.9)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in ./v3nv/lib/python3.6/site-packages (from requests<3,>=2.20.0->hca) (3.0.4)
Requirement already satisfied: crc32c in ./v3nv/lib/python3.6/site-packages (from dcplib<3,>=2.0.2->hca) (1.7)
Requirement already satisfied: pycparser in ./v3nv/lib/python3.6/site-packages (from cffi!=1.11.3,>=1.7->cryptography==2.3.1->hca) (2.19)
Requirement already satisfied: oauthlib>=3.0.0 in ./v3nv/lib/python3.6/site-packages (from requests-oauthlib>=0.7.0->google-auth-oauthlib<2,>=0.1->hca) (3.0.1)
You are using pip version 18.1, however version 19.1.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
(v3nv) quokka@qcore:~/data-store$ hca dss get-collections
{
  "collections": [
    {
      "uuid": "e9212ff7-86b7-4479-9111-c86521c4edf3",
      "version": "2019-05-07T163531.828088Z"
    },
    {
      "uuid": "b26f589a-2fe7-4161-8eb0-f8ca0282aa95",
      "version": "2019-05-07T162809.847327Z"
    }
  ]
}

What version and stage are you running this on?

I can reproduce this on prod with hca 6.1.0 (CPython 3.6.7 Linux-4.15.0-50-generic-x86_64-with-Ubuntu-18.04-bionic). Switching to dev resolved this issue for me.


Django 4.0.0 compatibility

Change deprecated and removed methods in Django 4.0

url => re_path
smart_text => smart_str
is_safe_url => url_has_allowed_host_and_scheme
force_text => force_str
ungettext => ngettext
urlquote => quote (urllib.parse)

Fix Signals init
Add pytz to requirements. Should migrate from pytz to zoneinfo at some point
Still a WIP. There are few tests that fail with 4.0 but I want to see what happens with the CI and if there are other things that might require attention.
Also gets the filebrowser_safe from my branch that is already 4.0 compatible TODO: this must be switched back as soon as upstream has merged my patch. Also should probably get the package from pypi not from the master branch in git.
Fixes: #2008 and #2007

https://github.com/stephenmcd/mezzanine/blob/a2fa5c8d96ac6e0a724d214135c5581cae1c8b52/.github/workflows/main.yml#L91 should probably be removed from the on push / pr  and another "deploy" workflow be implemented for just the master branch?

Great work @tswfi!


Allow Menus to showcase a KeyCode without a ModifierKey

Current implementation requires Menus include ModifierKeys, otherwise it throws an error. This should be a user decision (Main cases are Delete / Backspace / any of the F1-F12).

Looks all good, thanks for this.


feat(ingest/elastic): reduce number of calls made

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Merged as failure is related to download failure and ingestion tests passed.


fix(ndm-exporters): changed exporter service common label

Signed-off-by: Abhishek Agarwal<EMAIL_ADDRESS>Why is this PR required? What issue does it fix?:
https://github.com/openebs/monitoring/issues/79
What this PR does?:
This PR removes:

Removes the name and component label from the ndm operator's exporter service yamls.
Add a new label(common for both the cluster as well as node exporters) in the ndm exporter services that will be used for monitoring. Label to added be added is- app: openebs-ndm-exporter in both the operator as well as helm deployment.

Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[x] Fixes https://github.com/openebs/monitoring/issues/79
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #647 (c4b10dc) into develop (8472a70) will increase coverage by 0.07%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##           develop     #647      +/-   ##
===========================================
+ Coverage    46.62%   46.70%   +0.07%
===========================================
  Files           78       78
  Lines         3820     3820
===========================================
+ Hits          1781     1784       +3
- Misses        1880     1882       +2
+ Partials       159      154       -5

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/addhandler.go
75.42% <0.00%> (+1.02%)
: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 8472a70...8eefa36. Read the comment docs.

Why is this PR in hold-merge ?


Clarifying Signature Policy selection

when submitting a Development Services Access Request to Intel.

@askmish please let me know if more changes are needed.

@ruffsl looks good. Could you please just rebase the commit? We had some CI changes and this won't pass until your branch lays ontop of those changes. Will be something like...
git checkout this-branch
git fetch upstream master
git rebase -i upstream/master
git push --force your-fork this-branch```

@dcmiddle , thanks for the instructions. PR breach now rebased on to upstream master.

Hello, it looks like this commit introduced an error in the build system.
Take a look at this for sphinx issue for suggestions on how to prevent that from happening.
https://github.com/sphinx-doc/sphinx/issues/3921


Fix CI error: You cannot publish over the previously published versions: 1.0.0.

https://github.com/node-casbin/mongo-changestream-watcher/runs/8038332347?check_suite_focus=true

@Sefriol plz refer to: https://github.com/casbin/node-casbin about how to avoid updating version number in package.json manually

CI is basically a copy from mongoose-adapter. I do not think this would affect any future releases, right? I think it was just a bug for the first release where the initial package was added.
I have not modified the package.json after npm init.

@Sefriol can you make a new PR (like a typo fix) to test?

Pipelines seem to work now as they should @hsluoyz. Closing this.


The contents of this page don't align with

The contents of this page do not align with the title of this document in the left-hand navigation section. It says this page should describe the process to migrate from DTU to Vcore, but it instead describes network connectivity things. I believe this is the page the link should be pointing to: https://docs.microsoft.com/en-us/azure/azure-sql/database/migrate-dtu-to-vcore
Thanks for reviewing!

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

ID: d9d47e96-dd40-41f1-d13b-1815334792d4
Version Independent ID: 9108d2c4-6a4e-0a62-caac-1ce937dc2d13
Content: Use Azure Resource Health to monitor database health - Azure SQL Database
Content Source: articles/azure-sql/database/resource-health-to-troubleshoot-connectivity.md
Service: sql-database
Sub-service: performance
GitHub Login: @aamalvea
Microsoft Alias: aamalvea

@atsatrom We are checking this and will get back to you.

@atsatrom We thank you for your feedback. We have corrected this with a request https://github.com/MicrosoftDocs/azure-docs-pr/pull/117943 Once Approved the changes will be visible in 24 hours.


ML tutorials: reference overview of new ML.NET API

As the new API released with 0.6 version and while tutorials are not updated, would be useful to give the links to the overview of new API.
/cc: @JRAlexander

@JRAlexander thank you for review; I've addressed your feedback.

@pkulikov - Thanks for adding this. I'll merge it now.


fixtures and document.ready

A number of people seem to have encountered the same issue that it's difficult to use fixtures with javascript scripts that use document.ready or equivalent:
http://stackoverflow.com/questions/12715207/how-can-i-get-my-jasmine-tests-fixtures-to-load-before-the-javascript-considers
http://stackoverflow.com/questions/11191966/jquery-triggerclick-not-working-with-jasmine-jquery
https://github.com/modeset/teaspoon/issues/307
As an example please see my repo:
https://github.com/tansaku/ThermostatBDD/tree/document_ready_breaks_things
it all works fine (see master branch), but if the thermostat code is wrapped in a document.ready block (as in the document_ready_breaks_things branch) then several tests fail.  My hypothesis is because document.ready is activated before the fixture is loaded.  The fixture then loads the javascript with code in the document.ready function and so that javascript never gets called:
  $('#up').click(function(){
    var new_temp = parseInt($('#temperature').text()) + 1;
    $('#temperature').text(new_temp);
  });

  $('#down').click(function(){
    $('#temperature').text(parseInt($('#temperature').text()) - 1);
  });

versus
$(document).ready(function() {
  $('#up').click(function(){
    var new_temp = parseInt($('#temperature').text()) + 1;
    $('#temperature').text(new_temp);
  });

  $('#down').click(function(){
    $('#temperature').text(parseInt($('#temperature').text()) - 1);
  });
});

I've been trying to address the issue by using the jQuery.holdReady function but that doesn't seem to work:
https://github.com/tansaku/ThermostatBDD/tree/jquery_hold
One hacky fix we have is to load the jquery from a CDN.  If we do that then everything works, which I presume is because the longer loading time from the CDN holds off the  time at which the document.ready gets called ..., but I'd really love to find a more general solution.
The approach described here sounds promising:
http://stackoverflow.com/a/23061691/316729
but it would be great to get something into the jasmine-jquery code to help with this.  At the moment I can't understand why the jQuery.holdReady function doesn't work, but I'll keep thinking on it ...

I think I might have fixed this with the following:
  beforeEach(function(){
    jasmine.getFixtures().fixturesPath = '.';
    loadFixtures('index.html');
    $.holdReady(false);
  });

and calling $.holdReady(true); in the SpecRunner.html


on windows, functions of libbleu are not found

using microsoft visual compiler 14.0, fairseq 0.5.0
building libbleu dll by calling python setup.py build develop
functions in pyd are only exposed if they are declared as __declspec(dllexport).
problem is fixed if declaration of all exported C++ functions are changed, e.g.
extern "C" {
  __declspec(dllexport) void bleu_zero_init(bleu_stat* stat) {
    std::memset(stat, 0, sizeof(bleu_stat));
  }
}

Have the same issue, can you share more details about the fix? where to put this code snippet?

@myleott  this modification needs to be included in the next release as part of pip install fairseq for Windows.

I propose this should be re-opened. Requiring users to build your library from scratch is possible, but painful. I am not an expert at cross-compiling, but would it be as simple as adding the __declspec to the files in master? I don't know if that directive impacts Linux or OS X builds.


Remove thumbnails from functions catalog

After reviewing the spec, it looks like we need to this in the functions catalog:

[ ] Remove grid/list switcher so it's only list view
[ ] Remove thumbnail from list view
[ ] When removing thumbnail, set height of the rows
[ ] Still use aspect ratio at sm breakpoint

Note: these changes need made at the functions catalog level, not at the catalog item level if type function, because when there are catalogs with mixed types (e.g. collections), we'll still display placeholder thumbnails for functions so there is visual consistency.

@kingtraceyj can you add any detail? Or do we simply remove thumnails in the functions catalog?

If we remove the thumbnail images, what about the grid view? Or do we hide grid view for functions along with images?

The cards will look like the library overview list. Here are the redlines for that. I can do a functions specific redline though. We will need to add the asset tags that libraries doesn't have.

@francinelucca I'll get you an updated card with the asset and framework tags but this is the basic structure.

@francinelucca updated redlines for functions catalog! https://ibm.box.com/s/w6xjolukzbiv9jhqa8di207d7rfy96j4

@kingtraceyj I'm seeing this in your spec:

The catalog item component looks like this for all catalogs, all collections:

Why the change to move status top right and type and framework bottom left?

It would be nice for this to be consistent across all asset types.
We'll likely have other meta information that we'll want to show like usage counts and accessibility rating, which would go inline with status at bigger breakpoints. If status remains bottom left, then there's room to also show some of that information on small breakpoint if we find that valuable.

@kingtraceyj once the above questions are resolved, we can move this to "ready for development" in the design workspace.

@mattrosno yeah. i do think we should be consistent. i'm tying to remember why we went in that direction. @jeanservaas do you remember why we ended up switching the tags with status on mobile for functions?

@kingtraceyj can we keep the function catalog item content mobile positioning as-is so it is consistent for all catalog items?

@kingtraceyj we did this because we didn't have the real estate for the tags in the upper right hand corner in these small space scenarios.

@kingtraceyj since this became a design issue (per label), that means this is a design story point, so can you and @alisonjoseph make a dev issue for this and align on what needs done?

For both tiles with a thumbnail and without a thumbnail, small break points are the same. The status moves to upper right and tags move to lower left.
For a tile with a thumbnail, at medium breakpoints, the status moves to upper right and tags move to lower left.
For a tile without a thumbnail, at medium breakpoints, the status and tags stay in place. There is room for the tags to stay in the upper right.
Here are the specs for a catalog with a thumbnail:
medium:

small breakpoint:
content section same as medium but remove thumbnail

Catalog tile without a thumbnail:
medium break point:

small break point:

Created dev issue https://github.com/carbon-design-system/carbon-platform/issues/886


Add compatibility with 2024.2 eap

The first EAP of 2024.2 is available now.
Unfortunately, this requires to switch to the new version of the Gradle IntelliJ plugin: https://plugins.jetbrains.com/docs/intellij/tools-intellij-platform-gradle-plugin.html
Without this change, we're unable to support 2024.2

:tada: This issue has been resolved in version 0.70.0 :tada:
The release is available on:

v0.70.0
GitHub release

Your semantic-release bot :package::rocket:


feat: Enable message-db package to use a different PostgreSQL schema

Using the message_store schema is hardcoded: we should be able to use any other schema.

On the other end, MessageDbClient does not assume message_store hardcoded by default:
https://github.com/equinox-project/equinox-js/blob/main/packages/message-db/message-db-consumer/src/lib/MessageDbClient.ts#L29
Let's settle on one strategy.


Following the Microservices-Orchestration Example leads to error Expected to find an object with property ['orderId'] in path $ but found 'java.lang.String'

Following the exact How-To in Microservices-Orchestration using docker-compose local-kafka and camunda cloud leads to an error and causes the Connector-Sink to degrade with the error:
Expected to find an object with property ['orderId'] in path $ but found 'java.lang.String'
I unsuccesfully tried setting key.converter.schemas.enable=false and value.converter.schemas.enable=true which i believe is what has been done here

Closing this due to inactivity, feel free to reopen!

Hi,
i have same error.

Create "create instance --variables "{"orderId": 1}" order"
Manual push message {"eventType": "OrderPaid", "orderId": 1, "amount": 4000} to topic payment-confirm

And after have Degraded sink connector and follow log message -
Caused by: com.jayway.jsonpath.PathNotFoundException: Expected to find an object with property ['orderId'] in path $ but found 'java.lang.String'.


Node null if casting as a custom type using onready

Godot version:
Built from 3.2.2-stable.
OS/device including version:
Arch Linux btw
Issue description:
If I cast a node as a custom type (extending from Node) with onready var, the variable is null. But if I cast to Node it is not. However if I do this manually in _ready it actually works either way.
# assume this is present for all following examples:
onready var inventory: Inventory = $inventory

# doesn't work (prints null)
onready var candyStack: ItemStack = inventory.get_node("candy") as ItemStack
_ready():
  prints(candyStack)

# doesn't work (prints null)
onready var candyStack := $inventory/candy as ItemStack
_ready():
  prints(candyStack)

# doesn't work (prints null)
var candyStack: ItemStack
_ready():
  candyStack = inventory.get_node("candy")
  prints(candyStack)

# works (prints node)
onready var candyStack: Node  = inventory.get_node("candy")
_ready():
  prints(candyStack)

# works (prints node)
var candyStack: ItemStack
_ready():
  candyStack = $inventory/candy
  prints(candyStack)

I expected all four examples to have exactly the same outcome, the variable not being null. My first thought was that maybe technically onready variables are set after the _ready function runs, but thinking about it for two seconds is all it took to remember that's definitely not the case.

I'm closing for now. I seem to be getting non-deterministic results and want to make more sense of it before bringing it up.

Oops I figured out what was going on and it was my fault. My custom type was overriding _init and expecting arguments, which was understandably breaking casting. Sorry for the noise.


petition_separates_nonrent wasn't being asked

changed to
if trial_court_circuit == 21:
  petition_separates_nonrent

check for other uses of trial_court_county

changed in doc and defense_logic.yml
tested St. Louis County and non-St. Louis County and both work